Inline extend function in node implementation

This is a proposed solution for the Node.js 0.6 breakage I reported in #450 . This replaces the use of util._extend with an inlined version of the minimal implementation.

Coverage increased (+0.05%) to 63.804% when pulling a697a07a5db6423ef558cdf6b2d6690a41766657 on dougwilson:node-06-extend into c211947ea37e1b08ee78653d78defbda7438783d on visionmedia:master.

Nice find @dougwilson! 👍
I can't help but wonder what you're doing with Node v0.6 still, though 😄

🤷♂


Rename Dockerfile.arm to Dockerfile.armhf to more clearly reflect the specific subversion of ARM it targets

This will be even more important when we look to add arm64 (which is where ARM actually gets interesting in the server space, IMO).
cc @jfrazelle

LGTM

LGTM I'll update the Jenkins job after merge


Update the GitLab image to 9.0

DCOS specific:

Switch to the built-in healthcheck command
Add pre-install notes staging that single-node installs need to be host pinned

This update adds the features introduced in 8.17 and 9.0

Easily Search and Add Issues to Issue Boards
Squash and Merge (EE only)
Copy and Paste GitLab Markdown
Monitoring GitLab with Prometheus
Audit Users (EE only)
Configurable Mirror Synchronization Time (EE only)
Subgroups
Deploy Boards (EE only)
Export Issues (EE only)
Gitaly
Group search and filtering
Impersonation Tokens
Blocking manual actions in pipelines

See https://about.gitlab.com/2017/02/22/gitlab-8-17-released/ and https://about.gitlab.com/2017/03/22/gitlab-9-0-released/ for more changes and features.
This update includes the security fixes included in 8.16.5, and 8.17.4

GitHub importer assigning labels and comments to the wrong project
Several XSS vulnerabilities
Information Disclosure in Issue and Merge Request Trackers
SSRF when importing a project from a Repo by URL

See https://about.gitlab.com/2017/02/15/gitlab-8-dot-16-dot-5-security-release/ and https://about.gitlab.com/2017/03/20/gitlab-8-dot-17-dot-4-security-release/ for more information
This update includes several bug fixes for regressions. Details on these bug fixes can be found here:
https://about.gitlab.com/2017/01/27/gitlab-8-dot-16-dot-3-released/ https://about.gitlab.com/2017/02/02/gitlab-8-dot-16-dot-4-released/ https://about.gitlab.com/2017/02/17/gitlab-8-dot-16-dot-6-released/ https://about.gitlab.com/2017/02/28/gitlab-8-dot-17-dot-1-released/ https://about.gitlab.com/2017/03/01/gitlab-8-dot-17-dot-2-released/ https://about.gitlab.com/2017/03/10/gitlab-8-dot-17-dot-3-released/

Tested the CE and EE install using a single-node instance. Tested an upgrade from the previous version to the new version on CE.
Confirmed registry, git over ssh, and git over http were all working.
cc\ @ssk2

Awesome, thanks @twk3! For future reference, if you the copy the package in one commit and make changes in a second, it's a little easier to diff versions.


Add back Yiddish to Language Codes

@mahagr You said a friendly reminder was ok so here is one! 😉
Note that Yiddish was among the language codes until this point – that's why it's "add back".


Fixed chain.updateStatus of gov params path

Description

If enable the gov module in settings.json. The error repeats in this path.
/cosmos/gov/v1beta1/params
{
  "code": 12,
  "message": "Not Implemented",
  "details": [
  ]
}

cosmos sdk explains params_type defines which parameters to query for, can be one of "voting", "tallying" or "deposit".
So i fixed it but if is known issue please discard this PR.
Checklist

[x] Targeted PR against correct branch.
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[x] Run meteor npm run lint
[ ] Added an entry to the CHANGELOG.md file.
[x] Re-reviewed Files changed in the Github PR explorer.

Thanks! This PR fixes #491

thanks for reply @kwunyeung  sould i add CHANGELOG.md?

@MinseokOh yes please add an entry in CHANGLOG.md. Thanks!

Added in 33ecc20

@MinseokOh master has been updated. Please resolve conflicts.


Azure files share disk space free (percentage)

Hi, requesting update this documentation page on how to view/see Azure File share disk space free ( percentage).
I.E. if Azure file share set with 100 Gb quota and if provisioned/used space = 80 Gb , percentage free = 20 % available
I need to place it on a dashboard so that Operations team could see that live data
Currently, I can only see azure files space used ( out of the box  metric)

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

ID: 75687a2c-c9e0-262f-197b-4b88b8e6d27d
Version Independent ID: 58bd1e87-6399-d276-7258-b5022d1019a8
Content: Monitor Azure Storage services with Azure Monitor for Storage - Azure Monitor
Content Source: articles/azure-monitor/insights/storage-insights-overview.md
Service: azure-monitor
Sub-service: insights
GitHub Login: @lgayhardt
Microsoft Alias: lagayhar

@lgayhardt Thanks for the feedback !  I have assigned this issue to content author to investigate and update the document as appropriate.

@reassign:normesta
Norm, can you help with this question? Or reassign as appropriate?

#reassign:normesta
Norm, can you help with this question? Or reassign as appropriate?

hi , any update on this one ? thank you in advance

Hi @SwathiDhanwada-MSFT,
This issue is a storage insights issue and should be assigned to the Azure storage team. Please assign to that team and remove the Azure-monitor label.
Thanks,
Abby

@SwathiDhanwada-MSFT
Per Abby's comment, please assign this to the Storage Insights team and update the tags accordingly.  Thank you.

@SumanthMarigowda  Can you please assign to appropriate poc?


ffmpeg error

Error I received:
C:\Users\user\Downloads>python Subtitles.py
Extracting audio from S1E01...
Generating subtitles for S1E01... This might take a while.
Detected language: English
90%|█████████████████████████████████████████████████████████████▉       | 131739/146739 [07:44<00:52, 283.83frames/s]
Adding subtitles to S1E01...
Traceback (most recent call last):
File "", line 198, in _run_module_as_main
File "", line 88, in run_code
File "C:\Users\user\miniconda3\Scripts\auto_subtitle.exe_main.py", line 7, in
File "C:\Users\user\miniconda3\Lib\site-packages\auto_subtitle\cli.py", line 67, in main
).output(out_path).run(quiet=True, overwrite_output=True)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\user\miniconda3\Lib\site-packages\ffmpeg_run.py", line 325, in run
raise Error('ffmpeg', out, err)
ffmpeg._run.Error: ffmpeg error (see stderr output for detail)
I installed ffmpeg with pip install ffmpeg-python in my base env (before I used it with choco but I received an error saying ffmpeg couldn't be found).
I've put my .py file and the .mp4 file both in my Downloads folder, this is the content of the .py file:
import os
os.system('auto_subtitle "Downloads/S1E01.mp4" -o subtitled/')
Not sure what I did wrong.
Edit: I don't have an output folder (subtitled/) but I imagine this will get created if not found.
This is the full stderr:
stderr output:
Traceback (most recent call last):
File "", line 198, in _run_module_as_main
File "", line 88, in run_code
File "C:\Users\user\miniconda3\Scripts\auto_subtitle.exe_main.py", line 7, in
File "C:\Users\user\miniconda3\Lib\site-packages\auto_subtitle\cli.py", line 49, in main
audios = get_audio(args.pop("video"))
^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\user\miniconda3\Lib\site-packages\auto_subtitle\cli.py", line 84, in get_audio
).run(quiet=True, overwrite_output=True)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\user\miniconda3\Lib\site-packages\ffmpeg_run.py", line 325, in run
raise Error('ffmpeg', out, err)
ffmpeg._run.Error: ffmpeg error (see stderr output for detail)

I also get this error. It gets stuck, you need to kill the ffmpeg process for it to proceed.
I run this on a headless server to generate transcription by batches. Hopefully it gets fixed soon.

try:
pip install ffmpeg-python
instead of:
pip install ffmpeg

try: pip install ffmpeg-python  instead of: pip install ffmpeg

I did.

Error Correction and Usage Enhancement
Issue:
The command provided in the previous section encounters errors due to incorrect formatting and potential flag functionality issues.
Corrections:

Quotation Marks: Remove unnecessary double quotes around the file path.
Flag Functionality: Address potential issues with the -o flag.
Direct Terminal Usage: Suggest executing the command directly in the terminal for simplicity.

Correct Command:
auto_subtitle /home/nick/auto-subtitle/test1.mp4 --model tiny --srt_only True

Repository Reference:
For extracting subtitles from multiple videos simultaneously and obtaining both .vtt and .srt files, refer to my altered repository, similar to the provided command structure.
auto-subtitle Repository
Please refer to the last section of the README.md in the above Git repository for detailed instructions on how to perform bulk extraction of subtitles for all the videos in the folder

I can confirm that the suggested option by @Nishan666 worked for me, i.e. passing --srt_only True as a flag additionally to the base command 👍
# Not OK,  ffmpeg error
auto_subtitle video.mp4 -o subtitled/

# OK! Everything works and subtitles are printed to subtitled/ folder
auto_subtitle video.mp4 -o subtitled/ --srt_only True

Is there a decent way to overlay subtitles on video? I thought the whole point of this repo was to put subtitles on the video, not just make a text file with them.

I tried to fix this error #81

Your commit works perfectly for me. Thank you!

try: pip install ffmpeg-python  instead of: pip install ffmpeg

Thank you!!

I got the error when I spelled my input video incorrectly.

I tried to fix this error #81

this fix works perfectly, thank you so much. also to the creator of this app: to a weeb like myself you are a life saver. now i never need to worry anymore when an episode of a show is broken i can just use this tool and sub it myself. its so awesome. it does a great job at it too. so both the creator of this app and mashinow ARIGATO! thank you so much!


Update Process Layer/Onboarding

3_3_1_Onboarding.md

[ ] Update to the state from RuleBook "<IP_ADDRESS> Admission of service providers".
[ ] What informations are needed for the certification process, registration to DAPS and ParIS?
[ ] Availabilty Setup must be updated. DMZ can be used, but also external Cloud services etc. Currently no need to distinction betwenn internal/external connector, cause no implementation supports that.
[ ] Connector Configuration is very dependend on the connector implementation. For onboarding a connector we have to figure out what the minimal common configuration is (x509, URL, TLS certificate, ?).
[ ] Not sure if we need the security setup section in onboarding. We maybe should put the verification in the exchanging data
[ ] Clearify if there will be a process of automatically getting the x509 certificate. Currently it is transfered manually via EMail, Matrix, ...

Consider besides Connector (Data Provider/Consumer) onboarding also:

[ ] Broker
[ ] AppStore,
[ ] Clearing House,
[ ] Vocabulary Provider,
[ ] DAPS,
[ ] ParIS
[ ] Service Provider,
[ ] Data Owner,
[ ] Data User,
[ ] App Provider

Closed by #193


Identity blocked on first launch when config invalid

