Safe in read-only mode even though correct owner is connected

Title/Description
We've recently had multiple users report that they are confused about seeing a "Read only" mode label. Initially I thought it was just users connecting the wrong wallets to the app. But now I have experienced the same issue.
When opening a Safe via URL in a new browser tab and blocknative automatically connects to my Metamask account that is set on the correct owner, I still end up in read-only mode.

Also a few other issues appear:

Settings shows 0 owners
Safe name changes to LOADED SAFE
Threshold in settings shows "2 out of 0"

Maybe this is related to the onboard.js implementation.
Environment

Browser: Chrome
App Version: 2.16.0.
Environment:

production (mainnet)

Wallet: Metamask

Steps

Open Safe in one tab (-> Settings, owners etc are loaded correctly.)
Open the same safe via direct link in another tab

Expected

Usual Safe name
Safe is not read only.

Observed

Safe name changes to LOADED SAFE
Safe is read only
0 owners is displayed in settings

Updates

Dec 3, Tobi: Added steps and more details.

The cause of this bug seems to be a race condition between useLoadSafe and useSafeScheduledUpdates
Seems that in some place of the code we are fetching the owners ok:
  const [[, remoteThreshold, remoteNonce, remoteOwners = []], safeInfo, localSafe] = await Promise.all([
    generateBatchRequests<[undefined, string | undefined, string | undefined, string[]]>({
      abi: GnosisSafeSol.abi as AbiItem[],
      address: safeAddress,
      methods: safeParams,
    }),
    getSafeInfo(safeAddress),
    getLocalSafe(safeAddress),
  ])

but then we are assigning [] to the owners list, I think the empty is coming from the localStorage, but I'm not 100% sure. Will need to do more research

@lukasschor just making sure: You don't have a reliable way to reproduce this yet, right?

Yes, for me it's reliably reproducible. I think a requirement is to already have the interface open in one tap and open in one tab and opening it in another one using a deep-link.

Yes, for me it's reliably reproducible. I think a requirement is to already have the interface open in one tap and open in one tab and opening it in another one using a deep-link.

That's it! Nice one, I'll add the steps to the ticket. Thanks!
@liliya-soroka that's the key to #1454  as well :)

Reopen as there are reported accounts that switch owner's connection status while connected to the safe (without any user interaction). No network or user switch, nor page refresh.

I'm moving this to "requirements in progress" because I wasn't able to reproduce the issue. It's possible that's already fixed or it just a race condition that is not easy to reproduce. I asked Leandro to let me know in case he founds the bug again
CC: @fernandomg @alfetopito

I'm moving this to "requirements in progress" because I wasn't able to reproduce the issue. It's possible that's already fixed or it just a race condition that is not easy to reproduce. I asked Leandro to let me know in case he founds the bug again
CC: @fernandomg @alfetopito

I'm also not able to reproduce this. We had reports of an unstable RPC endpoint. @alfetopito which RPC endpoint are you using? I'm on https://rpc.xdaichain.com/
I would actually close this one as long as it is not reproducible and/or the issue doesn't persist.
fyi @dasanra in case you had ideas on what the issue might be here.

I'm also not able to reproduce this. We had reports of an unstable RPC endpoint. @alfetopito which RPC endpoint are you using? I'm on https://rpc.xdaichain.com/
I would actually close this one as long as it is not reproducible and/or the issue doesn't persist.
fyi @dasanra in case you had ideas on what the issue might be here.

I was testing most of the time with Nifty wallet on chrome https://www.poa.network/for-users/nifty-wallet
Not sure what endpoint they are using though.
Would assume they use the default's xdai endpoint? https://www.xdaichain.com/for-users/converting-xdai-via-bridge/use-alternate-or-custom-json-rpc-endpoints

I was testing most of the time with Nifty wallet on chrome https://www.poa.network/for-users/nifty-wallet
Not sure what endpoint they are using though.
Would assume they use the default's xdai endpoint? https://www.xdaichain.com/for-users/converting-xdai-via-bridge/use-alternate-or-custom-json-rpc-endpoints

Thanks for the info, I'll try with nifty wallet today then!

Thanks for the info, I'll try with nifty wallet today then!

Seems like Nifty wallet uses https://dai.poa.network.
The alternative is https://rpc.xdaichain.com/ which is what they recommend here: https://www.xdaichain.com/for-developers/developer-resources
I wasn't able to reproduce with Nifty wallet on Chrome. But I still think it's related to the rpc.

Seems like Nifty wallet uses https://dai.poa.network.
The alternative is https://rpc.xdaichain.com/ which is what they recommend here: https://www.xdaichain.com/for-developers/developer-resources
I wasn't able to reproduce with Nifty wallet on Chrome. But I still think it's related to the rpc.

I'm reopening this issue because I found the issue while fixing #1845 on volta. Seems that when this line runs:
  const [[, remoteThreshold, remoteNonce, remoteOwners = []], safeInfo, localSafe] = await Promise.all([
    generateBatchRequests<[undefined, string | undefined, string | undefined, string[]]>({
      abi: GnosisSafeSol.abi as AbiItem[],
      address: safeAddress,
      methods: safeParams,
    }),
    getSafeInfo(safeAddress),
    getLocalSafe(safeAddress),
  ])

on checkAndUpdateSafe the batch call hangs from time to time and returns undefined

I also have this issue while trying to add the 4th owner of a safe using the Ledger wallet. I had a safe with 3 owners. I added the 4th owner and they have only read rights. I thought there was a random issue so I deleted them and added another wallet (a totally different ledger) to it and I have the same issue.

currently having this issue on mobile, iOS 15 beta


Update installation.md

I have added one additional format for the windows user which include adding to PATH and convert to .exe format for running it.,

What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Checklist:

[ ] Fixes #
[ ] Signed the commit for DCO to be passed
[ ] Labelled this PR & related issue with documentation tag

Can someone look onto this matter, I feel Steps like adding to path was written for Linux/Macos user but not for windows plus after extraction we receive the file in .txt format which need to be in .exe


Update his-update-center.md

Changing 2016 CU4 to CU5 since it is now released

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

File
Status
Preview URL
Details

his/his-update-center.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Print Id of the subgraph from which the given subgraph is obtained through single edge extension in the output

Hi Keren,
I have gone through your tool gBolt and it is very nice.
I have one small requirement in my project explained below.
Suppose, the current subgraph that is generated is as follows
t # 0 * 94
v 0 0
v 1 1
e 0 1 0
x: <list of all graph Ids in which this subgraph 't#0' is present>
From the above subgraph, below subgraph can be generated by adding single edge 'e 0 2 1'
t # 1 * 94
v 0 0
v 1 1
v 2 1
e 0 1 0
e 0 2 1
x: <list of all graph Ids in which this subgraph 't # 1' is present>
In my project, I also want the Id of the subgraph from which the given subgraph is generated through single edge extension. So, I want the output as shown below for each subgraph present in the final output of gBolt.
t # 1 * 94 p # 0
v 0 0
v 1 1
v 2 1
e 0 1 0
e 0 2 1
x: <list of all graph Ids in which this subgraph 't # 1' is present>
Here, 'p # 0' indicates the Id of the subgraph 't # 0' from which the subgraph with Id '1' is obtained.
Can you please incorporate this change in your code?
It will be really helpful for my project.
Appreciate your help in advance.

No problem, please wait for some days (no more than a week). I have a great amount of staff to finish recently.

Hi Keren,
Thanks a lot for the update.
Sure, I can wait till that time.
Please inform me once you incorporate those changes.
On Tue, Sep 19, 2017 at 2:56 AM, Keren Zhou<EMAIL_ADDRESS>wrote:

