CI: Fix slow unittests

Here's the shortlist for slow unittests, this list is about 10 minutes of the 15-18 total

[x] Loggerd
[ ] Manuvers - improved but following_distance is still pretty slow
[ ] Laika - improved but still not great

42.32s call     selfdrive/controls/tests/test_following_distance.py::TestFollowingDistance::test_following_distance
34.05s call     selfdrive/locationd/test/test_locationd.py::TestLocationdProc::test_params_gps
30.40s call     selfdrive/car/tests/test_fw_fingerprint.py::TestFwFingerprintTiming::test_fw_query_timing
29.99s call     system/ubloxd/tests/test_ublox_processing.py::TestUbloxProcessing::test_ublox_gps_cutover
28.35s call     selfdrive/locationd/test/test_laikad.py::TestLaikad::test_laika_online
27.37s call     selfdrive/locationd/test/test_laikad.py::TestLaikad::test_laika_online_nav_only
26.18s call     selfdrive/locationd/test/test_laikad.py::TestLaikad::test_get_navs_in_process
13.21s call     system/ubloxd/tests/test_ublox_processing.py::TestUbloxProcessing::test_get_fix

once we fix these, we should enforce a default timeout of 10 seconds in pytest

We can disable the laika unit tests for now since it's been cancelled (leave process replay though).


Test TestLookupHost-2 failed when building

...
ok  	mime/multipart	0.128s
ok  	mime/quotedprintable	0.395s
--- FAIL: TestLookupHost-2 (0.01s)
hosts_test.go:132: LookupHost("localhost") = [<IP_ADDRESS> <IP_ADDRESS>], has duplicate addresses
FAIL
FAIL	net	2.072s
ok  	net/http	4.068s
...
$ go version
go version devel +202ef48 Fri Jun 12 04:33:50 2015 +0000 linux/amd64
$ uname -a
Linux kevin1 3.19.0-18-generic #18-Ubuntu SMP Tue May 19 18:31:35 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux

Quick check, does <IP_ADDRESS> or localhost appear more than once in your /etc/hosts/ file ?

Oh... Yes.
But it seems not a problem if  <IP_ADDRESS> appear more than once in /etc/hosts/ file.

It's not a real problem.  You can ignore this test failure.
The test is looking for a specific problem that can cause duplicate addresses to be returned.  If there really are duplicate addresses, the test will fail incorrectly.  Perhaps we should skip the test if testing.Short is true.


Add a contributing file

We should add a contributing file describing the desired workflow to make contributions to the repository.

IMHO we are using the standard GitHub flow so we should't need an extra file we could just reference the official GitHub docs.

Alright.


Delete symlink, fix path for elasticdump bin

Delete symlink for nodejs, existing good one.
Fix path for elasticdump bin in ENTRYPOINT command.

thanks!


Discord.JS V13 problem with Discord-Akairo

'"discord.js"' has no exported member named 'StringResolvable'. Did you mean 'UserResolvable'?
This is the message i get
These are the dependency i use:
"dependencies": {     "discord-akairo": "^8.1.0",     "discord.js": "^13.1.0"   },

Use the master branch
npm i discord-akairo/discord-akairo

As GoldenAngel2 stated: npm i discord-akairo/discord-akairo
This is the solution!!!


Usernames market is part of UsernameRegistrar.sol

In order to allow easier change of the payment logic, the registrar should not handle anything else then a public ledger (NFT).
This change would facilitate any future feature, such as changing payment token address, or anything.
The payment, locking and slashing conditions should be isolated from the registrar to enable future proofing.

108 makes upgradability smoother, because without it the migration process would be "every user needs to be migrated individually to a new registrar" as is the current behavior, but with the separation of the NFT ledger on a migration event the new  Username Registrar will become the new controller of this ledger, meaning that all users are migrated at once, and the NFT address won't change.


keeps re-rendering when using state data

When I use state data(ex. this.state.data) to render the list, it keeps refreshing when I add more data. Any solution for this?

Experiencing this also on Android.
"react": "16.6.3",
"react-native": "0.57.4",
"react-native-masonry-list": "^2.10.0"

+1

any one can fix that bug?

+1

+1

https://github.com/hejun041/react-native-masonry-list/blob/master/HowToUse.md
can solve sort bug and add extra data
hope help

Hi everyone,
New stable update will be coming out soon.  In a couple of days.  @hejun041, those are some good fixes.  You will be added to the contribution section, but next time, can you do a clean pull request?

Install the update
Type in the following to install the update.
$ npm install --save<EMAIL_ADDRESS>
or
$ yarn add<EMAIL_ADDRESS>

:tada: Efficiently Add More Images
Version *2.12.0 update: Without rerendering the images.
import MasonryList from "react-native-masonry-list";

//...
// example function to add more images
addMoreImages(newImages) {
    this.setState({
        images: this.state.images.concat(newImages)
    });
}

render() {
    return (
        <MasonryList
            images={this.state.images}
        />
    );
}
//...

:+1: If you like react-native-masonry-list, please be sure to give it a star. Thanks.


odcds: log abnormal message in higher level

The reason that client request can't find the on-demand cluster can be:

the on-demand request is not triggered
the requested cluster is not found
the request timed out

In the previous situation, the log level of failure reason 2/3 is 'debug', which is too low that we can't distinguish the reason via log. This change increases the log level so that user can troubleshoot the issue directly.

Commit Message: odcds: log abnormal message in higher level
Additional Description:
Risk Level: Low, only log level changed
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

/retest


[🐞] useTask$ + isServer doesn't strip/noop like useServerMount$

Which component is affected?
Qwik Runtime
Describe the bug
Currently useTask$ is slated to replace the other lifecycle hooks including useServerMount$. However, useServerMount$ currently strips/noops from the client bundles so server-side code is not sent to the client or available to the public in general.
However, useTask$ + isServer does not, as in the following code:
useTask$(() => {
  if (isServer) {
    // this is included in the client bundles
    console.log("SUPERSECRET123");
  }
});

The contents of the above if statement are included in the client bundles and ideally should not be.
As a bonus, it would also be nice for code aesthetic and reducing indentation if the inverted early return case could be stripped as well, nice DX though not required if it poses too much extra work:
useTask$(() => {
  if (!isServer) return;
  // everything in this block would be stripped
  console.log("SUPERSECRET123");
});

Reproduction
https://github.com/jordanw66/usetask-noop
Steps to reproduce

Clone repo.
Note respective strings inside src/routes/index.tsx.
Build.
Search on dist folder for strings.
The useServerMount$ string is correctly stripped, but the other one is present.

System Info
System:
    OS: Linux 6.1 Gentoo Linux
    CPU: (8) x64 Intel(R) Core(TM) i7-9700K CPU @ 3.60GHz
    Memory: 41.24 GB / 62.73 GB
    Container: Yes
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 16.17.1 - ~/.volta/tools/image/node/16.17.1/bin/node
    npm: 8.19.2 - ~/.volta/tools/image/npm/8.19.2/bin/npm
  npmPackages:
    @builder.io/qwik: 0.16.1 => 0.16.1
    @builder.io/qwik-city: 0.0.128 => 0.0.128

Additional Information
No response

useServerMount$() is deprecated in theory, but not in practice, it should still be there! agreed on this problem is very important. going to look with highest priority.
However what's your use case for useServerMount$() today?

@manucorporat if useServerMount$() is deprecated but still available, doesn't that mean at some point in later releases it would be removed? If not what's the plan for useServerMount$()

Fixed! https://github.com/BuilderIO/qwik/pull/2527


Fixes the weird placeholder strings appearing when language was changed