Expected Behaviour
Identity should wait until the first valid config is available (with orgid and privacy status) to process the force sync event.
Actual Behaviour
Currently the force sync event is blocked by the first invalid config, no other events are processed after that.
Reproduce Scenario (including but not limited to)
Steps to Reproduce

fresh app install
I set first config with missing orgid MobileCore.updateConfigurationWith(configDict: [“dummy”: “config”])
I set second config with valid orgid MobileCore.configureWith(appId: self.LAUNCH_ENVIRONMENT_FILE_ID) - which provides valid orgid
Identity extension does not generate ECID, all the processing that requires identity is blocked

Platform and Version
Identity 3x extension
Sample Code that illustrates the problem
let extensions = [AEPIdentity.Identity.self,
                  Lifecycle.self,
                  Signal.self,
                  Edge.self,
                  Consent.self,
                  AEPEdgeIdentity.Identity.self
                  //step-extension-start
                  , SampleExtension.self
                  //step-extension-end
                  , UserProfile.self
                  // step-assurance-start
                  , AEPAssurance.self
                  // step-assurance-end
                ]

MobileCore.registerExtensions(extensions, {
    // Use the App id assigned to this application via Adobe Launch
    MobileCore.configureWith(appId: self.LAUNCH_ENVIRONMENT_FILE_ID)

    if appState != .background {
        // only start lifecycle if the application is not in the background
        MobileCore.lifecycleStart(additionalContextData: ["contextDataKey": "contextDataVal"])
    }
})

MobileCore.updateConfigurationWith(configDict: ["dummy": "config"]) // this gets executed before the registerExtensions callback and blocks processing

Logs taken while reproducing problem

Likely solution: Fetch latest configuration shared state during bootup process of Identity rather than the config versioned at the first event.


Schema should allow pagination links to be null

I have cases where my pagination links next and prev are null. The current schema does not allow that, the proposed change does.

Ref https://github.com/json-api/json-api/issues/887

It's not quite as simple as it might seem.
Suggest closing in favour of #1555

Thanks for weighing in @Relequestual. Closing in favor of your schema update.


[Coverity CID :191002]Error handling issues in /tests/net/lib/mqtt_pubsub/src/test_mqtt_pubsub.c

Static code scan issues seen in File: /tests/net/lib/mqtt_pubsub/src/test_mqtt_pubsub.c
Category: Error handling issues
Function: publish_handler
Component: Tests
CID: 191002
Please fix or provide comments to square it off in coverity in the link: https://scan9.coverity.com/reports.htm#v32951/p12996

looks like we aren't handing the case where rc == EGAIN.  We need to do a 'continue' in that case.


Added IAEA API

with correct formatting

[x] My submission is formatted according to the guidelines in the contributing guide
[x] My addition is ordered alphabetically
[x] My submission has a useful description
[x] The description does not have more than 100 characters
[x] The description does not end with punctuation
[x] Each table column is padded with one space on either side
[x] I have searched the repository for any relevant issues or pull requests
[x] Any category I am creating has the minimum requirement of 3 items
[x] All changes have been squashed into a single commit

Hi @justADeni
Thanks for your contribution ;)
We have some problems with Public APIs, so we won't be able to review/add this improvement right now.
Read this issue for more information, please: https://github.com/public-apis/public-apis/issues/3104


[Proposal] Apply copy-on-write (COW) rule wherever possible

Only copy the data when possible. For example:
df = mutate(mtcars)

# currently, df is a copy of mtcars
# but if there are no changes made to mtcars, it should be:
# df is mtcars

Hey--I suspect that copying ends up being pretty cheap computationally and space-wise. AFAIK a lot of the internal references are preserved (e.g. to underlying "Blocks" / arrays), until there's something that would mutate them.
(I could definitely be wrong here though)

Thanks for pointing this out. I somehow agree with you. This is a low priority as this could anyway save resources.


Remove manual command buffer release logic

Reduces unnecessary overhead from releasing command buffers.

See https://developer.apple.com/documentation/metal/mtlcommandqueue/1508686-commandbuffer, command queue handles this for us. Can be confirmed by adding some stray calls to  _commandQueue.commandBuffer(); without submitting them (which will correctly deadlock).

https://github.com/ppy/veldrid/pull/26

This looks to cause a massive memory leak. Will reassess.

The memory management flow in veldrid is a bit too complex for me to take on. Will just close this for now and monitor the situation as we optimise elsewhere.


Add mistral support

This PR is the follow up to #605 , adding adapter support for MistralAI's Mistral model.
It should fix the previous failed tests.

Thank you for taking the time to review this. I had to code on a windows machine where I couldn't use or install Make but I will be able to do that very soon. I also noticed an unexpected behavior when doing inference after loading Mistral models with the adapters library. I will look into that before pushing again.


[FALSE-POSITIVE]

https://essayservice.com
We have noticed through the VirusTotal report that our website (Essayservice.com) is reported on phishing by Phishing Database.
This website is whitelisted by all other sources:
https://www.virustotal.com/gui/domain/essayservice.com/detection
We would like to solve this issue and are ready to prove that we have never resorted to any actions related to Phishing or any other possible malicious actions.
Also, we are ready to provide you with all the needed conforming material and information upon your request.

This is the second time this has happened. I just checked our lists and we do not have you listed.
These are the only entries we have containing the word "essay"
phishing-domains/output/hosts/INACTIVE/hosts:<IP_ADDRESS> mail.writemyessays.xyz
phishing-domains/output/hosts/INACTIVE/hosts:<IP_ADDRESS> writemyessays.xyz
phishing-domains/output/hosts/ACTIVE/hosts:<IP_ADDRESS> essayecomast.000webhostapp.com
phishing-domains/output/hosts/ACTIVE/hosts:<IP_ADDRESS> essaylordsvoucher.serveo.net
phishing-domains/output/hosts/ACTIVE/hosts:<IP_ADDRESS> essayparlor.com
phishing-domains/output/hosts/ACTIVE/hosts:<IP_ADDRESS> irishessays.com
phishing-domains/output/hosts/ACTIVE/hosts:<IP_ADDRESS> nativeessaywriters.org
phishing-domains/output/hosts/ACTIVE/hosts:<IP_ADDRESS> newessayhelp.live
phishing-domains/output/domains/INACTIVE/list:mail.writemyessays.xyz
phishing-domains/output/domains/INACTIVE/list:writemyessays.xyz
phishing-domains/output/domains/ACTIVE/list:essayecomast.000webhostapp.com
phishing-domains/output/domains/ACTIVE/list:essaylordsvoucher.serveo.net
phishing-domains/output/domains/ACTIVE/list:essayparlor.com
phishing-domains/output/domains/ACTIVE/list:irishessays.com
phishing-domains/output/domains/ACTIVE/list:nativeessaywriters.org
phishing-domains/output/domains/ACTIVE/list:newessayhelp.live

@mitchellkrogza
Thank you for your response. According to VirusTotal we are still listed in this base. Their support team advised that we should ask GitHub to send them an updated report of the Phishing Database. Whom can we contact to ask for this kind of help? We tried to contact GitHub Support, but they are not responding for almost a month.
Thank you for your help in advance.