No problem, please wait for some days (no more than a week). I have a
great amount of staff to finish recently.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Jokeren/DataMining-gSpan/issues/17#issuecomment-330361169,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQCLAmFfjxCtN-_qEfnugwK8r1GhdZ5Qks5sjt_sgaJpZM4Pah7h
.

--
Regards
Parnika Paranjape

Hi Keren,
Sorry to bother you again.
When I studied the command line options for gBolt, I found that there is already a provision to print the parent Id of each subgraph in the final output of gBolt using '-parent true' option.
However, there is one small issue with this output.
This issue is described in the attached file (both .docx and .txt formats are attached)
Issue.txt
Issue.docx
It will be really helpful for my project if you could incorporate the changes?
Appreciate your help in advance.

Sure, I will fix it at this weekend.

@parnika26 I have added thread ids in the output file, could you check out the correctness for me?

Hi Keren,
Sorry for the late reply.
I was testing the code on multiple inputs so it took a while to respond.
The thread Ids are correct and the remaining output is also correct.
Thank you so much for your support :)
Just one small concern.
I tried running gBolt on the attached input file at support 0.5 and it gave error 'Bus error (core dumped)'.
My system configurations are as follows:

Ubuntu 17.04 (64 bit OS) 2) RAM :- 32 GB 3) Processor:- Intel® Xeon(R) CPU E5-2695 v3 @ 2.30GHz × 28

Input FIle:- Please remove '.txt' extension from input file if needed.
spect_heart_1_train_data.txt
Error File:-
gBolt_error.txt
I have also tried running gBolt on another system with same configurations and 64GB RAM but got same error.
Can you please try to run gBolt at support 0.5 for the given input file on your system and let me know what could be the reason for such error?
Appreciate your help in advance.

Sure, I will look into the problem

I cannot generate results after running on my system for one minute. So I guess some while later it will throw a similar problem as yours.
I notice your graph data is very data, indicating there will be many frequent patterns as long as you set a low support. e.g. 0.5 * 60 = 3.
I will try to locate specific lines in my codes.

Hi Keren,
Thanks a lot for your quick reponse.
Can you please inform me after the issue is fixed?
Appreciate your help in advance.

Hi Keren,
In the current version, the output of gBolt contains the graphs with atleast one edge in it.
However, in my project I also need the graphs with single vertex (i.e. no edge in it).
Can you please also include the graphs with single vertex (i.e. no edge) in the output of gBolt ?
It will be really helpful for my project.
Appreciate your help in advance.

@parnika26 I will fix it tomorrow.

@parnika26 done

Hi Keren,
I downloaded the latest version of gBolt having the changes done by you yesterday.
I tried running it on various datasets.
However, the output of gBolt still did not have the graphs with single vertex (i.e. no edge).
Also, after the execution it produces the error 'Segmentation fault (core dumped)'
I am attaching the output of gBolt for Chemical_340 dataset and it does not contain the graphs with single vertex (i.e. no edge).
Can you please look into this issue and try to resolve?
Appreciate your help in advance.
Chemical_340.t0.txt

You can tell me your command that crushes the program.
The output nodes are in a separate file called *.nodes

Hi Keren,

Here is the command:-  ./gbolt -input_file Chemical_340 -output_file gBolt_Output/Chemical_340 -pattern true -support 0.2
The output node file *.nodes was not created after I ran above command for Chemical_340 or any other data file.

Appreciate your help in advance.
Attaching the file containing error.
gBolt_Error.txt

@parnika26 fixed.
You should specify -nodes option on the command line in order to get the .nodes file.

Hi Keren,
It is working now and also producing .nodes file.
However, in the nodes file, the list of graphs Ids are not present for each vertex (i.e. node).
The graph Ids are present for all other subgraphs with atleast one edge.
Can you please add the graph Ids for each vertex in the nodes file ?
For example,
t # 0 * 47
v 0 5
x:
Appreciate your help in advance.

Hi Keren,
My earlier message was incomplete hence, sending it again.
It is working now and also producing .nodes file.
However, in the nodes file, the list of graphs Ids are not present for each vertex (i.e. node).
The graph Ids are present for all other subgraphs with atleast one edge.
Can you please add the graph Ids for each vertex in the nodes file ?
For example,
t # 0 * 47
v 0 5
x: 0 1 3 4 5 6 7 8 11 13 15 24 27 28 30 33 44 47 49 50 51 54 55 57 59 63 66 67 68 69 76 77 80 87 91
Appreciate your help in advance.

@parnika26 fixed


Custom sort option

First of all let me say that this is not an issue, but just a question.
My case: I need to order a table throught four different db columns in order to obtain a 'natural sort', but of course if I add an order by clause to the definition of the initial query this propagates to any further attempt of sorting.
i.e.:
query = db.session.query().select_from(MyTable).\      order_by(MyTable.col1, MyTable.col2, MyTable.col3, MyTable.col4)
At a first glance it works, but at the same time it prevent any other attempt of dynamic sorting of the table.
So what I'm asking for is:
How to define a custom sort option for one of the columns? Is it possible to attache a keyword argument to the column definition in order to obtain my need?
As far as I can understand from the source code I cannot figure it out if it is possible... sorry I'm not a true programmer!
Many thanks in advance

you should be able to define multiple sorting columns in the the javascript of datatables. I belive this supports multi column ordering


interface with BioStructures

It would be nice if one could directly call dssp_run and stride_run with a data structure from BioStructures directly.
That is fairly easy in the current state of the package, since the binaries are external programs that only read temporary PDB files. There are some details concerning the headers of the PDB files that both packages accept (or mmCIF files, for DSSP), and I have handled those (for PDB) in the current implementation. Something similar, or converting a structure to the PDBTools format before calling the dssp_run(::Vector{PDBTools.Atom}) may be necessary (and similarly for stride_run). Trivial things, though.
The problem with supporting the BioStructures data is (from my first impression) that this is fairly huge dependency, for such a simple package. Thus, I would be willing to provide this functionality though a package extensions, such that it is only loaded if BioStructures is loaded separately.

It may become not so light if we manage to wrap SESCA as well to compute CD spectra. I think making the extensions is fine here. I'll work on that.

It is now possible to run DSSP and STRIDE on BioStructures objects and get the secondary structure code from within BioStructures: https://biojulia.dev/BioStructures.jl/stable/documentation/#Assigning-secondary-structure.
Other output of DSSP/STRIDE may be added in future.

Cool. I guess we close this here, right? Or do you think there is an integration that can be useful still?

Yes probably close this one.


Update readme.md to fix broken image links due to reorg [BUG FIX]

Fix Many Broken Images on marketplace page https://github.com/microsoft/vscode-docs-authoring/issues/779

redid PR with several other modules adjusted too
https://github.com/microsoft/vscode-docs-authoring/pull/781


Resolve pytest warnings

Based on #1585 from maint-1.8, but targeted for main.
test_hash.py did not have any warnings, but was re-written using parameters, similar to the other PR.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 84.631%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2153

Relevant Lines:
2544

💛  - Coveralls

Thanks!
Sidenote: we should probably also just remove this test_doctest.py (it's only used for a single file test_binascii_hex.txt I think, and I don't see a reason why those can't be normal pytest tests (if it's not already covered in existing tests)

The aims of this PR have changed, as it removes test_doctests.py and test_binascii_hex.txt, which is more-or-less covered by test_wkb.py. The re-written test_hash.py remains.


Show all available messages in bubble

Fixes https://github.com/AtomLinter/linter-ui-default/issues/12

@steelbrain did you take into consideration giving priority to errors so that errors appear first in the bubble? Also could you take a screenshot to see how it looks like?

@satazor The base linter does not know anything about the severity of messages, so it can not sort them based on that, and even if we do that, providers are free to use any message type so it would fail there.
I can not take a screenshot right now, but it is like two messages together in one bubble, they look nice


