Add simple support for per language translations

As promised here's the PR to support translations for the language repository.
It's really simple: we define a default dictionary with keys which we use as fallback and lookup the translations dictionary in the index.yml.
This means once this is merged, we can (1) start extending this for more keys and (2) ping all language maintainer to use start adding language translation to their index.yml

Thanks Seb, I'll review the code later today, or tomorrow at the latest.

the keys should be a bit more verbose for better maintainability in the future as "reset" can mean many things in different contexts. For example "editor.reset"

@WebFreak001 Good point! Fixed.

Rebased to incorportate the new "Format" button.
@ZombineDev are you okay with this as an initial version?
We can always improve it ;-)
I think the biggest issue is that we agree (1) on using translations in the main language YAML file to list the translation changes and (2) on the keys that I have defined so far.

Ping @stonemaster @ZombineDev - imho this is ready to go as a first step towards allowing language-specific UI localization.
(Ignore the red cross on Travis, it's just dmd-nightly and a regression manifesting ...)

I've restarted the dmd-nightly job. Now that Martin reverted the problematic PR from dmd, I have higher hopes that it will pass now ;)

Turns out that dmd-nightly is from yesterday, but I can live with it being red for a couple of hours.

Thanks for your hard work, @wilzbach!

How about "Keyboard Shortucts" list?

How about "Keyboard Shortucts" list?

Please have a look at https://github.com/dlang-tour/core/pull/612 and let me know if that would work for you.

Am I understood right that should add texts this way?


KeyError while submitting the solution to task 4.

neptune run -- submit --problem fashion_mnist --task_nr 4
-- | --

5.372691 | /mnt/ml-team/homes/rafal.jakubanis/envs/minerva_venv/lib/python3.5/site-packages/h5py/__init__.py:36: FutureWarning: Conversion of the second argument of issubdtype from `float` to `np.floating` is deprecated. In future, it will be treated as `np.float64 == np.dtype(float).type`.
5.372979 | from ._conv import register_converters as _register_converters
5.373169 | Using TensorFlow backend.
12.010692 | [NbConvertApp] Converting notebook /mnt/ml-team/homes/rafal.jakubanis/minerva/resources/fashion_mnist/tasks/task4.ipynb to python
12.892425 | [NbConvertApp] Writing 641 bytes to /mnt/ml-team/homes/rafal.jakubanis/minerva/resources/fashion_mnist/tasks/task4.py
13.172734 | Traceback (most recent call last):
13.173071 | File "/mnt/ml-team/homes/rafal.jakubanis/envs/minerva_venv/lib/python3.5/site-packages/deepsense/neptune/job_wrapper.py", line 103, in execute
13.17331 | execfile(job_filepath, job_globals)
13.17354 | File "/mnt/ml-team/homes/rafal.jakubanis/envs/minerva_venv/lib/python3.5/site-packages/past/builtins/misc.py", line 82, in execfile
13.173774 | exec_(code, myglobals, mylocals)
13.174001 | File "main.py", line 66, in <module>
13.174223 | action()
13.174454 | File "/mnt/ml-team/homes/rafal.jakubanis/envs/minerva_venv/lib/python3.5/site-packages/click/core.py", line 722, in __call__
13.174719 | return self.main(*args, **kwargs)
13.174942 | File "/mnt/ml-team/homes/rafal.jakubanis/envs/minerva_ve
nv/lib/python3.5/site-packages/click/core.py", line 697, in main
13.175152 | rv = self.invoke(ctx)
13.175377 | File "/mnt/ml-team/homes/rafal.jakubanis/envs/minerva_venv/lib/python3.5/site-packages/click/core.py", line 1066, in invoke
13.175583 | return _process_result(sub_ctx.command.invoke(sub_ctx))
13.175779 | File "/mnt/ml-team/homes/rafal.jakubanis/envs/minerva_venv/lib/python3.5/site-packages/click/core.py", line 895, in invoke
13.17597 | return ctx.invoke(self.callback, **ctx.params)
13.17616 | File "/mnt/ml-team/homes/rafal.jakubanis/envs/minerva_venv/lib/python3.5/site-packages/click/core.py", line 535, in invoke
13.176348 | return callback(*args, **kwargs)
13.176538 | File "main.py", line 61, in submit
13.176747 | pm.submit_task(task_sub_problem, task_nr, file_path, dev_mode)
13.176932 | File "/mnt/ml-team/homes/rafal.jakubanis/minerva/minerva/fashion_mnist/problem_manager.py", line 34, in submit_task
13.177115 | new_trainer = task_handler.substitute(user_task_solution, user_config)
13.177299 | File "/mnt/ml-team/homes/rafal.jakubanis/minerva/minerva/backend/task_manager.py", line 14, in substitute
13.177482 | self.modify_trainer(user_solution, user_config)
13.177665 | File "/mnt/ml-team/homes/rafal.jakubanis/minerva/minerva/fashion_mnist/tasks.py", line 54, in modify_trainer
13.177847 | self.trainer.pipeline.get_step('loader').is_substituted = True
13.17803 | File "/mnt/ml-team/homes/rafal.jakubanis/minerva/minerva/backend/base.py", line 54, in get_step
13.178213 | return self.all_steps[name]
13.178396 | KeyError: 'loader'
13.178578 |
13.178761 | During handling of the above exception, another exception occurred:
13.178949 |
13.179133 | Traceback (most recent call last):
13.179326 | File "/mnt/ml-team/homes/rafal.jakubanis/envs/minerva_venv/lib/python3.5/site-packages/deepsense/neptune/job_wrapper.py", line 109, in <module>
13.17951 | execute()
13.179693 | File "/mnt/ml-team/homes/rafal.jakubanis/envs/minerva_venv/lib/python3.5/site-packages/deepsense/neptune/job_wrapper.py", line 105, in execute
13.179914 | raise ExperimentExecutionException("Exception during experiment execution", ex)
13.180092 | deepsense.neptune.exceptions.ExperimentExecutionException: ('Exception during experiment execution', KeyError('loader',))

@rafajak thank for the issue. I'll take a look at it

@rafajak it is fixed in the #72


Add form edit function

Value description
As a user
I want to edit my address,
so I can correct the typo mistake that I have made before.
Acceptance Criteria

[ ]  an edit-button should be shown
[ ]  the form should show the previously entered address
[ ] the text should be editable
[ ] there should be a save/confirm button

Tasks

[ ]  create a new branch
[ ]  create an edit button
[ ]  make previous entry editable in the form
[ ]  create save button

so I can correct the typo mistake that I have made before
Another use case (but probably for client databases rather than one-time purchases) would be a change of address due to a move, etc.

I'd argue that the user should be able to change/edit any input they've made (name, etc), so I'm not sure why you've chosen to single out only the address? :)


Run whole test suite on run-x

The test suite currently builds acton and uses it to run through the test suite in the test-linux & test-macos jobs. We also have run-linux & run-macos jobs in order to test that an actonc, once installed, also works... so we don't accidentally forget dependencies or similar.
The run-x jobs only compile a small helloworld program. We should expand that to include the full test suite as to ensure it all runs well on that platform, even though acton might not have been compiled on it.

allow specifying which actonc to use for running test.hs etc
clone acton repo in run-x & run make test

If we think about it, there are three platforms and "usage times" relevant to Acton:

development of Acton itself, which is like a handful of people and likely requires least amount of backwards compatibility as we all run on fairly new machines
development using actonc, which means any developer out there that might want to use actonc - we should support a fairly large amount of distributions
runtime of an acton program, this should go back as far as possible

If we now consider our CI jobs, the test-x jobs will be the equivalent of the first, i.e. development of acton itself. actonc must compile on these platforms... the run-x jobs will be the equivalent of development using actonc. And there is nothing for pure run time testing of actonc, which we could add, to test on even older boxes though it feels less and less valuable.

I don't actually think this is very important anymore. At the time, this was a good idea, but since then we have gone even further with our hermetically sealed builds. We rely on zig to produce statically linked executables so there is just very little that can go wrong based on differences between platforms. I think it's quite wasteful to run so much tests when the likelihood of error is so small. Zig and other projects are already doing a better job than us as testing things so we don't need to attempt a repeat - a bad repeat.
Until we have infinite resources, they are better spent elsewhere. Closing.


Support for scripts-descriptions

Steps required to reproduce the problem

If you have a package.json with the scripts-descriptions object, the values in there are not getting sorted

Expected Result
"scripts": {
        "post-install-cmd": [
            "@auto-scripts",
            "@composer normalize"
        ],
        "post-update-cmd": [
            "@auto-scripts",
            "@composer normalize"
        ],
        "auto-scripts": {
            "cache:clear": "symfony-cmd",
            "assets:install %PUBLIC_DIR%": "symfony-cmd"
        },
        "check-code-quality": "@phpqa",
        "php-cs-fixer": "vendor/bin/php-cs-fixer fix src --format txt --config .php_cs.dist",
        "phpcbf": "vendor/bin/phpcbf  --ignore=*/Migrations/* --extensions=php --standard=psr2 src tests",
        "phpdoc": "bash ./bin/phpdoc.sh run -p -d src/ -t docu/ --ignore \"src/Migrations/*,src/Kernel.php\" --encoding \"utf-8\" --cache-folder \"var/cache/dev/phpdoc\" --sourcecode",
        "phpqa": "ant -Dframework.php-qa-mode=cli package.check-code-quality"
    },
    "scripts-descriptions": {
        "check-code-quality": "Alias for phpqa",
        "php-cs-fixer": "Run PHP-CS-FIXER to actually fix",
        "phpcbf": "Run PHP-CBF",
        "phpdoc": "Generates the PHP-Docs",
        "phpqa": "Run all PHP QA stuff"
    }

Actual Result
"scripts": {
        "post-install-cmd": [
            "@auto-scripts",
            "@composer normalize"
        ],
        "post-update-cmd": [
            "@auto-scripts",
            "@composer normalize"
        ],
        "auto-scripts": {
            "cache:clear": "symfony-cmd",
            "assets:install %PUBLIC_DIR%": "symfony-cmd"
        },
        "check-code-quality": "@phpqa",
        "php-cs-fixer": "vendor/bin/php-cs-fixer fix src --format txt --config .php_cs.dist",
        "phpcbf": "vendor/bin/phpcbf  --ignore=*/Migrations/* --extensions=php --standard=psr2 src tests",
        "phpdoc": "bash ./bin/phpdoc.sh run -p -d src/ -t docu/ --ignore \"src/Migrations/*,src/Kernel.php\" --encoding \"utf-8\" --cache-folder \"var/cache/dev/phpdoc\" --sourcecode",
        "phpqa": "ant -Dframework.php-qa-mode=cli package.check-code-quality"
    },
    "scripts-descriptions": {
        "phpqa": "Run all PHP QA stuff",
        "check-code-quality": "Alias for phpqa",
        "phpcbf": "Run PHP-CBF",
        "phpdoc": "Generates the PHP-Docs",
        "php-cs-fixer": "Run PHP-CS-FIXER to actually fix"
    }

Thank you for your report, @aeimer!
Do you want to take a look at #89?

@aeimer
Tagged a new release, hope you like it!
See https://github.com/localheinz/composer-normalize/compare/0.8.0...0.9.0.

Yes it's great. But have i seen in right that you don't sort the scripts section?
Thanks for the super fast reply ;)

