style: move voting power in cast your vote section

Moving voting power indicator within the "Cast your vote" section, to make sidebar more compact, also adding icons for both "Cast your vote" and "Results" sections.
Before

After

Imo it would make more sense to reach consensus on the final design of the proposal page, before implementing particular changes

Imo it would make more sense to reach consensus on the final design of the proposal page, before implementing particular changes

This is just a small change from what we have now, there been already many iterations, it take too long to get consensus.

Don’t see any reason to bypass design process like this, regardless of how small the change is. Makes much more sense to create one single PR to merge all UI style changes at once after consensus with designer.


Python error when adding target IP to end of CME command

Describe the bug
Specifying the IP of a target at the end of the CME command generates an error.
To Reproduce
Steps to reproduce the behavior:

In a terminal type a command similar to: cme smb -u users.txt -p pwd.txt <IP_ADDRESS>
Run it and the error will display.
See error below:
Traceback (most recent call last): File "/home/haribo/.local/bin/cme", line 8, in <module>  sys.exit(main()) File "/home/haribo/.local/pipx/venvs/crackmapexec/lib/python3.9/site-packages/cme/crackmapexec.py", line 77, in main  if os.path.exists(passw): File "/usr/lib/python3.9/genericpath.py", line 19, in exists  os.stat(path) TypeError: stat: path should be string, bytes, os.PathLike or integer, not TextIOWrapper

Expected behavior
Normal scan results would appear
Screenshots

Crackmapexec info

OS: Kali latest updated on 8/28/2021
Version of CME: 5.1.1dev
This is the version that comes with Kali

Additional context
Note that specifying the target IP before the command line options works:  cme smb <IP_ADDRESS> -u users.txt -p pwd.txt

Hello
Try: cme smb <IP_ADDRESS> -u users.txt -p pwd.txt  :)

@mpgn hehe yes I know that works. I mentioned that same syntax in my issue in the Additional context section. What I wanted to report was how an error was thrown if you specified the syntax in the way I had mentioned: cme smb -u users.txt -p pwd.txt <IP_ADDRESS>
Definitely not a pri 0 bug but something to perhaps queue up for later. :)

Ah, yeah I understand your point, but this exact stacktrace help me to understand better the initial problem ^^


JAVA-1660: Support netty-transport-native-epoll in OSGi container

I tested Alexandre's solution on a Linux box:

update <Import-Package> as he indicated
update BundleOptions, but with only one additional bundle:mavenBundle()
    .groupId("io.netty")
    .artifactId("netty-transport-native-epoll")
    .version(nettyVersion)
    .classifier("linux-x86_64")

run the tests and check that I see the message Found Netty's native epoll transport in the classpath, using it in the logs. Add a log of the channel class and check that it's the epoll one.
revert the change in BundleOptions and check that the test falls back to NIO.

Everything worked fine, so I'll proceed with this approach.

Merged.
Like for #902, you'll need to override Netty version in your app.

Thank you!, I have run a smoke test on my environment (linux, Apache Karaf) using 3.3.2 version and it works fine.


Project closed?

Have not been getting a response back from your endpoints for some time now? Has this project stopped?

It's not closed I'm just not happy with it and have decided to take down the server. Wasn't hit for days so not worth paying for. Alternatively I can recommend OWAPI which is pretty similar https://github.com/SunDwarf/OWAPI.

Makes sense. Thanks for the info.


feat: support custom ids

Description

This pull request aims to support custom ids in a way that:

Developers can specify a custom id type when defining a collection
Users can provide their own id values instead of the ones generated automatically by MongoDB

Related discussion: https://github.com/payloadcms/payload/discussions/262

[X] I have read and understand the CONTRIBUTING.md document in this repository

Type of change

[X] New feature (non-breaking change which adds functionality)
[X] This change requires a documentation update

Checklist:

[x] I have added tests that prove my fix is effective or that my feature works
[X] Existing test suite passes locally with my changes
[X] I have made corresponding changes to the documentation

I'm initializing the PR for peer review. I'll communicate here when it's ready for a final review / merge.

Custom ids are implemented using a new id property in the collection configuration.
Specifying { id: <Type> } in a collection lets payload know that:

The id field MongoDB is going to write in database should be of type <Type> (String, Number, etc.)
id field value will be provided by user in admin or API when a record is created

Key elements of the solution:

_id field is added to MongoDB schema when id is specified in collection (see https://mongoosejs.com/docs/guide.html#_id), to prevent automated id generation
ID field is added to admin UI when a new record is being created when id is specified in collection
id field is added to GraphQL creation mutation input type when id is specified in collection
Restrictions on ObjectId type are loosen since id can now be of any MongoDB-compatible type
Relationship schema building has been updated to support custom ids

Update:

Collections doc has been updated to reference the id property
All existings tests are passing

I'm not sure about how to create additional tests though. I've noticed existing tests reference existing native collections but I can't use them.
Besides this, the PR is ready for review by Payload team.

@jeanbmar — this looks great! I just reviewed everything and have to hand it to you. Good work.
We should write some tests for this one though. What we'll probably want to do is inject a new collection config into the demo app, that does use a custom ID, with which we can write some integration tests around to make sure that we can do the following functions (at a minimum):

Allows document creation with custom ID
Allows querying documents by custom ID (see your buildQuery changes)
Allows findByID using a custom ID
Allows setting relationship fields to collections with a custom ID, with all combinations of hasMany and many relationTos

There are probably a few other test cases we should cover here but this list should get us started at least.
Our team can handle writing these tests though. We have a big week this week, with a bunch of stuff coming out, but should be able to get to this shortly thereafter unless you are feeling up for it before then. What do you think?

@jmikrut Thanks!

One edge case I could see happening is that if a user specified that their ID was a number, instead of a string, GraphQL might freak out at the moment because it might expect a string to be passed as an ID. Not sure there - haven't tested it... just thinking out loud

I was wondering the same: as long as a string is passed to GraphQL (which is the case when using admin), Mongoose will cast the value to a Number afterward.

Our team can handle writing these tests though. We have a big week this week, with a bunch of stuff coming out, but should be able to get to this shortly thereafter unless you are feeling up for it before then. What do you think?

That sounds great, thank you. I will run a fork in the meantime.

I was wondering the same: as long as a string is passed to GraphQL (which is the case when using admin), Mongoose will cast the value to a Number afterward.

I tested GraphQL for custom ids that are both String and Number. With Number I really didn't like how it expected a string so I made changes to allow that to happen. I also added all the tests.
One outlier which isn't a big deal, but should probably be fixed is that we have id coming through as a field available in the data for update mutations. That really shouldn't be there given the id is immutable. This isn't really an issue with your PR as it has been there all along.
The only thing I haven't tested here is that you could have multiple types of IDs on one relation.  For example:
const CustomID = {
  slug: 'custom-id',
  labels: {
    singular: 'CustomID',
    plural: 'CustomIDs',
  },
  id: Number,
  fields: [
    {
      name: 'name',
      type: 'text',
      required: true,
    },
  ],
};

const NormalID = {
  slug: 'normal-id',
  labels: {
    singular: 'NormalID',
    plural: 'NormalIDs',
  },
  fields: [
    {
      name: 'name',
      type: 'text',
      required: true,
    },
  ],
};

const Relations = {
  slug: 'relations',
  labels: {
    singular: 'Relation',
    plural: 'Relations',
  },
  fields: [
    {
      name: 'relations',
      type: 'relationship',
      relationTo: [ 'custom-id', 'normal-id' ],
    },
  ],
};

In the example relations is expected to have both types of IDs and I would like to see that this is possible before merging.

Another thing to note that we could improve.
If you have a Number id on a collection, the admin UI will allows text input that will error when saving and the message isn't very user friendly.

In the example relations is expected to have both types of IDs and I would like to see that this is possible before merging.

I think it's possible in the sense it won't crash, but we should maybe throw an error in this case. I can't find a valid use case where one would want to match ids of different types.

Another thing to note that we could improve.
If you have a Number id on a collection, the admin UI will allows text input that will error when saving and the message isn't very user friendly.

Indeed, I think it's the drawback of allowing any Mongoose schema type as collection id type.
We could probably tighten things on the id type by allowing only a subset of Mongoose types such as ObjectId, String and Number. We could then accordingly switch between text and number admin types for the ID field. What do you think?

We literally just came to the same conclusion in our internal meeting this morning. I was planning to pivot the changes to do exactly this.
@jeanbmar Do you want to make the changes to lock it down to only text and number or should I?
Changes needed:

Change id input to support either text (current) or number
Add validation (might be fine without if the id input once input is corrected)
schema validation in joi for collection id should only be strings of 'text' or 'number'
cleanup code where possible (GraphQL switch case for types could be a simple if, etc.)
update documentation

Am I missing anything?
Lastly, is the config name property id sufficient? It would be more semantic to rename it to idType, idFieldType or possibly something different?

I'll give it a shot!

I've pushed the discussed changes.
I see two additional things to check / consider:

Relationships id type consistency should be checked during sanitize calls. It could happen in something similar to the following way, but I wasn't sure about TS types so I didn't implement it: https://github.com/payloadcms/payload/blob/cb422bb4c6758607592127f1bc4e0d1a20378945/src/collections/config/sanitize.ts#L123-L124

We might want to change this as well, though it's purely visual:
https://github.com/payloadcms/payload/blob/cb422bb4c6758607592127f1bc4e0d1a20378945/src/admin/components/views/collections/List/formatFields.tsx#L11

Your changes look good, thanks for taking that on.
I have one more thought about our approach and how this could be better yet.
Instead of having an independent property on the collection for the idType we could check in the actual collection fields for a field with name: id and let that be the replacement for auto-generating the id.
By doing it this way some really cool possibilities are available like:

change where the id input is shown
Hide the id field and write a hook to set the id before save
Use a custom components
Add custom validation logic

We will have to validate the config to prevent localization for the id field. The id field can't be in a group. It must be required.
How do we feel about pivoting to making the id field work like all the fields? Any other potential issues?
I can do some more work on this too.

I agree, using an id field would make sense and enable additional capabilities now that the id type is either text or number instead of Mongoose types. I can spend some time on it today, let's a have a look.

@DanRibbens I've pushed some changes as discussed, but I haven't handled the validation part. Could you take care of it?

Yes, I'll review the changes and finish up anything missing when I'm free.
This will be great to get merged!

Ah, my bad. I had some trouble with git. I rebased changes from master and resolved conflicts, but then git push was being rejected even though I had all the commits in history from jeanbmar/payload:master. I thought if I pushed with -f it would be fine, I probably shouldn't have done that.
I've reopening the PR under a new branch: https://github.com/payloadcms/payload/pull/313
Lesson learned: don't let remote PRs get stale and be don't -f. Sorry for the confusion.


Add benchmark-addr2line.py script

Add a benchmarking script that compares binutils, elfutils, LLVM, and this project implementation of addr2line.
The limitation of cargo bench is that it's designed for micro-benchmarking, where a benchmark is run ~100 times. Thus I came up with a separate Bash script that utilizes hyperfine.
Sample output:
Benchmark 1: gimli-addr2line libxul.so
  Time (mean ± σ):     115.5 ms ±   2.4 ms    [User: 86.1 ms, System: 29.3 ms]
  Range (min … max):   110.9 ms … 120.7 ms    25 runs

Benchmark 2: binutils-addr2line libxul.so
  Time (mean ± σ):      1.196 s ±  0.017 s    [User: 0.962 s, System: 0.233 s]
  Range (min … max):    1.184 s …  1.208 s    2 runs

Benchmark 3: llvm-addr2line libxul.so
  Time (mean ± σ):     493.1 ms ±   7.3 ms    [User: 445.9 ms, System: 47.1 ms]
  Range (min … max):   481.2 ms … 501.2 ms    6 runs

Benchmark 4: elfutils-addr2line libxul.so
  Time (abs ≡):         3.696 s               [User: 3.676 s, System: 0.018 s]

Relative speed comparison
        1.00          gimli-addr2line libxul.so
       10.35 ±  0.26  binutils-addr2line libxul.so
        4.27 ±  0.11  llvm-addr2line libxul.so
       31.99          elfutils-addr2line libxul.so
Benchmark 1: gimli-addr2line clangd-18
  Time (mean ± σ):     391.1 ms ±   4.2 ms    [User: 252.2 ms, System: 138.8 ms]
  Range (min … max):   383.9 ms … 395.7 ms    7 runs

Benchmark 2: binutils-addr2line clangd-18
  Time (abs ≡):         3.115 s               [User: 2.456 s, System: 0.658 s]

Benchmark 3: llvm-addr2line clangd-18
  Time (mean ± σ):     398.0 ms ±   3.7 ms    [User: 323.9 ms, System: 74.0 ms]
  Range (min … max):   393.8 ms … 405.4 ms    7 runs

Benchmark 4: elfutils-addr2line clangd-18
  Time (abs ≡):         2.393 s               [User: 2.359 s, System: 0.033 s]

Relative speed comparison
        1.00          gimli-addr2line clangd-18
        7.96          binutils-addr2line clangd-18
        1.02 ±  0.01  llvm-addr2line clangd-18
        6.12          elfutils-addr2line clangd-18
...

All right. After a little investigation, I utilized hyperfine which ends up with a minimal shell script. What do you think about it?
When it comes to the Docker script: do you want a script that will build a Docker image (docker build) and run it then? Should it be the same script the GitHub CI should run?

The hyperfine output isn't as pretty as the table you had, but I still think it's worth using it to get more reliable results.
Yes, I want a single command to run the benchmark on my non-openSUSE system, and it should avoid running the zypper commands every time. I'm not particularly familiar with using docker in github actions, but the current action spent a minute installing the packages, so seems like it might be a good idea to build an docker image for it too?

The hyperfine output isn't as pretty as the table you had, but I still think it's worth using it to get more reliable results.

Yeah, I think so. Moreover, one can use --export-markdown option to export the results and include them in README.md file.

Yes, I want a single command to run the benchmark on my non-openSUSE system, and it should avoid running the zypper commands every time.

Sure. If we run docker build && docker run in the script, then the container building will be cached and you won't have to install all the packages every time.

I'm not particularly familiar with using docker in github actions, but the current action spent a minute installing the packages, so seems like it might be a good idea to build an docker image for it too?

Me neither and I haven't found any easy solution for that. So do you want build a docker container that will be manually pushed to dockerhub and then used in the CI. What do you think?

Me neither and I haven't found any easy solution for that. So do you want build a docker container that will be manually pushed to dockerhub and then used in the CI. What do you think?

What you've got is fine. I don't think we want to push to dockerhub, it may be possible to change the caching in the action to keep the layers. I don't know. Only look into it more if you're keen.

All right, so let's leave the caching for now. I've just updated the PR so that the requested wrapper script is introduced (and used by GitHub CI). Can you please review it?

Also this has already been useful in highlighting the need for #318

One last nit: can we move these into a subdirectory? e.g. scripts? And maybe a scripts/docker for the dockerfile, since it uses the subdirectory as context for the build, so best to restrict it to what is needed.

Sure, it should be done right now.


Rename "Mavlink" stuffs to that of "MAVLink"

These classes use lower case for MavLink:
class MavlinkCommands {...};
class MavlinkParameters {...};
class MavlinkReceiver {...};
class MavlinkChannels {...};

In QGC source and elsewhere the spelling is MAVLink. Should I change these to use the correct case (e.g. MavlinkCommands to MAVLinkCommands, etc.)
Please suggest: @hamishwillee @julianoes @JonasVautherin .
Thanks.

If MAVLink is always implied then it isn't needed. If something is intended to be generic then it shouldn't be prefixed. You'd use a MAVLink where something must always be for working with MAVLink and only makes sense in that context.

OK. Thanks for rephrasing my question @hamishwillee . :)