[Distribution] publish client artifacts in public Maven repository

re-pack client modules into smaller number of artifacts?
use some maven plugin to do the job..

Done via OSS repository, tested on staging and snapshot repos. Waiting for release to promote to Maven Central.

exclude from promotion to Maven:

bagri-samples
bagri-test

do we need to deploy POM modules (bagri, bagri-server, etc)?

done! http://repo1.maven.org/maven2/com/bagridb/


query: Add Contains operator

Adding a |= operator to FrostDB.
It's basically LIKE '%$foo%' in SQL. It'll return true whenever the string contains the substring.

Do we need this if we could do this with regexp matching?

Interesting idea to use the regexp.
We still need this as it uses substring matching rather than full regexp which I assume to be a lot cheaper. We could benchmark this.

Interesting idea to use the regexp. We still need this as it uses substring matching rather than full regexp which I assume to be a lot cheaper. We could benchmark this.

Instead of regexp matching instead have the regexp operator downgrade itself into a Contains matcher if it detects it's regexp is a substring matcher.

I'll start fresh from the regexp angle.

What Thor said would be a great logical optimizer.


Arbitrary points localization implementation using compas.rpc

[ ] Add grasshopper file

 include localization from if_jamming into compas_rcf (and attribute)


Replace DataManager with Catalog/Database concepts

No issue to close
Description
replace data_manager and storage module with single catalog module that encapsulates all low level with binary data
Is it a feature that change user experience?
no
Client output
no

Pull Request Test Coverage Report for Build 459255048

1896 of 2206 (85.95%)  changed or added relevant lines in 41 files are covered.
9 unchanged lines in 5 files lost coverage.
Overall coverage decreased (-0.6%) to 89.304%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/catalog/src/lib.rs
4
6
66.67%

src/node/src/lib.rs
0
2
0.0%

src/node/src/query_engine/mod.rs
4
6
66.67%

src/catalog/src/sql/on_disk.rs
0
3
0.0%

src/catalog/src/in_memory.rs
231
237
97.47%

src/storage/src/lib.rs
10
19
52.63%

src/catalog/src/on_disk.rs
280
303
92.41%

src/schema_executor/src/lib.rs
233
266
87.59%

src/catalog/src/sql/in_memory.rs
335
372
90.05%

src/data_manager/src/lib.rs
56
109
51.38%

Files with Coverage Reduction
New Missed Lines
%

src/data_manager/src/lib.rs
1
57.26%

src/schema_planner/src/lib.rs
1
98.17%

src/query_analyzer/src/lib.rs
2
77.23%

src/schema_executor/src/lib.rs
2
89.52%

src/storage/src/persistent.rs
3
61.68%

Totals

Change from base Build 459225608:
-0.6%

Covered Lines:
11163

Relevant Lines:
12500

💛  - Coveralls

Pull Request Test Coverage Report for Build 459255048

1896 of 2206 (85.95%)  changed or added relevant lines in 41 files are covered.
9 unchanged lines in 5 files lost coverage.
Overall coverage decreased (-0.6%) to 89.304%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/catalog/src/lib.rs
4
6
66.67%

src/node/src/lib.rs
0
2
0.0%

src/node/src/query_engine/mod.rs
4
6
66.67%

src/catalog/src/sql/on_disk.rs
0
3
0.0%

src/catalog/src/in_memory.rs
231
237
97.47%

src/storage/src/lib.rs
10
19
52.63%

src/catalog/src/on_disk.rs
280
303
92.41%

src/schema_executor/src/lib.rs
233
266
87.59%

src/catalog/src/sql/in_memory.rs
335
372
90.05%

src/data_manager/src/lib.rs
56
109
51.38%

Files with Coverage Reduction
New Missed Lines
%

src/data_manager/src/lib.rs
1
57.26%

src/schema_planner/src/lib.rs
1
98.17%

src/query_analyzer/src/lib.rs
2
77.23%

src/schema_executor/src/lib.rs
2
89.52%

src/storage/src/persistent.rs
3
61.68%

Totals

Change from base Build 459225608:
-0.6%

Covered Lines:
11163

Relevant Lines:
12500

💛  - Coveralls


Jenkinsfile experiment

[ ] Try to get this working at all
[ ] Turn the scripts into a Makefile

Just noticed that I don't have write access to this repository and @ImmutableJenkins doesn't either, which will prevent Jenkins from setting the status of the commit here.

Also, let me know if something is missing on the workers to get this working.

There are 2 things missing:

local Ipfs (can be temporary instance), this is easy for me to work around
Java 8 - There is java 9, though it's quite new and still has a lot of compatibility issues. Downgrading it should be fine as J8 is still supported and most things don't need J9 features yet. I can work around this with docker if there is one


Add CUDA argument in evaluate.py to set the "is_cuda" flag in evaluate methods to False. (for non-Nvidia GPUs to use CPU)

Hi All,
I am using a Mac OS for my project so I am running into an issue when trying to evaluate my transformations. As I do not have Nvidia GPUs, I would like to use the CPU when working with PyTorch otherwise I would get an "AssertionError: Torch not compiled with CUDA enabled".
Mac OS users that do not have Nvidia GPU will have to set device = -1 to not use GPU:
MacOS: "AssertionError: Torch not compiled with CUDA enabled"
https://github.com/allenai/allennlp/issues/877
This seems to be stemmed from the fact that there is currently no way to change the is_CUDA flag that is being set to TRUE by default in the evaluate() method inside evaluate_text_classification.py to FALSE. (There is code to set the device to 0 or -1 based on the is_cuda flag.)
I am able to run my evaluations by changing the is_cuda flag in the code. It will probably be better to make it an argument so that future users who want to use CPUs instead of GPUs to be able to do it when running python evaluate.py -t [transformation] -task [task_type]
I will be happy to make the required changes if that's something we'd want.
Thanks,
Tim

Hi @timothy22000 : Thanks for bringing this issue to notice. Instead of introducing an argument in the evaluate.py script, I would suggest adding the following code in evaluate_text_classification.py to automatically detect if GPU is present or not and set the is_cuda flag accordingly in the evaluate method,
is_cuda=torch.cuda.is_available()

@kaustubhdhole : Let me know your thoughts.

Fix is merged. So, closing the issue.

Well, I solve this problem by changing CUDA into cpu where the bug happens.


XSS vulnerability

Project missing input sanitizers.
Example:
Simply adding a <img src=x onerror=alert(1) /> as a new "thought" will trigger an XSS:

Just a reminder.
This is a security issue that is probably being mimicked by other users.
If possible, it should get some attention.

Is this still an issue? I can't reproduce it on PR #36


Update homeassistant extension

Description

Sort StatesList in alphabetical order

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

What necessitated this?

Before the PR my sensors were not alphabetically sorted:

This is after the PR:

So we have a plan now for this PR to get it merged, right?


Mace doesn't support Clip or Constant

Onnx version 1.3.0, Mace version 0.10
I can optimize the onnx file using onnx_optimizer.py, but running convert fails with
Exception: Mace does not support onnx op type Clip yet
If I run convert with the unoptimized onnx file, I get a similar error:
Exception: Mace does not support onnx op type Constant yet
Any ideas when these ops will be supported?

Exception: Mace does not support tensorflow op type Enter yet
I have also this problem

again, Any ideas when these ops will be supported?


Re-running the training produces very different result

This is the notebook:
https://colab.research.google.com/drive/1857BKLnYP_3gspdYTwKRO7aD3lGUFYhg?usp=sharing
As you can see the last two cells are duplicates, but the first run always results in a completely bad score, and the second one is alright.
What is wrong here?