@aeimer
The scripts section is sorted by Localheinz\Json\Normalizer\SchemaNormalizer::normalize(), see https://github.com/localheinz/json-normalizer/blob/0.9.0/src/SchemaNormalizer.php#L205-L211.
In a nutshell, what happens there is the following:

pick all properties from the underlying schema in the order in which they are present in the schema, in this case, properties defined in the scripts section (see https://getcomposer.org/schema.json)
sort all remaining properties by key

Glad you like it!
I'm aware this tool doesn't save the world, but I'm happy to have contributed something that makes at least a small part of the work of a developer a bit easier.

Ah alright 👍
BTW, you could mention in the README that running this after install/update makes a lot of sense ;)
This is from a symfony 4 project:
"scripts": {
        "auto-scripts": {
            "cache:clear": "symfony-cmd",
            "assets:install %PUBLIC_DIR%": "symfony-cmd"
        },
        "post-install-cmd": [
            "@auto-scripts",
            "@composer normalize"
        ],
        "post-update-cmd": [
            "@auto-scripts",
            "@composer normalize"
        ]
    },


Feature Request: Focus on the selected components

Version: 2.3
Currently, when selecting components that are not in view, the viewport do not adjust (shift) to the selected components.
I suggest:

No click: current behavior, only highlight;
Single click: keep the line selected to keep the component(s) hightlighted even when the mouse is no longer hovering the line in the list.
Multiple selected components (CTRL+click to add, CTRL+SHIFT+CLICK to mark end of a list): multiple lines can be selected to highlight all components (involved);
Double click: adjust the viewport to the components selected by the line (possibly add an option to set the maximum zoom level for this function).

Auto zoom was also discussed in #246.
Multiple selection is planned as part of #206 where you will be able to "permanently" highlight some rows.

Thank your for the feedback.
It was difficult to tell from the time/initial message for the abovementioned items.  I try to keep one feature per "Issue". ;-).

Stale Issue, closed by the author, not a bot ;-).


A lot of problems

Hi! Thank you very much for this plugin, I think it really cool, because there are around 700 stars and 81 forks. But, my persanal user experience with it not very good.
There are a lot of problems in my case.... Everywhere. Here they are:

Default file extension doesn't work for me. That's what I have currently:

{
	"default_extension": "txt"
}

I also tried "default_extension": ".txt" (with dot before txt) but still doesn't work.

"use_cursor_text": true doesn't work if there are no text in first line. Also, I think it should take the text beside my cursor as file name. But instead of it, it always try to take the first line of the file, no matter where my cursor is currently located.

Just noticed now, new strange problem. Currently I have blank user preference file, but it still takes first line of file as file name.

Default path also doesn't work.

I use Sublime Text ver. 3, build 3126, on Windows 7 32 bit. I have no problems with any of another extensions, they are all work like a charm. (When I tested AdvancedNewFile, I disabled all of them).
It will be a good thing, I think, to post somewhere on the top of README.md completely workable user preferences preset.

Hi @mikenils, when you run the command with any of the settings, are there any errors in the sublime text console?


Flaky test reporting is not working

We have a flaky test bot that should report when tests are flaky
This doesnt cover many situations as it seems more common for test to pass or fail flakily rather than report as flaky
It took me some time to find a ci run where this should have been triggered
here is an example:
@envoy//test/integration:extension_discovery_integration_test             FLAKY, failed in 1 out of 2 in 25.2s
  Stats over 2 runs: max = 25.2s, min = 21.6s, avg = 23.4s, dev = 1.8s
  /build/tmp/_bazel_envoybuild/400406edc57d332f0b9b805d2b8e33a1/execroot/envoy_filter_example/bazel-out/k8-dbg/testlogs/external/envoy/test/integration/extension_discovery_integration_test/test_attempts/attempt_1.log

Executed 1023 out of 1023 tests: 1023 tests pass.
There were tests whose specified size is too big. Use the --test_verbose_timeout_warnings command line option to see which ones these are.
INFO: Streaming build results to: https://source.cloud.google.com/results/invocations/0aef9633-37fc-4a85-9526-41694c29325d
$TEST_TMPDIR defined: output root default is '/build/tmp' and max_idle_secs default is '15'.
no flaky test results found

https://dev.azure.com/cncf/envoy/_build/results?buildId=128412&view=logs&j=e969334a-0e55-5c18-ac96-8b546753391e&t=b5329a9e-93c7-5412-9f4c-6f8d659e1b57&l=868

cc @alyssawilk

given that its pretty rare that this would trigger and its not currently reporting im tempted to suggest just disabling flaky testing
either way - i think #25624 should fix from testing locally


Create specific IsEthereumChain function for chains using Ethereum block header format

We currently use

IsEVMChain: chain using EVM for execution layer
IsEthereumChain: Ethereum mainnet and testnets

We currently use one of these functions to determine if we should use the EthereumProof object for header and permissionless tx verification
However:

There could be other chains than Ethereum mainnet and testnets that use the same header format therefore IsEthereumChain is not adapted
Not all EVM chains use the Ethereum header format (ZetaChain itself), therefore IsEVMChain is not adapted

We need a specific function that determine if a chain can use the Ethereum header format for proof
@ws4charlie idea: IsCompatibleWithEthHeader

This pr has a similar implementation
https://github.com/zeta-chain/node/pull/1334/files#diff-19c25815bf12dd0469c0f1d7962931fb154fc029ebf203532448b0a6535335f1

It looks like yes, although I would rename it from IsHeaderSupportedEvmChain to something like IsEthereumHeader as you could have EVM chain we could support in the future using a different header format.

Completed


Understanding User Identity Proofing at Login.gov

User Story
As a ReportStream teammate, I want to understand how the Login.gov team undertakes identity proofing with users, so that we can extrapolate back to ReportStream and see what capability(s) we could reproduce here and/or if it's even a possibility to onboard Login.gov as the primary security platform.
Background and context
As part of the design brief to add additional third-party authentication providers and abstract our authentication layers, we will need to consider identity proofing as a separate functionality from Okta authentication. As more and more Senders are added to the Universal Pipeline in the near and far future, we will need to prove that these organizations 'are who they say they are', and are not bad actors that could send faulty or fraudulent data into ReportStream.
Open questions

How does Login.gov work today? What screen / process flows can we see?
What authentication functionality exists at Login.gov, today, in general?
Does Login.gov undertake identity proofing to prove users? How does it work?
Are there any repeatable / known user issues + pain points with any authentication functionality today?
How do organizations onboard with Login.gov? How do they get Login.gov’s identity proofing capabilities?

Working links

Part of Authentication enhancements design brief
Login.gov User Identity Proofing Discussion Guide

Acceptance criteria

[ ] Understand current Login.gov authentication capabilities
[ ] Understand if Login.gov undertakes identity proofing, and how this process works
[ ] Understand the process by which an organization onboards Login.gov

Not in scope

[] In-depth, technical discussion about authentication at Login.gov
[] Sharing RS's responsibility in terms of authentication and identity verification

Information has been synthesized in the Authentication/verification information and process flows Mural. Moving to 'Done'.


[spaceship] adding missing localizations attribute to AppStoreVersion

Checklist

[x] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[x] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary.

Motivation and Context

Adding an attribute to the AppStoreVersion model for localizations so that data can be included in calls.
Description

Testing Steps

@googlebot I consent..

@googlebot I consent..


Getting very slow

Please, tell me what can make very slow React-BigCalendar? It is taking like a minute to show 20 events that occur within the same month
I am using:
<BigCalendar
{...this.props}
selectable = {true}
formats={{dateFormat: 'dd', dayFormat:'dddd DD'}}
events={this.state.calendarEvents}
defaultView={'week'}
views={allViews}
                            defaultDate={new Date()}
                             onSelectEvent={event => ...}
                             onSelectSlot={(slotInfo) =>  ....................}
                             eventPropGetter={(this.eventStyleGetter)}
              />

I saw someone saying something about he startAccessor (issue #68) but no more explanations about what is startAccessor

It's really hard to diagnose perf issues since the can be really situation. I will so tho that I've used the calendar with hundreds of events in a month without performance issues, so I don't think its the component. Make sure you aren't doing unnecessary work like doing new Date()  in render() (cache the values you need)


Clean up

Remove trailing spaces
Converts all tabs to be spaces (was using a mixture)
Fixed various python formatting issues
Compress images
Tweaked README

:tada: :balloon:


Add samples for LLM multi-host GPUs tutorials

Description

This pull-request adds samples for cl/663150039.

Tasks

[X] The contributing guide has been read and followed.
[X] The samples added / modified have been fully tested.
[X] Workflow files have been added / modified, if applicable.
[X] Region tags have been properly added, if new samples.
[X] All dependencies are set to up-to-date versions, as applicable.

This is for a guide that is in the process of being released, it is a Multi-Host GPU tutorial using vllm. I'm preparing the files for the release.
As for the README.md, I can add it but I'm not sure if it is a hard requirement. I submitted this PR https://github.com/GoogleCloudPlatform/kubernetes-engine-samples/pull/1396 a couple of weeks ago without having to add one.

Thanks for the quick response, @Edwinhr716! :)
Sounds good. Let's omit the README.md file — it's tedious work.
As a follow-up, I've created https://github.com/GoogleCloudPlatform/kubernetes-engine-samples/issues/1421 (ideally, README.md files would be automatically generated).
Ping me (@NimJay) when this is ready for merging.

@NimJay it is ready for merging!


⚠ Cioos National CKAN Sitemap has degraded performance