Addressed as part of #310.

To me mavlink is sort of like a word which is why I wrote it that way. We can change it but I don't know if the churn is really warranted.

Probably not, but already done :-)


Remove unused BuildConfig class

There was BuildConfig class which had to be passed to the test project REST call but was not used anywhere. Since the REST call failed when this parameter was not provided, I decided to remove it after the discussion with @mrietveld.

I will update the documentation (together with some other fixes) when @mrietveld finishes the migration to AsciiDoc.

+1 if we somehow end up needing this functionality, I'd rather start from scratch than keep in code that's not needed anymore.
@manstis Would you do the honors? (Merge)


linestream_from_channel fails to call fin () after exception

https://github.com/ocaml-community/yojson/blob/b743857a28f02af68bcfadb515df0b21a9c6eb06/lib/read.mll#L1183
linestream_from_channel fails to call fin () after an exception, this may lead to a file descriptor leak. In particular linestream_from_file assumes fin is always called as exceptions are not handled

This makes sense, the issue can be closed


docs(code): update comments of dispatch property of EffectOptions

PR Checklist
Please check if your PR fulfills the following requirements:

[ ] The commit message follows our guidelines: https://github.com/datorama/akita/blob/master/CONTRIBUTING.md#commit
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[x] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: #612
What is the new behavior?
Updating comments of dispatch property of EffectOptions
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