@TeamEssayService  We (and not GitHub) should clarify with them how they fetch and update the data from here. This repository (and it's underlying infrastructure) is not automatically communicating with any external API or services. We just update files and make them available through GitHub.
I'm aware - through emails I get copies of - that they are fetching our whitelisting list every 15 minutes but I don't know if they are using the proper tool. Indeed, we have here a whitelisting mechanism which is totally different from what the world may know.
We need to clarify the situation but our whitelist list - if fetched - should be used in combination with the proper whitelisting tool. It may work if they follow how the rest of the world is parsing a whitelisting list, but in our case, we have a tool that does a bit more in its parsing mechanism.
Our whitelisting list - and tool, let us apply a regular expression, special filter or even automatically remove complements (www.example.com if example.com is given and vice-versa) which the rest of the word doesn't normally do.
I really hope that they use our whitelisting tool.
The reason we are using that whitelisting tool - even if I'm the solely author of it - is that it's more powerful than a normal whitelisting tool. It abstracts a lot of things (like the complement case) that enforced us - in the past - to have a huge whitelist list.
Whitelisting all subdomains of a given subject without our whitelisting tool is really not funny. Indeed, we had to list all of them one by one. With our whitelisting mechanism, it's only one line: REG ^.*\.example\.org$. Of course, it does not happen every day that such a rule is used but it's still a relief and something - as far as I'm aware of - that you don't see in a "normal" whitelist list.
To understand the system a bit deeply: We have our whitelist list but if a source we work with whitelist something on their end, it may get automatically deleted from here (without an explicit whitelisting from us). In that case, if VT is creating a database with all our entries they should compare from time to time. Their database should match 1:1 what we distribute here. From my perspective, it's not clear right now if they are doing that.
We will try to discuss forward with VT in how we can help them solve the problem. It looks like - from my perspective - that they save everything they get from here but don't synchronize it in order to get a 1:1 of what we distribute here. Meaning that unless we explicitly whitelist as a subject, it doesn't get removed from their database.
Have a nice day/night.
Stay safe and healthy.
Nissar

Note:

VT should be able to:

Sync our entries 1:1 from time to time.
AND
Use our whitelisting tool if they fetch our whitelist list.
OR
Leave whitelisting to our infrastructure. Meaning they only need to fetch our entries and sync 1:1 from time to time.

/cc @mitchellkrogza

@TeamEssayService I have emailed VT again as we have whitelisted you last week already but it appears they are not ingesting the new data as we update it.

@TeamEssayService the problem has been fixed by VT. They cleaned the URL from their system but not the domain.


Terminal windows crash when you RDP in

Environment
Microsoft Windows [Version 10.0.18362.449]
Windows Terminal (Preview)
Version: 0.6.2951.0

Steps to reproduce
Log on to a machine using RDP which is already running a terminal, in my case they where Ubuntu terminals.
Expected behaviour
The terminals should remain running.
Actual behaviour
All terminals crash.
Details from event viewer
Faulting application name: WindowsTerminal.exe, version: 1.0.1910.22001, time stamp: 0x5daf7ab2
Faulting module name: ucrtbase.dll, version: 10.0.18362.387, time stamp: 0x4361b720
Exception code: 0xc0000409
Fault offset: 0x000000000006db8e
Faulting process ID: 0x23ec
Faulting application start time: 0x01d59e0083a1795f
Faulting application path: C:\Program Files\WindowsApps\Microsoft.WindowsTerminal_0.6.2951.0_x64__8wekyb3d8bbwe\WindowsTerminal.exe
Faulting module path: C:\WINDOWS\System32\ucrtbase.dll
Report ID: 98ac8e04-9879-4c10-a5e1-c47ab781f51a
Faulting package full name: Microsoft.WindowsTerminal_0.6.2951.0_x64__8wekyb3d8bbwe
Faulting package-relative application ID: App

/feedback

Could you clarify where to find that, the feedback link in terminal just links to the github issues page, I can't find anything that looks like the above.

@stevenh That's in the "Feddback Hub" app

Sorry I must be missing something as the feedbackup Hub app doesn't have an option to provide feedback for the terminal app.
All I get is:

@stevenh You need to scroll that first list up to "Apps":

Once you select Apps, it'll populate the second dropdown with all your installed applications.

Thanks @zadjii-msft I must say that's terrible UI on Feedback Hub

It pre-selects an random category from the list
It hides the scrollbar really quickly, when it does display one, doesn't always see my screenshot.
It looks like the list is alphabetical but isn't Apps, Gaming and Xbox then Accessibility
Search doesn't work well, I had to open App and Features to find the exact name of the Terminal app Windows Terminal (Preview) as scrolling through hundreds in the list would have take ages.
No suitable option in the Which of the following best describes your problem? really no option to report an application crash?

I'm sure this was meant to make things easier but it's been a real pain to get to this point. Compare that to the simplicity of raising the github issue an it's night an day, so definitely some improvements that could be made.
Anyway I hope you got the info you required, not sure how its going to get associated to this issue as it didn't ask me for any linking information.

@stevenh After filing your feedback, could you click the "Share my feedback" button? That will give you an aka.ms link you can share that we can use to find the crash report.


Equipped inventory icons don't replace the background texture

The background texture should disappear when an item is equipped (e.g. the sword in the hand slot should go away)

Reference: this is how it should look

And this is how it looks right now

Can you recheck this? I think i implemented the missing draw-flag in the commit mentioned above.

Yeah, works!


Add sample_dtypes method to Dataset

The current Dataset.fit logic uses a "dangerous" to_ddf().head(1) call to sample the "real" pandas/cudf data to infer the data types.  It makes sense that we cannot trust the dtypes attribute of the underlying Dask collection (for cudf-based collections), but there is no guarentee that head(n) will return the first n rows of the dataset unless there are >=n rows in the first Dask partition.  To avoid this problem, this PR adds a simple Dataset.sample_dtypes method that will iterate through the Dask partitions until a non-empty partition is found.  Although this may be slow for a large Dask collection with many empty partitions, the common case should have the same performance (and we are clearly fixing a bug).

cc @oyilmaz-nvidia

This looks handy for the upcoming schema work. 👍🏻


Run application in Docker

Hi. This is a basic Dockerfile that would allow users to run postgis-preview via Docker. I have tested it with Docker Engine in production and it works perfectly.
Usage is:
# In the directory with the Dockerfile, build Docker image
docker build -t='postgis-preview' .

# Create container (port 3000 is exposed in Dockerfile)
docker create --name='postgis-preview' postgis-preview

# Start container
docker start postgis-preview

Thanks for making such a great application!

Thanks!

Hi, so I gave this a try finally, but I am still not understanding what is happening.  All of the above commands work fine.  When I do docker start postgis-preview it just says postgis-preview and puts me back to the command prompt.  Is there something running?  How do I access it.  Thanks.

Sorry about the delay - I just saw the notification.  When you run that command, the application is now running on http://IP_OF_DOCKER:3000. The container will now run exactly the same on any docker host (so a local dev environment, a remote production environment, on windows/linux/osx etc) and all of the dependencies are managed by Docker rather than the host. You can also run different instances of it on different ports if you want to compare new functionality or have it pointing at different databases etc. Please let me know if this does not make sense.


Try it out link is 404 broken link

FYI, link on the Readme page of this project is a broken link.

Yes, I know just started development couple days ago.


Command handler having issues

https://media.discordapp.net/attachments/731392965360091196/755416383138824282/unknown.png?width=1442&height=128
Why do I keep getting this error?

Please update the link as it it showing invalid.

Getting a 403 on this, please either provide another image (imgur) might be more helpful.
Or please close this issue if you've solved it now.

You won't get an error unless it can't access the commands folder, which shouldn't be an issue unless you modified something


NTR: isoprenoid diphosphate phosphatase activity

Dear GO editors,
PubMed=20110354 shows that UniProtKB/Q8IY26 catalyzes the dephosphorylation of presqualene diphosphate and the isoprenoid diphosphates :  geranyl diphosphate, geranylgeranyl diphosphate and farnesyl diphosphate that are  involved in cholesterol biosynthesis and protein prenylation. Could we create terms for these activities?
For presqualene diphosphate the reaction is (not yet public):
Reaction=H2O + presqualene diphosphate = H(+) + phosphate + presqualene monophosphate [Rhea:67968]
For farnesyl diphosphate the reaction is :
Reaction=(2E,6E)-farnesyl diphosphate + H2O = (2E,6E)-farnesyl phosphate + H(+) + phosphate [Rhea:48128]
For geranyl diphosphate the reaction is :
Reaction=(2E)-geranyl diphosphate + H2O = (2E)-geranyl phosphate + H(+) + phosphate [Rhea:47944]
Thank you for your help.
Lionel

Hi @ibreuza
Can you note a suggested parent.
@pgaudet do we want to go with something more generic for the farnesyl and geranyl, as they differ by one unit or should I go ahead?

I was trying to get a more general compound here: https://github.com/geneontology/go-ontology/issues/16475
but I did not manage, or we would not have RHEA links. What do you think?

If we don't have the rhea links then the terms are of no use;

would 'isoprenoid diphosphate phosphatase activity' fit all of the above? Then I could make just that one, without the substrate specific reactions as children

@lbreuza
I think this was a question for you

would 'isoprenoid diphosphate phosphatase activity' fit all of the above? Then I could make just that one, without the substrate specific reactions as children

Thanks, Pascale

would 'isoprenoid diphosphate phosphatase activity' fit all of the above?

The enzyme can dephosphorylate (sequentially) diphosphate and monophosphate. I believe that a better parent term would therefore be:
"isoprenoid phosphatase activity"

If we don't have the rhea links then the terms are of no use;

Here are the reactions from Rhea. Unfortunately some are not yet public but will be made public soon.
Reaction=H2O + presqualene diphosphate = H(+) + phosphate + presqualene monophosphate [Rhea:67968]
Reaction=H2O + presqualene monophosphate = phosphate + presqualene alcohol [Rhea:68024]
Reaction=(2E)-geranyl diphosphate + H2O = (2E)-geranyl phosphate + H(+) + phosphate [Rhea:47944]
Reaction=(2E)-geranyl phosphate + H2O = (2E)-geraniol + phosphate [Rhea:68020]
Reaction=(2E,6E,10E)-geranylgeranyl diphosphate + H2O = (2E,6E,10E)-geranylgeranyl phosphate + H(+) + phosphate [Rhea:68008]
Reaction=(2E,6E,10E)-geranylgeranyl phosphate + H2O = (2E,6E,10E)-geranylgeraniol + phosphate [Rhea:68016]
Reaction=(2E,6E)-farnesyl diphosphate + H2O = (2E,6E)-farnesyl phosphate + H(+) + phosphate [Rhea:48128]
Reaction=(2E,6E)-farnesyl phosphate + H2O = (2E,6E)-farnesol + phosphate [Rhea:48132]

Then I could make just that one, without the substrate specific reactions as children

It is unclear to me why we can have GO terms for BP that are compound-specific "farnesyl diphosphate metabolic process

Hi @lbreuza
GO aims to captures MFs that corresponds to the level of specificity of the enzyme, not for each substrate. In this case - does it correspond to one class of enzymes that has wide substrate specificity?
For BPs, we aim to capture processes that are biologically relevant. For example the paper you cite mentions 'interconversion of isoprenols and isoprenoid phosphates', at which level do you think conversion of one compound into another is interesting as a GO class? Ideally GO would group functions/processes, if we go in too much detail that doesn't address this goal.
I hope this is clear? Please let me know if it isn't.
Thanks, Pascale

would 'isoprenoid diphosphate phosphatase activity' fit all of the above? Then I could make just that one, without the substrate specific reactions as children

I probably inadvertently closed that ticket. The suggestion of @hdrabkin would fit to the specificity of that protein.

Trying to close:
Term: isoprenoid diphosphate phosphatase activity
Def:Catalysis of  the dephosphorylation of isoprenoid diphosphates
Parent: isoprenoid phosphatase activity   <<<<< would need to be added as child of GO:0016462?
Def ref: PMID:33246356
And I could add the RHEA s(when available) as narrow xrefs

Sounds good to me.
Thank you.

Just updated my annotations...Thanks.
I am closing the ticket


Change of the block size in OTA transaction (TZ-457)

Question
Hi, I have the OTA feature working on my project, but there's a way to increase the size of a single block per request?
By default is 64 bytes (and I don't know is constant) so if the image is too large it take up to 1 hour.
Additional context.
No response

will follow up in https://github.com/espressif/esp-zigbee-sdk/issues/201


#trivial Invalidate sessionManager when remoteImageManager deallocated

PINURLSessionManager has a retain cycle by NSURLSession's delegate, let's break it when imageManager deallocated.

          1 Warning

:warning:
Any source code changes should have an entry in CHANGELOG.md or have #trivial in their title.

  Generated by :no_entry_sign: Danger

Please add a test.

Test added.

          1 Warning

:warning:
Any source code changes should have an entry in CHANGELOG.md or have #trivial in their title.

  Generated by :no_entry_sign: Danger

Thank you @zhongwuzw !

awesome test, thanks!


Enable https redirect test

What this PR does / why we need it:
Upstream dependencies fixed a bug that stopped us from enabling this test. Enable it.
Release note:

NONE

🎉
/lgtm


EmptyReturnFixer - it's now risky fixer due to null vs void

ref #1638
https://3v4l.org/7HscE

:+1:  :ship:


client-side alternative to nuxtServerInit

I am running some localStorage operations in store. Is there a way to dispatch this action similar to nuxtServerInit but on client-side instead?

We are working on implementing a nuxtClientInit for spa mode.
Be aware that you can have a mismatch between client-side and server-side if you fetch some data that can change the DOM into your localStorage.
Also, you can actually reproduce nuxtClientInit yourself, see https://github.com/nuxt/nuxt.js/issues/240#issuecomment-326893386

Works like a charm :-) Excellent.. I will keep that in mind..


Show error message on API error

The search results page will seem like its loading when the search API has actually returned an error. For example:
https://climbdex.com/results?minGrade=undefined&maxGrade=undefined&name=&angle=any&minAscents=1&sortBy=ascents&sortOrder=desc&minRating=1.0&onlyClassics=0&gradeAccuracy=1&settername=&holds=&mirroredHolds=&board=kilter&layout=1&size=8&set=1&set=20&roleMatch=strict
It would be more useful if there was an error message, possibly with a link to report the issue here.

PR created adding parameter validations as well as toast messages for errors

@brmnjsh thanks for the PR!
Good question about logging. I hadn't really thought about setting up anything robust yet. I think given the very low number of users currently, I'd just stick with outputting to the console, assuming that's what you mean by local server based. I'm able to get the console output in my fly.io dashboard, so I can generally use that for debugging and such. There are probably a few other features that are higher priority to make Climbdex a better drop-in replacement for the existing apps before more robust logging becomes a priority, but it would eventually be a nice thing to have.

@lemeryfertitta alright! I'll log to console for now.

Deployed! Thanks again, @brmnjsh


Remove possibility to save values inside of properties

It is against the intended use.

:+1:

:+1:

in addition, there should be an emphasized, explicit note in the wiki on the intended use of properties


TonConnect crashes application on user rejected connection

Sorry for inconvenience but what that hell is wrong with this TonConnect sdk? Why OnWalletConnectError in TonConnect throws exceptions that are impossible to catch and handle because this method called from arbitrary task that is not awaited when we receive payload? User reject to connect just lead to application crash and you cant do anything with it. Does this sdk even was tested?

Why do we even throw these exceptions in that method?? Method already propogated error message to StatusChangeErrorsHandlers, what purpose of throwing additional exception that even impossible to handle?

Sorry for inconvenience but what that hell is wrong with this TonConnect sdk? Why OnWalletConnectError in TonConnect throws exceptions that are impossible to catch and handle because this method called from arbitrary task that is not awaited when we receive payload? User reject to connect just lead to application crash and you cant do anything with it. Does this sdk even was tested?

No this SDK, wasnt tested.


[google_maps_flutter][Android] Polylines not automatically reflecting on Flutter 3.0.0

issue
updating the polylines of GoogleMap widget is not reflecting immediately in Flutter 3.0.0.
steps to reproduce

clone test repo https://github.com/zopagaduanjr/polyline_delay
add api key
build using flutter 3.0.0
tap on the toggle polylines button to see behavior
build using flutter 2.10.3
tap on the toggle polylines button to see behavior

demo: https://www.youtube.com/watch?v=3s0e6NQg10Q
version
Flutter 3.0.0 • channel stable • https://github.com/flutter/flutter.git
Framework • revision ee4e09cce0 (2 weeks ago) • 2022-05-09 16:45:18 -0700
Engine • revision d1b9a6938a
Tools • Dart 2.17.0 • DevTools 2.12.2

@blasten Looks like it's not just markers.

This is a duplicate of https://github.com/flutter/flutter/issues/103686. I will update the issue title.
This issue was fixed by just triggering the invalidate for  the marker#update method call in https://github.com/flutter/plugins/pull/5787, but I realized that a lot of operations are triggering markers update. I added a change in https://github.com/flutter/plugins/pull/5787 to ensure that all operations that trigger update receive an invalidation.


Add parsy, and parse create table statement with it

This pull request implements some part of parser for the "create table" with use of the parsy library, to validate whether it can be used to rewrite the whole parser. The goal is to make it easier to modify the code, which right now uses complicated regular expressions to get the job done.
All primitives and operations used are documented on these two pages:

Primitives
Methods, operators, and combinators

Changelog:

Add ci_string and optional_space_around functions for creating parsers
Implement column definition parser in parsy
Modify parse_column to use the column definition parser instead of the previous field_re regular expression and python logic.

Hi,
I noticed you created functions instead of making them methods of the class sql_token.
As the project is using classes I'd like to keep it this way.
Can you please change the  functions you added to be private methods of the class sql_token?
Thanks

I'll do that, thanks.
I would like to keep some of the parser functions at the module-level (such as ci_string, or optional_space_around) because they are generic parsers and not related to SQL in particular. I'd move the post_process and some more specific parsers as private methods. Does that make sense?

I have moved the methods. Maybe some of the new parsers too can be moved into the class instead of the top-level.
I'd say we can keep the generic parsers outside and the sql-specific parsers inside the class.

I've tested the PR and works very nice. I'll merge the PR as soon as you add the shebang on top of the test_tokenizer.py file, thanks

Thanks for the review.

Regarding the functions outside the class they can remain functions if they can be reused for other dialects.

This condition makes sense and makes it very clear.


problem in config reader

why is tagScheme is initalised with noSeg?
and it is not getting updated upon reading the config file, so far in my experiments tagscheme always defaults to BIO and not BMES how do you set the tagScheme? through config file (I don't think you've even written code for that)

noSeg is just an initial value, it represents the input format does not have the BIO/BMES format (such as POS tagging). The self.tagScheme will be updated automatically after loading the input file.


Improve the new search bar

With the update of  #170 a new search bar was added to the site. There are some possible improvements that could be made to make it more flush with the UI design of the other parts of the websites.
UI improvements:

[ ] Have the border of the search bar match the white of the rest of the site. Its currently a black border.

[ ] I'm unsure about this because I can't judge fonts but make sure that the font of the search... match w/ rest of site

[ ] Sometimes a search list of kickers appears and sometimes it does not appear, unsure if bug or feature

Possible updates to the search:

[ ] Make a combination of the old and new search where it would be a magnifying glass then ex

"Placeholder text and search result dropdown menu are currently in Arial, should be same font as subscribe button / section headings" - Rachel


Compatibility with tailwind?

Bug report 🐞
Sadly geist doesn't seem compatible with tailwind, i.e. doing something like this does not work:
import { Page } from "@geist-ui/react";

<Page className="bg-red">

Version & Environment

Version of geist-ui/react: v2.2.0

Expected Behaviour
The behavior I expect is it just works.
Actual results (or Errors)
nothing happens

refs: #317 #240
Under the current design, geist is not affected by the class names of the default weights.

ah, thanks


Making precomputed annotation layer

For segmentation and image layers, I use cloudvolume to convert my data to precomputed format so that Neuroglancer can read it. Cloudvolume does not support conversion for annotation layers, so I am trying to find another way to do this. It is not clear to me what the precomputed layer folder would look like, i.e. the file format for the annotations and the format of the info file. I see that annotation layers can be saved in precomputed format, e.g.: precomputed://gs://neuroglancer-janelia-flyem-hemibrain/v1.0/synapses, but I don't know how to see the contents without cloudvolume:
from cloudvolume import CloudVolume
vol = CloudVolume('https://storage.googleapis.com/neuroglancer-janelia-flyem-hemibrain/v1.0/synapses')

but I get an error when I do this:
KeyError: 'scales'

The cloudvolume maintainer has made it clear to me that annotation layers are not supported, so I am not surprised to see this.

Unfortunately, the only software we have at the moment for generating this format is based on a Google-internal C++ framework similar to Cloud Dataflow, and therefore can't suitable for open source release.
To view the raw data, you can simply use gsutil, e.g.:
gsutil ls gs://neuroglancer-janelia-flyem-hemibrain/v1.0/synapses
You can also refer to the documentation here:
https://github.com/google/neuroglancer/blob/master/src/neuroglancer/datasource/precomputed/annotations.md

Thanks for the info. I am interested in trying this simply for the reason that loading in point annotations via the python API or a CSV upload results in a very laggy viewer. I am using annotations to label cell centers, and I have order 10^5 cells. These cells can be independent points, not related in any way. Do you have any sense whether loading in these annotations via an annotation layer will change the responsiveness of the viewer?
I am having a hard time understanding the hemibrane synapses annotation layer, even with access to the file contents of that layer directory. For example, it is not clear to me what these shard files are or how they are made. I see a broken link in the annotations.md doc regarding them: https://github.com/google/neuroglancer/blob/master/src/neuroglancer/datasource/precomputed/sharding.md

Storing annotations in the Neuroglancer JSON state directly as you have
been doing, while convenient for a small number of annotations, is
unfortunately not very efficient for a large number of annotations.
The precomputed annotation format is multi-scale, and spatially indexed,
and should be able to handle essentially arbitrarily many annotations.  We
have used it with hundreds of millions of annotations, for example.  There
is a limit to how many points Neuroglancer can actually render at once with
a reasonable frame rate, but that is in the hundreds of thousands.
The correct link to the sharded format is here:
https://github.com/google/neuroglancer/blob/master/src/neuroglancer/datasource/precomputed/sharded.md
For your use case, though, with only 10^5 points, you can avoid most of the
complexity of the precomputed annotation format and just write a single
chunk in non-sharded format.
In the end, you will have a directory tree with just two files, mirroring
the format and names of the following two files:
gs://neuroglancer-janelia-flyem-hemibrain/v1.0/synapses/info
gs://neuroglancer-janelia-flyem-hemibrain/v1.0/synapses/spatial0/0_0_0
You will write all of your annotations to the 0_0_0 file in the binary
format described here:
https://github.com/google/neuroglancer/blob/master/src/neuroglancer/datasource/precomputed/annotations.md#multiple-annotation-encoding
The info file you can strip down to just the following:
{
"@type" : "neuroglancer_annotations_v1",
"annotation_type" : "POINT",
"by_id" : {
"key" : "by_id"
},
"dimensions" : {
"x" : [ 8e-09, "m" ],
"y" : [ 8e-09, "m" ],
"z" : [ 8e-09, "m" ]
},
"lower_bound" : [ 0, 0, 0 ],
"properties" : [],
"relationships" : [],
"spatial" : [
{
"chunk_size" : [ 34422, 37820, 41362 ],
"grid_shape" : [ 1, 1, 1 ],
"key" : "spatial0",
"limit" : 10000
}
],
"upper_bound" : [ 34422, 37820, 41362 ]
}
You should change chunk_size and upper_bound to equal the size in voxels of
your dataset, and adjust "dimensions" for your dataset.  This won't allow
indexing by annotation id or by associated segment id --- if you want that,
you will have to write additional files.  But this will be enough to
display the points in Neuroglancer.
On Mon, Jun 29, 2020 at 11:59 AM Austin Hoag<EMAIL_ADDRESS>wrote:

Thanks for the info. I am interested in trying this simply for the reason
that loading in point annotations via the python API or a CSV upload
results in a very laggy viewer. I am using annotations to label cell
centers, and I have order 10^5 cells. These cells can be independent
points, not related in any way. Do you have any sense whether loading in
these annotations via an annotation layer will change the responsiveness of
the viewer?
I am having a hard time understanding the hemibrane synapses annotation
layer, even with access to the file contents of that layer directory. For
example, it is not clear to me what these shard files are or how they are
made. I see a broken link in the annotations.md doc regarding them:
https://github.com/google/neuroglancer/blob/master/src/neuroglancer/datasource/precomputed/sharding.md
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/google/neuroglancer/issues/227#issuecomment-651300757,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABAEJ2T744WWZP3IY5HGJNDRZDQCVANCNFSM4OHG753A
.

OK, thank you that is incredibly helpful. I am trying the simple example you suggested.
I am unsure about how to encode a point position vector in binary format. For the count variable, I plan to do something like the following (in a simple example where I only have 2 point annotations that I want to add). Correct me if this is not going to work -- I have little experience writing binary files. The example is in python.
count=2
count_bytes=count.to_bytes(64,byteorder='little',signed=False)

But for the position vector, should I convert a tuple or list to bytes, or something else?

You can use the python struct module for encoding.
For example: struct.pack('<Q', 2)  to encode the number 2 as uint64le.
For float32le, struct.pack('<f', 0.5)
On Mon, Jun 29, 2020 at 3:20 PM Austin Hoag<EMAIL_ADDRESS>wrote:

OK, thank you that is incredibly helpful. I am trying the simple example
you suggested.
I am unsure about how to encode a point position vector in binary format.
For the count variable, I plan to do something like the following (in a
simple example where I only have 2 point annotations that I want to add).
Correct me if this is not going to work -- I have little experience writing
binary files. The example is in python.
count=2count_bytes=count.to_bytes(64,byteorder='little',signed=False)
But for the position vector, should I convert a tuple or list to bytes, or
something else?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/google/neuroglancer/issues/227#issuecomment-651399790,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABAEJ2VT5AM2BKRKEJMBD3TRZEHUDANCNFSM4OHG753A
.

I see. I can now get annotations to show up and the viewer is indeed much more responsive than when storing the annotations in the JSON state even with 10^5 annotations.
The code I used is:
with open('0_0_0','wb') as outfile:
    total_count=len(coordinates) # coordinates is a list of tuples (x,y,z)
    buf = struct.pack('<Q',total_count)
    for (x,y,z) in coordinates:
        pt_buf = struct.pack('<3f',x,y,z)
        buf+=pt_buf
    # write the ids at the end of the buffer as increasing integers
    id_buf = struct.pack('<%sQ' % len(coordinates), *range(len(coordinates)))
    buf+=id_buf
    outfile.write(buf)

One thing that is unexpected is the spatial distribution of annotations does not match the spatial distribution of my coordinates in the z dimension only. x and y look fine. Here is a screenshot from loading in the precomputed layer via the binary 0_0_0 file I wrote with the above code:

This is what the distribution of annotations looks like (and what the true distribution is) if I load them in via the JSON state:

I can "fix" this by making the Spacing (cross section) and Spacing (projection) less than a certain value in the widget within the "Annotations" tab. I don't understand what these spacing values mean and why their default is making my annotations squished in the z dimension.
For reference my info file is:
{
   "@type" : "neuroglancer_annotations_v1",
   "annotation_type" : "POINT",
   "by_id" : {
      "key" : "by_id"
   },
   "dimensions" : {
      "x" : [ 5e-06, "m" ],
      "y" : [ 5e-06, "m" ],
      "z" : [ 1e-05, "m" ]
   },
   "lower_bound" : [ 0, 0, 0 ],
   "properties" : [],
   "relationships" : [],
   "spatial" : [
      {
         "chunk_size" : [ 2160, 2560, 687 ],
         "grid_shape" : [ 1, 1, 1 ],
         "key" : "spatial0",
         "limit" : 10000
      }
   ],
   "upper_bound" : [ 2160, 2560, 687 ]
}

Where 2160,2560,687 is the volume size of the image layer and dimensions matches the resolution key of the image layer, i.e. 5x5x10 microns.

Neuroglancer "subsamples" by showing only a prefix of the list of
annotations according to the spacing setting.  If you set "limit" to 1 in
the info file, you won't get subsampling by default.  If you want the
subsampling to do something reasonable, then you can randomly shuffle the
order in which you write the annotations.
On Tue, Jun 30, 2020 at 9:56 AM Austin Hoag<EMAIL_ADDRESS>wrote:

I see. I can now get annotations to show up and the viewer is indeed much
more responsive than when storing the annotations in the JSON state even
with 10^5 annotations.
The code I used is:
with open('0_0_0','wb') as outfile:
total_count=len(coordinates) # coordinates is a list of tuples (x,y,z)
buf = struct.pack('<Q',total_count)
for (x,y,z) in coordinates:
pt_buf = struct.pack('<3f',x,y,z)
buf+=pt_buf
# write the ids at the end of the buffer as increasing integers
id_buf = struct.pack('<%sQ' % len(coordinates), *range(len(coordinates)))
buf+=id_buf
outfile.write(buf)
One thing that is unexpected is the spatial distribution of annotations
does not match the spatial distribution of my coordinates in the z
dimension only. x and y look fine. Here is a screenshot from loading in the
precomputed layer via the binary 0_0_0 file I wrote with the above code:
[image: annotations_badspatial]
https://user-images.githubusercontent.com/42473349/86152650-83791000-bace-11ea-9956-e015f4686b5b.png
This is what the distribution of annotations looks like (and what the true
distribution is) if I load them in via the JSON state:
[image: annotations_goodspatial]
https://user-images.githubusercontent.com/42473349/86152533-5167ae00-bace-11ea-942e-4fe1056cb345.png
I can "fix" this by making the Spacing (cross section) and Spacing
(projection) less than a certain value in the widget within the
"Annotations" tab. I don't understand what these spacing values mean and
why their default is making my annotations squished in the z dimension.
For reference my info file is:
{
"@type" : "neuroglancer_annotations_v1",
"annotation_type" : "POINT",
"by_id" : {
"key" : "by_id"
},
"dimensions" : {
"x" : [ 5e-06, "m" ],
"y" : [ 5e-06, "m" ],
"z" : [ 1e-05, "m" ]
},
"lower_bound" : [ 0, 0, 0 ],
"properties" : [],
"relationships" : [],
"spatial" : [
{
"chunk_size" : [ 2160, 2560, 687 ],
"grid_shape" : [ 1, 1, 1 ],
"key" : "spatial0",
"limit" : 10000
}
],
"upper_bound" : [ 2160, 2560, 687 ]
}
Where 2160,2560,687 is the volume size of the image layer and dimensions
matches the resolution key of the image layer, i.e. 5x5x10 microns.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/google/neuroglancer/issues/227#issuecomment-651919531,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABAEJ2X6M3JAPV6S5TVVGV3RZIKKXANCNFSM4OHG753A
.

Got it, thanks for the help!

@jbms In addition to coordinates, is it possible to add the props attribute in the 0_0_0 binary file, so that it can store two types of points in one file (e.g., one for pre- and one for post-synapse)? Or how to implement this feature?
Thanks!

Yes you can encode properties as well, just refer to the format documentation.
I normally use a line segment annotation for synapses.  You can style the endpoints separately.

Got it. Thanks!

I have a question of how to arrange multiple properties in the annotation encoding (spatail index). Say I have 3 properties for POINT annotations, "cell_size" (float32), "cell_type" (uint16, range from 1 to 20) and "point_color" (rgba, for debugging, r=1, g=2, b=3, a=255). When I put the "cell_color" in the last position of the properties entry in the json file, and use the following code for each point (padding a 2 bytes of zeros at last),
struct.pack('<4fH4BH', x, y, z, cell_size, cell_type, r, g, b, a, 0)
everthing goes well as shown below (as all alpha=255, the "cell_color" property becomes rgb style).

But when I put rgba in the second postion in the properties specification of the json file, and use the following code (the last 2B can also be H),
struct.pack('<4f4BH2B', x, y, z, cell_size, r, g, b, a, cell_type, 0, 0)
the properties becomes weird shown in the figure below.

"cell_size" is normal, but "cell_color" and "cell_type" are not, channel r and g disappear, and b=3 becomes r channel, a=255 becomes g channel and "cell_type" becomes the b channel. I also try b_arr = pack('<2B', 1, 2) then unpack it as unpack('<H', b_arr), it is 513, which is the actual "cell_type" show in the figure.
So, I wonder if a property type is rgba, does it have to be at last position of the corresponding array in the json specification. Or is there any detail I missed in the documentation?

Also, I think the color bar above is not correct too. For example, as in the first figure above, #010203 is very close to black, but #0203ff (ff is the alpha=255 channel) is close to the blue color as shown.

Regarding the color bar, thanks for reporting that issue, there was a bug in the display of RGBA colors, which is now fixed.
The issue with the property order is that the order in which the properties must be encoded is not the same as the order in which the properties are specified in the info file, as described here:
https://github.com/google/neuroglancer/blob/master/src/neuroglancer/datasource/precomputed/annotations.md#single-annotation-encoding
In order to minimize the padding bytes required, properties that require 4 byte alignment (uint32, int32, float32) are encoded first, followed by properties that require 2 byte alignment (uint16, int16), followed by properties that require 1 byte alignment (uint8, int8, rgb, rgba).  For a given alignment, the properties are encoded in which the properties are specified in the info file.
So in your case, the encoding will be same regardless of the order in which the properties are specified in the info file.

Hi Jeremy,
I have been using precomputed layers with a single spatial index successfully thanks to your help earlier in this thread. I have started getting requests to load millions or tens of millions of annotations in a single precomputed layer. Without spatial indexing this is starting to get slow, so I am looking at the documentation to create multiple spatial indexes.  https://github.com/google/neuroglancer/blob/master/src/neuroglancer/datasource/precomputed/annotations.md#spatial-index
I think I understand the basic idea here. I do have a few questions though.

It's not explicitly said in your instructions, but I am assuming that for each cell (aka "chunk") at each spatial index, you write out a file in the spatial folder for that index, using 0-indexing for the cell numbers. For example if I am on spatial level 1 and have grid_shape=[2,1,1] I would have two chunks: [0,0,0] and [0,1,1] and those would be written out as: spatial1/0_0_0 and spatial1/1_0_0. Is that correct?
I'm also not sure about what the order of the annotations needs to be in each of these chunk files. In your instructions referenced above you say the order should be bit-reserved morton code (not sure what that is). But at the end of this section: https://github.com/google/neuroglancer/blob/master/src/neuroglancer/datasource/precomputed/annotations.md#multiple-annotation-encoding you say to order them randomly. Can you clarify?
When picking the subsets of annotations in each cell, your instructions state:

"Compute a subset emitted(level, cell) of remaining_annotations(0, cell) where each annotation is chosen uniformly at random with probability min(1, limit / maxCount(level))"
Let's say for example we have 10^6 annotations and we're calculating the subset emitted for level 0 for which we only have one cell 0_0_0. maxCount(level=0) = 10^6 since that is the total number of cells in this volume. And say we're using a limit of 10^4. The probability of selecting each annotation is then 10^4/10^6 = 0.01. Is this equivalent to just picking 10^4 of the total 10^6 annotations at random? I guess this min() function ensures that the limit is enforced since you'll never fill a cell with more than 10^4 annotations. And in the case where maxCount() is less than the limit you will pick all of annotations. Is that right?

There are two formats supported: unsharded and sharded:
https://github.com/google/neuroglancer/blob/master/src/neuroglancer/datasource/precomputed/annotations.md#unsharded-spatial-index
https://github.com/google/neuroglancer/blob/master/src/neuroglancer/datasource/precomputed/annotations.md#unsharded-spatial-index
For unsharded the format is indeed as you have described.
They should be ordered randomly; the bit-reversed morton code was an initial experiment I tried to produce a more spatially-distributed set of annotations, but it turned out to be a bad idea since it distorted the distribution.  I have corrected the documentation in the annotations.md file.  Thanks for pointing this out.
If you have just a single cell (e.g. at spatial0 level), it is indeed the same as just picking 10^4 annotations at random to include in the first level, and retain the remaining for the later levels.  However, once you have more than one cell, you need to retain the same proportion of annotations in all cells, in order to avoid distorting the distribution.  For example, suppose at level 1 you have two grid cells in total, and there might happen to be 10^5 annotations in cell 0 but only 10^4 annotations in cell 1.  To achieve the desired limit of 10^4 annotations, you need to subsample only 10% of the annotations in each cell, such that you end up with 10^4 annotations selected for cell 0 and 10^3 annotations selected for cell 1.  When displayed in Neuroglancer, cell 0 will show 10x as many annotations as cell 1, which will correctly preserve the distribution.  If instead you just picked 10^4 annotations for each cell, then the same number of annotations would be shown in both cells, giving the appearance that the density is the same in both cells.

Note: In principle it would be possible to sample with different ratios in each cell, so that every cell gets to its limit if possible, and then record the sampling ratio that was used for each cell, so that Neuroglancer can take it into account.  However, that is not supported by the current format, so instead we must use a uniform sampling rate.

That makes sense, thanks for the prompt reply.

I was able to generate a spatially indexed precomputed layer from ~10^6 point annotations. I used limit=10^4. It does load very fast in Neuroglancer in comparison to using only a single spatial index, so that's good. However, the subsampling seems significantly higher in the multi-spatial index layer, i.e. there are way fewer points displayed at any given zoom level. The screenshot below shows the single spatial index layer (yellow points) with limit=10^4 and the multi-spatial index layer (red points) also with limit=10^4.

Is this expected? I am in particular concerned that when zoomed in the multi-spatial index does not show that many points -- the user needs to be able to see ALL of the annotations when zoomed way in since the primary use case is validating detected cells. I also tried a limit=1 multi-spatial index layer (as you suggested earlier in the thread for no subsampling in the single-index layer case) and it shows even fewer points overall. It is possible I implemented the spatial indexing algorithm incorrectly.

Thank you for this helpful thread.  I'm trying to implement writing of annotation collections now, and I'm confused by this bit:

Neuroglancer "subsamples" by showing only a prefix of the list of annotations according to the spacing setting.  If you set "limit" to 1 in the info file, you won't get subsampling by default.

How is the size of the prefix (subsample) shown determined?  Experimentally, with the same 100 line annotations in my 0_0_0 index, I find:
Limit = 10000  →  all 100 lines shown
Limit = 50000  →  64 lines shown
Limit = 100000  →  32 lines shown
Limit = 200000  →  16 lines shown
Limit = 400000 →  8 lines shown

I imagine it's doing some calculation of the total number of lines it might see by multiplying "limit" by the number of chunks, and trying to pick a reasonable prefix (subsample) size.  But when there's only one chunk, I would expect that calculation to come out to some constant value, like 1000 annotations.  I can't understand why the lines shown is actually shrinking with increasing limit, or why it's OK to lie about the limit value (documented as "the maximum number of annotations per grid cell") in order to avoid this subsampling.

Note: It is clear now that the subsampling is somewhat confusing and we probably need to somehow improve how it is handled to be less confusing.
The idea is that the "Resolution" setting in the UI specifies a limit on the density (in screen space) of annotations that is desired.  Neuroglancer then tries to ensure that this limit is met within the densest chunk, the one with limit annotations --- other areas may be less dense.
Based on the current camera parameters and the assumption that annotations are uniformly distributed with each chunk, Neuroglancer determines the desired density of annotations, then multiplies this density by the chunk volume at the current spatial index level (initially the top-most/coarsest level).  The fraction of annotations to subsample is given by desired_number_of_annotations / limit (clamped to not exceed 1).  Note that by setting limit to 1, this will always equal 1, i.e. no subsampling.  Then we subtract the density obtained from this spatial index level from the desired density, and if the desired density is still greater than 0 (can only occur if we didn't do any subsampling at this level), continue with the next spatial index level, until either there is no finer spatial index level or the desired density is reached.


Add wiki.json

Adding synthetic monitoring for wiki. Part of: https://github.com/dfds/cloudplatform/issues/2600

atlantis plan

Locks and plans deleted for the projects and workspaces modified in this pull request:

dir: environments/demo workspace: grafana-cloud-objects-demo-ws-01


Fix: stop crashing when parsing response content into ErrorResponse

PR Checklist
PR Structure

[x] This PR has reasonably narrow scope (if not, break it down into smaller PRs).
[x] This PR avoids mixing refactoring changes with feature changes (split into two PRs
otherwise).
[x] This PR's title starts with name of package that is most changed in the PR, ex.
paymentservice.stellar, or all or doc if the changes are broad or impact many
packages.

Thoroughness

[x] This PR adds tests for the most critical parts of the new functionality or fixes.

What
Stop crashing when parsing response content into ErrorResponse.
Why
So we can better debug what's causing an issue.

Something went wrong with PR preview build please check

Reference Server Preview is available here:https://anchor-ref-pr612.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr612.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr612.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr612.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr612.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr612.previews.kube001.services.stellar-ops.com/

Something went wrong with PR preview build please check

Something went wrong with PR preview build please check

Reference Server Preview is available here:https://anchor-ref-pr612.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr612.previews.kube001.services.stellar-ops.com/


Upgrade watchify from 3.x to 4.x

Version 3.x depends on chokidar 2.x that is incompatible with nodejs14. Upgrading watchify to 4.x so that the compatible (and smaller) chokidar is installed.

@jmreidy I hope you don't mind me taking the liberty of opening this PR. Please let me know if there's anything I can help with and I would be happy to do so.

Thanks for the PR @mdblr.

Sure, I'll do a release over the weekend. I just want to take a closer look at the changes in watchify from 3.x to 4.x and check whether there is anything to watch out for before releasing.

...released as 6.0.0


Add location property to NavLink

fixes #4910

Sorry, didn't see the tests dir at first. I've added two tests.

LGTM.

Let's do this. No harm in adding this API, as far as I can tell.


Unable to run 'python3 demo.py --env_config objectgoal_suncg_mf'

Hello, I wanted to try to run python3 demo.py --env_config objectgoal_suncg_mf. However, I encountered the following problem:
jin@jin-System-Product-Name:~/minos-0.6.x/gym$ python3 demo.py --env_config objectgoal_suncg_mf
train:87 episodes
test:34 episodes
val:32 episodes
2019-02-08 16:30:41,115 INFO {'sim_id': 'sim00', 'machine': 'jin-System-Product-Name'}
2019-02-08 16:30:41,125 INFO sim00:Starting sim server at /home/jin/minos-0.6.x/gym/minos/server/server.js with port 56493
2019-02-08 16:30:42,453 INFO sim00:connect
2019-02-08 16:30:42,542 INFO sim00:inited
2019-02-08 16:30:42,554 ERROR Error calling start: Error fetching scene /home/jin/work//suncg//house/69e283eec2f85eb53d528dcfb2172ab9/house.json
new_episode(): failure in start/reset
Traceback (most recent call last):
  File "demo.py", line 80, in <module>
    main()
  File "demo.py", line 76, in main
    run_gym(args)
  File "demo.py", line 13, in run_gym
    env.configure(sim_args)
  File "/home/jin/minos-0.6.x/gym/gym_minos/envs/indoor_env.py", line 26, in configure
    self._sim_obs_space = self._sim.get_observation_space(sim_args['outputs'])
  File "/home/jin/minos-0.6.x/gym/minos/lib/RoomSimulator.py", line 227, in get_observation_space
    self.init()
  File "/home/jin/minos-0.6.x/gym/minos/lib/RoomSimulator.py", line 158, in init
    return self.new_episode()
  File "/home/jin/minos-0.6.x/gym/minos/lib/RoomSimulator.py", line 141, in new_episode
    if 'goalObservations' in result:
TypeError: argument of type 'NoneType' is not iterable
2019-02-08 16:30:42,558 INFO sim00:disconnect
2019-02-08 16:30:42,558 INFO sim00:Stopping the simulator
2019-02-08 16:30:42,558 INFO Counter()
2019-02-08 16:30:42,558 INFO sim00:Stopping child servers
2019-02-08 16:30:42,558 INFO sim00:Killing sim pid 9213
2019-02-08 16:30:42,566 INFO sim00:Stopped child servers
2019-02-08 16:30:42,566 INFO sim00:Simulator killed.

Then I comment the following two lines:
if 'goalObservations' in result:
            del result['goalObservations']

However, I still encountered the following errors:
jin@jin-System-Product-Name:~/minos-0.6.x/gym$ python3 demo.py --env_config objectgoal_suncg_mf
train:87 episodes
test:34 episodes
val:32 episodes
2019-02-08 16:32:23,475 INFO {'sim_id': 'sim00', 'machine': 'jin-System-Product-Name'}
2019-02-08 16:32:23,483 INFO sim00:Starting sim server at /home/jin/minos-0.6.x/gym/minos/server/server.js with port 28741
2019-02-08 16:32:24,828 INFO sim00:connect
2019-02-08 16:32:24,915 INFO sim00:inited
2019-02-08 16:32:24,926 ERROR Error calling start: Error fetching scene /home/jin/work//suncg//house/69e283eec2f85eb53d528dcfb2172ab9/house.json
new_episode(): failure in start/reset
Running MINOS gym example
Starting episode 0
Traceback (most recent call last):
  File "demo.py", line 80, in <module>
    main()
  File "demo.py", line 76, in main
    run_gym(args)
  File "demo.py", line 17, in run_gym
    observation = env.reset()
  File "/home/jin/anaconda3/lib/python3.6/site-packages/gym/core.py", line 104, in reset
    return self._reset()
  File "/home/jin/minos-0.6.x/gym/gym_minos/envs/indoor_env.py", line 64, in _reset
    res = self._sim.reset()
  File "/home/jin/minos-0.6.x/gym/minos/lib/RoomSimulator.py", line 180, in reset
    output = self._augment_response(observation, None)
  File "/home/jin/minos-0.6.x/gym/minos/lib/RoomSimulator.py", line 207, in _augment_response
    observation = response['observation']
TypeError: 'NoneType' object is not subscriptable
2019-02-08 16:32:24,931 INFO sim00:disconnect
2019-02-08 16:32:24,932 INFO sim00:Stopping the simulator
2019-02-08 16:32:24,932 INFO Counter()
2019-02-08 16:32:24,932 INFO sim00:Stopping child servers
2019-02-08 16:32:24,932 INFO sim00:Killing sim pid 9759
2019-02-08 16:32:24,939 INFO sim00:Stopped child servers
2019-02-08 16:32:24,939 INFO sim00:Simulator killed.

Please give me some help!
Thank you very much!

It looks like it's not finding the SUNCG data.  Is your SUNCG data in /home/jin/work//suncg/ ?
If it is located in /xyz/suncg , you should run
NODE_BASE_URL=/xyz python3 demo.py --env_config objectgoal_suncg_mf


Support HDP <IP_ADDRESS>

Here's HDP 2.3, which is handled as an update to 2.2, rather than it's own GA branch.

LGTM


[C#] When inside comment blocks (/* */), pressing the Period key on the keyboard should not trigger Content Assist dialog box.

VSCode Version: 1.25.1
OS Version: Windows 10 Professional

Issue:
When inside comment blocks (/* */), pressing the Period key on the keyboard should not trigger Code Assist dialog box.
It is annoying when you invoke the Content Assist dialog box, but it is even worse when you press the Spacebar in places where it should have been whitespaces. Like

aaa
bbb
ccc

And then you accidentally invoke the Content Assist on the first "period", and the first entry is accidentally applied when the "Spacebar" key is pressed down.
Steps to Reproduce:

In a blank C# file, start with a comment block. (/* */).
Type out an ordered list (List that starts with 1, 2, 3, ...).
The moment you type a period in the ordered list, you are invoking the Content Assist dialog box.
All I'm doing is pressing only Spacebar and Period keys inside the comment block. The Content Assist shouldn't be invoked inside the comment blocks at all.

Does this issue occur when all extensions are disabled?: Yes
Do I think this is a Visual Studio Code issue? Yes

Aw darn, I didn't realize I DIDN'T successfully disable ALL extensions. Apparently, there was one extension hidden away in the list of installed extensions, and that happens to be C# OmniSharp extension. Disabling that fixes the Content Assist dialog box when typing out the ordered list, but that means I don't get Content Assisting in C#.
I'll report this to OmniSharp instead. Sorry for this.

https://github.com/OmniSharp/omnisharp-vscode/issues/2441


Configure FTS + add an index on the date columns

Sort by date descending is likely the most common way of sorting, so that column should be indexed.
Also add FTS configuration for both notes and the OCR column on resources.

I haven't done the FTS on OCR yet. I'm going to move that to another ticket because it requires more thought.

Should have used porter stemming for this.


Sorry, my mistake. I can reproduce it on dd3d9169 with seed 253253.

@Vasniktel I haven't seen this crop up when doing recent fuzzing runs; I reckon the problem has been fixed in the meantime.
I made loads of fixes to the fuzzer towards the end of March.
Would you be happy to close it?

The problem has been fixed.


Search in collection group

I wanted to search through the subcollections of documents and fetch the documents which matches the criteria given.
I went through the firestore documents and found there is a feature known as collection group with which we can achieve this.
So, Is the collection group feature is implemented in fireorm?
If so, can we get any documentation on that.
Thanks in advance.

As of today is not implemented in fireorm, but it'll be added! (can't say a timeline though). If you think you can add the support, PRs are welcome!
Closing since is a duplicate of https://github.com/wovalle/fireorm/issues/125


trace: don't finish spans twice when they are propagated

Consider the following example where the span tracing the https://users.service/users request will be finished by fetchUsers function:
func fetchUsers() ([]*User, error) {
	span := traceClient.NewSpan("/users")
	defer span.Finish()

	// Create the outgoing request, a GET to the users endpoint.
	req, _ := http.NewRequest("GET", "https://users.service/users", nil)

	// Create a span to attach tracing information.
	rspan := span.NewRemoteChild(req)
	defer rspan.Finish()

	// do the request, etc.
}

On the users.service side, our APIs allow the propagated span to be finished by the remote service:
func usersHandler(w http.ResponseWriter, r *http.Request) {
	span := traceClient.SpanFromRequest(r)
        defer span.Finish() // NOT THE RESPONSIBILITY OF THIS SERVICE TO FINISH THIS SPAN!!

	req, _ := http.NewRequest("GET", "https://users.service/meta", nil)

	rspan := span.NewRemoteChild(req)
	defer rspan.Finish()

	// ...
}

In the new trace prototype we have been working on, we came up with the solution of introducing FinishFuncs. Each span creator is returning a FinishFunc if the span is finish-able:
ctx, finish := trace.ChildSpan(ctx, "/users")
defer finish()

If you propagate and resurrect a span in the remote server, we are not giving you a finisher. So it is explicit that it is not responsibility to finish the span:
func Span(ctx context.Context, name string, info []byte) (context.Context, error)

We should follow a similar approach here and make sure that our APIs restrict the users not to misuse the finish feature.

/cc @jba @zombiezen @broady @jcd2

Blocked by the behavior of SpanFromRequest, see https://github.com/GoogleCloudPlatform/google-cloud-go/issues/538.

Closing in favor of https://github.com/GoogleCloudPlatform/google-cloud-go/issues/549.


CCC: CI/CD

Steps:

Run tests on pull requests via Travis CI.
Commits to the master branch require tests via Travis CI, and these master branch commits push new npm modules and docker images.
Dev and QA and Prod branches build the local internal stacks. These have fixed URLs.


please add aiida-core as dependency

aiida-core should be specified as a dependency in setup.json
depending on the aiida version, consider something like
aiida-core>=0.12.3,<1.0.0

or
aiida-core>=1.0.0b6,<2.0.0

Thanks. This has been done.

thanks!


Is this ready for django 2.2?

Read the tox and it does not seem to cover 2.2 hence the question

Yeah, I'm running several apps on Django 2.2 in production that use this package.
I'll update the tox. Thanks for pointing this out.


Day 1

General comments:

Include some inline comments to help organize the code blocks
Include more scaffolding in the challenge questions

Day 1:

Introduction slides are very clear and informative. My main suggestion would be to include more intuitive examples alongside the mathematical descriptions
“In the things we didn’t cover” section, include a brief sentence or two elaborating what each line means.
Where does the Trump example fit into the overall structure of Day 1?

Hi @emgrabow9 -- Can you commit and push these changes? Please close this comment when you are done. Let me know if you have any questions. Thanks!


Don 985 add friendly captcha support for person creation

Codecov Report
Attention: Patch coverage is 40.62500% with 19 lines in your changes missing coverage. Please review.

Project coverage is 70.00%. Comparing base (dc2548d) to head (fc966a2).

Files
Patch %
Lines

...Application/Middleware/FriendlyCaptchaVerifier.php
11.11%
16 Missing :warning:

src/Application/Middleware/RecaptchaMiddleware.php
40.00%
3 Missing :warning:

Additional details and impacted files
@@              Coverage Diff              @@
##             develop     #141      +/-   ##
=============================================
- Coverage      71.17%   70.00%   -1.18%
- Complexity       238      245       +7
=============================================
  Files             38       39       +1
  Lines            902      930      +28
=============================================
+ Hits             642      651       +9
- Misses           260      279      +19

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


display_url

Missing feedback from pycon development
commit e8b896733a9e13a15396a625b71f607984af8487
Author: James Tauber<EMAIL_ADDRESS>Date:   Mon Feb 11 15:54:43 2013 -0500

    added display URL on sponsor to allow distinction between anchor text and actual href

Signed-off-by: Hiroshi Miura<EMAIL_ADDRESS>

it is used in templates such as
project_name/templates/sponsorship/list.html:
{{ sponsor.get_display_url }}


Update service steppers following async stepper changes

Needs stepper (#2223) + service (#2188) PRs to merge

Done


Use consolidated saucelabs account

Coverage remained the same at 100.0% when pulling 71da5553154e59f6c59c6b87ae9e3b6d9c66aa1e on switch-sauce into 431b5c9c9a5696884f7efe3468490bbe7db6254d on master.


chore(deps): update dependency javax.servlet:javax.servlet-api to v3.1.0

This PR contains the following updates:

Package
Update
Change
References

javax.servlet:javax.servlet-api
minor
3.0.1 -> 3.1.0
homepage, source

Release Notes

javaee/servlet-spec
v3.1.0
Compare Source

Renovate configuration
:date: Schedule: At any time (no schedule defined).
:vertical_traffic_light: Automerge: Disabled by config. Please merge this manually once you are satisfied.
:recycle: Rebasing: Whenever PR is stale, or if you modify the PR title to begin with "rebase!".
:no_bell: Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

Available now for Enterprise: Renovate Pro with real-time webhook handling and priority job queue.

PR has been edited
:construction_worker: This PR has received other commits, so Renovate will stop updating it to avoid conflicts or other problems. If you wish to abandon your changes and have Renovate start over you may click the "rebase" checkbox in the PR body/description.

PR has been edited
:construction_worker: This PR has received other commits, so Renovate will stop updating it to avoid conflicts or other problems. If you wish to abandon your changes and have Renovate start over you may click the "rebase" checkbox in the PR body/description.


ExtraMetadataTools - Add logo from SteamGridDB not showing in Playnite 9 Beta

I'm not seeing the option to download logos from SteamGridDB in the ExtraMetadataTools extension for Playnite9 for some reason. I've entered the API key several times but it hasn't made a difference.

Think I've spotted the problem - you seem to have two menuItem11 entries


How to individually access stat scores in template.

How to individually access stat scores in template (including the bonus's).
So for example:
If I want to use an info box to reformat and add in Dice Roller functionality:

[!infobox]
Stats

Type
Stat

STR
19 (+3) dice: 1d20 +3

Ideally I could access this with

|STR|{resource.str} ({resource.strbonus}) dice: 1d20 {resource.strbonus}|

Thoughts?
C

yes. I think I would probably wrap it in stats instead of scores (scores would keep giving you the current string... stats would give you the parts.
Would be wrapping exposing this: https://github.com/ebullient/json5e-convert-cli/blob/main/src/main/java/dev/ebullient/json5e/qute/AbilityScores.java ...

Fixed by 5e798498f9c24d59d364a50b796b018795492e60

Almost works perfectly, very close ;->
Tried testing with Goblin. Negative value in the modifier works but Zero or Positive mod doesnt get added to the dice roll:

STR:  8 (-1) dice: 1d20 -1
DEX: 14 (+2) dice: 1d20 2
CON: 10 (+0) dice: 1d20 0

C

oh. because I didn't decorate the value anymore (not a string). Woops.

Works perfectly now thanks!


Create sample_from_sphere_or_ball

The function here samples uniformly from either the ball or the sphere with a given norm with possibly different radii
Description
Please include a summary of the change, motivation and which issue is fixed. Any dependencies changes should also be included.
Fixes # (issue)
Type of change
Please check all relevant options.

[x] Improvement (non-breaking)
[x] Bug fix (non-breaking)
[x] New feature (non-breaking)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Testing
Please describe the tests that you ran to verify your changes. Consider listing any relevant details of your test configuration.

[ ] Test A
[ ] Test B

Test Configuration:

OS
Python version
ART version or commit number
TensorFlow / Keras / PyTorch / MXNet version

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Codecov Report

Merging #1804 (a5f0ff2) into main (e7f3b7f) will decrease coverage by 48.29%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main    #1804       +/-   ##
===========================================
- Coverage   85.49%   37.20%   -48.30%
===========================================
  Files         271      271
  Lines       23048    23048
  Branches     4103     4103
===========================================
- Hits        19705     8574    -11131
- Misses       2331    14017    +11686
+ Partials     1012      457      -555

Impacted Files
Coverage Δ

art/estimators/object_detection/utils.py
0.00% <0.00%> (-100.00%)
:arrow_down:

art/attacks/evasion/shapeshifter.py
5.89% <0.00%> (-93.30%)
:arrow_down:

...rt/defences/detector/poison/clustering_analyzer.py
8.22% <0.00%> (-89.25%)
:arrow_down:

art/attacks/evasion/carlini.py
7.55% <0.00%> (-86.74%)
:arrow_down:

art/attacks/evasion/pe_malware_attack.py
14.28% <0.00%> (-84.80%)
:arrow_down:

...ces/detector/evasion/subsetscanning/scanningops.py
11.59% <0.00%> (-84.06%)
:arrow_down:

art/attacks/evasion/square_attack.py
10.32% <0.00%> (-84.04%)
:arrow_down:

art/attacks/evasion/wasserstein.py
11.94% <0.00%> (-83.28%)
:arrow_down:

art/attacks/evasion/elastic_net.py
17.88% <0.00%> (-81.46%)
:arrow_down:

art/defences/trainer/adversarial_trainer.py
11.50% <0.00%> (-80.54%)
:arrow_down:

... and 166 more

Hi @nmegiddo Thank you very much for implementing the correct sampling from norm spheres or balls! I pull this PR to a dev branch to move the function to art.utils.


CpuPool is lame after fork

The following code hangs forever:
extern crate futures;
extern crate futures_cpupool;
extern crate libc;

use futures::future::Future;

fn main() {
    let pool = futures_cpupool::Builder::new().create();
    let pid: i32 = unsafe { libc::fork() };
    println!("Forked: {}", pid);
    let f: Box<Future<Item = String, Error = String>> = Box::new(pool.spawn_fn(move || {
        println!("Running on pool in PID {}", pid);
        Ok(format!("PID {} done", pid))
    }));
    println!("Joined on future: {}", f.wait().unwrap());

    if pid != 0 {
        let mut status = 0;
        unsafe {
            libc::waitpid(pid, &mut status, 0);
        }
    }
}

The CpuPool in the forked child appears to never schedule work. (If I create the pool after the fork, it runs fine).
Is there any way for me to detect that a CpuPool is unable to schedule work? Or for a CpuPool to re-initialise itself after a fork? Or a wrapper type I can use which will None itself out after a fork, or re-initialise the CpuPool after a fork? Or really any way of handling the situation?
Right now, I have code which, when I explicitly fork manually, re-initialises a RwLock<Option>, but requiring me to proactively do this when I notice that I've forked seems error-prone.

I think this is just how multithreaded programs work with forking? I don't think there's really anything we can do about this...

Yes, if a program has threads, you cannot fork unless you immediately call exec. This is just a limitation with how fork works. The issue is that, when you add threading to the mix, the fork might happen while another thread has locked a global mutex. In the forked process, that mutex will never be unlocked, leading to a deadlock.


Configure stylish-haskell to format datatypes correctly

Blocked by: https://github.com/input-output-hk/ouroboros-network/issues/2951
It seems that all rules in the style guide regarding datatype formatting can be governed by stylish-haskell (version > 0.12.x.x)

As it turns out stylish (even in latest <IP_ADDRESS>) still misses some critical components for us to use it. Mainly:
  #     equals - How to format equals sign between type constructor and data constructor.
  #     # Possible values:
  #     # - "same_line" -- leave "=" AND data constructor on the same line as the type constructor.
  #     # - "indent N" -- insert a new line and N spaces from the beginning of the next line.

We need to have ability to say "equals same line but NOT the data constructor
  #     first field: How to format first field of each record constructor.
  #     # Possible values:
  #     # - "same_line" -- "{" and first field goes on the same line as the data constructor.
  #     # - "indent N" -- insert a new line and N spaces from the beginning of the data constructor

We need to say how we indent first field and other fields (because in our case this is different.
This means that this issue is blocked until necessary changes are done in stylish

We for now don't plan on having an auto-format on datatypes, but might eventually reconsider. Closing for now.


Added the apps Astah SysML, Astah Professional and PDFExpert

Offenses:
Casks/astah-professional.rb:14:4: C: Final newline missing.
end

Casks/astah-sysml.rb:14:4: C: Final newline missing.
end

2868 files inspected, 2 offenses detected
RuboCop failed!

I added the missing newlines.

Thank you for the contribution. It was merged directly as c28b0c7db7fdd20246f9ae4f4dec04f3c7452ba2 to keep commit history cleaner. Your contribution is still credited to you.


Configuration file

development configuration file that can be overwritten in production by enviormental variables

When the server is configured with the enviroment variable: SERVER_HOST=thomsen-it.dk
Then the cookies work.
But there still an issue of the cookie is granted for "www.thomsen-it.dk" then it is not accessible for "thomsen-it.dk"
I would call this as a good enough solution for now.

Good for now I think. May need a revisit


Replace Pending boolean with Confidence

This flag was being used to determine whether to use latest block or finalized block for event reads (originally named Finalized in a draft of the CLIP, but renamed to Pending during review due to some miscommunications around its purpose).
Having it be a Confidence level from 0 to 10 allows for more granularity but still avoids products needing to pass a chain-specific # of confirmations.  Instead, the Confidence level should be translated into # of confirmations based on the chain, in the RelayConfig.
Choosing int32 instead of uint32 leaves the door open for special values like Confidence = -1 which could be used to represent the Pending block in the tx pool. (Presently this is not supported, and there are no known use cases, but better to be more flexible in case the need arises.)
Along with this, we should also start using it for method reads to make things more parallel between events & state, the distinction between which the product plugins are not supposed to know or care about.

Can you please open PRs for the other repos that will need changes so we can review the whole set at once?

Can you please open PRs for the other repos that will need changes so we can review the whole set at once?

Ah, I forgot this was still open. I think @ilija42 is deciding on the exact approach, and might already be including changes to this effect (or something different?) in his present work.  @ilija42 should I just close this one? If it's still relevant, I can add a linked PR to chainlink repo.


Exactly one relationship type must be specified for CREATE - Example

Hi, I am running your example with 1.0.3 version, I am getting the following error while running the query:
api.query("MATCH (a:person), (b:person) WHERE (a.name = 'roi' AND b.name='amit') CREATE (a)-[knows]->(a)")
Error:
Exactly one relationship type must be specified for CREATE

I think the query should be:
api.query("MATCH (a:person), (b:person) WHERE (a.name = 'roi' AND b.name='amit') CREATE (a)-[:knows]->(b)")
Change the [knows] to [:knows] by adding the ":“ before the relationship, and change the last (a) to (b), otherwise this query won't make any sence.


can we not use api of openai, but local vicuna instead?

Yes, of course let's make a class together, we can use the majority of the OpenAI Class but we'll have to change the api call to the weights initialization

@kyegomez probably we just go with Langchain or whatever that abstract LLMs models

A pr on this https://github.com/kyegomez/tree-of-thoughts/pull/8, it uses ooba's textgen webui as server.

@yhyu13
Yeah Let's make an langchain class

And @1lifetime1 help build the class for vicuna!


Rebase on current stable branch

Make master branch ahead of stable branch, to make sure count of revisions
remains OK whatever branch you are sitting on (issue #1096).

Note that files remain strictly identical between this branch and current master

As no code is changed by this, I am going to merge it and build a 3.3.0 beta 2.
3.3.0 final will be released as soon as we have enough feedback, to respect as much as possible the new 2-months release cycle.


Addresses are needed

For my invoices, I'm required to have the address for the organization paying under their name and my address under my companies name. You could use -toaddress and -fromaddress in order to match the other flags.
But, it might be better to have them in a database and pull it up that way. Then you don't have to have the full address on every command line. Add it once, parse it in, and store it by the name and reuse by the name later.

@raguay  https://github.com/maaslalani/invoice/pull/21
CC: @maaslalani
please review when you have few mins

But, it might be better to have them in a database and pull it up that way. Then you don't have to have the full address on every command line. Add it once, parse it in, and store it by the name and reuse by the name later.

Of just put to-address and from-address in a json or yaml file and using --import client_name.json once #24 is straightened out.

Yes, a JSON file would be better. It would then be easy to include address information from different countries (there are different formats). Also, doing batch jobs would be easy if it was in a JSON file already.

I would like to suggest 1 json file that would hold all clients:
{
  "axelfoley": {
    "First Name": "Axel",
    "Last Name": "Foley",
    "Address": "Street 123",
    "Zip": "90210",
    "City": "Beverly Hills",
    "State": "California",
    "Country": "USA"
  },
  "johndoe": {
    "First Name": "John",
    "Last Name": "Doe",
    "Address": "Avenue 123",
    "Zip": "90210",
    "City": "Beverly Hills",
    "State": "California",
    "Country": "USA"
  }
}

And have users just issue invoice generate --client axelfoley
In fact, we could add the 'From' entities in the same file, and the misc settings too.


[WIP] Add secure route

What changes this PR?
This patch is for https://jira.coreos.com/browse/SRVKS-154. It changes:

to let openshift route take over from Ingress's annotations.
to introduce following annotations to support secured route for OpenShift route.

serving.knative.openshift.io/tlsMode
serving.knative.openshift.io/certificate
serving.knative.openshift.io/httpProtocol

What are the annotaitons?

serving.knative.openshift.io/tlsMode is to specify a type of termination. Valid options are reencrypt, edge and passthrough.
serving.knative.openshift.io/certificate is to specify secret which has a set of certs to pass OpenShift route.
serving.knative.openshift.io/httpProtocol is to specify the desired behavior for insecure connections to a route. Value options are redirect, none and empty whihch is same with routes.spec.tls.insecureEdgeTerminationPolicy.

Example usage
Example 1. Create a Edge termination route
1. Add annotation and create edge/None routes
$ oc annotate ksvc hello-example serving.knative.openshift.io/tlsMode=edge serving.knative.openshift.io/httpProtocol=none --overwrite
service.serving.knative.dev/hello-example annotated

$ oc delete ingresses.networking.internal.knative.dev  hello-example

$ oc get routes.route.openshift.io  -n istio-system hello-example-0
NAME              HOST/PORT                                           PATH   SERVICES               PORT   TERMINATION   WILDCARD
hello-example-0   hello-example.default.apps.example.com          istio-ingressgateway   80     edge/None     None

2. Access test

Only secure access (https) works.

$ curl -k https://hello-example.default.apps.example.com
Hello World!

Example 2. Create a Passthrough termination route
1. Add annotation and create passthrough/None routes
$ oc annotate ksvc hello-example serving.knative.openshift.io/tlsMode=passthrough serving.knative.openshift.io/httpProtocol=none --overwrite
service.serving.knative.dev/hello-example annotated

$ oc delete ingresses.networking.internal.knative.dev  hello-example

$ oc get routes.route.openshift.io  -n istio-system hello-example-0
NAME              HOST/PORT                                           PATH   SERVICES               PORT   TERMINATION        WILDCARD
hello-example-0   hello-example.default.apps.example.com          istio-ingressgateway   443    passthrough/None   None

2. Set up TLS settings on Knatie serving's Gateway
Refer to https://knative.dev/docs/serving/using-a-tls-cert/  (NOTE Knative serving does not support passthrough, so it is necessary.)
2. Access to secure route

Only secure access (https) works.

$ curl -k https://hello-example.default.apps.example.com
Hello World!

Why WIP?

Need to add some more unit tests for making route with TLS settings.

Closing this PR as https://github.com/openshift-knative/knative-openshift-ingress/pull/15 is merged. Edge termination might be requested in the future, but let's re-open this if necessary.