Make EVSE mockable and add mock

Describe your changes
This set of changes makes EVSE derived from an interface similar to what we did with ComponentStateManager,
and adds a mock to our set of mocks for v201.
Checklist before requesting a review

[x] I have performed a self-review of my code
[x] I have made corresponding changes to the documentation
[x] I read the contribution documentation and made sure that my changes meet its requirements

CC: @shankari

It looks like after a rebase you need to also update
std::map<int32_t, std::unique_ptr<Evse>>& evses; and explicit SmartChargingHandler(std::map<int32_t, std::unique_ptr<Evse>>& evses);.
and change the the EvseInterface type
Except for me this looks good!

Updated :)

Contributes to #361


HDDS-7438. Add a createStreamKey method to OzoneBucket.

What changes were proposed in this pull request?
//OzoneBucket
  public OzoneDataStreamOutput createStreamKey(String key, long size)
      throws IOException {
    ...
  }

We propose to add the above method to OzoneBucket since there is a similar createKey in OzoneBucket.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-7438
How was this patch tested?
Added a new unit test.

@captainzmc , thanks a lot for reviewing and merging this!


⚠ Empornium has degraded performance

In cfe8518, Empornium ($EMP) experienced degraded performance:

HTTP code: 500
Response time: 1495 ms

Resolved: Empornium performance has improved in fe8b609 after 41 minutes.


Send Apilytics version info together with metrics

Codecov Report

Merging #7 (772e11a) into master (aa0e9ff) will not change coverage.
The diff coverage is 100.00%.

:exclamation: Current head 772e11a differs from pull request most recent head 16b9ed8. Consider uploading reports for the commit 16b9ed8 to get more accurate results

@@            Coverage Diff            @@
##            master        #7   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            3         4    +1
  Lines           68        76    +8
  Branches         6         7    +1
=========================================
+ Hits            68        76    +8

Impacted Files
Coverage Δ

apilytics/fastapi.py
100.00% <ø> (ø)

apilytics/__init__.py
100.00% <100.00%> (ø)

apilytics/core.py
100.00% <100.00%> (ø)

apilytics/django.py
100.00% <100.00%> (ø)

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 aa0e9ff...16b9ed8. Read the comment docs.


Can't import Bestiary 4 - Graeae

Can't import statblock:
Graeae      CR 5/MR 2
XP 1,600
Graeae (Pathfinder RPG Bestiary 4 134)
NE Medium monstrous humanoid (evil, mythic)
Init +3; Senses blindsense 30 ft., darkvision 60 ft.; Perception +10
Defense
AC 19, touch 13, flat-footed 16 (+3 Dex, +6 natural)
hp 65 (6d10+32)
Fort +3, Ref +8, Will +8
DR 5/epic; SR 16
Weaknesses eye of the graeae
Offense
Speed 30 ft., fly 60 ft. (good)
Melee 2 claws +8 (1d6+2)
Special Attacks luck ripple, mythic power (2/day, surge +1d6)
Spell-Like Abilities (CL 6th; concentration +5)
At will—arcane sight, fly, undetectable alignment (DC 17)
5/day—ill omen[APG]
3/day—augury, enthrall (DC 18), feast of ashes[APG] (DC 18)
1/day—ray of exhaustion (DC 19)
Statistics
Str 14, Dex 17, Con 12, Int 23, Wis 13, Cha 8
Base Atk +6; CMB +8; CMD 21
Feats Combat Casting, Iron Will[M], Toughness
Skills Bluff +5, Craft (alchemy) +15, Fly +16, Intimidate +8, Knowledge (arcana) +12, Perception +10, Spellcraft +12, Stealth +12, Survival +10, Use Magic Device +5
Languages Aklo, Common, Giant, Goblin, Sylvan
SQ coven, fate casting
Ecology
Environment any
Organization solitary, a pair, or coven (3-12)
Treasure standard
Special Abilities
Blindsense (30 feet) (Ex) Sense things and creatures without seeing them.
Combat Casting +4 to Concentration checks to cast while on the defensive.
Coven (Ex) Can work magic with 2 other hags to cast certain spell like abilities.
Damage Reduction (5/epic) You have Damage Reduction against all except Epic attacks (weapons with a +6 bonus).
Darkvision (60 feet) You can see in the dark (black and white only).
Eye of the Graeae (Su) Must remain within 30 ft of mystic eyeball, or become blind and lose all Su/Sp abilities.
Fate Casting (Su) As a full-rd action, use 1 power to answer a question as if divination.
Fly (60 feet, Good) You can fly!
Iron Will [Mythic] Roll Will save vs. spells, Sp, or Su abilities from non-mythic foe twice (take higher).
Luck Ripple (DC 19) (Su) As a swift action, 1 creature in 30 ft gets +2  or -2 to AC, attacks, saves, skills (choose 1) for d6 rds (Will neg).
Spell Resistance (16) You have Spell Resistance.
 I used Hero Lab to produce this statblock. For greater accuracy, you might want to use one of the online resources. Hero Lab can truncate some of the text -- or inject errors due to a lack of data. All the other statblocks from Bestiary 4 imported -- a total of more than 85% imported!


fix: also check for "EISDIR" in mkdirp

Fixes #10544.
Though the EISDIR error code is not mentioned in the POSIX standard,
FreeBSD and memfs both throw this error when trying to run mkdir on
/. So essentially we should treat it the same as EEXIST.
References:

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=59739
https://github.com/streamich/memfs/pull/326/

What kind of change does this PR introduce?
A bugfix
Did you add tests for your changes?
I'm not sure how I should add tests, because this issue only happens on BSD systems.
Does this PR introduce a breaking change?
No
What needs to be documented once your changes are merged?
Nothing

/cc @sodatea friendly ping

I think I need to work out a more generic way to deal with the root directory case, instead of relying on the error code because different OSes😅
I'll try to fix it this weekend.


ERROR: Could not consume arg: --config_name=local_ollama

I am trying to install this R2R with Ollama locally and following this document :  https://r2r-docs.sciphi.ai/cookbooks/local-rag
Could you confirm if this document is up to date and correct because even when followed to the dot it gives errors?
Do we need to clone the git repo to get it working with Ollama?
I already have ollama running on my system.
conda create -n r2r python=3.11 -y && conda activate r2r
pip install 'r2r[all]'
pip install 'r2r[local-embedding]'
mkdir r2r
cd R2R
touch local_ollama
-- and then pasted below config in local_ollama file:
{
"embedding": {
"provider": "sentence-transformers",
"base_model": "all-MiniLM-L6-v2",
"base_dimension": 384,
"batch_size": 32
},
"eval": {
"provider": "local",
"frequency": 0.0,
"llm":{
"provider": "litellm"
}
},
"ingestion":{
"excluded_parsers": {
"gif": "default",
"jpeg": "default",
"jpg": "default",
"png": "default",
"svg": "default",
"mp3": "default",
"mp4": "default"
}
}
}
and then I ran following command:
python -m r2r.examples.quickstart ingest_as_files --no-media=true --config_name=local_ollama
extractions in t=5.57 seconds. - 2024-06-21 07:39:39,108
r2r.pipes.embedding_pipe - INFO - Fragmented the input document ids into counts as shown: {UUID('f0c63aff-af59-50c9-81fc-2fe55004c771'): 17, UUID('c9bdbac7-0ea3-5c9e-b590-018bd09b127b'): 233, UUID('b722f1ec-b90e-5ed8-b7c8-c768e8b323cb'): 5, UUID('c996e617-88a4-5c65-ab1e-948344b18d27'): 3108, UUID('ba77307d-6c8a-549f-812a-3558697e2842'): 23, UUID('4a4fb848-fc03-5487-a7e5-33c9fdfb73cc'): 31, UUID('1a9d4d3b-bbe9-53b9-8149-67806bdf60f2'): 18, UUID('ef66e5dd-2130-5fd5-9bdd-aa7eff59fda5'): 11, UUID('c5abc0b7-b9e5-54d9-b3d3-fdb14af4d065'): 2094} - 2024-06-21 07:39:40,005
Time taken to ingest files: 31.94 seconds
{'processed_documents': ["File 'got.txt' processed successfully.", "File 'aristotle.txt' processed successfully.", "File 'pg_essay_1.html' processed successfully.", "File 'pg_essay_2.html' processed successfully.", "File 'pg_essay_3.html' processed successfully.", "File 'pg_essay_4.html' processed successfully.", "File 'pg_essay_5.html' processed successfully.", "File 'lyft_2021.pdf' processed successfully.", "File 'uber_2021.pdf' processed successfully."], 'skipped_documents': []}
ERROR: Could not consume arg: --config_name=local_ollama
Usage: quickstart.py ingest_as_files --no-media=true -

hey @fahdmirza - thanks for flagging, I will look into reproducing, my guess is that the release has fallen slightly out of date.

tested this today, the issue is solved with the latest image, can you confirm?

Now it has this error:
(r2r) Ubuntu@0068-kci-prxmx10127:~/R2R$ python3 -m r2r.examples.quickstart ingest_as_files --no-media=true --config_name=local_ollama
2024-06-23 21:22:45,567 - INFO - r2r.core.providers.vector_db_provider - Initializing VectorDBProvider with config extra_fields={} provider='pgvector' collection_name='demo_vecs'.
2024-06-23 21:22:45,624 - INFO - r2r.core.providers.embedding_provider - Initializing EmbeddingProvider with config extra_fields={'text_splitter': {'type': 'recursive_character', 'chunk_size': 512, 'chunk_overlap': 20}} provider='ollama' base_model='mxbai-embed-large' base_dimension=1024 rerank_model=None rerank_dimension=None rerank_transformer_type=None batch_size=32.
2024-06-23 21:22:46,622 - INFO - r2r.core.providers.llm_provider - Initializing LLM provider with config: extra_fields={} provider='litellm'
R2RApp.init, config =  <r2r.main.assembly.config.R2RConfig object at 0x7b950ee10e50>
ERROR: Could not consume arg: ingest_as_files
Usage: quickstart.py ingest_as_files - <group|command>
available groups:      USER_IDS | default_files | file_tuples | r2r_app |
user_ids
available commands:    analytics | app_settings | delete | document_chunks |
documents_overview | evaluate | ingest_documents |
ingest_files | logs | rag | search | serve |
update_documents | update_files | users_overview
For detailed information on this command, run:
quickstart.py ingest_as_files - --help

Hey Fahd,
Can you confirm whether or not you are still seeing issues after the latest docker has been published today?


Failed to establish the environment: The detected CUDA version (10.1) mismatches the version that was used to compile PyTorch (11.7). Please make sure to use the same CUDA versions.