Running tests (yarn test:all) and building libs (``yarn build:libs) is successful, building apps (yarn build:apps`) fails wth:
ERROR: libs/akita/src/lib/store.ts:48:14 - error NG2007: Class is using Angular features but is not decorated. Please add an explicit Angular decorator.

48 export class Store<S = any> {
                ~~~~~

libs/akita/src/lib/store.ts:48:14 - error NG2007: Class is using Angular features but is not decorated. Please add an explicit Angular decorator.

48 export class Store<S = any> {
                ~~~~~

I don't think this error is relevant to my change in this pull request.


Implement Extent Reports for Cucumber

If beforestep/afterstep hooks do not get fixed any time soon in maven-cucumber-reporting.
See here:
http://www.vimalselvam.com/cucumber-extent-reporter/
https://github.com/email2vimalraj/CucumberExtentReporter
http://extentreports.com/community/
<EMAIL_ADDRESS>http://toolsqa.com/selenium-cucumber-framework/cucumber-extent-report/

Dependency fixes required:
Issue #59
Issue #75


187 support mobile screen size

Please also update this branch to the latest. Looks like you did merge already, so you can't rebase this anymore...

Don't think you read this: https://github.com/penumbra-zone/web/pull/238#pullrequestreview-1734447421. It's still an issue.

oh, I fixed it but ...)

@grod220
is it ok?

@grod220 is it ok?

Yes, this works

oh, I fixed it but ...)

What do you mean? We should not have ... and show the whole testnet name.

I don't understand where the changes are missing)

but for mobile we can cut networks name?

but for mobile we can cut networks name?

I'm saying we should never cut the network name length for any window size


Misuses of cryptographic APIs

Hi
The following lines have cryptographic API misuses.  File name => jupiter-common/src/main/java/org/jupiter/common/util/MD5Util.java: Line number => 39: API name => MessageDigest:Operation on object of type java.security.MessageDigest object not completed. Expected call to digest, update File name => jupiter-common/src/main/java/org/jupiter/common/util/MD5Util.java: Line number => 54: API name => MessageDigest: File name => jupiter-common/src/main/java/org/jupiter/common/util/MD5Util.java: Line number => 50: API name => MessageDigest:First parameter (with value "MD5") should be any of {SHA-256, SHA-384, SHA-512}

Can you please send a pr for this？

MD5 is supported.
See references :
https://docs.oracle.com/javase/8/docs/api/java/security/MessageDigest.html#getInstance-java.lang.String-
https://docs.oracle.com/javase/8/docs/technotes/guides/security/StandardNames.html#MessageDigest


Support Production Workloads

I want Persul to support production workloads. This issue defines what that actually means in terms of numbers.
Numbers
PURL creation
Parameters
A rough estimation which I have repeatedly used, adjusted for nice fractions:

1.000 distinct users creating purls
1.000 purls created / per day * per user (= 40 / hour ~ 0.6 / minute)
86.400 purls created / day * user

breaks down into 3.600 / hour = 60 / minute = 1 / second

purls have a minimum lifespan of 10 years

Deductions
Given these numbers ...

86.400.000 purls would be created every year
864.000.000 purls would have been created after 10 years

Side note on data types:

32-bit integer IDs will be able to address ~4 billion PURLs.
With 64 bit that would go up to 18.446.744.073.709.551.615
UUIDs are 128 bit have an adress space large enough that I won't bother writing down the number. Don't quote me on that 😉

PURL resolution
In my use case I expect that PURLs will be resolved infrequently, potentially never.
Due to that expectation I don't yet have detailed numbers expect these:

Only 10% of all PURLs are ever resovled. The remaining 90% are created but never resolved.
Users expect a PURL to resolve within 1 second on average. This does not take into account the time the actual resolved URL is loaded by browsers since that is not under Persurls control.

Related

#28
#7

I ran the application (based on #42) on Railway while trying to set up a demo deployment (see #40). I ran the load tests which gave me these numbers (1 minute for each parameter set):

Concurrent PURL creators
 Average request time

1-25
 10-20ms

50
 50ms

100
 700ms

200
 800ms

250
 1.2s

500
 2.6s

This shows that there is definitely a bottleneck at least due to the addition of a RWMutex in #38.
Right now these numbers are okay with me and the show a potential for optimization. I am sure that the request times will go down once SQLite is replaced with a dedicated database #7.

I re-ran load tests on Railway using Postgres and a connection pool of max 10 connections. Results:

Concurrent PURL creators
 Average request time

1-25
 3ms - sometimes with a couple of 20ms

50
 3ms - more with a couple of 20ms

100
 10-20ms - still with a lot of 3-5ms requests

250
 10-30ms

500
 10-50ms

1000
 wide range from 10ms to 400ms

Service metrics:

Postgres metrics:

Summary

Generally response times have dramatically improved with the addition of postgres in #50.
1000 agents tried to create a PURL every 50ms. In the current test these agents created a total of 190.382 purls in 1 minute.

As far as request/response time of PURL creation goes, I consider Persul to be production ready.

[ ] I have not yet done any tests on resolve times which I may add later
[ ] Currently, the id columns of the purls table is an 32bit integer. A future version of persul will have to go for a 64bit integer. To avoid downtime due to reaching this limit, the system should implement a health check based on the remaining available row count in the purls table.


Ability to upload from memory

In my cases, I store images in db and receive it as []byte. In this PR I added ability to upload []byte image without usage filesystem.

BTW , Thank you for your commit , Please change your function and commit again :handshake:

@ahmdrz, I'm changed bytes to io.Reader. In the previous implementation we had to open the file twice: to write in the multipart form and to obtain the dimensions. I solved this problem using tee.Reader.

Thank you.


Paymentrequest stickersheet

Created a stickersheet that designers and developer can use in Sketch

Hey @mustafa-x
Here are few nits:

Add paymentrequest/stickersheet/index.html to paymentrequest/index.html as well.
s/tak/take in paymentrequest/stickersheet/index.html
Replace paymentrequest/stickersheet/index.html header with

---
feature_name: PaymentRequest Stickersheet
chrome_version: 53
feature_id:<PHONE_NUMBER>217792
---

hey @beaufortfrancois added changes just now :)

Thanks! Let me know when you're ready for review and want me to merge.

Feel free to review. I am waiting on another PR from Web Fundamentals as this links to that. Once that has been merged then we are good to go.

Here it is live!
https://googlechrome.github.io/samples/paymentrequest/
https://googlechrome.github.io/samples/paymentrequest/stickersheet/


default configuration and systemd results in duplicate log messages

nsd, per default, logs both to stderr and syslog. As systemd forwards
both into journal all messages are included twice.
Ideally one could make nsd just use syslog and nothing else but
this does not seem to be possible.
This bug has been filed in Debian as
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=803683
Also originally reported in https://www.nlnetlabs.nl/bugs-script/show_bug.cgi?id=4198

Wouter, this is amazingly fast ;-)

Thanks for the report, I added the option log-only-syslog, that you can enable.  That should solve the duplicate log item issue.  I do not want to change the default log outputs, because other users may depend on it.  So this new item allows to choose between log both syslog and stderr, log to only the logfile (with the logfile option) or log only to syslog with the added log-only-syslog: yes option.
Before syslog takes effect, in the startup sequence, it uses stderr still.  This is only when errors cause failure, mostly.  But because syslog is not opened yet, that would only print to stderr and thus also no duplicates.


Update twilio dependencies

Summary
By supporting Flex Plugins CLI 7.x, customers can use the new plugin validator against the template plugin.

Update Twilio CLI to latest release
Update Flex Plugins CLI to latest release
Update GitHub Actions workflow to not halt on validator warnings

Checklist

[x] Tested changes end to end
[x] Requested one or more reviewers

tested the branch with twilio cli 7.0.0 locally and all good.
LGTM


meta-rules support added

Sometimes, you end up with a lot of similar rules, this would be better supported with "meta-rules".
Simple example, defining two rules, one for "some_person", one for "another_person" in just one file.
name: "This rule detects %expanded_value% in your Data"
...
expanded_list_value: ["some_person", "another_person"]

More complex example, defining two rules, one for host1 and 50, one for host2 and 100, in just one file ;-)
name: "More than %expanded_value[1]% errors in %expanded_value[0]%"
type: frequency
num_events: "%expanded_value[1]%"
...
expanded_list_value: [["host1", 50], ["host2", 100]]

Can you resolve conflicts please :)

Can you resolve conflicts please :)


Is there a way to use react-cosmos with styled-components ?

Hi !
I'm trying to use react-cosmos with my components but I'm using styled-components and obvisouly an error occur :
Cannot read property 'xxx' of undefined.
My "components look like this :

I don't know how to export my theme, Can I have a "master" components taking my fixtures as children ? And import my theme in the master component ?
"Like a ThemeProvider"

Ok...
Nevermind ! Decorator on the way, my bad.
I put a decorator in src.

And it works.


Rename deploy and deployAll tasks to highlight its purpose

When using the plugin as a part of a continuous delivery pipeline, the name of the tasks can be confusing.
For example the task deployAll could be interpretet as deploying a single unit to a Maven Repository, a docker registry and a runtime enivroment all at once.
Suggestions for new names:
==== deploy
deployToCloud
cloudDeployment

==== deployAll
deployAllToCloud
batchCloudDeployment

I have to agree on this one. However, the common Gradle convention is to begin the tasks with the plugin name or purpose. e.g. cloudDeployAll
Also, currently you have to pass a --environmentId if working with only one. Again, the usual Gradle convention is to have dynamic tasks names.
deployer {
    marathon {
        id = "marathon-zwitscher"
    }
}

You now should have a cloudDeployMarathonZwitscher and cloudDeleteMarathonZwitscher task. The *All tasks are simply a shortcut for all if the kind.

@toplac, @lreimer: will rename the tasks as @lreimer suggested and will add the possibility to have dynamic task names.

@sjahreis Any update?


Update Flux Comparison example with Redux 1.0RC

Here it is: https://github.com/voronianski/flux-comparison/blob/master/redux

I'll take a look at this.

Thanks!

@gaearon should I only use Babel stage 2 on here too?

@rwillrich Yes, please!

Ok

I've submitted a PR to the project -> https://github.com/voronianski/flux-comparison/pull/53/files

Thank you!


CloudSdkAppEngineFlexibleStaging.stageFlexible() fails if app.yaml is malformed

Parsing malformed app.yaml here using SnakeYAML will throw ScannerException and ParserException that are RuntimeException, as noted here.
I think the staging logic should just log a warning and continue staging rather than exploding, very much like it currently does with IOException on app.yaml.

It may be rather right to fail staging. If so, we should throw AppEngineException as we currently do, e.g.,  when app.yaml or Dockerfile does not exist.

I have to admit it makes me a bit nervous to duplicate the gcloud
validation in the core library. I don't want to break people when we
shouldn't.
On Tue, May 9, 2017 at 6:01 PM, Chanseok Oh<EMAIL_ADDRESS>wrote:

It may be rather right to fail staging. If so, we should throw
AppEngineException as we currently do, e.g., when app.yaml or Dockerfile
does not exist.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/appengine-plugins-core/issues/406#issuecomment-300314672,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHf5HbX83NBbpnGAOgVmhOaxcesHvx5uks5r4OJLgaJpZM4NVqQz
.

Failing when that app.yaml is inaccessible due to IO errors or when it is malformed is not semantic validation, so IMO it is okay. It should and will fail anyway at a certain point of the deploy pipiline.


Force old FastTransforms to use FFTW <v1.6

Older versions of FastTransforms are incompatible with FFTW v1.6, so the only way to prevent these from being fetched seems to be to impose a hard compatibility bound on these and limit these to FFTW v1.5 and below.
cc @MikaelSlevinsky

Can you please elaborate for the record why there's an incompatibility? It'd sound like FFTW.jl had a breaking change in a minor version

FFTW used to store values of arguments as type-parameters. This was type-unstable without constant-propagation. FFTW v1.6 changed this to improve type-stability. The type parameters weren't a part of the API, so this isn't a breaking change. However, FastTransforms depends intricately on the type parameters as declared by FFTW, so this change in FFTW led to older versions of FastTransforms becoming incompatible with FFTW v1.6+.


NOMERGE: android hotfix hotfix ; push notifications were busted

@keybase/react-hackers

deployed as 210


[Docs]: Button Widget

Is there an existing issue for this?

[X] I have searched the existing issues

Documentation Link
https://docs.appsmith.com/reference/widgets/button
Discord/slack/intercom Link
No response
Describe the problem
Restructuring
Describe the improvement
Rehaul

Duplicate of #20019

@spark-c I see that you have opened a duplicate ticket on the other repo https://github.com/appsmithorg/appsmith/issues/20019 . Can you please close that and raise a PR for this one please?


4.1.1 Macos "display wireframe" does not work.

Godot version
4.1.1 Macos
System information
M1 Macbook Air big sur
Issue description
Wireframe mode does not work in 4.1.1 WHEN COMPATIBILITY RENDERER IS ENABLED

this is godot 3.5 ^

this is godot 4.1.1 ^
Steps to reproduce
have a 3d mesh
set perspective to display wireframe
enable the compatibility renderer
be on mac (?)
Minimal reproduction project
N/A

Compatibility mode is still under development, cc @clayjohn

Related to https://github.com/godotengine/godot/issues/55900.

None of the debug draw modes are implemented in OpenGL currently.


boards/stm32: add initial support for b-g431b-esc1 board

Summary
boards/stm32: add initial support for b-g431b-esc1 board
Impact
Testing

Very nice @raiden00pl ! This is a good board to brushless motor!


Heads-up: Copy edits

Docs Build status updates of commit d6fc868:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/install-the-tools.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Predict won't work

docker run with the latest worker image is never exits... it creates _predicted file and just hangs after that

to reproduce do:
a2ml new some-project -s iris.csv -t species
cd some-project
a2ml import
a2ml train
a2ml evaluate
wait till search is done, a2ml evaluate will print leaderboard
select on of the model id-s from leaderboard and do
a2ml deploy  --locally
a2ml predict ../iris.csv -m

@vlad-dplrn, please close the issue if it's gone


[Tabs] Migrate to hooks

Related to #15231.

[ ] Migrate the tests to testing-library.

I would propose that we prioritize removing the shallow tests over migrating the existing tests to testing-library first.

Nobody proposed migrating the existing ones. I merely introduced another API in case tests need to be refactored for other reasons (strict mode, shallow removal etc).

take ).to.not.be.ok

Agree :100: jest already identified this problem and reduced chainability to not only. Everything else is a single word improving grepability.
I wouldn't mind using jest-expect over chai's expect. I didn't even think about using it since I thought it was coupled with the test runner.

Nobody proposed migrating the existing ones. I merely introduced another API in case tests need to be refactored for other reasons (strict mode, shallow removal etc).

I believe that we (as a team) have introduced two ways of doing the same things (expect vs assert) and (enzyme vs testing-library). We should aim for simplicity, it's not OK to have two identical tools for the same job in the long run. However, we should also recognize that it's an necessary short term intermediate unindeal state for improving the problem Material-UI wants to solve in the long term.
So I thinks that we should progressively remove the assert and enzyme usage from the codebase. Does it make sense?

I wouldn't mind using jest-expect over chai's expect. I didn't even think about using it since I thought it was coupled with the test runner.

I would have a preference for jest expect as it might be more useful for people using our tests as inspiration. But, it's not very important from my perspective.

I will rebase the pull request to use the latest useEventCallback util and improve how the tests are written.


Log Error When ESM Import Fails

I had a pretty tough time debugging an issue that resulted from some bad code that was generated during a build step and which got executed at "import-time" rather than when rendering. This was difficult to figure out because ipywidgets simply reports something like failed to render model id <the-id> without logging the original error. This could be mitigated if anywidget reported errors that arose when importing the ESM module.

Oof, ya we definitely could improve this. Thanks for the PR. I hadn't considered the unhandled exception for a failed import. Generally there is an issue with obfuscation of the source code because we need to create a blob URL. Still trying to think of ways to improve the "debuggability" of this.

Thanks for getting that over the finish line!


remove unused variable from scope according final built instructions

Some variables are eliminated by optimized passes(such as OpFusion), we can filter out them according to arguments of the built instructions, and erase them from the scope to avoid unnecessary buffer allocation

I wonder why there are unused variables in scope, shouldn't all are used?

I wonder why there are unused variables in scope, shouldn't all are used?

BuildScope create all variables of the initial graph not the optimized graph where some variables maybe eliminated by OpFusion pass currently


Sync call is blocking indefinitely

What version of gRPC and what language are you using?
v1.37.1, C++
What operating system (Linux, Windows,...) and version?
Ubuntu 20.04
What runtime / compiler are you using (e.g. python version or version of gcc)
GNU 9.4.0
What did you do?
Using gRPC api's for synchronous call
cloned the code =>
git clone --recurse-submodules -b v1.37.1 --depth 1 --shallow-submodules https://github.com/grpc/grpc
Installed in /usr/local
What did you expect to see?
A quick handshake bw greeter_client and greeter_server Inside helloworld.
What did you see instead?
The client call is blocking, and the response from the server is getting delayed
Everything work's fine, when i repeat the steps with gRPC package v1.59.0 or higher
See TROUBLESHOOTING.md for how to diagnose problems better.
Anything else we should know about your project / environment?

Could u please help me out ?

v1.37 is 3 years old. Please upgrade to a newer version, you are missing years of fixes, features, and improvements.


Use multistage build to reduce the size of the Docker container

Uses multistage builds to improve the size of the final image.
Single "stage", (with .dockerignore): 259MB
Multistage: 89.2MB
Also documented, as coming back to why the Dockerfile is the way it is in the future will be (hopefully) less confusing.
I've also included a .dockerignore with minimal inclusions, might also be worth including .venv.
Feel free to change anything as you see fit.

Excellent. Thanks a lot for the detailed explanations in the comments.


expense tracker

Is your feature request related to a problem? Please describe.
No its not a problem. I want to add an expense tracker feature. I am a GSSOC participant.
Describe the solution you'd like
I will work using Javascript and html
Additional context
Add any other context about the feature request here.

@ridsuteri please assign it to me

@ridsuteri please assign this to me

@Amit366 this project is already present in the repository :(

@ridsuteri there are 178 issues opened and there are some participants who have 6 to 7 issues but aren't able to send a PR. Now from where can I work on a new project. Can you suggest any idea on which I can work.

see @Amit366 participants are coming up with new issues daily !
that clearly means there are still lots of project to be added in the repo .. also about participants having 6-6 issues , we will be asking them about their progress after certain days  and if they are unable to get the projects done in certain time ..those project will be assigned to anyone else ...


Change logo to have transparent background

On any background that isn't white it's quite jarring to have a white
cutout around the logo. Making the logo background transparent solves
this issue, although the logo might now be less distinct if the page
it's displayed on matches the logo colours.
Before:

After:

Hi @WalterSmuts thanks for the contribution. Yeah I actually also have an svg without background of the logo, however I was unsure if it was better that way on dark backgrounds.
A colleague of mine designed the logo and could probably make it work better, but unfortunately he is not available right now.
You are probably right, though and we should use the version without background for now. Thanks again for the contribution :)


docs: Fix typo in code example and update output

Hello! I found a typo in the code example documentation and would like to contribute a fix.
Changes made:

Fixed the typo in [docs/sem_agg.rst]
Updated the example output to match the current version

The correction helps maintain accurate documentation and ensures the example reflects the latest behavior.
Thank you for maintaining this project!

LGTM! Once the tests pass, you can go ahead and merge


missing AV1 support!

hello guys, i would like to check with you if possible to add AV1 support to ion-SFU.
i have post similar request in Pion here - https://github.com/pion/webrtc/issues/1670.
explaining that i'm using AV1 on chrome for almost a year...now, and missing it badely with latest ion-sfu which force me to downgrade into less efficient VP8 codec usage.
kindly advise if this might be interested for community to use amazing ion-SFU with modern and latest codec such AV1 which is both stable and much higher performancer than older VP8/9/264...
Best!

Hello @dannymn, support for AV1 should be straight forward, if you can contribute to detect the keyframes we can include it, we have not yet invested on it because its lack of support on other browsers.

hi @OrlandoCo  hope all well.
perhaps it is good time to raise AV1 support to ion-sfu. AV1 codec used widely by chrome both PC & Mobile devices.
also similar cool solution like livekit + Galene + plugNmeet supporting AV1 for quite some time already, i would be very happy and thankfull if ion-sfu may also benefit usage of much more modern codec like AV1 which consume less BW w/ greater quality, hence give better result over time. i belive other ion-sfu users like me will also appriciate anyone who can contribute here. many thanks!


Implementation of length penalty using logits processors

I realized that I  forgot to sign off all the little commits to fix the formatting.... So I squashed all the commits in this new PR. This includes the fixes proposed in the review comments.

That's strange, seem like the logits tensor has a different shape in Mixtral. I didn't see anything like that with Mistral-7b, I'll have to take a look. But it happens only for the requests with length_penalty, right?

@maxdebayser actually it turned out to be a preexisting bug in the typical p logits processor. But there was also a bug where we weren't checking for presence of the param properly so the length pen logits checker was always getting added. Fixes here: 2961da7ff4213403ba68acddab48f3efb44a2f90.

That explain's why I wasn't able to reproduce it :D. I tried with a quantized Mixtral 7Bx2 on my machine and was about to launch the full Mixtral 7Bx8 to test.


Get RGA working with Vulkan input when the output files are set to be in active directory.

Hi pzurita,
Please describe the issue that you are trying to solve, and provide instructions for reproducing it.

The issue is that I would get the error "Error: output directory does not exist: " if the output for the ISA, livereg, cfg, il, and/or statistics were set to be in the active directory rather than somewhere else. All of this in Windows 10, I don't use Linux.
These are the arguments that I passed in and which work with my pull request:
-s vulkan --asic Ellesmere --isa test.isa --livereg test.reg --cfg test.flow --il test.il --vert test.vert --frag test.frag

Done.


Fixed: ES2015 compatible Sample Code

Hi.
Sample code of 'Hello world' page is written in ES2015. So I fixed it.

var is es2015


Document the OS LTR plugin

Description
The OpenSearch LTR plugin doesn't have any information in the documentation website.
See this spreadsheet to track the migration effort: https://docs.google.com/spreadsheets/d/1b81JlMCurjhPi-kaGW8eBkiV7s3x4NrtHpyPO3mlghg/edit?gid=0#gid=0
Issues Resolved
Version
2.17?  We need to update the LTR plugin before this get's merged, it's very out of date.
See https://github.com/opensearch-project/opensearch-learning-to-rank-base/issues/38 for the issue tracking the work to get a updated plugin released.
Checklist

[x ] By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

We will be tracking the work in this spreadsheet https://docs.google.com/spreadsheets/d/1b81JlMCurjhPi-kaGW8eBkiV7s3x4NrtHpyPO3mlghg/edit?gid=0#gid=0

Closing in favour of https://github.com/opensearch-project/documentation-website/pull/8336


Credentials - Unable to add coinbase_advanced_trade

Describe the bug
When selecting random connectors for testing PMM controller, found coinbase_advanced_trade returning No module named 'coinbase' when adding API keys

Steps to reproduce bug

Setup deploy using setup.py
On Credentials, select coinbase_advanced_trade as connector and add API keys

Confirmed. I have the same issue.


Click listener on marker

Hello,
How do you add a click listener on a marker ? I haven't found anything about it.
Thanks in advance.

For now ,
you can get geopoint of an clicked  marker  by calling the attribute onGeoPointClicked  in OSMFlutter
and in  future releases, I will add another alternatives

ok many thanks !


Add the ability to set the viewport's minSize

https://github.com/Foliotek/Croppie/blob/680db73a21c43fbaef26bd3d4fbb2dbdb63c2341/croppie.js#L468
I would like to be able to set the viewport's minSize as i can't set images smaller than 50x50 pixels.

Up!

I'm open to PR's for this.

when you got the image data, you can create a canvas and put the image into canvas, then you can got any size what you want.

@xty1992a
I want to do it with the library, not manipulating the image AFTER i've cropped it.
If I needed to insert into a canvas and crop a a image every time I wanted an image smaller than 50x50, what is the point of using the lib?
Check the PR I made


Problem accessing dialogflow intent with Botkit v4.0.1

Hello,
Awesome plugin. Worked great with Botkit 0.7.x. However, I am having trouble integrating with Botkit 4.x.x. I can't seem to access message.intent when using the controller.hears function. See below:
controller.on('message', async(bot, message) => {
    console.log(message) // message contains message.intent
});

controller.hears('hello', 'message', async(bot, message) => {
    console.log(message) // message does not contain message.intent
});

I am not sure what I could be messing. Hopefully you can help me out?
node: 8.11.2
botkit-middleware-dialogflow: 2.1.0
botkit: 4.0.1

I realized that if I change this:
controller.middleware.receive.use(dialogflowMiddleware.receive)
to:
controller.middleware.ingest.use(dialogflowMiddleware.receive)
Then my message.intent property is available. Are there any drawbacks of using ingest as opposed to receive?

Now there some issues with middleware pipeline in botkit, but using ingest is a good workaround.


html2canvas does not work correctly with rotated view

Describe the bug
Using html2canvas as an alternative to dom-to-image-more does not work with a rotated view.
To Reproduce
Open https://openlayers.org/en/v6.5.0/examples/print-to-scale.html and https://openlayers.org/en/latest/examples/print-to-scale.html rotate the views and export.  Compare the results.
Expected behavior
An export including the specified controls, and the rotated canvas correctly positioned.

Looks like this is a html2canvas bug, and it is browser specific. In Safari, I get the expected result. In Chrome, I do have all controls on the page, but the rotated map is shifted.

It will work as expected in Chrome using WebGL for the tile layer (with preserveDrawingBuffer set) although currently a layer cannot be added to the map before the source is set.


JK BMS BLE Protocol

Hello, it's not about [esphome-jk-bms], but it's about reading daa drom JK BMS by my app...
Now i have this:

I send to BMS 2 command:
0xffe1:
aa5590eb9700df5288679d0a096b9af6709a17fd
aa5590eb9600f45fe2036dfbe038aabc441234b8
After that i start to recieve data each second, but I was able to decrypt some data. The main thing I need to find is the charge current and the discharge current. Help me please...
First pair: Data from Bluetoothe and after 10 seconds screenshot of app:

Second pair: Data from Bluetoothe and after 10 seconds screenshot of app:

My English not so good, but I hope you will understood me :)

Oh, I think I found charge current and charge power?

But where is discharge?

Ok, solved... it can be negative... Thank for everyone (no one) :)

You are on the right track! Keep in mind there are different time zones and everybody needs some rest. :-)

https://github.com/syssi/esphome-jk-bms/blob/main/components/jk_bms_ble/jk_bms_ble.cpp#L564-L578


Update sbt-mdoc to 2.2.14

Updates org.scalameta:sbt-mdoc from 2.2.9 to 2.2.14.
GitHub Release Notes - Version Diff
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 = "org.scalameta", artifactId = "sbt-mdoc" } ]

labels: sbt-plugin-update, semver-patch

Superseded by #146.

Superseded by #146.


⚠ SkipTheCommericals has degraded performance

In f578b77, SkipTheCommericals ($STC) experienced degraded performance:

HTTP code: 200
Response time: 1624 ms

Resolved: SkipTheCommericals performance has improved in 3d72d26.


Master Page Structure

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Summary 💡
While designing the pages, they are all built independently of each other. Actually I think it should be a main page. When they become the main template, they can create sub-templates.
It can be a standard template at first for the master page
Note: We will need one navigation component because Navigation can place it anywhere on the master page

Examples 🌈
No response
Motivation 🔦
No response

@tanerozel Thanks for creating this issue.
In Toolpad, we currently have the option to duplicate pages. This way, the first page can be used as a template. Does this address the concern you have mentioned?
Also for an internal app, generally the pages:

don't follow a same layout
consists of a few components that can be quickly drag-dropped on canvas. So, fairly quick to be built (UI-wise)

Regarding the navigation component, currently, we have a fixed side navigation in a deployed app (example). In the future, we'll improve it as discussed here

@tanerozel If I understand this issue correctly, you're looking for a shared layout between pages? Is this a duplicate of https://github.com/mui/mui-toolpad/issues/1064


arm64 ubuntu18.04 python3.7 installation error

hi,
i am trying to install the library for python3.7 using pip3 install pyrealsense2. i am getting an error Collecting pyrealsense2   Could not find a version that satisfies the requirement pyrealsense2 (from versions: ) No matching distribution found for pyrealsense2.
i even tried python3 -m pip install pyrealsense2 but still no luck.
i have been trying all the solutions in other issues mentioned here but did not work.
can someone provide me solution?

Hi @siddharthcb   The pyrealsense2 pip packages are for x86 / x64 processors and will not work with Arm processors.  On a computing device that uses Arm, the pyrealsense2 wrapper should be built from source code instead.

hi @MartyG-RealSense
thanks for the reply but it is installing with python3.6 on the same arm64 processor.
you meant it is not compatible with python3.7 in arm64 ? or not compatible for any version of python in arm64?

The packages should not be compatible with Arm processors.  The terms "Arm64" and "AMD64" (an x64 processor) are not the same.

thanks a lot. i will try building it from source.

hi @MartyG-RealSense .
i built from source using python 3.6.
suppose say if i have upgraded version of python3.7. How do i change the one that i have built from source?

As far as I know, you would have to build the pyrealsense2 wrapper from source with CMake again and target it at Python 3.7 using the CMake instruction -DPYTHON_EXECUTABLE=[full path to the exact python executable]

is it cmake -DPYTHON_EXECUTABLE=[full path to the exact python executable?

when i run this cmake .. -DCMAKE_BUILD_TYPE=Release -DBUILD_EXAMPLES=true -DBUILD_PYTHON_BINDINGS=bool:true -DPYTHON_EXECUTABLE=/usr/local/lib/python3.7
i get following error:
`CMake Error at wrappers/python/third_party/pybind11/tools/FindPythonLibsNew.cmake:95 (message):
Python config failure:
Call Stack (most recent call first):
wrappers/python/third_party/pybind11/tools/pybind11Tools.cmake:16 (find_package)
wrappers/python/third_party/pybind11/CMakeLists.txt:33 (include)
-- Configuring incomplete, errors occurred!
See also "/home/smartagri/librealsense/build/CMakeFiles/CMakeOutput.log".`

Please try the usr/bin path instead of lib:

DPYTHON_EXECUTABLE=/usr/bin/python3.7

Variations of the -DPYTHON_EXECUTABLE instruction that you can try are:
-DPYTHON_EXECUTABLE=...
(i.e just 3 full stops)
-DPYTHON_EXECUTABLE=/usr/bin/python3

thank you. issue solved.

Thanks very much for the update @siddharthcb


[Rancher2] Add NeuVector Page

Request Summary:
Add a page related to the new Neuvector integration
Details:
should link back to this: (https://open-docs.neuvector.com/deploying/rancher#deploy-and-manage-neuvector-through-rancher-apps-marketplace)

Docs team can handle this however they like. If fleet/longhorn have a pre-existing pattern.
Also NeuVector will need a top-line release note, as it is a flagship feature of 2.6.5.

closing as duplicate of https://github.com/rancher/docs/issues/4046


Ensure LD_LIBRARY_PATH set when building using a linux devkit

When building using a linux devkit, the LD_LIBRARY_PATH needs setting prior to the build to use the devkit sysroot lib64:lib

I never liked the REMATCH syntax, but if it works I'm ok with it :-) Do these need to be in prepareWorkspace.sh as well as linux.sh? I'll approve regardless for this cycle but it would be good to avoid the dupliation in the future if possible.

I could do that better yes

I never liked the REMATCH syntax, but if it works I'm ok with it :-) Do these need to be in prepareWorkspace.sh as well as linux.sh? I'll approve regardless for this cycle but it would be good to avoid the dupliation in the future if possible.

yep, i've resolved it into one place


Feature/add extra labels and annotations locust

This PR allows people to customize labels and annotations. This is primarily usefull for people who have kubernetes plugins that relay on custom annotations to find data.

/assign @mattfarina

/retest

/ok-to-test

/lgtm


Make the tests work

Tests are currently not run since this was basically copied from the s2i-node image.  And a good chunk of those tests might not be relevant to this image.
We need to see what tests we can keep and what we get rid of and then re-enable travis

We need to see what tests we can keep and what we get rid of and then re-enable travis

We have tests to check the connection with the test-example-app running.
I think this is enough since the other verification tests are done on the parent image.

There are a few parameters that we can send in, like DEPLOY_PORT, NPM_RUN and NPM_BUILD.  we might be able to test DEPLOY_PORT and maybe NPM_RUN,  but i'm not sure how to test the NPM_BUILD option


Using ajax loader

If it possible to have a loader gif before the image is displayed? My images are quite big and can take a few seconds to load.
thanks

Hi fsjsy,
It's possible for you to show a loading indicator, apply backstretch, and then pass in a callback function to hide the loading indicator. That said, I think it would be much easier to have jQuery preload the images for you. I've updated the slideshow example to include preloading code:
https://github.com/srobbin/jquery-backstretch/blob/master/examples/slideshow.html
Hope that helps.
–Scott


Hello, author. Is there any academic paper here

Hello, as I am writing an academic report, how can I cite you?

There isn't a published academic accompanying this work, so I'm not completely sure how to cite it formally. From my perspective, as long as you mention my name a link to this repository that's ok with me. Thanks for checking!

@ashkonf  Any example to use, please?

What format are you using?

@ashkonf  .txt

example data https://archive.ics.uci.edu/ml/machine-learning-databases/statlog/german/german.data

I just added: https://github.com/ashkonf/HybridNaiveBayes/blob/master/src/test.py


thanks

just used it for my project
THANKS
http://blog.devekko.com/make-favicons-ico-using-sketch/

+1 very convenient!

Do you know if it's working with the latest update of Sketch? I'm using 49.1 and I'm getting an error when trying to open the file that reads, "The document “favicon-exporter.sketch” could not be opened." No other details. Thanks in advance!


Add type annotations to IndexOpsMixin(base.py)

Codecov Report

Merging #1892 into master will decrease coverage by 2.57%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1892      +/-   ##
==========================================
- Coverage   94.20%   91.63%   -2.58%
==========================================
  Files          40       40
  Lines        9939     9862      -77
==========================================
- Hits         9363     9037     -326
- Misses        576      825     +249

Impacted Files
Coverage Δ

databricks/koalas/base.py
97.35% <100.00%> (-0.04%)
:arrow_down:

databricks/koalas/usage_logging/__init__.py
25.66% <0.00%> (-66.65%)
:arrow_down:

databricks/koalas/usage_logging/usage_logger.py
47.82% <0.00%> (-52.18%)
:arrow_down:

databricks/koalas/__init__.py
76.66% <0.00%> (-13.34%)
:arrow_down:

databricks/conftest.py
89.83% <0.00%> (-10.17%)
:arrow_down:

databricks/koalas/spark/functions.py
88.88% <0.00%> (-7.41%)
:arrow_down:

databricks/koalas/accessors.py
86.06% <0.00%> (-6.97%)
:arrow_down:

databricks/koalas/generic.py
90.68% <0.00%> (-4.68%)
:arrow_down:

databricks/koalas/namespace.py
78.66% <0.00%> (-4.64%)
:arrow_down:

databricks/koalas/frame.py
93.63% <0.00%> (-3.11%)
:arrow_down:

... and 14 more

Continue to review full report at Codecov.

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

Thanks! merging.


Add automatic UML-scheme rendering to .github/workflows

It's hard to read .puml files!

Implemented by 6f5ea4c7217fb832b9d8a4597f996a298157ca16

Or better, with PR #15


JSON Serialization for user-facing types

Is your feature request related to a problem?
The runner needs to know how to serialize certain objects to JSON, as they need to be sent to the VS Code extension.
Currently, these types would be:

[ ] Table (https://github.com/Safe-DS/Library/blob/main/src/safeds/data/tabular/containers/_table.py)
A JSON representation already exists by calling to_json_file. To use this here, a new method without files would be needed

[ ] Image (https://github.com/Safe-DS/Library/blob/main/src/safeds/data/image/containers/_image.py)
An image can be represented as a byte array by calling _repr_png_. This could be encoded to a string with base64.

Desired solution
As the json.dumps function is used to encode data, a custom JSONEncoder subclass could be used that encodes these additional values and falls back to the default encoder for all other values (like numbers).
The custom encoder can be specified using the cls argument of the json.dumps function.
Possible alternatives (optional)
No response
Screenshots (optional)
No response
Additional Context (optional)
Some types like Tables and Images are useful for inspection purposes.
They should be able to be displayed in the VS Code extension to show information about the data of a pipeline.
Currently, placeholders are sent in the data field of the placeholder_value message.

:tada: This issue has been resolved in version 0.5.0 :tada:
The release is available on:

v0.5.0
GitHub release

Your semantic-release bot :package::rocket:


Add 404 screen+

Add global 404 screen
Display resource fetch errors inline

Addresses #779.
Signed-off-by: PGobz<EMAIL_ADDRESS>

Closing until previous PRs are merged to minimize conflicts.


"Init composer" is not available before indexing of the project

As changing the library paths impacts the indexing, it would make sense to be able to configure it as soon as possible rather than having to wait until PhpStorm indexed all vendors as being project files (which can take some time)

"Init composer" is not a feature of the plugin, it is native PhpStorm feature, so I am not able to change this :(

oh, I thought it was coming from this plugin.
So closing this


Talk about OSD

In the "Useful Utilities" section, please talk about OSD.
They  are needed to change volume/brightness levels and display related notifications, as well as notifications for Caps Lock and Num Lock.

they are not needed. I don't use them, most people don't use them.
Furthermore, all OSD I've seen was eww / ags, which are mentioned

they are not needed. I don't use them, most people don't use them.
Furthermore, all OSD I've seen was eww / ags, which are mentioned

And how do you change volume and brightness?

awesome-hyprland has a list of OSDs here

And how do you change volume and brightness?

By directly invoking tools like light (brightness) or wpctl (volume). This doesn't display the value to the screen, but it does change it, and that's what matters

awesome-hyprland has a list of OSDs here

And how do you change volume and brightness?

By directly invoking tools like light (brightness) or wpctl (volume). This doesn't display the value to the screen, but it does change it, and that's what matters

Can we add those tools in the wiki then?
I didn't know they existed and ended up writing my own

proycon/wayout
it takes text from stdin and shows on screen
https://git.sr.ht/~proycon/wayout
available in nix as "proycon-wayout"
:( but it didn't worked for me...

proycon/wayout
it takes text from stdin and shows on screen
https://git.sr.ht/~proycon/wayout
available in nix as "proycon-wayout"
:( but it didn't worked for me...

Came looking for info on OSDs too, so it is a shame the topic was so quickly dismissed.
For anyone else that lands here, I found this repo that lists so many useful/essential additions, including options for OSDs, seemingly in spite of no one ever needing them... 🤨
https://github.com/hyprland-community/awesome-hyprland?tab=readme-ov-file#osd


[WFLY-13435] / [WFLY-13509] / [WFLY-13434] / [WFLY-13319] DUP Refactoring for SecurityDomain resolution.

This pull request depends upon https://github.com/wildfly/wildfly-core/pull/4276
As described in the WildFly Core PR this is predominantly about refactoring deployment unit processors so a security domain can be identified early and re-used by later deployment unit processors without repeating domain resolution logic.
The end result also means any virtual security domain created for MP JWT can be reused within an EJB deployment and a test case is added verifying the propagation of an identify established using JWT from the servlet container to the EJB container.
As described in the WildFly Core PR it will still be desirable to pull this resolution earlier so the security policy is defined before entering Phase.INSTALL but presently a deployment can be converted to a web application later in the INSTALL Phase.
https://issues.redhat.com/browse/WFLY-13435
https://issues.redhat.com/browse/WFLY-13509
https://issues.redhat.com/browse/WFLY-13434
https://issues.redhat.com/browse/WFLY-13319

Experimental jobs:
https://ci.wildfly.org/buildConfiguration/WF_WildFlyCoreIntegrationExperiments/213201
https://ci.wildfly.org/buildConfiguration/WF_WildFlyCoreIntegrationExperiments/213203

The mixed domain test failures on https://ci.wildfly.org/buildConfiguration/WF_WildFlyCoreIntegrationExperiments/213521 can be ignored, this was a -Delytron run and we do not normally run mixed domain tests on that profile.

@chengfang / @fl4via / @rachmatowicz / @tadamski FYI this PR is handling DUP refactoring to better collaborate on the identification of the Elytron security domain to use and share the resolved security domain across deployments.

@chengfang @tadamski I'm going to go ahead and merge this one as it's blocking other critical work. If there are concerns we can follow up later.


feat: rework TaggedTable

Closes #647
Summary of Changes

TaggedTable is now called TabularDataset,
It is moved from safeds.data.tabular.containers to safeds.data.labeled.containers. That's where all dataset classes for supervised learning will go, like the upcoming ImageDataset.
TabularDataset no longer inherits from Table.
TabularDataset now has a very small interface. It's only meant to be used as input for supervised ML models. Table manipulation is now solely done using the Table class.
tag_columns on Table is now called to_tabular_dataset. This makes it consistent with other conversion methods and emphasizes that this is a terminal operation and should only be used once one is done manipulating the table.
TabularDataset now has a public to_table method to get a Table again.

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

v0.22.0
GitHub release

Your semantic-release bot :package::rocket:


Glyphicons don´t show up with rails 4.2 and bootstrap 3.3.3 in development environment

I have fonts in /app/assets/fonts/bootstrap
In the file applications.js
//= require bootstrap-sprockets
//= require bootstrap
In the file application.css.scss
@import "bootstrap-sprockets";
@import "bootstrap";
In the file Gemfile
gem 'bootstrap-sass', '~> 3.3.3'
gem 'sprockets-rails'
gem 'sass-rails', '~> 5.0'
I don´t see the glyphicons when I load the main page with a proper code.
Can anybody help me?

Check your Chrome-browser console, you have any font path error?

Based on #831, seems like your file is supposed to be named application.scss instead of application.css.scss

Closing due to lack of further response from OP.


The colored text makes the translated portions different.

When translating text such as in the image, it seems that different colors are perceived as different text. Can this be solved by adding an option in the config?
I would like to know what the solution is.

you can try to disable RichText handling in config


Add support for rendering a custom wrapper around Element

This PR suggests to extend module API with possibility to provide a wrapper component to be rendered arounnd Element.
PR in element web https://github.com/vector-im/element-web/pull/25537 that is linked.

@weeman1337 thanks for your comments, I have provided the changes. Here is a PR in matrix-react-sdk https://github.com/matrix-org/matrix-react-sdk/pull/10321 that is linked. Is it fine just to keep this link just in a comment or should I move it to the description?

@maheichyk sorry this issue got out of sight. Usually the workflow here is to re-request a review one changes are done.

This way it will be on my review list :)

Is it fine just to keep this link just in a comment or should I move it to the description?

For a better discoverability add it to the description please.

@germain-gg we discussed this PR last Wednesday, it seems to be okay. Could we move this forward?

@germain-gg thanks a lot for the suggestion, sounds a good idea. I have added WrapperLifecycle and removed the banner related code. Please have a look.

@maheichyk could you update your branch with the base branch? It's preventing me from merging that pull request otherwise. And I do not have permissions as maintainer to push to your branch.

@germain-gg thanks for the merge! Could you please create a release of the module api, then I could finish: https://github.com/vector-im/element-web/pull/25537

@maheichyk It's been released as part of 2.1.0

@fnwbr yes, it is used in our opendesk-module that is public since some time now.


Powershell Core OSC9;9 string leaves residue in prompt after using grep from git-for-windows with pipe

This is a port of an issue previously filed here: https://github.com/MicrosoftDocs/terminal/issues/425.
Output of the bug
PS C:\Users\ACER> dir | grep 'pattern'           # no match
PS C:\Users\ACER> ←]9;9;"C:\Users\ACER"
PS C:\Users\ACER>
PS C:\Users\ACER> dir | grep 'Documents'     # matches
d-r--           10/9/2021 11:04 PM                Documents
PS C:\Users\ACER> ←]9;9;"C:\Users\ACER"
PS C:\Users\ACER>
PS C:\Users\ACER> grep -HnrIi 'damn' .\Documents\PowerShell\        # to show that without piping the bug doesn't appear
.\Documents\PowerShell\/Scripts/scratch.ps1:1:function Damn([string]$sym)
PS C:\Users\ACER>
PS C:\Users\ACER> dir | win32yank.exe -i  # to show that the problem is not with piping
PS C:\Users\ACER>

Reproduction steps

Put the following in the $PROFILE.function prompt
{
$loc = Get-Location

# Emulate standard PS prompt with location followed by ">"
$out = "PS $loc> "

# Add current working directory (FileSystem)
if ($loc.Provider.Name -eq "FileSystem") {
    $out += "$([char]27)]9;9;`"$($loc.Path)`"$([char]7)"
}