Fixes #2507
A brief description of what changes your PR introduces. The better this description, the quicker we can review, exchange feedback, and merge!
After changing language the placeholder was displaying weird strings and not the cities suggestion you added it was happening because of useCallback function that starts running and do not terminate the priveous instance of useCallback I added language dependency to useCallback and function will terminate whenever the language state changes.
Fixes #2507
✔ Compiles and passes lint tests
✔ Properly formatted
✔ Tested on desktop
✔ Tested on phone
Before Fixing Screenshot :

Initially wrong strings
After Fixing Screenshots

:

lgtm :)

OLD PR  #2327  already fixes issue and is better solution than this.


Update release-notes.md

#sign-off

#sign-off

@GennadNY
This pull request was opened in the public repo. PMs should work in the private repo, per the Microsoft Docs contributor guide. Would you make this content update in the private repo instead? Thank you! We'll close this PR.


HMSPROV-155 - trigger aws reservation from UI

I checked with adding hard-coded pubkey id and image id, and it works :)
should we merge it or do we want to wait for the pubkey page to be implemented?
What are we going to do regarding the image_id? how do we get it?

What are we going to do regarding the image_id? how do we get it?

In our sample page, we are hardcoding it already I believe, in real world scenario, we should be getting it as a prop to the component from Image builder on component initialization, so that should be fine just by exposing that prop as we do.

should we merge it or do we want to wait for the pubkey page to be implemented?

I would actually love if we have at least the select first, before merging it :thinking:
But I'd not block on it per se, so if we cant do the select dropdown first, I am fine merging without it :)

I'm closing this in favor of https://github.com/RHEnVision/provisioning-frontend/pull/57, the requests will be handle there with a progress screen


dateFilter cannot handle Date objects in Safari

dateFilter outputs NaN for date components when passed a js Date object.
template:
MM/dd/yyyy: {{dateObj |date:'MM/dd/yyyy'}}
output:
MM/dd/yyyy: NaN/NaN/0NaN
more detailed example:
http://plnkr.co/edit/1AcK4KcKiud7EK1k8Gd9
See output in iOS:
http://tinyurl.com/angular-date-ios
Failure occurs in Safari & Chrome in iOS 6.1.4
It also fails when passed a string in osx/Safari Version 5.1.9 (6534.59.8)

Thank you revolunet, I ran into this problem recently without knowing that safari had this issue.


Blank screen after deleting a scale color

Describe the bug
When I select a scale and delete a color the page goes blank
To Reproduce
Steps to reproduce the behavior:

Go to prism
Create a new palette or go to an existing palette
Select an existing scale or add a new sacle
On the right panel select delete color for the scale (screenshot below)

Expected behavior
Not exactly sure but the page probably shouldn't go blank
Screenshots

Environment
Chrome Version 102.0.5005.115 (Official Build) (x86_64)
Mac OS Catalina - Version 10.15.7
Additional context
Posted a video here: https://twitter.com/DamiDina/status/1539291589847171073

Looks like it only breaks when deleting the last color in the scale. @pavelkeyzik I think this is related to the bug your reported and fixed: https://github.com/primer/prism/issues/10
@pavelkeyzik Could you disable the "Delete color" button if the color is the last color in the scale?

@damidina The issue should be resolved. You can try to reproduce it again. If you're happy with the result, you can close the issue. If not, just let us know 😊


DO NOT MERGE: QualifiedIdentifier long running branch

https://github.com/microbiomedata/nmdc-schema/issues/1752

DO NOT MERGE INTO ANY MAIN BRANCH OF ANY REPO

I just had my 1:1 with @cmungall. Chris wanted to know more about the use cases for which this PR might be a solution. Chris, can you please say more about that?

@turbomam is this still relevant in light of re-IDing?


Add mutation for bulk delete order lines

Part of  #3927
Pull Request Checklist

[x] Privileged views and APIs are guarded by proper permission checks.
[x] All visible strings are translated with proper context.
[x] All data-formatting is locale-aware (dates, numbers, and so on).
[x] Database queries are optimized and the number of queries is constant.
[x] Database migration files are up to date.
[x] The changes are tested.
[x] The code is documented (docstrings, project documentation).
[x] GraphQL schema and type definitions are up to date.
[x] Changes are mentioned in the changelog.

I am putting this on hold, untill I finish #3891

@NyanKiyoshi, @akjanik Are we ready to merge it?

What about other bulk actions that you wanted to add @akjanik? Are we going to do them in separate PR (which is perfectly fine, just asking out of curiosity to know what's our plan)? Also, I don't know if you have it on your list, but we'll need bulkCancelOrder mutation to be able to cancel multiple orders (as an open, non-draft order cannot be deleted).

@maarcingebala To avoid issues with large PRs, I decided to divide work into small ones. ATM I am working on BulkPublishPages, but after that, I may focus on this bulk cancel mutation. If more missing mutations comes to your mind, feel free to mention them in #3927


Support no_std

Fixes #159.

r? @dnaq

Thanks. Will review and merge as soon as I have time.
On Fri, 5 May 2017 at 23:45, Keegan McAllister<EMAIL_ADDRESS>wrote:

r? @dnaq https://github.com/dnaq
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dnaq/sodiumoxide/pull/163#issuecomment-299583142, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABPVBSfoB-QYycS5JnRwApFNczu0Q2C6ks5r25hkgaJpZM4NO3yT
.

@dnaq: poke?

@dnaq: I need this for an important project. Are you actually interested in the patch or should I maintain my own fork of sodiumoxide?

Hopefully this weekend

Just tried to merge, but #[feature] isn't supported in stable, only in nightly. I'm a bit wary of adding features that only build in nightly. Do you know when these features will land in stable?
I'm not at all opposed to you vendoring your dependencies if need be, and I'm grateful for your patch, but I think compatibility with stable rust is quite important. The only exception so far has been to run benchmarks, which isn't in any way core functionality.
A solution might be to only build the non-allocating primitives when using no_std, then you wouldn't need collections and alloc. Would that work for you, and if so are you interested in amending your patch?

I don't see the problem. Many crates have optional features that require nightly. Every piece of code in sodiumoxide today still builds on stable. CI will confirm this. Users who don't care about no_std should see no difference at all; if that's not the case, I consider it a bug and will endeavour to fix it.
I do have the concern that we're not testing the no_std option in CI. If you want me to fix that before this PR lands, I can try to do that.
Someone did tell me that it's better to have a Cargo feature "std" that is on by default. (Cargo features should never remove functionality, and that could concievably happen with no_std, although it's not the case in the initial patch.) So I'll make that change when I get back from the woods on Sunday.

A solution might be to only build the non-allocating primitives when using no_std, then you wouldn't need collections and alloc. Would that work for you, and if so are you interested in amending your patch?

Doesn't help; no_std itself is unstable. Besides, it's a lot more work and probably not necessary for my use case.

I'm not at all opposed to you vendoring your dependencies if need be

I am very hesitant to mantain my own patches on top of a cryptography library. We all know how well that went for Debian.

Doesn't help; no_std itself is unstable.

I believe that it's possible to create #![no_std] libraries in rust, it's just binaries that are unstable at the moment. The book seems to agree with me. On the other hand anyone who wants to actually use sodiumoxide without std would have to be running nightly at the moment, so alloc and friends might not be a blocker.

I do have the concern that we're not testing the no_std option in CI. If you want me to fix that before this PR lands, I can try to do that.

This is one of my concerns, and any work you could put towards making the tests run automatically would really help.

I am very hesitant to mantain my own patches on top of a cryptography library. We all know how well that went for Debian.