I am closing the issue since it has been recommended to use Flax instead in the readme file


📝 Add tiledbsoma scrna guide

subj

Awesome, Sergei! 😄
Small asks:
Can you use a plain S3 path here rather than the global variable from the settings?

Also here (let's avoid using the unfamiliar storage settings as much as possible, people just want to see paths in the context of a tiledbsoma store because all the complexity is within the object itself; you don't have to organize a big number of paths and you can simply 'remember' your storage path of your array -- this is the opposite to when you work with 10k artifacts with different paths that you want to query for):

Can you print versioning information about this artifact?


readme: update broken link

current link is broken, updated, not sure its the right link but it works

fixed in https://github.com/watson-developer-cloud/abap-sdk-nwas/commit/c60720f37810eb4f9a5a630d606ee4cca25b54c6, closing


blank page after login with PTC

I've been having this issue since yesterday, today I pulled the last package.json update. It does recognize wrong PTC passwords as I only get the blank screen when I enter the correct one. Thanks.

I can confirm that the blank page issue persists after the package.json update. Using Google login here.

Did you run npm install after pulling?

 I actually didn't pull because I've made some changes of my own to my local copy of the project and simply edited pogobuf in package.json. It's working after re-running npm install. Thanks.

Alright I know this is a noob question, but I downloaded the pre-built package. And I am also getting the blank screen error. I found the package.json file on in the folders but I'm not sure what to do with it on my end in order for the pokemon journal to work again. It was working fine yesterday, can anyone point me in the right direction?

It's working after running npm install. Thanks

Could you link to a guide on how to compile and run this or update the pre-made packages please?  THanks

How can I install the latest version?
If I execute
npm install -g pokemon-journal
it looks like a version<EMAIL_ADDRESS>is trying to be installed and I get errors
I don't know how npm works and why this old version 0.1.2 is used.
If on the other hand I get the latest version from github into a local folder this version looks like<EMAIL_ADDRESS>but I don't know how to use it:
D:\Git_Repositories\pokemon-journal>npm run dev
produces errors I described before:

<EMAIL_ADDRESS>dev: concurrently --kill-others "npm run hot-server" "npm run start-hot"

In other words: it looks like there is a version 0.2.0, but how to compile or execute it?

New release here https://github.com/billyvg/pokemon-journal/releases/tag/0.2.1

Thanks for compiling the new version man. Much appreciated.


Round/trim number of digits in microseconds -Inputstream

Hi ,
We have input stream which contains  xml
For example :
Expected Input
<feed ....> . One of node may have date time like this

................
<m:inline>
<d:Birth m:type="Edm.DateTimeOffset">1987-07-25T00:00:00.1234567Z</d:Birth>
................
</m:inline>

The above inputstream we are getting from Htttp call.
we would like to trim millsec to 3 digits i.e 1234567Z to 123Z in InputStream , so we have used ReplacingInputStream class
String search= "([0-9]{4}-[0-9]{2}-[0-9]{2}T[0-9]{2}:[0-9]{2}:[0-9]{2}.[0-9]{3})([0-9]{4})(Z)*";
Stringreplacement = "$1$3";
byte[] pattern = search.getBytes("UTF-8");
byte[] replacementBytes  = replacement.getBytes("UTF-8");
				ReplacingInputStream ris= new ReplacingInputStream(content,pattern,replacementBytes);
		byte[]  newBytes=readBytes(ris);
		System.out.println(Arrays.toString(newBytes));

The code is not working.Same value is  printed 1987-07-25T00:00:00.1234567Z
Note : The regular expression to search & replace is correct , we have tested with other approaches . We dont want to convert InputStream to String or Pasrse inputStream (SAX)  then replace to avoid Memory consumption . So we like your approach replacing in Input Stream , If it works that would be great.
Please do the needful.
Best Regards,
Showkath.

Any updates on this issue ?
Regards,
Showkath.


Size stat is overridden on save/reload

Hi I come across several issues with ToyBox would like to seek advices here:

Several equipment's feature still stick around with the character even it is already unequipped so is this the base game issue or the mod ?

Just wondering if the size selection under the stats tab intended to override and stick it coz if it does it doesn't seem to work properly I have a Leopard animal companion which I wanna to change the size up but it wouldn't stick from reloading. If it's not intended would it be asking too much to have such feature added as override the animal actual game size please?

There is a minor issue with the Armies General when adding skill before loading the blueprint it will crash however when it try to type something in the search field it doesn't trigger the blueprints loading at least not in the same Armeies tab. As long as the blueprints are loaded no crash when adding skills

Thank you so much for looking at it!!!!

Hi,
Thank you for looking into this!

for example you could equip the ring that storyteller gave you early in
the game then unequip it check the feature tab under party session for that
character the ring feature still there even though you already unequipped
it. Same for some buff like cooking buff from Acorn pie even you already
changed other recipe and gain other buff.

Later today I could post some screenshots here it's better explain that way.
I suspect it could be the buff duration setting (I set it to 2) which I'll
try it the default setting later today.

the actual size. I got a Leppard animal companion and max out level and
according to the wotr wiki it only get to medium size and I just wanna have
it be a size larger to large. I tried the change it but once save / reload
it changed back to medium.

Thank you very much!!!
On Mon, May 8, 2023, 04:03 xADDBx @.***> wrote:

Okay so I had some time to take a look:

Pretty sure not a ToyBox issue; you coud explain in detail on what
exactly you mean with feature stick around (which features from which
weapons; does this only happen with certain features/weapons or with all;
how long etc.)
Which one doesn't stick around; the visual one or the one that
changes the actual stat?
I'll move this one into a new issue

—
Reply to this email directly, view it on GitHub
https://github.com/cabarius/ToyBox/issues/873#issuecomment-1537529324,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVQOC3CIVYHAOMICVNWA7MTXE752TANCNFSM6AAAAAAXUMPCGU
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hi,
Here are the example of the feature persistency:

that is what she is currently equipped.

the old Carapace armor feature still hanging there.
Hope this will help explain better.
Thank You!!!

I think I figured out the reason for the persistency. Some items have feature "ranking" / "level" button and if I increased the default ranking it will stay there. I have just tested the above  Carapace armor again without modifying the ranking / feature level it won't stay there after unequipped.

Which log do you require to investigate?


Grep and check for bad unicode characters

The problem is that one bad translation will blow up Spring init
See https://sakaiproject.atlassian.net/browse/SAK-47085

Cool

Agreed, any rogue character will break the translations and cause secondary effects that are very difficult to debug

@stetsche be honest, how many hours it took to understand and fix #11003  ?

I've raised https://github.com/sakaiproject/sakai/pull/11006 for testing.

@mpellicer I asked you not to ask! :laughing: 10h...
@ottenhoff Nice test!

Yes sorry about the 10h. I prob wasted 6h on it a few months ago. It was very difficult to find the origin.

No problem, that's not your fault! I agree it's pretty tricky to spot


Interactive Tutorial - Configuring a Java Microservice

Error when build project using maven with error message : Could not build image: toomanyrequests: You have reached your pull rate limit.

/sig docs
/language en
/kind support

This issue is already reported in #37994 and #38018.
/traige duplicate

Hi @Harisalghifary,
I had given an explanation for the same issue here #37994, you can check it once.
I hope that will resolve your query.

@Harisalghifary,
Please confirm are you still facing the same issue.


Google Play Music has been updated and is no longer scrobbling.

The Google Play Music website has a new interface and now the last.fm scrobbler isn't recognizing when songs are playing, it needs to be updated also.
Thank you :)

This should be fixed now #602

Thank you very much! :)
Sent by Outlook for Android
On Sat, May 16, 2015 at 2:55 PM -0700, "Oliver Faircliff"<EMAIL_ADDRESS>wrote:
This should be fixed now #602

