Missing documentation to make it work

Hi,
I'm just discovering the complex BUFR file format supplied by meteofrance as public radar data, and I'm far away from my comfort zone (web development & GIS "standard" formats).
For a few days I've been trying to find tools that will allow me to convert data to more friendly data formats.
Sadly, the local tables supplied by meteofrance only work with the OPERA tools, and so far it hasn't been a very good experience.
I'm trying to work with tools based on eccodes, in particular some Python librairies that look interesting (pybufr, eccodes-python).
So I tried to find a way to convert CSV files to eccodes definitions, but I discovered that it was a big problem for a lot of people and projects around the world.
Then I found your repository, and it was the best basis I can find for completing this task. Thanks for sharing your work.
I was able to understand how a few scripts work (convert_tableb_to_element.py, convert_tabled_to_sequence.py), but I can't go any further with my humble knowledge at the moment. I'm missing some documentation.
I had to add some error handling for an error caused by wrong data in one CSV file. 😔 Instead of valid data the meteofrance CSV contained '[GC threshold exceeded with 12,014,472 bytes in use.  Commencing GC.]', '[GC completed with 277,360 bytes retained and 11,737,112 bytes freed.]' and '[GC will next occur when at least 12,277,360 bytes are in use.]'.
With some guidance, I'll be glad to work on this interesting project and add some documentation too.
From what I've read in the last few days, this repo should be better known because it should be of interest to a lot of people. I was surprised to see so many people struggling with these BUFR files from meteofrance.

Hi,
I feel your pain.
This repository was setup just to keep my code somewhere safe. It is not pretty nor documented because it was basically a one time thing.
After I created the tables, I manually edited the items I actually use to have nice names, as the default is to create a name like entry004194.
I didn't upload the tables themselves because the copyright of the original tables are not clear and the work has to be done to rename every single item without clash.
I am happy to accept your help in documenting the code, and if we can legally post the resulting table, I am happy to do it also.

Thanks for your quick reply and for your positive response.
I had deduced that it was a repo for personal use, and the code didn't seem so ugly. 😉
It was quite clear how the code worked, my principal problem is to deduce the arguments and the paths to use.
As I'm just starting out with the BUFR format, it's more difficult to solve questions of this kind.
With your help we should be able to explain everything's needed to get the eccodes definitions. 🤞
I'll try to find more information about the licences, I haven't looked into it.
I have no precise idea of the best way to move forward with the project, but I can propose that I list what I've deduced and the points where I'm at a loss.
Or I let you list the steps needed to convert all CSV files, then I'll create the README content and add comments in the code.
I'll let you choose what suits you best.

I don't remember the steps anymore. The list should be best, so I can see exactly what you are doing and where you hit a snag.

Okay, I'll do my best to get you back on track. 😉
Folder structure and filenames
First, based on the definitions from the repo, I assumed the CSV files had been updated since.
It increased the difficulty to deduce the result in the repo folder definitions/ from different source files.
Here's a link to the current tables supplied by meteofrance : tables_bufr_361.
bufrtabb_11.csv
bufrtabb_13.csv
bufrtabb_16.csv
bufrtabd_11.csv
bufrtabd_13.csv
bufrtabd_16.csv
localtabb_247_8.csv
localtabb_85_12.csv
localtabb_85_14.csv
localtabb_85_20.csv
localtabd_247_8.csv
localtabd_85_12.csv
localtabd_85_14.csv
localtabd_85_20.csv

From the eccodes documentation about local configuration, I was able to understand vaguely the folder structure:
definitions/bufr/tables/[masterTableNumber]/local/[localTablesVersionNumber]/[bufrHeaderCentre]/[bufrHeaderSubCentre]

Hypotheses
(may be wrong)