That's a quite valid view. The reason that I have been holding off on a lot of merges lately is that I never merge anything into sodiumoxide without reviewing it and testing it. Lately I've been very short on time, but I've been taking the view that a stable and thoroughly vetted cryptographic library is more important for the rust ecosystem than a library with all latest bells and whistles. I know these might sound like big words for what is essentially a bindings library, but there are a lot of cryptographic libraries with weaknesses and weird behaviour, and I would like to avoid sodiumoxide becoming one.
That being said, embedded systems have been a target from the beginning, and #![no_std] is work towards that goal, so I would really like to merge your work when the CI-bit has been solved. Your patch is pretty small, and I can hopefully merge it by next weekend.

Ah, I didn't know that no_std is stable for libraries! I didn't get an
"unused #[feature]" warning; that might be a bug in rustc.
I totally agree about being careful. A lot of danger can creep in at the
API level even if the underlying primitives are sound. In this case though,
there is no API change.
So I'll switch to #[cfg(not(feature = "std"))], and also work on CI
integration. Should be ready in a day or two. Thanks!
On May 21, 2017 11:40 AM, "dnaq"<EMAIL_ADDRESS>wrote:
Doesn't help; no_std itself is unstable.
I believe that it's possible to create #![no_std] libraries in rust, it's
just binaries that are unstable at the moment. The book
https://doc.rust-lang.org/book/no-stdlib.html seems to agree with me. On
the other hand anyone who wants to actually use sodiumoxide without std
would have to be running nightly at the moment, so alloc and friends might
not be a blocker.
I do have the concern that we're not testing the no_std option in CI. If
you want me to fix that before this PR lands, I can try to do that.
This is one of my concerns, and any work you could put towards making the
tests run automatically would really help.
I am very hesitant to mantain my own patches on top of a cryptography
library. We all know how well that went for Debian.
That's a quite valid view. The reason that I have been holding off on a lot
of merges lately is that I never merge anything into sodiumoxide without
reviewing it and testing it. Lately I've been very short on time, but I've
been taking the view that a stable and thoroughly vetted cryptographic
library is more important for the rust ecosystem than a library with all
latest bells and whistles. I know these might sound like big words for what
is essentially a bindings library, but there are a lot of cryptographic
libraries with weaknesses and weird behaviour, and I would like to avoid
sodiumoxide becoming one.
That being said, embedded systems have been a target from the beginning,
and #![no_std] is work towards that goal, so I would really like to merge
your work when the CI-bit has been solved. Your patch is pretty small, and
I can hopefully merge it by next weekend.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dnaq/sodiumoxide/pull/163#issuecomment-302954662, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAbKRZtNDcDunZhVoPrenpcmu6gXrBbLks5r8IUkgaJpZM4NO3yT
.

@dnaq: OK I think this is ready, let me know if you have any remaining comments.

Sweet! Thanks. Now if only I hadn't bricked my Cortex-M dev board...


Chrome version?

Can you tell me any more about what version of Chrome we can expect to be installed?
We are locked to tag 6.0.8 of ng-cli-e2e because of issues upgrading to node 9
This caused an issue because of the update of ChromeDriver to 2.42 yesterday, which requires Chrome to be v68-70
The 6.0.8 tag of this image is downloading Chome 67.0.3396.79 (currently)
But when I look at the dockerfile, it is pulling:
https://dl.google.com/linux/direct/google-chrome-stable_current_amd64.deb
... which I would assume would download the newest version available.
Do you have any insight on why the 6.0.8 tag is pulling an older version?
(What my boss really wants is for the versions of both Chrome and ChromeDriver to be locked at some specific version to avoid these kinds of incidents.)

This is indeed an unfortunate situation:
During image build the browser is installed, f.e. chrome latest.
But the webdriver is installed during npm-install which happens at a later point in time. This is the reason that we failed to provide a working alpine variant of this image, since we did not accomplish to bundle the webdriver compiled with musl-libc and prevent a later installation of a different version through Angular CLI.
So the tagged image versions will not change, it will ship the chrome version that was current at the point in time the tag was build, but your project moves in time and uses newer dependencies that do not work with the old browser.
There are not much choices how to solve this:

you can update the image version (let me know what problems node 9 causes for you and I will look into it)
you can use the dockerfile and build your own images using current chrome but older angular cli/node

Actually, this should be fine from my side.  If the version of Chrome is already baked into the layer, we can just make our current approach permanent:
docker ... trion/ng-cli-e2e:6.0.8 \
    ./node_modules/webdriver-manager/bin/webdriver-manager update --versions.chrome 2.41 && \
    ng e2e -a="${APP}"

...which completely locks versions of both Chrome and ChromeDriver as desired.

Does that approach work for you? When I last checked I could not prevent angular-cli to update to the latest webdriver.
If it works it would be cool, if you could make a PR against the README to document that for other users as well.

Sure
https://github.com/trion-development/docker-ng-cli-e2e/pull/6


Stuck Loading On Nuxt

Trying to integrate this into my vue/nuxt project and was able to get it in as a plugin.
But not matter what m3u8 playlists I try, its just spinning.

No errors that I am seeing preventing it from playing. I also do see the response in the network tab loading the .m3u8
Any idea?

I believe it has to do with the sample rate.
I am testing with BunnyCDN. So im sure the sample rate would stay the same between encoding.

I copied your demo html into the page it loaded fine. It seems I must have waveforms generated. Ill further test that.

Hi!
Thanks for adding a link to your audio. That's helpful.
Your m3u8 contains an index to further nested m3u8 files (e.g. 640x360/video.m3u8) which file then contains ts (video*.ts) files. I'm afraid this nested structure cannot be consumed by the Stem Player.
However, I hope that this may help you generate audio in a format that can be played back.
The result should look something like this
Hope that helps...

https://d178eqdhz8mr00.cloudfront.net/testsong/plumb.m3u8
https://d178eqdhz8mr00.cloudfront.net/testsong/plumbtest.m3u8
The player doesn't spin forever on load, but it doesnt play the file. I can see the correct time loaded, but it spins after hitting play.

I see.. yours references .mp3 instead of .ts

Tried with mp3 and same thing at least for myself.
<soundws-stem label="Guitar A" src="https://d178eqdhz8mr00.cloudfront.net/testsong/finaltest/file.m3u8" waveform="https://d178eqdhz8mr00.cloudfront.net/testsong/maintest.json"></soundws-stem>

Deploying this now to try instead of manually in elasticoder converting.
https://github.com/stemplayer-js/api

Cool..
I'll send an invite through slack so we can assist

For reference, you can try this to decode audio: https://github.com/stemplayer-js/docker-process-audio
And this to generate waveforms: https://github.com/stemplayer-js/docker-generate-waveforms


HOSTS addition to block ads

Please add:
<IP_ADDRESS> tracking.scenepass.com
<IP_ADDRESS> cdn.creativesumo.com
<IP_ADDRESS> cdn3.bestcontenttrade.top
<IP_ADDRESS> cdn2.bestcontenttrade.top
<IP_ADDRESS> cdn1.bestcontenttrade.top
<IP_ADDRESS> cdn.bestcontenttrade.top
<IP_ADDRESS> a.bestcontentfood.top
<IP_ADDRESS> b.bestcontentfood.top
<IP_ADDRESS> a.bestcontenttrade.top
<IP_ADDRESS> b.bestcontenttrade.top
This blocks everything on motherless.com when the site forces you to disable adblock, at least in Canada. The videos will work and ads will display "cannot connect" instead.

Happy fapping.

Ping @sinfonietta, this one's yours, I think.
Closing.


Enable database access for API.

The production API deployed and ran successfully on Heroku, so we can consider #11 fixed!


Create a page for a single EventGroup