Hi, thanks for your wonderful work!
When I establish the environment for running the ops/make.sh. A error occurs as follows:
The detected CUDA version (10.1) mismatches the version that was used to compile
PyTorch (11.7). Please make sure to use the same CUDA versions.
However, I have installed the exact pytorch==1.13 and cuda version==11.7.

How can I solve this problem?
Thanks in advance!

please check the cuda version under the directory from torch.utils import cpp_extension; print(cpp_extension.CUDA_HOME).

Thank you for your quick response. I checked in this way and found out it is usr/local/cuda-10.1.
So what can I do next？
Thanks again.

A simple workaround maybe creating a symbolic link to your install coda-11.7, i.e., cd /usr/local; sudo ln -s /your/path/to/cuda-11.7 cuda.

Hi, thank you for your response. I cannot use the sudo order, so I found another way to solve this problem. Write in here to remind me:

conda activate NMRF
select the right cuda-nvcc version in https://anaconda.org/nvidia/repo, e.g.,
conda install nvidia/label/cuda-11.7.0::cuda-nvcc
select the right cuda-libraries-dev in https://anaconda.org/nvidia/cuda-libraries-dev, e.g., and install with conda
conda install nvidia/label/cuda-11.7.0::cuda-libraries-dev
then, running this:
export CUDA_HOME=/your/environment/path
run the setup.py to see if there is any wrong
according to the specific error, set the symbolic link to the environment, .e.g.,
export LD_LIBRARES_PATH=/xxxx/.conda/envs/NMRF/lib ls -la /xxx/.conda/envs/NMRF/lib/libcudart.so

Thank you again for helping me identify the difference between torch.version.cuda and cpp_extension.CUDA_HOME.


Custom font isn't loading on custom theme for Material UI

Current Behavior 😯
I'm using a custom font in my React web application. So I wrote this code:
import React, { FunctionComponent } from 'react'
import { createMuiTheme, MuiThemeProvider } from '@material-ui/core/styles'
import SofiaProLightTtf from '../../assets/font/sofia-pro-light.ttf'
import SofiaProTtf from '../../assets/font/sofia-pro-regular.ttf'

const sofiaPro = {
  fontFamily: 'Sofia Pro',
  fontStyle: 'normal',
  fontWeight: 100,
  src: `url(${SofiaProTtf})`
}

const sofiaProLight = {
  fontFamily: 'Sofia Pro Light',
  fontStyle: 'normal',
  fontWeight: 100,
  src: `url(${SofiaProLightTtf})`
}

const theme = createMuiTheme({
  typography: {
    fontFamily: 'Sofia Pro',
    body1: {
      fontFamily: 'Sofia Pro Light'
    }
  },
  overrides: {
    MuiCssBaseline: {
      '@global': {
        '@font-face': [sofiaPro, sofiaProLight]
      }
    }
  }
})

const Theme: FunctionComponent = ({ children }) => (
  <MuiThemeProvider theme={theme}>{ children }</MuiThemeProvider>
)

export default Theme

But it isn't working. So I tried to customize the font using plain CSS.
I found a workaround removing the overrides property in createMuiTheme and with this CSS:

But it is a very bad workaround...
I opened a question on Stack Overflow but nobody answered, so I think that it's a bug.
I'm using the latest Material UI version:
"@material-ui/core": "4.7.0",
"@material-ui/lab": "4.0.0-alpha.34",

Expected Behavior 🤔
We need be able to load a custom font using only createMuiTheme, without any custom CSS.
Steps to Reproduce 🕹

Create a new project. You'll need to add this rule on your webpack.config.js:

  {
    test: /\.ttf$/,
    use: 'file-loader',
    exclude: /node_modules/,
  },

Add some custom fonts on your project, and copy and paste the first code block on this issue

On your index.tsx, use:

import React from 'react'
import ReactDOM from 'react-dom'
import Theme from './theme'

const App = () => (
  <Theme>

hello world

  </Theme>
)

ReactDOM.render(<App />, document.getElementById('app'))

The custom font wasn't load.

Context 🔦
I need to use a custom font. Beside I found a workaround, I would like to have a better code.
Your Environment 🌎

Tech
Version

Material-UI
v4.7.0

React
16.12.0

Browser
Chore, Firefox and Safari

TypeScript
3.7.2

@oliviertassinari Thank you for you notice quickly my issue, but I think that it's a bug, not a support.
I already opened a question on Stack Overflow and nobody answered that.
I'm sorry if I'm doing something wrongly.

Using regular CSS sounds like a good solution too.

@oliviertassinari Well... I think that it isn't a good solution.
I wrote a slightly better code using tsx:
const Theme: FunctionComponent = ({ children }) => (
  <>


Refactor FindBlobsByTags method to accept OptionalParameters

Summary
Add OptionalParameters to FindBlobsByTags to provide the possibility of setting marker and maxresults. This is required to use result paging.
Work Item(s)
Fixes #893
Fixes AB#524630

Is it possible to get this one backported to BC 24.x?

Most certainly. I think I will backport the entire module before the 24.2 release to all changes backported.


Help - About PCB and Redox Manuform

Hello there,
I'm a hardware rookie, and I've never touched anything about hardware before. But I think the Redox Manuform model is really nice, and I would really want to have a keyboard like this of my own. So I would like to ask: Can the PCB from this project be used directly for the Redox Manuform model or what modifications do I need to make to make the provided PCB suitable for the Redox Manuform?

Looking at that I think it will be a fully hand wired keyboard. You don't use a pcb, just lots of wires and patience. A tricky project for your first hardware, but not impossible!

From: Jetiaime @.>
Sent: Friday, December 30, 2022 3:10:55 PM
To: mattdibi/redox-keyboard @.>
Cc: Subscribed @.***>
Subject: [mattdibi/redox-keyboard] Help - About PCB and Redox Manuform (Issue #159)
NetApp Security WARNING: This is an external email. Do not click links or open attachments unless you recognize the sender and know the content is safe.
Hello there,
I'm a hardware rookie, and I've never touched anything about hardware before. But I think the Redox Manuform model is really nice, and I would really want to have a keyboard like this of my own. So I would like to ask: Can the PCB from this project be used directly for the Redox Manuform model or what modifications do I need to make to make the provided PCB suitable for the Redox Manuform?
—
Reply to this email directly, view it on GitHubhttps://github.com/mattdibi/redox-keyboard/issues/159, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASXN7W4UO7KCSJCC7A7RVG3WP2DH7ANCNFSM6AAAAAATMVS4PA.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Hello @Jetiaime,
as @Chrismelba-Instaclustr already said the Redox Manuform is a handwire project. The Redox PCBs won't fit in the case and can't be used directly.


⚠ Documentation has degraded performance

In 5e08cbd, Documentation (https://docs.tonicpow.com) experienced degraded performance:

HTTP code: 200
Response time: 12944 ms

Resolved: Documentation performance has improved in 43bdefa.


Troubleshooting MaSIF-site errors with "./data_prepare_one.sh" for multiple PDBs of my protein complex

Hello,
I'm having troubles trying to run the "./data_prepare_one.sh" script for my protein complex of interest (NALCN) using MaSIF-site through Docker. I have no problem running the script for 4ZQK_A as shown in the Docker tutorial for MaSIF-site.
However, when I tried to run the script for the 4 different pdb-files for my protein complex of interest (7W7G, 7SX3, 7SX4, and 7WJI). I encountered the following errors:
For 7W7G_A, 7SX3_X, 7SX4_X, and 7WJI_X (by X, I mean any chain):

When running the ./data_prepare_one.sh for 7W7G_B, get this error:

I'm able to successfully run the script for 7W7G_C and 7W7G_D.
I suspect that the errors are due to the size or complexity of the protein chains, as A and B are substantially larger than C and D. However, since the small chains cannot be run on the other PDBs, and the error is different for 7W7G_A and 7W7G_B, I believe other factors might also be involved in these errors.
Cheers

I had the same error before. It occurs if you take ./data_prepare_one.sh with only one chain as input. You need to comment out this line: python -W ignore $masif_source/data_preparation/01-pdb_extract_and_triangulate.py $PDB_ID\_$CHAIN2

Hello, I'm having troubles trying to run the "./data_prepare_one.sh" script for my protein complex of interest (NALCN) using MaSIF-site through Docker. I have no problem running the script for 4ZQK_A as shown in the Docker tutorial for MaSIF-site. However, when I tried to run the script for the 4 different pdb-files for my protein complex of interest (7W7G, 7SX3, 7SX4, and 7WJI). I encountered the following errors: For 7W7G_A, 7SX3_X, 7SX4_X, and 7WJI_X (by X, I mean any chain):

When running the ./data_prepare_one.sh for 7W7G_B, get this error:
I'm able to successfully run the script for 7W7G_C and 7W7G_D. I suspect that the errors are due to the size or complexity of the protein chains, as A and B are substantially larger than C and D. However, since the small chains cannot be run on the other PDBs, and the error is different for 7W7G_A and 7W7G_B, I believe other factors might also be involved in these errors.
Cheers

Hi! can you give me a hint on how you can analyse local PDB files?


Interfaces and string return types swallow exceptions

During testing with exception flows, I noticed something curious. When the return type of an endpoint is either an interface or a string, the thrown exception is swallowed by a framework specific exception.
internal sealed class TestSomethingEndpoint : EndpointWithoutRequest<string>
{
    public override void Configure()
    {
        Get("/test");
    }

    public override Task<string> ExecuteAsync(CancellationToken ct)
    {
        throw new Exception("test exception");
        return Task.FromResult("message");
    }
}

Instead of the expected message test exception, I get the exception Uninitialized Strings cannot be created. with stack trace
   at System.Runtime.CompilerServices.RuntimeHelpers.GetUninitializedObject(Type type)
   at System.Runtime.Serialization.FormatterServices.GetUninitializedObject(Type type)
   at FastEndpoints.Endpoint`2.InitResponseDTO()
   at FastEndpoints.Endpoint`2.get_Response()
   at FastEndpoints.Endpoint`2.ExecAsync(CancellationToken ct)
   at Microsoft.AspNetCore.Diagnostics.ExceptionHandlerMiddleware.<Invoke>g__Awaited|6_0(ExceptionHandlerMiddleware middleware, HttpContext context, Task task)

A similar thing happens when I return an interface.
internal sealed class TestSomethingEndpoint : EndpointWithoutRequest<IEnumerable<string>>
{
    public override void Configure()
    {
        Get("/test/exception");
    }

    public override Task<IEnumerable<string>> ExecuteAsync(CancellationToken ct)
    {
        throw new Exception("test exception");
        return Task.FromResult(new[] { "message" }.AsEnumerable());
    }
}

Similar to the previous example, but this time the exception reads Cannot create an instance of an interface.
   at System.Runtime.CompilerServices.RuntimeHelpers.GetUninitializedObject(Type type)
   at System.Runtime.Serialization.FormatterServices.GetUninitializedObject(Type type)
   at FastEndpoints.Endpoint`2.InitResponseDTO()\r\n   at FastEndpoints.Endpoint`2.get_Response()
   at FastEndpoints.Endpoint`2.ExecAsync(CancellationToken ct)
   at Microsoft.AspNetCore.Diagnostics.ExceptionHandlerMiddleware.<Invoke>g__Awaited|6_0(ExceptionHandlerMiddleware middleware, HttpContext context, Task task)

Also, as a note, for some reason there is no downcasting allowed in the return type. When I specify the return type as IEnumerable<string> and I return an List<string>, there is a compilation exception: Cannot convert expression type 'System.Threading.Tasks.Task<System.Collections.Generic.List>' to return type 'System.Threading.Tasks.Task<System.Collections.Generic.IEnumerable>'.  I need to cast it to the corresponding interface via .AsEnumerable().

This is a bug in version 5.1.0, this is fixed in version 5.2.1.


weex内置websocket模块，调用时报堆栈溢出错误

This issue is outdated for a long time and will be closed now. You can create a new one if you still have questions.


Remote Settings update

When organisation starts using custom domain, remote setting has to be updated. Currently package does not allow to update remote setting, as recognizes that setting was already created, despite the fact, that URL is no longer valid. Entry has to be deleted from 'Remote Settings' setup node manually and then re-created using Welcome page.
minor #nicehave

Thanks, yeah that is a bit of a pain, marked as an enhancement. Cheers! 👍

I thought I'd bump this up again. We had a consulting client come to us in confusion when the DLRS configuration page stopped working after they (without asking us) enabled My Domain. We had to advise them to delete the setting and recreate. If it could update the existing setting that'd be wonderful.

Yeah once this enhancement is done, this will not be an issue. Architecture wise this is a better fix. https://github.com/afawcett/declarative-lookup-rollup-summaries/issues/607.

Interesting. Would trigger deployment still depend on the API though?

Yes, the tool would still need to use SOAP Metadata API wrapper, it will be a while if unlikely that they will allow the native MD API to publish code tbh. But at least making the above enhancement reduces the day to day / week to week need once the triggers are deployed.

Makes sense. I guess the remaining use case would still be, admin sets up DLRS, then later on either 1) has an instance migration, 2) creates a sandbox, or 3) enables My Domain. Next time they try to deploy a trigger on a new object they'd still be prompted to create the Remote Site Setting. If DLRS could be smart enough at this point to update the RSS rather than attempt unsuccessfully to create a new one, that'd still have value.

