RuntimeError: CUDA out of memory. Tried to allocate 464.00 MiB (GPU 1; 10.92 GiB total capacity; 8.43 GiB already allocated; 323.50 MiB free; 1.56 GiB cached)

Traceback (most recent call last):
File "train_panoptic.py", line 628, in
main(parser.parse_args())
File "train_panoptic.py", line 589, in main
global_step=global_step, loss_weights=config["optimizer"].getstruct("loss_weights"))
File "train_panoptic.py", line 296, in train
losses, _, conf = model(**batch, do_loss=True, do_prediction=False)
File "/opt/conda/lib/python3.6/site-packages/torch/nn/modules/module.py", line 493, in call
result = self.forward(*input, **kwargs)
File "/opt/conda/lib/python3.6/site-packages/torch/nn/parallel/distributed.py", line 376, in forward
output = self.module(*inputs[0], **kwargs[0])
File "/opt/conda/lib/python3.6/site-packages/torch/nn/modules/module.py", line 493, in call
result = self.forward(*input, **kwargs)
File "/opt/conda/lib/python3.6/site-packages/seamseg-0.1.dev31+g18e28cc-py3.6-linux-x86_64.egg/seamseg/models/panoptic.py", line 76, in forward
self.rpn_head, x, bbx, iscrowd, valid_size, training=self.training, do_inference=True)
File "/opt/conda/lib/python3.6/site-packages/seamseg-0.1.dev31+g18e28cc-py3.6-linux-x86_64.egg/seamseg/algos/fpn.py", line 84, in training
obj_logits, bbx_logits, h, w = self._get_logits(head, x)
File "/opt/conda/lib/python3.6/site-packages/seamseg-0.1.dev31+g18e28cc-py3.6-linux-x86_64.egg/seamseg/algos/fpn.py", line 57, in _get_logits
obj_logits_i, bbx_logits_i = head(x_i)
File "/opt/conda/lib/python3.6/site-packages/torch/nn/modules/module.py", line 493, in call
result = self.forward(*input, **kwargs)
File "/opt/conda/lib/python3.6/site-packages/seamseg-0.1.dev31+g18e28cc-py3.6-linux-x86_64.egg/seamseg/modules/heads/rpn.py", line 62, in forward
x = self.conv1(x)
File "/opt/conda/lib/python3.6/site-packages/torch/nn/modules/module.py", line 493, in call
result = self.forward(*input, **kwargs)
File "/opt/conda/lib/python3.6/site-packages/torch/nn/modules/conv.py", line 338, in forward
self.padding, self.dilation, self.groups)
RuntimeError: CUDA out of memory. Tried to allocate 464.00 MiB (GPU 1; 10.92 GiB total capacity; 8.43 GiB already allocated; 323.50 MiB free; 1.56 GiB cached)
^CTraceback (most recent call last):
File "/opt/conda/lib/python3.6/runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "/opt/conda/lib/python3.6/runpy.py", line 85, in _run_code
exec(code, run_globals)
File "/opt/conda/lib/python3.6/site-packages/torch/distributed/launch.py", line 235, in
main()
File "/opt/conda/lib/python3.6/site-packages/torch/distributed/launch.py", line 228, in main
process.wait()
File "/opt/conda/lib/python3.6/subprocess.py", line 1477, in wait
(pid, sts) = self._try_wait(0)
File "/opt/conda/lib/python3.6/subprocess.py", line 1424, in _try_wait
(pid, sts) = os.waitpid(self.pid, wait_flags)
when I run "python -m torch.distributed.launch --nproc_per_node=2 train_panoptic.py --log_dir LOG_DIR configurations/cityscapes_r50.ini  ../dataset_root/" on two 2080ti GPU of 12G memory and set the batch_size as 1 for both training and validation, it always show CUDA out of memory. If the code only run on the GPU of 16G memory.

The original settings from our paper, which the given cityscapes_r50.ini reproduces, are calibrated for a setup of 8x Nvidia V100 GPUs with 32GB of memory each. To be able to train on 12GB per card you need to reduce the training image size by properly setting the shortest_size and longest_max_size parameters to lower values.


Fix a issue in REPL related to user-defined sorts and lemmas

…red in SMT

Reviewer checklist

[ ] Test coverage: stack test --coverage
[ ] Public API documentation: stack haddock

@ttuegel @virgil-serbanuta I made a fix to REPL so that user-defined sorts and lemmas are declared in the SMT queries. However, i'm not sure how to write a test.

@yzhang90
Yes, the scripts directory from the link @virgil-serbanuta added contains repl integration tests. You can find more about repl scripts here https://github.com/kframework/kore/wiki/Kore-REPL#repl-scripts
We check the repl output after running each script against its corresponding .golden output file. The repl outputs a new conjuction of claims which indicate the state of the proof after executing the script. If the proof was finished the output is #True. You can look at the tiny semantics directory for how to add this sort of integration test. Each spec should have its own directory and a scripts directory which tests proofs of that spec.
So, if you think that you can test this by checking the final repl output (the set of claims) after running some repl commands on a spec and semantics of your choice you can add this sort of test. Otherwise, we also have some unit tests for the parser and the interpreter in https://github.com/kframework/kore/tree/master/kore/test/Test/Kore/Repl, if you think this sort of test would be more suitable in this case.
I hope this helps.

@virgil-serbanuta I added a TODO in the code. I'm busy with the sum-to-n proof and we can get back to the test later when I have time.


add _lastDraggedItem and _lastDraggedItemType to DragDropStore

example for #38.

I'm closing this, since it's a workaround for the real problem we'll be solving with #54 and later #38 itself.


fix(route_handler): fix unnecessary backward search

Signed-off-by: Fumiya Watanabe<EMAIL_ADDRESS>Related Issue(required)

Description(required)
In RouteHandler::getMainLanelets(), the default value of backward_distance, which is the 2nd argument of getLaneletSequence(), is std::numeric_limits<double>::max().
However, in the specific situation, searching backward lanelets becomes almost infinite loop.
To avoid this, in this PR, I fixed the code to give the bounded backward distance.

Review Procedure(required)
Before applying this change, it may enter the infinite loop by throwing the goal pose near the ego car many times.
Please check that it does not happen this error by applying this change.

Related PR(optional)

Pre-Review Checklist for the PR Author
PR Author should check the checkboxes below when creating the PR.

[x] Read commit-guidelines
[x] Assign PR to reviewer

If you are adding new package following items are required:

[ ] Documentation with description of the package is available
[ ] A sample launch file and parameter file are available if the package contains executable nodes

Checklist for the PR Reviewer
Reviewers should check the checkboxes below before approval.

[ ] Commits are properly organized and messages are according to the guideline
[ ] PR title describes the changes

Post-Review Checklist for the PR Author
PR Author should check the checkboxes below before merging.

[ ] All open points are addressed and tracked via issues or tickets

CI Checks

Build and test for PR / build-and-test-pr: Required to pass before the merge.
Build and test for PR / clang-tidy-pr: NOT required to pass before the merge. It is up to the reviewer(s). Found false positives? See the [guidelines][clang-tidy-guidelines].
Check spelling: NOT required to pass before the merge. It is up to the reviewer(s). See here if you want to add some words to the spell check dictionary.

@rej55 Do you have any tips to reproduce the bug?

I confirmed that sometimes route_handler stops to output a route when I sent goal pose repeatedly in the following situations

I confirmed that the above bug does not occur after merging PR.

However, sometimes the route that does not include the lanelet near the ego vehicle is output.
(In this case, id: 9298 & 9235)

Does this happen in your environment?

This weird bug happens just by changing the goal pose without changing the position of the vehicle.

I confimed that the bug was not occued.


docs: add new landing page and design

:tada: This PR is included in version 2.1.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Make WMAgent job wrapper slightly more verbose; fix one error message

Fixes #10259
Status
not-tested
Description
Be explicit on the worker node setup style that the job is running on (LCG, OSG, CVMFS) and create a record in the log for that.
In addition to that, fix the error message in case a python installation area cannot be located.
Is it backward compatible (if not, which system it affects?)
yes
Related PRs
none
External dependencies / deployment changes
none

I haven't tested it yet. But could you please confirm whether it addresses all the points that you mentioned in your issue, Stefano. @belforte

Tested on the agent as well. Merging


Using component props values inside noscript template crashes in Firefox

Describe the bug
When creating noscript tag, if child elements use component props values, then it crashes. In this case, the noscript element has an anchor tag where it's aria attribute is being set by component prop label and it results in error node.setAttribute is not a function.
const Comp = (props: {label: string}) => {
  return (

          hi

      nooooo

  );
};

function App() {
  return (
    <>
      <Comp label="hi"/>
    </>
  );
}

This maybe a silly useless issue to report, since this occurs in client side rendering, which doesn't make sense since creating a noscript jsx template will never be created and fired in a disabled javascript page. I'm reporting due to browser inconsistency where Chrome and Safari renders the noscript tag with children while Firefox fails. However I haven't tested this in SolidStart.
Your Example Website or App
https://playground.solidjs.com/anonymous/3876e2a9-a538-45ec-afc0-f4ab745bebd7
Steps to Reproduce the Bug or Issue

Create component with props and have noscript element with child elements.
Make noscript child element attribute use a component prop.

Expected behavior
It should render the noscript element in the DOM, with the aria-label set to the value of the component label prop.

      hi

  nooooo

Screenshots or Videos
No response
Platform

OS: macOS
Browser: Firefox
Version: 119.0

Additional context
No response

This is probably relared https://github.com/solidjs/solid/issues/1601#issuecomment-1463314273


iprintln!() fails to print after printing certain amount of data

Hello :crab:, when running the program below that repeatedly calls iprintln!(), it always stops printing at the same point in code, not finishing the printing tasks. It should print "Hello World ~" 77 times in a loop, but the program always stops printing in the middle of 26th invocation of iprintln!().
#[entry]
fn main() -> ! {
    if let (Some(dp), Some(mut cp)) = (
        stm32::Peripherals::take(),
        cortex_m::peripheral::Peripherals::take(),
    ) {
        // Set up the system clock. We want to run at 168MHz for this one.
        let rcc = dp.RCC.constrain();
        let clocks = rcc.cfgr.sysclk(168.mhz()).freeze();
        let mut itm = cp.ITM;
        for i in 0..77 {
            iprintln!(&mut itm.stim[0], "Hello World for the {}-th time", i);
        }

        loop {
            continue;
        }
    }

    // Failed to access peripherals..
    panic!();
}