[x] #61
[x] #62
[x] #63
[x] #64
[x] #65
[x] #66

This issue is completed.
However, there are still some problems with displaying on mobile devices.


When will the Search Guard and Search Guard SSL for Elasticsearch 6.7 be available?

Hi,
Elasticsearch 6.7 came put this week and we wanted to check to see when will the Search Guard and Search Guard SSL that is compatible with Elasticsearch 6.7 be available for us to download?
Thank you very much.

Found another post. I am good now..


Setup simple continuous test for grpc build on Windows

Figure out a simple way how to continously verify that grpc is building alright on Windows. Nothing complicated yet, any trivial solution will do.

Nico has a jenkins windows instance running on GCE.


Update betwixt to 1.6.1

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

Thank you for the contribution. It was merged directly as 9ce82261ea977d4507d9685744e9ca5a8361fdca. Your contribution is still credited to you.


Compiler Error only when archiving

I'm trying to archive my app, but for some reason I'm getting this errors:

Happens when I'm archiving or when just building using Any iOS Device.
Tried deleting my derived folder, uninstall Cocoapods and reinstall all the pods.
Nothing works.
Does any one have an idea?

Hi, can you still reproduce this on the current master?
I have tried this and it worked for me. If you can reproduce this, please reopen this issue and attach an Xcode project that showcases this behavior. I can imagine it having to do with how you import SwiftUI, but without seeing it for myself I cannot give you any further support.


Error Building on Mac M1

I ran into errors building on Mac M1 where owl can't be installed. The work-around exists in this PR but it hasn't been merged yet. So until then, running this to install owl will allow ProbZelus to build successfully:
opam pin -n git+https://github.com/mseri/owl.git#arm64 --with-version=1.1.0
PKG_CONFIG_PATH="/opt/homebrew/opt/openblas/lib/pkgconfig" opam install owl.1.1.0

Thanks a lot for documenting this.
Unfortunately owl is pretty unstable (even on Intel).
Hopefully this will be resolved in the next versions.


Fix prediction script

One of the last TensorFlow updates broke the ./ask.sh script. Probably it has to be written from scratch.

Hi @mommi84 I think I also just experienced this as well.
The output of python -m nmt.nmt  --vocab_prefix=../$1/vocab...  is <unk> <unk> <unk> <unk> <unk> <unk> <unk> <unk> which I'm assuming is <unknown> <unknown> ... etc
Any ideas what is going wrong here?
From what I can see this is a blocker to using NSpM is that correct?
Thanks for any assistance. I would like to try and fix this.

Hi,
Has there been a solution found?


minimal seed check failure repro

there is another flake. looking into it

@maxtaco done. I needed to add the user to the parent team as well (previously was only in subteam).

CI passing now


Added description about mock for callee

Description

Closes #305
Types of changes

[ ] Bug fix (changes which fixes an issue)
[ ] New feature (changes which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] ETC (build, ci, docs, perf, refactor, style, test)

Checklist

[ ] I followed the contributing guidelines.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] The PR title and commits are followed conventional commit form.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.3 out of 5 committers have signed the CLA.:white_check_mark: loloicci:white_check_mark: da1suk8:white_check_mark: dudong2:x: Jiyong Ha:x: kokeshiM0chiJiyong Ha seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Search box After searching for keywords, move the mouse up and down in the file area of the directory bar to search for overlapping text in the display boxes

Search box After searching for keywords, move the mouse up and down in the file area of the directory bar to search for overlapping text in the display boxes

And software dragging Windows in this state will be particularly hard

The test platform was Windows 10


[SPARK-33887][SQL] Allow insert overwrite same table with static partition if using dynamic partition overwrite mode

What changes were proposed in this pull request?
For dynamic partition overwrite, we do not delete partition directories ahead, We write to staging directories and move to final partition directories after writing job is done. So it is ok to have outputPath try to overwrite inputpath. In view of this, i think inserting static partition in dynamic partition overwrite mode is also supported, i adjust the judging conditions of dynamicPartitionOverwrite to allow this.

Why are the changes needed?
In some cases, we need overwrite static partition using itself.

Does this PR introduce any user-facing change?
No

How was this patch tested?
UT

Can one of the admins verify this patch?

ping @viirya


upgrade go to 1.19

Signed-off-by: ldpliu<EMAIL_ADDRESS>https://github.com/stolostron/backlog/issues/26491

/lgtm


Delay while typing text.

@mrazadar  while i type text i can see that it has a delay before render symbol

You can try a react-native-controlled-mentions that is not rich in configuration yet but enough stable for use in projects.
https://www.npmjs.com/package/react-native-controlled-mentions


Is it possible for Runeinfo to listen to webhooks?

Just a question. I want to know if I can use Mee6's timer plugin and trigger the Runeinfo bot to respond if its a specific bot command. See attached: https://i.imgur.com/sY4FpxN.png
I asked in the Mee6 Community discord and got this response. https://i.imgur.com/A4E6P1n.png
Is there any way to get this feature working by enabling the Runeinfo bot to listen to webhooks?

no


Paper Prototypes

Create Paper prototypes.
Need to have 2 designs
5.3.2, 6 in book
Assigned to : Shabnam Mustafa

Shabnam finished with the help of Jesus Perez


[help] Response as utf8

Hello, i believe it is not a problem with request-promise, but, here we go haha.
I have a nodejs crawler using request-promise and my problem is about the html response...
I get some elements in the body and then i console.log they, but, i get this character �
instead any special character.
So... is there a way in request-promise to set html response as utf8 ?
here is my code
https://github.com/mateusduraes/crawler-br-news/blob/master/index.js
If something is wrong, sorry me, im new in gh.
Thank you.

Hi @mateusduraes , request-promise is built on top of request and thus the documentation of request applies as well. It documents a encoding option that you can set. However, it is utf8 by default.
By looking at your code I don't think the encoding is the issue. It's rather that you lost the implicit binding of the log function to the console object. (If you don't understand what I mean please google implicit vs. explicit binding in javascript.)
The solution might be:
  requestPromise('http://www.em.com.br/')
    .then(getNews)
    .then(prettyjson.render)
-   .then(console.log)
+   .then((result) => {
+     console.log(result)
+   })

Hope this helps!

May I close this issue or is there anything I can help you with?

Hello bro, thanks for the suggestion but it doesn't worked.
By the way i still trying to find the solution, and yes, you can close the issue.
Thanks for the attention!


Allow omitting unit

Hello and good day. I'm not really sure how to contribute to this project, if I could just open a PR or not, so if this was not OK please let me know and lets close it.
The thing is, I saw an open issue where it was requestes to allow ommiting units, just like the regular sleep command, where it defaults to seconds when no unit is passed. So I did kind of fix to that. I'm not a go programmer, I recently started learning it, so I don't know if my implementation was a good idea or not. I mutated the state of arg[0], this sounds fishy, but I looked like it wasn't used anywhere else.
Anyways, if this is a valid feature (ommiting units), please let me know. And if it is the case, I think it could be stated in the help/manual/readme that it defaults to seconds.
Solves #34

hey, yeah this looks good! thanks
wanna add to the help as well?

I'm not sure how to add the help :sweat_smile: But I'd be glad to add it. The actual help message apparently is "timer is like sleep, but with progress report", so I'm not sure how to insert the information about defaulting to seconds, since there is not really a mention to units.
I added a line to the README, but I also don't know how the man pages are generated, I think it would be nice to mention the default unit as seconds in the man page as well.

man pages are auto-generated, this looks good, thanks!


Why so dark?

I love the sidebar icons but theme is too dark for me, can i customize it by my self to make it more lighter like in image below?
its itg flat theme with spacegray dark color scheme

Yeah. you can do whatever you want :)