Thanks @camisotro will up the priority on this one.

Would the simplest solution be to change  to  in the client-side callout?

Fixed in v2.11

I am getting this error again after the most recent release and the workaround listed above is no longer working. Currently process:

Click "Remote Site Setting" button on Welcome tab
Remote Setting is created with "https://thedesaifoundation--dlrs.visualforce.com" domain
I changed this Remote Setting name and then repeated but the same error still shows up: "Salesforce Metadata API connection failure, click the link below to configure."

Advice?

Same here: pushing the "Create/Update Remote Site Setting doesn't solve the "Salesforce Metadata API connection failure" issue.

I for the first time now get this error. Have uninstalelled/re; put in remote site entry manaully; tried in sandbox. Nothing. Are others still getting this? Any new ideas?

I am getting the same error that @stevebackman and @danielconde13 described above, pushing the "Create/Update Remote Site Setting doesn't solve the "Salesforce Metadata API connection failure" issue.  Any new ideas to try or solutions?

There have been a lot of Warnings from Salesforce around Custom Metadata access for Spring '20 which a lot of people were upgraded to over the weekend.  Now for users to access Custom Metadata (Which is vital to DLRS), you need to give users access to the specific Custom Metadata via their Profile or a Permission Set.  I would try that and report back.

Thanks @JimBTek I looked into the Custom Metadata access changes for Spring '20 as you recommended and that looks like it was the problem, things are working correctly after making the changes recommended in the Salesforce documentation at  https://releasenotes.docs.salesforce.com/en-us/winter20/release-notes/rn_forcecom_development_custom_metadata_require_cruc.htm

@JimBTek  @jimhubert  I tried adding the dlrs Custom Metadata in the System Administrator profile, create/update the remote site settings and still I'm still receiving this issue.

Hello @JimBTek I am facing hte same issue and curious if another solution was found? I uninstalled/reinstalled. Then I deleted the remote site and recreated manually.  I also assigned the Custom Meta Data access to my profile via permission sets and profile.  Any suggestions would be greatly appreciated!

Hey all - please note - that if you can enable My Domain on the org - this whole connection issue goes away - since the tool then leverages a platform feature that allows SF API access without a remote site.

@afawcett  I have an org getting this issue with My Domain enabled. They see this error when they go to the Manage Lookup Rollup Summaries tab. I've removed the existing remote site setting and re-created it. I turned off both lock sessions options in session settings.  I still see this message in the debug of the Welcome screen
"Web service callout failed: WebService returned a SOAP Fault: INVALID_SESSION_ID: Invalid Session ID found in SessionHeader: Illegal Session faultcode=sf:INVALID_SESSION_ID faultactor="


Fix/get pytwin logger exposure

Codecov Report

Merging #70 (430a640) into main (b5e119a) will decrease coverage by 0.28%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #70      +/-   ##
==========================================
- Coverage   87.06%   86.78%   -0.28%
==========================================
  Files           9        9
  Lines        1794     1794
==========================================
- Hits         1562     1557       -5
- Misses        232      237       +5


units for deeplabcut tracking of body points

2019_07_01_Nsyb_NLS6s_walk_fly2.npz has a key dlc that stores the output of deeplabcut tracking of 8 points on the fly. These values range is 0~460, but what's the unit?
@bendichter

I asked about this and last I checked they didn't have unit conversion working


Gamepad support

Added gamepad support for GLFW and the web. Also added a demo to show how to use it.

Coverage increased (+0.1%) to 41.011% when pulling e20783904c1b9f1f1ebf79d3b77178381f98bb15 on gamepad-support into d6f9c6aafb3d9124495edf7115b1b2fb0c35b9c8 on master.


Clecio's NRM ES dashboard moved to core accounts

Describe the issue
NRM is paying for the ES, we want to move it to core accounts to take that burden off them
Which Sprint Goal is this issue related to?

Note: 'Milestone' is a ZenHub term that we use synonymously with 'Sprint'.

The 'Milestone' description (created in ZenHub) should clearly list Sprint Goals. This section should indicate which 'goal' this issue is related to.
Additional context

ES = Elastic Search

Definition of done

Clecio needs to be able to interface with ES front end
Consult with Clecio about how to handle Dev/Test/Prod

Consulted with NRM OneTeam Tech lead for additional background on the need described in this ticket. The team is taking the approach of building a PoC over the next several months and has made arrangement to avoid/reduce costs so shuffling their resources around to reduce cost is not a priority from their perspective. Consulted CPF PO and concluded that this ticket could be closed and perhaps the concern re-evaluated in ~6 months once the PoC is complete or further along.


metrics every 30m, 2 fails in a row to trigger

Change to have 2x failed checks in a row, but check twice as fast.
I assume the original intention was to run every 30m, not once per 60m
(on :30).

:+1:


Add test for validating ArgoCD UI login

What type of PR is this?
/kind enhancement
What does this PR do / why we need it:
As a QE trying to automate manual tests, we want to have a test validating the ArgoCD UI login workflow as part of the Kuttl test suite.
We have seen bugs with ArgoCD UI login with dex failing many times with our release candidates. There have been delays in identifying them as they are UI tests and currently, we don't have any Automation in place for UI testing.  We need to find a way to add this test as part of the Kuttl suite perhaps by testing the API endpoints so that we find any bugs related to the login workflow as quickly as possible.
Have you updated the necessary documentation?

[ ] Documentation update is required by this PR.
[ ] Documentation has been updated.

Which issue(s) this PR fixes:
Fixes https://issues.redhat.com/browse/GITOPS-2724
Test acceptance criteria:

[ ] Unit Test
[ ] E2E Test

How to test changes / Special notes to the reviewer:
To test, you can run:
kubectl kuttl test ./test/openshift/e2e/sequential --config ./test/openshift/e2e/sequential/kuttl-test.yaml --test 1-103_validate_argocd_ui_login

/test all

Closing this, please reopen if required


UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 0: ordinal not in range(128)

when I try to run the preprocessing I get the following error on MacOS:
  File "preprocessing.py", line 92, in <module>
    stop_words = [replace_umlauts(token) for token in stopwords.words('german')]
  File "preprocessing.py", line 52, in replace_umlauts
    res = res.replace('ä', 'ae')
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 0: ordinal not in range(128)

when using
python preprocessing.py dewiki.xml corpus/dewiki.corpus -psub

nevermind, finally got it working: with python3 everything works fine 🙂

Thanks for pointing this out. Would it be helpful, e.g. to always provide $ python3 .... examples in the readme?

I don't know - maybe at least mention it somewhere... although: now it is mentioned here 🙂

Ok. It is already mentioned at the beginning of the readme:


Search not working

Good afternoon, I am making this post because I have just redownloaded slimCat on my new hard disk and I've noticed something not working.
I'm currently using Puma 4.11 (and it's not even telling me to get the Slow Cheetah version like it used to before my old hard disk died last week) and when I go to the search tab... It's totally empty.

As you can see from the picture, I even tried to type stuff in and see if it'd let me do anything, but it'll constantly tell me that I need to narrow down the search 'cos it's getting too many results.
Checking around, I noticed another issue on here where it was suggested to delete the "search.xml" and did that, but it didn't solve the issue, sadly. What should I do to fix this?
Thanks in advance.

4.11 is no longer being maintained, so when the searching system was changed, it broke. If you want search, use 5.xx.


[MM-8338] Option to send as message when an invalid slash command is entered

Summary
Currently when a user enters an invalid slash command, they are asked to add a space at the beginning to allow for sending their message, if they wish to circumvent the processing of the slash command.
This ticket's feature is to allow the user to click a link within the displayed error to "force send" the message, skipping the slash command processing. The link is put into focus so pressing "enter" for a second time will send the message as well.
Ticket Link
Jira Ticket
GitHub Ticket
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] Ran make check-style to check for style errors (required for all pull requests)
[x] Ran make test to ensure unit and component tests passed
[x] Added or updated unit tests (required for all new features)
[x] Has UI changes
[x] Includes text changes and localization file (.../i18n/en.json) updates
[x] Touches critical sections of the codebase (auth, posting, etc.)