logs from ITM are stored locally in itm.txt, and the log is always the same as below
(no matter how many times I try running).
Hello World for the 0-th time
Hello World for the 1-th time
Hello World for the 2-th time
Hello World for the 3-th time
Hello World for the 4-th time
Hello World for the 5-th time
Hello World for the 6-th time
Hello World for the 7-th time
Hello World for the 8-th time
Hello World for the 9-th time
Hello World for the 10-th time
Hello World for the 11-th time
Hello World for the 12-th time
Hello World for the 13-th time
Hello World for the 14-th time
Hello World for the 15-th time
Hello World for the 16-th time
Hello World for the 17-th time
Hello World for the 18-th time
Hello World for the 19-th time
Hello World for the 20-th time
Hello World for the 21-th time
Hello World for the 22-th time
Hello World for the 23-th time
Hello World for the 24-th time
Hello World

Is there a maximum limit of data that iprintln! can send in total?
If not, I guess this should be considered a bug..
Metadata

I used stm32f407g-disc1 board to run this program.
rustc --version --verbose

rustc 1.46.0-nightly (feb3536eb 2020-06-09)
binary: rustc
commit-hash: feb3536eba10c2e4585d066629598f03d5ddc7c6
commit-date: 2020-06-09
host: x86_64-unknown-linux-gnu
release: 1.46.0-nightly
LLVM version: 10.0

How are you consuming ITM? For whatever it's worth, your example works for me with an ITM consumer that we have purpose built that consumes ITM via the TPIU; are you using the STLink/V2 on the Discovery and then OpenOCD? And is your program stopping, or is the ITM data being dropped?

How are you consuming ITM? For whatever it's worth, your example works for me with an ITM consumer that we have purpose built that consumes ITM via the TPIU; are you using the STLink/V2 on the Discovery and then OpenOCD? And is your program stopping, or is the ITM data being dropped?

Thank you so much for testing the example on your side! :smile_cat:
I am using STLink/V2-1 on the Discovery and with OpenOCD.
The program runs to completion and reaches the infinite loop at the end,
and only the ITM data is dropped :crying_cat_face:
(Using gdb-multiarch),
I tried executing the code instruction-by-instruction using the n command
to see if doing that could make the program successfully run all invocations of iprintln!()..
But pressing n just once simply executes the program all the way to the end..
(gdb) n