Reply to this email directly or view it on GitHub:
https://github.com/david-sabata/web-scrobbler/issues/605#issuecomment-102702223

It's only partially fixed. Songs played from Google Play Music website using version 1.47 are successfully scrobbled to Last.fm but as stand-alone tracks, not linked to the appropriate album.

@eocsor I've made a pull request that fixes this: https://github.com/david-sabata/web-scrobbler/pull/608

The album scrobbling should be fixed now too.


How to specify a provider?

Hello, I am trying to follow this tutorial but am not able to use the Azure app config to store my flags. I have tried to use just the application.yaml to pass the flags specified here but they do not seem to be being read. Is there a way to specify a different provider? I would ultimately like to store these in mongo alongside some other config that we store.

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

ID: 7af9b1df-7ce6-465b-bb32-f1cc91039085
Version Independent ID: a59d2917-6fe5-eb90-8222-f6f23933db56
Content: Tutorial for using feature flags in a Spring Boot app - Azure App Configuration
Content Source: articles/azure-app-configuration/use-feature-flags-spring-boot.md
Service: azure-app-configuration
GitHub Login: @mrm9084
Microsoft Alias: mametcal

It actually appears that the issue is not that it isn't reading my flags in application.yaml but that the sample thymeleaf is not handling the Mono being bound. If I just pass a Boolean it works fine. Seems like the demo is missing something?

@4everinbeta Thank you for bringing this to our attention. I'll review the demo and see if it needs adjustment.

@4everinbeta, sorry to hear the issue. If you are trying to use feature management with  Azure App Configuration you will want to look at the quickstart guide, https://docs.microsoft.com/en-us/azure/azure-app-configuration/quickstart-feature-flag-spring-boot which also has information of feature flags, specifically getting them from App configuration. Also if you want to check what the resulting project from this demo should look like we have a copy in our repo, https://github.com/microsoft/spring-cloud-azure/tree/master/spring-cloud-azure-samples/feature-management-sample.

Thank you @mrm9084 but I am actually trying to use it without Azure app config. Currently these applications are still run in our on-site datacenter and all configuration must live onsite as well. I was going to start with just application.yaml and move it to another source onsite such as mongo if I could get another provider to work.

@4everinbeta I found the problem with the demo this line model.addAttribute("Beta", featureManager.isEnabledAsync("Beta"));  Should be model.addAttribute("Beta", featureManager.isEnabledAsync("Beta")).block();

Thank you! One slight change (I put the .block() inside the parenthesis for addAttribute! Thank you so much! Do you happen to know if there is hot reload of file updates? I'm thinking no as that is not the behavior I'm seeing but noticed it is working that way in .net core.

@4everinbeta Refresh isn’t built into the package. A refresh event would need to be triggered to do that. If you implement ApplicationEventPublisherAware and she publish a refreshEvent, it will trigger an update. An example can be seen here https://github.com/microsoft/spring-cloud-azure/blob/master/spring-cloud-azure-appconfiguration-config/src/main/java/com/microsoft/azure/spring/cloud/config/AppConfigurationRefresh.java, refresh is on line 115. If you are interested in this evening added to the feature management package create an issue on this project https://github.com/microsoft/spring-cloud-azure.

Thanks again for the great information @mrm9084! I am going to create an issue as I feel that it makes sense for the java version to have as close as possible to parody with the .net version.


Loader > texture image doesn't contain valid dimensions in Safari 13

Expected Behavior
When loaded in Chrome, resources.sig.texture['width || height'] is correctly set to the image texture's dimensions (897 x 309)
Current Behavior
When loaded in Safari, these dimensions both come through as 1. :(
Possible Solution
?
Steps to Reproduce
Load up the codepen and inspect the console
Environment

pixi.js version: 5.1.15
Browser & Version: Safari 13.0.1
OS & Version: MacOS 10.14.6
Running Example: https://codepen.io/glomotion/pen/JjPVMZd

Ooops. its because im loading a webp. Nevermind! :)


Create extension for additional installers

To cover the following:

VSCode Extensions
Google Chrome extensions

This would cover:

[ ] googlechrome-editthiscookie
[ ] googlechrome-emberinspector
[ ] googlechrome-github-expandinizr
[ ] googlechrome-zenhub
[ ] vscode-csharpextensions
[ ] vscode-ember-cli
[ ] vscode-ember-frost
[ ] vscode-icons

@pascalberger you might be interested in this for some of your VSCode extensions as well.
@bcurran3 you might be interested in this for some of your googlechrome extensions.

@gep13 How would the extension look like for VSCode? And what is its advantage over directly installing the extension?

@pascalberger the extension would likely contain the two lines that you currently include in the chocolateyInstall.ps1 file, and the function would likely take a single parameter which is the name of the VSCode extension that is being installed.  The benefit being that "if" the installation mechanism for VSCode extensions ever changed, then you change it in one place, the chocolatey extension, instead of in all the packages that are currently doing it themselves in the chocolateyInstall.ps1 file.

@gep13 OK, got it :)

@gep13 any update on this one?

@AdmiringWorm I haven't forgotten about this one, just somewhat down the list of things to do just now 😢

Closing due to no activity

If I can figure out how to make a helper (which is on my todo list for at least one other idea), I'll do the Chrome extension helper.
New baby due in 4 days, behind in some work, and busy being lazy right now!

I now think that this is probably not worth it. Majority of packages should be embedded. As such, perhaps extensions for AU would be more welcome to have.


chore: oops forgot to update lockfile

Current dependencies on/for this PR:

master

PR #3   👈

This comment was auto-generated by Graphite.