An issue in model conversion

Hello
How are you?
Thanks for contributing this project.
I met an issue when converting the vggface2 model, which is based on ResNet50 model.

I tried this conversion on both tf1.x  and tf2.x but failed.
I send the source Keras model.
https://we.tl/t-MjAHF2SPiK
Please fix the issue asap.
Thanks

It should work since fac474a, version 0.1.1 on pip.

Thanks


stable issue or memory error for

hello ,when I using 1.06 version to do bwa-meme some how like that
" bwa-meme mem  -M -a -t 8 $2 $3 $4 |mbuffer -m 8G |samtools sort --reference $2 -o $5.cram -O CRAM -@ 8 -"
but it turns out exit without any error report?
how could it be?

Hi,
Can you elaborate more in the log?
It would be helpful if you attach the log output.
In my local environment I get something like this.

Great, if you have any more problems related to this, please reopen this issue.


npx prisma will use a Prisma server version: 1.32.0-beta

Describe the bug
When using npx to login, init and deploy (I should have installed it though, I know) it uses the version 1.32.0-beta for the server, and when using the cli after installing it globally, it says this:
Warning: Your Prisma server and Prisma CLI are currently out of sync. They should be on the same minor version.
Prisma CLI version: prisma/1.34.3 (darwin-x64) node-v8.11.3
Prisma server version: 1.32.0-beta
For further information, please read: http://bit.ly/prisma-cli-server-sync
To Reproduce
Steps to reproduce the behavior:

npx prisma login
npx prisma init
npm i -g prisma
npx prisma deploy
prisma deploy

Expected behavior
To have the same version on npm i -g prisma as on npx prisma (maybe?)
Screenshots
If applicable, add screenshots to help explain your problem.
Versions (please complete the following information):

Prisma CLI version: prisma/1.34.3 (darwin-x64) node-v8.11.3
Prisma server version: 1.32.0-beta
npm: 6.5.0

Additional context
Add any other context about the problem here.

Unable to reproduce this at all. Can you please try again after cleaning your npm cache(npm cache clean).
if you still able to reproduce this, it is a bug in npx not in prisma.

`➜  ~ which prisma
/Users/rjr/.nvm/versions/node/v8.15.1/bin/prisma
➜  ~ nvm use 10.16
Now using node v10.16.0 (npm v6.9.0)
➜  ~ which prisma
/Users/rjr/.nvm/versions/node/v10.16.0/bin/prisma
➜  ~ prisma -v
Prisma CLI version: prisma/1.34.5 (darwin-x64) node-v10.16.0
╭──────────────────────────────────────╮
│                                      │
│   Update available 1.34.0 → 1.34.5   │
│    Run npm i -g prisma to update     │
│                                      │
╰──────────────────────────────────────╯
`


toolTip:tooltipIsOpen

Can't get the value of tooltipIsOpen flag. Add please eventEmitter with new state of toolTip.

i think it is implemented in such a way that you can progromatically set the isOpen value from the outside, it isn't designed to be an event.
That being said, if that is the purpose then it isn't working that way either

It looks like isOpen was not implemented (yet). I'll create a pull request once the feat-rc5 branch is merged into main.

How was it? I can't get it to work as well.

Fixed with #939.


question: how to install mistune-contrib?

Just install mistune through pip. Works like a charm.
Now, I need a table of content, how to install and use mistune-contrib?

pip install mistune-contrib

thanks.


Rename ellipsis button

The naming of the feature is too explicit on the appearance (that it displays an ellipsis). Figure out better naming (for the themable part etc). Maybe "overflow-button" or something...

Also, switch the implementation to use the ellipsis character in the theme instead of three dots "..." in the template

Let's add separate comments for the options below and vote with 👍 or 👎

Option 1: "overflow-button"

Option 2: "more-button"

Option 3: "show-all-button"


Compiled tracking module is not detected

I've compiled OpenCV for Windows with the tracking module from opencv-contrib. This has produced opencv_world4100.dll.
Usually, placing this file alongside the executable in target/release would be all the program needs to compile but for whatever reason it's not able to detect the tracking module.
Is there anything specific I need to do to enable this to work? I can't figure out what I'm missing.
I'm using the latest Rust and OpenCV.
Thank you.
build.log

Yes, that path was still set to use an old installation. Thank you for the help and God bless package managers


B #4148: Added missing showback xsd

Signed-off-by: Kristian Feldsam<EMAIL_ADDRESS>

Why this is still not merged? What can I do to help with merging this?

Sorry for the delay... some other PR's where in queue. Thanks for your contribution!


Cinnamon tree does not grow to stage 1 with bonemeal

But the cocoa tree does :)

cinnamon trees actually grow all the way on one go. they are sort of halfway between fruit trees and regular trees


Timeline for next release?

Would be nice to have a new release that supports Tmux 2.4. It looks like support is already in master. Is there anything in particular holding up the next release?

Good point. I’ll release a new version shortly.

Done: v2.3.0


Ability to hide elements from federated schema but preserve in internal schema

Hi,
I haven't found a way to hide fields from federation, i.e. exclude them from service { sdl }. Coupled with Strawberry validating that types must have "one or more fields", this means that my outward facing Query has to have at least one field from every backing service I want to use Strawberry to serve. Currently I'm making void: str @deprecated fields, which are rather unsightly.
Thanks, Dan.

Upvote & Fund

We're using Polar.sh so you can upvote and help fund this issue.
We receive the funding once the issue is completed & confirmed by you.
Thank you in advance for helping prioritize & fund our backlog.

@djmcgreal-cc is this something that's supported by federation itself?

Hi Patrick,
Yes, a backing service decides what to include by adding it to the service { sdl } }.

Hi Patrick, Yes, a backing service decides what to include by adding it to the service { sdl } }.

Can you show me an example of this in a code first framework? I don't think I have ever seen it :)

I can't help you there I'm afraid, I don't have enough experience of other frameworks. I don't think it's related to choice of paradigm though. Apollo's buildFederatedSchema method doesn't require a Query definition, so it's possible to contribute to a subgraph that doesn't extend from the Query root.
Other than an ability to hide fields from federation, can you think of another way to add a subgraph via Strawberry that doesn't also introduce redundant fields into the Query type?
Thanks, Dan.

This should be supported by using @inaccessible https://www.apollographql.com/docs/federation/federated-types/federated-directives/#inaccessible


NumberFormatException in routing with Locations

I have the following test case in LocationsTest.kt
    @location("/f/{id}/{name}/") class FLocation(val id: Int, val name: String)
    @location("/f/{id}/{name}/{offset}/") class FOLocation(val id: Int, val name: String, val offset: Int)
    @location("/f/m/foo") class TestLocation(val foo: String?)

    @Test fun `precendece routing`() = withLocationsApplication {
        application.routing {
            get<FLocation> {
                call.respond("1: plain")
            }
            get<FOLocation> {
                call.respond("2: offset")
            }
            get<TestLocation> {
                call.respond("3: test")
            }
        }
        urlShouldBeHandled("/f/m/foo", "3: test")
    }

Produces a java.lang.NumberFormatException: For input string: "m". I would have expected this to pass. Is it a bug or why is this ?

That's because foo, even if nullable, doesn't have default value and hence should be a parameter in query, like /f/m/foo?foo=bar. But if you add = null to foo parameter it still will loose to /f/{id}/{name} because match quality of missing parameter is low and beats better match quality of /m exact match to {id} parameter.
Any suggestions are welcome.

Ah yes. I forgot for a moment that = null isnt implied by the nullable type.
Thanks