return $out
}

Open a new PWSH tab in windows terminal.
Make sure git is in the $env:path and invoke:dir | grep -i "pattern"

Versions of tools used

Windows Terminal: 1.15.2003.0 (why does wt -v not print in the console? I cannot even copy from the popup window.)
PowerShell Core: PowerShell 7.1.4
GIt for Windows: git version 2.33.0.windows.2

Discussions elsewhere
I mentioned in another issue -- may or may not be related -- that Windows Terminal crashes randomly if I use OSC9;9 string in the prompt. But I've been using the preview build for a week and it hasn't crashed yet. Update: I have had no crashes since then.

why does wt -v not print in the console?

It is too sad to countenance. Funny enough, if you press Ctrl+C in any standard Windows message box, it DOES unexpectedly give you a hard copy of it:
---------------------------
Help
---------------------------
Windows Terminal Preview
1.16.2004.0
---------------------------
OK
---------------------------

I'm guessing this is related to msys restoring the console mode and turning off VT input . . . let me go digging, because I wrote this up before.

let me go digging, because I wrote this up before.

This is way too fast a response. I feel guilty now. Take your time.

Funny enough, if you press Ctrl+C in any standard Windows message box, it DOES unexpectedly give you a hard copy of it

Live and learn.

I feel guilty now.

