New landing page about self-hosting Zulip

Here's how I'm imagining this could look:

Hero: add space above the buttons.
The first 4 sections ("100% free and open-source software" through "Easy to set up and maintain"): Add alternating background colors.
Not sure about what to do with the quote. Currently using it instead of an icon.
For the next 4 sections, I was thinking we could have them in a 2x2 grid of rectangles with text and an icon in each rectangle. (The icon could be above or, potentially, below.) I want to de-emphasize these compared to the first 4 sections; I think they are more for Igor and less for Linda/Franz.
Because I copied the plans boxes from /for/education, they are too big on the bottom.
We should make the font bigger.

I have icon options picked out for each section in a Paper doc. We will need to double-check the licenses, but I think first we need to decide whether we are going in this direction.
I would guess that we'll be able to re-use components from this page for our upcoming Zulip vs. X pages, so it's worth spending some time figuring out what we like.
@amanagr @timabbott what do you guys think?
@amanagr would you be up for playing with the CSS for this? Obviously, the text has not been finalized, but it will be roughly the amount of content you see (i.e. just a couple of lines per heading).

OK, I did another content pass on this; would be glad to get @timabbott 's help to wrap up the content work.

For the visuals: @amanagr

Could we put the quote on a background that's more like the "Apps for every platform" part of the homepage, i.e. a bluish-greenish gradient?
For the plans section, can we put it on the same gradient we use at the top of /plans? I like that one better, and I think it's good for these to be visually consistent.
For the "Reliable and scalable", etc. boxes, we'll want the text to be more vertically centered. Since I think we're keeping the text pretty short, we could probably make them boxes less tall.

I pushed a bonus commit that adds a sentence to one of the top region things that I think obviates the need for a "Runs on" thing on the bottom.

As I'm looking at this, one more change:

Can we try left-aligning the body text in the 4 boxes? Somehow the centered text is bugging me, especially when the line breaks happen to be awkward. It's possible the titles can stay centered.

I'm moving "we're here to help" into one of the four boxes, which has now freed up.
@amanagr could you try using this icon for it? https://www.iconfinder.com/icons/326539/faq_help_live_question_icon

With the changes I made, we need to remove the white stripe between the feature boxes and the plans:

@alya updated!

Nice!
Can we try centering the images in the 4 boxes even though the text is left-aligned? Somehow it doesn't look quite right with the icons being on the left.

Done

@amanagr could you please update the CSS to use the same italics we have on /plans for the relevant part of the plans boxes on this page?

@amanagr one more: could you please replace the icon for "No lock-in" with https://www.iconfinder.com/icons/298757/database_icon?

Thanks! Yeah, I can take care of the links as soon as I'm done fiddling with the page.
Is it a problem that there are failing checks?

I just pushed a commit adding links to /self-hosting.
@amanagr can you please make the version number variable I added to the install button in the /self-hosting page hero work? We used to have it on /hello, but I want to just link to /self-hosting from there now.

I'll make a separate commit for documentation updates to link to /self-hosting as well.

@amanagr another thing to fix is that alignment in the bottom area looks off at narrow (but not the narrowest) windows:

I wasn't sure what's the best way to fix this lint error -- should the section be renamed, or is there something better to do?
templates | ERROR:Duplicate ID(s) detected :Id 'feature-pricing' present at following files:
templates | Line 247:templates/zerver/for-education.html
templates | Line 266:templates/zerver/self-hosting.html

https://github.com/zulip/zulip/pull/19984#issuecomment-1036545164

I can increase height there, but then the other boxes will have a lot of extra vertical space. Generally, there the text in the 4 boxes have a similar number of words, so these problems don't arise. Can we reduce some content in the 4th box?

https://github.com/zulip/zulip/pull/19984#issuecomment-1036572811

I can take of care of CI when you are done with your changes.

https://github.com/zulip/zulip/pull/19984#issuecomment-1036444229

Done.

#19984 (comment)

I can increase height there, but then the other boxes will have a lot of extra vertical space. Generally, there the text in the 4 boxes have a similar number of words, so these problems don't arise. Can we reduce some content in the 4th box?

Hm, maybe? It would also be good if the images and headings were aligned in the top 2 boxes, though.

So, image and title have fixed distance from top now. So, we can add some content in the first two boxes to make fill the extra space.

Cool, thanks! Could you reduce the bottom margin on all the boxes? It's nice to have them equal to each other, but I don't think they have to be square. I'll add some more text as well.

For the top buttons, the current version of the emphasis isn't quite working for me. Would you mind changing it back to having the three buttons be styled the same? I'll start a #design thread to brainstorm a bit, but it's not a launch blocker.
Also, somehow a duplicate button snuck in. :)

Done.

Can you fix the CI issues?

2022-02-17 15:07:29 [documentation_crawler] ERROR: Please check link: https://zulip.com/self-hosting/ this is the only failing test after the latest push which will be fixed on deploy right?

I applied the above change (also fixed weird whitespace in one place) and merged the parts that don't modify ReadTheDocs as the series ending with e93320c40d1a99bdd813d5ec309efd50cd71f562, and pushed back to this pull request the remaining piece; we can merge it once it passes CI (aka once we've deployed what we've already merged to Zulip Cloud).

Since the self-hosting page is now deployed to zulip.com, I've rebased the remaining commit and tagged it to merge once CI passes.


[RadioComercial] Add extractor

IMPORTANT: PRs without the template will be CLOSED
Description of your pull request and other information

This extractor was created specifically for the Portuguese radio station Radio Comercial.
Its main function is to fetch and download podcast episodes.
Presently, it offers two extract functions:

Downloading individual podcast episodes by directly using their URL links.
Downloading all the episodes of a specified season for a particular podcast, utilizing the `playlist concept.

Valid URLs that are covered by this extractor:

Single episode: https://radiocomercial.pt/podcasts/convenca-me-num-minuto/t3/convenca-me-num-minuto-que-os-lobisomens-existem
Entire podcast playlist: https://radiocomercial.pt/podcasts/as-minhas-coisas-favoritas
Playlist specific to a particular season of a podcast: https://radiocomercial.pt/podcasts/convenca-me-num-minuto/t3

Template

Before submitting a pull request make sure you have:

[X] At least skimmed through contributing guidelines including yt-dlp coding conventions
[X] Searched the bugtracker for similar pull requests
[X] Checked the code with flake8 and ran relevant tests

In order to be accepted and merged into yt-dlp each piece of code must be in public domain or released under Unlicense. Check all of the following options that apply:

[X] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[ ] Fix or improvement to an extractor (Make sure to add/update tests)
[X] New extractor (Piracy websites will not be accepted)
[ ] Core bug fix/improvement
[ ] New feature (It is strongly recommended to open an issue first)

Copilot Summary

🤖 Generated by Copilot at bd9904c
Summary
🎙📻🎧

This pull request adds two new extractors for yt-dlp, RadioComercialIE and RadioComercialPlaylistIE, which enable downloading audio and playlists from the Portuguese radio station Radio Comercial. It also fixes a minor formatting issue in _extractors.py.

Oh we are the coders of the sea
And we write extractors with glee
We pull the audio from RadioComercial
On the count of three, heave ho, heave ho!

Walkthrough

Add support for Radio Comercial extractors (link, link)

Import new classes RadioComercialIE and RadioComercialPlaylistIE from radiocomercial.py in _extractors.py (link)
Define new classes in radiocomercial.py that inherit from RadioComercialBaseExtractor (link)

Remove an empty line from _extractors.py for formatting consistency (link)

Same as https://github.com/yt-dlp/yt-dlp/pull/8507#issuecomment-1792785496. Please instead edit the description and push additional commits onto the other branch

@seproDev thank you very much for taking the time to look into this, and provide those valuable suggestions. I'm working on those and will update the code once complete.

I'm currently having an issue extracting playlists from the website because of how it deals with unavailable episodes. At first, I was using a Python set to gather the episode links from the elements with class tm-ouvir-podcast, avoiding the duplicates entries (this class shows up twice for each episode).
The trouble is, any unavailable episode defaults to the same URL (like radiocomercial/podcast/). This behaviour can be seen in test number 4 of the RadioComercialPlaylistIE class, in which two episodes are unavailable on the same page (first).
This situation leads to two problems:
Problem 1: When I was using a Python set and more than one episode was missing, I ended up with fewer episodes than the expected number (_PAGE_SIZE). This stopped the scanning of additional pages.
To fix this, I changed the code to get rid of the duplicate tm-ouvir-podcast classes. Now, I  use lists instead of sets, ensuring the episode count always matches the _PAGE_SIZE, by keeping the unavailable episodes entries.
Problem 2: The unavailable episodes cause an error when the downloader tries to process them.:
[RadioComercialPlaylist] Playlist TNT - Todos No Top - Temporada 2023: Downloading 41 items of 41
[download] Downloading item 1 of 41
[RadioComercial] Extracting URL: https://radiocomercial.pt/podcasts/tnt-todos-no-top/2023/t-n-t-29-de-outubro
[RadioComercial] t-n-t-29-de-outubro: Downloading webpage
[info] t-n-t-29-de-outubro: Downloading 1 format(s): 0
[download] Destination: T.N.T 29 de outubro [t-n-t-29-de-outubro].mp3
[download] 100% of   86.69MiB in 00:00:13 at 6.62MiB/s
[download] Downloading item 2 of 41
ERROR: No suitable extractor (RadioComercial) found for URL https://radiocomercial.pt/podcasts/tnt-todos-no-top/2023/
[download] Downloading item 3 of 41
[RadioComercial] Extracting URL: https://radiocomercial.pt/podcasts/tnt-todos-no-top/2023/t-n-t-15-de-outubro
[RadioComercial] t-n-t-15-de-outubro: Downloading webpage`

I think I initially suggested using a PagedList over a generator function 🙃
Sorry about that