Having thought about this a bit more it seems I would expect two things about this case:

If the {id} paramter is not an Int then the url should not match that route - ie. same as if the route had not been installed. In a regex based router it would be something like to route("/f/{\d+}/{[^/]+}/"). Parameters not matching is not an exceptional case either so it shouldnt result in exceptions that get printed to the console (might be CallLogging but even then).
A constant path segment should win over a dynamic one simply because it is more specific and specific should win over more general rules.

I agree that type-check could be there, but it's simply not designed how, e.g. not sure if it should miss the routing with a 404 given /f/x/something with the routing above. Or should issue some other 4xx error.
As for constant vs non-constant, it's complicated. As of now, we check the quality of the whole route trie path, not just immediate child, and you do have parameter in the /f/m/foo route, it's just happens to be non-path parameter, and it's missing. Missing param qualifies lower than existing param, hence the result.

Consider routing like this:
routing {
   route("{id}") {
      accept("text/html") {
        get { … }
      }
   }
   route("{name}") {
      accept("application/json") {
        get { … }
      }
   }
}


How to get error details when an error occurs ?

Hi,  can i get the real error details when an error which like paper jam occurs ?
scanner :  Canon DR-6030C TWAIN  and  HUAGOSCAN G400 TWAIN
LRESULT CALLBACK TwainCallbackProc(WPARAM wParam, LPARAM lParam, LONG_PTR UserData)
For example，I set the callback as above by calling DTWAIN_SetCallback ,  I want to get the lParam  'TWAIN_ERR_FEEDERJAMMED'  when paper jam， but  wParam is 1038, lParam is -1109 and -1023  which is not the real error code.
What can I do to get the actual error code ?  please.

  See also DTWAIN_IsDeviceEventSupported
Also, given the above, the DTWAIN_TN_DEVICEEVENT is the notification you should be handling.  However, this will only be successful if:
a) the device supports device events,
b) the device properly detects a paper jam and generates the correct error message
c) sends in the TWAIN message pump a MSG_DEVICEEVENT message that DTWAIN will process, and thus relay this in a
    DTWAIN_TN_DEVICEEVENT notification.

Unfortunately I do not have this scanner to verify if it can actually return the error codes that you mentioned.   This is basically what you need to verify.

Closing issue


feat(check:policy): Add pnpm repo policies

First set of repo policies for when using pnpm.
For any pnpm-lock.yaml files:

Always require pnpm-workspace.yaml.  This is needed because we have one in the root and it needs one at the package level to override that.
Disallow package-lock.json in the same directory
Require the "preinstall" script in the package.json.

/azp run Build - build-tools


[Public Catalog] Hard code the install path for the code stream quickstart. ETA Oct 18

In order to support the codestream quickstart and install path we'll need to hardcode the button once the quickstart is added to the repository to direct users to the install nerdlet being built by devex.
note: There will be no installPlan for this "quickstart" but it will live in the quickstart repo.
Acceptance Criteria

[ ] are there other cases where we'd be doing this? -- @jpvajda
[ ] Obtain the direct URL to the nerdlet for install of the code stream quickstart. -- #help-dev-experience
[ ] when the quickstart details page is added, the install quickstart button should have the URL hardcoded to point the user the install nerdlet

@jpvajda we actually already do this for other quickstarts so the pattern is already provided for here ⚠Internal Link

I confirmed there is s specific time we need to flip this on for this GA, it's 8am PST on Oct 21st. cc @aswanson-nr. Perhaps @rudouglas can take this one given his timezone.


Link to the binder-examples org in the docs

I think we should have a link to binder-examples somewhere early on in the documentation:
https://github.com/binder-examples
This is a collection of example repositories that are all short and to-the-point to show off functionality. I don't think this would be a proper "gallery", it's more about showing how to compose various files in the repository.
Thoughts?

partially addressed by #29

I think this is done now that there's the sample repositories section


Bubble sort python

Add bubble sort (python)
Issue #41
FileName BubbleSort.py

Seems correct! The branch needs to be rebased with the python branch here, otherwise a lot of different and unwanted changes can happen. Can you once do that?

Seems correct! The branch needs to be rebased with the python branch here, otherwise a lot of different and unwanted changes can happen. Can you once do that?

sure

done

Seems correct! The branch needs to be rebased with the python branch here, otherwise a lot of different and unwanted changes can happen. Can you once do that?

sure

rebase done

The original branch seems to have been based off the master branch, which is C++ implementations.
My suggestions:

Create a new branch in your repo, based on the current python branch of ekdnam\algorithms.
Here, add bubblesort.py (what you have done currently here) in the reqd directory.
Create a new PR to the python branch.
Done!


Add design document for HIP-745

Problem
HIP-745 introduces a new way to create a transaction without erroring if all the fields are not provided. The design for this features needs to be documented for each SDK before it is planned for implementation.
Solution
Document the design for HIP-745
Alternatives
No response

Added issue to each SDK repo


[WIP] Call appyBlockDefaults from addBlock/insertBlock

@sneridagh @davisagli
Here's the proposal: we add a new .initialValue function to the block configuration, which can be called like this:
    value = blocksConfig[type].initialValue({
      id,
      value,
      formData: newFormData,
    });

Thoughs on naming and function signature?

@tiberiuichim btw, each time you press the "commit suggestion" button, it creates a new commit that triggers all the CI workflows. You can avoid this by going to the "Files changed" tab, where you have the option to "Add suggestion to batch" and then make a single commit with multiple suggestions included.

@tiberiuichim could you please add the changelog if it's ready? I would merge it right away!

@tiberiuichim ping!


Fix non-terminating fromPParams

Issue Number
#1657
Overview

[x] I made fromPParams not loop indefinitely and break the tip-following.

Comments
The node tip now updates correctly:
$ cardano-wallet-shelley network information
Ok.
{
    "network_tip": {
        "epoch_number": 22,
        "slot_number": 8397
    },
    "node_tip": {
        "height": {
            "quantity": 12738,
            "unit": "block"
        },
        "epoch_number": 22,
        "slot_number": 8360
    },
    "sync_progress": {
        "status": "ready"
    },
    "next_epoch": {
        "epoch_start_time": "2020-05-18T14:15:00Z",
        "epoch_number": 23
    }
}
 $ cardano-wallet-shelley network information
Ok.
{
    "network_tip": {
        "epoch_number": 22,
        "slot_number": 8398
    },
    "node_tip": {
        "height": {
            "quantity": 12739,
            "unit": "block"
        },
        "epoch_number": 22,
        "slot_number": 8398
    },
    "sync_progress": {
        "status": "ready"
    },
    "next_epoch": {
        "epoch_start_time": "2020-05-18T14:15:00Z",
        "epoch_number": 23
    }
}

Could have added a test that fromPParams terminates/doesn't throw, but that seemed a bit too specific. When we add Test.Integration.Scenario.API.Shelley.Wallets I think those scenarios will cover this.

bors r+


Move endinness from type to var

Currently, endianness is associated with the NC_TYPE_INFO_T
object. This is a holdover from HDF5 where type objects could
be cloned to have different endian values. This is inconsistent
with the netcdf API (nc_def_var_endian), which is var based.
Changes:

store endianness with variable
move the endianness in NC_TYPE_INFO_T to put it
into the HDF5 specific structure pointed to
by NC_TYPE_INFO_T.format_type_info field.

Needs some fixes for hdf4

We must also consider what effect this has on HDF5 generated files that use endianness in their types.
That is, if there is a HDF5 file with a compound type which has an endianness set, we can currently still handle that. We need to continue to do so. I can knock up some tests if you wish...


DBZ-7780 NO_DATA snapshot mode validation throw DebeziumException on restarts if snapshot is not completed