Ha, don't! It's not a problem at all.
The symptoms here feel very similar to the ones dissected in https://github.com/microsoft/terminal/issues/4921#issuecomment-599135352, which we traced back to an issue in the MSYS2 runtime that Git uses. I'm not sure whether the state of this issue has changed between 2.33 (the version you have installed) and 2.37 (the current version).

I can reproduce issue 4921. Although, I need to press enter once, then ←]9;9;"C:\Users\ACER" appears. Also, if I run a command that produces an exception (apparently powershell errors and exceptions are different), the output is like this.
C:\Users\ACER> git gui
C:\Users\ACER> wrong_cmd hello world
←[91mwrong_cmd: ←[91mThe term 'wrong_cmd' is not recognized as a name of a cmdlet, function, script file, or executable program.
Check the spelling of the name, or if a path was included, verify that the path is correct and try again.←[0m
C:\Users\ACER> ←]9;9;"C:\Users\ACER"

So, basically terminal loses the ability to recognize the escape/special characters as escape/special characters, right?

I'm not sure whether the state of this issue has changed between 2.33 (the version you have installed) and 2.37 (the current version).

This issue persists with Msys2's grep command, too. And I regularly update msys.

Yea, this looks like the same thing. I'm game to call this a dupe (and reopen if I'm wrong).
/dup #4921


Add setCrop method to BitmapText game object

Please do not update the README or Change Log, we will do this when we merge your PR.
This PR (delete as applicable)

Adds a new feature

Describe the changes below:
Add setCrop method to BitmapText game object.

It seems that pushScissor can't rotate, i.e. can't apply crop feature on a rotated bitmaptext.
So I will close this PR.


Template reference variables not recognized by the linter

OS

Mac OSX (El Capitan)

Versions.
angular-cli: 1.0.0-beta.22-1
node: 4.4.2
os: darwin x64

Repro steps.
Linter doesn't seem to understand template reference variables.
For example, for a given block of HTML:
<input type="text" #myInput>
<p>{{ myInput.value }}</p>

With the default tslint config, I get the following error when running ng lint:
The property "myInput" that you're trying to access does not exist in the class declaration.
Other than that, HTML linting is a great addition to the linter, thank you!

This is a bug with codelyzer itself, please file it on their repo: https://github.com/mgechev/codelyzer


Service Principal Name incorrect

The instructions give a wrong app name.

If you don't see Microsoft Azure Windows Virtual Machine Sign-In as a result, the service principal is missing from the tenant.

It should say ...

If you don't see Azure Windows VM Sign-In as a result, the service principal is missing from the tenant.

The wrong app name is also used in the code to check for and create the Service Principal
(https://learn.microsoft.com/en-us/entra/identity/devices/howto-vm-sign-in-azure-ad-windows#missing-application)

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

ID: c946fe0f-e18d-2529-2fbf-bce04759fbca
Version Independent ID: 885a61d1-6096-5aa0-fe8d-f1ec8d55e542
Content: Log in to a Windows virtual machine in Azure by using Microsoft Entra ID - Microsoft Entra ID
Content Source: docs/identity/devices/howto-vm-sign-in-azure-ad-windows.md
Service: active-directory
Sub-service: devices
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

#label:"assigned-to-author", "doc-enhancement", "triaged"
#reassign:@MicrosoftGuyJFlo

Thanks for catching that. Some names have been changing in the background lately.


[sql lab] fix encoding error in logging message

In Sql Lab, user types in a query with unicode like this:
SELECT * FROM "tb_name" WHERE city = "Lübeck"
When sql_parse.py and sql_lab.py tried to log message like:
logging.info("Parsing with sqlparse statement {}".format(self.sql))
This cause exception thrown, and user got Internal Server Error from Sql Lab.

Coverage decreased (-0.2%) to 68.9% when pulling 7ad234ab6e2717db82ed30dc32676af8a7534d53 on graceguo-supercat:gg-SQLlabEncoding into 3dfdde130a88d3a12a6e084adc07d284ed6090e6 on apache:master.

Coverage decreased (-0.2%) to 68.9% when pulling 7ad234ab6e2717db82ed30dc32676af8a7534d53 on graceguo-supercat:gg-SQLlabEncoding into 3dfdde130a88d3a12a6e084adc07d284ed6090e6 on apache:master.

Coverage decreased (-0.2%) to 68.961% when pulling d8a769fc3dec5e74624ae62a5da266dbcba1b247 on graceguo-supercat:gg-SQLlabEncoding into 3dfdde130a88d3a12a6e084adc07d284ed6090e6 on apache:master.

Coverage decreased (-0.2%) to 68.961% when pulling b27b5bf70dbfbaa2ad0fe919abca2f3a88172ce5 on graceguo-supercat:gg-SQLlabEncoding into 3dfdde130a88d3a12a6e084adc07d284ed6090e6 on apache:master.

Coverage decreased (-0.2%) to 68.961% when pulling b27b5bf70dbfbaa2ad0fe919abca2f3a88172ce5 on graceguo-supercat:gg-SQLlabEncoding into 3dfdde130a88d3a12a6e084adc07d284ed6090e6 on apache:master.

Coverage increased (+0.005%) to 69.123% when pulling 786b12ff28c1bf40312f430d907cb856619d3ae3 on graceguo-supercat:gg-SQLlabEncoding into 3dfdde130a88d3a12a6e084adc07d284ed6090e6 on apache:master.

Coverage decreased (-0.2%) to 68.962% when pulling b8b5ea9122a7c186dc3ce54a545097e180e3a00f on graceguo-supercat:gg-SQLlabEncoding into 8223729e1ebee9fa0a1116c736037f750e8b2578 on apache:master.

Coverage increased (+0.009%) to 69.129% when pulling eeb6b8682bd9ce7eb8f594862a74b2049866dc56 on graceguo-supercat:gg-SQLlabEncoding into 8223729e1ebee9fa0a1116c736037f750e8b2578 on apache:master.

Coverage increased (+0.009%) to 69.129% when pulling eeb6b8682bd9ce7eb8f594862a74b2049866dc56 on graceguo-supercat:gg-SQLlabEncoding into 8223729e1ebee9fa0a1116c736037f750e8b2578 on apache:master.


[1.20.1]Server Startup Failure: Conflict between Hephaestus and Create leading to NoClassDefFoundError

Minecraft Version
1.20.1
Fabric/Quilt Version
0.14.22
Fabric/Quilt API Version
0.87
Hephaestus Version
1.20.1-<IP_ADDRESS>
Describe your issue
When both Hephaestus and Create mods are present, the server fails to start due to a NoClassDefFoundError. Specifically, Hephaestus is attempting to access a class from the Create mod (io/github/fabricators_of_create/porting_lib/entity/events/LivingEntityEvents) which it cannot find, leading to a server crash.
https://github.com/Fabricators-of-Create/Create/issues/1061
Crash Report
https://github.com/Fabricators-of-Create/Create/files/12385245/latest.log
Other mods
Create
Tried reproducing with just Tinkers?
Yes
Performance Enchancers
Sodium
Searched for known issues?
I did not search

i have the same problem. is there any process?

will they try to fix it for 1.20.1? i really need it for this modpack im making but if they cant fix it then thats ok

Would that fix also be released for 1.20.1? I was really looking forward to this mod


Support for registering traefik to consul, AWS ELB, etc.

Do you want to request a feature or report a bug?

Feature
What did you expect to see?

Hi! I'm Træfiker :robot: the bot in charge of communication regulation.
Thanks for your interest in Traefik!
We dedicate the issue tracker to bug reports and feature requests only. My advanced AI has spotted that your issue might be a configuration problem or relates to something that doesn't look like a bug.
To confirm this, please join our Community Forum and reach out to us on the Traefik section.
In case I'm wrong (well, that would be embarrassing :sweat_smile:), my developers will re-open the issue and fix me!
In the meantime, you can double check Traefik's documentation.


Strategy for inserting rows with table-values parameters

With SQL Server you can used table-values parameters to perform bulk DML very quickly and elegantly. Unfortunately, this is tedious:

Create a table type
Write an INSERT statement
Create a DataTable or an IEnumerable
Execute a command

Here is a proposal for how EF could pull off all of this transparently for inserts and deletes in SaveChanges:
Generate a table type programmatically for all tables that require it. 3 issues:

The table type probably should be created in a separate transaction so that concurrent SaveChanges calls do not contend. But really this is optional because type creation is a one-time initialization.
A type name must be chosen and the type's structure might change over time. This can be solved by appending a strong hash of the type's structure to it's name. That way table types are immutable. If the table structure changes a new type will be created. The hash would include column names, their order, data types, nullability and the primary key. Old TVP types are simply never cleaned up. They only arise on schema change which is rare.
This requires DDL permission. EF could check on startup for these permissions. Alternatively, the feature could be opt-in.

I think all of this would work for deletes as well. Updates are trickier because there is a great variety of columns that might change or not change. Maybe EF can use a single type for all updates and simply ignore some columns for some updates.
Non-issues:

Generated values (identity, defaults). The OUTPUT clause can return them elegantly.
Performance. For SaveChanges calls with few rows the existing row-by-row strategy should be used. Over the network the TVP strategy is probably better starting with 2-3 rows due to roundtrip times. On the same machine I measured the threshold to be 10 for a particular workload.
Semantics. I don't think the semantics of a SaveChanges call would be affected in any way. It's simply a "go faster" feature.
"Do we really need this given that SqlBulkCopy exists?": TVPs can kind of compete with SqlBulkCopy. They are an integer factor slower but like 2 orders of magnitude faster than row-by-row inserts. 2 OOM go a long way. Often, this will be good enough. I think SqlBulkCopy usage will drop dramatically once this feature is available in EF.
Topological ordering of DML. I don't see any issues here. Doing an entire table at once should always result in a valid topological order. Alternatively, EF could detect safe cases and fall back to row-by-row otherwise.
Row order. Index uniqueness validation logically happens at the end of a DML statement. Therefore the order of rows in the TVP does not matter for correctness.

Performance benefits:

Many-row inserts are much more efficient because SQL Server can "see" all rows at once and maintain all indexes optimally (usually by writing to indexes at a time in sorted order).
Less round-trips. In a network a round-trip might cost 0.5ms.
Less CPU on both client and server.

Who says that EF is not suitable for bulk inserts? Right now that might be the case but it does not have to be so. This would be awesome.
The point of this ticket is to present a viable plan for how this could be implemented in EF. Some inspiration for the dev team.

Would make a great PR!

@GSPP I agree with @ErikEJ, this sounds like a great PR. Currently EF7 supports batching, so things are already factored in a way that multiple operations of the same type are grouped and submitted to the server together. I would expect TVPs to have a perf advantage over what we do, but I don't know how much.

I think TVPs would only have a measurable perf advantage when inserting 100s or 1000s of rows (which is not what you would normally do with EF OLTP systems)

@ErikEJ I understand why you might think that but measurements turn out to now support this view.
I have meant this TVP feature to be used with "bulky" inserts (100-1M rows). It is not primarily targeted at OLTP inserts (dozens of rows) but the perf benefits are visible there as well. I can only encourage you to try it because the gains can be staggering.
I'd like to quote this piece:

Many-row inserts are much more efficient because SQL Server can "see" all rows at once and maintain all indexes optimally (usually by writing to indexes at a time in sorted order).

Do not underestimate the difference that a better plan can do. Per-index maintenance in bigger batches can be significantly faster than issuing new queries and round-trips all the time. Even without indexes, imagine a single-row insert (small row, no indexes). 90% of the execution time is per-statement and per-batch overhead. The useful work (the per-row work) is small. With TVPs the per-statement and per-batch overheads exist once for many rows (granted, they will be a little higher but not by much).

Not related but FYI I was just thinking about a similar trick for PostgreSQL/Npgsql: switch to PostgreSQL's COPY (bulk data transfer) protocol for modification batches with many inserts.

@GSPP Sounds great, I have actually implemented this for doing INSERTs in a single table with 3-10 rows, and it Works great. DDL permissions could be a showstopper, however.

Not only for NULL. You can also do so for other distinct values. Is it possible to see where a value comes from? I often join multiple tables iwth basically the same condition (owner, tenant) and get multiple parameters generated.

This is generated seen as bad practice or risky because it causes additional query plans to be generated.

I think you might be able to make a single TVP for each entity if you include an Ignore flag.
For insert values this would present a challenge since you would have to lookup the default value.


Writing direction in bullet/numbered lists doesn't persist when sharing document in view mode

When sharing a document in view mode, the writing direction of text in any lists always appears from left to right.
Here is an example document in edit mode:

Here's the same document when shared in view mode:

Notice that the paragraph with the bullet point becomes unreadable in view mode.

oh.. I'll take a look.
it seems that text direction isn't properly preserved for server-side rendering.

it seems that I have an open pull request regarding this issue but it didn't get merged :(
https://github.com/facebook/lexical/pull/4942
For now I copied the code with the fix, hopefully I can get the lexical team to merge it so I'd revert this mess :)
https://github.com/ibastawisi/matheditor/commit/a7a1301


GPU Memory failed to allocate

I am trying to run colabfold on >2500 aa sequence. I read the other thread #7 regarding adjusting the following settings:
#!/bin/bash
export TF_FORCE_UNIFIED_MEMORY=1
export XLA_PYTHON_CLIENT_MEM_FRACTION=4.0
colabfold-conda/bin/python3.7 runner.py

If I follow https://sbgrid.org/wiki/examples/alphafold2 recommendations:
TF_FORCE_UNIFIED_MEMORY=1
XLA_PYTHON_CLIENT_MEM_FRACTION=0.5
XLA_PYTHON_CLIENT_ALLOCATOR=platform

Either way, I have been running into the following error:
tensorflow/core/common_runtime/gpu/gpu_device.cc:1871] Adding visible gpu devices: 0, 1, 2, 3, 4, 5, 6, 7
jax._src.traceback_util.UnfilteredStackTrace: RuntimeError: Resource exhausted: Failed to allocate request for 3.25GiB<PHONE_NUMBER>B) on device ordinal 0
Checking nvidia-smi indicated that the primary half of the memory of one GPU is being used and not the rest. I tried restarting my ssh sessions too to make sure nothing else was being unknowingly used in the background prior to running this command.