New API method get_pod_for_build() (fixes #238)

Querying pods requires a different API endpoint as it is a Kubernetes API rather than specifically Origin, so I've added a new Openshift constructor parameter to allow for that.
There is a new PodResponse class, which OSBS.get_pods_for_build() returns a list of (cf. list_builds).
I've added a test case for it, which required some mock JSON. To fetch that, I also added a command line verb for it as well... which ironically has decreased overall test coverage.

Can one build have multiple pods? If not I'd prefer if the method returned the single pod (or None).

I've changed it so the method returns a single pod. Rather than returning None is there is no single pod to return, I've made it raise an exception for consistency with get_build().

Looks good! Wanna squash it before merging?

Re-based and squashed.

LGTM, merged.


Focus stacking?

Now that manual focus is possible on iOS 10+, automatic focus stacking is feasible. Does your framework support focus stacking?

Probably not, since I haven't done any work in that area.


Autocomplete off for docs

Dependant on #141 (how I've done it)
Currently the browser's default autocomplete can get in the way of our one:

I changed one line to stop that (Although it will seem like loads because it's comparing to master, not #141 )

I've rebased this upon v1-dev now that #141 has been merged

All fixed on the live build 😄


README doesn't explain anything.

Hello, can you please update the README to explain the setup.
Thank you

+1 - this sounds like a great plugin but the instructions are lacking

+1, I agree would be really useful to have better instructions. @RGSMA perhaps you could help out with some instructions if you have got it working.

Go to IFTTT and add the Webhooks service to your account

Note your api key in the Webhooks settings

Create Webhook notifications (see screenshot)

Update your JSON file (see screenshot)

Restart homebridge

Thank you! Was unable to figure out how to get the API key etc. in ifttt.


Confusing _ (underscore) for private, virtual, signals, callbacks, lifecycle functions

The current function naming api is confusing in it's _ use.
In the engine source (C++) it is used for the private functions. And in godot it is obviously used for lifecycle callbacks like _ready(d) or _input(d) but it is also used when creating signal connections via the editor _on_button_pressed().
But for builtin api's this is not always the case. There are a lot of virtual functions which can be overwritten by gdscript and are treated like callbacks which don't have a _ (underscore).
Example: editorImportPlugin
all the get_... methods which are basically callbacks (virtual functions) don't use the _ syntax.

Also mentioned here #14466

When you consider that nothing is really private in GDScript, the underscore is quite pointless. It can be an internal convention for users to make in their method names, but engine provided methods don't really need to be marked this way (and considering that they are called by the engine itself, they should be considered public anyway).
As said, it's not used consistently, so much that many contributors got it wrong, which made it more inconsistent. If it were up to me I would remove the underscore prefix from everything.
For the hidden methods and properties I think it's okay though. They don't show up in docs and aren't supposed to be called from the scripts anyway.

The way I always understood it is that any functions that could be considered responding to a signal/event (ready, input, on button pressed) should be prefixed with a _, so as to signify that they're not supposed to be called directly by other game code; and anything else (whether it's getting data or performing an action) should not. Personally, it's not confusing to me, but I can understand how someone coming from another language might be confused.
I am curious if you have any suggestions to better categorize the functions, though.

@LikeLakers2 yea when using just the engine I got the same impression than u.
Just recently I found inconsistencies with some virtual functions like get_name() from the EditorPlugin class. (But there are a lot of others too).
And when dealing with engine source code it has no coherent structure with gdscript (which is fine. the users should not care about the source code naming)
My proposal would be to rename all virtual, (auto-called function) to _function_name()
so that it becames a consistent indicator that those functions are called from inside the engine event handlers/value getters... and you don't have to be confessed where the function is used. or why it never gets called, but only declared.

isn't this a python-ish convention like PEP8 or something?? When you name a function which shouldn't be called outside a class, i.e. private, you add _ to its beginning.
I don't know how it is on the source code, but in the API it seems consistent to me. I mean, if you are connecting a notification to a method, this method should be called by the notification only, shouldn't it? I mean, you can get a pretty hard time debugging a method that is called both by a built-in notification (signal) and by a custom method.
And for lifecycle functions it makes even more sense...is not recommended at all to call _process or _ready outside of the class itself, or _draw. :thinking: So it seems that these would be equivalent to privatefunctions, wouldn't they?
Since GDScript, as Python, doesn't have public and private scopes, this is a really important convention. The problem, yes, can be it consistency, but it doesn't seem to be inconsistent to me, as an user.
As for the core, Since C++ has such scopes, I think they are only using this convention for user exposed methods, is this the case?

Well, you're not supposed to call EditorImportPlugin::import(), but it does not start with an underscore. If the engine calling it mean they're not private, then the same applied to the lifecycle callbacks. OTOH,
VisualScriptCustomNode has all virtual methods prefixed.
For _ready(). _process(), etc. I believe it's not recommended to call them at all, not even inside the class. They are more like some kind of magic methods than private ones.
In any case, there's no consistency at all. If the underscore convention is really useful, there should be a clear documented rule to apply it, otherwise each one thinks it's something different and it loses the meaning.

I think it's worth considering removing them entirely (including methods like _ready and _process). Is there a real utility for the underscore?
This change would require fixing conflicts (like get and _get) and it would greatly break compatibility.

I definitely advocate going through those compatibility breaking changes.
I like the _ since it allows me to type it and let autocomplete show me all the virtual functions.
I still prefer giving virtual functions (those which are bound with: 	BIND_VMETHOD(MethodInfo("_process", PropertyInfo(Variant::REAL, "delta")));) a naming hint. on_functions_name, or _function_name.

I also prefer having the underscores present for "virtual" concepts, but the inconsistencies need to be cleared, so I'd rather just add underscores to everything virtual that doesn't yet have it. If a method with an underscore is exposed to the scripting API, but isn't supposed to be called (like the import thing vnen mentioned), then why don't we just not make them visible in the scripting API?
It would also be nice if we could specify whether a method is virtual (intended to be overridden) vs. private (intended not to be called externally) in user-defined script functions, but I suspect that will be a task for #20318.

Feature and improvement proposals for the Godot Engine are now being discussed and reviewed in a dedicated Godot Improvement Proposals (GIP) (godotengine/godot-proposals) issue tracker. The GIP tracker has a detailed issue template designed so that proposals include all the relevant information to start a productive discussion and help the community assess the validity of the proposal for the engine.
The main (godotengine/godot) tracker is now solely dedicated to bug reports and Pull Requests, enabling contributors to have a better focus on bug fixing work. Therefore, we are now closing all older feature proposals on the main issue tracker.
If you are interested in this feature proposal, please open a new proposal on the GIP tracker following the given issue template (after checking that it doesn't exist already). Be sure to reference this closed issue if it includes any relevant discussion (which you are also encouraged to summarize in the new proposal). Thanks in advance!


Jörmungandr binary roundtrips

Issue Number
#219
Overview

I've added roundtrips testing decode (encode signedTx) === signedTx and the same for addresses.
As a workaround for throwing Binary encoders I used unsafePerformIO with Control.Exception.try

Comments

Generators now only generate valid transactions. Potential remaining points for objections:

Use of unsafePerformIO instead of monadicIO
Shrinkers - we could remove them


Adding 8 vs 9 info with codeblocks

Content must go through a pre-merge checklist.
Pre-Merge Content Checklist
This documentation has been checked to ensure that:

[x] The title, template, and date` are all set
[x] File does not use CRLF, but uses plain LF (hint: use cat -ve <filename> | grep '^M' | wc -l and expect 0 as a result)
[x] Has passed bart check

[x] Has run npm run build-index
[x] Has been manually tested by running in Spin/Bartholomew (hint: use PREVIEW_MODE=1 and run npm run styles to update styling)

[x] Headings are using Title Case
[x] Code blocks have the programming language set to properly highlight syntax and the proper copy directive
[x] Have tested with npm run test and resolved all errors

Just FYI; replaces https://github.com/fermyon/developer/pull/346


login.microsoftonline.com is not resolved in WSL

Environment
Windows build number:
Win32NT 10.0.18363.0 Microsoft Windows NT 10.0.18363.0
Your Distribution version:
Ubuntu 20.04
Whether the issue is on WSL 2 and/or WSL 1:
WSL1
Linux version 4.4.0-18362-Microsoft<EMAIL_ADDRESS>(gcc version 5.4.0 (GCC) ) #1049-Microsoft Thu Aug 14 12:01:00 PST 2020
Steps to reproduce
In a fresh installed Ubuntu 20.04
$ sudo apt update
$ sudo apt upgrade
$ sudo apt install -y apt-transport-https lsb-release gnupg curl

Then the following command gives the error
$ nslookup login.microsoftonline.com
;; Truncated, retrying in TCP mode.
;; Connection to fec0:0:0:ffff::1#53(fec0:0:0:ffff::1) for login.microsoftonline.com failed: connection refused.

Change the /etc/resolv.conf and add google ones <IP_ADDRESS> and <IP_ADDRESS>. Try again the nslookup to that specific domain it will work.
$ nslookup login.microsoftonline.com
Server:         <IP_ADDRESS>
Address:        <IP_ADDRESS>#53

Non-authoritative answer:
login.microsoftonline.com       canonical name = a.privatelink.msidentity.com.
a.privatelink.msidentity.com    canonical name = prda.aadg.msidentity.com.
prda.aadg.msidentity.com        canonical name = www.tm.a.prd.aadg.trafficmanager.net.
www.tm.a.prd.aadg.trafficmanager.net    canonical name = san.current.a.prd.aadg.trafficmanager.net.
Name:   san.current.a.prd.aadg.trafficmanager.net
Address: <IP_ADDRESS>
Name:   san.current.a.prd.aadg.trafficmanager.net
Address: <IP_ADDRESS>
Name:   san.current.a.prd.aadg.trafficmanager.net
Address: <IP_ADDRESS>
Name:   san.current.a.prd.aadg.trafficmanager.net
Address: <IP_ADDRESS>
Name:   san.current.a.prd.aadg.trafficmanager.net
Address: <IP_ADDRESS>
Name:   san.current.a.prd.aadg.trafficmanager.net
Address: <IP_ADDRESS>
Name:   san.current.a.prd.aadg.trafficmanager.net
Address: <IP_ADDRESS>
Name:   san.current.a.prd.aadg.trafficmanager.net
Address: <IP_ADDRESS>

Change the Windows DNS settings to the google ones too. Try the nslookup again and it will fail.
$ nslookup login.microsoftonline.com
;; Truncated, retrying in TCP mode.
;; Connection to fec0:0:0:ffff::1#53(fec0:0:0:ffff::1) for login.microsoftonline.com failed: connection refused.

It only fails for this domain as far as I've seen and only when the DNS servers are the same in Linux and Windows.
Expected behavior
The WSL should successfully resolve the domain without changing the resolv.conf  file, using the same DNS configured in windows.
Actual behavior
The WSL can't resolve the domain.

Please collect WSL logs and post the backlink to the feedback item here. Also please identify any third-party networking software that has been installed on Windows. Any insight on what /etc/resolv.conf was pointing at before you changed it to eights would also be useful. It is returning ipv6 addresses.

Thanks for the answer! The thing about the logs is that I can't see what is been collected and this is happening on my corporate machine so I have to be careful with the information that is posted. Do you know a way to review the log information before publishing it?
About third party applications, I have installed Cisco AnyConnect for the VPN (but the problem happens out of the VPN) and McAfee as anti-virus. I can't identify any other software but as I said, this is my work PC so it might have something else.
This is my resolv.conf content before the change:
# This file was automatically generated by WSL. To stop automatic generation of this file, add the following entry to /etc/wsl.conf:
# [network]
# generateResolvConf = false
nameserver <IP_ADDRESS>
nameserver <IP_ADDRESS>
nameserver fec0:0:0:ffff::1

The workaround is to remove auto generated values by WSL in /etc/resolv.conf and adding google NS.
# [network]
# generateResolvConf = false
nameserver <IP_ADDRESS>

Bit old but I think I know why this happens as I ran into this issue today. This was in Kali Linux and not using WSL. The DNS response for login.microsoftonline.com is too large to fit into the DNS UDP packet. Because the original RFC (before eDNS) limited the packet size to 512 bytes. You can see that nslookup shows the UDP response was truncated, so it fails over to use TCP.
└─$ nslookup login.microsoftonline.com
;; Truncated, retrying in TCP mode.
I was using network manager for DNS (default in Kali). I changed over to systemd-resolved and it fails over to TCP just fine now.

This could be related to https://github.com/microsoft/WSL/issues/5256

This happened to me when running a vscode dev container through Docker and WSL2. /etc/resolv.conf was showing the Docker internal DNS server (?): <IP_ADDRESS>. Changing this to any other DNS server (<IP_ADDRESS> or <IP_ADDRESS>) fixed the issue for me. The comment from @n0-0x01 put me on the right track. Thank you!

Is this issue supposed to be fixed for real? I'm running Linux servers on bare metal and I get random DNS timeouts (we have 5 second timeout for resolving the DNS name) when trying to resolve login.microsoftonline.com. As such, this issue seems wider than just WSL and it causes random failures for the OpenID Connect protocol documented here: https://docs.microsoft.com/en-us/azure/active-directory/develop/v2-protocols-oidc which requires connecting to login.microsoftonline.com to validate the ID token.
I'm not aware of any other DNS failures at the same time so this is specifically about resolving domain login.microsoftonline.com.


Python: Parallel function calls result in event loop RuntimeErrors

Describe the bug
When testing the sample chat function in azure_chat_gpt_api.py in a parallel setting, the calls to kernel.invoke all succeed, but a RuntimeError is raised about the event loop being closed.
To Reproduce
Modified azure_chat_gpt_api.py sample to call the chat function in parallel with asyncio.gather:
# Copyright (c) Microsoft. All rights reserved.

import asyncio
import logging

import semantic_kernel as sk
import semantic_kernel.connectors.ai.open_ai as sk_oai
from semantic_kernel.contents.chat_history import ChatHistory
from semantic_kernel.utils.settings import azure_openai_settings_from_dot_env_as_dict

logging.basicConfig(level=logging.WARNING)

system_message = """
You are a chat bot. Your name is Mosscap and
you have one goal: figure out what people need.
Your full name, should you need to know it, is
Splendid Speckled Mosscap. You communicate
effectively, but you tend to answer with long
flowery prose.
"""

kernel = sk.Kernel()

service_id = "chat-gpt"
chat_service = sk_oai.AzureChatCompletion(
    service_id=service_id, **azure_openai_settings_from_dot_env_as_dict(include_api_version=True)
)
kernel.add_service(chat_service)
req_settings = kernel.get_prompt_execution_settings_from_service_id(service_id=service_id)
req_settings.max_tokens = 2000
req_settings.temperature = 0.7
req_settings.top_p = 0.8
req_settings.auto_invoke_kernel_functions = True

chat_function = kernel.create_function_from_prompt(
    prompt=system_message + """{{$chat_history}}{{$user_input}}""",
    function_name="chat",
    plugin_name="chat",
    prompt_execution_settings=req_settings,
)

history = ChatHistory()
history.add_user_message("Hi there, who are you?")
history.add_assistant_message("I am Mosscap, a chat bot. I'm trying to figure out what people need.")

async def process_one_chat(user_input):
    answer = await kernel.invoke(
        chat_function,
        user_input=user_input,
        chat_history=history
    )
    print("Answer:", answer)

async def main() -> None:
    inputs = ["how are you", "what can you tell me", "what do you do", "tell me a story"]
    tasks = [process_one_chat(user_input) for user_input in inputs]
    await asyncio.gather(*tasks)

if __name__ == "__main__":
    asyncio.run(main())

This results in Mosscap's answers being printed, followed by 4 instances of:
Exception ignored in: <function _ProactorBasePipeTransport.__del__ at 0x000001959CBC6AF0>
Traceback (most recent call last):
  File "C:\Users\sawidder\AppData\Local\miniconda3\envs\sk-dev\lib\asyncio\proactor_events.py", line 116, in __del__
  File "C:\Users\sawidder\AppData\Local\miniconda3\envs\sk-dev\lib\asyncio\proactor_events.py", line 108, in close
  File "C:\Users\sawidder\AppData\Local\miniconda3\envs\sk-dev\lib\asyncio\base_events.py", line 719, in call_soon
  File "C:\Users\sawidder\AppData\Local\miniconda3\envs\sk-dev\lib\asyncio\base_events.py", line 508, in _check_closed
RuntimeError: Event loop is closed

Expected behavior
Event loop handling should properly close all async loops.
Screenshots
N/A
Platform

OS: Windows
IDE: VS Code
Language: Python
Source: pip package version 0.9.1b1

Additional context
It may be helpful to add integration tests for this case :)

Hi @sarah-widder, the issue you're experiencing, RuntimeError: Event loop is closed, occurs in Windows when async tasks try to complete after the asyncio event loop has been closed, often during cleanup at the end of a script. A workaround is to explicitly close the event loop and create a new one before the program exits, ensuring all async operations are completed and resources are properly cleaned up, thus preventing the error.
When I add the following to the script, there's no longer an error:
if __name__ == "__main__":
    loop = asyncio.get_event_loop()
    try:
        loop.run_until_complete(main())
    finally:
        loop.close()
        asyncio.set_event_loop(asyncio.new_event_loop())

Thanks @moonbox3 this works for me as well!

Thanks for confirming. I will close this issue -- please re-open or create a new issue, as needed.


Various  VPC Updates

Add create_vpc for conditional module deployments
Update to latest cloud_workspace module 0.0.2
Update  min terraform version to 1.0.1

Resolves:  https://github.com/aws-ia/terraform-aws-vpc/issues/6
Resolves:  https://github.com/aws-ia/terraform-aws-vpc/issues/4
Resolves:  https://github.com/aws-ia/terraform-aws-vpc/issues/3


Setup semantic release

⚠ Once merged, a v1 will be published because semantic-release does not support version starting with 0.x.x
The explanation: https://semantic-release.gitbook.io/semantic-release/support/faq#can-i-set-the-initial-release-version-of-my-package-to-0-0-1
At the same time, I've removed size-limit in favor of preact-comporessedSize-action
What's next ?

I'll create a v1 branch
Make a pull request to support RHF 7 and publish the V2

awesome! thank you @jorisre


fix delete pull head ref for DeleteIssue

fix #19655

Codecov Report

Merging #20032 (eb77538) into main (909fb6a) will increase coverage by 0.02%.
The diff coverage is 0.00%.

:exclamation: Current head eb77538 differs from pull request most recent head eda99e7. Consider uploading reports for the commit eda99e7 to get more accurate results

@@            Coverage Diff             @@
##             main   #20032      +/-   ##
==========================================
+ Coverage   46.79%   46.82%   +0.02%
==========================================
  Files         968      968
  Lines      134270   134274       +4
==========================================
+ Hits        62835    62872      +37
+ Misses      63741    63698      -43
- Partials     7694     7704      +10

Impacted Files
Coverage Δ

routers/web/repo/issue.go
36.71% <0.00%> (-0.06%)
:arrow_down:

services/issue/issue.go
31.84% <0.00%> (ø)

modules/indexer/issues/bleve.go
0.00% <0.00%> (-64.54%)
:arrow_down:

modules/indexer/issues/db.go
0.00% <0.00%> (-48.00%)
:arrow_down:

services/repository/check.go
0.00% <0.00%> (-13.14%)
:arrow_down:

services/repository/review.go
50.00% <0.00%> (-12.50%)
:arrow_down:

modules/indexer/issues/indexer.go
43.70% <0.00%> (-10.00%)
:arrow_down:

services/cron/tasks_basic.go
90.90% <0.00%> (-9.10%)
:arrow_down:

modules/process/manager_exec.go
86.04% <0.00%> (-6.98%)
:arrow_down:

modules/indexer/stats/indexer.go
51.06% <0.00%> (-6.39%)
:arrow_down:

... and 24 more

Continue to review full report at Codecov.

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

Could you explain the bug and how this PR resolve it?

Could you explain the bug and how this PR resolve it?

just a nit. the format of pull head ref is  refs/pull/{index}/head, not refs/pull/{index}.

Please send backport to v1.17

Honestly could we please try to write better PR opening comments. Explain what the PR is doing and why.


Repository 1.0.6 not available in HACS

hi,
don't know if it a bug but HACS don't accept 1.0.6.
got the message
<Integration home-assistant-libs/python-matter-server> Repository structure for 1.0.6 is not compliant
thx Rudi

Duplicate of #113
Please, check if your issue was already reported before creating a new one.
Thanks! 👍
../Frenck

Sorry 🙈
Rudi
Am 01.12.2022 um 18:11 schrieb Franck Nijhof @.***>:

Duplicate of #113https://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fhome-assistant-libs%2Fpython-matter-server%2Fissues%2F113&data=05|01||c7f9792e7a5b4e0ccd1808dad3bf1f2e|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|638055115052476739|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=oqxxflZxJO5qCs3ZxW6lWslgXjcjpRepvBIZnp%2BvQC8%3D&reserved=0
Please, check if your issue was already reported before creating a new one.
Thanks! 👍
../Frenck
—
Reply to this email directly, view it on GitHubhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fhome-assistant-libs%2Fpython-matter-server%2Fissues%2F121%23issuecomment-1334089874&data=05|01||c7f9792e7a5b4e0ccd1808dad3bf1f2e|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|638055115052476739|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=IrN1qRk5e17jE%2BDVBRgAeH0HVc9cZ%2BPNGwhq8r6bnPE%3D&reserved=0, or unsubscribehttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAK3OPIW5BQEEMPZDJGWIBGDWLDL47ANCNFSM6AAAAAASRAQMSE&data=05|01||c7f9792e7a5b4e0ccd1808dad3bf1f2e|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|638055115052476739|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=wy7dWZnzcACtNbNmjP1XYGYtzxFA%2FIqp4P5Z%2FJ7py2E%3D&reserved=0.
You are receiving this because you authored the thread.Message ID: @.***>


Remove the conditionals between ns and ns-pub

this is facilitating the workflow described in https://trello.com/c/JSjcrW4H/16-nightshade-public-and-iviewer-missing#comment-5c8aa690f9edb563c6b6db5b . The goal is to have just one ansible environment (playbook and other files) which governs both ns and ns-pub servers and also, to have iviewer on the ns-pub.
Further PR on MT will follow, to go forward with the other steps in https://trello.com/c/JSjcrW4H/16-nightshade-public-and-iviewer-missing#comment-5c8aa690f9edb563c6b6db5b
cc @sbesson @manics

Can you delete https://github.com/openmicroscopy/prod-playbooks/blob/master/templates/omero-web-config-for-webapps_ns-web-pub.j2 since it won't be used.

@manics done

Travis is still failing. Even after restart, the error is different though, originally there was the error fixed by https://github.com/openmicroscopy/prod-playbooks/pull/160, now there is a new one which I cannot decipher

@manics travis failed again

Conflicting PR. Removed from build TRAINING-merge#270. See the console output for more details.
Possible conflicts:

Upstream changes

nightshade-webclients.yml

--conflicts

Changes look fine. I'm didn't follow the pub-omero testing, but I agree with @manics that I can see having this deployed even if pub-omero is not completely consistent with ns-web yet.

Merging and running in check-mode against ns-*


Fix the bug that data cannot be saved due to repeated requests three …

Fixes: #3753

It looks like you're losing the selected field - at least in some scenarios. I agree with the first part of your fix. Can you try the following to see if it addresses your issue?
async addNewMapping(selectedField: Field, compoundSelection: boolean) {
if (!compoundSelection) {
this.cfg.showMappingDetailTray = false;
this.cfg.mappings!.activeMapping = null;
}
const mapping: MappingModel = new MappingModel();
if (selectedField != null) {
mapping.addField(selectedField, false);
this.updateTransition(mapping);
this.selectMapping(mapping);
await this.notifyMappingUpdated();
} else {
this.selectMapping(mapping);
}
}

It looks like you're losing the selected field - at least in some scenarios. I agree with the first part of your fix. Can you try the following to see if it addresses your issue?
async addNewMapping(selectedField: Field, compoundSelection: boolean) {
if (!compoundSelection) {
this.cfg.showMappingDetailTray = false;
this.cfg.mappings!.activeMapping = null;
}
const mapping: MappingModel = new MappingModel();
if (selectedField != null) {
mapping.addField(selectedField, false);
this.updateTransition(mapping);
this.selectMapping(mapping);
await this.notifyMappingUpdated();
} else {
this.selectMapping(mapping);
}
}

thanks, this following can address my issue，I reworked the code according to yours, you see if there is any problem？

Can you file a bug report, squash the commits into one and fix the commit message?
https://github.com/atlasmap/atlasmap/wiki/AtlasMap-Developer-Memo#commit-message