closes: https://issues.redhat.com/browse/DBZ-7780

/packit test --labels oracle

@Naros Seems even 5h is not enough for oracle-21-xs. whenSnapshotMultipleTablesAndConnectorRestartsThenOnlyNotAlreadyProcessedTableMustBeProcessed this is taking about 10 min and executed multiple time. I created https://issues.redhat.com/browse/DBZ-7793
If you agree I'll go to merge this.

Applies to main (2.7) and 2.6


feat(testing): Introduce support for the Cypress framework

Description
This PR adds support for the Cypress framework. It's been tested on the latest version of Cypress (v13).
All utilities related to Cypress are exported from the subpath @clerk/testing/cypress.
Also, this PR adds an Error to be thrown if someone tries to import from the package's root path (@clerk/testing).
Usage
Firstly, add your Clerk keys (CLERK_PUBLISHABLE_KEY and CLERK_SECRET_KEY) to your environment variables file (e.g. .env.local or .env.).
You can find these keys in your Clerk Dashboard.
All Cypress related utilities are exported from @clerk/testing/cypress. Make sure that your import paths are correct!
On your cypress.config.ts, you must use the clerkSetup function to set up Clerk for your tests.
Keep in mind that you must pass the Cypress config object to the clerkSetup function and also return the new config object from the setupNodeEvents function.
// cypress.config.ts
import { clerkSetup } from '@clerk/testing/cypress';
import { defineConfig } from 'cypress';

export default defineConfig({
  e2e: {
    setupNodeEvents(on, config) {
      return clerkSetup({ config });
    },
    baseUrl: 'http://localhost:3000', // or your app's URL
  },
});

Then, you can use the setupClerkTestingToken function to bypass bot protection on your tests.
// cypress/e2e/app.cy.ts
import { setupClerkTestingToken } from "@clerk/testing/cypress";

it("sign up", () => {
  setupClerkTestingToken();

  cy.visit('/sign-up');
  ...
});

Checklist

[x] npm test runs as expected.
[x] npm run build runs as expected.
[ ] (If applicable) JSDoc comments have been added or updated for any package exports
[ ] (If applicable) Documentation has been updated

Type of change

[ ] 🐛 Bug fix
[x] 🌟 New feature
[ ] 🔨 Breaking change
[ ] 📖 Refactoring / dependency upgrade / documentation
[ ] other:

!snapshot

Hey @anagstef - the snapshot version command generated the following package versions:

Package
Version

gatsby-plugin-clerk
5.0.0-beta.45

@clerk/testing
1.1.0-snapshot.v641a910

Tip: Use the snippet copy button below to quickly install the required packages.
gatsby-plugin-clerk
npm i<EMAIL_ADDRESS>--save-exact

@clerk/testing
npm i<EMAIL_ADDRESS>--save-exact


Disabling arm64e testing in two tests

I wasn't aware that arm64e was a thing in open-source, but these two
tests are failing on Apple Silicon both on the main branch and on the
rebranch branch, so I'm disabling it for now.
The failing tests are:

ModuleInterface/arm64e-fallback.swift
stdlib/Reflection_objc.swift

rdar://83059989

@swift-ci please test

@swift-ci please smoke test

OSS Swift is supposed to support arm64e only to the extent that it can load an arm64e interface if an ordinary arm64 interface is not available. Fixing that part of the bug in #39315.


reflectx: fork from sqlx and code tidy up

Codecov Report

Merging #8 (9192540) into main (5de04e2) will decrease coverage by 7.52%.
The diff coverage is 74.72%.

@@            Coverage Diff             @@
##             main       #8      +/-   ##
==========================================
- Coverage   83.78%   76.25%   -7.53%
==========================================
  Files           2        3       +1
  Lines          37      219     +182
==========================================
+ Hits           31      167     +136
- Misses          4       39      +35
- Partials        2       13      +11


Fix 1345

Here is a branch I've worked on which should eliminate the #1345 bug and also add docs and refactor code. There were two problems, in ##phi nodes the vreg has more than one "parent" so which one was chooses as parent was random. I made it so the one with the lowest vreg nr always is choosen. The linear-scan step had the same problem so I changed the key from { start end } to { start end vreg } and because vreg nr is unique, it should make it so compilation becomes more deterministic.

Merged, thanks @bjourne !!

@bjourne any idea what this failure is?
USE: project-euler.044

results in this:
==== project-euler:

resource:extra/project-euler/project-euler.factor

 3: USING: definitions io io.files io.pathnames kernel math math.parser
15:     project-euler.041 project-euler.042 project-euler.043 project-euler.044
                                                                               ^
Generic word lookup-base-pointer* does not define a method for the POSTPONE: f class.
Dispatching on object: f

==== project-euler.044:

Generic word lookup-base-pointer* does not define a method for the POSTPONE: f class.
Dispatching on object: f

Strange. I must have missed that.


Driver [expo] not supported

Fresh install, getting this error after installing the package.

You may be installing an old version. 1.x used a class name for the driver, 2.x introduced "expo"


rules to download genbank sourmash databases

In the Snakefile workflow, I use databases that are located on my local compute cluster, meaning I didn't need to download them as part of the workflow. I'm including rules to download them below:
The input paths for the sourmash gather rule would need to change in the Snakefile to reflect the output paths recorded below
rule download_sourmash_db_genbank_bacteria:
    output: "inputs/sourmash_dbs/genbank-2022.03-bacteria-k31.zip"
    threads: 1
    resources:
        mem_mb = 800,
        time_min = 30
    shell:'''
    wget -O {output} https://dweb.link/ipfs/bafybeigkcvizvhe3xzxsuzv3ryf3ogvgvcmms2e5nfk7epl5egts22jyue
    '''

rule download_sourmash_db_genbank_archaea:
    output: "inputs/sourmash_dbs/genbank-2022.03-archaea-k31.zip"
    threads: 1
    resources:
        mem_mb = 800,
        time_min = 30
    shell:'''
    wget -O {output} https://dweb.link/ipfs/bafybeidn6epju7yrdxrktq5wjko2yiwp6nrx3mq37htiuwecm7lffrbcdi
    '''

rule download_sourmash_db_genbank_fungi:
    output: "inputs/sourmash_dbs/genbank-2022.03-fungi-k31.zip"
    threads: 1
    resources:
        mem_mb = 800,
        time_min = 30
    shell:'''
    wget -O {output} https://dweb.link/ipfs/bafybeidhhwvwujkteno5ugwgjy4brhrv5dff2aumifcuew73qolfktdndq
    '''

rule download_sourmash_db_genbank_protozoa:
    output: "inputs/sourmash_dbs/genbank-2022.03-protozoa-k31.zip"
    threads: 1
    resources:
        mem_mb = 800,
        time_min = 30
    shell:'''
    wget -O {output} https://dweb.link/ipfs/bafybeicpxjhfrzem7f34eghbbwm3vglz2njxo72vpqcw7foilfomexsghi
    '''

rule download_sourmash_db_genbank_viral:
    output: "inputs/sourmash_dbs/genbank-2022.03-viral-k31.zip"
    threads: 1
    resources:
        mem_mb = 800,
        time_min = 30
    shell:'''
    wget -O {output} https://dweb.link/ipfs/bafybeibqsldwsztjf66rwvwnb6hamjtsfkmdk5bmfqbzwrod6wwwkqz2ya
    '''