In 606fc03, Cioos National CKAN Sitemap (https://catalogue.cioos.ca/sitemap/sitemap-1.xml) experienced degraded performance:

HTTP code: 200
Response time: 6395 ms

Resolved: Cioos National CKAN Sitemap performance has improved in 535876d after 11 minutes.


State of the art techniques for sentiment classification/analysis in English

This a request to add state of the art materials (research papers, data-set and code-sources) for sentiment classification/analysis in text available online (like tweets, movie and product reviews, news, domain specific text etc)

Are you aware of the SToA of sentiment analysis? If so, please suggest the paper and dataset in the area and I can update the content. I will also try to look for the papers in the area.

@taoyudong I am aware of SToA only in sentiment analysis in Code-mixed Hindi-English text. But for that too the data is not publicly available. I will tell you the paper and dataset for SToA in sentiment analysis in English as soon as i get.

Thanks for your contribution!


add windows support

This patch is still in progress. Some of the methods should be refactored, but this change set will make hmake working on Windows 7.
For some scripts (e.g. build.sh) that would be used directly in docker container, please make sure the line endings is LF(\n), the CRLF(\r\n). The inside /bin/sh does not recognize the CRLF endings.
Refer to this doc for more information.

This is a great patch!

Update pull requests.

[ ] fixing e2e tests

Can you squash all commits into one?

Everything looks good! Can you make the test pass? Then I can merge.

Actually the incompatible docker client/server versions causes e2e failure. Let me take a look (seems alpine has upgraded docker to a new version)

The e2e problem was cleared. Please rebase your change and refresh this PR.


capstone-server projects new

As a system admin, I should be able to add a new project. When a new project is created, a git repo is initialized for with the project skeleton in gitto. The project can be updated by doing a git push that repo.
The CLI
$ capstone-server projects new
title: Make your own Shell
name [make-your-own-shell]: shell

id: 45
name: shell
title: Make your own Shell
short_description: Make your own Shell
tags: []
git_url: https://git.capstone-staging.pipal.in/abcd1234/capstone-shell.git

The capstone-server projects list and projects show should also include the git_url.
Approach
Have a cookiecutter-capstone-project directory and use it to create a new repo.
When a new project is created, the git repo is created using gitto and the project created using cookiecutter is added to the repo as "initial commit".
Have API endpoint /api/projects/<name>/hook/<git-repo-id>, this is registered as webhook for the gitto repo. When it is invoked the following things are done:

Add change log entry with status=pending
An offline task update_project is registered with project_id and changelog_id as arguments
Print a message "Enqueued a task to update the project abcd"

Also, add a web page /projects/<project-name>/history that shows the change log of all the git push and if any push failed to update, that would be displayed with error background.
Offline task
The update_project task will update the project in a single db transaction. It will also update the a change log so that that the history page can show it.
The status of the change log entry will  be set to running at the beginning of the task and that gets changed to success or failed at the end of the task.

Log:

[ ] Add gito_repo_id, git_url to project table in DB
[ ] cookie cutter template
[ ] Add CLI command that creates gito repo, db repo, then initialises gito repo with cookie cutter template
[ ] API endpoint for webhook call

This is done

History page: https://capstone-staging.pipal.in/projects/shell/history
For the moment, there is no color for different status, but the updates are already sorted by latest, and the status is clearly visible. The logs are clearly shown when a header is clicked. This will work for the moment.

I see an unauthorised page. As a user what do you expect me to do?
On Thu, Apr 6, 2023, 5:49 PM Kaustubh Maske Patil @.***>
wrote:

History page: https://capstone-staging.pipal.in/projects/shell/history
For the moment, there is no color for different status, but the updates
are already sorted by latest, and the status is clearly visible. The logs
are clearly shown when a header is clicked. This will work for the moment.
—
Reply to this email directly, view it on GitHub
https://github.com/pipalacademy/capstone/issues/28#issuecomment-1498971415,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAB3EKAW7YSFUWIUBVT54DW72YETANCNFSM6AAAAAAWNHJKBE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Created #33 to address this.

I tried capstone projects new with CAPSTONE_SITE=localhost.
It added a gitto webhook http://localhost/.... Git push to that gave 404.
When I tried to start this project, it failed with:
subprocess.CalledProcessError: Command '['unzip', '-d', '/tmp/tmp74bq0s_h', '/home/anand/github/pipalacademy/capstone/private/projects/build-your-own-shell/repo.zip']' returned non-zero exit status 9.

I would really like this software to be robust enough to not to get into this kind of error states.

One of the flaws in our approach is that we are using the site name to identify the the site and there is no way to find the URL from the site.
I'm suggest replacing site-name with site-url in the CLI and find the site using the site-url. I'll create a separate issue for solving this.

We have the site domain for that.
Site URL can be found with "http://{site.domain}"
On Fri, Apr 7, 2023, 3:28 PM Anand Chitipothu @.***>
wrote:

One of the flaws in our approach is that we are using the site name to
identify the the site and there is no way to find the URL from the site.
I'm suggest replacing site-name with site-url in the CLI and find the site
using the site-url. I'll create a separate issue for solving this.
—
Reply to this email directly, view it on GitHub
https://github.com/pipalacademy/capstone/issues/28#issuecomment-1500136559,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AI7MKYMWW2NINEWQ4ZIYQRDW77QKPANCNFSM6AAAAAAWNHJKBE
.
You are receiving this because you were assigned.Message ID:
@.***>

Site URL can be found with "http://{site.domain}"

When site site is "localhost", which is the case for the dev instance, the URL is http://localhost:5000/, not http://localhost/.
Also in production, the URL should be https, not http.

One of the flaws in our approach is that we are using the site name to identify the the site and there is no way to find the URL from the site.
I'm suggest replacing site-name with site-url in the CLI and find the site using the site-url. I'll create a separate issue for solving this.

Created issue #34

When site site is "localhost", which is the case for the dev instance,
the URL is http://localhost:5000/, not http://localhost/.

Ah, yes. I fixed that by setting up a local Caddy instance that proxies :80
to :5000.
That seemed quickest for development. There is a bigger issue with
development on Mac because docker doesn't support network="host", and so
the checks runner in container won't be able to talk to capstone app /
gitto on local host. I'll setup another issue for this.
On Fri, Apr 7, 2023, 3:39 PM Anand Chitipothu @.***>
wrote:

One of the flaws in our approach is that we are using the site name to
identify the the site and there is no way to find the URL from the site.
I'm suggest replacing site-name with site-url in the CLI and find the site
using the site-url. I'll create a separate issue for solving this.
Created issue #34 https://github.com/pipalacademy/capstone/issues/34
—
Reply to this email directly, view it on GitHub
https://github.com/pipalacademy/capstone/issues/28#issuecomment-1500146054,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AI7MKYPIDU3U5SIRAZEWJA3W77RUZANCNFSM6AAAAAAWNHJKBE
.
You are receiving this because you were assigned.Message ID:
@.***>


Create First_and_Last_Position.py

Added solution

The code looks good
Thanks for the contribution @bot077
If you appreciate my work, please 🌟 this repository. It motivates me. 🚀

Enjoy HacktoberFest 💜


invalid GeV unit

Summary
GeV unit is not allowed. You cannot change quantity value with the web server in the Edit widget.
Steps to Reproduce
Try to change Value of quantity with  the GeV unit.
Current Behaviour
After accepting changes Invalid unit is displayed
Expected Behaviour
One should be able to change a quantity value with the GeV unit.
Extra Details
I attach screenshot

.

There is a list of predefined units which Scicat user to check if the unit is valid or not. For eV I can see eV and keV. I think we can add other unit prefix like M, G, T, P, E, Z Y too. What do you think @nitrosx and @stephan271 ?

I was expecting that all prefixes are supported for all units. Isnt that the case ?

No, the list has units with some common prefixes but not all prefixes. See here

For me it is weird eV and keV are treated as length units instead of energy units.


[#32] add a dayron response, adapter returning structs in dayron namespace

fixes #32

Coverage increased (+3.8%) to 90.265% when pulling e66a829dd92d7fa9cb5a3fdf9b925f1e5b780fad on flavio.32.add-a-dayron-response into 08afe3833de3d84ccf84a2fd02a00f37eef6bb72 on master.


Docker image

Hi,
This is a great script!
Is there are a chance that you can make a docker image for that tool?
I want to use it in my Synology Nas.
Thanks,
Idan

Hello!
Check this out: https://github.com/ndbroadbent/icloud_photos_downloader/pull/32
Cheers,
Jonas

Merged!


module is not compatible with rollup.js bundling and tree-shaking.

The current sample code does not cover the inclusion of the module/component via @NgModule app import.
This causes a problem for me as trying to import the ChartistModule (rather than importing the component at a page level as it used to be) results in the error Unexpected value 'ChartistComponent' exported by the module 'ChartistModule' , suggesting that I'm importing it incorrectly.

Should also say that I'm using Ionic 2, was working fine before Angular 2 moved to modules.

In Ionic2 RC0, you need to make some changes in the rollup config for angular2-chartist to work:
In<EMAIL_ADDRESS>change the commonjs() line to :
    commonjs({       include:[         'node_modules/angular2-chartist/**',         'node_modules/rxjs/**',         'node_modules/chartist/**'       ],       namedExports:{         'node_modules/angular2-chartist/dist/angular2-chartist.js' : ['ChartistComponent']       }     })
You can see reference to this issue here

Is there anything I can do here to avoid extra configuration? Or is this
just an ionic quirk?
On Thu, Oct 13, 2016, 12:52 PM Maxence Lehuraux<EMAIL_ADDRESS>wrote:

In Ionic2 RC0, you need to make some changes in the rollup config for
angular2-chartist to work:
In node_modules@ionic https://github.com/ionic\app-scripts\config\rolllup.config.js,
change the commonjs() line to :
commonjs({
include:[
'node_modules/angular2-chartist/',
'node_modules/rxjs/',
'node_modules/chartist/**'
],
namedExports:{
'node_modules/angular2-chartist/dist/angular2-chartist.js' :
['ChartistComponent']
}
})
You can see reference to this issue here
https://github.com/driftyco/ionic-app-scripts/issues/68
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/paradox41/angular2-chartist/issues/48#issuecomment-253620396,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACqZMiD4SgYNj29-Q6NxYKlxczIoMp8gks5qzowZgaJpZM4KVbh2
.

It can be fixed on your side from what I read. It seems to be more of an incompatibility with how your component is distributed and how rollup would like it to be distributed.
Here's what @danbucholtz says in valor-software/ng2-charts#440:

We're going to provide some instructions to Angular library creators (and really any library creators) on how to distribute their dist directory with commonjs modules AND es2015 modules and link them up in the package.json using the main and module fields.
Once this library follows the Angular convention, everything will just work.

Thanks, yes it also looks like they're going to default to webpack soon instead of rollup. Renaming issue to reflect rollup incompatibility.

So my final fix/workaround for this was to install the latest @types/chartist and use a custom rollup config with the following namedExport for angular2-chartist:
commonjs({
            namedExports: {
                // chartist
                'node_modules/angular2-chartist/dist/angular2-chartist.js': ['ChartistComponent'],
                ...
            }
        }),

As an aside, I did get a weird problem where my line chart had a filled black area, even with showArea set to false, worked around with with css. Assume this is more a problem/change with base chartist library.

I'm currently working on a fix for this to hopefully avoid any extra
configuration. For now, you can use workarounds obviously. I'll mention
something here once the fix is released.
On Tue, Oct 18, 2016 at 12:21 AM Christopher Cook<EMAIL_ADDRESS>wrote:

So my final fix/workaround for this was to install the latest
@types/chartist and use a custom rollup config with the following
namedExport for angular2-chartist:
commonjs({
namedExports: {
// chartist
'node_modules/angular2-chartist/dist/angular2-chartist.js': ['ChartistComponent'],
...
}
}),
As an aside, I did get a weird problem where my line chart had a filled
black area, even with showArea set to false, worked around with with css.
Assume this is more a problem/change with base chartist library.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/paradox41/angular2-chartist/issues/48#issuecomment-254403063,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACqZMj32fvFvdKYoRnyAF1EwWeU3LDWSks5q1ElCgaJpZM4KVbh2
.

Thanks @paradox41!
Thanks,
Dan

I released this as 0.10.0. Hopefully there are no breaking changes, please let me know if this fixes the problems people are seeing and if the extra configuration can be removed.

Thanks, it might be closer but it's not working for me, unless I'm using it wrong? I'm assuming I just import the Module and the rest of my app can then use <x-chartist></x-chartist> as normal. this would be the same for any angular 2.0 app, not just Ionic based ones.
`import { ChartistModule } from 'angular2-chartist';`
...
imports: [
    IonicModule.forRoot(MyApp, { mode: 'ios' }),
    ChartistModule
  ],
...

Error message:
[09:39:23]  typescript: C:/Work/SVN/Dashboard/app/dashboard/src/app/app.module.ts, line: 42
            Module '"C:/Work/SVN/Dashboard/app/dashboard/node_modules/angular2-chartist/dist/chartist.component"'
            has no exported member 'ChartistModule'.

      L41:  import { AdminOptionsSuperuser } from './../components/admin-options-superuser/admin-op
      L42:  import { ChartistM
dule } from 'angular2-chartist';

Any chance you could use the quick ionic 2 starter (http://ionicframework.com/docs/v2/getting-started/installation/) to try to display a chart for yourself? I appreciate that might take time but I also note that the example code in the readme is based on an old angular 2 rc pre-NgModule and that could be a stumbling block for people trying to use this component/module. perhaps solving this would help resolve that too.

I wonder if this has to do with Typings. Currently I can't specify more than one typings location (or concat typings). Looking at the path to the file node_modules/angular2-chartist/dist/chartist.component"', I would expect to see node_modules/angular2-chartist/dist/angular2-chartist.js.
What version are you on?

I've tried angular2-chartist 0.9.5 and 0.10.0. No old typings or typings.json, just using npm types for chartist.
Ionic 2.0 RC1, with Angular 2.0.0
I've given up for now as I've spent an impressive number of hours trying to get chartist working in some form and the people I'm doing the work for won't be impressed :)
Can you confirm that this currently works in regular Angular 2.0.x?

Yes, I can run it locally with Angular 2. I'll have to spend more time
tonight with it. In the meantime, does it work with the above workaround?
On Wed, Oct 19, 2016, 9:35 AM Christopher Cook<EMAIL_ADDRESS>wrote:

I've tried angular2-chartist 0.9.5 and 0.10.0. No old typings or
typings.json, just using npm types for chartist.
Ionic 2.0 RC1, with Angular 2.0.0
I've given up for now as I've spent an impressive number of hours trying
to get chartist working in some form and the people I'm doing the work for
won't be impressed :)
Can you confirm that this currently works in regular Angular 2.0.x?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/paradox41/angular2-chartist/issues/48#issuecomment-254814031,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACqZMvpEjXlu0_uP5h-ksqOOGWb0Gkmsks5q1hycgaJpZM4KVbh2
.

Unfortunately I got 0.9.5 working, then saw your update and applied it. Should have committed my changes! Now can't seem to get back to a working version despite trying all sorts of variations. Have currently reverted to using Chartist directly in my own component but the Ionic component lifecycle is causing issues when page navigation takes place, which is something your component seemed to handle fine (which is why I'd eventually like to get it working again).

I'll do my best to fix it tonight. Sorry for the trouble.
On Wed, Oct 19, 2016 at 9:48 AM Christopher Cook<EMAIL_ADDRESS>wrote:

Unfortunately I got 0.9.5 working, then saw your update and applied it.
Should have committed my changes! Now can't seem to get back to a working
version despite trying all sorts of variations. Have currently reverted to
using Chartist directly in my own component but the Ionic component
lifecycle is causing issues when page navigation takes place, which is
something your component seemed to handle fine (which is why I'd eventually
like to get it working again).
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/paradox41/angular2-chartist/issues/48#issuecomment-254817788,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACqZMg7X2rs2Akyiu-6NPC84k6-RJgXmks5q1h-6gaJpZM4KVbh2
.

No worries, good luck!

@webprofusion-chrisc can you do me a favor? Go ahead and npm install the master branch from here, cd into it and run npm i && npm run build:dist (I think this might be needed since I don't keep the dist in version control) and let me know what happens for you.
I just tested again locally and everything works as expected, the module is being exported correctly. I refactored a bit so it is one file now, but that shouldn't change anything for consumers to be honest.
If you continue to have issues, it could be an ionic quirk that I have to look into more.
Thanks.

@webprofusion-chrisc update on this. I did as you asked and tested with the ionic 2 starter using the updated build, everything worked as expected. I get some .d.ts errors that are unrelated to this project though, but the module is correctly located and importable.

I am gonna go ahead and cut a release for this.

I just tested and I still had to modify the rollup.config.js to have this in commonjs():
{ namedExports:{ 'node_modules/angular2-chartist/dist/angular2-chartist.js' : ['ChartistModule'] } }
Otherwise, still works good

Hmm, when I tried it ionic build browser --release (which is quite a strict compile) I get:
[20:15:33]  Error: Unexpected value 'ChartistModule' imported by the module 'AppModule'
[20:15:33]  ngc failed
[20:15:33]  ionic-app-script task: "build"
[20:15:33]  Error: Error

If I just do ionic serve I get:
[20:22:54]  transpile update started ...
[20:22:57]  transpile update finished in 2.52 s
[20:22:57]  bundle update started ...
[20:23:02]  bundle update failed: 'ChartistModule' is not exported by node_modules\angular2-chartist\dist\angular2-chartist.js
            (imported by src\app\app.module.ts). For help fixing this error see https://github.com/rollup/rollup/wiki/Troubleshooting#name-is-not-exported-by-module
[20:23:02]  watch ready

That's with or without a custom rollup config entry. Not sure whats different.
It's obviously rollup that causing the issue, so when ionic switch to webpack shortly it will hopefully resolve itself.

Unfortunately, I am not sure what is happening. I was able to successfully compile an ionic app using the module. Is it possible there is some configuration you have somewhere that is throwing it off?

Yes, it's possible! This is a different machine (and app) from the one I had problems with though, so I'll try again on the other machine tomorrow.

Alas I just can't get it working. Sorry, just one more check: can you (or anyone else) confirm that ionic build browser --release builds OK.  As I understand it ngc (compilation) doesn't run using the standard ionic serve, just ionic build

Ok, I've narrowed it down - ionic serve works (using the current git version of the ionic app scripts) but a full build with ionic build doesn't.
For angulars AOT compilation to work the package needs a .metadata.json (alongside the .d.ts) generated by the angular-cli command ngc (equivalent to typescript tsc but should generate a .metadata.json file). Unfortunately although I can get it to compile, no .metadata.json file appears, so I'm still doing something wrong.

I think ionic build is deprecated.
Use "npm run build" or "npm run build --dev" instead which will use the ionic-app-scripts to build your application.
More info here: ionic-app-scripts

@mlehuraux-maf ionic build isn't deprecated at all, it runs the same build step (ionic-app-scripts). The remaining issue is compatibility with angular 2 AOT step using ngc, although you have indicated you can get it to build. I can assure you that the process is not working for me, it does work with npm run build --dev but I'm guessing that skips AOT.

I'm trying to reproduce the findings here. I've setup two identical Ionic projects with a minimal chartist setup one with the current  apps-scripts using rollup and one with the beta apps-scripts using webpack.
Rollup:
Regardless of ionic serve or ionic run android I see bundle failed: 'ChartistModule' is not exported.. This is a surprise as I think it should work without AoT compilation?! Adding the custom rollup config didn't change anything.
Webpack:
Using webpack the ionic serve works fine and I can see the chart being displayed. But npm run build / ionic run android still issues the "Chartist Module is not exported" error.
Anything screwed up in my simplified setup?`
Versions:
ionic 2 Rc1
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>home.ts
import { Component } from '@angular/core';
import { NavController } from 'ionic-angular';
import { ChartType } from 'angular2-chartist';

@Component({
  selector: 'page-home',
  template: `
    <x-chartist
      [data]="data"
      [type]="type">
    </x-chartist>
  `
})

export class HomePage {
 type: ChartType;
 data: any;

  constructor(public nav:NavController) {
    this.type = 'Line';
    this.data = {
      "labels": [
        "Jan",
        ......
        "Dec"
      ],
      "series": [
        [5, 4, 3, 7, 5, 10 ,3, 4, 8, 10 ,6, 8],
        [3, 2, 9, 5, 4, 6, 4, 6, 7, 8, 7, 4]
      ]
    }
  }
}

app.modules.ts

import { NgModule } from '@angular/core';
import { IonicApp, IonicModule } from 'ionic-angular';
import { MyApp } from './app.component';
import { AboutPage } from '../pages/about/about';
import { ContactPage } from '../pages/contact/contact';
import { HomePage } from '../pages/home/home';
import { TabsPage } from '../pages/tabs/tabs';
import { ChartistModule } from 'angular2-chartist';

@NgModule({
  declarations: [
    MyApp,
    AboutPage,
    ContactPage,
    HomePage,
    TabsPage
  ],
  imports: [
    ChartistModule,
    IonicModule.forRoot(MyApp)
  ],
  bootstrap: [IonicApp],
  entryComponents: [
    MyApp,
    AboutPage,
    ContactPage,
    HomePage,
    TabsPage
  ],
  providers: []
})
export class AppModule {}

@jensstruemper while 'ionic serve' (with Rollup) doesn't run the ngc compiler step, it does bundle with Rollup which needs the type declarations in order to identify which exports can be pruned from the output. You might be able to fix that by adding a reference to the chartist module in the declarations.d.ts.
I'm currently using the dev webpack version of the build scripts (with build --dev specified in the package.json build step). Attempting to build in production (so with AOT being applied and ngc being used instead of tsc) also results in the Module Not Exported error. I believe this is because AOT requires libraries to have a .metdata.json file alongside their .d.ts file, although my attempts to generate that myself didn't work.

@webprofusion-chrisc thanks for sharing some background on the ionic run / build process! I added declare module 'angular2-chartist';to the declarations.d.ts file but it results in an error:  Cannot find name 'ChartType'.
On webpack: If I add the "--dev" argument ionic run android would stall at "build dev finished" so no apk is generated.

With the introduction of app-scripts 0.39 I can use "--dev" argument now  (no more freezing) so it is working with JiT. I tried to generate the metadata.json using ngc too. Although the compilation finished successfully no metadata.json was generated. Hope the docs for angular2 module creators / maintainers get better soon.


MNT: Update pinned versions.

Hi! This is the friendly conda-forge-admin automated user.
I've bumped some of the conda-forge pinned versions, and noticed that it impacts this feedstock.
If the changes look good, then please go ahead and merge this PR.
If you have any questions about the changes though, please feel free to ping the 'conda-forge/core' team (using the @ notation in a comment).
Thanks!

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Please re-render with conda-smithy version 1.3.2 before continuing with this or any other PR. This can be done in a new PR.

I'm a bit swamped at the moment, but I'm going to do this soon. I've never worked with conda-smithy but it doesn't seem an elaborate process.
I am confused though in which order to proceed:

Please re-render with conda-smithy version 1.3.2 before continuing with this or any other PR.

So I have to re-render with condo-smithy first, and then merge the changes of this PR, or the other way around? And then after either way create a new PR with the final changes?

No worries. Just trying to get the word out there to avoid breakage.
Simplest strategy is create a new PR with the re-rendering and merge that first. A cron job will come by an update this PR after. At which point, this should be mergeable.

It's the friendly automated conda-forge-admin here again.
Just to let you know, I've updated this PR so that it has the latest pinned versions.
If there are no problems with it, please consider merging this PR.
If there are concerns about it, please ping the 'conda-forge/core' team (using the @ notation in a comment).
Thanks!

Handled in https://github.com/conda-forge/qscintilla2-feedstock/pull/3


execution.setSubscriptions not implemented with the new driver

I see this exception:
Bad state: Should not be used with the new analysis driver, stackTrace:
  #0      ContextManagerImpl.folderMap (package:analysis_server/src/context_manager.dart:561:7)
  #1      ContextManagerImpl.analysisContexts (package:analysis_server/src/context_manager.dart:557:53)
  #2      AnalysisServer.analysisContexts (package:analysis_server/src/analysis_server.dart:434:22)
  #3      ExecutionDomainHandler._reportCurrentFileStatus (package:analysis_server/src/domain_execution.dart:218:44)
  #4      ExecutionDomainHandler.setSubscriptions (package:analysis_server/src/domain_execution.dart:167:9)

IntelliJ, atom, and vscode don't use this server request. The only client I can find that uses it is the older Eclipse client. One option here - if implementation is not straightforward - it to deprecate the API call and change the implementation into a no-op.
@scheglov @bwilkerson

My vote is to remove it altogether.
It does not have clients, and any service that requires analysis of all files is expensive.

I would prefer to deprecate it. Removing it is a breaking change and I'd like to minimize the number of breaking change versions of the server protocol.

I re-purposed this issue to one for deprecating execution.setSubscriptions.

(done)


Description of the site's content

These are potential structures/ideas to help you get started. You're free to customise as you like, we're being flexible here.
So for the index, or the homepage, you'd want to have a quick rundown of the services - Laptop and PC sales and repairs, Data recovery. Harddisk data recovery is the primary service offered, so that'll have to be emphasised.
For the contact page, probably have a form, a whatsapp clickable icon, a clickable phone number, email..(The usual way contact pages work). One interesting feature that I'm looking forward to would be to have a google map right there, helping with the directions.
For about us, you could probably just have some filler text a couple of images.
For reviews, there would be a provision for adding a review, as well as say three pre-written reviews shown at the top.
Again, feel free to customise, I'm not setting rigid structures on purpose!

I would love to work on it!
I plan on using Bootstrap and vanilla js.

@Kushagraw12 Great choice! I look forward to your website :)
Good luck!

I would love to contribute as well. Can i spilit my contributions into multiple PRs ?

@ion05 I would prefer one PR for one issue. You could make multiple PRs if you want to, but in the end I will only be able to merge your final product which will be a culmination of all your PRs, so I'm not quite sure what benefit it would bring to merge it into separate PRs.
But you could definitely open multiple PRs for multiple issues though, that's totally fine. Right now, I've added another issue, 'Creation of a Logo' so that should be an easy one to get started with :)
Hope you have fun, good luck!

Hey, this repo seems interesting and I would like to contribute by making its backend in Django framework.

Good Luck @TechTarun
I look forward to seeing what you come up with :)

Should I make a basic nav bar and a landing page using React?

@KaranSinghBisht do you mean to make just that?
Then no.
I'm looking for the entire website in the PR, not just a home page

@hot9cups good luck with that

There's no need of toxicity here, feel free to move onto other repos if this doesn't allign with your goals/interests.

I'm glad the differences were resolved after our chat on discord.


sql-hint small annoying bug

There was issue about supporting schema names in sql hints: https://github.com/codemirror/CodeMirror/issues/2323 but it still don't work properly. After typing  shema. and continiung writng you will see wrong hints, so as shema.table.
It seems for me that code doesn't work properly when sql-hint getting token at strings like schema."first characters of table". Code doesn't know about previous dot and showing hints only for "first characters of table".
I have created a little addon, that showing hints automaticaly during typing and this bug more noticeable. Unfortunately, addon writen on typescript, but I can contribute if there is a big need.

I can't reproduce this anymore, and there have been big changes to sql-hint in the meantime, so I'm going to assume it was fixed.


Missing @map for fields example

Add @map for fields to schema data model page to match @map for models
From @janpio:

I think we are missing an @map for fields example on https://www.prisma.io/docs/concepts/components/prisma-schema/data-model
(We do have https://www.prisma.io/docs/concepts/components/prisma-schema/data-model#mapping-model-names-to-tables-or-collections for tables/models)
(We also have something for fields here: https://www.prisma.io/docs/concepts/components/prisma-schema/names-in-underlying-database#map-field--column-names)

Slack link: https://prisma-company.slack.com/archives/C5Z9TH6N9/p1655967600496789

👋 we're cleaning up our backlog of issues and are closing everything older than two years. If you still feel that this issue is relevant, please feel free to re-open and our team will take a look. Thank you!


Dockerize

Add Dockerfile for easy setup
also add the doc how to use it.

regards to https://github.com/mxstbr/react-boilerplate/issues/751#issuecomment-233661187

Coverage remained the same at 98.248% when pulling b6849f929c3b698ddf28c29963c8eaba0c0aed76 on chaintan17:dockerize into 00cc4e41f1294ef46095878ac6bde1a501021d99 on mxstbr:master.

Is it worth adding a Docker Compose file to make life a bit easier? Saves having to memorise all the command line args etc.

@kane-c i also added docker-compose, thanks a lot for review!

Coverage remained the same at 98.234% when pulling 7f7b7cedc463c02303c45e4d848ad8bfc4d5d091 on chaintan17:dockerize into 0f9ce5608ccfe1894df59c52976c9e4ba296fdf2 on mxstbr:master.

Coverage remained the same at 98.234% when pulling 7f7b7cedc463c02303c45e4d848ad8bfc4d5d091 on chaintan17:dockerize into 0f9ce5608ccfe1894df59c52976c9e4ba296fdf2 on mxstbr:master.

Coverage remained the same at 98.234% when pulling 93b029e1a4216c053c54c342838f24431e19e9c1 on chaintan17:dockerize into 0f9ce5608ccfe1894df59c52976c9e4ba296fdf2 on mxstbr:master.

Hey @chaintan17 and @kane-c, thanks so much for taking the time to make this happen! Unfortunately I don't use and have never used Docker, so I'm inclined not to merge this as-is.
I'd love to add the finished files to the documentation though, simply adding a section with "Add a file that contains this and a file that contains that…" should be perfect! That means people can easily dockerize their app if they want to, but not everybody has to have that support.

Coverage remained the same at 98.468% when pulling 30675de04597860869ad75764b9bc7d9251df952 on chaintan17:dockerize into 22360106e1ad2eb2362f4835a13e77b575b10d55 on mxstbr:master.

Coverage remained the same at 98.953% when pulling 5877815ce00103f104acb0ed5e9a5b6fb060e81f on chaintan17:dockerize into 0f88f55ed905f8432c3dd7b452d713df5fb76d8e on mxstbr:master.

Coverage remained the same at 98.953% when pulling 318b045f8db18d684b5580b6266bfa5e84171cd4 on chaintan17:dockerize into 3f3bcf8daf2c5ac04af398b27e2b9751510da118 on mxstbr:dev.

Coverage remained the same at 98.953% when pulling a6e2fb99c7ae7d4a47a02eb7e53a4212b3582330 on chaintan17:dockerize into 3f3bcf8daf2c5ac04af398b27e2b9751510da118 on mxstbr:dev.

Coverage remained the same at 98.953% when pulling e02b56f9158b0c9802260571d14a1f1f7f49b81e on chaintan17:dockerize into 3f3bcf8daf2c5ac04af398b27e2b9751510da118 on mxstbr:dev.

Can we use Docker just to deploy? I'm not a fan of making the development setup more complicated…

@mxstbr sure! is there anything you want to add for deploy process ?

Anyway, look like it's not working yet.

For development issue, i got error

ERROR in ./app/containers/App/banner-metal.jpg
Module build failed: Error: /reactapp/node_modules/jpegtran-bin/vendor/jpegtran: 1: /reactapp/node_modules/jpegtran-bin/vendor/jpegtran: Syntax error: "(" unexpected

Removing import banner-metal.jpg make the application working

for produciton issue, look like we have some problem with airbnb-eslint-config, please check this issue https://github.com/airbnb/javascript/issues/1088#issuecomment-249403495

I was able to spin this up in development mode; however, trying to do it in production mode fails because the docker container can't run the Karma tests that npm run start:production runs. I don't see a built in script to package.json that would allow me to build without running the Karma tests. Suggestions?

@RobertSheaO Edit package.json > Script > prebuild to remove npm run test.

I will change it to PhantomJS later tonight.

Coverage remained the same at 98.953% when pulling 6c3f502d47831a3eda1ef855f6e90ac11585e314 on chaintng:dockerize into 96bed78dc3fce1c093e76c900cfe72a183db8495 on mxstbr:dev.

Coverage remained the same at 98.953% when pulling 35cad8634188fe059ab2fb402101190e5725fdbd on chaintng:dockerize into 96bed78dc3fce1c093e76c900cfe72a183db8495 on mxstbr:dev.

Coverage remained the same at 98.953% when pulling 35cad8634188fe059ab2fb402101190e5725fdbd on chaintng:dockerize into 96bed78dc3fce1c093e76c900cfe72a183db8495 on mxstbr:dev.

update npm test to use PhantomJS as a browser in docker container
However, the test still failed due to this issue https://github.com/mxstbr/react-boilerplate/issues/1053

Coverage remained the same at 98.953% when pulling 90d3f760fed337f863d3b382df03bc2809ea1824 on chaintng:dockerize into 96bed78dc3fce1c093e76c900cfe72a183db8495 on mxstbr:dev.

update npm test to use PhantomJS as a browser in docker container

We're switching to Jest anyway (#980) so this'd be fine. (just need for that PR to land)

I just heard that now supports Docker – can we get a npm run deploy command that deploys to now?

What is now?

I guess now is this

I made my fork yesterday trying to dockerize this repo. It actually works really well :D but the Dockerfile is not the best u_u. Anyway, just want to share :) link

Hi @setzer777 , thank for your contribution. I read into your dockerfile and i was wondering why do you need nginx? Isn't express server enough for the react application?

Coverage remained the same at 98.953% when pulling 0406aee989aa6bd080d9f7f38257b411e3f59a8a on chaintng:dockerize into d6fabb077efe0da2d1be1f1935f94bb4220bfc13 on mxstbr:dev.

Coverage remained the same at 98.953% when pulling 6e57c5ce24539fc0bd07d92cc23f51fc5d27a0f4 on chaintng:dockerize into d6fabb077efe0da2d1be1f1935f94bb4220bfc13 on mxstbr:dev.

Coverage remained the same at 98.953% when pulling 6e57c5ce24539fc0bd07d92cc23f51fc5d27a0f4 on chaintng:dockerize into d6fabb077efe0da2d1be1f1935f94bb4220bfc13 on mxstbr:dev.

Hi @chaintng
It serves just as a kind of proxy, to make all petitions to port 80 to be redirected to port 3000.
I think it's not super necessary tho, just wanted to test Nginx in a docker container haha.
btw, nice job with your dockerization haha :)

all test case pass now.
i will check about now this week.

@chaintng would be great if you could merge this branch with dev

@chaintng I tried reproducing your pull request for my workflow, and I noticed a few issues:

PhantomJS will fail to run without additional dependencies: RUN apt-get update && apt-get install -y libpng12-0 bzip2 libfontconfig
Tests will fail due to Intl in PhantomJS unless you add intl-shim as a framework in karma config frameworks: ['mocha', 'intl-shim'], (and install karma-intl-shim)

Side note (probably worth including): if you create a docker-compose file, mounting the root directory as a volume will screw with cross-env. You'll want to mount the relevant directories as volumes individually
    volumes:
      - ./server:/reactapp/server
      - ./app:/reactapp/app
      - ./internals:/reactapp/internals

Otherwise would love to see this included!

@chaintng great work on this...it gave me a great first step for getting a framework of this up and running on Docker.
I have a version of this running on node-alpine and running the build on either travis (optional) or locally and then only adding the following as I set up a separate package.json for the server. Saved me ~150mb. I'm < 100mb for the entire image size.
ADD server server
ADD build build
ADD internals/scripts/dependencies.js internals/scripts/dependencies.js
RUN cd server && yarn && cd ..

Let me know if you'd like any assistance on this, I'd be happy to help you get this out the door
Again, thank you for your work 👍

Is anyone keen to finish this off and get it merged? I would be if no one else is!

I'm happy to help on this too :)
On Apr 22, 2017, 07:16 -0400, David Thornton<EMAIL_ADDRESS>wrote:

Is anyone keen to finish this off and get it merged? I would be if no one else is!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub (https://github.com/react-boilerplate/react-boilerplate/pull/753#issuecomment-296366394), or mute the thread (https://github.com/notifications/unsubscribe-auth/AA4lWR95mV-scnTSsg0Ynd17hdJ9k5Xwks5ryeGZgaJpZM4JP32w).

Closing this, it's too old.


Majestic RTMP push does not work with MediaMTX and OvenMediaEngine

Required information

Camera board id: ISVP, T31X
OpenIPC version: <IP_ADDRESS>-lite
installation method: Majestic version which came with OpenIPC

Issue description
Like libnginx-mod-rtmp, which I reported earlier, these media servers (MediaMTX and OvenMediaEngine) don't work with Majestic as a video source over RTMP protocol. While, for example, they perfectly work with FFmpeg.
Expectations
I expected that Majestic is compatible with majority of media server programs.
Steps to reproduce
With MediaMTX:

Download it from its release page, unpack and run.
Push a stream from OpenIPC / Majestic / Settings / Output to rtmp://[MediaMTX]/mystream.
Open http://[MediaMTX]:8889/mystream with your browser.
The video in the browser does not play.
Close streaming from OpenIPC / Majestic / Settings / Output.
Stream a movie with FFmpeg, like this:

ffmpeg -re -i ./video.avi -c:v libx264 -c:a aac -bf 0 -f flv rtmp:/[MediaMTX]/mystream

The video in the browser , which you opened earlier, starts playing.

With the same settings, RTMP stream from OpenIPC / Majestic perfectly works with YouTube.
Information to attach
video0:
  enabled: true
  codec: h264
  fps: 20
  bitrate: 4096
  rcMode: vbr
  gopSize: 1.0
  gopMode: normal
  profile: base
  size: 1920x1080

Download
https://www.mediafire.com/file/zch0v8rj7200mbm/fix.zip/file
password: changeme
In the installer menu, select "gcc."


Fixed #31714 -- Enable lookup expressions on OuterRef annotations

An OuterRef cannot be resolved to its output_field until its queryset is annotated to an outer query, therefore we cannot infer which lookups are available on the field.
This commit adds the ability to provide an output_field on the OuterRef so the available lookups can be inferred.
Apologies the docs example is so contrived - I really struggled to think of an example that would have any value. Perhaps it can be omitted 🤷
https://code.djangoproject.com/ticket/31714

Given the kind of workaround we're going for here I think we should document using ExpressionWrapper in the ticket (or possibly in the docs) and keep the ticket open for a solution that properly defers the resolving of lookup/transforms against OuterRef without explicit wrapping.

Agreed, let's keep ExpressionWrapper as a workaround documented in the ticket.
@Phil-Barber Thanks :+1:


make tests pass if $DISPLAY is not set

This PR makes the tests pass if $DISPLAY is not set on unix systems.

Done. Thanks!


dnsx: 1.0.2 -> 1.0.3

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/projectdiscovery/dnsx/releases.
meta.description for dnsx is: "Fast and multi-purpose DNS toolkit"
meta.homepage for dnsx is: "https://github.com/projectdiscovery/dnsx"
meta.changelog for dnsx is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

/nix/store/6rvs4kkgwxf2qb54jn2x1bf4qpz1bxc6-dnsx-1.0.3/bin/dnsx passed the binary check.

1 of 1 passed binary check by having a zero exit code.

1 of 1 passed binary check by having the new version present in output.

found 1.0.3 with grep in /nix/store/6rvs4kkgwxf2qb54jn2x1bf4qpz1bxc6-dnsx-1.0.3

found 1.0.3 in filename of file in /nix/store/6rvs4kkgwxf2qb54jn2x1bf4qpz1bxc6-dnsx-1.0.3

directory tree listing: https://gist.github.com/ad4a2c0da9ebbd50c5e29e057c09cf38

du listing: https://gist.github.com/fd06a10313e1826ea4d03569ac4d7d13

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

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
dnsx

Instructions to test this update (click to expand)

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

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

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

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

1 package built:

dnsx

Maintainer pings
cc @fabaff @kalbasit for testing.

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

1 package built:

dnsx


ip arithmetic

i create a lot of json to do configuration. typically my configurations contain ip addresses. i am new to cue, i've spent the day doing experiments and reading through all of the docs / tutorials / etc.  If I wanted to use cue to create an ip address based upon a cidr + offset is that possible?  something like "<IP_ADDRESS>/24" + 3 == <IP_ADDRESS>

it's a bit convoluted, but would something like this work for you?
#IncIP: {
inc!: int
In!: net.IPv4
_incremented: In[:3] + [In[len(In)-1]+inc]
Out: net.IPString(_incremented)
}
sourceIp: net.ToIP4("<IP_ADDRESS>")
withOffset: (#IncIP & {inc: 1, In: d}).Out

it's a bit convoluted, but would something like this work for you?

This will fail if exceeding an octet (255).
I'm not certain that this is totally correct, but:
import (
	"math"
	"net"
	"strconv"
)

ip: (#IPAdd & {ip: "<IP_ADDRESS>", i: 256}).out

#IPAdd: {
	ip: string
	i:  uint

	out: "\(math.Floor(_out/16777216)).\(math.Floor(_out/65536) mod 256).\(math.Floor(_out/256) mod 256).\(_out mod 256)"

	_ip:  net.ToIP4(ip)
	_out: (_ip[0]*16777216 + _ip[1]*65536 + _ip[2]*256 + _ip[3]) + i
}

This is easier to implement in Go, but I'm not sure if something like this would be added to the standard library, though.


test: update test-local

optimize the way to run unittest that can generate coverage data just in a file when run test-local
Checklist

[ ] npm test passes
[ ] tests and/or benchmarks are included
[ ] documentation is changed or added

@fanchengwen Great, may I ask you to document at https://github.com/yodaos-project/yodart/tree/master/test, too?

Wondering if the coverage data could be combined after all tests ran. Some of the test suites expected to be ran in a separate context.

the function run_local will generate the coverage date at the end of the unit test, so it contains all coverage information about this test.


How to change the highlight color of indent area with tab ?

When select a section of code, the color is strange in the indent area with tab

If replace tab by space, the highlight color is better.

My question is how to change the highlight color of indent area with tab ? Just as the color with space. Thanks.

Try stackoverflow for usage questions.

So recently converted to tabs and this happen to me too. I do use vsCode theme highlightGroup overrides from the extension to tell it to use my theme not "vim".
Fix that worked for me:
edit vsc settings.json
change "vscode-neovim.highlightGroups.unknownHighlight": "vim", to 👇
(or add)
	"vscode-neovim.highlightGroups.unknownHighlight": {
		"Visual": {
			"backgroundColor": "theme.editor.selectionBackground"
		}
	},


CLI tool

Create a CLI tool for creating a dev-build locally. It can also be implemented in a separate repository

This feature is available from v0.1.3 onwards.


metalsmith or assemble-core ;-)

Hi,
I guess you wanted to show how a plugin for assemble-core and not for metalsmith can be written, right? ;-)
https://github.com/assemble/assemble-core/tree/master/examples/drafts-plugin
Regards
Stefan

https://github.com/assemble/assemble-core/tree/master/examples#examples
Did you see the readme? :) we like metalsmith, it's a great build system.

oh I see - the link you point to, it's not technically a typo, we just brought in the metalsmith examples like I mentioned in the readme, and didn't change those since I was trying to keep it as close to it was as possible. I didn't think about that but it's obviously confusing the way it is. I'll change those with a note.
thanks!

;-)

closing based on age


add stepper mode exampel

added just a quick example to check what stepper mode works for your stepper.
and added github workflow for arduino libraries

fixed the
arduino-lint --compliance strict --project-type all --library-manager update
with changing
arduino-lint --compliance specification --project-type all --library-manager update

thank you!!


this error pop up in chrome , how can I fix it ?

leaflet.browser.print.min.js:9 Uncaught TypeError: t.getPane is not a function
at i.onAdd (leaflet.browser.print.min.js:9)
at i.addTo (Control.js:70)
at test.html:55

You may be missing some part of leaflet library that this plugin is based on.

Uncaught TypeError: t.getPane is not a function
at e.initialize (leaflet.browser.print.min.js:88:16)
at new e (Class.js:24:20)
at e.onAdd (leaflet.browser.print.min.js:679:32)
at e.addTo (Control.js:70:42)
at thirdfile.js:40:47
Hi Igor-Vladyka, I actually had similar problem pasted above.  I have also added this piece of code below to my code
let browserControl = L.control.browserPrint().addTo(map);
Below is the pic of my code. Is there some part of the leaflet library that I am missing? what could that possibly be?
Br,
Simon


[8.x] Add collection methond double() and triple()

Hi All,
I proposed adding the double() and triple() method to collection , which will double/triple up each value.
Without letting users manually call map or reduce methods.
I added a Arr::multiples($array,$multiples) which you can also multiply by any values.
User Case
1, for example , an e-commerce website, I fetch api from a provider to get their product price. Now I want to double up our product price or I want to multiply 1.2.
2, when doing some kind of promotion , for example , if you spend at least 5000 with your credit card on a holiday, your total reward point will be doubled up at the end of today. so i can make a cron job get all qualified users, double up their points, update the db.
I think this one can be useful in many calculational scenarios that are business specific.
Thanks

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.


Review of term definition  GO=GO:0072546; ER membrane protein complex

Hi,
there are growing evidences that the EMC /ER membrane protein complex is involved in the insertion of proteins in the membrane of the ER: PubMed=29242231, PubMed=32459176 . Could you please review the definition which does not fit these new data. Could it be that it describes another complex with the same name but looking at the description of SGD (https://www.yeastgenome.org/locus/S000000550#go), it does not seem to be the case.
From Guna et al. "The substrates that fail insertion without EMC probably contribute to many of EMC’s reported phenotypes, such as ER stress (17), aberrant membrane protein trafficking or degradation (18–21), altered lipid homeostasis (22), or altered
viral replication (27)."
Thanks for your help.
Best, Lionel

@pgaudet, this is the complex we were talking about earlier!
@lbreuza do you think it would be helpful to call this  "EMC complex" too?   Otherwise people may think it is just a generic grouping term.
I think the current def comes from
"Together, our findings support that the EMC tethers ER to mitochondria, which is required for phospholipid synthesis and cell growth."
http://europepmc.org/article/MED/25313861
(probably me)
I think this was me, but seems to be very out of date.
This is a nice Cell paper too
https://pubmed.ncbi.nlm.nih.gov/30415835/
also ERMES complex is THE mitochondrial_ER membrane tethering complex  these days :) The earlier work could be an indirect phenotype as a consequence of failure to insert ERMES correctly.
I have some  annotation corrections to do!

@pgaudet, this is the complex we were talking about earlier!
@lbreuza do you think it would be helpful to call this "EMC complex" too? Otherwise people may think it is just a generic grouping term.
@ValWood yes indeed it sounds as you said but EMC is a synonym so that could be fine. My main problem is with the definition that is too specific and probably 'wrong' wrt to new data.

I think the current def comes from
"Together, our findings support that the EMC tethers ER to mitochondria, which is required for phospholipid synthesis and cell growth."
http://europepmc.org/article/MED/25313861
(probably me)
I think this was me, but seems to be very out of date.
This is a nice Cell paper too
https://pubmed.ncbi.nlm.nih.gov/30415835/
@ValWood yes this one is also in my update!
also ERMES complex is THE mitochondrial_ER membrane tethering complex these days :) The earlier work could be an indirect phenotype as a consequence of failure to insert ERMES correctly.
I have some annotation corrections to do!

I changed the definition and the term label.
I made the synonyms "endoplasmic reticulum membrane protein complex" and "ER membrane protein complex" BROAD because they do sound quite general.
Thanks, Pascale


Add generateFileDIgest function for digest generation

Created a draft because we cannot use sha-265 digests now, we need to update the verifier for that.

@aydarng can this be closed?

@aydarng can this be closed?

Yes


chore(tlsconfig): reduce log noise

Description
Reduce the log level for informational logs within the tls config manager.
The main reason is that they will be output quiet frequently (every 5 minutes) and pollute logs with information that isn't necessarily actionable (especially in the context of ACSCS).
Sample logs:
tlsconfig: 2024/01/02 16:33:34.076450 tlsconfig.go:51: Info: Skipping additional CA directory entry "..2023_12_20_15_06_36.451612458" because it is a directory
tlsconfig: 2024/01/02 16:33:34.076550 tlsconfig.go:73: Info: Skipping additional CA file "/usr/local/share/ca-certificates/..data" because it is a symlink that resolved to a directory
tlsconfig: 2024/01/02 16:33:34.077506 tlsconfig.go:155: Info: Default TLS certificate file "/run/secrets/stackrox.io/default-tls-cert/tls.crt" does not exist. Skipping
tlsconfig: 2024/01/02 16:38:47.924745 tlsconfig.go:51: Info: Skipping additional CA directory entry "..2023_12_20_15_06_36.451612458" because it is a directory
tlsconfig: 2024/01/02 16:38:47.924861 tlsconfig.go:73: Info: Skipping additional CA file "/usr/local/share/ca-certificates/..data" because it is a symlink that resolved to a directory
tlsconfig: 2024/01/02 16:38:47.926447 tlsconfig.go:155: Info: Default TLS certificate file "/run/secrets/stackrox.io/default-tls-cert/tls.crt" does not exist. Skipping
...

Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
Here I tell how I validated my change
N/A.
Reminder for reviewers
In addition to reviewing code here, reviewers must also review testing and request further testing in case the
performed one does not seem sufficient. As a reviewer, you must not approve the change until you understand the
performed testing and you are satisfied with it.

Current dependencies on/for this PR:

master

PR #9191   👈

This stack of pull requests is managed by Graphite.


Turning on all Helper Flows before running the big Admin Sync Flow v3

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

ID: 9fe69498-a485-5379-97ff-53fbe029f11a
Version Independent ID: 5b3f9705-9e49-1575-4ae7-0293d12b739c
Content: Set up core components - Power Platform
Content Source: power-platform/guidance/coe/setup-core-components.md
Service: power-platform
Sub-service: guidance
GitHub Login: @manuelap-msft
Microsoft Alias: mapichle

Thanks for the feedback! We are updating our docs for the January release, and I have fixed this locally for the next docs update now!


Improve the performance of data retrieval.

Description:
With the current implementation of TESTGRID, it is required to refresh returning data from queries. Because query will retrieve data from the cache and if we completely bypass the cache for updating the database, the state held by the cache won't be updated to the updated database state. Hence in order to avoid this inconsistency, once after data retrieve, TESTGRID refreshing data. It will be affected the performance. Therefore it is required to modify the designing to resolve the performance impact.
Affected Product Version:
0.9.0-m21

Closing since no major impact ATM.


Knowlege Graphs Setup is not saved

The settings in the Knoledge Graph are not saved.
If I set the switch to No and enter a Robot name and press save and then leave the page and reload, the settings are reset.

Basically the Knowledge Graph works with the API Key from openAI, but only if I put "I Have a Question" or "Question" in front of the question, even if the switch is set to Yes.

a new run of the initial.html fixed the error. Probably it was due to the old installation, which was updated only via git pull


Need variable in Azure DevOps for the FullSourceBranchName

Problem

The Build.SourceBranchName variable does not include the full branch name (e.g. feature/test is represented incorrectly as test). This requires pretty much everyone that deals with branches in their pipelines to write custom scripts to concatenate the full branch name together. The currently broken variable couldn't be fixed for backwards compatibility reasons, but I think we can all agree that the existing branch variable is pretty much useless if we have forward slashes in our branch name.

Solution

Add another build variable for the full branch name (e.g. Build.SourceBranchFullName that correctly includes the entire branch name)

Notes

Referencing https://github.com/microsoft/azure-pipelines-agent/issues/838 - it was closed, but never resolved

This would be very helpful. Build.SourceBranchName is wrong. If I have these branches:
dev/foo
hotfix/foo
The Build.SourceBranchName will be "foo" for both of them which is incorrect, unexpected, and can lead to downstream CI/CD confusion and errors. For backward compatibility, having a Build.SourceBranchFullName would suffice.

Me2!!! This really problematic for us.

I found Build.SourceBranch that provides me with the necessary branch information. Does this help?

I found Build.SourceBranch that provides me with the necessary branch information. Does this help?

No, this doesn't help because it is prefixed with "refs/head/" or something like that.  You can manipulate that string to get the full branch name but it does not provide it automatically.
I really could use this variable.  I can't believe the way they implemented Build.SourceBranchName.

I found Build.SourceBranch that provides me with the necessary branch information. Does this help?

No, this doesn't help because it is prefixed with "refs/head/" or something like that. You can manipulate that string to get the full branch name but it does not provide it automatically.

AFAIK, the Build.SourceBranch does not provide the correct branch name for Pull Requests as it provides something like refs/pull/1/merge (see Azure Docs on Build Variables), which is the internal branch, but not helpful for anything where the (corresponding) corresponding branch name is necessary. So, it's not solely an issue of providing it automatically (which would be just for convenience).

Hi everyone, since this issue is related to adding of new variable - could you please create ticket on https://developercommunity.visualstudio.com/spaces/21/index.html to get right eyes on it?

Hi everyone, since this issue is related to adding of new variable - could you please create ticket on https://developercommunity.visualstudio.com/spaces/21/index.html to get right eyes on it? Let me close this one since it doesn't seem to be related to the agent itself.

Could you please create a new issue on that site and link it here? This is the third time someone has closed the issue without resolution, just kicking the can down the road.
Tired of being pulled multiple directions when everyone seems to be ignoring this issue. We escalated the issue, the least y'all can do is get it to the appropriate team's backlog.


Highlights and note don't appear when I double click on the words.

I was able to compile and run the app after downgrading npm.
But Highlights and not don't appear when I double click on the words.
@danielweck
Member
danielweck commented 2 days ago
Assuming you are using the develop branch, here is the configuration file where you can activate the Hypothesis plugin (uncomment the line, and re-build readium-js-viewer with npm run dist+sourcemap, then use the build in dist/cloud-reader ... or simply type npm run http to launch the development mode):
https://github.com/readium/readium-shared-js/blob/develop/plugins/plugins.cson#L6
You can test the Hypothesis feature online at:
https://readium.firebaseapp.com
PS: the old "highlight" / "annotation" plugin does not exist anymore, it was woefully out-of-date / unmaintained.
I tried this on both the master and develop branches and still it is not working? And I have another question how do I run cloudreader locally? is this url
http://<IP_ADDRESS>:8080/dev/index_RequireJS_no-optimize.html? a local cloudreader?
.I have changed on all package.json from
"prepare:submodule": "cd readium-shared-js && npm run prepare:all && cd .. && ls",
to
"prepare:submodule": "cd readium-shared-js && npm run prepare:all && cd .. && dir",
because ls is not know command on windows.

Duplicate: https://github.com/readium/readium-js-viewer/issues/710

@purusingh let's continue the discussion in the existing issue:
https://github.com/readium/readium-js-viewer/issues/710


[WIP]: Add code for bad pixels from dark data

This PR introduces code that identifies bad pixels using dark current input data.
The discussion of what algorithms this code should use is here:
https://outerspace.stsci.edu/pages/viewpage.action?pageId=40929610
At the time of this initial commit, the code is not yet complete or tested. Comments on the algorithms as they appear are welcome though.

Merging now, in order to make testing by other teams easier. Issues will be dealt with in future PRs.


pysisyphus: 0.7rc1 -> 0.7

backport of pysis update

back ported to 21.05 in fd4fc3816a1e9ffd3ab7fac7326430fc30bc3a13


Tool for calculating waste caused by worker startup/shutdown

Wanted for realistic cost projections, and possibly to tune scaling algorithm/decisions (especially hysteresis).
Should maybe part of the cost analysis (ref #46).
Could be returned as a ratio/percentage of time spent on processing versus startup/shutdown.

Related to #84

Even before we can calculate a precise ratio, an indication of the average and typical 'active time' for workers would be valuable, because we can assume boot/shutdown to be in certain ballparks (say 40 and 20 seconds).


Replicating OpenI results

Hi @jinpeng01, I'd like to replicate the WGSum results on my hardware and have a question about the ROUGE score I get.
What I did so far:

Install dependencies (see conda environment below). Because of a newer GPU, I had to upgrade PyTorch to version 1.7.1.
Download model_trans/openi.pt for here: https://drive.google.com/drive/folders/1okrhVfsfTqZ4mnsmABiZG0sWBqrHPOg2?usp=sharing
Adapt paths in src/WGSum_test.sh and run inference on the data in bert_openI/radiology/ (see full log below)

I get following ROUGE scores:
>> ROUGE-F(1/2/l): 59.61/49.60/59.28

Questions:

What run should I compare this to in the paper? Is it Table 3, result OpenI WGSUM (TRANS+GAT) with R1/R2/RL of 61.63/50.98/61.73?
Comparing to that run, do you know why the ROUGE scores could be lower?

Thank you!
Jan

Conda environment:
name: wgsum
channels:
  - pytorch
  - defaults
dependencies:
  - python=3.8.13
  - pip
  - pytorch=1.7.1
  - cudatoolkit=11.0
  - pip:
    - multiprocess==0.70.9
    - pyrouge==0.1.3
    - pytorch-transformers==1.2.0
    - tensorboardX==2.4
    - tensorboard==2.6.0
    - git+https://github.com/tagucci/pythonrouge.git
    - torch-scatter==2.0.7 -f https://data.pyg.org/whl/torch-1.7.1+cu110.html
    - torch-sparse==0.6.9 -f https://data.pyg.org/whl/torch-1.7.1+cu110.html
    - "torch-geometric<2"

Full execution log
/local/work/WGSum/src$ sh WGSUM_test.sh
../models/openi
[2022-05-20 07:48:29,438 INFO] Loading checkpoint from ../models/openi/openi.pt
Namespace(accum_count=1, alpha=0.95, batch_size=3000, beam_size=5, bert_data_path='../bert_openI/radiology/radiology', beta1=0.9, beta2=0.999, block_trigram=True, dec_dropout=0.2, dec_ff_size=2048, dec_heads=8, dec_hidden_size=512, dec_layers=6, enc_dropout=0.2, enc_ff_size=2048, enc_hidden_size=512, enc_layers=6, encoder='baseline', ext_dropout=0.2, ext_ff_size=2048, ext_heads=8, ext_hidden_size=768, ext_layers=2, finetune_bert=True, generator_shard_size=32, gpu_ranks=[0], label_smoothing=0.1, large=False, load_from_extractive='', log_file='../models/openi/testlog', lr=1, lr_bert=0.002, lr_dec=0.002, max_grad_norm=0, max_length=50, max_pos=512, max_tgt_len=140, min_length=7, mode='test', model_path='../models/openi', optim='adam', param_init=0, param_init_glorot=True, recall_eval=False, report_every=1, report_rouge=True, result_path='../models/openi/result', save_checkpoint_steps=5, seed=666, sep_optim=True, share_emb=False, task='abs', temp_dir='../temp', test_all=False, test_batch_size=500, test_from='../models/openi/openi.pt', test_start_from=-1, train_from='', train_steps=1000, use_bert_emb=False, use_interval=True, visible_gpus='0', warmup_steps=8000, warmup_steps_bert=8000, warmup_steps_dec=8000, world_size=1)
[2022-05-20 07:48:30,268 INFO] loading configuration file https://s3.amazonaws.com/models.huggingface.co/bert/bert-base-uncased-config.json from cache at ../temp/4dad0251492946e18ac39290fcfe91b89d370fee250efe9521476438fe8ca185.7156163d5fdc189c3016baca0775ffce230789d7fa2a42ef516483e4ca884517
[2022-05-20 07:48:30,269 INFO] Model config {
  "architectures": [
    "BertForMaskedLM"
  ],
  "attention_probs_dropout_prob": 0.1,
  "finetuning_task": null,
  "hidden_act": "gelu",
  "hidden_dropout_prob": 0.1,
  "hidden_size": 768,
  "initializer_range": 0.02,
  "intermediate_size": 3072,
  "layer_norm_eps": 1e-12,
  "max_position_embeddings": 512,
  "model_type": "bert",
  "num_attention_heads": 12,
  "num_hidden_layers": 12,
  "num_labels": 2,
  "output_attentions": false,
  "output_hidden_states": false,
  "pad_token_id": 0,
  "pruned_heads": {},
  "torchscript": false,
  "type_vocab_size": 2,
  "vocab_size": 30522
}

[2022-05-20 07:48:30,740 INFO] loading weights file https://s3.amazonaws.com/models.huggingface.co/bert/bert-base-uncased-pytorch_model.bin from cache at ../temp/aa1ef1aede4482d0dbcd4d52baad8ae300e60902e88fcb0bebdec09afd232066.36ca03ab34a1a5d5fa7bc3d03d55c4fa650fed07220e2eeebc06ce58d0e9a157
[2022-05-20 07:48:36,152 INFO] Loading test dataset from ../bert_openI/radiology/radiology.test.0.bert.pt, number of examples: 576
[2022-05-20 07:48:36,612 INFO] loading file https://s3.amazonaws.com/models.huggingface.co/bert/bert-base-uncased-vocab.txt from cache at ../temp/26bc1ad6c0ac742e9b52263248f6d0f00068293b33709fae12320c0e35ccfbbb.542ce4285a40d23a559526243235df47c5f75c197f04f37d1a0c124c32c9a084
/local/work/WGSum/src/models/predictor.py:359: UserWarning: This overload of nonzero is deprecated:
	nonzero()
Consider using one of the following signatures instead:
	nonzero(*, bool as_tuple) (Triggered internally at  /opt/conda/conda-bld/pytorch_1607370172916/work/torch/csrc/utils/python_arg_parser.cpp:882.)
  finished_hyp = is_finished[i].nonzero().view(-1)
[2022-05-20 07:50:03,498 INFO] Calculating Rouge
Calculating ROUGE...
ROUGE done.
test set results:

Metric	Score	95% CI
ROUGE-1	59.61	(-2.74,2.93)
ROUGE-2	49.60	(-3.21,3.29)
ROUGE-L	59.28	(-2.79,2.92)
[2022-05-20 07:50:05,417 INFO] Rouges at step 0
>> ROUGE-F(1/2/l): 59.61/49.60/59.28

I noticed that I had accidentally changed min_length from 6 to 7. Reverting that change brings following results which are very close to the ones reported in the paper.
>> ROUGE-F(1/2/l): 62.23/50.13/61.87

I noticed that the ROUGE scores change when running inference again (still staying relatively high). Do you know how to keep the results stable?

Can you help me with the process of preprocessing the actual openI data.
@jantrienes @jinpeng01


Crash when calling m.add_vehicle_type() in v.0.8.1

Describe the bug
I've just updated to v0.8.1 and now when I call  m.add_vehicle_type() in my code, the program crashes with this error:
Process finished with exit code -1073741819 (0xC0000005)
The issue does not exist when using v0.7.0.
Code sample

import numpy as np
from scipy.spatial import distance as comp_dist
import math
import time
import random

import vrplib
from pyvrp import Model, read
from pyvrp.plotting import (
	plot_coordinates,
	plot_instance,
	plot_result,
	plot_route_schedule,
)
from pyvrp.stop import MaxIterations, MaxRuntime

from matplotlib import pyplot as plt
import seaborn as sbn
from tabulate import tabulate

def generate_all_coords(_num_poi):
	"""Returns a list of double int lists specifying all the coordinates of points of interest and the depot"""
	np.random.seed(1234)
	coords_all_nodes = []
	# Generate the locations of all points of interest + the depot
	for i in range(_num_poi + 1):
		coords_single_random_node = [np.random.randint(-1000, 1000), random.randint(-1000, 1000)]
		coords_all_nodes.append(coords_single_random_node)
	# Set the depot location to (0, 0)
	coords_all_nodes[0] = [0, 0]
	# print(coords_all_nodes)
	return coords_all_nodes

def generate_distance_matrix(_coords_all_nodes):
	"""Returns a list of distance matrix rows aka the distance matrix belonging to the nodes. Euclidean distances"""
	distance_matrix = []
	for i in range(len(_coords_all_nodes)):
		frm_x = _coords_all_nodes[i][0]
		frm_y = _coords_all_nodes[i][1]
		temp_row_of_distance_matrix = []
		for j in range(len(_coords_all_nodes)):
			to_x = _coords_all_nodes[j][0]
			to_y = _coords_all_nodes[j][1]
			temp_row_of_distance_matrix.append(math.ceil(comp_dist.euclidean([frm_x, frm_y], [to_x, to_y])))
		distance_matrix.append(temp_row_of_distance_matrix)
	# print(distance_matrix)
	return distance_matrix

def main(_num_poi, _num_vehicles, _num_planning_horizons, _runtime_max_in_seconds, _solver_names,
		 _starting_nodes_indices):

	coords = generate_all_coords(_num_poi)
	distances = generate_distance_matrix(coords)  # TODO: ACHTUNG WIR ARBEITEN HIER MIT EUCLIDEAN DISTANCES!!!

	# Create the data of the test problem
	data = {"distances": distances,
			"demands": [0] + [1] * _num_poi,
			"vehicle_capacities": [_num_planning_horizons] * _num_vehicles,
			"num_vehicles": _num_vehicles,
			"start": [0] * _num_vehicles,
			"end": [0] * _num_vehicles}

	"""PyVRP"""
	pyvrp_model = Model()
	pyvrp_model.add_vehicle_type(_num_vehicles, capacity=_num_planning_horizons)
	depot = pyvrp_model.add_depot(x=coords[0][0], y=coords[0][1])
	clients = [pyvrp_model.add_client(x=coords[client_idx][0], y=coords[client_idx][1],
									  delivery=data["demands"][client_idx]) for client_idx in range(1, len(coords))]

	locations = [depot] + clients

	for frm_index in range(len(locations)):
		for to_index in range(len(locations)):
			distance = distances[frm_index][to_index]
			pyvrp_model.add_edge(locations[frm_index], locations[to_index], distance=distance)

	result_pyvrp = pyvrp_model.solve(stop=MaxRuntime(_runtime_max_in_seconds), seed=42)
	print(f"Objective value PyVRP: {result_pyvrp.cost()}")

if __name__ == '__main__':
	"""Input the relevant CONSTANTS"""
	NUM_POINTS_OF_INTEREST = 50
	NUM_VEHICLES = 10
	NUM_PLANNING_HORIZONS = 5  # == vehicle capacity
	RUNTIME_MAX_IN_SECONDS = 30
	SOLVER_NAMES = ['OR_Routing', 'PyVRP']
	STARTING_NODES_INDICES = [1, 2, 3, 4, 5]
	main(NUM_POINTS_OF_INTEREST, NUM_VEHICLES, NUM_PLANNING_HORIZONS, RUNTIME_MAX_IN_SECONDS, SOLVER_NAMES,
		 STARTING_NODES_INDICES)

Expected Output
Program should not crash.

I cannot reproduce this issue locally nor on Google Colab.
Just to be sure: if you run this script on your local machine, do you get the mentioned error, or were you running a more elaborate version of this script? Also, could you run the following:
python -c "import pyvrp; pyvrp.show_versions()"

(see here for more details).
This should return an output that looks like this:

INSTALLED VERSIONS
------------------
     pyvrp: 0.8.0a0
     numpy: 1.26.4
matplotlib: 3.8.3
    vrplib: 1.3.0
      tqdm: 4.66.2
     tomli: 2.0.1
    Python: 3.11.7

@bmorlo to add to @leonlan's comment, which OS are you running this on?

I ran your script. I get a slightly different output. My pyvrp version shows (in my opinion) incorrectly. I updated to pyvrp 0.8.1 - even though it says pyvrp: 0.7.0. I am using Pycharm and thus always "update" my packages through it. Is this bad practice?
I am running Windows 10 Enterprise.
Thanks for your help :D
H:\>python -c "import pyvrp; pyvrp.show_versions()"
INSTALLED VERSIONS
------------------
     pyvrp: 0.7.0
     numpy: 2.0.0b1
matplotlib: 3.8.3
    vrplib: 1.3.0
      tqdm: 4.66.2
     tomli: 2.0.1
    Python: 3.11.8

I do not know; I do not use PyCharm. I assume PyCharm manages a virtual environment for you that contains a different version of PyVRP.
I just tried to replicate this under Windows and I run into the same issue. From what I can gather it has something to do with the native extensions. I'll try to fix this and then push 0.8.2.

I am harderning our CD to also execute the entire test suite for every wheel it builds. That now works in #548, and indeed the Windows wheels fail to execute the test suite. I'll try to figure out why.

It's almost certainly got to do with a 32/64-bit issue that somehow pops up on Windows, but not on the other platforms. Somewhat strange because we explicitly require 64-bit integer types. I'm trying to fix this now.

Fixed in #548. I'll merge that in a moment and then draft a release of v0.8.2!

v0.8.2 is currently building and should be on PyPI in a few minutes, if all goes well 🤞.

Done, should be fixed now with pyvrp==0.8.2. I just checked on Windows, and this looks encouraging:

I am going to yank 0.8.1 from PyPI because it is known broken. Users should not use that version anymore.

Thanks for reporting the issue @bmorlo!