masterTableNumber is always 0
localTablesVersionNumber represents the different versions of a table
bufrHeaderCentre is the center producing the data, meteofrance is represented as 85
bufrHeaderSubCentre seems to always be 0 (I'm trying to get more information about that)
the source file bufrtabb_11.csv seems to match the local table 0 version 11 for meteofrance (85), so the converted file should be located in tables/0/local/11/85/0/
the command python convert_tableb_to_element.py tables_bufr_361/bufrtabb_11.csv will print the content of the bufrtabb_11.csv as an element.table. The file is not created by the script so it's necessary to redirect output to the file.
the command python convert_tabled_to_sequence.py tables_bufr_361/bufrtabd_11.csv will print the content of the bufrtabd_11.csv as a sequence.def. The file is not created by the script so it's necessary to redirect output to the file.
convert_codetable_to_eccodes.py and copy_codetables.py should be used to create the files in the codetables subfolders, but I was not able understand how to do that (I tried a lot of things, nothing worked...).
the script update_entry_names.py is not necessary to create the eccodes definitions, but is used to give more meaningful names to the entry.... fields
the script is not able to generate all the files and the folder structure in one shot for now, and it's necessary to make multiple operations manually

First questions
(more to come I suppose)

what is wrong in my hypotheses ?
can you specify the sys.argv[] arguments in the different scripts ?
how to generate the codetables subfolders ?

* what is wrong in my hypotheses ?

You got all correct.

* can you specify the `sys.argv[]` arguments in the different scripts ? I couldn't figure out how to define these variables correctly : `refcsv`, `basecsv`, `basepath`, `outpath`, etc.

These are parameters for the script itself, i.e: copy_codetables.py reference basepath/ outpath/
Maybe I didn't understand your question.

* how to generate the `codetables/` files ? I tried to execute `convert_codetable_to_eccodes.py` with different CSV as source file, then I tried a generated `element.table`, trying to find a match with the `get_fields()`function. But nothing worked.

* the files in the repo definitions `codetables/` subfolders contain content I can't find in the current CSV files. Perhaps the source files used to generate these tables contained this data, but I doubt it. For example, the file `definitions/bufr/tables/0/local/11/85/0/codetables/1192.table` contains `Composite complete`, `Composite francaise`, `Composite Royaume-Uni Irlande`, `Composite suisse`, etc. Where does this data come from?

The codetables are made using other files. They are btc085.010,  btc085.011 and  btc085.012. I don't have a link to them. I need to clarify the copyright before I upload them.

You got all correct.
That's good news, thank you. When I work on documentation I'll try to explain all that clearly.

The codetables are made using other files. They are btc085.010, btc085.011 and btc085.012. I don't have a link to them. I need to clarify the copyright before I upload them.
Everything's clear now, some files used by the scripts are not in the repo ! I hadn't considered this possibility when I was trying to understand how it works. After reading your reply I looked everywhere on the web for them, but they were impossible to find. They do not seem to be made available by meteofrance. If these files are not shared by meteofrance, it seems difficult to go further because I suppose these files are updates, like the tables of descriptors ?

Maybe I didn't understand your question.
Sorry, that wasn't clear. I couldn't figure what file the reference was, or what basepath I should use (the folder containing the OPERA CSV ? or one of definitions folder ? or one folder from eccodes default definitions ?)
Thanks to your answer I understood that some files were missing, so it was logical that I couldn't figure what the reference file was.

I started looking about licences for the tables of descriptors. On the public data page where public radar API are listed and where tables of descriptors can be downloaded, there is a mention about public data licence : Royalty-free under Etalab open license. The source to indicate is "Météo-France". Some suggestions: "Source: Météo-France" or "Information created from Météo-France data".
In this licence we can read:

You are free to reuse the "Information":

Reproduce, copy, publish and transmit "the Information";
Broadcast and redistribute "the Information";
Adapt, modify, extract and transform the "Information", in particular to create "Derived Information";
Exploit the "Information" commercially, for example by combining it with other with other "Information", or by including it in your own product or application.

You need to :

Mention the authorship of the "Information": its source (at least the name of the "Producer") and the date of its last update.
The "Re-user" may fulfill this condition in particular by indicating one or more hypertext link(s) (URL) to the "Information", ensuring effective acknowledgement of its authorship. This mention of authorship must neither confer an official character on the re-use of the "Information", nor suggest any recognition or endorsement by the "Producer", or by any other public entity, of the "Re-user" or of its re-use.

Thank you, I'll see if I can make it work now.
Do you remember how you found those unfindable files ? It would have been great to be able to put a link in the documentation. 🤔
I just realized that the data from these reference tables must be present somewhere in the OPERA software suite to allow correct decoding of meteofrance data, if the tables supplied by meteofrance don't contain this data. 🤔
I'll make some research.

Thank you, I'll see if I can make it work now. Do you remember how you found those unfindable files ? It would have been great to be able to put a link in the documentation. 🤔
I got from MetroFrance directly per email.
I just realized that the data from these reference tables must be present somewhere in the OPERA software suite to allow correct decoding of meteofrance data, if the tables supplied by meteofrance don't contain this data. 🤔 I'll make some research.
They must exist, it would make no sense not to have them available somewhere, as they are integral to encoding and decoding the BUFR files.

Hi,
In the last few days I had to work on other tasks, so I'm going back to work now.
I've still taken the time to read some of the OPERA and eccodes documentation, and I'm beginning to understand some of the BUFR elements, but I still have a lot to learn. A clever but complex format...
I also sent an email to meteofrance to ask if they have a solution to read their BUFR format with eccodes, maybe things have changed since the data was made publicly available (january 2024).
Can you confirm I'm heading in the right direction ? This is what I plan to do :

Generate element.table + sequence/def files from *.csv

bufrtabb_11.csv -> tables/0/local/11/85/0/element.table
bufrtabb_13.csv -> tables/0/local/13/85/0/element.table
bufrtabb_16.csv -> tables/0/local/16/85/0/element.table
bufrtabd_11.csv -> tables/0/local/11/85/0/sequence.def
bufrtabd_13.csv -> tables/0/local/13/85/0/sequence.def
bufrtabd_16.csv -> tables/0/local/16/85/0/sequence.def
localtabb_247_8.csv -> tables/0/local/8/247/0/element.table
localtabb_85_12.csv -> tables/0/local/12/85/0/element.table
localtabb_85_14.csv -> tables/0/local/14/85/0/element.table
localtabb_85_20.csv -> tables/0/local/20/85/0/element.table
localtabd_247_8.csv -> tables/0/local/8/247/0/sequence.def
localtabd_85_12.csv -> tables/0/local/12/85/0/sequence.def
localtabd_85_14.csv -> tables/0/local/14/85/0/sequence.def
localtabd_85_20.csv -> tables/0/local/20/85/0/sequence.def

Generate code tables then copying the code tables in the matching folder based on the element.table in this folder.
Update entry name

I'll try to generate all folders and files automatically in one shot.

I encounter one warning during codetables generation, I suppose you encountered it too ?
python convert_codetable_to_eccodes.py .local/btc085.012
8202.table: Mismatched entries
{'filename': '8202.table', 'count': 14, 'items': {'0': 'Brute', '1': 'Adaptation statistique', '2': 'Prevision reactualisee', '3': 'Prevision validee par un previ', '4': 'Interpolation spatiale filtree', '5': 'Adaptation statistique filtree', '9': 'Prevision "AS BEST"', '16': 'Safran analogue', '17': 'Safran/Aladin', '20': 'Sympo horaire', '21': 'Sympo quotidienne', '31': 'VALEUR MANQUANTE'}}

I don't have a good understanding of the numbers in the list, but thanks to your logs I can see that 14 entries are needed, 14 entries are present in the file for the table 8202, but 3 entries start with a 16 (id?). So the 16 key in the dictionary is overwritten and only the last value is present.
I assume this is an update rather than an error, do you have any information about that ?
# file btc085.012, line 167

 008202  14  0 [C5 ] Type de prevision
          0  1 Brute
          1  1 Adaptation statistique
          2  1 Prevision reactualisee
          3  1 Prevision validee par un previ
          4  1 Interpolation spatiale filtree
          5  1 Adaptation statistique filtree
          9  1 Prevision "AS BEST"
         16  1 Safran/Arpege
         17  1 Safran/Aladin
         16  1 Safran
         16  1 Safran analogue
         20  1 Sympo horaire
         21  1 Sympo quotidienne
         31  1 VALEUR MANQUANTE

One last question : I looked into the btc085.01x files. The files seemed to contain the same data. I was able to confirm that with the diff command. I suppose the files were generated from a database at different times, but the data didn't change between each release.

I am traveling and it is a bit hard to work on this right now, but you are
correct on all your assumptions.
The warning is due to the duplicate value. If the table is from WMO, either
it was already broken or MF modified it.
I used what MF sent me to assemble the required files, but if you found the
originals, the better! I am not 100% sure that we need to duplicate these
tables. If eccodes does lookup in the standard tables to decode the meaning
of the item, then no duplication is better. I used what they sent me
because I assumed that they extended these tables, and on that case, local
definition takes precedence over the global ones.
On Tue, Jun 25, 2024, 22:15 Mathieu HAAGE @.***> wrote:

I'm understanding that bufrtab*.csv supplied by meteofrance are in fact
"official" tables from WMO, and the definitions are already present in the
ecCodes definitions. These tables must correspond to the
masterTablesVersionNumber in the BUFR headers, and meteofrance local
tables are correspond to the localTablesVersionNumber.
I'll try to generate ecCodes defintions for these files with your script
then I'll do a diff with the files already present in the ecCodes
definitions folders : definitions/bufr/tables/0/wmo.
If this is confirmed, there will be 6 fewer files to generate and my
understanding of the subject will have increased a little more. 🤞
—
Reply to this email directly, view it on GitHub
https://github.com/ReallyNiceGuy/eccodes_local_tables/issues/1#issuecomment-2188927770,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABR6CQTJEAXEKXNQKSELRFTZJFUOHAVCNFSM6AAAAABJNX5IESVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOBYHEZDONZXGA
.
You are receiving this because you commented.Message ID:
@.***>

Once again thank you for your help, your validation is all I needed.
I don't expect you to be working on it at the moment. 😉
Have a nice trip, I'll keep digging.

Thank you.
I am available to help again.

Hi,
I'm sorry, after a rather dynamic start I can't find time to work on the project right now.
I am finishing renovating my house so I can move in soon.
I hope to pick up where I left off very soon.


Keep rendering components after error in prod

In production mode, don't stop rendering components after finding that
one is missing. Instead render a no-op (but overridable component) and
log an error.
[fix #36]
@patric-eberle could you try this branch out?
I think this can be a minor release but I'm not sure. There is a chance that someone could be using the error message from Ember Islands in production mode by extending the ember-islands component and overriding the didInsertElement hook.

@mitchlloyd I will try to test your solution somewhen this week.

Thanks for reviewing! Released as version 1.2.0.


No update possible due to missing button

Hello Devs, today i noticed that there is no button to upgrade the adapter. I'm not sure if this is a problem of iob or the adapter.
Best regards.
Plattform:
linux
Betriebssystem:
linux
Architektur:
x64
CPUs:
4
Geschwindigkeit:
3295 MHz
Modell:
Intel(R) Core(TM) i5-8259U CPU @ 2.30GHz
RAM:
15.4 GB
System-Betriebszeit:
00:13:32
Node.js:
v18.17.1
time:
1718267911911
NPM:
9.6.7
Adapter-Anzahl:
624
Datenträgergröße:
58.8 GB
Freier Festplattenspeicher:
16.7 GB
Aktive Instanzen:
55
Pfad:
/opt/iobroker/
Betriebszeit:
00:04:23
aktiv:
_nodeCurrent:
18.17.1
_nodeNewest:
18.20.3
_nodeNewestNext:
18.20.3
_npmCurrent:
9.6.7
_npmNewest:
9.6.7
_npmNewestNext:
10.7.0

You do not note the admin version in use. Most likely you use adminn7. This is a know Bug at admin 7.0.1. Please see admin issues  Admin 7.x.x. has been revoked..
Please downgrade to latest admin Version  (6.x.x).  This shizld solve the problem.

Oh ok thank you very much. Now it works. Wasn't aware of this bug.
👍🏻


Add "Heal" function to the modeler for access via Q3D to the API

Description of the feature
Sometimes model healing is needed for an analysis to be able to be completed. The options are available in the GUI and work but I haven't been able to access them via API for the modeler in Q3D.
I only noticed that a colinear_healing method is available for HFSS 3D Layout.
Steps for implementing the feature
No response
Useful links and references
No response

Hi @jefebalconete,
We have the heal option but at the moment is only available for Maxwell2D and Maxwell3D (as well as the simplify option).
I need to refactor that method in order to make it available for different design types.
This fix will be most probably available in the next release.
Thanks for letting us now,
Regards,
Giulia


create a QR code on pdf

https://github.com/codemasher/php-qrcode (PHP7!)
I want to have a QR code at the bottom of the pdf file. A short descriptio like see more about this vehicle on top.
I ask myself how to implement such a thing

done via several tickets


PyTorch 1.0 from a nightly release not found

Hello! I think one of the prerequisites, PyTorch 1.0 from a nightly release, is not available any more, or at least I couldn't find it. Would you provide an available source or some alternatives?

Hi I recently tested on higher PyTorch version v1.5 and it works fine.


Unable to install dispatch locally

Describe the bug
When trying to install dispatch i get the following error
Traceback (most recent call last):
File "/usr/local/bin/dispatch", line 8, in
sys.exit(entrypoint())
File "/usr/local/lib/python3.9/site-packages/dispatch/cli.py", line 794, in entrypoint
dispatch_cli()
File "/usr/local/lib/python3.9/site-packages/click/core.py", line 1130, in call
return self.main(*args, **kwargs)
File "/usr/local/lib/python3.9/site-packages/click/core.py", line 1055, in main
rv = self.invoke(ctx)
File "/usr/local/lib/python3.9/site-packages/click/core.py", line 1654, in invoke
super().invoke(ctx)
File "/usr/local/lib/python3.9/site-packages/click/core.py", line 1404, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/usr/local/lib/python3.9/site-packages/click/core.py", line 760, in invoke
return __callback(*args, **kwargs)
File "/usr/local/lib/python3.9/site-packages/dispatch/cli.py", line 23, in dispatch_cli
configure_logging()
File "/usr/local/lib/python3.9/site-packages/dispatch/logging.py", line 6, in configure_logging
level = LOG_LEVEL.upper()
AttributeError: 'int' object has no attribute 'upper'
Cleaning up...
This is done locally on a mac

Did you pull the latest version of the code. This should have been fixed in https://github.com/Netflix/dispatch/pull/2544/

I just did yes

well sorry I was using git clone https://github.com/Netflix/dispatch-docker.git

which is mentioned here https://hawkins.gitbook.io/dispatch/installation

Change latest in https://github.com/Netflix/dispatch-docker/blob/master/docker-compose.yml#L22 with master. The changes mentioned above haven't been released yet.


fix: backport v5 operationID accessor changes

🚥 Fix #507

🧰 Changes
When working on v5 I made a lot of changes to how code snippets are generated to cleanup operationID accessors that can't be cleanly used as a method accessor, like get-a-pet into getAPet. The problem is that I shipped these code snippet changes to production prematurely as our v5 work is still in an open beta.
This work here backports those changes to the v4.x release series so folks coming to api for the first time from a production code snippet.
🧬 QA & Testing
See the couple tests I added but to test this locally you can load into packages/api and run this code:
const sdk = require('./src')({
  openapi: '3.0.0',
  info: {
    version: '1.0.0',
    title: 'Single Path',
  },
  servers: [
    {
      url: 'https://httpbin.org',
    },
  ],
  paths: {
    '/anything': {
      get: {
        operationId: 'get-pet',
        responses: {
          200: {
            description: 'OK',
          },
        },
      },
    },
  },
});

(async () => {
  await sdk.getPet().then(console.log);
  await sdk['get-pet']().then(console.log);
})();

You will see a screen like this:

What you won't see are any "that operation ID could not be found" errors about the first sdk.getPet() call.

Published directly to v4.5.2 from here. https://github.com/readmeio/api/releases/tag/v4.5.2


Usb port disappears

on reset of the board  - USB Serial Port Com4 shows up
couple second-later it disappears
cannot load the  ESP

it is running ...  How do I stop it ??
/* Hello World Example
This example code is in the Public Domain (or CC0 licensed, at your option.)
Unless required by applicable law or agreed to in writing, this
software is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR
CONDITIONS OF ANY KIND, either express or implied.
*/
#include <stdio.h>
#include "sdkconfig.h"
#include "freertos/FreeRTOS.h"
#include "freertos/task.h"
#include "esp_system.h"
#include "esp_spi_flash.h"
void app_main(void)
{
printf("Hello world!\n");
/* Print chip information */
esp_chip_info_t chip_info;
esp_chip_info(&chip_info);
printf("This is %s chip with %d CPU core(s), WiFi%s%s, ",
        CONFIG_IDF_TARGET,
        chip_info.cores,
        (chip_info.features & CHIP_FEATURE_BT) ? "/BT" : "",
        (chip_info.features & CHIP_FEATURE_BLE) ? "/BLE" : "");

printf("silicon revision %d, ", chip_info.revision);

printf("%dMB %s flash\n", spi_flash_get_chip_size() / (1024 * 1024),
        (chip_info.features & CHIP_FEATURE_EMB_FLASH) ? "embedded" : "external");

printf("Minimum free heap size: %d bytes\n", esp_get_minimum_free_heap_size());

for (int i = 10; i >= 0; i--) {
    printf("Restarting in %d seconds...\n", i);
    vTaskDelay(1000 / portTICK_PERIOD_MS);
}
printf("Restarting now.\n");
fflush(stdout);
esp_restart();

}

USB CDC is also a task for ESP32. It restarts or resets to re-enumerate.

but I'm in a lockup system - cannot access the USB port because the system is constantly rebooting  as shown in the program above
how do I break out of this program if my USB is constantly being reset USB CDC is also a task for ESP32. as you say ....it restarts or resets to re-enumerate.

Press and hold the BOOT button, then short press the reset button, and then release the BOOT button to enter the BOOTLOADER mode.

### Thank you


[Bug]: Azure Playground TypeGraph window is showing error

Describe the bug
In Azure playground, it is showing error in TypeGraph window.
Something went wrong:
TypeError: Cannot convert a Symbol value to a string
Try again

Reproduction
https://azure.github.io/typespec-azure/playground
Checklist

[X] Follow our Code of Conduct
[X] Check that this issue is about the Azure libraries for typespec. For bug in the typespec language or core libraries file it in the TypeSpec repo
[X] Check that there isn't already an issue that request the same bug to avoid creating a duplicate.
[X] The provided reproduction is a minimal reproducible example of the bug.

Not seeing this when I follow the link

I repro, if you pick the arm samples and load the type graph

Just an issue with union with no variants. Repro in the core playground here https://typespec.io/playground?c=dW5pb24gRm9vIHsKICBzdHJpbmcsCiAgaW50MzIKfQ%3D%3D&e=%40typespec%2Fopenapi3&options={}


fix: Code is required is displayed in the wrong field

Hi @Transmigration-zhou. Thanks for your PR.
PRs from untrusted users cannot be marked as trusted with /ok-to-test in this repo meaning untrusted PR authors can never trigger tests themselves. Collaborators can still trigger tests on the PR using /test all.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.


False positive with pointer returned through struct value (speex)

Thanks for the tool release! It's great to see new code in the static analysis space.
I found a false positive with the speex library. It sees a leak when a pointer is allocated and then returned as a member in a struct value. It may be more complicated than that, since I wasn't able to reproduce with a simpler test case.
src/skeleton.c:138: error: MEMORY_LEAK
   memory dynamically allocated to fp.message_header_fields by call to calloc() at line 135, column 82 is not reachable after line 138, column 5

https://github.com/xiph/speex/blob/master/src/skeleton.c#L138

this bug is not found anymore with the latest versions of infer and speex


Update the genpkey documentation

This is a significant update to the genpkey documentation to cover numerous options that were not documented.
In pulling this together I discovered two bugs which I have also fixed as part of this PR.
One bug causes a seg fault if an invalid digest name is given as a parameter value. The other bug is incorrect handling of some DSA parameter generation options. This actually has possible security consequences but analysis indicates that the risk is negligible (see the commit message for the "Pick a q size consistent with the digest for DSA param generation" commit for full details).
This PR is for master only. Various of these changes should also be applied to 1.1.0 but I will create a PR for that once this one has been reviewed.
Fixes #5739.

This got merged. Closing.


Connection refused

I get a connection error below:
C02TL289G8WL:~ akarpov$ mongo MongoDB shell version v3.4.0 connecting to: mongodb://<IP_ADDRESS>:27017 2019-05-03T12:41:01.599-0500 W NETWORK  [main] Failed to connect to <IP_ADDRESS>:27017, reason: Connection refused 2019-05-03T12:41:01.601-0500 E QUERY    [main] Error: couldn't connect to server <IP_ADDRESS>:27017, connection attempt failed : connect@src/mongo/shell/mongo.js:234:13 @(connect):1:6 exception: connect failed

There is a related link to mongodb-runner. Download the mongodb-runner as well. With it, you start the mongo.
https://github.com/mongodb-js/runner


Link wallet errors to Zendesk search

Story
As a wallet user who encounters an error, I want a one-click query to Zendesk search with my specific error, so I can get articles and FAQs with a solution.
As a NEAR Wallet developer improving the product, I want to see the list of most common Zendesk search queries over time, so I can correctly prioritize new articles and bugfixing
Acceptance Criteria

[ ] Given the error error-one generate a search URL https://nearhelp.zendesk.com/hc/en-us/search?utf8=%E2%9C%93&query=error-one
[ ] If clicked, open a search result on Zendesk containing the error query
[ ] Have the link to the search query inside the error box (top right corner) so the user can click on it and land on Zendesk
[ ] Have the link opened in a new window, to avoid interruptions of the user flow
[ ] Have the right call to action, such as "Click Here to find a solution"

From the screenshot below, the search query should be "2FA code was not verified correctly"

Unhappy Path

The user clicks the error link, but it doesn't open any Zendesk page
The user clicks the error link, and Zendesk doesn't find any article

NOTES
Zendesk provides a CSV file with a ranked list of most common queries (as of today it is staking) and the most clicked article for that specific query.
The goal for this feature is to periodically check the most common queries, and:

extend the existing articles with increasingly better answers to the query
add articles to new unanswered queries
fix any bugs or improve the UX to reduce the incidence of a given error

@corwinharrell I need a quick validation of the idea, and the ideal copy for the call to action.
Right now is "click here to find a solution" but I don't really like it

@jimmy3dita I think simply saying "View FAQ" is enough for the user to assume that they might find an answer there.


Arbitrum: Indexer lagging

The Arbitrum mainnet indexer is lagging behind the block of the actual chain head. If left for long periods of time, the chain head on the indexer drifts from the chain. Restarting the docker service will "catch up" the indexer, which will again start lagging.
The reason for the lag is unknown, as using fast RPCs like Alchemy with higher rate limits don't seem to work.
Tasks:

[x] Find a solution to fix the indexer
[x] Investigate solutions for testing indexed events against other data sources (RPC, cryo, dune, etc.)

Rate limits were set too low to index the chain, solved in #106


"watchdog Kernel module" functionality with diffrent supported parameters.

This test verifies "watchdog module" with different scenarios like

1. Watchdog module load and unload.
2. watchdog with action 1/ Reboot LPAR.
3. watchdog with action 0/ Halt LPAR.
4. watchdog with action 2 /dump collect
            a. dump collect to a local disk
            b. dump collect to a NFS disk
            c. dump collect to a SAN FC disk

@FarooqAbdulla02 gentle reminder

@FarooqAbdulla02 update please

Run logs with reference names :
watchdog_module_load_unload.log
watchdog_with_action_zero-SHUTDOWN.log
watchdog_dumpcollection_with_localdisk.log
watchdog_with_action_one-REBOOT.log
watchdog_kdump_over_NFS.log
watchdog_kdump_over_SAN.log

@abdhaleegit
here are the logs for "kdump" with existing watchdog code.
kdump_local_run_logs.log


Compile Error with CUDA 11

I try to compile it with CUDA 11 but failed.
I guess it is caused by 'Support for Kepler sm_30 architecture based products was dropped in CUDA 11'
log is here:
-- The CXX compiler identification is GNU 10.1.0
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: /usr/bin/c++ - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- The C compiler identification is GNU 10.1.0
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: /usr/bin/cc - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Found git, set version to: v5.3.3 (master-e7de04d)
-- System is: Linux (Linux)
-- Looking for CL_VERSION_2_2
-- Looking for CL_VERSION_2_2 - found
-- Found OpenCL: /usr/lib/libOpenCL.so (found version "2.2")
-- Found OpenCV: /usr (found version "4.3.0")
-- Looking for pthread.h
-- Looking for pthread.h - found
-- Performing Test CMAKE_HAVE_LIBC_PTHREAD
-- Performing Test CMAKE_HAVE_LIBC_PTHREAD - Failed
-- Looking for pthread_create in pthreads
-- Looking for pthread_create in pthreads - not found
-- Looking for pthread_create in pthread
-- Looking for pthread_create in pthread - found
-- Found Threads: TRUE
-- Found CUDA: /opt/cuda (found version "11.0")
--
-- Config summary:
--      OpenCV: 4.3.0
--      OpenCL: 2.2
--      CUDA: 11.0
--      Unicode: TRUE
--      Installing models to: /usr/share/waifu2x-converter-cpp
--      Not building test binaries
--      Building for: Unix Makefiles-x86_64
--
-- Configuring done
-- Generating done
-- Build files have been written to: /build/waifu2x-converter-cpp-cuda-git/src/build
Scanning dependencies of target conv
[  4%] Building C object CMakeFiles/conv.dir/conv.c.o
[  8%] Linking C executable conv
[  8%] Built target conv
Scanning dependencies of target gensrcs
[ 17%] Generating modelHandler_CUDA.ptx30
[ 17%] Generating modelHandler_OpenCL.cl.h
nvcc fatal   : Value 'sm_30' is not defined for option 'gpu-architecture'
make[2]: *** [CMakeFiles/gensrcs.dir/build.make:91: modelHandler_CUDA.ptx30] Error 1
make[2]: *** Waiting for unfinished jobs....
make[1]: *** [CMakeFiles/Makefile2:128: CMakeFiles/gensrcs.dir/all] Error 2
make: *** [Makefile:160: all] Error 2

System Info:
Arch Linux
Rolling 2020-07-25
CUDA 11.0.2-1

Same here

Hello friends,
I was having this error also when trying to compile this project on Mint 20 (Ubuntu 20.04) with CUDA 11.0 (from nvidia apt ubuntu repo).
I was able to get it working with some small modifications, see: https://github.com/ploober/waifu2x-converter-cpp/commit/3c811819e236beae4c0c27cc87b643f6bb942792
To summarize the change: If CUDA 11+ is detected, it will only build with a model handler for -arch=sm_60, which avoids the issue with -arch=sm_30 being unsupported in CUDA 11. If CUDA below version 11 is detected it will fall back to the original behavior.
@DeadSix27 if this is suitable for PR I can put it up. If not that's fine, I see there is #166 where this could potentially be addressed in a more universal way.

workground(?)
diff --git a/CMakeLists.txt b/CMakeLists.txt
index a13b35d..c7941af 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -378,20 +378,34 @@ if(CUDA_FOUND)
             DEPENDS src/modelHandler_CUDA.cu
         )
     endif()
+    if (CUDA_VERSION_MAJOR LESS 10)
+        add_custom_command(
+            OUTPUT modelHandler_CUDA.ptx30.h
+            COMMAND ${CONV_EXE} modelHandler_CUDA.ptx30 modelHandler_CUDA.ptx30.h str
+            DEPENDS ${CMAKE_CURRENT_BINARY_DIR}/modelHandler_CUDA.ptx20 conv
+        )
+        add_custom_command(
+            OUTPUT ${CMAKE_CURRENT_BINARY_DIR}/modelHandler_CUDA.ptx30
+            COMMAND ${CUDA_NVCC_EXECUTABLE} ${CUDA_NVCC_FLAGS} -arch=sm_30 -ptx -o ${CMAKE_CURRENT_BINARY_DIR}/modelHandler_CUDA.ptx30 ${CMAKE_CURRENT_SOURCE_DIR}/src/modelHandler_CUDA.cu
+            DEPENDS src/modelHandler_CUDA.cu
+        )
+    endif()
     add_custom_command(
-        OUTPUT modelHandler_CUDA.ptx30.h
-        COMMAND ${CONV_EXE} modelHandler_CUDA.ptx30 modelHandler_CUDA.ptx30.h str
-        DEPENDS ${CMAKE_CURRENT_BINARY_DIR}/modelHandler_CUDA.ptx30 conv
+        OUTPUT modelHandler_CUDA.ptx35.h
+        COMMAND ${CONV_EXE} modelHandler_CUDA.ptx35 modelHandler_CUDA.ptx35.h str
+        DEPENDS ${CMAKE_CURRENT_BINARY_DIR}/modelHandler_CUDA.ptx35 conv
     )
     add_custom_command(
-        OUTPUT ${CMAKE_CURRENT_BINARY_DIR}/modelHandler_CUDA.ptx30
-        COMMAND ${CUDA_NVCC_EXECUTABLE} ${CUDA_NVCC_FLAGS} -arch=sm_30 -ptx -o ${CMAKE_CURRENT_BINARY_DIR}/modelHandler_CUDA.ptx30 ${CMAKE_CURRENT_SOURCE_DIR}/src/modelHandler_CUDA.cu
+        OUTPUT ${CMAKE_CURRENT_BINARY_DIR}/modelHandler_CUDA.ptx35
+        COMMAND ${CUDA_NVCC_EXECUTABLE} ${CUDA_NVCC_FLAGS} -arch=sm_35 -ptx -o ${CMAKE_CURRENT_BINARY_DIR}/modelHandler_CUDA.ptx35 ${CMAKE_CURRENT_SOURCE_DIR}/src/modelHandler_CUDA.cu
         DEPENDS src/modelHandler_CUDA.cu
     )
     if (CUDA_VERSION_MAJOR LESS 9)
         set(GPU_CODE ${GPU_CODE} modelHandler_CUDA.ptx30.h modelHandler_CUDA.ptx20.h)
-    else()
+    elseif (CUDA_VERSION_MAJOR LESS 10)
         set(GPU_CODE ${GPU_CODE} modelHandler_CUDA.ptx30.h)
+    else()
+        set(GPU_CODE ${GPU_CODE} modelHandler_CUDA.ptx35.h)
     endif()
 endif()

diff --git a/src/modelHandler_CUDA.cpp b/src/modelHandler_CUDA.cpp
index af879fe..429200c 100644
--- a/src/modelHandler_CUDA.cpp
+++ b/src/modelHandler_CUDA.cpp
@@ -1,17 +1,17 @@
 /*
 * The MIT License (MIT)
 * This file is part of waifu2x-converter-cpp
-*
+*
 * Permission is hereby granted, free of charge, to any person obtaining a copy
 * of this software and associated documentation files (the "Software"), to deal
 * in the Software without restriction, including without limitation the rights
 * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
 * copies of the Software, and to permit persons to whom the Software is
 * furnished to do so, subject to the following conditions:
-*
+*
 * The above copyright notice and this permission notice shall be included in all
 * copies or substantial portions of the Software.
-*
+*
 * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
 * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
 * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
@@ -47,9 +47,14 @@ static void *handle;
         static const char prog20[] =
             #include "modelHandler_CUDA.ptx20.h"
             ;
-    #endif // CUDART_VERSION < 9000
-    static const char prog30[] =
-        #include "modelHandler_CUDA.ptx30.h"
+    #endif // CUDART_VERSION < 10000
+    #if CUDART_VERSION < 10000
+        static const char prog30[] =
+            #include "modelHandler_CUDA.ptx30.h"
+            ;
+    #endif // CUDART_VERSION < 10000
+    static const char prog35[] =
+        #include "modelHandler_CUDA.ptx35.h"
 	;
 #endif // HAVE_CUDA

@@ -172,9 +177,15 @@ namespace w2xc
 			return false;
 		}

-		const char *prog = prog30;
+		const char *prog = prog35;
 		// cuda 9.0 doesn't support Compute 20

+#if CUDART_VERSION < 10000
+		if (cap_major < 3)
+		{
+			prog = prog30;
+		}
+#endif // CUDART_VERSION < 10000
 #if CUDART_VERSION < 9000
 		if (cap_major < 3)
 		{
@@ -428,7 +439,7 @@ namespace w2xc
 			(nInputPlanes == 64 && nOutputPlanes == 64))
 		{
 			CUfunction f = 0;
-
+
 			if (nInputPlanes == 128 && nOutputPlanes == 128)
 			{
 				f = dev->filter_i128_o128;
@@ -558,7 +569,7 @@ namespace w2xc
 		}
 		else if (nInputPlanes == 128 && nOutputPlanes == 3)
 		{
-			void *args[] =
+			void *args[] =
 			{
 				&packed_input,
 				&packed_output,
@@ -610,7 +621,7 @@ namespace w2xc
 					1,
 					sizeof(float) * nInputPlanes * (GPU_BLOCK_SIZE+2) * 3,
 					dev->stream,
-					args,
+					args,
 					NULL
 				);
 			}

If you do not necessarily need to build the whole thing by yourself, you can get the AppImage here:
https://github.com/jeffshee/waifu2x-appimage/releases/tag/v5.3.3a

If anyone looking at this issue is still experiencing the error and wants a one-line fix:
sed -i 's/arch=sm_20/arch=sm_60/g' CMakeLists.txt && sed -i 's/arch=sm_30/arch=sm_60/g' CMakeLists.txt
Then run cmake just as you would normally.
Feel free to replace "sm_60" with your preferred value:
https://arnon.dk/matching-sm-architectures-arch-and-gencode-for-various-nvidia-cards/

fixed by PR #252


Works with Modules Intellisense

Hi! In order to use a Module Intellisense feature, the module name variable has to be deleted and also the quotes around it, then open single quotes again.

Extensions with Intellisense for Modules: leizongmin.node-module-intellisense, christian-kohler.npm-intellisense
A possible solution could be a user option for this extension that allows you to include the single quotes as part of the snippet variable.

Probably not the answer you're looking for, but one can always trigger IntelliSense like so ⌃Space.

Perhaps have a way to:
imp+moduleName to default to => `import 'moduleName' from 'moduleName';
?

I assume that this is addressed by the answer so I will close it. Feel free to open it again if there is any new info.


[Mac][minor] Add libsodium homebrew dep and reformatting

To solve issue #3474

I can't test locally but looks good. As long as we are not depending on any specific version of these (e.g. fmt) this is a nice change.

Thanks for the approval!
I left fmt not upgraded to latest because Meta internal is still using an old version: 8.0.x. If upgrade, it will break other libraries depending on velox library.
At some point, if the whole velox repo is upgraded to c++ 20, we can get free upgrade to std::format, which is even better!

Since velox team decided to use cmake for dependency management, I will trim this PR for some minor changes and formatting.

use cmake for dependency management

Just to clarify the recent work on resolve_dependency works with system dependencies but allows us to fallback to building the deps from source if they are not found. (or the user explicitly chooses with VELOX_DEPENDENCY_SOURCE=BUNDLED)

use cmake for dependency management

Just to clarify the recent work on resolve_dependency works with system dependencies but allows us to fallback to building the deps from source if they are not found. (or the user explicitly chooses with VELOX_DEPENDENCY_SOURCE=BUNDLED)

cc @kgpai - would you be aligned with @assignUser on the dep management direction as well as the fallback alternative?

Since the velox team decided to go with cmake system dependencies rather than brewed libraries, will close this PR for now.
Fallback brewed deps are optional at this point.


UI.json loading is unstable, leads to keys not being available

With the removal of PageContext there's been a little bit of over optimization on my part. useTranslation is running more code than it needs to and I should move it's logic back into a simpler context provider.

Closed by #103


Moving to OAI Spec 3.0

This is a tracking issue for OAI 3.0 spec support in Swagger Codegen here:
https://github.com/swagger-api/swagger-codegen/issues/6598
Currently, java is being worked on. There is no Go support at the moment.
Attempting to run swagger-codegen-cli-3.0.0-20180112.231857 results in code with the following:
type Event struct {
	Uuid *ModelString `json:"uuid,omitempty"`
	Timestamp *TimeTime `json:"timestamp,omitempty"`
	Body *Object `json:"body,omitempty"`
}

This should be:
type Event struct {
	Uuid string `json:"uuid,omitempty"`
	Timestamp *time.Time `json:"timestamp,omitempty"`
	Body interface{} `json:"body,omitempty"`
}

Fixed in f434cfbe99ff558cfe04a9d8ebd7cfb90a38c12f
Still using Swagger spec 2.0 but OpenAPI Generator 3.x supports OpenAPI Spec 3.0.


Release Version 2.1.1

Fixes #52 (the maven releaser already did the release, this just makes the docs instruct users to get the new version)
@jwngr please review.

@giacomodeglinnocenti FYI

Many thanks.


Qiskit 1.0

Summary
Details and comments

 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.2 out of 3 committers have signed the CLA.:white_check_mark: quantumjim:white_check_mark: grace-harper:x: drewvandethYou have signed the CLA already but the status is still pending? Let us recheck it.


Catch and fix Google Healthcare API errors

This change adds support for Google Healthcare API DICOMweb servers, such as the NCI's Imaging Data Commons.
The problem: Google Healthcare API raises an error if AvailableTransferSyntaxUID is a field, or if SOPClassUID is used as a search filter.
The SOPClassUID should definitely be allowed as an instance-level search filter, as documented in Table 10.6.1-5. Required Matching Attributes. However, this has apparently been a long-standing problem of nearly four years (see here), so it may not be fixed anytime soon. And even if it is fixed, the Imaging Data Commons may not update their software anytime soon. It would be highly advantageous to support such a large DICOMweb repository by working around the issue.
The fix in this PR is as follows:

The two search_for_instances() calls are still performed identically as before, as long as there are no HTTP errors.
If there is an HTTP error with a 400 status_code, and a message is present matching the errors from Google Healthcare API, then the search_for_instances() arguments are patched to work for Google Healthcare API, as follows: a) AvailableTransferSyntaxUID is simply removed, if present. b) SOPClassUID is manually filtered, if present (meaning it is not supplied in the search_filters, but only instances with a matching SOPClassUID are returned).

These changes shouldn't have any impact on any situations except where an error occurs from a Google Healthcare API server. And in that case, the function calls are patched and then work properly.
The following example works after this fix:
from wsidicom import WsiDicom, WsiDicomWebClient

url = 'https://proxy.imaging.datacommons.cancer.gov/current/viewer-only-no-downloads-see-tinyurl-dot-com-slash-3j3d9jyp/dicomWeb'
study_uid = '2.25.227261840503961430496812955999336758586'
series_uid = '<IP_ADDRESS>.4.1.59<IP_ADDRESS>4438926.1589741711.1637717011470.2.0'

client = WsiDicomWebClient.create_client(url)

slide = WsiDicom.open_web(client, study_uid, series_uid)

Fixes: #141

Hi @psavery
Thanks for your pull request. When parsing dicom files we have tried to handle as many implementations error as possible, so of coarse we should strive to do the same when reading from DICOM web.
Your approach inspired me to make some changes that would enables us to (hopefully) re-use the _search_for_instances() method also for other implementation errors. Is it ok if I push to your branch?

@erikogabrielsson Sure, feel free to push to it! :slightly_smiling_face:
We would love to have these changes in soon, so that we can start utilizing wsidicom with that large database!

Another possibility would be to merge these changes, and add new features as a separate PR.

Thanks so much, @erikogabrielsson! Can we get a new release so we can start using these features?

Released in 0.19.0


Fixed #168: Prevent the policy tests from running in CI

@jiridanek it is not totally clear to me why the policy tests are still being run on Ubuntu. It looks like the build is being downloaded from an earlier step - https://github.com/skupperproject/skupper-router/runs/5539179536?check_suite_focus=true#step:3:14  ??


[BUG] alias are ignored in model_dump for the documents in Link

Describe the bug
Hi, another bug related to migration to pydantic v2. In the case you have a Document with a field of type Link to another Document, when you try to perform a model_dump of the root Document requiring the use of the alias this directive is applied only for the root Document
To Reproduce
class C(Document):
    a: Annotated[str, Field(alis="_a")] = 'f'

class D(Document):
    c: Link[C]

c = C()
await c.save()
c = await C.get(c.id)
d = D(c=c)
await d.save()
d = await D.get(d.id)
await d.fetch_all_links()
d.model_dump(by_alias=True)

This results in
{'_id': '6503b090ba1e38cf02b18fc1', 'c': {'id': '6503b090ba1e38cf02b18fc0', 'a': 'f'}}

Expected behavior
The example above should return
{'_id': '6503b090ba1e38cf02b18fc1', 'c': {'_id': '6503b090ba1e38cf02b18fc0', '_a': 'f'}}

Additional context
python=="3.11.5"
pydantic=="2.3.0"
beanie=="1.22.1"

In FastAPI this bug prevents the use of Link in the response models
Thank so much for your work

Hi! Thank you for the catch. I'll check and fix it this/next week.


babel can't extract text from template with "with" statement

my template has the following code:
{% with messages = get_flashed_messages() %}
  {% if messages %}

    ×

    {% for message in messages %}

{{ message }}

    {% endfor %}

  {% endif %}
{% endwith %}

but pybabel can't extract text from this file unless I remove the {% with %} statement.
is there any mistake I made in this case?

My application templates folder structure :
/templates
    /app
          /_layouts
                /main.html
                /email.html
    /website
          /_layouts
                /main.html

when I extract files, babel doesn't extract files which in templates/website/_layouts/main.html. If I change file name from templates/website/_layouts/main.html to templates/website/layouts/main.html babel extract this file.
babel.cfg
[python: **.py]
[jinja2: **/templates/**.html]
extensions=jinja2.ext.autoescape,jinja2.ext.with_

I think babel ignores folders with leading underscores.

babel extract files in "templates/app/_parts/" but it doesn't extract files in "templates/app/_layouts/" and "templates/website/_layouts/". Does babel have a problem with "_layouts" word?

Maybe you can make a testcase for this and open a new ticket.


[IMPROVEMENT] Enhance the code quality in the instance-manager instance and disk gRPC server methods.

Is your improvement request related to a feature? Please describe (👍 if you like this request)
Enhance the code quality in the instance-manager instance and disk gRPC server methods. Refactor the implementation to align with Object-Oriented principles.
Describe the solution you'd like

Describe alternatives you've considered

Additional context

Backend update.

Pre Ready-For-Testing Checklist

[x] Where is the reproduce steps/test steps documented?
The reproduce steps/test steps are at:

run e2e tests and everything should be ok

[ ] Does the PR include the explanation for the fix or the feature?

[x] Have the backend code been merged (Manager, Engine, Instance Manager, BackupStore etc) (including backport-needed/*)?
The PR is at

https://github.com/longhorn/longhorn-instance-manager/pull/369
https://github.com/longhorn/longhorn-instance-manager/pull/370
https://github.com/longhorn/longhorn-manager/pull/2448

[x] Which areas/issues this PR might have potential impacts on?
Area: gRPC services in instance-manager pod
Issues


chore: replaced with dapr-kit logger

Description

Currently we use log package for logging whereas we have our own logger available at github.com/dapr/kit/logger which we should make use it
Issue reference

Please reference the issue this PR will close: #[issue number]
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[x] Created/updated tests
[] Extended the documentation

I would like to see the logger name kept, we shouldn't have unnecessary package name collisions especially with the standard library.
The service grpc and http documentation should use the standard library / be open to the user.

you mean to change the name of  log to something elsevar log = logger.NewLogger("dapr.actor.manager")

I would like to see the logger name kept, we shouldn't have unnecessary package name collisions especially with the standard library.
The service grpc and http documentation should use the standard library / be open to the user.

you mean to change the name of log to something elsevar log = logger.NewLogger("dapr.actor.manager")

Yep, I just think it looks cleaner than using a naming convention that clashes with the standard library where it doesn't interface cleanly.

@mikeee can we think in this way .. it had same name so I was able to quickly migrate to new liberary . so maybe that's a plus point over using different names . just thinkering for this let me know if you agree or else ill change

ok so shall we do like for example logClient in the client package

I'm not entirely sure about using kit/logger in a user written application would be preferable.
@qustavo what are your thoughts about this change?
The recent change made by @qustavo allows for a user-specified logger as part of the client created, perhaps this addresses the need to migrate entirely to kit/logger and not change the expected output/functionality of the current logging implementation

I'm not entirely sure about using kit/logger in a user written application would be preferable. @qustavo what are your thoughts about this change?

From what I can see, the logging requirements for the sdk are minimal. We log.Print[f,ln] and log.Fatal[f] and the current logging implementation is sufficient. That being said, we could support a custom logger injection if SetLogger accepted an interface rather than a concrete implementation, so that you can bring your own logger, although that might be overkill.
In general I'd wait until the need for a more elaborated logging strategy is needed.
@sadath-12 is there a use case for using kit/logger that I'm overlooking?
https://github.com/dapr/go-sdk/blob/7e5ee73aa842660f3bb2c958947d1faa3f665022/client/client.go#L68


Fix: Tabs dispatch change event on init

Related Issue: #881

Thanks again for the report.
Fixed in v0.49.2


cmd/go: go list reports net as "stale" right after ./make.bash (but only inside GOROOT/src)

What version of Go are you using (go version)?

$ go version
go version go1.16 darwin/amd64

Does this issue reproduce with the latest release?
Yes.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/dmitshur/Library/Caches/go-build"
GOENV="/Users/dmitshur/Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/dmitshur/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/dmitshur/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_amd64"
GOVCS=""
GOVERSION="go1.16"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/dev/null"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch x86_64 -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/zb/5p8cwfhj29gf_m8vdy8ylmlr00jwcj/T/go-build203663307=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
I tried building Go from source at Go 1.16 and 1.15.8, comparing whether the net package is reported as stale or not right after a ./make.bash.
What did you expect to see?
I expected that net is reported not stale after ./make.bash with both Go versions.
What did you see instead?
net is reported as not stale when using 1.15.8 both inside GOROOT/src and outside:
$ cd $(mktemp -d)  # Or equivalent that doesn't involve symlinks.
$ git clone https://go.googlesource.com/go && cd go
$ git checkout go1.15.8
$ cd src && ./make.bash && cd ..
$ export PATH="$(pwd)/bin:$PATH"
$ go list -f '{{.ImportPath}}: {{if .Stale}}stale; reason={{.StaleReason}}{{else}}not stale{{end}}' net
net: not stale
$ cd src
$ go list -f '{{.ImportPath}}: {{if .Stale}}stale; reason={{.StaleReason}}{{else}}not stale{{end}}' net
net: not stale

net is reported as not stale when using 1.16 outside GOROOT/src, but stale inside:
$ cd $(mktemp -d)  # Or equivalent that doesn't involve symlinks.
$ git clone https://go.googlesource.com/go && cd go
$ git checkout go1.16
$ cd src && ./make.bash && cd ..
$ export PATH="$(pwd)/bin:$PATH"
$ go list -f '{{.ImportPath}}: {{if .Stale}}stale; reason={{.StaleReason}}{{else}}not stale{{end}}' net
net: not stale
$ cd src
$ go list -f '{{.ImportPath}}: {{if .Stale}}stale; reason={{.StaleReason}}{{else}}not stale{{end}}' net
net: stale; reason=stale dependency: golang.org/x/net/dns/dnsmessage

I ran into this while investigating #44714 with @toothrot, and we both thought this was worth reporting so that it can be investigated as needed.
(While writing up this issue report, I realized it makes a difference whether the go list command is run in GOROOT/src or outside, which is not something I realized earlier.)
CC @jayconrod, @bcmills, @matloob.

This is possibly a side-effect of the way the vendored std module dependencies work in module mode.
When you are inside GOROOT/src, the module dependencies of std have their actual, external-module identities. (That makes it possible to run commands like go mod tidy and go mod vendor.)
However, when you move outside of GOROOT/src, those vendored dependencies can no longer be ordinary module dependencies: their versions are fixed to what is shipped in std and the packages no longer retain their module identities. For example, the vendored copy of golang.org/x/net/dns/dnsmessage turns into the package vendor/golang.org/x/net/dns/dnsmessage.
When that happens, the package itself becomes stale, because its identity (and therefore its debug information, and perhaps its export metadata) has changed.

This probably changed as of CL 251159, which removed the special-case code for std.
That special-case forced the vendored std module dependencies to their vendor/… paths even when running the go command within GOROOT/src.
@jayconrod, @matloob: do you think we should restore that special-case (so that the dependencies of packages in std are identical whether inside or outside of the std module), or keep it as it is in 1.16 (so that the dependencies reported for those packages for general go commands match the paths used in go get and go mod tidy commands)?

I'd lean toward restoring the special case (I hate saying that). It's pretty confusing that these packages have multiple identities. It's taken me longer than I'd like to admit to re-read your comments and the CL to work out what's going on.

Yeah, I agree. I have a CL in progress. 😁


pks cli command to login should be replaced

In the given document pks cli command given for login is in correct
pks login -a api.pks.example.com \ --client-name automated-client \ --client-secret randomly-generated-secret -k
`Error: unknown flag: --client-name
Usage:
pks login [flags]
Examples:
pks login -a  -u  -p  [--ca-cert  | -k]
Flags:
-a, --api string              The PKS API server URI
--ca-cert string          Path to CA Cert for PKS API
-h, --help                    help for login
-p, --password string         Password
-k, --skip-ssl-verification   Skip SSL Verification
-u, --username string         Username
`
should be replaced with pks login -a  -u  -p  [--ca-cert  | -k]

@shausy Thanks for this. That flag should still work. Is it possible you're using an older version of the PKS CLI? Please download the latest version of the PKS CLI from Pivotal Network using these instructions and see if the issue continues.

Sure thanks for the info. I will check and update

Thanks @shausy. Closing this but feel free to reach out again if you have any more questions or concerns.


fix cache populate incorrect content when numBackgroundThreads>1 on b…

…roker
it's a follow up of #3943. the same bug exists on broker side.

my mis-understanding. it is different with what happens in #3943 now, close this PR.


BindingResolutionException in Container.php line 744:

Target [Illuminate\Contracts\Auth\Registrar] is not instantiable.

Hi @sourabh13
I know it is long time ago you postede this issue, but where do you get this error, if it still present to you. ? :)

Hello,
First of all thnx for the multiauth package, it was a great help to me and actually I got past that issue. It was an error at my end .. :). Sorry forgot to close this issue.


Align text in the centre

Is there any way to align text centre?
I set .textAlignment to centre, but when I type down one letter, it jumps back to left.

You could custome your markdown previewer or viewer or render to make a center style. No need to make center in editor.

Hi, sorry for the delay. You can fiddle with the text alignment either here or here. Please reopen this issue if you are still having trouble or need more help. You may also try stackoverflow.com for general questions or help.


[BUG] Build fails on Manjaro/Arch Linux

Describe the bug
Building GPTQModel on Manjaro fails. I have the suspicion it is because of Arch Linux, from my research, I think the error means I should try to build with gcc-12 and g++-12 but switching to those with variable export didn't work.
I guess the problem will disappear if you have a pip version.
Part of the exceptions:
Emitting ninja build file /run/media/knut/HD/huggingface models/language models/LLMs/GPTQModel/build/temp.linux-x86_64-cpython-39/build.ninja...
Compiling objects...
  Or use conda to create a virtual python env with python 3.10/3.11. I want to see if the error is caused by python or your g++.

I tried 3.11 in a virtualenv but I get the same error. so that means it's likely g++? I tried version 11,12, 13. I'd guess I'm supposed to use 12 on my system, given my cuda version.

setup.py has c++ 17 flag so maybe you need to have 17 installed.

For reference our ci and dev env is based of Ubuntu 22.04 and any dev/compiler.
I would recommend upgrading your compiler to something similar to Ubuntu 22.04 or use a container system such lxd or docker to compile without having to touch your base is which can be a pain to reconfigure.

ok thanks.

on the sporadic attempts I've undertaken, i've not been able to build gptqmodel on my PC. as far as I know, I'm using the right gcc version. this keeps me from adopting your tool.

I mean I could indeed use a container, but I haven't bothered - I just want to install this the regular way.

Is Manjaro officially supported by Nvidia for cuda toolkit? Nvidia is has very tight (strict) integration of their driver/cuda combo.
Also, launching an official Nvidia docker image is only one line of code.
We do not have the resources to debug arch or other distros that is not nvidia sanctioned.

got that, I'll use the image


Implementing Fibonacci Search in java

💥 Proposal
(A clear and concise description of what the proposal is.)
Have you read the Contributing Guidelines on Pull Requests?
Yes

/assign

Pls assign it to me would like to create in java


Package Relay / V3 package RBF deployment

i.e. port https://github.com/bitcoin/bitcoin/pull/25038
and implementation of https://github.com/bitcoin/bips/pull/1382

completed


Stretched background

Currently the background image (clouds.png) is stretched across the screen, would it be possible to tile this or atleast make sure the aspect ratio is retained from the original image? Possibly (as a new feature) would also be great to support some parallax scrolling by combining two background images.

Nice, on it!

Done though probably could do with some better colouring on the layers - I just darkened the trees a bit to fit them in.

Fantastic work! As for the sprites I'll probably re-do them to add some more detail but your effort is certainly exactly what I meant. Great job! I'm currently at 63% of redoing all tiles, and 60% of "other sprites" (particles, logo, background etc.). Once everything is done I'll do a pull request and then you can all complain and give feedback to improve the visuals ;)


Use to use the build CI image in ci.yml

Description
Use to use the build CI image in ci.yml.
Issues Resolved
Part of https://github.com/opensearch-project/opensearch-build/issues/3966
https://github.com/opensearch-project/job-scheduler/issues/524
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff

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

Hey @peterzhuamazon can you please take a look at this PR?
I assume as suggested in meta issue need to only the build and test workflows and not the bwc test workflows, also did not use the option options: --user root as there are no packages that needs to be installed for job-scheduler.
Thank you
Adding @joshpalis @minalsha @bbarani

Looks like we need the options: --user root to allow the repo checkout inside the container, added back.

Hey @joshpalis can you please take a look at the failed tests ? are they flaky ?

@prudhvigodithi , yes this is a flaky test, which has been identified in the handoff document.
org.opensearch.jobscheduler.sampleextension.SampleJobRunnerIT > testJobUpdateWithRescheduleJob FAILED
    java.lang.AssertionError: expected:<1> but was:<0>
        at __randomizedtesting.SeedInfo.seed([14A61049E22DA007:C548E907BD31869D]:0)
        at org.junit.Assert.fail(Assert.java:89)
        at org.junit.Assert.failNotEquals(Assert.java:835)
        at org.junit.Assert.assertEquals(Assert.java:647)
        at org.junit.Assert.assertEquals(Assert.java:633)
        at org.opensearch.jobscheduler.sampleextension.SampleJobRunnerIT.testJobUpdateWithRescheduleJob(SampleJobRunnerIT.java:81)
REPRODUCE WITH: ./gradlew ':opensearch-job-scheduler-sample-extension:integTest' --tests "org.opensearch.jobscheduler.sampleextension.SampleJobRunnerIT.testJobUpdateWithRescheduleJob" -Dtests.seed=14A61049E22DA007 -Dtests.security.manager=false -Dtests.locale=hr-HR -Dtests.timezone=America/Indiana/Indianapolis -Druntime.java=11

I'll re-run the failed checks

Hi @prudhvigodithi could you help backport this to 2.x?
Thanks.

I have created a backport PR https://github.com/opensearch-project/job-scheduler/pull/535
@peterzhuamazon @joshpalis @cwperks please take a look.


Fix for #1965

I think I have found a fix for issue #1965. The indentation issue would have also come when using subscriptions so I've tried to fix that too.
I could not test my changes. I followed the written instructions but had the following problem:

First I changed the path of extensions.graphback.model in the .graphqlrc file,

Then I ran the command in the templates folder and received this error,

While I wasn't able to test it I think the changes I've made should fix the issue.

@RinkiyaKeDad amazing change and nice PR description, thanks. @craicoverflow mind having a look at this one?

@RinkiyaKeDad the build: https://github.com/aerogear/graphback/runs/1165819855?check_suite_focus=true if failing and it is related to the change you made.
Do you mind running yarn test -u in the packages/graphback-codegen-client/ directory to update the snapshot? Thanks.

Looks good. Please update snapshots

Hey @machi1990 I ran the command you told in order to update the snapshots. Recevied this error:

Please help me. Thanks.

Please help me. Thanks

Every repo has contributimg.md file that everyone should follow. it is not perfect but it has all info and it will resolve problems. It is ok to ask maintainers to solve problems with code, but we cannot solve problems with your environment etc.

Really sorry @wtrocki. Didn't know this :sweat_smile:. Will take care from next time.

File in question: https://github.com/aerogear/graphback/blob/master/CONTRIBUTING.md
If you follow steps your problem will be resolved.

Can you please just tell me if this is a  problem with my environment?

I see that the package.json of graphback-codegen-client mentions @graphback/core as a dependency and just be sure I even ran yarn install in the graphback-codegen-client directory where it gave me a success message saying "Already up-to-date". This means that @graphback/core should be present in node_modules but then why is it having trouble importing it?

Can you please just tell me if this is a problem with my environment?

I see that the package.json of graphback-codegen-client mentions @graphback/core as a dependency and just be sure I even ran yarn install in the graphback-codegen-client directory where it gave me a success message saying "Already up-to-date". This means that @graphback/core should be present in node_modules but then why is it having trouble importing it?

Thanks for following up on this one. The error you are seeing is because the project was not built. Can you run the below command in the project root?
yarn
yarn build

Once that is done, you can run the yarn test -u command in the graphback-codegen-client package.
Let me know if that work.

https://github.com/aerogear/graphback/blob/master/CONTRIBUTING.md#building

Sorry, I only ran yarn install after reading the CONTRIBUTING.md since I was under the assumption that building is something you only did before deployment :sweat_smile: Thanks a lot for helping me out :)


unpin beta

Now that we've released should be good to unpin beta / any future testing can just use /stable anyways?

Would it make sense to have a /latest that's always unpinned?

Let's wait this week and merge next Monday? You can merge your sim changes to stable if they are low risk.

There is /stable. I want the team focus to be on the release fixes and not to be distracted to future releases.

Also /beta is always latest and /stable is the last release stable channel.
I just pin beta for a week after release to ensure folks don't spend time on non-urgent things.

@abchatra good to go?


Add WORLD_OPTION_SETTINGS env variable

Allows configuration of all options for the game via a .env file.

This looks very interesting and very according to the Docker-Best-Practice way.
There is also another PR from @gchai under #40 thats more focussed on very intensive script usage.
Hmmm, both are very interesting and ALSO BOTH missing documentation about the "truck-load" of settings we would deliver 🤣
Can we all please stick the heads together and make 1 GREAT solution to this?

This solution is very Docker-Best-Practice way with using .env files to keep settings and secrets private. But its also fragile and very update heavy on the context of Pocketpair releasing a new config and we have to update this all the time and maybe the gameserver will not start until ALL peole update the config or the image or we fix the bug, because of missing params or failed validation or 100 other things we didnt thing about. The worst thing here can happen is that hundreds of server restart, get new server files and NONE is starting, which would be really bad and people surely would be sad or pissed off 🤣 and increase drastically traffic to the "issue-section" 🤣
The other solution is very scripting heavy and extends quite far the servermanager, but respects 100% the Default-Config that comes with the gameserver over SteamCMD no matter what happens. The worst thing here can happen is that the gameserver just starts and 1 config-param was not set/applied and its back to default-value then, BUT its always running. Which has a certain value for a "customer-perspective".
Any thoughts or problems i missed or problems with my analysis?

Seems like the discussion and a draft on the readme started over at #40 🤷 @jtmckay please help out too over there.

What I like about this solution, is that I don't have to re-build the docker image every time I need to change/add a setting. I would assume that Pocketpair would not make a breaking change where a previous config no longer functions, but maybe I'm giving them too much credit.
I actually didn't see the alternative solution until I already submitted this PR. I just thought I'd share back what I found useful, incase it's useful back here, since this was useful for me, and I appreciated it @jammsen. Happy to go either way.

What I like about this solution, is that I don't have to re-build the docker image every time I need to change/add a setting. I would assume that Pocketpair would not make a breaking change where a previous config no longer functions, but maybe I'm giving them too much credit.
I actually didn't see the alternative solution until I already submitted this PR. I just thought I'd share back what I found useful, incase it's useful back here, since this was useful for me, and I appreciated it @jammsen. Happy to go either way.

Thanks for your help and honesty, i value this much!
With #40 as solution we can add .env too, i asked that question at the end of my latest comment on #40 . There should be no need to rebuild the image the way i want this to be resolved and an .env would be also in question for my solution.

Thanks for the contribution, closing this in favor of #40 which is already commited by me


make pdk fails with :: ModuleNotFoundError: No module named 'yaml'

Describe the bug
When we install openlane with "git clone https://github.com/The-OpenROAD-Project/OpenLane.git" and following step #make pdk command fail with below error:
Traceback (most recent call last):
File "./dependencies/tool.py", line 15, in
import yaml <---
ModuleNotFoundError: No module named 'yaml'
git clone  /home/indir/OpenLane/pdks/skywater-pdk
fatal: repository '/home/indir/OpenLane/pdks/skywater-pdk' does not exist
make: *** [Makefile:87: /home/indir/OpenLane/pdks/skywater-pdk] Error 128
Desktop (please complete the following information):
Important Note::This issue arise only when using below OS.

OS:Ubuntu 20.04 and Centos 7 and above

Workaround is install pyymal with below command and then try #make pdk
#sudo pip3 install pyyaml
#make pdk
I have tested at end above solution working fine.
@donn  Please verify is this valid and approve it, then I will Close this GitHub issue.
Thanks..

This isn't a workaround. This is in the Readme.


Split up [github] testers

Update github contributors badge for create-service-tester.

Conflicts nominally with #2470. Let's merge that ASAP and this right afterward so as to unblock the rest of #2320.

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @paulmelnikow!

  Generated by :no_entry_sign: dangerJS against 39437ab35a0ea2716efa15b42ff73d15b59b3809


How can i Cancel the request

This code is not Canceling the request, why?
     RestangularConfigurer.addFullRequestInterceptor(function (element, operation, route, url, headers, params, httpConfig) {
            var defer = $q.defer();
            defer.reject();
            httpConfig.timeout = defer;
            return {
                element: element,
                headers: headers,
                params: params,
                httpConfig: httpConfig
            };
        });

That's the correct way to cancel the request.
See also https://github.com/mgonto/restangular/issues/1377#issuecomment-226625799


unable to open image

not sure what this all means, signal, null. you could have wrote some meaningful errors
{ [Error: Command failed: convert: unable to open image platforms/android/res/drawable/icon.png': No such file or directory @ error/blob.c/OpenBlob/2701. convert: WriteBlob Failed platforms/android/res/drawable/icon.png' @ error/png.c/MagickPNGErrorHandler/1630.
] timedOut: false, killed: false, code: 1, signal: null }

i think ionic resource is the way to go


Rozwalak2024

Pull Request
This PR is for a

[ ] New Publication(s)
[ ] Correction

For the following list(s):

[ ] ancientmetagenome-environmental (README)
[ ] ancientmetagenome-hostassociated (README)
[ ] ancientsinglegenome-hostassociated (README)

Publication Information
This is to close
PR Workflow

Open this PR with sample metadata on the samples metadata sheet (:tada: you're already here!)
Wait for checks for sample metdata to pass
(If checks fail) make corrections, and push changes to this branch (no need to open a new PR!)
(Once passed) comment on this PR @spaam-bot please autofill <table_name> <project_id> to get a half-filled template! (may take a minute or so to get the comment with the file)
Fill in the template, and verify autofilled data correct!
Once filled in, append(!) the new rows from the TEMPLATE file to the end of the corresponding library metadata file, and update this PR
Wait for checks for library metadata to pass
Request review!

Pre-review checklist (new publications)

[ ] Publication is published

Preprints currently not accepted?

[ ] Checked the publication is not already in the database?
[ ] Checked samples in this publication are not previously published data?

Newly re-sequenced metagenomes are OK!

[ ] Samples are shotgun metagenomes and not amplicon data

Note: hostassociated-singlegenome may also contain whole-genome enriched data

[ ] Checked the list follows conventions as described in the corresponding sample type's README file (e.g. using ERS/SRS accession codes for ENA/SRA)?
[ ] Library metadata has been added
[ ] Changelog is updated to include the publication under 'Added'?
[ ] Pull request has passed validation checks (see automated comment from github-bot)?
[ ] (If applicable) Made a separate PR to add new categories to controlled lists (stored under /assets/enums, e.g. material or archive)? If so the corresponding PR is here:

@jfy133 Can you help me with these problems?

@rozwalak if you look closely at the first error, you'll see you have a space at teh end of your project_id string ;)
The second, you need to make a second PR to update the corresponding enum file: https://github.com/SPAAM-community/AncientMetagenomeDir/tree/master/assets/enums, as described in the error :)

Thanks @rozwalak !


Managed failover workflow improvements

What changed?

Support graceful failover in managed failover workflow.
Check remote task list poller number prior to failing over a domain.

Why?

Support batch failover
Prevent dis-config and no worker on the to-be-active cluster.

How did you test it?
Unit  tests update

Potential risks

Release notes

Documentation Changes

Pull Request Test Coverage Report for Build f9bba1ff-3720-481b-a9a9-030d4642d2dc

49 of 57 (85.96%)  changed or added relevant lines in 3 files are covered.
96 unchanged lines in 11 files lost coverage.
Overall coverage decreased (-0.02%) to 56.399%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

service/worker/failovermanager/workflow.go
32
36
88.89%

tools/cli/adminFailoverCommands.go
12
16
75.0%

Files with Coverage Reduction
New Missed Lines
%

common/task/weightedRoundRobinTaskScheduler.go
1
89.64%

tools/cli/adminFailoverCommands.go
1
39.56%

service/history/execution/mutable_state_builder.go
2
69.62%

service/matching/matcher.go
2
91.46%

common/cache/lru.go
3
90.73%

common/persistence/nosql/nosqlplugin/cassandra/workflow.go
3
50.23%

service/history/execution/context.go
6
73.83%

common/persistence/nosql/nosqlExecutionStore.go
9
57.52%

service/history/queue/transfer_queue_processor.go
12
58.22%

common/persistence/nosql/nosqlplugin/cassandra/workflowUtils.go
20
77.03%

Totals

Change from base Build d54024c7-1c91-4254-b249-84d53ddf009b:
-0.02%

Covered Lines:
79935

Relevant Lines:
141731

💛  - Coveralls


cursorWordPart{left,right} moves too far

Issue Type: Bug
cursorWordPartLeft and cursorWordPartRight move the cursor too far. The cursor moves past the next underscore instead of right before it.
Here's a gif of TextMate, which as far as I know was the first program to have a shortcut for something like this:

Here's Sublime Text, which works the same way:

And here's VS Code's implementation, which moves past the underscore instead of right up to it. This makes it difficult to change one part of a variable name, because if you select and then type it deletes the underscore.

I was surprised that I was unable to find anyone else reporting this issue. Perhaps this shortcut isn't used that much? I use it all the time which is why I noticed it. Any chance it could be swapped to work the same way as TextMate and Sublime Text?
VS Code version: Code - Insiders 1.44.0-insider (708b019bb4e20f07cf89df9f1d943af3d38d7a70, 2020-03-18T05:43:24.249Z)
OS version: Darwin x64 18.7.0

You can add _ to your editor wordSeparators.  Put this into your settings:
  "editor.wordSeparators": "`~!@#$%^&*()-=+[{]}\\|;:'\",.<>/?_"

note the added _ at the end.  Now wordPart moves will act as you wish with respect to underscores.

Thanks for the quick reply! Unfortunately that doesn't do the right thing. If I add _ to my word separators, then the insertion point (starting from the left) moves like this:
|foo|_|bar|
What I want is this:
|foo_|bar|

Thanks for the quick reply! Unfortunately that doesn't do the right thing. If I add _ to my word separators, then the insertion point (starting from the left) moves like this:
|foo|_|bar|
What I want is this:
|foo_|bar|
(which is what TextMate and Sublime Text both do by default)

I am confused, your gifs show TextMate and ST doing this:
|foo|_bar|  .  Adding the underscore to wordSeparators gets you half-way there though, not all the way.

Whoops - you're right. I accidentally put the pipe in the wrong spot. I've fixed my comment.

Wow!! Thanks for the super fast fix! I am very excited for this to show up in my insiders :)

I don't see any change in this behaviour as of
Version: 1.45.0-insider (user setup)
Commit: 44c5185373d5f209cd6c0b5d29a216fb801da34e
Date: 2020-04-10T18:12:33.780Z
Electron: 7.2.1
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Windows_NT x64 10.0.19603
did this get merged yet?

@ArturoDent Yes, the change is in any VS Code >=1.44. There is a test that asserts the change and it has been verified independently by @Tyriar .  Is it possible you are not testing cursorWordPartLeft but cursorWordLeft and assuming that would have changed ?


Possible regression in f8a6f99

In https://github.com/FNA-XNA/FNA/issues/23 a fix was made to support a VS -> PS variable of type COLOR2 to MojoShader in commit https://github.com/flibitijibibo/MojoShader/commit/f7e6b196b935b19c949e2025f778d37a77c86a2d however the next commit https://github.com/flibitijibibo/MojoShader/commit/f8a6f991a3fba48561f7bda89c57db800a8390ad which added a Non-ES fix I think added a regression.
As the code changed from:
        if (usage_str == NULL)
        {
#if SUPPORT_PROFILE_GLSLES
            output_line(ctx, "varying highp vec4 io_%i_%i;", usage, index);

#else
            output_line(ctx, "varying vec4 io_%i_%i;", usage, index);
#endif
            output_line(ctx, "#define %s io_%i_%i", var, usage, index);
        } // if

to:
        if (usage_str == NULL)
        {
#if SUPPORT_PROFILE_GLSLES
            if (support_glsles(ctx))
                output_line(ctx, "varying highp vec4 io_%i_%i;", usage, index);
            else
#else
            output_line(ctx, "varying vec4 io_%i_%i;", usage, index);
#endif
            output_line(ctx, "#define %s io_%i_%i", var, usage, index);
        } // if

which results in effectively:
        if (usage_str == NULL)
        {
            if (support_glsles(ctx))
                output_line(ctx, "varying highp vec4 io_%i_%i;", usage, index);
            else
                output_line(ctx, "#define %s io_%i_%i", var, usage, index);

so when it generates the GLSL it only outputs the #define ... line and not the io_ variable declaration.
Perhaps the #else should be replaced by an #endif

https://github.com/flibitijibibo/MojoShader/commit/d5a811b90107862d2a39533d13f1b75aa6e1690f


Office Tabs

Description of the new feature / enhancement
Dear PowerToys Team,
I am writing to request a new feature that would be very useful to me in my work. It is about being able to add or delete sheets within the same Word or PowerPoint document, just as it can be done in Excel.
I just want that the rest of office products have something like the tabs of a browser, which follows the principle of the sheets of Excel, to see different documents of the same type (PowerPoint, Word, etc). I just want that this feature to be extended to the other prodcuts of the suite.
Scenario when this would be used?
In any moment that someone needs files of the same category, as an example, different word documents but that respond to a unique subject.
This would allow me to better organize the information and have more flexibility when editing or presenting my projects.
Supporting information
Currently, there is no option in Word or PowerPoint similar to what is in Excel, which allows adding/deleting new sheets, within the same document (physical file).  think this would be a very valuable improvement for many users who use these programs frequently. I thank you for your attention and I hope you can consider my suggestion.

/feedbackhub

Thanks guys, awesome work with all the tools of PowerToys, i used it frecuently and i do love it. keep it like that.


cannot install KICS older then V1.5.1 using the tag on linux machine

the URL for the download file has changed since V1.5.2 and the installation script does not support it
curl -sfL 'https://raw.githubusercontent.com/Checkmarx/kics/master/install.sh' | bash -s -- -b /usr/local/bin v1.5.1 ==> WORKS
curl -sfL 'https://raw.githubusercontent.com/Checkmarx/kics/master/install.sh' | bash -s -- -b /usr/local/bin v1.5.2 ==> FAILES

Hello, @daviduash  🙂 Thank you so much for reaching us!
KICS deprecated the availability of binaries in the GitHub releases assets as of version 1.5.2. The latest (and last) binary available is 1.5.1: https://github.com/Checkmarx/kics/releases/tag/v1.5.1
However, KICS is available as a Docker image. You can see the documentation here.
Let me know if you need any help 😊

At my company the desktop security configuration prevents using bind mounts with Docker.  This makes it very inconvenient to use Kics in Docker form.  Please reconsider supporting standalone binaries again.


yaml interface testing

I intentionally broke the yaml schema in my pr to see if tests would fail. They seem to pass.


added city/weather api data retrieval. added function that applies temperature status and puts data into objects

added function to get city data. function to get weather from randomly selected cities. Function to add temperature tag and put it into an object.

Fixed issue from previous pull request.


Fix named message stream copy to

Is the same thing needed in NetworkBehaviour line 832 and NetworkSpawnManager line 457?

Is the same thing needed in NetworkBehaviour line 832 and NetworkSpawnManager line 457?

¯_(ツ)_/¯
Depends how the stream is meant to be used.
Which is why I'm not a huge fan of using raw streams.
If those lines were changed in the same context as the ones I flagged, then my guess would be yes. However, their position might already be at zero, or they also might be expecting to be used differently.

@ShadauxCat Is the fact that the stream has trailing nulls a recent change, or it was always like this?
If it was always like this and it's the expected behavior, I can just change the test to assert that the content starts with what I'm expecting.
Otherwise, we might want to remove the trailing nulls from the stream we give back the the end-user in the message handlers.

@ShadauxCat Is the fact that the stream has trailing nulls a recent change, or it was always like this?

@bendoyon I'm pretty sure that's fixed by #985


Image Flickers when Updated from 0.42.3 to 0.43.4 and changed from listview to flatlist

Updated RN from 0.42.3 to 0.43.4 and changed newsfeed component from listview to flatlist
So, whenever we scroll the feed the image flickers
<FlatList
ref={(c)=>this._listViewRef = c}
onViewableItemsChanged={(visibleRows, changedRows) => console.log("yes it works")}
refreshing={this.state._refreshing}
data={this.state._dataSource}
renderItem={({item})=>this._renderRow(item)}
enableEmptySections={true}
onEndReached={()=>{
if(this.state._stopLazyLoad == false){
return this.fetchNewsFeed()
}
}}
extraData={this.state}
onEndReachedThreshold={height*4}
ListHeaderComponent={()=>}
ListFooterComponent={()=>{
if(this.state._refreshing == true){
return null;
}
return <View style={{flex:1,justifyContent:'center',alignItems:'center'}}>
}}
/>

React Native version: 0.43.4
Platform: android
Development Operating System: Linux
Dev tools: Android Studio 2.3,Android SDK Tool 25.2.5

Are you using dataSource from ListView? if so that is wrong. it should be a array of data not the datasource from ListView

No i am using a json array and data is properly visible as before.. just the issue is images keeps on flickering.. I am not getting why it behaving like this @stoffern

@bgoyal2222 sounds like you have issues with the FlatList rerendering on scroll / onEndReached
what does this function do? fetchNewsFeed
onEndReached={()=>{
if(this.state._stopLazyLoad == false){
return this.fetchNewsFeed()
}
}}

@stoffern that is for when the user reach to the of my feed.. to make a call and fetch further data and display

Is that an async call or are you slowing stuff down with it?

ya thats a fetch api call @dobrienlzd

A few ideas:
Where is the render row function?  Seems like that would be interesting to see!

Put a timer around this request so you know how long in milliseconds it take. Also just return stubbed data and see if you still notice the flickering.
Upgrade to 0.44.0 and let us know what happens?
Let's try to get the rending time on the item via callbacks.  See if there is a real difference between 0.42 and 0.43 or not.

Just pick whichever is easiest first. But FlatList get's lots of attention.

Thanks for the help.. I am not sure but it seems to be solved when i removed image prop resizeMode='contain'


Adds support for mentioning users in comments, reviews and model descriptions

Resolves #150 by adding mention support and mention notifications.


mWeb - New Workspace button is not visible correctly after deleting a workspace

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:
Action Performed:

Go to STG ND in Safari on IOS.
Create a lot of workspaces by Settings --> Workspaces --> New Workspace.
Now open any workspace settings and delete it.
Notice the New Workspace button

Expected Result:
The New workspace button should not change its position and show remain at the same place.
Actual Result:
The New workspace button is not visible and hides below the browser bar.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[x] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.43-0
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/7062dd48-b241-4b2a-a442-2437b8cb63f8
https://github.com/Expensify/App/assets/93399543/7bb08cd5-f3ba-4b7f-93af-2b9e4c24c3e4
Expensify/Expensify Issue URL:
Issue reported by: @Talha345
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1689877033802469
View all open jobs on GitHub

Reviewing today!

not reproduced, new workspace is above URL. Closing


Issues with the DynamoDB extension

Hello all,
I have been struggling with using the DynamoDbAsyncClient. Does anyone here have an idea what I may be doing wrong?
See https://github.com/quarkusio/quarkus/discussions/28884

This is the wrong place to submit issues for this subject.


Throwing java.nio.BufferUnderflowException during commands deserialization

During experimenting with the sample project (sbt 'project example' 'runMain aecor.example.App'), it turned out that when you send a command to aggregate (in particular, to TransactionAggregate), you get such error:
aecor.encoding.WireProtocol$Decoder$DecodingFailure: null
	at aecor.encoding.WireProtocol$Decoder$DecodingResult$.fromTry(WireProtocol.scala:42)
	at aecor.example.domain.transaction.TransactionAggregate$$anon$1$$anon$10.decode(TransactionAggregate.scala:38)
	at aecor.runtime.akkapersistence.AkkaPersistenceRuntimeActor.aecor$runtime$akkapersistence$AkkaPersistenceRuntimeActor$$handleCommand(AkkaPersistenceRuntimeActor.scala:174)
	at aecor.runtime.akkapersistence.AkkaPersistenceRuntimeActor$$anonfun$receiveCommand$1.applyOrElse(AkkaPersistenceRuntimeActor.scala:160)
	at akka.actor.Actor$class.aroundReceive(Actor.scala:517)
	at aecor.runtime.akkapersistence.AkkaPersistenceRuntimeActor.akka$persistence$Eventsourced$$super$aroundReceive(AkkaPersistenceRuntimeActor.scala:75)
	at akka.persistence.Eventsourced$$anon$1.stateReceive(Eventsourced.scala:681)
	at akka.persistence.Eventsourced$class.aroundReceive(Eventsourced.scala:192)
	at aecor.runtime.akkapersistence.AkkaPersistenceRuntimeActor.aroundReceive(AkkaPersistenceRuntimeActor.scala:75)
	at akka.actor.ActorCell.receiveMessage(ActorCell.scala:527)
	at akka.actor.ActorCell.invoke(ActorCell.scala:496)
	at akka.dispatch.Mailbox.processMailbox(Mailbox.scala:257)
	at akka.dispatch.Mailbox.run(Mailbox.scala:224)
	at akka.dispatch.Mailbox.exec(Mailbox.scala:234)
	at akka.dispatch.forkjoin.ForkJoinTask.doExec(ForkJoinTask.java:260)
	at akka.dispatch.forkjoin.ForkJoinPool$WorkQueue.runTask(ForkJoinPool.java:1339)
	at akka.dispatch.forkjoin.ForkJoinPool.runWorker(ForkJoinPool.java:1979)
	at akka.dispatch.forkjoin.ForkJoinWorkerThread.run(ForkJoinWorkerThread.java:107)
Caused by: java.nio.BufferUnderflowException: null
	at java.nio.HeapByteBuffer.get(HeapByteBuffer.java:151)
	at java.nio.ByteBuffer.get(ByteBuffer.java:715)
	at boopickle.DecoderSize.readByteArray(CodecSize.scala:235)
	at boopickle.BasicPicklers$$anon$5.unpickle(Pickler.scala:379)
	at boopickle.BasicPicklers$BigDecimalPickler$.unpickle(Pickler.scala:129)
	at boopickle.BasicPicklers$BigDecimalPickler$.unpickle(Pickler.scala:120)
	at boopickle.Pickler$$anon$1.unpickle(Pickler.scala:19)
	at boopickle.PicklerHelper$class.read(Pickler.scala:49)
	at boopickle.Default$.read(Default.scala:144)
	at boopickle.TuplePicklers$$anon$4.unpickle(TuplePicklers.scala:17)
	at boopickle.TuplePicklers$$anon$4.unpickle(TuplePicklers.scala:15)
	at boopickle.UnpickleState.unpickle(Pickler.scala:586)
	at aecor.example.domain.transaction.TransactionAggregate$$anon$1$$anon$10$$anonfun$decode$1.apply(TransactionAggregate.scala:39)
	at aecor.example.domain.transaction.TransactionAggregate$$anon$1$$anon$10$$anonfun$decode$1.apply(TransactionAggregate.scala:38)
	at scala.util.Try$.apply(Try.scala:192)

I didn't dive into this deeply, but I noticed that such error occurs if you perform deserialization in several steps using UnpickleState. If you do it at once like this:
val (command, fromAccountId, toAccountId, amount) = boopickle.UnpickleImpl[Tuple4[String, From[AccountId], To[AccountId], Amount]].fromBytes(bytes)

Then, it works ok.

Thank you, I’ll take a look


[zh] sync /architecture/cri.md

sync with en weight
try to improve something

content/zh-cn/docs/concepts/architecture/cri.md

Preview: https://deploy-preview-37680--kubernetes-io-main-staging.netlify.app/zh-cn/docs/concepts/architecture/cri/

/lgtm


V2 : Cleanup : Decompose OrthoNormalBasis into functional API

So, we have multiple things inside orthoBasis:

to2D ; used in reTesselateCoplanarPolygons
to3D ; used in reTesselateCoplanarPolygons
getProjectionMatrix ; used in connectors
getInverseProjectionMatrix; used in connectors

For the projection matrices:
note: a projection matrix, is just another matrix (as in a set of operations on mat4 s) so perhaps we can do something in that direction: just adding new functions to mat4 ?
The proposed (closed) PR #160 adds functionality to plane, toXYPlaneTransforms(). This function produces matching projection and inverse project between planes.

No longer relevant as this library is part of OpenJSAD.org


ARROW-8217: [R] Try unskipping failing test on Win32 in test-dataset.R

Neal found that the test passed in a debug build, let's see if the test still fails on master

Good grief. I'll start bisecting by hand on my fork to see if I can determine which patch fixed this. In the meantime we can merge this, I'm going to remove the commented-out code

OK, @nealrichardson @fsaintjacques I think the mystery is solved. This appears to have been fixed by b9b9ece9c (see the RTools runs at https://github.com/wesm/arrow/branches/all?query=bisect -- ignore the red X's and click inside to see whether the RTools run succeeded or failed). So apparently there was some memory corruption happening that only provoked a segfault in the 32-bit build.
Good question is whether valgrind would have detected this on Linux builds of the R library or similar.

😅 all's well that ends well? Merge this?

Well, what's weird is that the failing test should not have been affected by the bug that was fixed in b9b9ece, so there is still some mystery here of understanding where the bad memory access or undefined behavior originated. If it comes up again we can revert to this discussion and produce a build prior to b9b9ece so that we can reproduce the segfault again and get a gdb backtrace

+1, anyway, merging this


Added dashtools docker --init <image_name> command

TODO

Add to README
Add to changelog

Changed version to 1.7.0


Completable types block on reactive-streams subscriptions

onSubscribe is falling back to the Iterable (non reactive) subscribe in Value.
subscribe inside each completable should be overriden to delegate to the active reactive instance.

Release in 10.1.0


Implement table mode for smt list

Imported from Bitbucket issue 10,  created by apdavison on 06-26-2010 at 20:47, last modified: 06-29-2010 at 12:19

Done in r125:b98d2f263d97
Trac comment by apdavison on 06-29-2010 at 12:19


Use alloc/init for compatibility with MRR mode

When running in manual retain-release mode (i.e. not ARC), the NSRegularExpression object that is created in -stringByDeletingIgnoredCharacters should not be autoreleased (otherwise messages to the object will eventually result in a crash).

Whoops; good catch.


Background running again in scenario causing tests to fail

Hello,
I running tests on a mobile app and there seems to be a problem with the background. I think this error is within cucumber. As you can see on the images below, I'm running multiple tests with a background. However when the first scenario runs it fails because it runs the background again within the scope of the scenario after already running it.
I.e. In the Log In scenario, it is running the "Given I have accepted terms and conditions" step again even though it is not stated in this scenario and it has already run in the background.
The wierd thing is when I run the scenario in isolation, it runs perfectly.
Have you encountered this problem before? Is this a problem within cucumber?

@mattwynne

Which version of Cucumber are you using? We had some odd problems like this with the 1.x series but I'd be surprised to see anything like this with 2.x
Would you be able to create a simple way for us to reproduce this? Perhaps a very small repo? It might help you to diagnose / isolate where the problem is anyway.

Ah I'm using 1.3. So I will update and let you know if the problem still persists.
Thanks

I have currently catched this issue in one of my project that uses Calabash with Cucumber 1.3. After I removed an HTML formatter that was configured with --format 'Calabash::Formatters::Html' -o <%= html_report %>, the issue was gone.
You might want to check the formatters you have, it is reported at https://github.com/cucumber/cucumber-ruby/issues/892 that they can cause issues with background steps.

Updated my cucumber and it works now. Thanks