In order to preserve the text that caused the error, in case the user has changed the text since, the text is stored on the error at this.state.serverError.submittedMessage. I'm storing it on the error, as it is only relevant for as long as the error is displayed.

I have introduced an options parameter in the doSubmit() method in the create_post component, with the options for options.message for overriding what message should be submitted, and options.ignoreSlash, to signal that we want to avoid any slash command processing.
doSubmit = (e, options = {}) => {
...
const post = {};
post.message = options.message || this.state.message;
...
if (post.message.indexOf('/') === 0 && !options.ignoreSlash) {

The frontend is able to determine if the error is the invalid command error by the following logic:
error.server_error_id === 'api.command.execute_command.not_found.app_error';

The error string is built client-side, so the error message received from the server is ignored when the server_error_id matches this.

Moving comment over from GitHub ticket

I some questions regarding this feature.
If the user changes the text after encountering the error:

should the error continue displaying?
should clicking the "send as message" link clear the edited text?

As of now, the error does continue displaying, and the text is cleared out on forced submission of the error'd text.
Also, currently the invalid command error occurs when the beginning word is just '/' . I want to verify that this is expected behavior since we're looking at invalid commands.

@esethna the feature is complete, ready for testing

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: https://i-08403df01cb3fd171.test.spinmint.com
Test Admin Account: Username: sysadmin | Password: sysadmin
Test User Account: Username: user-1 | Password: user-1
Instance ID: i-08403df01cb3fd171

rebased since PR is out of sync with tip

The logic for handling keypresses to give focus to the main post textbox is strict about
only allowing character or number keys to make the textbox gain focus. You'll notice that if you
press backspace while having the upper chat in focus, the textbox will not gain focus.
Relevant lines:
components/create_post.jsx#L660
utils/post_utils#L161
I think adding the case to handle backspace for just this scenario of the invalid slash command
may not be the right solution here. If we instead make the textbox keep focus after submit, pressing enter at that time could force submit the message the same way the clickable link in the error message does. The point of giving the link focus instead was to make it so a regular submit from the textbox could never force submit the message, is this correct? Letting the textbox keep focus may not be the right solution either.
Here are some options I can think of:

Decide it's okay that backspace doesn't give the textbox focus
Add the case in the key listener to listen for backspace, possibly just for this invalid slash command case. I think this decision would require review
Let textbox keep focus, allow second press of enter to force submit the message. This kind of defeats the purpose of the link in the error message

I would lean to option 3. I don't think it defeats the purpose of the link in the error message as some users will still use the mouse to click it.
To be clear though ^, we should only send the message on the second "ENTER" if the message has not changed. If there were edits to the command, and the command trigger still does not exist, the error message should appear again.

@esethna going with option 3, PR is up to date with requested changes

Spinmint test server destroyed

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: https://i-0bd719f0ce026d886.test.spinmint.com
Test Admin Account: Username: sysadmin | Password: sysadmin
Test User Account: Username: user-1 | Password: user-1
Instance ID: i-0bd719f0ce026d886

Spinmint test server destroyed

requested changes have been, sorry if this ping is unnecessary

requested changes have been, sorry if this ping is unnecessary

No problem. Ping actually brought me here again . I don't have this in my review queue so i would have missed it if not for the mention. Thanks 👍
Don't hesitate to mention me if there is any update.

No problem. Ping actually brought me here again . I don't have this in my review queue so i would have missed it if not for the mention.

Thanks, I still have my PR etiquette/workflow training wheels on. Any constructive criticism on that would be welcomed!

@sudheerDev updated with fix for nested FormattedMessage components


[Event Request] Table 36 "Sales Header" - OnBeforeSetDefaultSalesperson

We are migrating our code from NAV 2018 to BC16.4. In the Table 36 "Sales Header", in the procedure SetDefaultSalesperson, we need to add our code at specific places. To do so, we need a IsHandled flag and an event publisher, like this :

With this integration event :

Do you think it could be possible ?
Thank you for your work.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


self-signed cert on IdP causes signature validation errors?

i'm not entirely sure what is going on here, or if this is even a bug, but i cannot get the sls endpoint to verify the signature provided by the IdP, failing with:
signatures.c:346(xmlSecOpenSSLEvpSignatureVerify) obj=rsa-sha1 subject=EVP_VerifyFinal msg=signature do not match errno=18
searching seems to suggest the underlying xmlsec library does not trust the cert chain, which seems plausible, because the cert is self signed...
it should be noted that other endpoints (sso, acs, and slo) are functioning properly, but IIRC do not require signing (i only enabled this when i was told our domain requires it).
i was unable to find reference documents on how signing with HTTP-Redirect binding is suppose to work, but reading the code tells me the the GET query is used verbatim, after first removing Signature=..., at which point the remaining query string is validated using the provided Signature + IdP cert, is this correct?
if more information is needed i can provide the arguments being passed to OneLogin_Saml2_Utils.validate_binary_sign along with the IdP certificate, once i first verify that is safe to publish... if not, i can send via private channel.

@xtfxme, any news?

No, time constraints let me to implement a server-side expiry instead for
now. However I should be able to get you some additional details by the end
of the week.
On Jan 5, 2015 12:25 PM, "Sixto Martin"<EMAIL_ADDRESS>wrote:

@xtfxme https://github.com/xtfxme, any news?
—
Reply to this email directly or view it on GitHub
https://github.com/onelogin/python-saml/issues/33#issuecomment-68751642.

@xtfxme any progress?

No :-( been too busy elsewhere and the project using this does well enough without.
if you'd like to leave it open I will return when possiblex men days of future past otherwise feel free to close and open a new one when the time comes

Ok

I think you can provide the data since it is public (and no user data is provided in a SAML Logout).
Related to the SLO process, I'm using the same method to validate Signature on SAML Logout Request and SAML Logout Responses, the OneLogin_Saml2_Utils.validate_binary_sign method that you already discoevered. Maybe the problem is the algorithm used, the default is the RsaSha1.
Can you enable the debug and check what return the line: https://github.com/onelogin/python-saml/blob/master/src/onelogin/saml2/utils.py#L952

Hi, I got similar issues but with different type :
"func=xmlSecOpenSSLEvpSignatureVerify:file=evp_signatures.c:line=368:obj=rsa-sha256:subj=unknown:error=18:data do not match:details=EVP_VerifyFinal: signature does not verify
(18, 'Signature is invalid.')"
I used "openssl req -new -x509 -days 3652 -nodes -out sp.crt -keyout sp.key" this command to create new sp x.509 cert and private key and provided in sp configuration.
Not sure what is happening. or what is the cause. My doubt is with this log "data do not match" . Do you have any idea on this.


build runtime error

build from wenet/runtime/libtorch/docker/Dockerfile
log:

cc @pengzhendong

your libtorch version dose not match system arch (x86/aarch64/... etc)

root@docker-desktop:~/wenet/runtime/libtorch/build# arch
aarch64
which libtorch should i use?

your libtorch version dose not match system arch (x86/aarch64/... etc)

system archis aarch64, which libtorch should i use?


Syntax error

Hi, i'm getting this error when executing the script and later adding the app name + domains.
/usr/local/bin/sple.sh: Syntax error: "(" unexpected root@asdfsth:~/serverpilot-letsencrypt#

my bad. I was executing the script with the sh commando of mac! Sorry


Edit button in "Magic notes" app

Describe the solution you'd like
There should be an edit button alongside the delete button

@Vishal-raj-1 can you assign me this issue?

@frappelatte28 Sure !! You can start.

Hey @frappelatte28 Are you working on this? Any updates?

Hey @urvashi-code1255! Actually, I am working on the other feature request that I have mentioned earlier. And I am facing some issues in that I tried my hands and did my research but that issue is not resolving. I am attaching the screenshot of the error that is showing up, let me know if you can help me with it :)

click on source where you will see the error at line number 27 of init.js . check this

yeah I did check the source you mentioned but I dont understand what is happening and why it is happening

@frappelatte28 did you make edit features in this note app ?

No, I didn't, if somebody else wants to work on this issue please assign them.

Can I work on this?

@khushi935 sure !!

https://user-images.githubusercontent.com/62712155/109026624-f2524b80-76e5-11eb-9884-cdd39f7c1cdd.mp4
Have a look at this.


Several devices issue

I had to add usb_vid, usb_pid to the Embed.toml probably due to the fact, that several devices were found on my Windows PC (a person next to me using Linux had the same problem):
[default.probe]
protocol = "Swd"
usb_vid = "0d28"
usb_pid = "0204"

Here is the error:
PS C:\Users\.....\microbit\src\03-setup> cargo embed --target thumbv7em-none-eabihf
   Compiling semver-parser v0.7.0
    ...
   Compiling rtt-check v0.1.0 (C:\Users\.....\microbit\src\03-setup)
      Config default
      Target C:\Users\.....\microbit\target\thumbv7em-none-eabihf\debug\rtt-check
       Error The following devices were found:
              [0]: BBC micro:bit CMSIS-DAP -- 0d28:0204:9906360200052820E521F0BA07F91E12000000006E052820 (CMSIS-DAP)
             [1]: CMSIS-DAP v1 -- 0d28:0204:9906360200052820e521f0ba07f91e12000000006e052820 (CMSIS-DAP)
              Use '--probe VID:PID'
              You can also set the [default.probe] config attribute (in your Embed.toml) to select which probe to use. For usage examples see https://github.com/probe-rs/probe-rs/blob/master/probe-rs-tools/src/bin/probe-rs/cmd/cargo_embed/config/default.toml .

Interesting.  Thanks much for the report! What is the other device that is showing up here?
Anyhow, we can add a note to the book. If you want to do a PR, it would be welcome. Otherwise I'll get it in a bit.


Add defensive checks to s2n_array.

Description of changes:
As part of ongoing verification work of s2n, found a potential overflow condition in the underlying s2n_array code.  This does not affect users of s2n as the affected code cannot be triggered by untrusted input.  Adding defensive checks to protect against future issues.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

:exclamation: No coverage uploaded for pull request head (defensive_array@7545823). Click here to learn what that means.
The diff coverage is n/a.


update readme - steps to create and execute a mongo container

This may reduce friction for developers who are novices with database/containers etc. Btw, great project. I really like this and I am using it as a model to look at for my own project called ludwig.

Thanks for contributing to this project! i ask because i want to understand for my own knowledge. why should a developer use docker in that situation? you can review the code for setting up an admin account. perhaps working with docker in a non-dockerized app could be challenging for a developer.

I appreciate your quick reply back. Firstly, I wanna applaud you for your chatter app. It is clean, well designed and well documented.
On docker:

Docker prevents an engineer from needing to explicitly install a mongo engine on their machine.
 But why is an engineer gonna wanna waste the time?
On your point of developer challenges - not a practical argument  - if a developer need to go through your code, have database knowledge, etc, then you are already attributing intelligence to the developer and docker is a trivial learning curve.
Irrespective, if you take my MR or leave it, I want to applaud you for your very clean code, design patterns and very clean design. Chatter is somethiing I am going to reference when building chat into my boilerplate.
My application Ludwig is a boilerplate which I am creating and I am just beginning to mess around with chat. I am choosing a path which is not socket.io. Just working with the native Websocket so I am going to use your app as a reference cuz I am new to websockets and am doing some things which are causing some unexpected behavior.
My honest opinion is you should ditch socket.io client and go native.
I'd love for you to fork my app, mess around with it and give me some pointers on the chat websocket side of things.

@mydungeon  Oh, okay, nice! thank you for your explanation and suggestions. im currently working on a PWA app and im interested in incorporating websockets into it. however, i lack knowledge about websockets and how they working. this project is quite old, and at that time, I simply wanted to build a MERN stack app :).
But after hearing your explanation, I realize the importance of improving my understanding of websockets. thank you for highlighting that. In the future, I plan to develop another chat app using Next.js 14, and I promise it will be better.
once again, thank you for your valuable suggestions!

Good to know you intend to incorporate websocket on your PWA app. I'd be interested in what knowledge we can share as we are both learning through this tech. Keep in touch and again, I encourage you to clone or fork my Ludwig repo but understand if you are too busy.


Add Trenitalia and Trenitalia France

Despite the 2 being the same company in different countries, the process is not the same so I separated them.
Also the italian translation is made with LibreTranslate because as the company is mainly italian I thought it would be very important

Thanks for your contribution!


⚠ Bofrost.fr has degraded performance

In a9b2f8a, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 921 ms

Resolved: Bofrost.fr performance has improved in 3f273df.


Uppsala, Sweden in country-region-data but not in selector

Uppsala, Sweden is included in the country-region-data data.json, along with 20 other Swedish regions, but for some reason it doesn't appear in the country-regions-selector under "Sweden" (the other 20 do).
Its syntax in data.json appears to be OK (pasted below). Why might this be?
            {
                "name": "Uppsala",
                "shortCode": "C"
            },

Thanks for providing this invaluable resource and cheers,
Will

@willkuhn I suspect this is because the live demo uses an older version of this package, as Uppsala does indeed appear in the data.

@lauralouiset I added Uppsala county along with a few other edits to data.json on Sep 22, 2019. I don't know why it was missing earlier. I wasn't aware of the live demo (nor am I sure now whether I'm linking this comment to that edit in the most appropriate way; the Guidelines seem to be silent on that).

oh sorry -- wasn't aware you had fixed this. I suggest you close this issue if it has been resolved.


Insert new line between each method of "convert function to class"

Suggestion
I use this handy feature quite a lot:

very useful, but i also wish that it could insert a new line between each and every method too.
🔍 Search Terms

List of keywords you searched for before creating this issue. Write them down here so that others can find this suggestion more easily and help provide feedback.
✅ Viability Checklist

My suggestion meets these guidelines:

[ ] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[ ] This wouldn't change the runtime behavior of existing JavaScript code
[ ] This could be implemented without emitting different JS based on the types of the expressions
[ ] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)
[ ] This feature would agree with the rest of TypeScript's Design Goals.

⭐ Suggestion

📃 Motivating Example

💻 Use Cases

It's very easy to add these lines with multi-cursor editing, but quite annoying to remove them if they were automatic. Without a more clear motivation / volume of feedback to change this, I don't really see a reason to.


Expose cAdvisor web interface in caddy

Right now by default the cAdvisor web interface is not exposed. It could be exposed with caddy like the pushgateway or Alertmanager.

Closed by #273.


Lookup album image from Discogs if possible if attaching from file fails

When syncing the media library, album art is looked up from the file, and if not found, is left blank even when the art can be found on discogs.
This change automatically queues a job to attempt to attach the album art from discogs if not found when attaching from the file - and discogs lookup is available.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

4 of 5 (80.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 96.721%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

app/jobs/attach_album_image_from_file_job.rb
4
5
80.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
767

Relevant Lines:
793

💛  - Coveralls

My goal here is to have all images automatically loaded on sync. Currently in my library I have a lot of missing images and they don't get loaded until I click into the album, which is strange behavior to me.
However, I'm worried that this may cause a rate limit issue on the discogs api, need to investigate.

According to the doc of discogs API https://www.discogs.com/developers#page:home,header:home-rate-limiting.  There are some custom http headers we can use to  track the rate limit of the API.

Hi, since this PR hasn't been updated for a long time. I made a continuation of this PR: https://github.com/blackcandy-org/black_candy/pull/343. So I will close this PR. Thanks for your contribution.


Improved UV diagnostic surface shader.

Map U to R and V to G. B is set to 0.
Fix discontinuity across triangles.
Works correctly for uv coordinates greater than 1.
old:

new:

Interesting, would this would have fixed the uv problem with alembic import?

Interesting, would this would have fixed the uv problem with alembic import?

No. This only affects the diagnostic mode visualization. There are no changes to UVs.
I didn't know uvs in Alembic were wrong. The Alembic support has to be rewritten
in any case. At this point, I'm wondering if moving directly to USD would be better.

Thanks,
IIRC, USD and abc can be used complementary.

Thanks!


2nd Try: UI add, lots of other stuff for 3.0.

This addresses issues #30, #33 (in part), #40, #46, #48, #52, #53, #54, #55, #56, and maybe more.  With this change there may be one more sprint if there is time before April.  Otherwise, bug fixes only for 3.0.

Alpha merge.


Bugfix/race condition

Submit a pull request
CLA

[x] I have signed the Stream CLA (required).
[x] The code changes follow best practices
[x] Code changes are tested (add some information if not applicable)

Description of the pull request

please share more information about the problem. ie: a race output or just point out where we need to sync access

@tbarbugli there are methods which updates internal channel state from response, if they will be called concurrently - client could get race condition

@Bogdan-D yes that was obvious; channel state response is not a type one should share outside the controller, that type is just to return a response. Is there any place where we use it on different go-routines? If not then this is not something we should merge (if yes the solution might not even be to add synchronization)

@tbarbugli we don't have any goroutines here, but clients could use it in any way they want, so they will be surprised about race conditions may appear (an example i've found from imgur, they are using go sdk in http handler and in such case they can't control concurrent requests)

You're supposed to document concurrency features explicitly like “Multiple goroutines may invoke methods on a Conn simultaneously.” in the net package.

this will not be merged for now


Update Editor Property Capitalizations to include ReLU and SiLU

Does anyone know what we use these for?

Seems to be some ML stuff not present in Godot: https://sh-tsang.medium.com/review-silu-sigmoid-weighted-linear-unit-be4bc943624d
I assume OP wants this for a plugin/extension.
This isn't the right approach for this, we don't want to hardcoded strings in the core engine because some third-party plugins need them.
If there's a need for it, we should add an API that lets plugins register new conversions. CC @timothyqiu


Fix HTTP body encoding

while updating values in Japanese, HTTP body needed to be utf8 encoding

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: emmieclover sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


GM Vision doesn't work at Scene Darkness 1

With Scene Darkness set to 1, GM Vision doesn't work.  By moving Scene Darkness to 0.95, GM Vision will work.
The scene darkness is the only change between the below screenshots.

Don't set Darkness Color to #000000. It's not a valid option at the moment. I'll need to do something about this.

Fixed in v1.8.5.


Azure Resource Manager Endpoint Metadata missing loginEndgraphEndpointpoint

When retrieving the endpoint metadata for an Integrated System running 1905, there is no loginEndgraphEndpointpoint property in the JSON.  The only properties that are returned are

galleryEndpoint
graphEndpoint
portalEndpoint
authentication

loginEndpoint
audiences

When I use the graphEndpoint value for the activeDirectoryResourceId I can save the file and login, but the Extension shows the following message under App Service:
Error: The access token has been obtained for the wrong audience or resource 'https://graph.windows/net/'.  It should exactly match with one of the allowed audiences '{Audience URI}'.
When I use the audiences value for the activeDirectoryResourceId I get the following message when trying to log in:
Visual Studio Code
An error occurred while signing in:
AADSTS650057: Invalid resource. The client has requested access to a resource which is not listed in the requested permissions in the client's application registration. Client app ID: aebc6443-996d-45c2-90f0-388ff96faa56(Visual Studio Code). Resource value from request: {Audience URI}. Resource app ID: 052c2b1c-afba-4985-a4f2-842432c36269. List of valid resources from app registration: . Trace ID: d04cd65e-41f9-484d-a48e-cdee819fe900 Correlation ID: 88ae3661-3b06-49e1-b917-6c94cec779b3 Timestamp: 2019-06-21 19:01:28Z
It seems like the audience URI is the correct value to use, but the must be an additional step missing from the instructions.
Environment Information:
4-Node Integrated System
Microsoft Build 1.1905.0.40
Identity Provider Azure Active Directory
AAD Tenant is different from Registration Tenant

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

ID: 5d8803f5-7736-6a29-98df-69c121d43c95
Version Independent ID: d88f693f-cade-48ee-df03-d2363298ef0d
Content: Connect to Azure Stack using Azure Account Extension in Visual Studio Code
Content Source: azure-stack/user/azure-stack-dev-start-vscode-azure.md
Service: azure-stack
GitHub Login: @mattbriggs
Microsoft Alias: mabrigg

@SqlTechMike Thanks for the feedback. I am reaching out about this issue, and will get back to you as soon as I have more information.

@SqlTechMike Thanks for your feedback! We will have an update for the topic posted shortly. #in-progress

@SqlTechMike you will need to register your VS code application ID by updating the Azure Stack AAD Home Directory.
Try running those 3 PowerShell lines and then see if this still does not give the proper output.

@SqlTechMike We've updated the topic. Hopefully it will resolve your issue. It should be available later this afternoon. Travis has summarized the gist of the update to the article. Thanks again for leaving feedback.
#please-close

Thank you, running the PowerShell created the missing application registrations and the connection from Visual Studio Code now works properly.


Add automated preview link table

Fixes #126

Consider adding preview links to articles that reference any snippets/images/include files that are updated in the PR too (even if the article itself isn't changed). I believe that's what the OPS system does.


Gatsby: footer improvement and some other minor fixes

Status: waiting for Matěj Stehlík to approve the footer design

Good job! :) Any related trello tasks?

No, I just enhanced it a little bit as our designers had some objections to it before the first release and we didn't have time to fix it back then.
I also fixed HTML links, which now work as Gatsby links with this SPA experience.
Then I fixed breadcrumbs at pages with area lists and I removed target="_blank" from our Link component – well, we don't really use it anyway, because we don't have any external links in the code.


Nested key claim name

I want to know how one can use nested jwt payload in kong?
imagine I have the following payload:
{
    'level1': {
        'level2': 'something',
    },
}

now if I would like to say that my key_claim_name is going to be level1.level2 it says that's not possible through the following error:
{
    "message": "No mandatory 'level1.level2' in claims"
}

+1 I have the exact same issue.
@meysam81 Did you ever find a solution to this?

+1 I have the exact same issue.
@meysam81 Did you ever find a solution to this?

I ended up modifying the underlying source code in the docker container, committing it, and reusing it in my future deployments.
here's my modification:
In the docker container, open the following file:
/usr/local/share/lua/5.1/kong/plugins/jwt/handler.lua
replace the following code:
local jwt_secret_key = claims[conf.key_claim_name] or header[conf.key_claim_name]

with this one:
local jwt_secret_key = claims[conf.key_claim_name] or header[conf.key_claim_name] or (claims["CUSTOM_KEY"] and claims["CUSTOM_KEY"][conf.key_claim_name])


signal-desktop: 5.0.0 -> 5.1.0

This is the most important Signal-Desktop release in a while as we
finally have Wayland support via Electron 12 \o/ #YAY :D
https://github.com/signalapp/Signal-Desktop/issues/3411
https://github.com/signalapp/Signal-Desktop/commit/5bcf50ede96fed18dccc207e89bbae7ceb2d2dff#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519R231
https://github.com/signalapp/Signal-Desktop/commit/c6d5d852a9f9668268803bb5a20d94b4a505dd87#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519R232
https://en.wikipedia.org/wiki/Electron_(software_framework)#Versions

Motivation for this change
Warning: This should use '~/.config/Signal Beta/ but testing this might still cause data-loss!
signal-desktop-beta --enable-features=UseOzonePlatform --ozone-platform=wayland works without XWayland but is also affected by #103049 (which makes sense but is really unfortunate for the Wayland support...). I.e. the 1st launch should be fine but when closing the window and re-launching it no window appears unless --disable-gpu is used, the compositor is restarted, or  '~/.config/Signal Beta/ is removed.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Will the 21.05 release include this change? If so, then I think this is a risky change for our users. Why not generalize the signal-desktop derivation and produce two, one which is a stable release and the other which is the beta release and allow users to choose if they want the beta release.

@ixmatus oh, no worries, this is only a draft for testing until the next stable release. I should've stated that I won't merge this until then.

Why not generalize the signal-desktop derivation and produce two, one which is a stable release and the other which is the beta release and allow users to choose if they want the beta release.

Yeah, I was surprised how many changes are required to switch to the "beta channel" and might add an abstraction for that so that users could more easily try out beta releases if they want to (but I don't plan on maintaining a signal-desktop-beta in Nixpkgs).

@primeos this dropped officially today! https://github.com/signalapp/Signal-Desktop/releases/tag/v5.1.0
Do you want to continue with this PR? Otherwise I can add my own.

Do you want to continue with this PR?

Yeah, seams reasonable but thanks for the offer :)


⚠ Bofrost.at has degraded performance

In 5419225, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1581 ms

Resolved: Bofrost.at performance has improved in 5a376f7.


Google Maps API warnings in console

The following warnings appear in the chrome console when first interacting with a Google terrain map in Nunaliit. Is Google about to break stuff again?
Google Maps API warning: NoApiKeys https://developers.google.com/maps/documentation/javascript/error-messages#no-api-keys util.js:220 Google Maps API warning: RetiredVersion https://developers.google.com/maps/documentation/javascript/error-messages#retired-version util.js:220 Google Maps API warning: SensorNotRequired https://developers.google.com/maps/documentation/javascript/error-messages#sensor-not-required

Need to replace the script element in index.html with:
<script src="http://maps.google.com/maps/api/js?v=3.23"></script>

Are you going to move this into demo as the default or is there some code to change to get over this version hump and keep current with Google for now? I worry that setting this will break all the atlases at once in the future when Google deprecates it.

I too am seeing these message in the Javascript console and am feeling quite uneasy given past experiences with Google infrastructure.  I will perform the fix indicated by JP, however please keep us up to date with required actions.
On Jun 16, 2016, at 11:42 AM, Amos Hayes<EMAIL_ADDRESS>wrote:

Are you going to move this into demo as the default or is there some code to change to get over this version hump and keep current with Google for now? I worry that setting this will break all the atlases at once in the future when Google deprecates it.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

This issue may need further consideration. See link below:
http://googlegeodevelopers.blogspot.ca/2016/06/building-for-scale-updates-to-google.html

Looks like it broke today.

I believe this has been resolved with the inclusion of an Google API Key property which can be set using the nunaliit atlas configuration command nunaliit config. See wiki for further details.
Alternatively, if the atlas is not using a Google API Key, alternative WMS backgrounds can be used including Open Street Maps (OSM). See wiki documentation on module documents for further details.


.btn-default but no .btn-md ?

A philosophical question that occurred to me:
So, .btn itself doesn't have a default color, which is why .btn-default exists.
Given that .btn doesn't have a default color, why does it have a default size? Perhaps it should have no default size and there should be e.g. a .btn-md class that explicitly sets a "normal" size? Or if this difference is legitimate, what's the rationale behind it?

I think .btn is more related to the layout and  ".btn-default", ".btn-primary", ... are related to style. It's a great sepparation between layout and style, and ".btn-md" is somehow still layout-related. I think we should not bloat the DOM with extra classes because most of the time, if we want to edit the default button size, we could simply edit it with SASS.
In conclusion, it is very likely that we use the default button size, far more likely than using the "btn-default" so I consider the current separation of layout / style is good.

I remind you that we already have .btn-lg and .btn-sm classes.

Indeed, but it's most likely to use the default button size so having to specify it explicitly only adds more classes than necessary to the DOM. It makes sense to have layout-related properties by default while leaving style-related properties optional.
Even though it could make sense, I don't think we have to compromise the simplicity of Bootstrap where it's unnecessary. Again, it's far more likely to want to change the style of the button than its size, and forcing an extra class on a website that is only using the default size could mean just bloating the markup.
Please let me know if I am wrong :)

And yes, the -lg and -sm classes are great, but not as likely to be used compared to -primary and -default (or the others).

@cvrebert: Note that .btn-default was renamed to .btn-secondary: f399d21ef5e20bd736d5cf2946acb982378cd2f4

Default sizing is required to keep consistent look throughout framework. .btn-md would just add clutter, as would .form-control-md, .input-group-md, .modal-md, .pagination-md... All of which would be required in case .btn-md is added, to fully follow this naming convention.
The better question is here why .btn doesn't have a default color. .btn-secondary styles could be applied to .btn by default and .btn-secondary perhaps replaced by something useful like .btn-inverse from v2.

The better question is here why .btn doesn't have a default color.

That was a deliberate change made in v3.

Honnestly, adding .btn-md and requiring it for every normal buttons sounds to me like adding classes for the sake of adding classes. Don't do that.

My assumption there is that it's perfectly acceptable to have a default button size, but not necessarily a default button style. Specifying what style—or semantic/contextual variation—you need makes more sense than specifying an initial size as well.
I understand the argument, but I don't think we need to go that far with the class and minimal override approach we've taken elsewhere :).
(Holler here if that doesn't make sense though!)


Website broken in Chrome when Redux DevTools extension enabled

The https://reakit.io website is a blank white page in Chrome when the Redux DevTools extension is enabled, due to an error mounting the React app.
The errors are:
bundle.80726854.js:80
TypeError: Cannot read property 'match' of undefined
    at isFiltered (<anonymous>:1:4965)
    at Object.x [as send] (<anonymous>:1:73931)
    at Provider.callbackFn (bundle.80726854.js:12)
    at mf (bundle.80726854.js:80)
    at lf (bundle.80726854.js:80)
    at rh (bundle.80726854.js:80)
    at Xg (bundle.80726854.js:80)
    at Yg (bundle.80726854.js:80)
    at If (bundle.80726854.js:80)
    at wh (bundle.80726854.js:80)

bundle.80726854.js:80
Uncaught TypeError: Cannot read property 'match' of undefined
    at isFiltered (<anonymous>:1:4965)
    at Object.x [as send] (<anonymous>:1:73931)
    at Provider.callbackFn (bundle.80726854.js:12)
    at mf (bundle.80726854.js:80)
    at lf (bundle.80726854.js:80)
    at rh (bundle.80726854.js:80)
    at Xg (bundle.80726854.js:80)
    at Yg (bundle.80726854.js:80)
    at If (bundle.80726854.js:80)
    at wh (bundle.80726854.js:80)

bundle.80726854.js:80
Uncaught Error: Minified React error #200; visit https://reactjs.org/docs/error-decoder.html?invariant=200 for the full message or use the non-minified dev environment for full errors and additional helpful warnings.
    at ca (bundle.80726854.js:80)
    at t (bundle.80726854.js:80)
    at Gh (bundle.80726854.js:80)
    at Object.render (bundle.80726854.js:80)
    at bundle.80726854.js:123

Disabling the Redux DevTools extension causes the site to appear again.
I'm using Chrome 70.0.3538.110 and the Redux DevTools extension version 2.16.2, which should be the latest version.

Thank you for reporting it @jacobwgillespie. I'm investigating it.

It seems to be a bug in redux devtools extension (related https://github.com/zalmoxisus/redux-devtools-extension/issues/591)
I just disabled devtools for now.

Thank you!  It appears that Redux DevTools v2.16.3 was also released which fixes this issue, in case you want to add back devtools in the future.


Tek Quetzal should be on the shopping list

I'm flying around in The Island and I ran into a Tek Quetzal. When I went to add it to my Larkator shopping list so I could track both normal and Tek Quetzals, I realized Tek Quetzals aren't an option to select. Tek Raptor, Tek Parasaurs, and Tek Stegosauruses are.
So if you could add Tek Quetzals to The Island list, that would be super.
Also, it seems there's a Tek Rex as well: https://ark.gamepedia.com/Tek_Creatures

Suddenly Tek Quetzal and Tek Rex are on the list. I'm so confused.


Refactor bash files

Substitute core bash files by python scripts.
Renaming and removing files, and create corresponding entry points.

PR is ready.
I'm working on testing the 'core' python scripts, but might open a new PR.

There was an issue with the subprocess call, that should now be fixed.
Also the path to the r0 list files was not complete (Why did you change that?)

Also the path to the r0 list files was not complete (Why did you change that?)

Probably mistake from my side 😅 . I will be doing the tests in a moment !


MdInput underline hover is missing

Bug, feature request, or proposal:
Something between bug and feature request
What is the expected behavior?
I just saw, while scrolling through the material guidelines, that the underline of the MdInput should have a hover effect.
https://material.io/guidelines/components/text-fields.html#text-fields-states
Then go to hover.
What is the current behavior?
There is no hover effect for the underline
What are the steps to reproduce?
See the docs.
What is the use-case or motivation for changing an existing behavior?
Adapt the material guidelines correctly.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Angular Material Beta 8

I believe this falls under https://github.com/angular/material2/issues/3984

True, Thanks @willshowell


What does the name "Nydus" means?

Something like a nydus worm? https://liquipedia.net/starcraft2/Nydus_Worm_(Legacy_of_the_Void)
Feel free to let your imagination fly and consider it to be anything extremely fast ;)


Azure - Wix Connection

veloex/velo-external-db:latest Stopped working. Please help because all my service is down. Thanks!

there was a change in app service, I updated the image, please redeploy.


Create @celo/encrypted-backup backup for ODIS hardened password encrypted backups

Description
This PR created the @celo/encrypted-backup package with an implementation of the password/PIN encrypted backup scheme intended for account recovery within crypto wallets such as Valora.
Core to the implementation are two functions createBackup and openBackup. Creating a backup takes the secret data and the user password or PIN along with a config for how to backup should be secured. Opening a backup takes the backup structure, which includes all the relevant configuration information, and the user password or PIN.
In order to secure the backup this PR implements:

Key derivation from the user's password or PIN
Key hardening by mixing in the output from ODIS with the unhardened key and a SequentialDelayDomain
Mixing in a value secured by an ODIS circuit breaker to address the possibility of ODIS compromise.
Computational hardening with scrypt or PBKDF2 as provided by Node crypto

Key derivation steps are completed on setup and on opening the backup to ensure proper decryption.
Other changes

Implemented a circuit breaker client to utilize the Valora implementation of the circuit breaker

Tested

Existing tests cover changes including refactoring of packages.
Circuit breaker client is covered by unit tests against a mock of the service.
Backup is covered by tests against mocks for the circuit breaker and ODIS.

Related issues

Fixes https://github.com/celo-org/celo-monorepo/issues/8816

Backwards compatibility
No backwards compatibility concerns
Documentation

Relevant documentation is included in https://github.com/celo-org/docs/pull/150
Most new methods have reasonably descriptive ts-doc comments to generate automatic API docs and to help readers of the code.

LGTM but maybe we could add simple function to verify domain by its schema? I am wondering how to use this general "KnownDomainSchema" for our purpose. I think we will have to check type of domain and check its specific schema.

I am wondering how to use this general "KnownDomainSchema" for our purpose. I think we will have to check type of domain and check its specific schema.

Given an object which is not known for sure to be a Domain, using KnownDomainSchema.decode(obj) will check that it is some known domain. Since domainHash and the functions to check signatures accept the KnownDomain arg, that's exactly what you want. For logic that is used for a particular domain, such as choosing which rate limiting function to apply, the best way to to use the isSequentialDelayDomain and related functions to be implemented as we add domains.
An alternative to this is to take the unknown object and try decoding it as each specific domain type (e.g. try SequentialDelayDomainSchema.decode(obj) then try LinearBackoffDomainSchema.decode(obj) if that fails and so on. I'm not sure why we'd do this over using KnownDomainSchema.decode(obj) then just checking for each type after handling any decode errors in one place.

Added do not merge so that this doesn't get merged before https://github.com/celo-org/celo-monorepo/pull/9154


Specification: left-nav contains subsection entries

Visit https://opentelemetry.io/docs/reference/specification/ on desktop and you'll see the following: the left-nav (red box in screenshot below) shows flattened collection of pages from arbitrary sections, as you can see from the screenshot below.
What we need to do to fix this is to ensure that every section has an index (README) page.

Tasks:

[x] https://github.com/open-telemetry/opentelemetry-specification/pull/2479

Related:

#1279
Contributes to #906

This has mostly be addressed, I'm going to close this in favor of tracking the remaining fixes via:

https://github.com/open-telemetry/opentelemetry-specification/issues/2558


NanoMapper == O.R.M?

It has been suggested that NanoMapper could be used as a simplistic O.R.M. by mapping against various System.Data classes (such as DataRow for example).
Need to figure out how this would work as we'd need to map to the indexes not the properties of the DataRow, etc.

This should be easily possible after #6 is complete


Update flyte.org content

(WIP)
We want to create a new feature-gallery (slides as content areas) to showcase the ease of using flyte. Initially it will have 5 slides (copy will be provided) and each slide will contain a title, a subtext and a video. Users should be able to scroll/click through the slides.

Slide content: title, subtext, movie
Should support clicking/scrolling through slides
Videos should autostart on view and loop from there
Should support {n} slides/updates

This work was related to flyte.org site update. It's work currently done.
Anyway it has a separate repository connected to it, so issue is closed here.


snappymail: 2.17.2 -> 2.17.3

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/the-djmaze/snappymail/releases.
meta.description for snappymail is: "Simple, modern & fast web-based email client"
meta.homepage for snappymail is: "https://snappymail.eu"
meta.changelog for snappymail is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.17.3 with grep in /nix/store/s50avcrfdjmn0h8mwnhpd1rwn87akch5-snappymail-2.17.3
found 2.17.3 in filename of file in /nix/store/s50avcrfdjmn0h8mwnhpd1rwn87akch5-snappymail-2.17.3

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
snappymail

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/s50avcrfdjmn0h8mwnhpd1rwn87akch5-snappymail-2.17.3 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A snappymail https://github.com/r-ryantm/nixpkgs/archive/5a125f8b2f401d030655d419125f806310e7c08a.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/s50avcrfdjmn0h8mwnhpd1rwn87akch5-snappymail-2.17.3
ls -la /nix/store/s50avcrfdjmn0h8mwnhpd1rwn87akch5-snappymail-2.17.3/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

snappymail

Maintainer pings
cc @Mic92 for testing.

Result of nixpkgs-review pr 188898 run on x86_64-linux 1

1 package built:

snappymail


lstn scan not failing without any output

Describe the bug
Running lstn scan with this package.json fails without any output locally.
However, the same project works fine with the listendev/action@scan action (logs)
Please include:

lstn version 0.3.1
macOS ventura 13.0

Steps to reproduce the behavior

Clone https://github.com/garnet-org/demos.git
Run lstn scan .
View output

Expected vs actual behavior
Instead of returning verdicts, it fails without any output in the local terminal.

This is likely caused by connection issues to the npm registry, maybe a timeout issue or something like that.
I'll investigate further.
Another thing I remember is that we didn't experience this in easylstn (the prototype before scan). I noticed that this problem exists in to as well when you do things like this.
lstn to --endpoint https://npm-staging.listen.dev react '*'

So I suspect that also doing the dependencies one at time (as opposed to doing them in bulk) will also be beneficial.
Additionally, doing them one at time allows us to have the nice progress output (like we showcased in easylstn) and in case of any failure we only skip the dependencies with an actual problem vs all of them.

This happens because of the --timeout flag which is setup at 60 seconds by default.
NPM registry connections are strictly related to this since most of the time that connection is the slowest part of the process. The previous statement is proven by the fact that this happens locally but not on the action. Most of the times 60 seconds are more than enough on the action but not locally.
To improve this, I propose to have a better "progress tracker" that prints the errors and shows "where we are" while processing the request.

@fntlnz the proposed solution in #187 was tested for the earlier case, and it is working now! so this solution would solve our issue for now.

@jadoonf the one in #187 is not a solution to this problem, even if it gives you the perception of solving it...
The problem here is that the default --timeout (60 seconds) may be too little and the CLI doesn't have a progress indicator that lets users understand that.
I see 2 separate topics here:

giving commands a progress indicator (that has nothing to do with the fact that the requests happen in parallel or not) (see #190)
always communicating we are exiting because of a short timeout

So, as a recap: this issue is easily solved without code changes just by raising the timeout (eg., --timeout 3000).

To prove this is a problem regarding the timeout (and it's also a flaky problem): see my output.
I run lstn scan ciao1/ where ciao1/ directory contains the package.json file of the demos repository.


Avoid showing uninstalled wallet

I don't know if this is a bug or a feature request, but i would like to avoid showing Browser/Frame wallets on the modal if the user does not have some of them installed. Is there any way to check it?
Thanks in advance.

Helo @afiorenza we noted your feedback earlier and worked on a more contextual modal which shows a personalized list of wallets :) Stay tuned to our social handles for further updates!


pkg: no package-lock.json in this library

match the best practice of a js library

Coverage remained the same at 100.0% when pulling f649e4ca3edb85f8d3da7a0c95c15edf5e801400 on unlock into 63862794d20008f0c310a83a18adbc41362ab081 on master.


SW-6475 Amends to styling and layout of Workflow

Codecov Report
Patch and project coverage have no change.

Comparison is base (6cc2dc2) 57.95% compared to head (9adc005) 57.95%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #310   +/-   ##
=======================================
  Coverage   57.95%   57.95%
=======================================
  Files          13       13
  Lines         918      918
=======================================
  Hits          532      532
  Misses        346      346
  Partials       40       40

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


TE9 TC-LC-1.1 chip tool Darwin testing: Support needed for feature map, event list attributes

As per the Label cluster testcase TC-LC-1.1, feature map and event list attributes needs to be present to read, which are missing in chip tool Darwin. Chip tool has feature map attribute but not event list. Sharing the screenshot for the same.

.

Event list is https://github.com/project-chip/connectedhomeip/issues/15011
Feature map was fixed in #18276


Update scrapy_delayed_requests.py

Hello.
Somehow I missed this, I apologise. However with Scrapy 1.0 this functionality can be implemented in a more simple way so I will archive this project. See README for more details.


how  many gpus is need?

Thanks for your great jobs, I would like to know how many gpus is need to train this model?

Hi, @xmu-xiaoma666 ,
Thanks for your interest in LSeg!
We've mentioned the details in Section 4.1.
Hope this helps.
Best,
Boyi


Need help

Hi,
I tried to understand the meta model implemented in the postit.json and postitDi.json and I've got some questions

I didn't find any reference on attributs isReadOnly, isVirtual and subsettedProperty in moddle doc, are they implemented or just unused attributs ?
whats the difference between PostitRootBoard and PostitPlane, why using this to elements for encapsulate drawing elements ? can we mix them into a unique element ?

Thanks a lot for your help and congratulation for your work.

Hi,
Thanks for your answer.
I refactored the project to support multiple planes for one board.
So by now, there's only one board that defines postit elements and their attributes like name, title, date...
And one or more plane that define postit DI like bounds, color, ... and RefId to postit element.
One postit element can have multiple DI, depends on the plane selected for drawing shapes.
In that case, for creating shapes, instead of walking through the board (because there is no more reference to a DI), I'm walking through DIs define in the selected plan and add busniessObject for each DI element but as busniessObject has also a DI attribute, it creates a refence loop...
So I'm asking me if it's necessary to keep businessObject ?
Thanks for your help.

Thanks for your details! That's an implementation detail for sure. As I'm interested: is there a specific reason for you to make those changes? Maybe sharing some information about your use case would help me to understand 🙂

Hi,
Here is my use case:
The postit board defines businessObject and those businessObject can be used in different planes. A businesObject can have a different shape for each plane, each shape defined by a specific Di moddle element for this plane.
So, through PostitTreeWalke, for each businessObject find in the board, a shape element is created with Di moddle element definition. I attached businessObject and diObject both directly as references to shape object and there is no more need to have a reference to Di moddle element in the businessObject.

Thanks for your insights. That's highly technical though, can you share a real-world use case on why you need those architecture changes? What do you want to achieve? Is this something related to having different post-it boards on a single page?


Move Azure terraform example to Standard SKU for IP addresses

What this PR does / why we need it:
This fixes broken LoadBalancer functionality in azure CCM (it refuses to add Basic IPs into the LB)
Which issue(s) this PR fixes:

Fixes #
What type of PR is this?
/kind regression
Special notes for your reviewer:
Does this PR introduce a user-facing change? Then add your Release Note here:

Move Azure terraform example to Standard SKU for IP addresses

Documentation:

NONE

/test pull-kubeone-e2e-azure-default-stable-upgrade-containerd-external-from-v1.27.11-to-v1.28.7

/test pull-kubeone-e2e-azure-default-install-containerd-external-v1.27.11

/test pull-kubeone-e2e-azure-default-stable-upgrade-containerd-external-from-v1.27.11-to-v1.28.7

/test pull-kubeone-e2e-azure-default-stable-upgrade-containerd-external-from-v1.27.11-to-v1.28.7
/test pull-kubeone-e2e-azure-default-stable-upgrade-containerd-external-from-v1.28.7-to-v1.29.2
/test pull-kubeone-e2e-azure-default-stable-upgrade-containerd-from-v1.27.11-to-v1.28.7

/test pull-kubeone-e2e-azure-default-stable-upgrade-containerd-external-from-v1.28.7-to-v1.29.2

/cc @xmudrii


Add support for optional username and password auth

I added optional support to connect to the mqtt server with a username and password, as I (and a pretty decent number of other folks) run my mqtt container, history db, etc, on a separate server. If you have any feedback on the code, please do let me know as this is quite literally my first attempt at writing Rust code, so I am not familiar with any best practices or common conventions.
That said, I spun up a separate local mosquitto instance and commented out the username and password field from my config, and connected to ensure that everything still worked as before with the new additions. Evidence of this can be seen below.

Screenshot of connection and data publish with user/pass removed from config

LGTM but could you maybe update the Readme as well?

LGTM but could you maybe update the Readme as well?

Sure, give me just a few and I will get that added to the PR

My apologies, I forgot to comment back when I added that and pushed it to the branch.


fix project tree tests failing on windows

There were a few project tree tests failing only on windows caused by #14182. The tests were failing because the projectTreeItem's uri got changed to be the file system uri, which was then used in the fsPathToProjectUri conversion when constructing file and folder tree items. This only caused the windows tests to fail because the root path in those tests was set to 'z:\' in windows and '/' for mac and linux.
Only the uri of the vscode.TreeItem needs to be the file system uri, so changing the projectTreeItem's uri back to just the basename fixed the tests and kept the git status showing in the tree. Most of the other changes are updating the baseTreeItem's uri to projectUri to make it less confusing since the file and folder tree items also have their fileSystemUri.
I verified that adding new items, folders, nested items and git status still works with these changes.

Pull Request Test Coverage Report for Build 659303712

15 of 24 (62.5%)  changed or added relevant lines in 6 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 43.348%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

extensions/sql-database-projects/src/models/tree/projectTreeItem.ts
1
3
33.33%

extensions/sql-database-projects/src/models/tree/dataSourceTreeItem.ts
1
4
25.0%

extensions/sql-database-projects/src/controllers/projectController.ts
1
5
20.0%

Totals

Change from base Build 659250625:
0.0%

Covered Lines:
25413

Relevant Lines:
53484

💛  - Coveralls


Add test for 'Position' function for PostgreSQL

I'd like to pick this one up

I'm not sure the current definition is actually correct. A position function in PosgtreSQL looks like position(search_string in main_string). The definition currently in expr.scala is  val Position = FunctionDef[(String, String), Int](FunctionName("position")) which generates a query looking like position(search_string, main_string) which is an invalid syntax. This is probably applicable for other functions ('replace' for example). I think the parsing of these functions should be different, but I'm quite new to the library so I'm not sure where to begin. Any ideas @jczuchnowski ?

This is still an open issue.


feature support qualify stmt

Proposed changes
like bigquery
https://cloud.google.com/bigquery/docs/reference/standard-sql/query-syntax#qualify_clause
Examples

SELECT
  item,
  RANK() OVER (PARTITION BY category ORDER BY purchases DESC) as rank
FROM Produce
WHERE Produce.category = 'vegetable'
QUALIFY rank <= 3

/*---------+------*
 | item    | rank |
 +---------+------+
 | kale    | 1    |
 | lettuce | 2    |
 | cabbage | 3    |
 *---------+------*/

You don't have to include a window function in the SELECT list to use QUALIFY.


Receiving 'nil' for apiRequestsCount and responseTime

Everything about the API and ForecastIO is working well, except having trouble with getting the metadata for the apiRequestsCount and responseTime. Not sure if the issue is in my code, in ForecastIO, or in the DarkSky API
My code:
self.client.getForecast(latitude: lat, longitude: lon, excludeFields: [.alerts, .daily, .flags, .minutely]) { result in
            switch result {
            case .success(let currentForecast, let requestMetadata):
                //  We got the current forecast!
                if let currentFeelsTemperature: Double = currentForecast.currently?.temperature {
                    print(currentFeelsTemperature)
                    DispatchQueue.main.async {
                        self.feelsLikeTmpLbl.text = String(Int(round(currentFeelsTemperature)))
                    }
                }
                if let summaryInfo: String = currentForecast.currently?.summary {
                    print(summaryInfo)
                    DispatchQueue.main.async {
                        self.summaryLbl.text = summaryInfo
                    }
                }
                print(requestMetadata)
            case .failure(let error):
                //  Uh-oh. We have an error!
                print(error)
            }
        }

Returns:
RequestMetadata(cacheControl: Optional("max-age=3600"), apiRequestsCount: nil, responseTime: nil)

That's strange—the unit tests for RequestMetadata look good to me. I'll take a closer look at this soon. I'll be busy early this week, but I'll try to take a look this weekend and update you on the issue.

Hey, sorry for the delay! I found some time for this issue, and the fix is now available in ForecastIO 5.1.0. It looks like the headers require lowercase strings, which is why you weren't getting the metadata. Sorry about that and the delay! Let me know if you still have any problems with this.

Thanks for the update! Everything seems to be working great now.


Fix rare crash of appleseed on exit

It's hard to see what changed in intersector.cpp. It seems you changed the order of some declarations

Line endings were busted, I fixed them. Nothing else has changed.

Thanks for the review @oktomus!


rd param allows other domain

rd param at /oauth2/start currently allows only the path starts with /.
Otherwise the request is redirected to / even if rd param is required to allow other domain URI.
This commit will work on following nginx snippet:
location /secret {
    satisfy any;
    auth_request /oauth2/auth;
    error_page 401 = /oauth2/start?rd=$scheme://$host$uri;
}

which is useful for multiple path required to authenticated.

seems to be conflicted with #464 and #544


HOST = <IP_ADDRESS> not running on windows

Is there a way around this ??

What is the context? Can you describe what you are trying to do and where it is failing?

Duplicate of #12, See pull request #9


Add long long to user parameters

This adds long long as a type that can be set through the yt_set_UserParameter library of functions.

@matthewturk Could you please also help update the doc (doc/SetCodeSpecificParameter.md)?
(Only need to paste that new API)
Thanks!


Add CloudInformationModel for tutorial environments

See issue #2278 for more info

Updates path in information architect notebook for CloudInformationModel (archives)
Builds CloudInformationModel.json during build
Adds CloudInformationModel.json to assembly, so that it's findable in container/archive environment

Testing ...

PR appears to have the intended effect, and the glossary is loaded. However see #2778 - output from notebook is not as expected.

Due to the issue around the licensing & changes needed to the parser to load the update file, I am closing this PR. Once those issues are addressed this change remains a viable approach.


Cannot read song name and artist in full screen

As you see, black text on black shifting background makes it very hard to read the text.

Thabks gor pointing this out, i missed this while checking for inconsistencies


Test PR job

@kvendingoldo Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

Please use your own fork to test github’s PR features.


request->put() return no values when using ContentTypes Middleware

I'm working in a restful api and I think I found a bug.
To reproduce the bug you should use the ContentTypes Middleware and send a put request (not a post & _method=put).
The request should have "Content-Type: application/json" and a valid json string in the body.
Then when you try to load the values using $app->request->put(); you'll get an empty array.
I think this bug it's in the Request::post() function, in this line:
$this->env['slim.request.form_hash'] = Util::stripSlashesIfMagicQuotes($_POST);

I guess this could be a solution... but not sure about the consequences:
$this->env['slim.request.form_hash'] = Util::stripSlashesIfMagicQuotes($this->env['slim.input']);

As a workaround I suggest to put a  $_POST = $env['slim.input'] in the call() function of the middleware. See below.
public function call()
{
    $mediaType = $this->app->request()->getMediaType();
    if ($mediaType) {
        $env = $this->app->environment();
        $env['slim.input_original'] = $env['slim.input'];
        $env['slim.input'] = $this->parse($env['slim.input'], $mediaType);
        $_POST = $env['slim.input'];
    }
    $this->next->call();
}

That method only works with the content-type is application/x-www-form-urlencoded. Otherwise, you have to use the $request->getBody() and decode appropriately. I'm hoping to get native JSON and XML support built into 3.x.