GenBank and GTDB rs207 produced the same gather results (e.g. both missed viruses, no additional species in genbank), so leave workflow as is. Dumping genbank gather rule here for future
rule sourmash_gather_mgx_genbank:
    input:
        sig= "outputs/sourmash_sigs/{sample}.sig",
        db1 = "/group/ctbrowngrp/sourmash-db/genbank-2022.03/genbank-2022.03-bacteria-k31.zip",
        db2 = "/group/ctbrowngrp/sourmash-db/genbank-2022.03/genbank-2022.03-archaea-k31.zip",
        db3 = "/group/ctbrowngrp/sourmash-db/genbank-2022.03/genbank-2022.03-fungi-k31.zip",
        db4 = "/group/ctbrowngrp/sourmash-db/genbank-2022.03/genbank-2022.03-protozoa-k31.zip",
        db5 = "/group/ctbrowngrp/sourmash-db/genbank-2022.03/genbank-2022.03-viral-k31.zip",
    output: "outputs/sourmash_gather/{sample}_k31_scaled2000_genbank.csv"
    conda: "envs/sourmash.yml"
    benchmark: "benchmarks/sourmash_gather_k31_scaled2000_genbank_{sample}.tsv"
    threads: 1
    resources:
        mem_mb = 64000,
        time_min = 480
    shell:'''
    sourmash gather -o {output} -k 31 {input.sig} {input.db1} {input.db2} {input.db3} {input.db4} {input.db5}
    '''


_

_

@resynth1943 I have to close this request without merging, because you requested to add a non-existent domain to this repository: Host porthcounty.org.uk not found: 3(NXDOMAIN) Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for understanding!

It's literally the email dude

Hi. Can you please delete this pull request?


Adding support for schema name

Is your feature request related to a problem? Please describe.
Adding support for schema name.
One of the examples was Snowflake
Currently Hierarchy of Snowflake data looks like
Database Name -> Schema Name -> Table Name.
OM supports
Service Name -> Database Name -> Table Name.
Describe the solution you'd like
Adding support for schema names.

@ayush-shah can you paste the snowflake listing page that you were showing for reference here

cc: @harshach


feat(DeleteItems): Delete a file or entire directory

What this PR does 📖
1. Remove one file or entire directory tree in Storage
a. Removing a file
https://user-images.githubusercontent.com/63157656/221327764-2570e9f3-f914-455c-a90b-e869ab5daf21.mov
b. Removing a entire directory tree
https://user-images.githubusercontent.com/63157656/221327914-b4f6be17-173e-471f-a6bd-bb4479fd8cde.mov

Which issue(s) this PR fixes 🔨

Resolve #

Special notes for reviewers 🗒
Additional comments 🎤

Worked well on my side @lgmarchi really neat!


\mathcal crashes KaTeX

Not sure if this is already reported or expected behaviour: https://github.com/getnikola/nikola/issues/2174#issuecomment-157078262
If there's \mathcal in my math, KaTeX doesn't render and Javscript gives the following error in the console:
Error: KaTeX parse error: Expected 'EOF', got '\mathcal' at position 9:

\mathcal{ABC} works on http://khan.github.io/KaTeX/.  What version of KaTeX are you using?

I'm using this:
<link rel="stylesheet" href="//cdnjs.cloudflare.com/ajax/libs/KaTeX/0.5.1/katex.min.css">

Is there a link which always points to the latest release of katex.min.css?

What version of the JS do you have?

Oh, Nikola's template uses version 0.3.0..
<script src="//cdnjs.cloudflare.com/ajax/libs/KaTeX/0.3.0/katex.min.js">

That should be updated. Is there a link to the latest release which could be put to templates so that it doesn't need to be updated in the templa? Alternatively, Nikola could provide a setting in conf.py to specify which version to use, that'd be ok too.

@jluttine unfortunately there isn't.  Please see https://github.com/Khan/KaTeX/issues/239 for details.

Well, #239 mentions http://cdn.jsdelivr.net/katex/latest/katex.min.js as a solution (currently at 0.5.1).

I'm not sure who's updating jsdelivr.  @cben glad to hear that it's up to date.

I think it updates automatically from npm.


Fixed and extended modification of eglCreateContext's attribute lists

I fixed an issue in the looping of eglCreateContext's attribute list where it was going until the attribute name was 0 instead of EGL_NONE.
I've also added code to raise the requested context version to 3.2 or 3.1 depending on support. I first try 3.2 because if supported RenderDoc will end up using 3.2 functions (e.g. glIsEnabledi) instead of the extension versions. If not supported, I fallback to 3.1 as gl_rendertext requires it.

I don't understand this change, you say "it was going until the attribute name was 0 instead of EGL_NONE." but EGL_NONE is defined to 0 so I don't see the distinction.
Also we definitely should not modify the version requested by the application during capturing, any changes to application behaviour should be as minimal as possible. Can you explain in more detail why you want to do that? You mention gl_rendertext requiring a version - if this is true it should be fixed instead.

Hi,
EGL_NONE is a bit unintuitive and defined as 0x3038 in egl.h..
In the case of raising the context version. I'm experimenting with getting RenderDoc to work with Imagination Technology's PVRVFrame (libEGL/libGLESv2 emulation libraries) on Linux. Its context creation is a little stricter than most drivers so if you request a 3.1 context you'll literally get a 3.1 context. As a result you won't be able to use any 3.2 functions even if eglGetProcAddress returns them. Due to the way the real function pointers are loaded in RenderDoc in some situations it'll end up using 3.2 core functions even if the context is 3.1 e.g. glIsEnabledi will get used instead of glIsEnablediOES because eglGetProcAddress will return a valid function pointer for glIsEnabledi first. This is unfortunately why I added the attempt to raise the context to 3.2.
In the case of gl_rendertext, for its shaders it appear to be using glsl version 310 so requires a 3.1 context. For my testing I was using a simple application that only requested an ES 2.0 context so the rendertext shaders were failing to compile.
I do agree though, modifying the application behaviour isn't ideal but was the simpler fix.
Cheers,
Scott

Eurgh, sorry that's my mistake, I assumed EGL_NONE was defined like GL_NONE to 0 and didn't check carefully when searching. That would be far too sensible...
I still think this should still be fixed elsewhere, fixing it like this could easily have knock-on effects elsewhere in more subtle ways as it's unknown what effect changing the version requested will have - it could even break application code that queries for the active version.
The problem you describe is similar to one on linux, where mesa will return function pointers for unsupported functions that just print an error message, so the code should generally handle checking explicitly for the necessary extensions before calling a function. It sounds like maybe this is just missing a couple of cases to account for GLES being different.
The text shaders should be pretty simple - they use GLSL 150 on desktop which is significantly older, so perhaps the GLES version could just be reduced? It's worth checking if the shaders actually use any features that aren't available in ESSL 200 - and if so which ones, since it should be possible to refactor the text rendering.
If you want to split apart the EGL_NONE fix that can be merged in, but the other changes I think will have to be reconsidered as above.

No problem. I'll get the EGL_NONE fix split off now and will have a look at fixing the other issues in a different way.


sphinx-togglebutton: Make solution admonitions collapsed by default

Add this extensions as a hard dependency and sets up the solution
to start collapsed by default.
https://github.com/executablebooks/sphinx-togglebutton
Review: general check unless someone wants to takelocally and
compile it.  A test is in the docs: directives.rst

I did not manage to make it work on my windows laptop. But it is probably not super important at this stage. Shall we merge it?

On Sun, Sep 13, 2020 at 05:09:44AM -0700, Anne Fouilloux wrote:

I did not manage to make it work on my windows laptop. But it is probably not super important at this stage. Shall we merge it?

Interesting... I see it works here::
https://coderefinery.github.io/sphinx-lesson/branch/rkdarst--togglebutton/directives/
I'd propose we merge and continue debugging... then we can see if it
works live and get those previews