@seproDev all good, the issues with using a PagedList weren't immediately apparent

Both of the problems you mentioned mean that a PagedList is not a viable option for this playlist extractor.
Instead we'll need to just use a generator function (e.g. _entries()), and we can use playlist_from_matches() which casts the URLs to an orderedSet and then constructs the url_results for us. We can also use RadioComercialIE.suitable() to ensure the URLs are not bogus

Thank you very much for taking the time and providing the proper solution for this use case. This has been a great learning experience so far.


github-runner: 2.297.0 -> 2.298.2

Description of changes

Release notes: https://github.com/actions/runner/releases/tag/v2.298.2
diff: https://github.com/actions/runner/compare/v2.297.0...v2.298.2
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Anyone test this yet? I'll test it out tomorrow if not.

Just tested on x86_64-linux, works.

Thanks so much!

Was this not backported to 22.05?


Define and add class "food choice management process"

Add a class:
label food choice management process [ONS:0000128]
superclass information content entity
definition the process by which an individual makes choices about food consumption, based on multiple determinants
Rationale: individuals make their "food choices" [ONS:0000132] as a result of a decision-making process that we are defining as the "food choice management process". In this way, each individual "dietary pattern" [ONS:0000094] becomes the result of the "food choice management process" that includes a balance of healthy and unhealthy "food choices" which are influenced by the "determinant of diet" class

Integration and minor corrections
label food choice management process [ONS:0000128]
superclass process
definition the food choice management process refers to the complex decision-making process an organism carry out to realize choices about which food to consume, if multiple options are available

I would change "realize" to "make" because realizing in English is taking the notion of something or recognizing, instead of doing something. I also think that it should be "an organism carries out" instead of "carry"


How can I take some actions after it successfully connects or fails?

@Module({
  imports: [
    MikroOrmModule.forRoot({
      // I can't find such an init callback
    }),
  ],
  controllers: [AppController],
})
export class AppModule {}

You can use onModuleInit from nest
https://docs.nestjs.com/fundamentals/lifecycle-events#asynchronous-initialization


AS3 Difficulties with Monitor_HTTPS_passphrase

Environment

Application Services Version: 3.22.1
BIG-IP Version:<IP_ADDRESS>

Summary
Attempting to create an AS3 declaration for a custom HTTPS monitor with a username and password for the monitored application.

How can we confirm the base64 encoded value matches the actual password
the Username contains a back slash which isn't accepted in json/AS3.

Steps To Reproduce
Steps to reproduce the behavior:

Submit the following declaration:

monitor portioin of the declaration:
            "as3-https-monitor": {
              "class": "Monitor",
              "monitorType": "https",
              "send":"GET /Ping.html HTTP/1.1\r\nHost: example.ca\r\n",
              "receive": "Pong",
              "username": "mydomain\F5Monitor",
              "passphrase": {
                "ciphertext": "f5base64abc123",
                "protected": "eyJhbGciOiJkaXIiLCJlbmMiOiJub25lIn0",
                "ignoreChanges": true
                }

2. Observe the following error response:
```json
This is due to the back slash in the username

{
    "code": 500,
    "message": "Unexpected token F in JSON at position 3806",
    "referer": "restnoded",
    "originalRequestBody": "",
    "errorStack": []
}

Expected Behavior
When configured successfully TMSH looks like this:
ltm monitor https example.ca_monitor {
adaptive disabled
app-service /web-partition/example.ca.app/example.ca
defaults-from /Common/https
description "This checks the status of the webservice"
destination :https
interval 5
ip-dscp 0
partition web-partition
password "***"
recv Pong
recv-disable none
send "GET /Ping.html HTTP/1.1\r\nHost: example.ca\r\n"
time-until-up 0
timeout 16
username mydomain\F5Monitor
Actual Behavior
with a single \ I get a 500 error message returned (see above). if I POST a double slash\ TMSH configures the following:
username mydomainF5Monitor
missing the \

Here is the solution to my pollution:
In order to define a secret password, the ciphertext needs to be added. the value of ciphertext is password converted in to Base64. for example, put ZjVmNQo= which is the base64 of the value f5f5.
Here's what I did to get the base64 of the value f5f5:
echo 'f5f5' | base64
ZjVmNQo=
To reverse the process:
echo 'ZjVmNQo=' | base64 --decode
f5f5
For the protected property value:
1. eyJhbGciOiJkaXIiLCJlbmMiOiJub25lIn0,  use with secret simply base64url-encoded into ‘ciphertext’.
2. eyJhbGciOiJkaXIiLCJlbmMiOiJmNXN2In0,  ‘ciphertext’ contains base64url-encoded SecureVault cryptogram  --> typically referenced when defining a certificate's key passphrase. for more info:

   JWE: https://clouddocs.f5.com/products/extensions/f5-appsvcs-extension/latest/refguide/schema-reference.html#jwe
   JSON Web Encryption (JWE): https://tools.ietf.org/html/rfc7516

Finaly, if the defained Username contains a backslash “\” to separate the domain from the user. use the following format to escape the backslash:
   "username": "domain\\\\user",


Need a boost ? [Feature]

It seems from README that wechaty-puppet used by go-wechaty are all outdated, and as per commit logs and #74, no attempts had been made to incorporated the current/new wechaty-puppets. I.e., it seems the project died after the wechaty-puppet-padplus had died, which is what I really don't want to see.
Describe the solution you'd like
A clear and concise description of what you want to happen.
看一下这个
https://github.com/wechaty/summer-of-wechaty/issues
这个是目前其他的mentor已经发出来的project，在我看来，go wechaty的功能迫切需要完善，这同一个项目里面不要说放一两个学生，我觉得就是放上十几个也没问题，他们都是在做同一个项目，只不过分别到具体的方面不一样
具体到这个特别的项目，分工如何，看什么做了，什么还没做，指导我都不行，不过我可以在具体的某一些点上做一些帮助。比如，只是说比如，我可以帮着做基于go 的 unit test 框架，不过还是那点，让我做框架设计肯定不行，我能帮到的就是在大的框架结构下面做一些具体的工作

Thank you @suntong for pushing this project moving forward!
Recently I read more go source code with the gRPC / OpenAPI projects, I feel that I start loving GO a lot more than before!

I'm really glad that you are feeling this way recently. I've done programming in more than 10 languages throughout my life now, and Go is the one that I love the most.
For many other languages, (especially python, cough, cough), the programming journey is, "this should work" -> "why not working" -> "Oh, man, this one need special twist?" -> "Oh, this one as well" -> "My God, here needs another twist", and the debugging/troubleshooting goes on and on...
For Go, the programming journey is, "should do this way but would that work?" -> "OMG, I can't believe it works in first attempt!!!" Now for most of my Go programs, I'd just write once, and need really little further tweaking, and it works! This is an amazing experience that I had never before. Not even C/C++.

 My Go based docker are just Alpine Linux base Docker image, with a complete package index, but it is just only 5 MB in size, plus the compiled Go executable, and NOTHING MORE!!!
No any other programming language on earth can do that!

It seems from README that wechaty-puppet used by go-wechaty are all outdated, and as per commit logs and #74, no attempts had been made to incorporated the current/new wechaty-puppets. I.e., it seems the project died after the wechaty-puppet-padplus had died, which is what I really don't want to see.

@suntong go-wechaty 还没有停止，可能是大家比较忙，我已经在解决一些issue😏, 并且提了一些 pr,

Glad to know, thanks for the update @dchaofei
If you have some time, please take a look and plan for
https://github.com/wechaty/summer-of-wechaty/


Getting error WARNING (moor): It looks like you've created the database classAppDatabase multiple times Please help

Hello, we are Trying to find a way to fig the problem with this error.
flutter: WARNING (moor): It looks like you've created the database classAppDatabase multiple times. When these two databases use the same QueryExecutor, race conditions will ocur and might corrupt the database.  Try to follow the advice at https://moor.simonbinder.eu/faq/#using-the-database or, if you know what you're doing, set moorRuntimeOptions.dontWarnAboutMultipleDatabases = true Here is the stacktrace from when the database was opened a second time: #0      GeneratedDatabase._handleInstantiated (package:moor/src/runtime/api/db_base.dart:90:30) #1      new GeneratedDatabase.connect (package:moor/src/runtime/api/db_base.dart:69:12) #2      new _$AppDatabase.connect (package:system_wall/classes/localdata/SQL.g.dart:8261:55) #3      new AppDatabase.connect (package:system_wall/classes/localdata/SQL.dart:94:62) #4      _backgroundConnection.<anonymous closure> (package:system_wall/classes/localdata/SQL.dart:75:42) <asynchronous suspension> #5      _backgroundConnection.<anonymous closure> (package:system_wall/classes/localdata/SQL.dart) #6      LazyDatabase._awaitOpened (package:moor/src/utils/lazy_database.dart:30:26) #7      LazyDatabase.ensureOpen (package:moor/src/utils/lazy_database.dart:43:12)
We are using Kiwi in the newest version.
We also use moor_ffi and Moor un the newest version
The Kiwi code:
void main() {   KiwiContainer kiwi = KiwiContainer();   kiwi.registerSingleton((container) => AppDatabase());   runApp(MyApp()); }
Our only Call of the Kiwi for now
static KiwiContainer kiwi = KiwiContainer(); static AppDatabase database = kiwi.resolve<AppDatabase>();
Our Database Constructor
`class AppDatabase extends _$AppDatabase {
//static final AppDatabase instance = AppDatabase();
AppDatabase() : super(LazyDatabase(() async {
MoorIsolate isolate = await _createMoorIsolate();
DatabaseConnection isolatedConnection = await isolate.connect();
final isolatedDatabase = AppDatabase.connect(isolatedConnection);
return isolatedDatabase.executor;
}));
@override
int get schemaVersion => 15;
//neu für isolate
AppDatabase.connect(DatabaseConnection connection) : super.connect(connection);
....
}`
we copy pasted the _createMoorIsolate from this website
https://moor.simonbinder.eu/docs/advanced-features/isolates/
I hope you can help us

Sure! In your case, the problem is that the constructor of AppDatabase creates a second AppDatabase (through the connect constructor).
Unfortunately in this case, there's no way to synchronously connect to an isolate, so you'd have to make it a Future:
static Future<AppDatabase> create() async {
  MoorIsolate isolate = await _createMoorIsolate();
  DatabaseConnection isolatedConnection = await isolate.connect();

  return AppDatabase.connect(isolatedConnection);
}

But thanks to this issue, I had the idea for DatabaseConnection.delayed that will be available in the next moor version. Then, you'll be able to use:
factory AppDatabase() {
  return AppDatabase.connect(DatabaseConnection.delayed(Future(() async {
    MoorIsolate isolate = await _createMoorIsolate();
    return isolate.connect();
  })));
}

This allows you to construct the AppDatabase synchronously while using a background connection.

If you only use the AppDatabase on one isolate (which, judging by the code you posted, you do), you can just do this now:
factory AppDatabase() {
  final executor = LazyDatabase(() async {
    MoorIsolate isolate = await _createMoorIsolate();
    DatabaseConnection connection = await isolate.connect();
    return connection.executor;
  });

  return AppDatabase.connect(DatabaseConnection.fromExecutor(executor));
}

You could also skip the connect constructor and just use AppDatabase(executor) like you would regularly. In this approach, stream queries won't work across databases on different isolates. If you only use the database on a single isolate, this is not a problem at all.
I hope that helps, let me know if need anything more!


Fix recompilation when batch processing is not triggered

Description
This patch provides a fix to avoid continuous recompilation of execution plans that do not use batch processing.
TornadoVM was set to recompile every time, but we only need to recompile under certain conditions only when using batch processing, or when changing the device.
Side note: We need the benchmarking suite for the AERO project.
Problem description
The issue was that the benchmark suite, and therefore, user code, can trigger recompilation over and over again due to a condition for batch processing.
Backend/s tested
Mark the backends affected by this PR.

[X] OpenCL
[ ] PTX
[ ] SPIRV

OS tested
Mark the OS where this PR is tested.

[X] Linux
[ ] OSx
[ ] Windows

Did you check on FPGAs?
If it is applicable, check your changes on FPGAs.

[ ] Yes
[X] No

How to test the new patch?
$ tornado --enableProfiler console -m tornado.benchmarks/uk.ac.manchester.tornado.benchmarks.BenchmarkRunner blackscholes

$ tornado-test -V uk.ac.manchester.tornado.unittests.batches.TestBatches

Test: class uk.ac.manchester.tornado.unittests.batches.TestBatches
	Running test: testSameInputSizeAndTypeRestriction ................  [PASS]
	Running test: test100MBSmall             ................  [PASS]
	Running test: test100MBSmallLazy         ................  [PASS]
	Running test: test100MB                  ................  [PASS]
	Running test: test100MBLazy              ................  [PASS]
	Running test: test300MB                  ................  [PASS]
	Running test: test300MBLazy              ................  [PASS]
	Running test: test512MB                  ................  [PASS]
	Running test: test512MBLazy              ................  [PASS]
	Running test: test50MB                   ................  [PASS]
	Running test: test50MBInteger            ................  [PASS]
	Running test: test50MBShort              ................  [PASS]
	Running test: test50MBDouble             ................  [PASS]
	Running test: test50MBLong               ................  [PASS]
	Running test: testSameInputSizeAndTypeRestrictionJavaArrays ................  [PASS]
	Running test: testSameInputTypeRestriction ................  [PASS]
	Running test: testSameInputTypeRestrictionJavaArrays ................  [PASS]
	Running test: testSameInputSize          ................  [PASS]
	Running test: testSameInputSizeJavaArrays ................  [PASS]
	Running test: testSameInputSizeJavaToTornado ................  [PASS]
	Running test: testSameInputSizeTornadoToJava ................  [PASS]
	Running test: testSameInputSizeAndTypeJavaToTornado ................  [PASS]
	Running test: testSameInputSizeAndTypeTornadoToJava ................  [PASS]
	Running test: testBatchNotEven           ................  [PASS]
	Running test: testBatchNotEven2          ................  [PASS]
	Running test: testBatchNotEven2Lazy      ................  [PASS]
	Running test: testBatchThreadIndex       ................  [PASS]
Test ran: 27, Failed: 0, Unsupported: 0

Validation has been fixed too.


#17040 workaround for no handles && use browser for image editor

Adds browser view for images. The view is enabled in the new preference page Editors -> Data Editor -> Appearance -> Image
Closes #17040, closes #17489

The image option isn't available as an option in Preferences in the current build. Also, it isn't available in Jenkins build (CE)
https://build.dev.dbeaver.com/job/DBeaverDesktop/job/dbeaver-ce/478/

verified


⚠ Fortnite has degraded performance

In 4725564, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1565 ms

Resolved: Fortnite performance has improved in 68201b1.


Factor Tracer out of graphql-tools

Remove tracer agent from GraphQL-tools
bump graphql peer dependency to 0.6.0

[x] Update CHANGELOG.md with your change (include reference to issue & this PR)
[x] Make sure all of the significant new logic is covered by tests
[x] Rebase your changes on master so that they can be merged easily
[x] Make sure all tests and linter rules pass

Coverage decreased (-0.08%) to 99.631% when pulling df747f47d2daef8b37a65d3b9a0c1d2f4f90eee8 on tracer-refactor into 69216ae2a94f8736457201d2773489c88d50cf0e on master.


Update formatting and sentence structure

A paragraph was formatted as code.
Changed sentence structure to keep the list as the last part.

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

@SvenAelterman This is a good fix. Thank you! @abiolao96 I am going to merge this in. #sign-off

@rothja: I'm sorry - only the author of this article, @Abiola, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


[dv] Key Sideload Parameter Support

This PR includes support for parameterisation of key sideload agent. Also includes minor fixes where we use key sideload agent (AES, KMAC and OTBN), in order to use the parameterised version correctly.
Signed-off-by: Canberk Topal<EMAIL_ADDRESS>

This apparently became a bit more invasive due to widening of the port resulting with X propagation in KMAC and AES. I trimmed them in their respective testbenches. I am open to suggestions regarding handling of the issue, do you think this solution is acceptable?

This apparently became a bit more invasive due to widening of the port resulting with X propagation in KMAC and AES. I trimmed them in their respective testbenches. I am open to suggestions regarding handling of the issue, do you think this solution is acceptable?

Would it work if we parametrize this interface with different width? Like what we did for push-pull agent?
If that works, maybe you can leave a TODO in this PR and implement that in a separate PR.

This apparently became a bit more invasive due to widening of the port resulting with X propagation in KMAC and AES. I trimmed them in their respective testbenches. I am open to suggestions regarding handling of the issue, do you think this solution is acceptable?

Will this work: https://github.com/lowRISC/opentitan/blob/master/hw/dv/sv/push_pull_agent/push_pull_agent.sv#L8
Configure the width when we create the agent?

Hi Cindy, it worked! (after bits of painful changes to the key sideload agent). This PR now introduces the support for parameterisation of key sideload with the default option supporting AES & KMAC. Still touches both of them for some minor fixes in order to execute without syntax errors.

I think we cannot do such a formulation because the bits of KEY_T depends on the number of key shares. So, we cannot infer the key width without that parameter (in the end I believe we would end up switching KeyWidth with Shares).

But isn't Shares a global defined in keymgr_pkg.sv? I think it's the same for each key type?  I now changed the commits to have only KEY_T as a parameter (and KeyWidth is now just a localparam under key_sideload_item)

Commit 3/4 are the result of this type of error, am I missing something?

Oh, I see.  Let's go with this!


Juliyasmith/ch17282/migrate ui tests in notifications to jest

https://app.clubhouse.io/augur/story/17282/migrate-ui-tests-in-notifications-to-jest

Coverage increased (+31.8%) to 57.493% when pulling a286599f869503d816ed281a1f5143730381db7c on juliyasmith/ch17282/migrate-ui-tests-in-notifications-to-jest into 9282398e7473d0b3279184e410bb3a9c37947562 on master.


Fix name fetchers on pod mutator

#169414951
Fixes #666

#169414951
Fixes #666

Can you also add a unit test for  this label ?


nc_test/run_diskless.sh: ncgen fails, and error is concealed

Environment

Netcdf-C 4.7.3 with HDF5 1.10.5
Multiple platforms: Linux, Mac OS
GCC versions 7.3 and 9.2

Steps to reproduce

Standard install of netcdf-c 4.7.3
configure, make, make check
Capture console output from make check

Symptoms

Unit test log file contains unexpected ncgen error message.  Also notice the final status on the last line is PASS, when FAIL should actually be indicated somewhere.

... netcdf-c-4.7.3/nc_test 21> tail -4 run_diskless.log
**** Test extended enhanced diskless netCDF with persistence
**** tst_diskless2.nc created
lt-ncgen: tmp1.cdl line 53: Enumconstant associated with a non-econst type
PASS run_diskless.sh (exit status: 0)

Error is not counted in any of the make check test summaries.

make  check-TESTS
make[2]: Entering directory '/home/.../netcdf-c-4.7.3/nc_test'
make[3]: Entering directory '/home/.../netcdf-c-4.7.3/nc_test'
PASS: t_nc
PASS: tst_small
PASS: nc_test
PASS: tst_misc
PASS: tst_norm
PASS: tst_names
PASS: tst_nofill
PASS: tst_nofill2
PASS: tst_nofill3
PASS: tst_atts3
PASS: tst_meta
PASS: tst_inq_type
PASS: tst_utf8_validate
PASS: tst_utf8_phrases
PASS: tst_global_fillval
PASS: tst_max_var_dims
PASS: tst_formats
PASS: tst_def_var_fill
PASS: tst_err_enddef
PASS: tst_default_format
PASS: tst_diskless6
PASS: run_diskless.sh      <-------------
PASS: run_diskless5.sh
PASS: run_inmemory.sh
PASS: run_cdf5.sh
==================================================================
Testsuite summary for netCDF 4.7.3
==================================================================
# TOTAL: 25
# PASS:  25        <-------------
# SKIP:  0
# XFAIL: 0
# FAIL:  0         <-------------
# XPASS: 0
# ERROR: 0
==================================================================

This breaks into two separate issues immediately, but I am reporting them together.

Ncgen fails on an enum inside a compound type.  I am not sure whether this is a real ncgen bug, or a problem in the CDL input.  This appears to be repeatable outside of the test suite.

run_diskless.sh includes an ncdump/ncgen cycle, but fails to properly detect and report a failure within the cycle.

Looks like we have alot to fix here. The enum const in a compound
rings a bell. I think this may reflect either an ambiguity in CDL
or a problem in the library.
In any case, I will try to get up to speed on the underlying cause.

This problem was originally found by the ncgen test suite:
https://gitlab.com/remikz/nccmp/issues/10
Also there is a little more information and a simple reproducer in the original bug report to netcdfgroup:
https://www.unidata.ucar.edu/mailing_lists/archives/netcdfgroup/2019/msg00113.html

Here are some results from other versions of netcdf-c, including my own tests and other people's reports.
<IP_ADDRESS> -- Correct output file
4.6.2 -- Assertion failure
4.6.3 -- Conflicting reports, fails for me
4.7.0 -- Assertion failure
4.7.2 -- Enum failure
4.7.3 -- Enum failure
So, I guess you would call this a regression.

Thanks @Dave-Allured !

I think the enum part was fixed by PR https://github.com/Unidata/netcdf-c/pull/2109

I believe Dennis is correct, as I am not able to duplicate this issue any longer.


[Minor Versioning] - Add deserialize_for_version trait

What is this task and why do we need to work on it?
After we introduce minor version upgradability to the sequencer, we will need a trait that deserializes types such as the Header + Payload by version.
What work will need to be done to complete this task?

[ ] Create a deserialize_for_version trait that can be implemented in sequencer
[ ] Remove Delta trait entirely (we ended up not dealing with state deltas generically, since this can just be a concrete type within the sequencer). This is important because currently we have Delta in the Decide event, which requires it to be serializable, but we don't need this
[ ] Remove ValidatedState from Decide event. Same as above, we don't use this anymore in the sequencer and we don't want ValidatedState to be serializable.
[ ] Remove Serialize, Deserialize trait bounds from ValidatedState trait (this should be trivial after the above)
[ ] Replace Deserialize trait bound with DeserializeVersion in remaining HotShot traits (BlockHeader, BlockPayload, any others?)
[ ] Wherever we need to deserialize messages containing these objects, we need to use deserialize_for_version and make sure we have the consensus version plumbed into that part of the code. This is the part I'm not sure about, might be hard.

Are there any other details to include?
No response
What are the acceptance criteria to close this issue?

[ ] Add deserialize_for_version test that serialize some mock types to different representation depending on some version enum
[ ] No HotShot traits require Deserialize anymore

Branch work will be merged to (if not the default branch)
No response

Actually BlockPayload already doesn't use serde, it has its own encode/from_bytes methods, so we can just add a version parameter to from_bytes


[Feature request] Publish Android app via F-Droid.org

It would be really, really great if you could consider also publishing the app via F-droid.org, please?
F-Droid is the  "Alternative FOSS market place" for Android. Users who opt out of running Google spyware blobs don't have access to the Play Store, and F-droid is the de-facto primary alternative.
There is currently a build of the Geant 'Eduroam CAT' on there, but it looks like that might have been an unofficial side-project by someone at Swansea Uni, and it has not had an update since 2019 - ergo support for it seems to be moribund at this point.
Thanks

Hi; this was mentioned in #90 also; we tried to publish in F-droid actually, but it seemed at the time we were unable to submit Ionic based Apps to F-droid.
We are working on a new version that doesn't use this framework, and we should definitely try again in due course. (We do have an .apk for download, but I'm not sure if that helps much.)

We do have an .apk for download

Good to know; but I couldn't find it :)

Hi, hmm you're right we could have included an .apk in the release. I just added it for 1.0.17 (though that's officially flagged iOS) and 1.0.16.

We were rejected on F-Droid before, but I think we can try again now: 05e62061a5a7b190b4b1d6bd1897097709c324fa

An alternative possiblity might be a self-hosted F-droid compatible repo, and publishing the URl for that. I don't know how easy this is to set up, but I do know that quite a few projects that have minor licensing/build issues that keep them out of the main F-Droid publish through their own repos to get around restrictions.

Another alternative would be to ask @IzzySoft , whether he could add the App to his repository, until the issues are resolved for the app to get into the main F-droid repository, as an temporary workaround: https://android.izzysoft.de/repo/info

That would require the APK being available at e.g. releases/, though. The last one I can find there is 2 years old.

The app has been rescanned for issues by F-Droid and here are the current results.

I think at this point, it'd be better to bet on the new app. However, Fastlane is something we need anyway so we might be able to fix that.


Documentation

Documentation for the functions in functions.php is ready, everything else is missing.
http://amphp.org/docs/socket/

yes, we need document. not just run server, but also run client

Documentation is now available at https://amphp.org/socket.


Fix RGV env variable specified as a path

What was the end-user problem that led to this PR?
The problem was that using RGV (the environment variable we use to select the proper rubygems version to test bundler against) with a path was not quite working.
Setting RGV to a path (RGV=..) is used in the rubygems repo to test the vendored copy of bundler (that lives at bundler/ inside that repo) against the checked out copy of the repository.
What was your diagnosis of the problem?
My diagnosis was that the previous approach to switch the rubygems version, namely, reexec'ing with --disable-gems and explicitly requiring our "version switching code" was not quite working because nested subprocesses would end up "leaking" to the system copy of rubygems again.
What is your fix for the problem, implemented in this PR?
My fix is to use RUBYOPT to configure the path to the rubygems copy that should be used, since I found that even the require "rubygems" present's in ruby's gem_prelude.rb respects the -I option in the RUBYOPT env variable.
Why did you choose this fix out of the possible options?
I chose this fix because it's simpler and works better.

@bundlerbot merge


[security] add datasource sources

This PR provides the concept of a datasource source which is a key abstraction for identifying the tables or druid datasources that a superset datasource derives its data from. This will be useful for people who override the security manager to identify the underlying tables and check if the person trying to access a slice has access to the underlying table.
@john-bodley

Putting this on hold for noe.


Connection problem, giving up. Some problem

I am trying to connect to Kali from Windows Subsystem for Linux. I installed xrdp, started it and connect with RDP in the port. Prompt appears with login an then a window with this:
connecting to sesman ip <IP_ADDRESS> port 3390
sesman connect ok
sending login info to session manager, please wait...
login successful for screen 10
started connecting
connection problem, giving up
some problem
I tried to edit the file in /etc/X11/Xwrapper.config changing "allowed_users=anybody".
Any ideas?
Thanks!

I tried to edit the file in /etc/X11/Xwrapper.config changing "allowed_users=anybody".

Don't be fooled by old information on the web. Stand on your own legs. AFAIK Kali Linux is a Debian based distro, if  /usr/share/doc/xrdp/README.Debian or some documents like that of your installation doesn't mention about editing Xwrapper.config, you don't need to edit.
You're also a pioneer using xrdp on Kali Linux on WSL. See also #1101.

You might be able to get your RDP connection working by using the Xvnc back-end instead of the Xorg back-end. Instructions here: https://github.com/neutrinolabs/xrdp/issues/1101#issuecomment-382033147

That worked! Thank you so much!


CI

Let's set up circle CI and/or Travis.
Tests fail on my machine currently btw.

Submitted a patch in #5

Someone with admin on this repo can go here: https://circleci.com/gh/ipfs/node-ipfsd-ctl and click Follow

done
On Tue, Jun 16, 2015 at 1:40 PM, Harlan T Wood<EMAIL_ADDRESS>wrote:

Someone with admin on this repo can go here:
https://circleci.com/gh/ipfs/node-ipfsd-ctl and click Follow
—
Reply to this email directly or view it on GitHub
https://github.com/ipfs/node-ipfsd-ctl/issues/4#issuecomment-112563171.


Option to check indentation of structured data

CLIs who outputs structured data often format the data with some indentation. It'd be great to be able to check them along with the structure of the data (#93)

Checking both indentation while analyzing higher level structure data is out of scope of the project, it'd require format-preserving parsers for each format and a lot of complexity.  Its probably best to focus some limited tests on the format and be simple enough to not need structured data comparisons from #93 and the rest on the content of the data and use #93.
(to clarify for my comment in #91, my intent had been to post your use case to #93 to remind us that allowing people to bypass structured data was important)


Need way to send Kate "Thank you" cheese wheel....can't find DM method

Thanks for updating the Wiki...it worked perfectly.
I really didn't want to spend the $300+ for the newer iComfort S30 controllers on 2 home systems ($600+).
Please let me know how I can compensate you for your time in creating and maintaining this.
Yours "warmly"...
Sean.

@sshrum01, thank you so much for your note! It made my whole night :) I’m tickled by how many people take the time to implement this skill, and I’m happy to have shared something I made for fun.
Happy new year!


Issue with dwolla account creation

From a user:

This may be an issue unique to me, but I was trying to pay for  and the link provided for Dwolla did indeed let me set up an account, but I could not find out what my PIN was supposed to be.  When I got in touch with Dwolla support, they said that I had created an account that was limited to only receiving money and not sending it.  I just wanted to let you know in case your site had the wrong link.  Thanks!

We should look into this.

Another issue from a user:

I've been trying very hard to use Dwolla to pay for my registration! Only, once I created my account, on Dancerfly it asks for my PIN, which I was never provided. I asked Dwolla for technical support, and they said a basic account does not have a PIN. And yet the Dancerfly registration will not accept it without a PIN :(

Another issue from a user (similar in that it's dwolla-related):

I noticed I have $XXX of pending payments logged in XXXXX. It's all the Dwolla payments that have happened since the beginning of the year. If I go to my Dwolla account, they show transacted (and I have the $), however they don't show as completed in Dancerfly. I'd say this ranks pretty highly on the list of things that needs to be looked at if for no other reason than Dancerfly and Dwolla are clearly not talking to each other properly.

How are you testing dwolla things in the test environment? It's not showing up as an option for me to click on when I'm on the dev server.

Potentially relevant documentation: https://developers.dwolla.com/resources/account-types/access-api-accounts.html

Closing this - we're not going to fix it. We're going to remove dwolla.


Stream projection

@AlexanderLitus, please implement storage changes as described in these tasks:
http://app.projects.tm/task/cdc7a4dd-1d57-43ea-a545-8dd469500497
http://app.projects.tm/task/06895f3f-aa30-47b8-8b52-317315d071dd


[BUG] Escape Menu needed to walk after reset

Acknowledgement of preliminary instructions

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

What problem did you encounter?
Now, ill need to do more testing, but as far as im aware, this is a bloxstrap issue, anyway, when ever i reset, i have to open the escape menu to be able to walk, there aren't any errors in the log.

What version of the escape menu are you using?

Default

What other options have you enabled in the Bloxstrap Menu?

just FPS unc-locker and hide GUI

just FPS unc-locker and hide GUI

What about in the flag editor?

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


Main Loop update rate

In Main loop logic is currently limited to updating at a rate of once per second. This is causing the mouse hold function to perform a bit unexpectedly - should be easily fixed once testing print statements removed.
Could probably be resolved by splitting up the logic in the loop to seperate timers / printing information out but this might get a bit clunky (will have to perform logical tests e.g. for the timer multiple times?)

Refactored main loop, logic detached from printing. Timer logic now checked seperately to seeing what should be printed, and is only checked by the console_info_print() method provided the date/time has changed by at least 1 second.


Hard Coded Group name for k3s servers

As already mentioned in #321 , some variable names are not ideal and should be prefixed.

token and other parameters should be 'k3s_' prefixed

The host group names for "server" and "agent" are not ideal either. The name is ambiguous. Additionally it is a host variable that is also active on other plays in the same playbook. It could possibly cause collisions with them.
Hence, in my playbook I renamed the server group to "k3s_server" and the agent group "k3s_agent". While this works for the agent playbook, it does not work for the server playbook. Unfortunately, the group names are hard coded into the latter.
This issue comes up, every time the following call is made:
groups['server']

My idea would be to introduce a variable, e.g. inventory_server_group_name and have that name default to server. In that case we would just have to change every occurence of groups['server'] to:
groups['{{ inventory_server_group_name }}']

I am willing to implement this solution myself and provide it as pull request, yet I would like to hear the admin's opinion first.

Hi @fkonradmain, I understand your issue, but could use your help better understanding "how" you and potentially other users utilize this playbook in concert with others. My day to day use of large scale ansible playbooks is limited. I typically am using only 1 or 2 separate playbooks on an inventory.
Because of my use case and potentially other beginners, when I took over as maintainer, I didn't see a reason change all the vars and the group names and add k3s_ it seemed redundant to me. You are using the k3s-ansible playbook, of course all the stuff in the inventory is for K3s.
In the K3s context, server and agent are the official names we give node roles. The naming inside the K3s space is not ambiguous, but very clear and used throughout the k3s-io org and docs.
I'm trying to balance the clarity and ease-of-use for first time ansible users (K3s is so easy to deploy that we often get people with no experience in Kubernetes starting with K3s) and people who may be utilizing this playbook in large scale enterprise deployments.
What does a typically inventory look like for you? Why is server and agent to ambiguous? What other playbooks do you utilize that come in conflict with this one?  Would the groups being changed to k3s_server and k3s_agent be enough to resolve your issue?

Currently the server role will try to join servers with the first node in the server group to create the control plane.  This means that we can not  manage multiple clusters within the same inventory with the hard-coded group names.
Making configurable groups gives us more flexibility.   I have a PR that addresses this issue in #331


Damp the effect of myriad, small allocations in optimizers

Triggering TagBot for merged registry pull request: https://github.com/bsc-quantic/Registry/pull/98

Triggering TagBot for merged registry pull request: https://github.com/bsc-quantic/Registry/pull/102

Triggering TagBot for merged registry pull request: https://github.com/bsc-quantic/Registry/pull/108


fails to give global minima

I am trying to solve a qubo problem using dwave tabu. I am using the python interface with the following commands.
from tabu import TabuSampler response = TabuSampler().sample_qubo(Q) print(response.first.energy)
every time I run, I am getting different values of energy and solution. I am getting energy close to the global minimum, but not global minima.
whereas qbsolv command gives global minima every time. what is the problem or am I missing something?

dwave-tabu and qbsolv are different solvers. They both use the tabu algorithm, but tuned differently. Also, qbsolv is a hybrid solver in a sense that it solves subproblems using a subsolver like QPU (or tabu again in "classical" mode).
Ultimately, each of them will perform better on some problems and worse on other. Unless you're running them on a simple problem, minimizing which is easy. In that case you might have found a bug. Can you share more details about your problem and/or the problem itself? Thanks.

Closing as stale.


Joken.Config get_current_time and validate_time_claim calls updated …

… to Joken.Helpers module for update on Joken
https://github.com/bryanjos/joken/commit/c72001a0b15717e3bc60a4c0ebc2acae86f51b20

Thanks for that.
I've also updated the dependencies, and I've just updated the Openmaize hex package.


Rename workflows, variables

Anatomical only right now. But so you can see where I'm going with this and stop me if I'm misinterpreting.

Yup - this is what I had in mind. We should probably also add a line to CONTRIBUTIONS about it

Along with moving from t1w to anat in workflow names, thinking of moving from bold to func, to mirror BIDS directories. (Not changing filenames at this point.)

Wasn't sure how readable this would be all in one, so did it one or two workflows at a time. Hopefully each commit is pretty understandable.

I'll can add to contributions if I get a minute tonight or first thing in the morning.

I think there are still a couple of errors in documentation.

Thanks for the bump. I forgot to check the build status of this one.

Sorry about the delay. This should be good.

Thanks!


Angular example template without any modification is so slow in production

Why is the Angular example template without any modification so slow in production?
It takes about 1.06 sec to refresh http://localhost:5000/home on local server.
How can this be a Production quality example?
1 second is too much. For most of the server admins this is not acceptable.
I am using DotNetCore version 2.0.3 on Windows x64.
To reproduce: (just the basic, without any modifications):
"dotnet new angular"
"npm install"
"dotnet restore"
"dotnet run", which says: "Hosting environment: Production"
In Chrome browser, press F12 and go to Network. And Paste http://localhost:5000/home to URL field.
Then hit Refresh sporadically. For getting the "home" or "localhost" document which is 6KB the query time is:

1st Refresh: 2.5 seconds. // Fine. First time the preparations take some time.
Hit Refresh very quickly (in every 2-3 seconds): 45msec
Hit Refresh while waiting about 5 seconds between Refreshes: 1.06 sec

So, it is 1 second in general for a random user accessing the website with some infrequency. With local webserver! Is it the best we can do?
Could you enlighten me why it is that slow or fix it please?
Thanks.

I wouldn't expect it to have a 1s delay when deployed fully, for example to Azure or to a full IIS instance. However if you need to track down the exact timings you would need to insert more logging or other instrumentation so you can get the timestamps relating to different parts of the request (e.g., when ASP.NET first starts receiving the request, when it starts making a call to Node for prerendering (if you have that enabled), when Node responds, when it starts serving the response HTML, and when it finishes serving the response HTML).
As it happens, the next version of the Angular template is structured very differently because it's based on Angular CLI. You might want to try out a preview of that version and see how it performs for you.

Probably related to keepAliveTimeout: https://github.com/aspnet/JavaScriptServices/issues/1392


Inline Swagger HTML + Danet Version Bump

When you run deno vendor the local files don't include the swagger.html.

Fixed in 1.6.0


Tab bringing up completion window and not putting spaces in.

Hello,
I just switched to Spyder, I really like it so far but there is on thing I dont like. I couldnt find the option that when I press tab it always puts 4 spaces there. It either brings up the completion window or it indents the whole line (when I have the "always indent" option enabled. Is there a way to change that? Its because I like putting spaces in some sections so its better readable, and having to press space so often is really annoying in my opinion. You can see what I mean in the picture.

Hi @PhilipeDE1, thanks for the suggestion.
I can confirm this behavior:

and there is currently no way to disable it. We should probably have an option for enabling disabling this? What do you think @spyder-ide/core-developers ?

Cheers!

Putting in an option to disable it would be good.
Would that option also disable the tab completes in the console?

Would that option also disable the tab completes in the console?

Good question!

I think the editor and console options for that should be different.

Is this ever going to be an option?  I have a long-term habit of writing a line, then tabbing over to add a comment (from my old C days).  But, when tab, a menu pops up, sometimes inserting a [seemingly] random word.  I can sometimes short-circuit this behavior by entering a space or two, but this has been driving me nuts for some time.


Allow Status Page URL To Be Configurable

Allowing the status page runbook url to be configurable using a environment variable. This allows consumers of Response to change the configuration without having to touch the keyword handlers file which might introduce conflicts with their forks down the line.
Worth noting, the commits look a little noise because of how we were pulling in master using a PR (which we've now stopped) but also to do with how we're forking and contributing changes back up using cherry picking using a branch of our master.

Ready for review @evnsio @adamlphillips

The status page handler is in there as an example of how to use the keyword handler.  I'm guessing it's not that useful to everyone, and adding a config variable feels pretty heavyweight.


Adds reproducible build assertion tests

Resolves #568
Checklist

[x] I have viewed, signed, and submitted the Contributor License Agreement.
[x] I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
[x] I have added an integration test, if necessary.
[x] I have reviewed the styleguide for guidance on my code quality.
[x] I'm happy with the commit history on this PR (I have rebased/squashed as needed).

These tests will pass once this composer-install PR get merged and a new composer-install release is cut.


Edge detection fixes

Be sure to update the client info core module docs in Propaganda.

Coverage remained the same at 94.524% when pulling 0fa23d553a7d5d35685db5485f9e4913f7e581ab on edge_detection_fixes into 6c580de05d0bd20966ff925cf8dbb617f34404b6 on master.


Error during includes file scripting: File Dialog2, File version is later than the current LabVIEW Version

While running the script includes VI on the example, I ran into this error:

LabVIEW version I tested:

2014, SP1. Version 14.0.1, 32 bit.
People shouldn't have to do this on the example. I will try running this on some other things as well.

Investigate cross linking issue.

I tried doing this again and I was not able to reproduce it.

not reproducible.


Rename gnostic OpenAPIv2 dependency

At some point, the upstream code was renamed and the capitalisation removed.
This can cause issues with Git as it doesn't detect the renames very well.
During the rebase, this was not handled correctly and the files were left as the old name within the git tree (rather unhelpfully they were correct on my system).
This should now fix the issue going forward with an explicit rename within the git history

/approve


Fix NPM errors

I see a couple of outdated NPM packages... would be worth fixing...
minimatch
graceful-fs

Found the issue, we need to upgrade to gulp4. Beware it's a diffferent package name.

Closing because doublon with #550


Local Documents and Dropbox?

It would be great if the online editor of markdown could open local documents (from the server) and as an extra could connect with dropbox.
Thank you very much.

This is indeed something I'm working towards. Opening local documents is probably coming up in the next update, after I've fixed a few bugs with the current version. Dropbox integration is also something I would like to implement at some point, though there's a few other features that have a higher priority right now.
Thank you for opening Markants first feature request :smile:

As of version 0.4.0, you can now load and save Markdown documents from your local drive.

Well done! It will now be one of my favorite web app.
Best regards.


Accept forwarded messages

Forwarded messages currently get no response for some reason. Fix it!

Oh wait, it works. Oops.


Simplified arithmetics compute

Closes #527.
The functions are now offered based on the name instead of Operator (which was removed).
Also, the functions no longer error, and instead panic. As usual, can_* can be used to check if the operation is supported, and checking the len of the arrays is required.

@yjhmelody , thanks! Could you clarify what do you mean with

the user that the type does not match, it will be better than unwrap directly

So, the design here is: the statically-typed versions of the operators do not check for the type; the dynamically-typed versions of the operators check for type.
I am uncertain about this choice, though: my hypothesis here:

query engines know the logical types associated to the arrays
they also know whether an operation is valid for a given operator (via can_*)

thus, they can match for a valid datatype via
if can_add(array1.data_type(), array2.data_type()) {
    Ok(add(array1, array2))
} else {
    // custom add for arrow2-unsupported type or error
}

Could you clarify your thoughts on this? Also @houqp @ritchie46 @sundy-li

I agree with you. I just suggest not to use the unwrap method directly, but to use the expect method

iirc unwrapping a Result and will show the result's error message on the panic.


Allow for custom user checks

Kuberhealthy could be turned into a great platform for users to easily write synthetic checks that include all the features Kuberhealthy provides today including metrics support and check lifecycle management.  It would be great if users could use some kind of SDK to build their own check container.  This would make Kuberhealthy a great facilitator for writing synthetic checks for any application.
I propose that a few elements of Kuberhealthy be created:

A new check type that executes a job or daemonset, tracks it to completion, and shows the results in the usual prometheus and dashboard interfaces
A new CRD that represents a "custom check" and the settings the custom check should use

check timeout
container URL
always/never pull container
kuberhealthy check status reporting URL
environment variables the pod should have applied

A gRPC custom check status reporting handler so that checks can report status after running
An SDK for users to easily develop checks of their own in any language.  Include application framework templates for several popular ones

The general flow of these custom checks would be:

Kuberhealthy custom check CRD is created
Kuberhealthy operator loads settings from CRD
Kuberhealthy spawns job with check and manages a timeout
Check runs and reports in to kuberhealthy status handler endpoint with gRPC
Kuberhealthy shows most recent status of custom check alongside other checks on the prometheus endpoint and JSON status endpoint

Approved

Approved. The sdk could either live in this project, or in a Kuberhealthy SDK repo

@lolimjake and I had a good design discussion for this today.  It breaks down into a few parts:

[ ] A new check will be created that manages the lifecycle of a pod from end to end
[ ] A new endpoint will be created that receives status updates from running pods and writes to the state CRD
[ ] The status endpoint will drop any requests from pods that are not in its pod name whitelist
[ ] A new CRD will be created that defines external checks, which image they run, how often, and what the timeouts are

Work on this continues in the 2.0.0 branch

This is working as a prototype in the 2.0.0 branch.  Let's switch to filing bugs.  Closing!


Sweep paper wallet for BCH

We would like Copay to provide an update on subject feature and let the community know if there are any plans to add on this function? As of now, sweeping is only applicable for BTC..

Nice idea, but you should open this ticket in https://github.com/bitpay/copay

Thanks. Is there any plan? It will make so many lives easier.
On Sat, Oct 21, 2017 at 9:07 AM, Javier Donadío<EMAIL_ADDRESS>wrote:

Nice idea, but you should open this ticket in https://github.com/bitpay/
copay
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bitpay/copay-recovery/issues/84#issuecomment-338393265,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AfFvB_Dxm3nAcM19bIaCaFOMJP65Pmrlks5sueyvgaJpZM4QBaTN
.

--
Majid Ashraf
Research Associate at NJCEP, NJIT.
Vice President Indian Graduate Association, NJIT
Department of Chemical, Biological and Pharmaceutical Engineering,
New Jersey Institute Of Technology,
East Newark, 07103-3513
Voice :<PHONE_NUMBER>)


added aria-label to delete button

👷 Deploy Preview for cg-eleventy-blog processing.

Name
Link

🔨 Latest commit
674fde73bd23cfe9f1e3dd1093dfb78fbea8d12a

🔍 Latest deploy log
https://app.netlify.com/sites/cg-eleventy-blog/deploys/65ee888bdcfdc70008c10b67


Datum round-trip corrupts datum bytes

Problem Report
The cardano-ledger code provides no guarantees about round-tripping the serialization; For example, if a datum uses fixed-length arrays in it's CBOR encoding, the cardano-ledger will re-serialize it with indefinite-length arrays.
WIth cardano-db-sync, this manifests as storing the wrong bytes for a datum.  For example, consider this datum:
https://testnet.cexplorer.io/datum/06269f665176dc65b334d685b2f64826c092875ca77e15007b5f690ecc9db1af
(cexplorer.io is powered by cardano-db-sync)
If you grab the bytes seen at the bottom of the page, they will not hash to 06269... as expected.  It turns out, this is because the bytes on the wire are different:
D8798441FFD87982D87982D87982D87981581CC279A3FB3B4E62BBC78E288783B58045D4AE82A18867D8352D02775AD87981D87981D87981581C121FD22E0B57AC206FEFC763F8BFA0771919F5218B40691EEA4514D0D87A80D87A801A002625A0D87983D879801A000F4240D879811A000FA92E

This can cause issues with down-stream applications that depend on this data, such as making the UTXO un-spendable, since the app has the incorrect datum as a witness.
We originally encountered this with Ogmios, and the lovely @KtorZ helped me figure out what was going on, and fixed it in Ogmios.  For an example of this fix, and/or a more detailed writeup, see:
https://github.com/CardanoSolutions/ogmios/commit/3f614c3cf15b6e418fd461a34853b750a3408c4f
I also opened a ticket for discussion with the Ledger team on this, seen here:
https://github.com/input-output-hk/cardano-ledger/issues/2943

Thank you for reporting. It seems reasonable to use the memoized bytes instead of redoing the serialisation both in terms of hash correctness and performance.

@kderme yes indeed. The principle that the ledger follows is to remember the original bytes and to never re-serialise, and db-sync should do the same.

Related problem in cardano-api https://github.com/input-output-hk/cardano-node/pull/3860

Datum test failed when run on DBSYNC_REV="13.0.4-rc1-fix-datum" :
>       assert (
            db_cbor_hex == orig_cbor_hex
        ), "Datum bytes in db-sync doesn't correspond to the original datum"
E       AssertionError: Datum bytes in db-sync doesn't correspond to the original datum
E       assert 'd8799f41ffd8799fd8799fd8799fd8799f581cc279a3fb3b4e62bbc78e288783b58045d4ae82a18867d8352d02775affd8799fd8799fd8799f581c121fd22e0b57ac206fefc763f8bfa0771919f5218b40691eea4514d0ffffffffd87a80ffd87a80ff1a002625a0d8799fd879801a000f4240d8799f1a000fa92effffff' == 'd8798441ffd87982d87982d87982d87981581cc279a3fb3b4e62bbc78e288783b58045d4ae82a18867d8352d02775ad87981d87981d87981581c121fd22e0b57ac206fefc763f8bfa0771919f5218b40691eea4514d0d87a80d87a801a002625a0d87983d879801a000f4240d879811a000fa92e'

I believe this has been fixed and the observed issue is actually in CLI https://github.com/input-output-hk/cardano-node/issues/4433. Feel free to reopen if not

For example, if a datum uses fixed-length arrays in it's CBOR encoding, the cardano-ledger will re-serialize it with indefinite-length arrays.

Just to clarify for posterity, the ledger does not re-serialize any user-submitted data!


Processing demo list 14 causes the geocoding process to crash

Overview
Processing demo list 14 causes the geocoding process to crash. There may be bad data in the list but whatever's in it breaks the exception handling.
Expected Behavior
geocoding list 14 succeeds
Actual Behavior
geocoding list 14 fails with the following message:
Exception ignored in: <function CPointerBase.__del__ at 0x7fbe04b19620>
Traceback (most recent call last):
  File "/usr/local/lib/python3.7/site-packages/django/contrib/gis/ptr.py", line 36, in __del__
  File "/usr/local/lib/python3.7/site-packages/django/contrib/gis/geos/libgeos.py", line 155, in __call__
  File "/usr/local/lib/python3.7/site-packages/django/utils/functional.py", line 36, in __get__
  File "/usr/local/lib/python3.7/site-packages/django/contrib/gis/geos/libgeos.py", line 159, in func
ImportError: sys.meta_path is None, Python is likely shutting down

Steps to Reproduce

upload demo list 14
parse demo list 14
geocode demo list 14

We ran into a problem before with that exception masking an actual problem with inconsistent fixture data:
https://github.com/open-apparel-registry/open-apparel-registry/pull/195#issuecomment-462849518

Likely the same as 292


Label Writer 280 prints empty by LibreOffice

hi, tried to print Libre Office -> Dymo-driver -> Cups -> Lable Writer 280 but it dose
not work.
I was selected device Lable Writer.
There was not model 280 !
It writes empty or nothifng !!!
joni

The label printer drivers are part of CUPS, please report an issue there.
See also LPrint for a newer label printer driver package.


Broken links in README.md

The README.md contains a few links returning 404:

The links to the Web, Android and iOS apps
The link to the Contributor guide

@rosariopfernandes thanks for catching this!  All fixed now.


All but type T check and assignment of T is not working

TypeScript Version:  2.4.0 / nightly (2.5.0-dev.201xxxxx)
Code

interface A {
    a: string;
}

interface B {
    b: string;
}

function isA(x: any): x is A {
    return !!x.a;
}

declare let anull: A | null;
if (anull == null) {
    anull = { a: ' ' } as A;
}
anull.a; // No error because it's A

declare let ab: A | B;
if (!isA(ab)) {
    ab = { b: ' ' } as B;
}
ab.b // Error b does not exists on type A | B

Expected behavior:
ab to be of type B
Actual behavior:
ab to be of type  A | B
http://www.typescriptlang.org/play/#src=
interface A {
    a%3A string%3B
}

interface B {
    b%3A string%3B
}

function isA(x%3A any)%3A x is A {
    return !!x.a%3B
}

declare let anull%3A A | null%3B
if (anull %3D%3D null) {
    anull %3D { a%3A ' ' } as A%3B
}
anull.a%3B %2F%2F No error

declare let ab%3A A | B%3B
if (!isA(ab))%20%7B%0A%20%20%20%20ab%20%3D%20%7B%20b%3A%20'%20'%20%7D%20as%20B%3B%0A%7D%0Aab.b%20%2F%2F%20Error%20b%20does%20not%20exists%20on%20type%20A%20%7C%C2%A0B%0A

If ab is originally of type A, then the if (!isA(ab)) branch will do nothing, and ab will still be an A at the bottom.
No compile error if you remove the ! in !isA(ab).

Sorry, the code I gave was wrong, here is the correct one:

interface A {
    a: string;
}

interface AB {
    a: string;
    b: string;
}

function isAB(x: any): x is AB {
    return typeof x.b === 'string';
}

declare let ab: A | AB;
if (!isAB(ab)) {
    ab = { a: '', b: ' ' } as AB;
}
ab.b // Error b does not exists on type A | AB

The type A | AB is effectively the same as A because we remove subtypes from unions when they have a corresponding supertype in the same union.

Is it intended when we assign a subtype, that we keep the super type?
Here is a simpler reproduce:

declare let ab: A | AB;
ab = { a: '', b: ' ' } as AB; // A | AB why?
ab.b // Error b does not exists on type A | AB

We only do control-flow based autocasts for unions, not for super/subtypes. The following does work:
declare let x: number | string;
x = 0;
2 * x;

Thanks for the explanation


Susan/check status in dropdown

Description
Display checkmarks on filter when selected in dropdown
Type of change

[x] New feature (non-breaking change which adds functionality)

Test and Reproduce
Outline the steps to test or reproduce the PR here.

Impacted Areas in Application
List general components of the application that this PR will affect:
Which pages?

[ ] All Projects Page

Specific components

Testing Script:
###All Project Page

Switch between projects
Search for a sample
Navigate through different pages of samples
Select Upload new sample button
Select sample to see generated report

Checklist:

[x] I have run through the testing script to make sure current functionality is unchanged
[x] I have done relevant tests that prove my fix is effective or that my feature works
[x] I have spent time testing out edge cases for my feature
[x] I have updated the test script or pull request template if necessary
[x] New and existing unit tests pass locally with my changes

this seems to be in conflict with lazyloading a bit though since url no longer shows the filter=""

hey @yunfangjuan, could u review this with existing master code so that i can merge lazy loading to it?


[Python] FizzBuzz

@amanmehara, I have added the file for FizzBuzz and made sure that they satisfy the guidelines given and optimized the code for readability, added comments, and made it suitable for further addition of numbers other than 3 and 5.

I have made the required changes. Thanks for giving me the opportunity to contribute.

Hey @amanmehara, please check out the changes that I have made and approve this pull if they satisfy all the criteria. If I must make any further changes, please inform me of the same.


Unclickable links in chrom(ium)

I'm running debian, in both chrome and chromium (but not iceweasel/firefox) anchors on the template are not clickable for up to one second after another anchor has been clicked. This applies to the language tabs on the top left which do not change the page upon rapid clicking (rapid being about .5 seconds apart). This also applies to headings on the left which when clicked rapidly give no indication of having been pressed. This can be tested on the Gem API website
At the same time, somehow Stripe fixed this issue for their site.

Hmm, I can't seem to reproduce the error on my Mac...how difficult would it be to record a video of this behavior?
(Also, Stripe doesn't use Slate :smile:)

Closing due to inactivity and because I couldn't reproduce, but feel free to reopen @Firescar96 if you're still having problems!


Typescript Resolvers Plugin: Ability to provide a custom visitor

In Redwood, we need the ability to slightly modify how the TypeScriptResolversVisitor works, while keeping all of the other functionality within the typescript-resolvers plugin as is.
Describe the solution you'd like
Ability to pass a customResolver in the config via an import string
Describe alternatives you've considered

passing visitor when we instantiate the plugin (but this does not match existing patterns of configuring codegen plugins

Additional context
See draft PR for discussion

Describe alternatives you've considered
We've also considered calling the original typescript-resolver plugin (with the original visitor) as it is, and then calling our own visitor. And then trying to surgically replace specific parts of the output string from the original plugin with the output from our custom visitor. But it feels inefficient and prone to breaking when the output from the original plugin or visitor changes.


In ngTagsInput how to show validation message for different cases?

I am using ngTagsInput Angular plugin for getting multiple email ids.
In my code 3 validation has been added those are as follows:
1.For Mandatory fields.
2.Field should accept only an email id.
3.It should not allow duplicate email id.
The above cases are working fine. But, I want to show the error message dynamically according to the above one of the case has occurred. Please help me out !!!

ngTagsInput uses ngModelController to set form's validity, so you can validate it as you would do with any form. Take a look at this post.

Don't get it how to validate ng-tags using ng-messages, required for example. Can you help me?

Sorry, my fault, your directive accept name attribute. Thanks a lot, awesome code.

But when you add required == true. By default the control is in Error state.
When I checked in in DOM, the state was ng-invalid ng-invalid-required, even though I havent touched the element. Seems strange. Can you help me?
Thanks,
-/Sammi

This issue was selected to be spring-cleaned by a script. More information can be found here.
If anyone thinks this was a mistake and this issue should be reopened, please leave a message below explaining why. Before doing so, please consider reading the CONTRIBUTING file.


it doesn't take altitude into account?

Thanks Alonso. I have a query regarding removing and addding geo coordinate from AR view because my geocoordinates long, lat, altitude cahnges over time. the problem i am having is its not removing the view, instead its stacking on top of the same view.
            location: CLLocation.init(coordinate: flightAnnotation.coordinate, altitude: flightAnnotation.altitude, horizontalAccuracy: 0, verticalAccuracy: 0, course: flightAnnotation.heading , speed: flightAnnotation.speed, timestamp: NSDate(timeIntervalSinceNow: timeInterval) as Date))
        arGeoCoordinate.dataObject = flightAnnotation.title
        arGeoCoordinate.calibrate(usingOrigin: userLocation, useAltitude: true)

        if let annGeo = flightAnnotationsGeo[flightAnnotation.title!]{
            arKitEngine?.remove(annGeo)
        }
        arKitEngine?.add(arGeoCoordinate)

Can you help why its not remoing teh geo coordinates from ar view

plus in landscape orientation the image bubble doesn't rotate?

Orientation changes are currently not supported while in AR view. When the engine is started, everything is computed based on the current orientation.

Closing as idle. Please reopen if necessary.


[Tabular] Quantile regression minor fix  and aggregation model

*Description of changes:

Add a neural network based quantile aggregation (ensemble) model. (cc @jwmueller)

TabularQuantileAggregatorModel: PyTorch based, aggregation (ensemble) model.
This is wrapped by LinearAggregatorModel and is only activated when self.bagged_mode is True.
Hyperparameter tuning is available.

Also minor fix for bugs in quantile regression models.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Job PR-1180-1 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-1180/1/index.html

Job PR-1180-7 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-1180/7/index.html

Job PR-1180-10 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-1180/10/index.html

Job PR-1180-14 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-1180/14/index.html

looking forward to trying this out!

Taesup & I are doing a final pass to make sure nothing's changed since, but if not we will probably aim to merge it asap

Job PR-1180-16 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-1180/16/index.html

Job PR-1180-18 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-1180/18/index.html

This PR is pending.


Autoclose address autocomplete list

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/6235
Tests | QA Steps

Go to Bank account, Company Information step.
Search for an address on Company Address field.
Autocomplete List should show up.
Click somewhere outside the autocomplete list. the list should close.
Click back to Company Address field. The list should show up.
Now press Tab on your keyboard.
Focus should move away from Company Address field.
Click somewhere outside the autocomplete list. the list should close.
Click back to the Company Address field. The list should show up.
Now press the tab the number of times equal to the items shown on the autocomplete list.
As soon as you press the Tab one more time, the focus should move to the next field Phone number and list should close.

[x] Verify that no errors appear in the JS console

Tested On

[x] Web
[ ] Mobile Web
[x] Desktop
[ ] iOS
[ ] Android

Screenshots

Web

https://user-images.githubusercontent.com/24370807/149243519-f378dc60-87f8-42e4-8083-37d35bfeeb1a.mp4
Mobile Web

Desktop

Not able to test it on the development server due to CORS errors.
iOS

Android

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

@NikkiWines Can I apply for the same job to get compensated?

Yep. @kevinksullivan, @parasharrajat ended up fixing https://github.com/Expensify/App/issues/6235 by forking the repo and implementing a solution on the fork. Can we hire him for that job and issue payment for it once this PR has been on prod for a week?

🚀 Deployed to staging by @NikkiWines in version: 1.1.31-2 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by @AndrewGable in version: 1.1.32-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Add Mistral 7b

Is your feature request related to a problem? / 你想要的功能和什么问题相关？
May be that addition will be great
Describe the solution you'd like. / 你想要的解决方案是什么？
Mistral 7b hugging face...
Describe alternatives you've considered. / 你考虑过的其他方案是什么？
No response
Additional context / 其他信息
No response

https://huggingface.co/mistralai/Mistral-7B-Instruct-v0.1

What about this one?

Any progress in this model?


Identifiers validation failed for dataset ibbi observations

Identifier validation failed for the dataset ibbi observations:

Crawler attempt: 7
Publishing organization: Blåst Spel AS
Publishing country: NORWAY
Installation: ibbi HTTP Installation
Number of occurrences indexed for this dataset at the time of the issue: 1
Cause: GBIF ID problems exceed 50% threshold: 100% duplicates; 2 total records; 2 absent records
Pipelines execution steps: http://api.gbif.org/v1/pipelines/history/execution/3809025/step

New IDs sample:

Old IDs sample:

GAIA:BU37j27gZYniB2eNQ4g3

Publisher email
Hello,
I am contacting you from the GBIF Secretariat about a dataset published by the ibbi observations : https://doi.org/10.15468/m94g82.
We noticed that the occurrenceIDs were changed. We have temporarily paused the ingestions of this dataset.
As you might already know, when an occurrence record has a new occurrenceID for a given dataset, our system considers it to be a new occurrence.
This means that it will be given a new gbifid and a new occurrence URL (like this one: https://www.gbif.org/occurrence/1252968762) and the old gbifid and URL will be deprecated.
In this case, this means that the occurrence URLs would be deprecated when ingesting the newest versions of these datasets.
We would like to check with you if those changes were intentional. Do you know if this is the case?
Please let us know, thanks! We are happy to resume the dataset ingestion.
Note that some users rely on those occurrence URLs and gbifids (like https://bionomia.net for example). In an attempt to improve
the stability of the occurrence URLs and gbifids, we have implemented a warning system to detect these type of changes in datasets (see this news item).
If the data publisher can provide us with a list of old and new occurrenceIDs per record, we can avoid the identifier and URL changes. Could that be an option?
Please let us know if you have any question. Thanks!
All the best,

You can skip/fix identifier validation using the registry UI.

Identifier validation still  failing for the dataset ibbi observations:

Crawler attempt: 9
Publishing organization: Blåst Spel AS
Publishing country: NORWAY
Installation: ibbi HTTP Installation
Number of occurrences indexed for this dataset at the time of the issue: 1
Cause: GBIF ID problems exceed 50% threshold: 100% duplicates; 2 total records; 2 absent records
Pipelines execution steps: http://api.gbif.org/v1/pipelines/history/execution/3809032/step

New IDs sample:

Old IDs sample:

GAIA:BU37j27gZYniB2eNQ4g3

Publisher email
Hello,
I am contacting you from the GBIF Secretariat about a dataset published by the ibbi observations : https://doi.org/10.15468/m94g82.
We noticed that the occurrenceIDs were changed. We have temporarily paused the ingestions of this dataset.
As you might already know, when an occurrence record has a new occurrenceID for a given dataset, our system considers it to be a new occurrence.
This means that it will be given a new gbifid and a new occurrence URL (like this one: https://www.gbif.org/occurrence/1252968762) and the old gbifid and URL will be deprecated.
In this case, this means that the occurrence URLs would be deprecated when ingesting the newest versions of these datasets.
We would like to check with you if those changes were intentional. Do you know if this is the case?
Please let us know, thanks! We are happy to resume the dataset ingestion.
Note that some users rely on those occurrence URLs and gbifids (like https://bionomia.net for example). In an attempt to improve
the stability of the occurrence URLs and gbifids, we have implemented a warning system to detect these type of changes in datasets (see this news item).
If the data publisher can provide us with a list of old and new occurrenceIDs per record, we can avoid the identifier and URL changes. Could that be an option?
Please let us know if you have any question. Thanks!
All the best,

You can skip/fix identifier validation using the registry UI.

Was let through because it is two occurrence records.


chore(merge-back): 2.39.0

See CHANGELOG

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: 316fb1580253f1def75d033ec5386c6c29744197
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


[graphql-playground-react] Change 'Hit the Play Button' to 'Press the Play Button'

Changes proposed in this pull request:

Updates empty state to use less violent verb 'play' rather than 'hit' (we have a client who commented on this and there doesn't seem to be any way to modify this phrase without forking).

Updates jest snapshots to match

Please let me know if you have any concerns. Thanks!

cc @timsuchanek @acao

@prichey thanks for this!


eigen3 correction

@vigsterkr UPDATE: I applied the fix to develop as it is definitely necessary. The rest is optional for cleaner cmake options.
I renamed them to start with LINALG_ so that they are grouped if ppl check the options and not scattered all over without it being clear that they change linag behaviour. Also, ccmake which shows the cached options now does't show a binary ON/OFF, but the string. Just like CMAKE_BUILD_TYPE
This might interfere with some builds. Let me know

mmh, no travis-ci somehow

travis is green

The viennaCL buildbot requires to change DLinAlgBackend=VIENNACL .. to LINALG_DEFAULT_BACKEND=VIENNACL ..

@karlnapf any reason why not to merge this?

needs a  buildbot change, see comment above. I would prefer this.
Not important as it is refactored anyways. Merge if it is easy to change buildbot scripts, otherwise just close

i've merged and updated the buildbot params for the viennacl job, i'm getting the following output from cmake
-- ViennaCL set as default global linear algebra backend library
-- Eigen3 set as default core module (linalg) backend library
-- Eigen3 set as default reduction module (linalg) backend library
-- Eigen3 set as default eigen solver module (linalg) backend library

@karlnapf why Eigen3 is still the eigen solver module?

This has to be fixed

I guess this can wait until we re-structure no?


Under Multiple goroutines, when using Empty and NewCustom method in same time，connection will enter the pool

the condition is below:
One app has multiple goroutines to deal with many requests.  Every request operates the redis. After Issues happen in  the network or environment, every request operate the redis falied and every time will re-init redis pool. Before re-init redis pool, one goroutine in the app firstly invoke Empty() and then NewCustom(). So Empty() can't close all connection in pool in one goroutine and the other goroutine will add connection into pool. because the code in pool.go below:
mkConn := func() error {
client, err := df(network, addr)
if err == nil {
p.pool <- client
}
return err
}
if size > 0 {
	// make one connection to make sure the redis instance is actually there
	if err := mkConn(); err != nil {
		return &p, err
	}
}

// make the rest of the connections in the background, if any fail it's fine
go func() {
	for i := 0; i < size-1; i++ {
		mkConn()
	}
	close(p.initDoneCh)
}()

i think the bold block code should replace p.Put(client) with p.pool <- client. It ensure  when one goroutine invoke empty() ,anothers can't invoke NewCustom() to add client into the pool.

Hi @steedyu, thanks for submitting an issue here. I'm afraid I don't fully understand your problem. NewCustom makes a completely new Pool instance, which will not effect the first Pool in anyway, no matter what it's doing. I'm confused what you mean by:

So Empty() can't close all connection in pool in one goroutine and the other goroutine will add connection into pool.

Also, just a heads up, that radix.v2 is no longer really supported, so I won't be adding any new features to it. All future development is being done on radix/v3, which is faster, more feature-rich, and more flexible. Thanks!

Ok. Thank you for your reply. I will try to use  radix/v3.


Typo in docs / closure-actions.md

Shouldn't this be this.attrs.boom() instead of this.attr.boom()!?

Hey @nfc036 , attrs is correct. This is glimmer specific thing as you can see here https://github.com/tildeio/glimmer/blob/1759c16defc546b034b97e37141187652ed93859/packages/%40glimmer/runtime/lib/component/interfaces.ts#L87

So I would kindly suggest to merge this pull request to correct the type in the docs.

Ah, kill me. I misread it..

;-)

Thanks @nfc036 and sorry for the confusion.