export CUDA_VISIBLE_DEVICES=0 or any value to use only one gpu, no solution for the out of memory pb than getting a bigger gpu...

Yeah so I rerun the command every couple of seconds. Doesn't go over 50%. Yeah, still crashes with the same error.


Update what-is-azure-databricks.md

There was a typo in it. The ML library was wrongfully named 'MLib' instead of 'MLlib' in the document.

@Leviter : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


fixed typo in Chinese of width

Fixes #[632]
Changes:

change "阔" to "宽" for width
Screenshots of the change:

also changed the typo in Coordinates example that misses a "1".

thanks!

thanks!


Archiving/Unlisting

Applies to both projects and versions.
Archiving
Channels with any associated versions cannot be deleted, but archived where the channel, and all versions are hidden from view from users without permissions.

Versions archived due to an archived channel, cannot be manually un-archived. The offending channel must be un-archived.
Versions can be manually archived/un-archived at the users discretion.
Projects can be manually archived/un-archived at the users discretion.

Unlisted
Add a setting for project channels so any versions released on that channel is unlisted, meaning it can be linked to, but doesn't show up in any version lists (for people w/o permissions).

Versions CAN be manually re-listed if associated with a channel configured to unlist versions. I consider the setting on the channel, as just a preset for what the version should start out as, not permanently stay.
Projects can be unlisted/re-listed at the users discretion.