Breakpoint 4, main () at src/main.rs:20
20      #[entry]
(gdb) n
(gdb) ^C
Program received signal SIGINT, Interrupt.
stim_practice::__cortex_m_rt_main () at src/main.rs:37
37              loop {
(gdb)

Update:
I tested the above example program on another board(Nucleo-429ZI), and it also exhibits the same behavior :crying_cat_face:
So far I've tested two devices and they both exhibit the same behavior. Both belong to the STM32F4 series.

stm32f407g-disc1
Nucleo-F429ZI

Maybe try with probe-rs' ITM support?

Maybe try with probe-rs' ITM support?


feat: use our own github runner to connect to the db to run migrations

Custom GitHub Runner
The intention of this PR is to start using a custom GitHub runner in order to allow the pipelines to connect to our internal database and perform different actions required while deploying our networks, such as, dumping the database and running migrations.

We are finally using a separated database per network. Since most of this job was accomplished internally in our infrastructure there's no proof to demonstrate the split, but you already have access to the console and check the logs.
What really matters, is the deploy pipeline, because before deploying, I'm running a db dump and running the migrations back, making sure the schema is fresh.
Let me know if you have questions or needs more clarification.

you are using yarn v1 for the install etc, which doesn't support our workspaces, and that messes up with the lock file. I'll try to make it work with v2


🐛 [cloud_firestore] sentinel value breaks wrappers

Bug report
Following this issue a "sentinel value" was added to query parameters.
From:
  Query<Map<String, dynamic>> where(
    Object fieldOrFilter, {
    Object? isEqualTo,
    Object? isNotEqualTo,
    Object? isLessThan,
    Object? isLessThanOrEqualTo,
    Object? isGreaterThan,
    Object? isGreaterThanOrEqualTo,
    Object? arrayContains,

To:
    Object? isEqualTo = notSetQueryParam,
    Object? isNotEqualTo = notSetQueryParam,
    Object? isLessThan = notSetQueryParam,
    Object? isLessThanOrEqualTo = notSetQueryParam,
    Object? isGreaterThan = notSetQueryParam,
    Object? isGreaterThanOrEqualTo = notSetQueryParam,
    Object? arrayContains = notSetQueryParam,

However this is a breaking change for any kind of wrapper as null will be passed:
  AppQuery<T> where(
    AppFieldPath fieldPath, {
    Object? isEqualTo,
    Object? isNotEqualTo,
    Object? isLessThan,
    Object? isLessThanOrEqualTo,
    Object? isGreaterThan,
    Object? isGreaterThanOrEqualTo,
    Object? arrayContains,
    Iterable<Object?>? arrayContainsAny,
    Iterable<Object?>? whereIn,
    Iterable<Object?>? whereNotIn,
    bool? isNull,
  }) {
    final queryRef = _ref.where(
      fieldPath.toFirestore(),
      isEqualTo: isEqualTo,
      isNotEqualTo: isNotEqualTo,
      isLessThan: isLessThan,
      isLessThanOrEqualTo: isLessThanOrEqualTo,
      isGreaterThan: isGreaterThan,
      isGreaterThanOrEqualTo: isGreaterThanOrEqualTo,
      arrayContains: arrayContains,
      arrayContainsAny: arrayContainsAny,
      whereIn: whereIn,
      whereNotIn: whereNotIn,
      isNull: isNull,
    );
    // ...

I believe this (breaking) change was not required since there is already a isNull parameter. Exposing notSetQueryParam to the public api doesn't sound too good, so if that previous change wasn't necessary I suggest reverting those changes as this will be a breaking change for libraries wrapping firebase. For instance fake_cloud_firestore has diverged in behavior from cloud firestore.
Note: The dart default parameter proposal would have been helpful here

/cc @Lyokone

Hey @cedvdb, thanks for the update. Upon review, we have decided that this does constitute a breaking change and will roll this back in the next release 🙏


pull request for e2e test at 11/08/2016 07:26:40

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

Details

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Update manifests/e/Eugeny/Tabby/1.0.172/Eugeny.Tabby.locale.en-US.yaml

Fixes an issue where this package has multiple different monikers
Microsoft Reviewers: Open in CodeFlow

[Policy] Reset Labels


Bug: TCP_NODELAY should be in IPPROTO_TCP level

@carllerche Please merge. This is a fatal bug!

@zonyitoo Can you rebase and squash.  I think the only commit we want in this PR is https://github.com/zonyitoo/nix-rust/commit/e2cac27a5fa4c926f7a6173ce204c3b7320635e2
Otherwise, LGTM.

@posborne Updated.

Thanks! merged: aa0863b0b9b31630159b5a36db694b43b3500bdc
(Sorry for the delay, I've been traveling :smile:)


Twig filter syntax

It doesn't play well with underscores:
Filter without underscore:
{{ users|length }}

    {{ users
        |
            length
         }}

Filter with underscore:
{{ "string with spaces"|url_encode }}

    {{ "string with spaces"
        |url_encode }}

Thanks for reporting this!

Thank you for fixing it.


Fix condition builder

name conflicts are not resolved in ParamUtil.addExpression
(eg. when condition uses same property as update)

:tada: This issue has been resolved in version 4.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Configure Azure PipelinesCode workflow

This pipeline configuration builds the application and produces a build for the Release configuration.

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


setup all-contributors bot and badge for README.md

@all-contributors please add @roannav for code

@all-contributors please add @roannav for audio, a11y, and content


Song crashes when I try to go in the chart editor

Describe your problem here. If you are modding, please try testing your problem in a clean version of the engine instead, this ensures the problem is actually caused by the engine itself. Also, if you're using an older version of JSE, please try the latest action build in the Actions tab or the latest release build. The issue has probably been fixed since that version. Oh yeah, and also be sure to check the pinned 'JS Engine: Known Issues' issue, to double check if your issue hasn't already been found!
So I was trying to test out my custom character I made so I went in my song and when I clicked 7 key on it it crashed.

Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No I did not im pretty sure

you're missing a json named "thearchy-reboundway-hard"

you're missing a json named "thearchy-reboundway-hard"

👆


Update using-the-exchange-server-2013-management-pack-for-troubleshoo…

…ting.md
Code snippet update

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange-2013-guide-management-pack/using-the-exchange-server-2013-management-pack-for-troubleshooting.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


What's the meaning of "squeeze_idx" in eval_datasets.py?

Hello!
When I read the code in eval_datasets.py, I have a question about the function "read_label" of class VOSTest.
Could you please tell me the meaning of "squeeze_idx"? When I debug the code, I find that in "getitem", it have used "obj_idx" as "squeeze_idx" in "read_label", but it seems no difference between the input label and "squeezed_label", so what's the effect of "squeeze_idx"? And under what circumstances will it be used?
Thank you!

A good question.
In some video sequences of YouTube-VOS, the object number may are smaller than the maximum object index. For example, there are only 3 objects in a sequence, but their indices are 1, 3, 10 instead of 1, 2, 3. Under such kind of cases, we have to squeeze the indices to be compact.

A good question.
In some video sequences of YouTube-VOS, the object number may are smaller than the maximum object index. For example, there are only 3 objects in a sequence, but their indices are 1, 3, 10 instead of 1, 2, 3. Under such kind of cases, we have to squeeze the indices to be compact.
Thanks for your reply.
After your explanation, I guess the advantage of making indices to be compact is that when extracting the labels of all targets, it can have a smaller tensor. For example, input label is [11, 480, 910], objects' indices are 1, 3, 6, the extracting result is [7, 480, 910], but after compacting it is [4, 480, 910]. So it reduces time and memory, is that right?

Yes, it will reduce time and memory when objects' maximum index is larger than the size of the identity bank. But if objects' maximum index is smaller than the size of the identity bank, the time and memory are always the same.

Got it, thank you very much!


@orta => Favorites logged out

When you are logged out and you tap the YOU section, we will show the sign in modal on top of whatever your current view is, instead of taking you to an empty YOU section.
fixes https://github.com/artsy/eigen/issues/108.
Also includes a bunch of changelog updates.

rockin.

Looks good :+1:


[SQL][Minor] Remove the OverrideFunctionRegistry

Remove the OverrideFunctionRegistry from the Spark SQL, as the subclasses of FunctionRegistry have their own way to the delegate to the right underlying FunctionRegistry.

Merged build triggered.

Merged build started.

Test build #36662 has started for   PR 7260 at commit 2ca8459.

Test build #36662 has finished for   PR 7260 at commit 2ca8459.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Merged build triggered.

Merged build started.

Test build #36688 has started for   PR 7260 at commit 164d093.

Test build #36688 has finished for   PR 7260 at commit 164d093.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Removing a class is not "minor".  That should be reserved for things like fixing a spelling mistake in documentation.

Sorry, @marmbrus , I am not so sure your mean, are you suggesting to create a associated jira issue or not removing the class for now?

Please create a JIRA
On Jul 7, 2015 5:18 PM, "Cheng Hao"<EMAIL_ADDRESS>wrote:

Sorry, @marmbrus https://github.com/marmbrus , I am not so sure your
mean, are you suggesting to create a associated jira issue or not removing
the class for now?
—
Reply to this email directly or view it on GitHub
https://github.com/apache/spark/pull/7260#issuecomment-119379745.

Alright thanks. Merging this in.


Hook up build server for linting/test running on PR

Arrange for a Jenkins job to run linting/tests and report back pass/fail on PR submit.

I haven't personally used it but Travis CI looks like a good fit.

I haven't used Travis either but it might be fun to set it up. I have a Jenkins server set up at the moment to handle this I just need to set up the job and add a script. I'll have it ready this weekend once the eslint integration branch gets merged in. Tests would be next...

Integrated with Jenkins server to build and lint. Creating separate issue for test running.


Use new PS format

With introduction of Az PowerShell module for Azure the syntax have changed to be New-AzAutomationSourceControl. It would be nice to spell that out.
Thanks!

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

ID: 83c90e64-b615-711f-a53d-fc76606e2ecd
Version Independent ID: 2d164036-6886-4440-50f7-369f99f41cea
Content: Source Control integration in Azure Automation
Content Source: articles/automation/source-control-integration.md
Service: automation
Sub-service: process-automation
GitHub Login: @MGoedtel
Microsoft Alias: magoedte

@trailmax Thank you for your feedback, we will further look into it and get back to you at the earliest.

@trailmax Currently the documentation reflects the power shell command-lets  pertaining to new Az module only.

We do have a note at the starting of the article about the usage of new Azure PowerShell Az module.

Other than the details listed above,  could you please let us know if you have any further feedback to improve our documentation on the usage of new Azure Powershell Az module. Thank you

@BharathNimmala-MSFT Indeed it does. Apologies, I must have overlooked it or was looking on a wrong page in a frenzy trying to resolve my issue at hand.


Fix another broken link

There was a second broken link that I missed on the first PR today
Let's check to see if my GH action to check links will be green before merging this one.

Now it's red but the link reported is a false positive.


OIPA-2100

https://app.zenhub.com/workspaces/iaticloud-5b8e865bb2cc7b6c78ece322/issues/zimmerman-zimmerman/iati.cloud/2100

:tada: This PR is included in version 5.2.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


feat: externalize ngWalker to support custom lint rules

relates to #657

Thanks for the quick PR! My idea was to externalize the content of the angular directory into a separate package. Does this cover your use case completely? Keep in mind that there are more visitor abstractions for traversal of template, style, and expression ASTs.

Hi @mgechev, Thank you for your input. I only needed NgWalker for my use case, but I think it makes sense to export the angular directory contents, there's so much goodness, everyone could use to build rules.
Just one question, When you are saying a separate package, do you mean npm module like @codelyzer/angular and @codelyzer/rules something like that?

Yea I am trying but couldn't find anything.. locally the command runs without any problems.

@piyukore06 if TypeScript won't complain, I'd suggest exporting only the base visitors without any extra internal APIs. This way, we'd be able to make changes in them without introducing issues for consumers of codelyzer.

@mgechev thank you for your comments! They lead me to a better understanding of, where to look and what should be exported.


Endless mining?

The last purchase contract and the last refund more than 8 hours ago, the buyer hub is empty, but there is a miner in the wallet and a worker (online) in f2pool

3 times


Bad path for tsc.exe when using TypeScript 1.4 or 1.5 with Visual Studio Online Builds for Visual Studio 2015 Projects

This issue is similar to issue #3493.
When running a build using the new build system via Visual Studio Online, the following errors occur:

This occurs whether I am using TypeScript 1.4 or 1.5 in my project (the highlighted part of the path reflects the TS version being used).  My last successful build finished at 5:03 PM, EST on 8/31/2013 and without any code changes, my builds are all failing now with this same set of errors.

As mentioned in https://github.com/Microsoft/TypeScript/issues/3493#issuecomment-13682268, we have  contacted the VSO team and they are working on fixing the issue. this is not caused by any changes on the TypeScript side.

Closing issue, as this is a problem with VSO, not TypeScript.


Adding personal .svg Icons from a folder in app.jsx

Hey, so I am trying to add my own icons to the appRoutes cosnt in app.jsx(in variables) however I can't seem to import these icons in any way. Any suggestions on how to use our own icons?
I have them as files in a folder

Hi @EINazare ,
We have single path icons working using this code. Thank you for showing the way!
How would you represent an svg with multiple paths in the code above?
Example:

</svg

Hello @intelage-steve ,
Unfortunately i do not know how you would do that.
I will take a look at this when i'll have some spare time, until than, maybe someone else will come to your help.
Best,
Manu


Change primary and accent Colors

Hello,
Here is the important part of my setup:
    "react-native-material-kit": "^0.5.1",
    "react-native": "0.52.1",
    "react-navigation": "^1.0.0-beta.19",

Thanks to react-navigation, my "App.js" looks like :
import React, {Component} from 'react';
import {Root} from './config/router';

class App extends Component {

  render() {

    return <Root/>;
  }
}

export default App;

and a screen called "HomeScreen" is my entrypoint.
I tried to change the default primary/accentcolor with this line, in "HomeScreen" :
setTheme({
    primaryColor: MKColor.Purple,
    primaryColorRGB: MKColor.RGBPurple,
    accentColor: MKColor.Lime,
    accentColorRGB: '205,220,57',
  }
);

class HomeScreen extends Component {
  ...
}

(quite ugly, but it's for testing purposes ;) )
I use theses colors in the headers of my screens :
  static navigationOptions = {
    title: "SomeTitle",
    headerStyle: {
      backgroundColor: theme.primaryColor,
    },
    headerTintColor: theme.accentColor,
  };

But it appears that the entrypoint's (HomeScreen) header has the default colors, but every child's screen has the correct header...
I don't really know what I'm doing wrong.
(also tried to put theses lines in the App.js, no success)
Thanks for any help :)

Well, with react-navigation, default parameters can be sets in the router : see here
With theses parameters here, setting the primary/accent colors in my "HomeScreen" works.
This solves my use-case :)


Type inference problem

Where is the syntax wrong?

@sunshinejr Thank you very much for your answer, how Moya implements multiple API simultaneous requests？

@wwdc14 Moya doesn't have an API support for multiple simultaneous request, no. You would have to create it yourself.

Ok, thank you for your answer.

Hey @wwdc14 no problem, happy to help! 👍


its not flinging

scroll not fling

Hi @klodianpepkolaj , thanks for your feedback. Could you describe the issue in more details (device model, steps to reproduce etc.)
As you can see on the demo-gif fling works under normal circumstances.

@shchurov tried already on 3 different devices with different SDK versions , everything works fine except flinging

@klodianpepkolaj  I can't come up with anything that could cause this issue. Works just fine for me. Could you try to debug it? The class name is TouchHandler, it's pretty straightforward.

@shchurov yes thats what i am trying to do, but not able to find the error, API 19, 21,22, already tried

@shchurov made there is an error in the code you have commited, please update the code here in github

@klodianpepkolaj It's the latest version, may I ask you to make a video of that issue?

@shchurov the issue was, when on developer options, and animation scale 0x, it doesnt work, but I found another solution for this that the fling works even when developer options are on. thanks

@klodianpepkolaj great, I just thought yesterday that it could be related to those animation settings. Glad that you resolved it.


[2.0] Further timeline speedups

Implements further speedups to the timeline resource, across the board.
To summarize the changes, at the core, the crux of the problem with the Timeline resource is it seems resources aren't actually meant to be used in this fashion. Building a Resource from a bunch of modular parts is inherently slow, because has to keep reloading all of the modules that it uses, at the step of DialogicUtil.get_event_by_string(). Then when it actually parses the event and fills out the data in it, that further reloads the module and adds even more load time, which I had previously cut down on by adding a thread for deferred processing at runtime.
There seems to be no way to implement a cache for this directly in the Resource loader, because the Resource loader checks the resource script file as a whole as the very first thing, and marks it wholesale unusable if any thing is missing. For example, at runtime I added a cache to the event scripts, and tried having the Timeline loader use it, but the resource loader loads before the Autoload for DialogicGameHandler, and even trying to add it in a conditional to check if it's loaded yet, it was still throwing an error that it was missing and it marks that resource type unusable for the game's runtime.
The only solution to this is by caching elsewhere, and processing it as a secondary step. Stripping the Timeline Loader itself down to just building a barebones event list of just strings, and then making the events themselves after the cache can finally build. In runtime, a cache on the  DialogicGameHandler that builds the script cache while it's loading the subsystems. In editor will be a bit more complicated, since it will have to monitor for changes in the filesystem and rebuild as well, but it will probably have to be attached to the EditorView.  then all the normal event creation will need to check the cache instead of actually iterating through the files calling DialogicUtil.get_event_by_string() . For saving, that seems to be taking a long time as well, and I haven't looked at it but I suspect it's doing the same multiple loads thing, and will  have to switch to the cache as well.

Commits as of this writing:
Implements it on the runtime for loading only. Changes TimelineUtil.text_to_events() to simply load the .dtl file, strip out the blank lines, and build an array with just the text values for each line. As all the work processing it is deferred until later, total load time for just the resource loading at runtime comes down to ~10 ms.
Initial load time of test timeline of 220 events before changes: around 9000 ms
Load time after this change: 30 ms

I was... not expecting anywhere near this much of an improvement. I was hoping at best to get it down to 1/4 of the original load time, to go from 9000 to 1500 ms
And having it process all the events at this stage as well only adds about 10 more ms:

The deferred loader thread is simply not needed any longer. It's completely redundant now.

summary of where i have to leave off for the moment: print's are littered everywhere at the moment for performance checking, but saving for Text timelines is implemented, and both Text and Visual timeline loads work. Character matching isnt working in Visual timeline yet, nor is saving
however, there is the following... strange issue, whcih I thought at first started with #1129, but is even still getting it trying to undo those changes to get rid of TimelineUtil. I suspect is an issue with the current Godot version, ultimately. but on project start the following errors will occur, and will repeatedly show up over and over:
'ERROR: Script does not inherit a CustomResourceLoader: res://addons/dialogic/Resources/TimelineResourceLoader.gd.
at: (core/io/resource_loader.cpp:979)
ERROR: Script does not inherit a CustomResourceSaver: res://addons/dialogic/Resources/TimelineResourceSaver.gd.
at: (core/io/resource_saver.cpp:211)'
due to this, Saving never completes, and trying to open a new timeline from the file picker gets this error:
'editor/editor_node.cpp:1201 - Condition "!res.is_valid()" is true. Returning: ERR_CANT_OPEN'
HOWEVER
For some bizzare reason, touching those two files fixes this for the session, until you close and reopen Godot again. So opening up the TimelineResourceLoader and TimelineResourceSaver files, adding a space in there somewhere, and then saving, will let it work fine

visual timelines should save and load properly now as well. need some more thorough testing

the timleine resource errors are gone for me now. at the moment the indents aren't being handled correctly and that's being lost, once I got that resolved eveyrhting should be good

didnt have much time to work on it today, but fixed the tab calculation, so now saving and loading properly for the Visual timeline editor.  the tabs are being lost, however, in the switch between Text and Visual timelines, so something different is happening with those that I still need to figure out

alright! now saving works properly from both timelines by themselves, and they both can switch modes properly from one to the other without issue. Only one final problem left now: how the "Test Timeline" button works is now screwing it up overriding the file before it runs. once that's done I'll document all the final changes

Finally! All of them are good!
There are a few extra steps on the saving on each editor that are probably not necessary, but I was having a lot of trouble getting the data to switch back and forth between the Visual and Text editors to work properly without them. But overall, there are the following changes:

Adds an event cache, attached to the EditorNode in editor, and attached to the DialogicGameHandler at runtime. This immensely speeds up the actual loading and creation of events, due to not constantly reiterating through the script files and reloading them over and over as it was before. in editor, it should probably rebuild that if a new one is detected (if someoe adds custom events, for example), but doesn't currently.
Adds a character cache of all the current characters in the project. this also is attached to EditorNode in editor, and DialogicGameHandler at runtime. at runtime, when creating the Character or Text events, it will search for them in the character cache and use it from there, speeding up load time.  due to an editor-only error trying to find the EditorNode in the scene tree from in the actual DialogicEvent creation to pull the character cache, a reference is passed to them at node creation as a meta property - this is a temporary handling, and will be fixed with other revamps to the charatcter suggestor.  as with the event cache, it should rebuild in-editor if a Character is saved, which I didn't because the Character editor was being rebuilt at the same time and im not sure if its 100% yet.
TimelineUtil was removed, and it's two functions were moved elsewhere, for reasons following
Timeline Loader is stripped down to barebones, and only initially generates an array of the strings of the events, instead of DialogicEvents themselves. This is the only way, unfortunately, to allow the loader to use the event cache
Timeline Saver is stripped down as well. Due to the translation functions being in the saver as well, a DialogicEvents timeline is needed here, but it only takes the text from the event_node_as_text property from each DialogicEvent
To facilitate the above two changes, a property was added to DialogicTimeline, _events_processed. This is set on load as a flag to denote that the timeline has not been converted into Events yet, and is set to true once it has been handled. Additionally, the same flag is used for saving to denote whether the events have been converted back into text again and it's ready to save, and the saver will reject it if the timeline is passed to it with it set to true
adds a further protection as well to saving, and will not save if the timeline is 0 events. this should get rid of other causes of  losing data because of a problem with the data sent to the saver. due to this change, users will need to delete the timelines if they want to blank them out
EditorView.process_timeline(): this is where one of the two TimelineUtil functions went. It goes through the array of text events from the loader - or from an editor - and creates Event nodes from it and populates it, and sets _events_processed once its completed. Additionally, this also prepares a DialogicTimeline for saving, by populating the event_node_as_text with the current event text
Dialogic.process_timeline(): corresponding version of the processor for runtime. this will automatically be called by both start_timeline() and preload_timeline(), and so only particually need to be used separately if creating a timeline in code at runtime from strings, rather than builidng it from DialogicEvent nodes from the event cache
TextEditor now loads the data from events from event_node_as_text only, discarding the rest. When saving, it builds a new array from the editor of just text node, as if it were the same as loading a Timeline (since the text mode is identical to .dtl file), and sends it back to EditorView.process_timeline() before saving. It
VisualEditor - which has now had it's files renamed to be that instead of TimelineEditor - has the most speedups. it loads and stores AddEventButton.tscn to instance on each use when populating the event buttons, instead of loading it, whihc speeds it up a bit and also cuts out a lot of the 'invalid UUID' warnings that was being generated by each load of it before. It also builds the buttons based on the event cache, cutting out further file searches and loads.    when saving, like TextEditor, it builds a new array of text, created from each event node's _store_as_string(), and sends it back to EditorView.process_timeline() before saving. a bit redundant to send it back first on this one, instead of just storing the '_store_as_string()' back in each node and sending it on to the saver, but its for safety, and also fixed some issue with the data not exchanging correctly when switching back and forth between TextEditor and VisualEditor. it then switches the timeline array back to that same text array after saving, for that same reason.
Both timeline editors are now properly and completely hooked up to the set_resource_unsaved() signal in the toolbar, so there's visual notification if any changes have been made. Both timelines will now only save if there has been changes, to save on unnecessary file saving. This unfortunately doesn't apply to the not-current timeline, so other timelines still open in Godot's memory will not reflect this in the recent file suggestions
Due to the above issue with it not applying to other open timelines, a meta property is set on the timelines as well, timeline_not_saved. This is set as true if a change is made, and set as false once it is saved. As (due to the recent suggestion selector in the toolbar) saving the project itself will try saving all of the open timelines, not just the current one on the timeline editor, the Timeline Saver checks for this meta property on the DialogicTimeline before doing anything else, so the non-current timelines don't try saving as well, possibly throwing errors and blanking the file because they aren't  garaunteed to have been prepared for save in wherever Godot is holding them open
made the current editor mode save properly, so it wouldnt keep reverting back to visual editor on project restart (this seemed to start with a different recent change, not sure which, it used to remember it before)
due to the massive speedup having the event cache brings, gets rid of the old processing thread that was previously used for optimization entirely. this means Dialogic.process_timeline() will parse every event when run, although it still isn't loading nested timelines due to mainly memory constraints. function can be added to this later if needed in the preloader, I still need to test on my RPG for Dialogic's response time now compared to 1.x where there was a noticable stutter on every timeline start.
adds some queue_free() in a few places as _exit_timeline() functions, to try and minimize the leaked resources at project close. doesn't get rid of them all, though.
also includes adjustments to the 1.x Converter for the changes from other recent PR to the Character file format, which I needed to include to rebuild my timelines as I was testing because they kept being corrupted by in-progress work and i was running out and it would not run due to those changes. also includes an icon in the folder, so Godot doesnt show a missing file error
adds a fix for DialogicUtil.get_event_scripts() so it doesn't generate errors if there are empty folders in the Event paths without event.gd files

And I think that's all of them


pull request for e2e test at 01/22/2017 17:15:12

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


BlocklyController.trashRootBlock(..) now respects the block's deletable flag.

Also, adding new method .trashRootBlockIgnoringDeletable(..) as a means to override this behavior.
Partial fix for #305.

This change is

I think this will break things as is.

Review status: 0 of 3 files reviewed at latest revision, 2 unresolved discussions.

blocklylib-core/src/main/java/com/google/blockly/android/control/BlocklyController.java, line 761 [r1] (raw file):

    /**
     * Remove a block and its descendants from the workspace and put it in the trash, regardless of
     * block's deletable state.  This method should only be used by for programmatic manipulation of

remove 'by'

blocklylib-core/src/main/java/com/google/blockly/android/ui/Dragger.java, line 596 [r1] (raw file):

        mDraggedConnections.clear();
        if (!mController.trashRootBlock(mPendingDrag.getRootDraggedBlock())) {
            // TODO(#305):

I think this is going to cause issues. All the connections will end up abandoned and the block may become unusable.

Comments from Reviewable

Doesn't finishDrag need to be called to add all the connections back to the manager? It looks like we'd skip that and just empty the list of connections instead. Is something else adding those back in the right location?

Review status: 0 of 3 files reviewed at latest revision, 2 unresolved discussions.

Comments from Reviewable

Minor nit in comments then

Review status: 0 of 4 files reviewed at latest revision, 2 unresolved discussions.

blocklylib-core/src/main/java/com/google/blockly/android/control/BlocklyController.java, line 761 at r1 (raw file):
Previously, RoboErikG wrote…
> remove 'by'

Still remove 'by'

Comments from Reviewable

Review status: 0 of 4 files reviewed at latest revision, 2 unresolved discussions, all commit checks successful.

blocklylib-core/src/main/java/com/google/blockly/android/control/BlocklyController.java, line 761 at r1 (raw file):
Previously, RoboErikG wrote…
> Still remove 'by'

Done.

blocklylib-core/src/main/java/com/google/blockly/android/ui/Dragger.java, line 596 at r1 (raw file):
Previously, RoboErikG wrote…
> I think this is going to cause issues. All the connections will end up abandoned and the block may become unusable.

Resolved.  Connections are now re-added in the aborted delete case.

Comments from Reviewable


Updates for Feb 2016 EIA Release

proxy to version 1.1
generate impact report script supports secure services

Will not merge this pull request because other changes has been made in the develop branch since this pull was created.  Will close this pull request and create another that includes the latest updates.


Filter float by search_model_fields

Hello Arthurio,
if one or more fields in search_model_fields are floats, there will be no usable result by the filter.
For example:
class ArticleFilter(Filter):
    search: Optional[str]
    ean: Optional[float]
    description: Optional[str]

    class Constants(Filter.Constants):
        model = Order
        search_field_name = "search"
        search_model_fields = ["ean", "description"]

In my case there are no results because of the float datatype. It must be cast to string or need to use with equals operation (not ilike).
I found this article on StackOverflow. I hope you can help me.
I use SQLAlchemy as ORM with MSSQL.
Thank you alot for this fine piece of software <3 <3

Since I haven't heard back from you I'll close this. Feel free to create a new issue with more details.


import and export es6

Cannot export or import clases when using es6.
Exporting class:
export class authRestService
{
  constructor($resource, env) {
    this.$resource = $resource;
    this.env = env;
  }
}

Importing class and extending:
import {authRestService} from './authrest-service';

(function () {
  'use strict';

  class Auth extends authRestService {
    constructor($resource, env) {
      super($resource, env);
    }

    get() {
      return 'Auth';
    }
  }

  /**
   * @ngdoc service
   * @name components.service:Auth
   *
   * @description
   *
   */
  angular
    .module('components')
    .service('Auth', Auth);
}());

i get the following error on the console,

authrest-service.js:3 Uncaught ReferenceError: exports is not defined(…)(anonymous function)
auth-service.js:11 Uncaught ReferenceError: require is not defined(…)(anonymous function)

This has been a pretty big issue for a while. ng-poly does not currently have a module bundler setup, so import/export aren't going to work.
There's a part of me that wants to get Webpack implemented (also would help with the huge Gulp files), but I haven't used Angular in ages.
I'm sorry, but this is probably something that won't be fixed anytime soon. A warning for this ought to be added to the readme.

@dustinspecker thank you for taking your time to respond, do you suggest any approach (some npm module ect.) so i can resolve this and added to the gulp files? and maybe create a PR for this repo, it would be nice to have it.


SignalR / Support ServiceWorkers

Is your feature request related to a problem? Please describe.
Looks like the SignalR Javascript bundle does NOT support ServiceWorkers, due to lacking of XMLHttpRequest object in SW scope.
Describe the solution you'd like
Provide an additional HttpClient implementation that uses the fetch primitive instead.
Describe alternatives you've considered
No alternatives :(
Additional context
N/A

cc @davidfowl this seems similar to an issue @halter73 merged in #7058. Thoughts on this?

It’s not the same thing service workers an web workers are different and though there might be some overlap in the fixed I’m not sure this covers everything. It seems like we’ll need to switch to fetch when available as part of supporting service workers.
I’m not 100% sure about using SignalR in service workers though @anurse did some investigation here

@bradygaster @davidfowl I made some changes and it works now. How can I submit a PR?

Sure that would be great!

How can I submit a PR?

https://github.com/aspnet/AspNetCore/blob/master/CONTRIBUTING.md#submitting-a-pull-request
@unsafecode Thanks for looking into this. The reason we just added web worker support is because an external contribute submitted PR.

PR submitted at #7672

@unsafecode While the PR is good and we're likely going to merge it because fetch support is awesome (we punted it here https://github.com/aspnet/SignalR/issues/1240) but I don't understand the use case for signalr inside of a service worker? How do you plan to use it?

@davidfowl We're build an enterprise-wide PWA that:

Uses Saga-based distributed transactions on microservices-oriented application, so we needed a way to notify the client when operations actually complete
We wanted to re-use the same approach to send "push notification" without WebPush. And SignalR in SW enables to trigger Notifications.

Can you describe how SignalR fits in? Doesn't the service worker die on idle? Isn't the point to avoid a long running connection in the first place? Service Workers are about intermittent connectivity while SignalR/WebSockets are about long running connections. Here's a quote from one of the many blog posts on this topic:

So, while WebSockets and Service Workers kind of overlap, both being used for client-server communications, WebSockets would be used more for continuous communication so both the client and the server can send messages to each other without having to create new connections, and Service Workers would be used more for one-off client-server communications like syncing an app and the database after the app was offline for some time, like a burst that can then be dropped when the sync is completed

Doesn't the service worker die on idle?

Yes, that does seem to make it pointless to run SignalR in a service worker.  In the scenario for the server to push notifications even when the web app isn't active, service workers have a role using Web Push API (e.g. Push Notifications and ASP.NET Core) but @unsafecode why don't you want to do it that way?

We don't believe there's a scenario for SignalR in service workers right now.


[87] move resource definition to irods_client side of config

Pretty sure this also needs to be updated/changed/moved...
https://github.com/irods/irods_client_s3_api/blob/1c767a443b68b56e2b1466d52bc9035396b39186/src/main.cpp#L270-L272
Input welcome.

----------------------------------------------------------------------
Ran 66 tests in 275.889s

OK (skipped=1)

real    4m59.731s

but that makes me think we're not testing against a non-default resource... will poke at that next.

I just realized that code block you referenced isn't part of your PR.
You can change that line to the following:
using json_pointer = nlohmann::json::json_pointer;
irods::s3::set_resource(config_value.value<std::string>(json_pointer{"/irods_client/resource"}, ""));

gotcha - thanks.

As a matter of fact, replace those 3 lines with the 2 lines I posted.

yes, since it doesn't come out of the s3_servers side anymore.

Right.

broke it (we ARE honoring the non-standard resource)...
then fixed it...
----------------------------------------------------------------------
Ran 66 tests in 276.863s

OK (skipped=1)

real    5m0.698s

okay, truly done, can squash once eyeballed.

Do the tests still pass with the resource change?

yes - https://github.com/irods/irods_client_s3_api/pull/88#issuecomment-1909162671

Ah I see. What's the non-standard resource?

newResc is the one created, and then used in the /config.json

I'm back now.

squashed - please one more look before #'ing.

#'d


Updating the .Net Core supported versions

Adding that .Net core 2.2, 3.0 and 3.1 are now supported by AWS serverless functions

https://aws.amazon.com/blogs/developer/announcing-amazon-lambda-runtimesupport/
https://aws.amazon.com/blogs/developer/net-core-3-0-on-lambda-with-aws-lambdas-custom-runtime/
https://aws.amazon.com/blogs/compute/announcing-aws-lambda-supports-for-net-core-3-1/

Thank you!


Restrict course selection when candidates Apply Again

Context

When a candidate's application is unsuccessful, they can opt to Apply
Again. One of the business rules for the Apply Again process is that
they can only apply to one course at a time.
Changes proposed in this pull request

Add UI changes to prevent the adding of additional courses under Apply Again.
Add a form validation to prevent the same when navigating to the course selection screens by URL.

Application Form

Course Choice Review

Validation error

Guidance to review

Can be tested manually with an application in the Apply Again phase. Probably easiest to use the DuplicateApplicationService in the console.
Check the component refactor makes sense.

Link to Trello card

https://trello.com/c/0Tj02wBa
Things to check

[ ] This code doesn't rely on migrations in the same Pull Request
[ ] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[ ] API release notes have been updated if necessary
[ ] New environment variables have been added to the Azure config

Found this bug, steps to reproduce:

Log in as<EMAIL_ADDRESS>on this Review app
Go to https://apply-for-te-1391-apply-mzq34j.herokuapp.com/candidate/application/courses/provider
Gorse SCITT
Biology 7 to 14
Boston Spa Academy
💥

Have done a bit of manual testing and looks like it's watertight (I wasn't able to work around the validation). Using the back button can be confusing but I think this is a longstanding issue. e.g. if I add my one permitted course and then hit Back a couple of times I end up with a validation error:

@tvararu That's the existing experience when you get a validation error on PickSiteForm. Agreed this could be better, though. My latest commit changes that render call, and instead redirects to the application form with a flash warning set.

WDYT?


[DO NOT MERGE] Sse test failure

So this bug is that we're calling SSEOutputStream.close()  whose docs say "we close and if it's already closed then it's fine, you can keep closing it forever you will not get an exception", which is the right thing to do IMO.
But the implementation calls ResteasyAsynchronousResponse.complete(), which then forwards to a vertxFlush() which throws because it's already closed (by the client).
Now, perhaps the right thing to do is to ignore any IOException in SSEOutputStream.close()  but I do wonder what the behaviour of ResteasyAsynchronousResponse.complete() should be. Should it consider that complete() should throw if already closed or not?

Just an added note for the others.  The code change of interest is in file
resteasy-core/src/main/java/org/jboss/resteasy/plugins/providers/sse/SseEventOutputImpl.java
This proposed fix is to address https://issues.redhat.com/browse/RESTEASY-2542 SSE tests intermittent
failures on Travis CI

Hum, why do I not see any CI running?

@FroMage

Hum, why do I not see any CI running?

I think that's because the PR was marked as draft. Now, is this still needed, given the discussion going on in #2348  ?

Indeed not.


#41 - double quotes fixer enabled

It should fix problems with single and double quotes in PHP files. I'm not closing issue, because there're other things to fix in it.

Please, resolve composer.lock conflicts

Please, resolve composer.lock conflicts

On my way.


Modal CSS glitches on Mobile

There a a few CSS glitches when viewing the Modal on a mobile device, namely two:

The "×" button is too far on the right and partly off-screen and possibly hard to click (on a small display, e.g. smartphone)
When scrolling down, while the Modal is open, the background "dim" isn't continuing. Maybe this is intentional, but I don't think so.

I attached two screenshots with the two bugs.
Apart from that, great project & awesome blog post!

Two minor changes to the modal CSS would fix these issues:
Changing the value of  position for .js-modal to fixed will make the modal follow the user when they scroll.
Changing the value of right for .js_modal_close to 0px will stop the close button being cut off on small screens.

Happy to take a pull request on this 🙂

Now fixed by @VerifiedJoseph and deployed.


Fixed birthday again back to what chris did

SOrry guys my bad.

Changes Unknown when pulling 9696faab0f4f5d451af56d337fb80f8fa4dbf872 on chilipadiboy:master into ** on CS2103AUG2017-W10-B3:master**.

Can use git revert instead of manually revert back to the working version.


Crash on startup with Mekanism

https://pastebin.com/2LaZsNXh
Tested with Mekanism <IP_ADDRESS>9 and VanillaFix 1.1.1 132 on Forge <IP_ADDRESS>68

Nevermind, I wasn't using the latest version since it's not on curseforge. sorry about that-.


B#40274 Clean tmp dir in image microservice

Can you then add the bash script to scripts/cronjob/clean-tmp-dir-image-microservice.sh that includes the actual command to do so, like cd microxxxx && docker-compose exec image_app bash remove-tmp-files.sh?


RFE: Handle artifacts that move to new GAV?

Sometimes artifacts move to a different GAV signature, so they're out of date but no further updates appear. Is it feasible to detect this as a new version and include it in the report?
Failing that - or in addition - perhaps there could be a report on dependencies that appear to be the latest version, but are years old? That could be a clue that either they've moved to a new GAV, or they're no longer maintained.

Sounds like a reasonable addition for me, but I don't know if the required date information are available to Maven.

I still think this has value.

It can be possible only if artifacts maintainers prepare proper relocation information.
https://maven.apache.org/guides/mini/guide-relocation.html


counterpartylib.lib.exceptions.ParseTransactionError: index out of range

10|counterparty  | ------------------------------ start ------------------------------
10|counterparty  | [2024-04-10 02:48:04][INFO] Connecting to address indexer.
10|counterparty  | [2024-04-10 02:48:04][INFO] Acquiring lock.
10|counterparty  | [2024-04-10 02:48:04][INFO] Connecting to database (SQLite <IP_ADDRESS>).
10|counterparty  | ERROR:root:SQLITE_LOG: recovered 626 frames from WAL file /home/adam/.local/share/counterparty/counterparty.db-wal (283) SQLITE_NOTICE_RECOVER_WAL
10|counterparty  | [2024-04-10 02:48:04][INFO] Starting API Server.
10|counterparty  | [2024-04-10 02:48:05][INFO] Resuming parsing.
10|counterparty  | [2024-04-10 02:48:05][INFO] New block inserted 838540
10|counterparty  | [2024-04-10 02:48:06][INFO] New transaction inserted 255930ade36ea75bb1c9373417fe627bc8e9f484da1489e8535ae9fb59adfa86
10|counterparty  | [2024-04-10 02:48:06][WARNING] ParseTransactionError for tx 255930ade36ea75bb1c9373417fe627bc8e9f484da1489e8535ae9fb59adfa86: index out of range
10|counterparty  | [2024-04-10 02:48:06][ERROR] Unhandled Exception
10|counterparty  | Traceback (most recent call last):
10|counterparty  |   File "/home/adam/counterparty-core/counterparty-lib/counterpartylib/lib/blocks.py", line 142, in parse_tx
10|counterparty  |     dispenser.parse(db, tx, message)
10|counterparty  |   File "/home/adam/counterparty-core/counterparty-lib/counterpartylib/lib/messages/dispenser.py", line 314, in parse
10|counterparty  |     action_address = address.unpack(message[LENGTH:LENGTH+21])
10|counterparty  |   File "/home/adam/counterparty-core/counterparty-lib/counterpartylib/lib/address.py", line 56, in unpack
10|counterparty  |     if enabled('segwit_support') and short_address_bytes[0] >= 0x80 and short_address_bytes[0] <= 0x8F:
10|counterparty  | IndexError: index out of range
10|counterparty  | During handling of the above exception, another exception occurred:
10|counterparty  | Traceback (most recent call last):
10|counterparty  |   File "/home/adam/.local/bin/counterparty-server", line 8, in <module>
10|counterparty  |     sys.exit(server_main())
10|counterparty  |   File "/home/adam/counterparty-core/counterparty-cli/counterpartycli/__init__.py", line 20, in server_main
10|counterparty  |     server.main()
10|counterparty  |   File "/home/adam/counterparty-core/counterparty-cli/counterpartycli/server.py", line 230, in main
10|counterparty  |     server.start_all(catch_up=args.catch_up)
10|counterparty  |   File "/home/adam/counterparty-core/counterparty-lib/counterpartylib/server.py", line 543, in start_all
10|counterparty  |     blocks.follow(db)
10|counterparty  |   File "/home/adam/counterparty-core/counterparty-lib/counterpartylib/lib/blocks.py", line 823, in follow
10|counterparty  |     new_ledger_hash, new_txlist_hash, new_messages_hash, found_messages_hash = parse_block(db, block_index, block_time)
10|counterparty  |   File "/home/adam/counterparty-core/counterparty-lib/counterpartylib/lib/blocks.py", line 200, in parse_block
10|counterparty  |     raise e
10|counterparty  |   File "/home/adam/counterparty-core/counterparty-lib/counterpartylib/lib/blocks.py", line 196, in parse_block
10|counterparty  |     parse_tx(db, tx)
10|counterparty  |   File "/home/adam/counterparty-core/counterparty-lib/counterpartylib/lib/blocks.py", line 161, in parse_tx
10|counterparty  |     raise exceptions.ParseTransactionError(f"{e}")
10|counterparty  | counterpartylib.lib.exceptions.ParseTransactionError: index out of range
10|counterparty  | Traceback (most recent call last):
10|counterparty  |   File "/home/adam/counterparty-core/counterparty-lib/counterpartylib/lib/blocks.py", line 142, in parse_tx
10|counterparty  |     dispenser.parse(db, tx, message)
10|counterparty  |   File "/home/adam/counterparty-core/counterparty-lib/counterpartylib/lib/messages/dispenser.py", line 314, in parse
10|counterparty  |     action_address = address.unpack(message[LENGTH:LENGTH+21])
10|counterparty  |   File "/home/adam/counterparty-core/counterparty-lib/counterpartylib/lib/address.py", line 56, in unpack
10|counterparty  |     if enabled('segwit_support') and short_address_bytes[0] >= 0x80 and short_address_bytes[0] <= 0x8F:
10|counterparty  | IndexError: index out of range
10|counterparty  | During handling of the above exception, another exception occurred:
10|counterparty  | Traceback (most recent call last):
10|counterparty  |   File "/home/adam/.local/bin/counterparty-server", line 8, in <module>
10|counterparty  |     sys.exit(server_main())
10|counterparty  |   File "/home/adam/counterparty-core/counterparty-cli/counterpartycli/__init__.py", line 20, in server_main
10|counterparty  |     server.main()
10|counterparty  |   File "/home/adam/counterparty-core/counterparty-cli/counterpartycli/server.py", line 230, in main
10|counterparty  |     server.start_all(catch_up=args.catch_up)
10|counterparty  |   File "/home/adam/counterparty-core/counterparty-lib/counterpartylib/server.py", line 543, in start_all
10|counterparty  |     blocks.follow(db)
10|counterparty  |   File "/home/adam/counterparty-core/counterparty-lib/counterpartylib/lib/blocks.py", line 823, in follow
10|counterparty  |     new_ledger_hash, new_txlist_hash, new_messages_hash, found_messages_hash = parse_block(db, block_index, block_time)
10|counterparty  |   File "/home/adam/counterparty-core/counterparty-lib/counterpartylib/lib/blocks.py", line 200, in parse_block
10|counterparty  |     raise e
10|counterparty  |   File "/home/adam/counterparty-core/counterparty-lib/counterpartylib/lib/blocks.py", line 196, in parse_block
10|counterparty  |     parse_tx(db, tx)
10|counterparty  |   File "/home/adam/counterparty-core/counterparty-lib/counterpartylib/lib/blocks.py", line 161, in parse_tx
10|counterparty  |     raise exceptions.ParseTransactionError(f"{e}")
10|counterparty  | counterpartylib.lib.exceptions.ParseTransactionError: index out of range
10|counterparty  | Unhandled Exception

Ugh this is a problem also with v9.61.1:
[2024-04-10 02:49:41][INFO] Running v1.1.5 of counterparty-server.
[2024-04-10 02:49:41][INFO] Running v9.61.1 of counterparty-lib.
[2024-04-10 02:49:41][INFO] Acquiring lock.
[2024-04-10 02:49:41][INFO] Connecting to database (SQLite 3.24.0-r1).
[2024-04-10 02:49:41][INFO] Connecting to backend.
[2024-04-10 02:49:41][INFO] AddrIndexRs connecting...
[2024-04-10 02:49:41][INFO] Connected to AddrIndexRs!
[2024-04-10 02:49:41][INFO] Starting API Server.
[2024-04-10 02:49:51][INFO] Resuming parsing.
[2024-04-10 02:49:57][WARNING] ParseTransactionError for tx 255930ade36ea75bb1c9373417fe627bc8e9f484da1489e8535ae9fb59adfa86: index out of range
[2024-04-10 02:49:57][ERROR] Unhandled Exception
Traceback (most recent call last):
  File "/counterparty-lib/counterpartylib/lib/blocks.py", line 129, in parse_tx
    dispenser.parse(db, tx, message)
  File "/counterparty-lib/counterpartylib/lib/messages/dispenser.py", line 262, in parse
    action_address = address.unpack(message[LENGTH:LENGTH+21])
  File "/counterparty-lib/counterpartylib/lib/address.py", line 54, in unpack
    if enabled('segwit_support') and short_address_bytes[0] >= 0x80 and short_address_bytes[0] <= 0x8F:
IndexError: index out of range

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/bin/counterparty-server", line 11, in <module>
    load_entry_point('counterparty-cli', 'console_scripts', 'counterparty-server')()
  File "/counterparty-cli/counterpartycli/__init__.py", line 16, in server_main
    server.main()
  File "/counterparty-cli/counterpartycli/server.py", line 172, in main
    server.start_all(db)
  File "/counterparty-lib/counterpartylib/server.py", line 494, in start_all
    blocks.follow(db)
  File "/counterparty-lib/counterpartylib/lib/blocks.py", line 1483, in follow
    new_ledger_hash, new_txlist_hash, new_messages_hash, found_messages_hash = parse_block(db, block_index, block_time)
  File "/counterparty-lib/counterpartylib/lib/blocks.py", line 211, in parse_block
    raise e
  File "/counterparty-lib/counterpartylib/lib/blocks.py", line 205, in parse_block
    parse_tx(db, tx)
  File "/counterparty-lib/counterpartylib/lib/blocks.py", line 148, in parse_tx
    raise exceptions.ParseTransactionError("%s" % e)
counterpartylib.lib.exceptions.ParseTransactionError: index out of range

Fixed here: https://github.com/CounterpartyXCP/counterparty-core/pull/1620


Clean up doc comments for APIs so documentation page looks right

This issue is to track updates we should do to the doc comments so that results look like

[ ] Part types should not list the types they're mapping, but rather have that as part of the doc for the member itself. For example see https://review.learn.microsoft.com/en-us/dotnet/api/documentformat.openxml.packaging.imageparttype?view=openxml-3.0.1&branch=smoke-test
[x] PresetTextWrap should be PresetTextWarp in https://learn.microsoft.com/en-us/dotnet/api/documentformat.openxml.drawing.presettextwrap?view=openxml-2.8.1

for #2: link to corrected topic is https://learn.microsoft.com/en-us/dotnet/api/documentformat.openxml.drawing.presettextwarp?view=openxml-3.0.1&viewFallbackFrom=openxml-2.8.1


clarity on fusion interface

Lets say you have two adapters, rte and mnli.  Now you want to train rte and fusion together, using mnli as a frozen adapter.
What is the correct protocol to achieve this?  Something like this?
model.train_adapter(['rte'])
fusion_adapters = ['rte','mnli']
fusion = ac.Fuse(fusion_adapters)
model.add_adapter_fusion(fusion)
model.set_active_adapters(fusion)
also, not sure it matters, but on the latest development branch

As I understand, you would like train both the fusion layer and the RTE adapter. If this is the case, the steps to achieve look a bit different:
First, add and activate AdapterFusion for training:
fusion_adapters = ['rte','mnli']
fusion = Fuse(fusion_adapters)
model.add_adapter_fusion(fusion)
model.train_adapter_fusion(fusion)

Note that train_adapter_fusion() only "unfreezes" the fusion layer, but not the adapters. Thus, we have to activate the rte adapters separately. As train_adapter() would change the activated setup, you can use a small trick:
model.apply_to_adapter_layers(lambda i, layer: layer.enable_adapters(Stack("rte"), True, False))

Now, the fusion layer and the rte adapter are activated for training, while the mnli is kept frozen.
Hope this helps!


html: Support passing custom_template as string.

Will be used in DVCLive for https://github.com/iterative/dvclive/issues/460

Codecov Report
Base: 96.04% // Head: 96.54% // Increases project coverage by +0.49% :tada:

Coverage data is based on head (79fb691) compared to base (d0023df).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #118      +/-   ##
==========================================
+ Coverage   96.04%   96.54%   +0.49%
==========================================
  Files          19       19
  Lines         683      694      +11
  Branches      100      100
==========================================
+ Hits          656      670      +14
+ Misses         22       20       -2
+ Partials        5        4       -1

Impacted Files
Coverage Δ

src/dvc_render/html.py
93.87% <100.00%> (+6.37%)
:arrow_up:

tests/test_html.py
100.00% <100.00%> (ø)

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

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


Possible duplicate files in debug build

In
https://github.com/flutter/flutter/blob/99e201e2ef3f8e996c9320feffa513007b67a4d8/packages/flutter_tools/lib/src/build_system/targets/ios.dart#L467
and
https://github.com/flutter/flutter/blob/99e201e2ef3f8e996c9320feffa513007b67a4d8/packages/flutter_tools/lib/src/build_system/targets/android.dart#L56
, the build system copies Artifact.vmSnapshotData and Artifact.isolateSnapshotData from the host artifacts to the assets during the debug build mode. However, in dart_snapshot.cc, it shows that these files are not used under the Android platform, while they are used under the iOS platform.
This copying action appears to be redundant for Android, as the files are not used. On the other hand, for iOS, the build system relies on the snapshot build for the host platform.
A potential problem arises if iOS enables the compressed pointers feature in the future. The host snapshot may become incompatible with the snapshot generated for iOS, potentially causing offset errors or other issues.
Edit: The snapshot files can be removed for iOS too, they can be found by symbol mapping, and the version in symbol are compiled for target platform.

Proposal to Optimize Snapshot Handling in Flutter's Build System for Android and iOS
Problem Statement
In the Flutter build system, there is a redundancy where Artifact.vmSnapshotData and Artifact.isolateSnapshotData are copied to assets during the debug build mode for both Android and iOS platforms. However, these snapshot files are not utilized on Android, making the copying action unnecessary. Furthermore, there is a potential compatibility issue for iOS if compressed pointers are enabled in the future, as the host-generated snapshots may not be compatible with the snapshots required by iOS, leading to offset errors or other issues.
Solution Proposal

Remove Redundant Copying for Android:

Skip copying Artifact.vmSnapshotData and Artifact.isolateSnapshotData for Android during the debug build mode, as they are not used.

Ensure Compatibility for iOS:

Introduce a mechanism to ensure that the snapshots generated are compatible with the iOS platform, especially if compressed pointers or other features are enabled in the future.

Detailed Steps

Update Android Build System:

Modify the build script for Android to exclude the copying of snapshot data files.

// flutter/packages/flutter_tools/lib/src/build_system/targets/android.dart
// Remove or comment out the line copying vmSnapshotData for Android
final String vmSnapshotData = environment.artifacts.getArtifactPath(Artifact.vmSnapshotData, mode: BuildMode.debug);

Enhance iOS Build System:

Ensure the iOS build system checks for compatibility and handles snapshot generation appropriately.

// flutter/packages/flutter_tools/lib/src/build_system/targets/ios.dart
final String vmSnapshotData = environment.artifacts.getArtifactPath(Artifact.vmSnapshotData, mode: BuildMode.debug);

Add a check to ensure compatibility when generating the snapshot.

// Ensure that the snapshot generation considers iOS-specific configurations.
if (environment.platform == TargetPlatform.ios) {
  // Code to handle compatibility check or generation logic
}

Potential Issues and Mitigations

Compatibility Check:

Ensure that any changes do not affect the functionality of the build system. Thorough testing on both Android and iOS is necessary.

Future-proofing for iOS:

Implement a mechanism to detect changes in the snapshot generation process (e.g., enabling compressed pointers) and adjust the build process accordingly.

Documentation and Communication:

Clearly document the changes and communicate them to the development community to avoid confusion and ensure smooth transitions.

Conclusion
By removing the redundant copying of snapshot files for Android and ensuring compatibility for iOS, we can optimize the Flutter build system, reducing unnecessary actions and preparing for future changes. This approach will streamline the build process and mitigate potential issues arising from future iOS features.

cc @cbracken


Support optional remote signer in kosud

Overview
For optimal security, Tendermint recommends using an external signer process on production chains for validators. This prevents double signing, and reduces (or eliminates) the risk of the keys  being compromised.
Background
This is a really good primer on the topic!
The PrivValidator interface provides signing functionality for Tendermint validators. The simplest is FilePV (used by go-kosu currently) which uses priv_validator_state.json and priv_validator_key.json.
Currently, we create our in-process tendermint node with a custom constructor, CreateNode.
It assumes usage of FilePV:
node, err := node.NewNode(
		app.Config,
		pv.LoadOrGenFilePV(                                // <= FilePV!
			app.Config.PrivValidatorKeyFile(),
			app.Config.PrivValidatorStateFile(),
		),
		nodeKey,
		proxy.NewLocalClientCreator(app),
		node.DefaultGenesisDocProviderFunc(app.Config),
		node.DefaultDBProvider,
		node.DefaultMetricsProvider(app.Config.Instrumentation),
		logger,
	)

Requirements

The kosud CLI should provide the option to configure a remote signer, by configuring a SignerListenterEndpoint and SignerClient
The SignerClient becomes the PrivValidator passed to node.NewNode()
The default should be FilePV (as it stands now)

For example (just an idea):
// not exactly sure what is required by SignerListenerEndpoint, more flags may be needed
kosud start -E [URL] --remote-signer "<IP_ADDRESS>:8000"

Testing
We may have to write a simple remote signer implementation in order to properly test.

This has been partially implemented in #320. We still need to wait for Tx signing support.

Do you think we close this issue since technically the implementation satisfies the above? Or leave it open since its not fully compatible with a live network?

Since the functionality is not there yet, I'd keep it open. And close it when enabled.


CHANGELOG

We can start adding a changelog info in the future once we have a stable version. I suggest we add it in the README just below the version.


Add deprecation warning to apk-install

Proposal to deprecate apk-install now that there is an official alpine Docker image that does not include this helper script. Dropping apk-install will encourage more consistent usage of the canonical method of installing packages. It minimizes the differences between official and Glider Labs images, which is good for (new) users and good for maintainers.
This also puts pressure on a more proper solution to removing the cache, such as adding a --no-cache option to apk add upstream, as opposed to maintaining a minimal script in a non-official image.

No comments? Should we merge this?

Can you remove apk-install from the docs? If it is going to be deprecated then we shouldn't mention it.

:+1:

Not sure why we should learn the canonical way to install packages. If that's the reason, we could also remove the package manager and learn how to compile from source (exaggerating here).
I'm a big fan of apk-install, instead of apk add --update <packages> && rm -rf /var/cache/apk/*, and was looking forward to more snippets in this repo to simplify package management and/or other routinely used tasks.
In the same vein, we created blendle/docker-base to help us extract common tasks to a higher level.
Obviously, we can simply move this to our own repo, but just voicing my opinion here that apk-install was one of the reasons to move to gliderlabs/alpine (amongst many others).

Thanks for the feedback. Anybody else?
On Mon, Jun 1, 2015 at 2:31 AM, Jean Mertz<EMAIL_ADDRESS>wrote:

Not sure why we should learn the canonical way to install packages. If
that's the reason, we could also remove the package manager and learn how
to compile from source (exaggerating here).
I'm a big fan of apk-install, instead of apk add --update  &&
rm -rf /var/cache/apk/*, and was looking forward to more snippets in this
repo to simplify package management and/or other routinely used tasks.
In the same vein, we created blendle/docker-base
https://github.com/blendle/docker-base to help us extract common tasks
to a higher level.
Obviously, we can simply move this to our own repo, but just voicing my
opinion here that apk-install was one of the reasons to move to
gliderlabs/alpine (amongst many others).
—
Reply to this email directly or view it on GitHub
https://github.com/gliderlabs/docker-alpine/pull/18#issuecomment-107340985
.

--
Jeff Lindsay
http://progrium.com

i like apk-install myself and would prefer to keep it if it's not too much trouble.
i can also see the appeal of an option, such as apk add --update --clean-cache <foo>, in upstream.

:heart: apk-install. I'll leave this open for a bit longer. But I would prefer to go along with the community. So far, it sounds like a helper people do like and use.

I don't use apk-install, but it would be nice if --clean-cache option will go to Alpine upstream.

I'm chiming in to say that even though we've only been using this Alpine image for a month because previously we were Ubuntu users, I think the script serves no purpose that most don't already do manually themselves, we never even noticed this existed in the first place and we only need to clean the cache once so it really adds no benefit.

How about adding a --no-cache option in upstream apk to make it not save any file on disk in the first place?

@ncopa --no-cache would work fine for packages, but from this name I don't expect APKINDEX.tar.gz would be cleaned as well. I do understand that this behaviour is not usual for standalone Alpine installation though, but nevertheless.

@frol better suggestions for a flag that disables all caching, including APKINDEX.tar.gz?

added request upstream: http://bugs.alpinelinux.org/issues/4905

@ncopa How about using two flags? --no-cache + --no-local-index?

@ncopa Another option might be --no-footprint.

We ended up with --no-cache for this feature. It got accepted upstream: http://git.alpinelinux.org/cgit/apk-tools/commit/?id=c43bfed8deaa0dab47c54db9b8f374853d345a6b
This will be available from v3.3.0_rc2.

@ncopa Will this still cache .apk files? Reading the help text "Read uncached index from network" I assume it won't. If so, it is not a replacement for apk-install yet.

apk will not cache .apk files by default. To enable .apk file caching you need create /etc/apk/cache symlink pointing to cache directory. So nothing will get written in any cache dir.
This should be slightly faster too as it reduces disk IO.

@ncopa This sounds great! Thank you!

Rebase and push if we still want to add this warning.


Publish viewer

The last stable version of the viewer should be published somewhere publicly. I propose:
https://viewer.playcanvas.com/
One cool thing we could do then, is add a right click context menu option for GLB files in the Editor Asset panel 'View...' which will launch the viewer in a new tab (or maybe an iframe) to leverage some of the viewers functions. But that'd be a new feature request in the Editor repo. 😄

If viewer.playcanvas.com will be used to view Editor's assets, then there is some considerations for authentication and session, to comply with assets access permissions.
Although viewer should probably work anonymously as well. Simple and straight forward.

Hmm. Well, we already have:
https://playcanvas.com/editor
So maybe:
https://playcanvas.com/viewer
...is better. And would presumably not have the authentication/access problem of a new sub-domain.

Done, get it at https://playcanvas.com/viewer!


How to include header after connected

We know the package can connect to server via -c option and include header via -H option on connecting.
But, How can we include header after connection?

You can't, headers can only be specified during the WebSocket opening handshake. After that headers no longer exist in the WebSocket protocol.


Using Hardware4*.java code in another project

I'm one of the developers on the Oshi project, providing cross-platform JNA-based hardware information.
A user has requested CPU serial number and I found this project after reading your reply to this StackOverflow post.
I'd love to just copy your Hardware4*.java files straight over to my project, with perhaps some minor modifications, but want to respect your licensing.  Is it sufficient to include your name as author with our EPL license?     Are there any other ways to get your files (releasing a jar, publishing on Maven central, etc.) without copying the source?  Would you be interested in contributing a PR yourself?

Hi @dbwiddis, feel free to do what you need to do :) It's MIT so you have a wide spectrum of possible choices. If you want to avoid copying source code I can release it to Maven Central even now.

@dbwiddis, just a note. Many things happen after I first wrote this piece of code and it seems like some functionalities I was depending on in the past are now deprecated (e.g. lack of HAL on Ubuntu makes lshal unavailable) making it hard to get SN without a sudo :( On Windows things does not happen to change but I'm not sure about Apples. Therefore it may be hard to have portable solution if you decide to use secure-tokens code in your project.

Yeah, it's not so much using the code as the formalities of copying the license wording exactly.  Just wanted to run it by you first.  I actually already have a util function to do commandlines so I'm mostly only stealing the idea I hadn't thought about, which was a series of fallback attempts.  dmidecode first, lshal second, possibly cpuid.    And I'll probably replace your exceptions with default values.   I'll probably add in MAC address listing as well.


Implemented the duplication of text based tabs. Fixes microsoft/vscode#25563

The commit does the following:

Adds a new Duplicate entry to the active tab context menu when is a text editor (it also comes with a ⌘K + ⌘U keyboard shortcut).
When a tab is duplicated a new Tab is created with the same text, language and cursor position.

Hi @bpasero, this is my first pull request to the Visual Studio Code project and I'd like to understand what does the backlog milestone represent for this team. Is this a place were tickets go indefinitely? Or is there a plan and criteria to merge these requests into master?
I'm mostly trying to understand how long it could be before I can start using the feature without having to compile from the source.

@PedroC88 I would like to understand how this pull request implements anything that has been asked for. When I understand #25563 correctly, people want a command that duplicates a file physically on disk and then trigger the rename so that you just have to type the name. Your PR however seems to do something different?

@bpasero  I did my change based on a personal need and looked through the issues to see if someone had reported it already. I may have misunderstood the OP on his request and, if that's the case, then you're right in that my pull request doesn't fix his problem. My solution isn't about physical files, but about duplicating the active tab within the workspace. I have reached out to the OP in #25563 to confirm whether my solution relates to his request or if a new feature-request should be created. Please allow a few days for him to get back at me so I can confirm.
Assuming I may need to create a new feature-request... What would I need to do to link the pull request to that issue?
P.S. I see you're the most active person developing VSCode so I apologize for taking so much of your time in guiding a noob, I'm just trying to make my changes as clean as possible.

@PedroC88 have you considered doing an extension to provide this command? I think everything would be there from the API side. If something can be done as extension we typically prefer that.

@bpasero I guess I could try creating my own extension or forking mssql but I have really never coded an extension and it would mean basically starting over and relearning VSCode from an extension perspective. I would also prefer it if VS supported the ability to duplicate a tab out of the box as I'm guessing this would be useful for more people around the world and not just me.
Would it be too complicated to check my code and merge it to master? (Note that I'm willing to improve on any feedback that you guys provide).

@PedroC88 we have very good documentation for extension authoring: https://code.visualstudio.com/docs/extensions/overview


IoT is missing support for public key infrastructure features #2079

list_things, register_ca_certificate, register_certificate, describe_ca_certificate, list_certificates_by_ca, delete_ca_certificate, update_ca_certificate, get_registration_code
Without this APIs, it's hard to mock a service via moto when someone gets device certificate signed by their own certificate authority vs relying in AWS IoT default CA.
Implemented above APIs.
Resolving #2079

Coverage decreased (-30.7%) to 62.149% when pulling 8277c8a9c1b7b94a033ee26f843162d999165afc on sahilkoladiya:iot_ca_support_6 into 09855801ba323326db1a745c4ddd6178749d95d4 on spulec:master.

It looks like this was replaced with #2089


Convert reStructuredText to Markdown

Initial version by @Caznix, fixups by @GeckoEidechse
Co-authored-by: Cazdotsys<EMAIL_ADDRESS>

Merging based on the fact that after prompting the involved people with enough time to respond there were no requests for changes ^^


fail2ban log and unban

I'm running a znc instance as a semi-official bouncer for the engineers and developers at work, where we use an internal IRC server as our main communication method. We're running znc behind a F5 LTM hardware load balancer, and most of our users are using irssi as their client.
I'm seeing issues where all of a sudden nobody can connect to znc (current connections are OK), everyone's client gets the "ERROR :Closing link [Please try again later - reconnecting too fast]" message. My guess is that this is because of the load balancer configuration, znc sees all clients as having the same IP address.
While a reconfiguration of the load balancer is required to fix this, I'd really like 2 additions to the fail2ban module:

an optional log file of every denied connection
an admin command to reset the ban for a client, or remove a client from the ban list.

Want to back this issue? Place a bounty on it! We accept bounties via Bountysource.

Ok, so #411 is useless due to my lack of C++ knowledge.
Any thoughts on this, or update on #32? We're running a common ZNC instance at our company for our internal IRC network, and currently have ~45 users on the ZNC instance. When everyone is in the office and has the same source IP, and someone's client is misbehaving, fail2ban kicks everyone off. Some sort of logging (or even better, unban) is desperately needed...

I think this is not releated to fail2ban, but "anonymous IP limit" in webadmin and increasing it to something (lot higher) might help.

We've tried disabling fail2ban and it fixes it. The problem is that sometimes someone types the wrong password into their IRC client, and if their client is configured to reconnect too fast, fail2ban trips. Our office is behind NAT, so ZNC sees everyone in the office as coming from the same IP.
The more general issue is, why dosn't fail2ban log when it bans an IP, and why isn't there a way to either manually unban an IP, or to whitelist allowed IPs...

Our office is behind NAT, so ZNC sees everyone in the office as coming from the same IP

whitelist allowed IPs

This will render fail2ban useless, you either have this only IP whitelisted, or not.

I was thinking that maybe some VPN solution between the office and ZNC could work.

@DarthGandalf Sorry if I was unclear about that... our office is behind NAT. Our ZNC instance is hosted offsite, and publicly available on the Internet so that we can connect from our phones, laptops, etc. I'd be fine with whitelisting just our office IP, our real concern (why we're running fail2ban) is if anyone in the outside world happens to discover our ZNC instance.


AndroidX deprecation and protobuf error

I keep having this issue with AndroidX and my projects will not compile, has anyone been able to figure out a solution?
Note: C:\dev\flutter.pub-cache\hosted\pub.dartlang.org\flutter_blue-0.6.3+1\android\src\main\java\com\pauldemarco\flutter_blue\FlutterBluePlugin.java uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: C:\Users\anegri\dev\flutter\certificate\bmi-calculator\build\flutter_blue\generated\source\proto\debug\javalite\com\pauldemarco\flutter_blue\Protos.java uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
Running Gradle task 'assembleDebug'...
Running Gradle task 'assembleDebug'... Done                         7.1s
Gradle build failed to produce an .apk file. It's likely that this file was generated under
C:\Users\anegri\dev\flutter\certificate\bmi-calculator\build, but the tool couldn't find it.

I think you need to enable Jetifier. My graddle properties are:
android.enableR8=false
android.useAndroidX=true
android.enableJetifier=true

This should be resolved in the latest version.

@pauldemarco confirmed in 0.7.2, the problem is not resolved.
Launching lib\main.dart on Android SDK built for x86 in debug mode...
Note: C:\flutter\.pub-cache\hosted\pub.dartlang.org\flutter_blue-0.7.2\android\src\main\java\com\pauldemarco\flutter_blue\FlutterBluePlugin.java uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: C:\Users\user\Documents\GitHub\flutter-boilerplate\build\flutter_blue\generated\source\proto\debug\javalite\com\pauldemarco\flutter_blue\Protos.java uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
√ Built build\app\outputs\apk\debug\app-debug.apk.
I/FlutterBluePlugin(25553): setup
Connecting to VM Service at ws://<IP_ADDRESS>:50881/gzCe4B4xrjY=/ws
D/EGL_emulation(25553): eglMakeCurrent: 0xaab85480: ver 2 0 (tinfo 0xaab835a0)

@pauldemarco confirmed in 0.7.2, the problem is not resolved.
Launching lib\main.dart on Android SDK built for x86 in debug mode...
Note: C:\flutter\.pub-cache\hosted\pub.dartlang.org\flutter_blue-0.7.2\android\src\main\java\com\pauldemarco\flutter_blue\FlutterBluePlugin.java uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: C:\Users\user\Documents\GitHub\flutter-boilerplate\build\flutter_blue\generated\source\proto\debug\javalite\com\pauldemarco\flutter_blue\Protos.java uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
√ Built build\app\outputs\apk\debug\app-debug.apk.
I/FlutterBluePlugin(25553): setup
Connecting to VM Service at ws://<IP_ADDRESS>:50881/gzCe4B4xrjY=/ws
D/EGL_emulation(25553): eglMakeCurrent: 0xaab85480: ver 2 0 (tinfo 0xaab835a0)

Same as you, I can't compile. What can we do?
I have same as @pauldemarco
android.enableR8=false
android.useAndroidX=true
android.enableJetifier=true
Nothing works

Same here, the deprecation message is still there.


fix: add state to controlled components

Issue #, if available:
Description of changes:

adding state to controlled components so they get updated with the default value
selectField as a default placeholder before the state is updated

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Approved, though not sure why you need a placeholder for SelectField

Approved, though not sure why you need a placeholder for SelectField

the placeholder is there to indicate to customers they need selection an option as we don't want to assume which default enum to set for the react state

Codecov Report

Merging #649 (459a15e) into tagged-release/forms-and-views (b911878) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@                       Coverage Diff                       @@
##           tagged-release/forms-and-views     #649   +/-   ##
===============================================================
  Coverage                           90.20%   90.20%
===============================================================
  Files                                  97       97
  Lines                                3614     3615    +1
  Branches                              970      970
===============================================================
+ Hits                                 3260     3261    +1
  Misses                                329      329
  Partials                               25       25

Impacted Files
Coverage Δ

packages/codegen-ui-react/lib/forms/form-state.ts
96.15% <ø> (ø)

...codegen-ui-react/lib/forms/form-renderer-helper.ts
94.30% <100.00%> (+0.04%)
:arrow_up:

...lib/generate-form-definition/helpers/form-field.ts
98.82% <100.00%> (ø)

Continue to review full report at Codecov.

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

Approved, though not sure why you need a placeholder for SelectField

the placeholder is there to indicate to customers they need selection an option as we don't want to assume which default enum to set for the react state

Got it


Bullet point-ed, added content to Mem&Attn Nets

Fixed internal hyperlinks in ToC
Added Augmented RNN in Memory and Attention Networks
Converted M&A Networks section from free flow to bullet points for readability

thanks!

Happy to help clean and update this repository @keon!
Will try to add more recent packager from facebook/FAIR, and papers from NIPS 2017 and EM-NLP in relevant sections.