Maybe this doesn't have to be done by initial release, for the moment we can just not allow deleting of channels with any versions.

archived projects should have a banner, on archival we should ask for a custom archival message ("I dont want to maintain this shit anymore, consider X and Y as alternatives" or something along those lines)


Add output image (with keypoints) scaling

Now, for small camera resolutions, an image of the same size is displayed, but it is preferable to have an image with keypoints scaled according to the screen parameters.

Solved in https://github.com/TimPushkin/FeatureDetectAndroid/pull/35


Fix IRGenModule::getMaximalTypeExpansionContext to use the proper associated context

In a per file compilation mode we want to use the file context -- which is stored in the AssociatedContext -- rather than defaulting to the current Swift Module context.
rdar://114344533

@swift-ci test


ui: make two level footer

Closes https://github.com/inveniosoftware/invenio-app-rdm/issues/256
Current code:

Example with according to issue:

More zoom to display the shadow:

Looks good! Here my comments:

Colors:

Use blue from the header (left side) as the background color of the top footer
Darken that blue (see Zenodo SCSS for a good value to darken it with)

Shadows looks good
Would put the language in the bottom footer bar, but smaller size.
Menu:

Less spacing between menu items (note the menu header is fine.
Colors of the menu items you could do with specifying white and make the color a bit transparent.

Applied changes:

Made footer text colors into variables
Changed footer colors, made light equal than the top and used Zenodo's dark blue
Moved language to lower footer
Reduced spacing between menu items
Made link color white with transparency


Help needed with links and navigation inside of a pdf.

I understand that you can use pdf.textWithLink and pass some options and you can jump from page 1 to page 12 in one click. The thing is you need to know the page number. Is there a way where I can create some kind of a point in a pdf and go to it based on name/class or something.
I have a table in my pdf that can be 1 page long or even 15 pages long. now because autoTable creates the new pages by itself I cant really track of it and I would still love to have an ability to quick get to these pages.
I read about something called annotation in the docs but i never really saw an actual helpful example, also I am using jspdf on node.js and not in the client side.

You could use hooks to remember which cell is on which page and add the links afterwards.

for example use:
didDrawPage: (HookData) => {}
to save the page to a variable and then use that variable in the link text?

Thanks for the tip! I managed to succeed and here is an example below for anyone looking to do the same.
I have a service that generates those autoTables so I have a function as such:
  getVulnsSummaryTable(params: { doc: typeof jsPDF; vulns: any[] }) {
    const { doc, vulns } = params;
    let pageNumber;

    const rows: Array<Array<string>> = [];

    vulns.forEach((element) => {
      var row = [element.legend, element.name, element.description, element.severityLevel, element.assetIdentifier];
      rows.push(row);
    });

    doc.autoTable(['', 'Vulnerability name', 'Description', 'Severity level', 'Asset name'], rows, {
      headStyles: {
        fillColor: [22, 50, 75],
      },
      margin: { right: 24, left: 24 },
      startY: doc.pageCount > 16 ? doc.autoTableEndPosY() + 32 : 80,
      didDrawPage: (data: CellHookData) => {
        pageNumber = data.table.startPageNumber;
      },
    });

    return pageNumber;
  }

And because I return the page of where the table starts, I can later use it as a hyperlink and give 'quick' links to the table pages:
const vulnsSummaryStartPage = jspdfTableService.getVulnsSummaryTable({ doc, vulns: vulnsForTable.slice(0, 2) });

 doc.textWithLink('See full list', 24, 90, {
      pageNumber: vulnsSummaryStartPage ,
      magFactor: 'XYZ',
      zoom: 1,
    });

Also in case you put the links later like myself, you can use it as such:
doc.insertPage(3); // 3 for example..
doc.setPage(3);

// your links page..


Add negative sound duration handling

Important: There was a test case in Edison that had a play tone block with a duration of 0. Since the new handling has been added, that test now fails. Therefore I have changed the tone duration to 20 ms, as well as the corresponding XML file so that the JUnit test runs successfully.
Fixed issue #469 for all robot types that support visitToneAction. For example, in WeDo,
Previously:

Now it skips over negative sound duration:

The approach is correct, but there needs to be a warning for the user so they know that nothing will be executed, otherwise they might not notice.


Box 'geerlingguy/drupal-vm' could not be found. Attempting to find and install...

Issue Type

Bug Report / Support Request

Your Environment
$ vagrant up
==> drupalvm: Box 'geerlingguy/drupal-vm' could not be found. Attempting to find and install...
drupalvm: Box Provider: hyperv
drupalvm: Box Version: >= 0
==> drupalvm: Loading metadata for box 'geerlingguy/drupal-vm'
drupalvm: URL: https://vagrantcloud.com/geerlingguy/drupal-vm
The box you're attempting to add doesn't support the provider
you requested. Please find an alternate box or use an alternate
provider. Double-check your requested provider to verify you didn't
simply misspell it.
If you're adding a box from HashiCorp's Vagrant Cloud, make sure the box is
released.
Name: geerlingguy/drupal-vm
Address: https://vagrantcloud.com/geerlingguy/drupal-vm
Requested provider: [:hyperv]
bash: $: command not found

Your OS

Windows (10 pro)

Full console output
$ vagrant up
==> drupalvm: Box 'geerlingguy/drupal-vm' could not be found. Attempting to find and install...
drupalvm: Box Provider: hyperv
drupalvm: Box Version: >= 0
==> drupalvm: Loading metadata for box 'geerlingguy/drupal-vm'
drupalvm: URL: https://vagrantcloud.com/geerlingguy/drupal-vm
The box you're attempting to add doesn't support the provider
you requested. Please find an alternate box or use an alternate
provider. Double-check your requested provider to verify you didn't
simply misspell it.
If you're adding a box from HashiCorp's Vagrant Cloud, make sure the box is
released.
Name: geerlingguy/drupal-vm
Address: https://vagrantcloud.com/geerlingguy/drupal-vm
Requested provider: [:hyperv]
bash: $: command not found
Summary
i have posted full console result please guide me how to resolve this issue, i am trying to resolve it from two days

I don't have any boxes supporting the HyperV platform; you'd have to install VirtualBox and use that instead. Otherwise you can change the box name in the config.yml file and use a box for Debian 10 or Ubuntu 18 or 20 from Vagrant Cloud that is built for HyperV.

by installing virtual box, it's installed successfully and accessible by ip http://<IP_ADDRESS>/
but Adminer, MailHog, PimpMyLog links are not working

by installing virtual box, it's installed successfully and accessible by ip http://<IP_ADDRESS>/ but Adminer, MailHog, PimpMyLog links are not working

i have added below lines in etc/hosts file.
<IP_ADDRESS> drupalvmsetup.local
<IP_ADDRESS> www.drupalvmsetup.local
<IP_ADDRESS> adminer.drupalvmsetup.local
<IP_ADDRESS> xhprof.drupalvmsetup.local
<IP_ADDRESS> pimpmylog.drupalvmsetup.local
<IP_ADDRESS> dashboard.drupalvmsetup.local
all links are working else main site link
http://drupalvmsetup.local/

by installing virtual box, it's installed successfully and accessible by ip http://<IP_ADDRESS>/ but Adminer, MailHog, PimpMyLog links are not working

i have added below lines in etc/hosts file.
<IP_ADDRESS> drupalvmsetup.local <IP_ADDRESS> www.drupalvmsetup.local <IP_ADDRESS> adminer.drupalvmsetup.local <IP_ADDRESS> xhprof.drupalvmsetup.local <IP_ADDRESS> pimpmylog.drupalvmsetup.local <IP_ADDRESS> dashboard.drupalvmsetup.local
all links are working else main site link http://drupalvmsetup.local/

browser's output
Not Found
The requested URL was not found on this server.
Apache/2.4.29 (Ubuntu) Server at drupalvmsetup.local Port 80

Guest hangs after 5 minutes of inactivity on  windows resolved:
Open Config.yml file
Under
vagrant_synced_folders
type: nfs
changes to
type: virtualbox