changelog: drop dev release

It's just confusing for ~all users

While it is confusing, we've already published a 0.3.1-dev on pub, so I'd keep the section.

While it is confusing, we've already published a 0.3.1-dev on pub, so I'd keep the section.

Yep. But it'll ALWAYS be here: https://pub.dev/packages/web/versions/0.3.1-dev/changelog
For 99.999% of folks, it's just confusing to see the dev release


[bug]can't get head notch

MediaQuery.of(context).padding.top; return 0.0 even iPhone X

Is it too early to call this method?
At the beginning, the screen data has not been obtained,
This is a problem with flutter itself


Automated EKCO version update 0.21.0

Automated changes by the cron-ekco-update GitHub action
  Adds [EKCO add-on](https://kurl.sh/docs/add-ons/ekco) version 0.21.0.

This is not ready to go out

Will close in favor of https://github.com/replicatedhq/kURL/pull/3440


Cart Issue : Items added to the cart while signed in will be refreshed when you sign out and then sign back in.

Items added to the cart while signed in will be refreshed when you sign out and then sign back in.
Reproduce with the following steps:

Sigin in the User
Put some items in a cart
Sign out User
Sign in same User
See the cart -> there is no items which was put on step2

Cause:
On the step3 (Sign out User), cart will be newly created as guest user. On the step4 newly created cart will be used for signed in user.
Ideal behavior:
On the step4,

If there are no items in guest cart and there are items in signed user cart, signed user cart will be used
If there are items in guest cart and there are items in signed user cart, pop up shown which asks which is used between guest cart and signed user cart

Commenting as this is an interesting issue I'd like to try taking on. The general concept of cart persistence and merging across guest and logged-in sessions is a common feature in modern e-commerce.
Issue:
We need to define and implement the expected behavior for the shopping cart when users interact with it across different states: as guests, logged-in users, and when transitioning between these states (e.g., adding items as a guest and then logging in, or logging out while having items in the cart).
Acceptance Criteria:

Cart Merging Logic: Ensure that when a user logs in after using a guest cart, the items in both carts are merged, with quantity updates for duplicate items.
Persistent Cart Across Devices: Investigate and potentially implement a feature to keep the cart synchronized across multiple devices and browsers when users log in or out.
Guest Cart Expiry: Define how long guest carts should persist in case the session is inactive and how to handle cart expiry gracefully.


DocumentDB v0.9

*Issue #, if available: Custom lens WA for DocumentDB
Description of changes:
v0.9
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Would you remove .DS_Store file from the commit?


mmcv and mmdet compatibility?

➜  3d-multi-resolution-rcnn git:(master) ✗ CUDA_VISIBLE_DEVICES=0 ./tools/dist_train.sh configs/3d-multi-resolution-rcnn.py 1 --validate

~/.local/lib/python3.8/site-packages/torch/distributed/launch.py:178: FutureWarning: The module torch.distributed.launch is deprecated
and will be removed in future. Use torchrun.
Note that --use_env is set by default in torchrun.
If your script expects `--local_rank` argument to be set, please
change it to read from `os.environ['LOCAL_RANK']` instead. See
https://pytorch.org/docs/stable/distributed.html#launch-utility for
further instructions

  warnings.warn(
~/.local/lib/python3.8/site-packages/numpy/core/getlimits.py:499: UserWarning: The value of the smallest subnormal for <class 'numpy.float64'> type is zero.
  setattr(self, word, getattr(machar, word).flat[0])
~/.local/lib/python3.8/site-packages/numpy/core/getlimits.py:89: UserWarning: The value of the smallest subnormal for <class 'numpy.float64'> type is zero.
  return self._float_to_str(self.smallest_subnormal)
~/.local/lib/python3.8/site-packages/numpy/core/getlimits.py:499: UserWarning: The value of the smallest subnormal for <class 'numpy.float32'> type is zero.
  setattr(self, word, getattr(machar, word).flat[0])
~/.local/lib/python3.8/site-packages/numpy/core/getlimits.py:89: UserWarning: The value of the smallest subnormal for <class 'numpy.float32'> type is zero.
  return self._float_to_str(self.smallest_subnormal)
Traceback (most recent call last):
  File "./tools/train.py", line 7, in <module>
    from mmdet.datasets import get_dataset
  File "~/.local/lib/python3.8/site-packages/mmdet/datasets/__init__.py", line 3, in <module>
    from .cityscapes import CityscapesDataset
  File "~/.local/lib/python3.8/site-packages/mmdet/datasets/cityscapes.py", line 17, in <module>
    from .coco import CocoDataset
  File "~/.local/lib/python3.8/site-packages/mmdet/datasets/coco.py", line 16, in <module>
    from mmdet.core import eval_recalls
  File "~/.local/lib/python3.8/site-packages/mmdet/core/__init__.py", line 3, in <module>
    from .bbox import *  # noqa: F401, F403
  File "~/.local/lib/python3.8/site-packages/mmdet/core/bbox/__init__.py", line 8, in <module>
    from .samplers import (BaseSampler, CombinedSampler,
  File "~/.local/lib/python3.8/site-packages/mmdet/core/bbox/samplers/__init__.py", line 10, in <module>
    from .score_hlr_sampler import ScoreHLRSampler
  File "~/.local/lib/python3.8/site-packages/mmdet/core/bbox/samplers/score_hlr_sampler.py", line 3, in <module>
    from mmcv.ops import nms_match
  File "~/.local/lib/python3.8/site-packages/mmcv-1.4.2-py3.8.egg/mmcv/ops/__init__.py", line 2, in <module>
    from .assign_score_withk import assign_score_withk
  File "~/.local/lib/python3.8/site-packages/mmcv-1.4.2-py3.8.egg/mmcv/ops/assign_score_withk.py", line 5, in <module>
    ext_module = ext_loader.load_ext(
  File "~/.local/lib/python3.8/site-packages/mmcv-1.4.2-py3.8.egg/mmcv/utils/ext_loader.py", line 13, in load_ext
    ext = importlib.import_module('mmcv.' + name)
  File "/usr/lib/python3.8/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
ModuleNotFoundError: No module named 'mmcv._ext'
ERROR:torch.distributed.elastic.multiprocessing.api:failed (exitcode: 1) local_rank: 0 (pid: 3285279) of binary: /usr/bin/python
Traceback (most recent call last):
  File "/usr/lib/python3.8/runpy.py", line 194, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/usr/lib/python3.8/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "~/.local/lib/python3.8/site-packages/torch/distributed/launch.py", line 193, in <module>
    main()
  File "~/.local/lib/python3.8/site-packages/torch/distributed/launch.py", line 189, in main
    launch(args)
  File "~/.local/lib/python3.8/site-packages/torch/distributed/launch.py", line 174, in launch
    run(args)
  File "~/.local/lib/python3.8/site-packages/torch/distributed/run.py", line 710, in run
    elastic_launch(
  File "~/.local/lib/python3.8/site-packages/torch/distributed/launcher/api.py", line 131, in __call__
    return launch_agent(self._config, self._entrypoint, list(args))
  File "~/.local/lib/python3.8/site-packages/torch/distributed/launcher/api.py", line 259, in launch_agent
    raise ChildFailedError(
torch.distributed.elastic.multiprocessing.errors.ChildFailedError:
============================================================
./tools/train.py FAILED
------------------------------------------------------------
Failures:
  <NO_OTHER_FAILURES>
------------------------------------------------------------
Root Cause (first observed failure):
[0]:
  time      : 2022-01-06_12:52:47
  host      : lvision-MS-7C59
  rank      : 0 (local_rank: 0)
  exitcode  : 1 (pid: 3285279)
  error_file: <N/A>
  traceback : To enable traceback see: https://pytorch.org/docs/stable/elastic/errors.html
============================================================

Hi, did you ever successfully run training and inference with custom dataset?


I want to set base_directory or prefix for uploaded file or image

I want to have prefix uploaded file to "/site/" because my framework in site folder. because my public in "../public_html/", the problem is if i set base_directory it make folder in framework but not in public.

I can fix it with set base_direcctory = '../site/public' . thank you

I have same problem, ../../ doesnt works

Duplicate of #506 .


Fix warning for Algolia SSR with React 18

I'm unfortunately not able to fix this easily. It also seems to be related to a reported issue at the Algolia repo.  The code seems to work fine even with the warning, and the warning is only visible server side, so it will be visible in our Radix logs for the newsroom.
My suggestion will be to move this to Dependent on others column, and see if Algolia is able to sort this out. In the meantime, I have changed the code to use react-instantsearch-hooks-web  instead of react-instantsearch-hook as the newly documented Algolia examples suggest. The main difference is that the web library includes the work in progress React components along with the hooks, so it will create a better foundation for using Algolia components when we continue the work for the newsroom. :)

This is an issue with React 18 and hasn't been fixed upstream yet:
https://github.com/algolia/react-instantsearch/issues/3441
https://github.com/facebook/react/issues/22796
Don't think there's anything we can do on our end with this yet - but it shouldn't cause any issues to have the warning.

Any news about this one @SvSven ?

Someone should have a look to see if it is already fixed

[InstantSearch] The package react-instantsearch-hooks is replaced by react-instantsearch-core. The API is the same, but the package name has changed. Please update your dependencies and follow the migration guide: https://www.algolia.com/doc/guides/building-search-ui/upgrade-guides/react/
[InstantSearch] The package react-instantsearch-hooks-router-nextjs is replaced by react-instantsearch-router-nextjs. The API is the same, but the package name has changed. Please update your dependencies and follow the migration guide: https://www.algolia.com/doc/guides/building-search-ui/upgrade-guides/react/
[InstantSearch] The package react-instantsearch-hooks-server is replaced by react-instantsearch. The API is the same, but the package name has changed. Please update your dependencies and follow the migration guide: https://www.algolia.com/doc/guides/building-search-ui/upgrade-guides/react/
[InstantSearch] The package react-instantsearch-hooks-web is replaced by react-instantsearch. The API is the same, but the package name has changed. Please update your dependencies and follow the migration guide: https://www.algolia.com/doc/guides/building-search-ui/upgrade-guides/react/


es2015-fy readme

I've just walked by and noticed a typo in readme. So I fixed it.
Converted usage examples to es2015 by the way.

Excellent. Thank you :)


Add ListOptions to ListWatched

Supoprt #238

I fixed it!

@willnorris  ping.
Can we get feedback on this PR?

made above changes I mentioned and merged as c57bb02.  Thanks!


Backend for new recursive joins

This is an ongoing PR. The details, and accompanying issues and further optimizations, will be written soon.

I addressed all your comments. I think you can take over now and merge.

Also, forgot to mention. There are 2 TODOS for you in on_disk_graph.h.
// TODO(Reviewer): Do I need to do other.graphEntry.copy() in the constructor?
// TODO(Reviewer): I believe I have to make this copy constructor public. Can you double check?" before merging and taking care of those TODOs.

Can you take care of those and remove those todos before merging?


Can the size of each mask be adjusted?

I use the SIM super-resolution dataset to train Resnet50 because SIM images lack high-level semantic information and have low inter-block correlation, resulting in poor reconstruction quality. I think it may be necessary to have smaller mask blocks. For example, if the default size of each mask block in the code is (32, 32), is there a hyperparameter that can be adjusted to (16, 16)? Or do I need to modify the code myself, and if so, in which part of the code specifically?

@pouletdatrid u can refer to our https://github.com/keyu-tian/SparK/tree/main/pretrain#some-details-how-we-mask-images-and-how-to-set-the-patch-size


Rewrite from scratch!

It's time to bring this project back to life. The first thing I want to do is remove some of the dependencies (or all of them—we'll see). Also on the table is bumping fish requirements to 2.3.
This time my primary focus is improving response time (the length of time taken for fish_prompt and fish_right_prompt to complete and show you the prompt) and adding powerful features that don't get in your way, e.g., better git support.
In terms of design and options, I want to be able to switch between different themes: Powerline, Classic, etc., and customize the color palette (as opposed to customizing individual glyph colors resulting in several variables with similar names).
Ultimately, I want a minimal, fast, zero-dependency, non-obtrusive, aesthetically-pleasing prompt that is easy to hack on, but not overblown with configuration options.
Stay tuned. 👋

A fork of Metro exists, which works with current versions of Fisherman: https://github.com/fishpkg/fish-prompt-metro

@fdcds Basically, yeah. @fishpkg is a vessel for fish packages I built that are now in long term maintenance. Active development is not happening anymore, but I review and accept pull requests for features or bug fixes.
This repository will be replaced by a new prompt that I'm working on, I just haven't gotten around to finish it. 😄

Waiting.


If a preload is highlighted by wrong file name, renaming the file does not update the script editor

Godot 3.2 alpha2
Say you have this in your script:
const MyShader = preload("./../my_shader.shader")

If the file name is actually the_shader.shader, it will cause the line to be highlighted red and produce an error.
Now if you rename your file to be my_shader.shader, it should match the script. However, the script editor still shows the error and only updates if you do an edit.
It probably also happens in the opposite case, where a path is initially correct but a rename makes it wrong.

I guess renaming a file should cause the current script to be parsed and checked for errors again.

Still valid in 6c173e2
Opposite case is bugged too, i.e. preloading a valid resource and then renaming the file doesn't produce error.


Fix front tag utils issues caused by tag not found

Hi! When trying to process several XML JATS files (generated by CERMINE), I found several issues in some functions in front_tags_utils.py. All the issues were caused because some tag could not be found. I guess that the logic thing if a tag is not found is to return an empty string, and that's what I've implemented. Please let me know if this is correct :)

lgtm, merging, thanks!


chore: use EDC M5 version

What this PR changes/adds
Uses the latest EDC M5 version in local builds and CI checkout.
ALso fixes a broken test.
Checklist

[ ] added appropriate tests?
[ ] performed checkstyle check locally?
[ ] added/updated copyright headers?
[ ] documented public classes/methods?
[ ] added/updated relevant documentation?
[ ] added relevant details to the changelog? (skip with label no-changelog)
[ ] formatted title correctly? (take a look at the CONTRIBUTING and styleguide for details)

Suggesting an alternative fix for the broken test: https://github.com/agera-edc/IdentityHub/pull/45

Suggesting an alternative fix for the broken test: agera-edc#45

@marcgs that is fine, we can take that. Lets merge that first, and I'll rebase my code onto it.

@marcgs @cpeeyush due to a pending PR in EDC upstream it is better to indeed merge this variant of the fix, as it will accommodate the aforementioned upstream PR as well.


Add support for Foyer forcefields

Description
Add support for Foyer forcefields
Todos

[x] Add ForceFieldSource and build system protocol
[x] Add tests

Questions

[ ] When running a test enthalpy of vaporization calculation I get NotImplementedError() in assign_parameters_gas

Status

[ ] Ready to go

Wow, this is a nice surprise!
Make sure you add foyer into the test environment.
Don't worry about the OpenEye parts of the matrix, but you can force the tests to run in those cases by adding if: always() to that step in the workflow.

The traceback looks like it's not coming from a development/editable install - does from openff.evaluator import __file__; print(__file__) point to your clone of this repo?

Yes, I pip installed it to test it with the production script

The error I'm seeing in tests:

openff.interchange.exceptions.UnsupportedCutoffMethodError: When using openmm.CustomNonbondedForce, vdW and electrostatics cutoff methods must agree on whether or not periodic boundary conditions should be used.OpenMM will throw an error. Found vdw method 1, and electrostatics method 0,

usually stems from the periodicity not being set. If Topology.box_vectors is set, which I assume it would be after the PACKMOL protocol finishes, then Interchange.box should also be set. Unless the error bubbles out from a system that is not periodic, in which case you're running up against a nasty confluence of corner cases in which the correct behavior is undefined (https://github.com/openforcefield/standards/issues/7). We're hoping to resolve this within a month or so (we have a meeting in two weeks about it) but as a short-term stop-gap, you could just set the gas-phase calculations to have a large (say, 5 nm or greater) periodic box. If you choose this option please note it in a comment in the code so it can be reverted later.

@aehogan I think this is ready to go? Is there anything more you were looking to do, and have you been able to use this in your studies? If there aren't issues you've ran into, I think I'll give this another review and look to merge it into a release soon. How does that sound to you?
The changeset is smaller than I originally anticipated, but thinking through it again, it shouldn't be a surprise that it doesn't need to do much more than swap out the force field source.

@mattwthompson There are still one or two more issues with running a full delta H vaporization simulation I need to iron out.
Besides that I was going to change out the xml test with a methane so that part of the code is actually tested.

Sounds good, let me know when you think it's ready - no rush

@mattwthompson should be good now

Awesome work!

Thanks! Delta H vaporization, delta H mixing and densities have all run successfully now.


Expose usage metrics for Realtime model

Handles #1010  to include usage (number of input/output tokens) in the response_done event for realtime model
Conforms to the usage object model as shown at https://platform.openai.com/docs/api-reference/realtime-server-events/response/done at time of submission

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

Hey, ruff checks seems to fails.
Can you run ruff format && ruff check --fix?


Hide #nav-tree scrollbars for chromium browsers

The ::-webkit-scrollbar CSS extension will hide the scrollbars without disabling #nav-tree scrolling ability.
https://user-images.githubusercontent.com/83611172/129077130-d71d2fa4-2116-4f06-8109-fd71951a3579.mp4

First of all: Thanks for you contribution!
I'm not sure if I like that, though. While it looks a lot cleaner for sure, it hides the critical information that the view is scrollable. 🤔
I'll think about it... Maybe as an option that is disabled by default?

First of all: Thanks for you contribution!
I'm not sure if I like that, though. While it looks a lot cleaner for sure, it hides the critical information that the view is scrollable. 🤔
I'll think about it... Maybe as an option that is disabled by default?

Thank you for your feedback. Instead, I suggest the following extensions: -webkit-scrollbar, -webkit-scrollbar-thumb and -webkit-scrollbar-track. Below is a non-tweaked sample to demonstrate the result. You may adjust it to represent doxygen-awesome-css light and dark color scheme and padding. 🙂
https://developer.mozilla.org/en-US/docs/Web/CSS/::-webkit-scrollbar
nav-tree
#nav-tree::-webkit-scrollbar {
  width: 10px;
  height: 10px;
}

#nav-tree::-webkit-scrollbar-thumb {
  background: rgb(90 90 90);
}

#nav-tree::-webkit-scrollbar-track {
  background: rgba(1, 0, 0, 0);
}

code:
code::-webkit-scrollbar, div.fragment::-webkit-scrollbar, pre.fragment::-webkit-scrollbar {
  width: 10px;
  height: 10px;
}

code::-webkit-scrollbar-thumb, div.fragment::-webkit-scrollbar-thumb, pre.fragment::-webkit-scrollbar-thumb {
  background: rgba(90, 90, 90);
}

code::-webkit-scrollbar-track, div.fragment::-webkit-scrollbar-track,  pre.fragment::-webkit-scrollbar-track {
  background: rgba(1, 0, 0, 0);
}

global:
- chrome bug: The browser vertical scroll is rendered invisible. Requires JS and the above #nav-tree to avoid.
- https://stackoverflow.com/a/66069228

::-webkit-scrollbar {
  height: 10px;
}

::-webkit-scrollbar-thumb:horizontal {
  background: rgba(90, 90, 90);
}

::-webkit-scrollbar-track:horizontal {
  background: rgba(1, 0, 0, 0);
}

Styling the scrollbars in a way that was satisfactory to me was harder than expected.
While the changes that you propose fix the scrollbars on Windows/Linux, they have the side effect that the default scrollbars on macOS are also overwritten, which IMO leads to a worse experience on the platform.
I've added an experimental separate stylesheet that looks and behaves similar to the macOS scrollbars, to better blend in on the platform while also looking good on Window/Linux: doxygen-awesome-fancy-scrollbars.css
Let me know what you think about it!
https://user-images.githubusercontent.com/21294002/152699943-0f74677a-8177-4c85-89f8-25708bca9cab.mov
And I've additionally fixed Firefox scrollbars as well! 😊  Scrollbar-styling on Firefox is limited but it's better than nothing:

(Don't mind the other visual changes in the video/image, what you see is WIP of some other cool stuff that I am currently working on! 😊 )


Proceed to the main page even if the controllers do not respond

Disable non-responding controller tab in this case

Or #7 and abort


LIQUIDITY SYSTEM.

Describe the bug

why when I try to add a liquidity balance in the KLAYTN / USDT token I get a notification as shown which I include below
and I don't think the liquidity system is like other DEXs that automatically detect token balances
How to reproduce
Steps to reproduce the behavior:
Expected behavior
maybe you can change the liquidity system and Earn system a little
making it easier and less transaction fees so that novice investors interested in investing a little money
Attachments
If applicable, add data, logs or screenshots to help explain your problem.
Environment (please complete the following information)

klaytn-dex-frontend release version that exhibits the issue
OS and its version [e.g. macOS/10.14]
Wallet and its version used [e.g. Kaikas, metamask]

Additional context
Add any other context about the problem here.

@Cynwon You should have no problem adding KLAY/USDT Liquidity combo now!
Thanks


fix: preventing creating tmp dir

BREAKING CHANGE: You need to have a folder called static in the root folder of your container

Is it possible to restore this test?
I assume, we can't create a static dir in the test before Spring starts, but the workdir of the test should be the project root. So we may be able to provide the directory in the codebase. Otherwise it may be an option to adjust the test to accept new directories with docdir in it but no other ones.


Fix PNG images being loaded without premultiplying its alpha channels

PNG images are guaranteed by the file standards to not contain premultiplied alpha channels, and use straight alpha channels. As a result, PNGs will be rendered wrongly as the un-premultiplied alpha channels are used to render the image.
This PR fixes LoadImage. It will now check if the loaded image is a PNG and if so, premultiplies the alpha channels, then flags the image as having a premultiplied alpha channel. If images that are not PNG, it will assume that they are already premultiplied and simply flag them as premultiplied. This PR also adds two shards PremultiplyAlpha and DemultiplyAlpha for manually premultiplying the alpha and demultiplying alpha of an image.
Some refactoring is also done for the premultiplying and demultiplying of the alpha channels. LoadImage's code for loading byte inputs and files have also been streamlined in the process.
Note: Checking image file type for PNG can be done according to the specifications @ http://www.libpng.org/pub/png/spec/1.2/PNG-Structure.html
Additional list of file signatures: https://en.wikipedia.org/wiki/List_of_file_signatures
Fixes #671

Also, you need to move the premultiple/unpremultiply template functions to a header (can add imaging.hpp) if you want to use them in serialization.cpp

CI should be fixed now, seems to have been an issue with visibility of the implemented functions within the header file

@Yttruire why this is still not merged? cc @guusw

@Yttruire Can you add some add a test case for PremultiplyAlpha & DemultiplyAlpha in the image script (or a new script)


mCS vs. Ember

I've caught a problem. Any version of mCS later than 3.0.0 is not initialize correctly in Ember.

ember -v
version: 1.13.8
node: 0.10.32
npm: 2.13.4

i'm using v3 with ember 1.13.8 also 2.1 without issue.

Ok, I've got something. It conflicted with defiantjs v1.2.6.


Add Go modules

This greatly improves the experience for people wanting to build the project from source.

Bumping this. Let's drop godep and get this using mainline modules!

+1


DateField - allow the selection of specific weekdays and disable the other weekdays

Description of the bug
I'd like to have a way to tell the DateField that only specific days can be selected. For our use case, we would like to specify that only Sundays can be selected.
Playroom reproduction
No response
Version
0.16.4
What browsers are you seeing the problem on?
No response

Hi,
it is a feature not available yet that I would like to have

Hi @FreedAlberto, this has been released in version 0.17.3!


draft: Add invocation_args_to_dict to ProviderContext

resolves #5524
Description

Added a invocation_args_to_dict contextproperty to the ProviderContext class.
Added a test in test_context.py using a MacroContext
Added a test in test_builtin_functions that prints {{invocation_args_to_dict}} to the logs. Again using a MacroContext

Checklist

[X] I have read the contributing guide and understand what's expected of me
[X] I have signed the CLA
[X] I have run this code in development and it appears to resolve the stated issue
[X] This PR includes tests, or tests are not required/relevant for this PR
[X] I have opened an issue to add/update docs, or docs changes are not required/relevant for this PR
[ ] I have run changie new to create a changelog entry

@gshank the Structured Logging Schema check failed but the integration tests pass when I run them locally.
I was looking into using --log-format-json in the test itself:
    def test_builtin_invocation_args_to_dict_function(self, project):
        _, log_output = run_dbt_and_capture(
            [
                "--debug",
                "--log-format=json",
                "run-operation",
                "validate_invocation",
                "--args",
                "{my_variable: test_variable}",
            ]
        )
        # Result is checked in two parts because profiles_dir is unique each test run
        parsed_logs = []
        for line in log_output.split("\n"):
            try:
                log = json.loads(line)
            except ValueError:
                continue

            parsed_logs.append(log)

And then running the assert on the parsed_logs list of dicts.
What are your thoughts on this?


Extract nodes from trained Decision Tree

Hi,
I'm trying to extract the nodes learned from a classification decision tree model in order to export them to some UI and visualize them, unfortunately it seems that there is no way to access them from the Model class.
Did I miss something?
If not, is there any plan to let the user access the DecisionTreeTablePtr, as it is already possible for other models such as the betas of a linear model or the centroids of a kmeans model?
Thanks

Current version of Intel DAAL hides the implementation of decision tree model, so you cannot visualize its contents.
However, the decision forest algorithm provides NodeVisitor feature that gives an ability to traverse the Model.
In the upcoming release we plan to introduce node visitor feature in decision tree algorithm as well.
This example shows the usage of node visitor in decision forest algorithm.

Thanks for your answer, looking forward to having access to the NodeVisitor for decision trees.


Fix get_window_safe_area on Android

Fixes: #61467
metrics removed, was not used in the function.
Test: https://github.com/godotengine/godot/issues/61467#issuecomment-1875902228

Cherry-picked for 4.2.2.


Windows without HW virtualization doesn't work

Is there a way to set docker-machine to be 32 bit by default instead of 64bit? It fails silently on Kitematic initial startup.

+1
I'm looking at using kitematic as a containerised app launcher for distance education use, which means that we're always for looking to be able to configure s/ware distributions for lowest common denominator, naive user, use.

KM needs to check for VT-X, and in the bigger picture, docker-machine may need to allow creating VMs with VT-X disabled.
Thanks

some type of virtualization is needed at this time. Sorry.


Fix Benchmark CI run

Resolves https://github.com/facebookincubator/velox/issues/1210

@mbasmanova This should resolve the issue. The CI still seems to fail since the main branch does not have this change.
The linux-build failures are unrelated.

The linux-build failures are unrelated.

@majetideepak Deepak, would you rebase on top to main. https://github.com/facebookincubator/velox/pull/1209 should allow linux-build CI job to pass.

This should resolve the issue.

Curious how did you verify that these changes fix ci/circleci: linux-benchmarks-basic job.

@mbasmanova The CI job said that the executable was missing. I verified that. This change fixes the issue. I see that the benchmark executable is built and present as expected.


[CSL-2379] Use OOP in wallet spec

This uses the approach describes in http://www.well-typed.com/blog/2018/03/oop-in-haskell/ to model the wallet. I think it worked out very well, the relation between the various models is now much clearer.
This closes https://iohk.myjetbrains.com/youtrack/issue/CSL-2379.

I think we should merge this as-is. Added a small sanity check to make sure stuff was working as expected.

@edsko We just need to please the CI (I think it's the usual business of regenerating pkgs/default.nix and then we're ready to hit that merge button methinks 😉


docs: add Windows 2022 node image release notes for 2024-6OOB

What type of PR is this?
/kind documentation

What this PR does / why we need it:
add Windows 2022 node image release notes for 2024-6OOB. Reference: https://github.com/Azure/AgentBaker/pull/4541
Which issue(s) this PR fixes:

Fixes #
Requirements:

[x] uses conventional commit messages

[x] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2627

Relevant Lines:
3458

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2627

Relevant Lines:
3458

💛  - Coveralls


update references to github org

the github org has been renamed. links to github repos that reference the old org get redirected properly and still work. however, github.io links that reference old org do not redirect and no longer work as expected. all references to previous org should be updated the new org to resolve broken URLs

@DTAIEB @mikebroberg this pull request contains updates to all references from the old github org to the new github org except for those in /docs and /docsrc (didnt want to step on Mike's toes).


api: add /debug and /metrics

What problem does this PR solve?

Issue Number: close #425
Problem Summary:
Currently, we are using /api/debug and /api/metrics, which are inconsistent with other components. Thus, TiDB Operator, TiUP, and Dashboard need to handle it specially.
What is changed and how it works:
Add /debug and /metrics.
Check List
Tests

[ ] Unit test
[ ] Integration test
[x] Manual test (add detailed scripts or steps below)
[ ] No code

zhangmingdeMacBook-Pro:tiproxy zhangming$ curl http://127.1:3080/debug/pprof/goroutine > g
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  5137    0  5137    0     0   584k      0 --:--:-- --:--:-- --:--:-- 2508k
zhangmingdeMacBook-Pro:tiproxy zhangming$ curl http://127.1:3080/metrics > m
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100    44  100    44    0     0   6075      0 --:--:-- --:--:-- --:--:-- 44000
zhangmingdeMacBook-Pro:tiproxy zhangming$ curl http://127.1:3080/api/debug/pprof/goroutine > g
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  4980    0  4980    0     0   563k      0 --:--:-- --:--:-- --:--:-- 2431k
zhangmingdeMacBook-Pro:tiproxy zhangming$ curl http://127.1:3080/api/metrics > m
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100    48  100    48    0     0   7313      0 --:--:-- --:--:-- --:--:-- 48000

Notable changes

[ ] Has configuration change
[ ] Has HTTP API interfaces change
[ ] Has tiproxyctl change
[ ] Other user behavior changes

Release note

Please refer to Release Notes Language Style Guide to write a quality release note.
None

/retest

/retest


feat: improve e2e test environment

This PR aims to solve the problem submitted by #81

Let APISIX manage the go-runner in the e2e test runtime, just like the production environment;
Remove unused HTTPS port.

Codecov Report

Merging #82 (4486aa5) into master (160e058) will decrease coverage by 1.74%.
The diff coverage is n/a.

:exclamation: Current head 4486aa5 differs from pull request most recent head 7357c87. Consider uploading reports for the commit 7357c87 to get more accurate results

@@            Coverage Diff             @@
##           master      #82      +/-   ##
==========================================
- Coverage   81.23%   79.49%   -1.75%
==========================================
  Files          11       11
  Lines         746      746
==========================================
- Hits          606      593      -13
- Misses        102      114      +12
- Partials       38       39       +1

Impacted Files
Coverage Δ

internal/server/server.go
56.91% <0.00%> (-10.57%)
:arrow_down:

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 160e058...7357c87. Read the comment docs.

Please make the CI pass, thanks!

Please make the CI pass, thanks!

I am trying~

@Chever-John Do you still have time to complete this PR?

@soulbird
I think this PR will not make sense now.
It had been a long time.
I will close this PR.

@soulbird I think this PR will not make sense now. It had been a long time. I will close this PR.

ok


Is there a plan to update the release?

The code has been updated 6 months ago but the release is still back in 2022. Could we see an updated release that we can run?

Honestly, I thought I did when I made those changes, I built it but never uploaded them. This is mostly for an internal library that I maintain at my job, so I did not think anyone would actually use it. I'll upload a release when I get a chance to tomorrow.

Please do! I'm trying to create a stub generator for my codebase as well since our Python code interfaces with our C# code and yours seem to handle a lot. Would you be able to update the readme as well? Kinda confused on how to run this.

Release updated


Check for partitionable slots when determining if VM idle

Partitionable slot1@ will always show unclaimed idle even when they have dynamic slots under them that are busy. Need to check for presence of partitionable / dynamic slots and make sure all slots idle before allowing idle time to be set to prevent shutdown before KeepAlive time is up.

fixed a logic error - https://github.com/hep-gc/cloud-scheduler/commit/9d0667c78ca0b212dd26f0c6cec4a11774509e54
vm wasn't being flagged as found and then was regarded as having finished retiring.


chore: add gitter integrations

Adds ci and codecov integrations to gitter.

Current coverage is 91.79% (diff: 100%)

Merging #651 into master will not change coverage

@@             master       #651   diff @@
==========================================
  Files           179        179
  Lines          1987       1987
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           1824       1824
  Misses          163        163
  Partials          0          0

Powered by Codecov. Last update 4e40c2b...7d8a1c5


Unable to deploy along with single spoke (nonHA) deployment -  "Gateway with dummy does not exist"

Env:
Aviatrix nonHA spoke deployment (mc-spoke) + NAT (mc-overlap-nat-spoke)
Module 1.1.1
TF  1.5.2
Controller 7.1.x
Issue:
Following error noted during Terraform run :
Error: failed to configure policies for 'customized_snat' mode due to: rest API edit_gw_customized_snat_config Post failed: Gateway with name dummy does not exist
│
│   with module.spoke1_nat[0].aviatrix_gateway_snat.gw_2[0],
│   on .terraform/modules/spoke1_nat/main.tf line 43, in resource "aviatrix_gateway_snat" "gw_2":
│   43: resource "aviatrix_gateway_snat" "gw_2" {
│
╵
╷
│ Error: failed to update DNAT for gateway(name: ) due to: rest API update_dnat_config Post failed: following parameters are required: gateway_name, policy_list
│
│   with module.spoke1_nat[0].aviatrix_gateway_dnat.dnat_rules_gw2[0],
│   on .terraform/modules/spoke1_nat/main.tf line 118, in resource "aviatrix_gateway_dnat" "dnat_rules_gw2":
│  118: resource "aviatrix_gateway_dnat" "dnat_rules_gw2" {
│

Observations:
Issue with single spoke (non HA); the NAT resource blocks for gw2 still being parsed, HOWEVER, the single spoke snat and dnat is provisioned correctly,   though ERROR above is still seen.
Deploying  spokegw HA works fine.

The 'test-non-ha' branch  change helped deploy  Avtx spoke + nat module without errors as before.
HOWEVER,  if a 2nd 'terraform apply' is run, the same error is reported:
(1)
module.spoke1_nat[0].aviatrix_gateway_dnat.dnat_rules_gw2[0]: Creating...
╷
│ Error: failed to configure policies for 'customized_snat' mode due to: rest API edit_gw_customized_snat_config Post failed: following parameters are required: gateway_name
│
│   with module.spoke1_nat[0].aviatrix_gateway_snat.gw_2[0],
│   on .terraform/modules/spoke1_nat/main.tf line 43, in resource "aviatrix_gateway_snat" "gw_2":
│   43: resource "aviatrix_gateway_snat" "gw_2" {
│
╵
╷
│ Error: failed to update DNAT for gateway(name: ) due to: rest API update_dnat_config Post failed: following parameters are required: gateway_name, policy_list
│
│   with module.spoke1_nat[0].aviatrix_gateway_dnat.dnat_rules_gw2[0],
│   on .terraform/modules/spoke1_nat/main.tf line 118, in resource "aviatrix_gateway_dnat" "dnat_rules_gw2":
│  118: resource "aviatrix_gateway_dnat" "dnat_rules_gw2" {
│

2
If the variables.tf file for the NAT module is set as below (same error):
locals {
is_ha = (var.spoke_gw_object.ha_gw_size != null) || (var.spoke_gw_object.ha_gw_size != "")
}
3
if the variables.tf file for the NAT module is set with this:
locals {
is_ha = var.spoke_gw_object.ha_gw_size != ""
}
'Terraform apply works without any issues and shows 'no changes'

Hi,
An update on this issue, the following modification tested using the 'test-non-ha' branch and results all good,  can this be merged to main branch or ok to push to 'test-non-ha' branch as well?
variables.tf  >  locals declaration.
Replaced :  " is_ha = var.spoke_gw_object.ha_gw_size != null"
With :  "  is_ha = (var.spoke_gw_object.ha_gw_size == "") || (var.spoke_gw_object.ha_gw_size == null) ? "false" : "true""
Tests:
(A)
(1) Test spoke HA + nat - "github.com/terraform-aviatrix-modules/terraform-aviatrix-mc-overlap-nat-spoke.git?ref=test-non-ha"
RESULT:    deploys
module.spoke1_nat[0].aviatrix_gateway_dnat.dnat_rules_gw1[0]
module.spoke1_nat[0].aviatrix_gateway_dnat.dnat_rules_gw2[0]
module.spoke1_nat[0].aviatrix_gateway_snat.gw_1
module.spoke1_nat[0].aviatrix_gateway_snat.gw_2[0]
module.spoke_azure_211.aviatrix_spoke_gateway.default
module.spoke_azure_211.aviatrix_spoke_transit_attachment.default[0]
module.spoke_azure_211.aviatrix_vpc.default[0]
(2) Modify (nat_attached = false) and re-apply
RESULT:  deploys
module.spoke_azure_211.aviatrix_spoke_gateway.default
module.spoke_azure_211.aviatrix_spoke_transit_attachment.default[0]
module.spoke_azure_211.aviatrix_vpc.default[0]
(3) Modify (nat_attached = true) and re-apply
RESULT:  deploys
module.spoke1_nat[0].aviatrix_gateway_dnat.dnat_rules_gw1[0]
module.spoke1_nat[0].aviatrix_gateway_dnat.dnat_rules_gw2[0]
module.spoke1_nat[0].aviatrix_gateway_snat.gw_1
module.spoke1_nat[0].aviatrix_gateway_snat.gw_2[0]
module.spoke_azure_211.aviatrix_spoke_gateway.default
module.spoke_azure_211.aviatrix_spoke_transit_attachment.default[0]
module.spoke_azure_211.aviatrix_vpc.default[0]
(B)
*** adding ***
is_ha = (var.spoke_gw_object.ha_gw_size == "") || (var.spoke_gw_object.ha_gw_size == null) ? "false" : "true"
(1) Test spoke nonHA + nat - "github.com/terraform-aviatrix-modules/terraform-aviatrix-mc-overlap-nat-spoke.git?ref=test-non-ha"
RESULT:
module.spoke1_nat[0].aviatrix_gateway_dnat.dnat_rules_gw1[0]
module.spoke1_nat[0].aviatrix_gateway_snat.gw_1
module.spoke_azure_211.aviatrix_spoke_gateway.default
module.spoke_azure_211.aviatrix_spoke_transit_attachment.default[0]
module.spoke_azure_211.aviatrix_vpc.default[0]
(2) Test spoke nonHA remove NAT and re-apply
RESULT: OK
data.aviatrix_spoke_gateway.morf
module.spoke_azure_211.aviatrix_spoke_gateway.default
module.spoke_azure_211.aviatrix_spoke_transit_attachment.default[0]
module.spoke_azure_211.aviatrix_vpc.default[0]
(3) Only re-apply
RESULT:  OK


docs: install source-map-explorer as a dev dependency

@ianschmitz @iansu @mrmckeb please let me know if you have any feedback on this PR, otherwise it is ready to be merged 😊 thanks

Hi @tobiasbueschel, thanks for your time on this.
The current stance on this from the team is that we just install all dependencies the same way, as the distinction a) doesn't affect bundle size, and b) can cause issues on some CIs that automatically install dependencies only.
So for simplicity, we can just leave this one as is :)


bson MODULE_NOT_FOUND error on startup

I've seen this since I've started working on the Gambs

2016-08-05T22:29:15.692225+00:00 app[web.1]: { [Error: Cannot find module '../build/Release/bson'] code: 'MODULE_NOT_FOUND' }

SO: http://stackoverflow.com/questions/28651028/cannot-find-module-build-release-bson-code-module-not-found-js-bson
One user suggests simply upgrading to 3.8.23 fixes: http://stackoverflow.com/a/30570552/1470725, we're on 3.8.8

Tried upgrading to 3.8.39 and running npm install but still see the errors locally:

7:56:10 AM web.1 |  { [Error: Cannot find module '../build/Release/bson'] code: 'MODULE_NOT_FOUND' }
7:56:10 AM web.1 |  js-bson: Failed to load c++ bson extension, using pure JS version

Updating Mongoose to v4 didn't do the trick, as some threads suggest.  Not too worried about this one - labelling as NO


future training: area<=0 check for ann_file parsing, dataset.py will not load bbox only

I'm picking up the code from dataset.py and noticed all my custom bounding boxes were coming up empty.
This is b/c of the check on 71 in dataset.py:
           if ann['area'] <= 0 or w < 1 or h < 1:                 continue
area is apparently only computed from segmentation masks and not bounding boxes...thus if you have a custom dataset with no seg masks, your bboxes won't be loaded.
I think just having the w and h check is sufficient and avoids people hitting this when training on custom datasets.
https://github.com/cocodataset/cocoapi/issues/36

@lessw2020 good point, I'll make a note to fix that. BTW, if you haven't been following, the code on the training branch is working, sort of... I managed to find a few hours to adapt some training code from my image models repo for this task...
So far it has

distributed training with apex AMP + sync-bn support
basic augmentations matching the TF automl code
ability to turn on EMA weight averaging

Major missing/broken things,

need to release timm with a 'momentum' opt matching tf.train.MomentumOptimizer (SGD w/o Nesterov enabled by default)
proper validation metric, currently just loss, no full coco eval or fast mAP calc being done
saving the wrong part of model right now, saving full bench, need to save just .model
full reproduction of automl level model accuracy, first crack only hit .26 mAP, this seems on par with other 3rd party attempts, but not good enough. It is most likely due to low LR caused by stability issues, I think I've fixed the stability issue and currently training at higher LR

Hi @rwightman,
Thanks for the update - that's great news as I'm full swing into setting up custom detector using EffDet and hoping to move from Signa over to your model soon :)
A couple points that may or may not be useful for you:
1 - I've been using augmentations from albumentations github with good success in terms of improving results (via loss) and they have a nice pipeline for chaining things together with OneOf and various level probabilities.
2 - Re: training and opt - I recently added gradient centralization to Ranger and saw a nice jump in obj detection training with it.  I'm also testing SPS right now and added in gradient centralization there and saw good success, so the net is you may want to try gc for improving results.  (not part of TF training but it's a nice improvement in most things I've tested it on.  )
3 - Proper loss metric would be a huge help.  Related, would you consider integrating Distance and/or complete IOU loss for the bbox regression?  https://github.com/Zzh-tju/DIoU
I haven't tested but it appears to be superiour than traditional regression setup.
Thanks for the progress and updates and feel free to ping me if there are specific things I might be able to help out on.
btw - do you envision this supporting ONNX or similar so it could be ported to TFLite for mobile there?  I've been working in TF directly and really not liking it but do need to be able to produce a mobile TFLite version for my work.
Hope you are doing well!

@lessw2020 extra goodies like DIoU etc will come in the experimentation phase, right now still trying to finish off the 'replicate official' baseline. I looked at albumentations but it seemed challenging to fit their defaults to replicate the Google baseline, also, their bbox format diff than the yxyx google, and they use lists of np array instead of one fat numpy array for some reason.
For experiments I have an impl of EvoNorm on deck too, hopefully will make it possible to train the larger models without having access to clusters of V100s. Heh.

@rwightman  - great to see your progress on the training branch!  EvoNorm sounds great.  I'm going to get setup with your training branch tomorrow and start riding along and hopefully can help a bit.
Yes albumentations has some oddities but the flip side is I will say it's very fast and has a lot of good augs though arguably most can be implemented pretty fast directly.
Anyway, l greatly appreciate your progress on this and will get synced up tomorrow with the training branch.
Hope you are doing well!


Why is WebHook setup such a pain?

I've been bashing myself for hours with a simple Echo-style (in the examples folder) bot via webhook, and can't get it to work.
This is my code (I did a fresh npm install of this Bot API repo just 12 hours ago)
const
TelegramBot = require('node-telegram-bot-api'),
TOKEN = '<MY_BOT_TOKEN_FROM_BOTFATHER>',
opts = {
  webHook: {
    port: 443,
    key: `${__dirname}/YOURPRIVATE.key`,
    cert: `${__dirname}/YOURPUBLIC.pem`
  }
},
bot = new TelegramBot(TOKEN, opts);

bot.setWebHook(`https://<MY_IP_ADDRESS>/bot${TOKEN}`, {
	certificate: opts.webHook.cert
})

bot.on('message', msg => {
	console.log(`ChatType: ${msg.chat.type}\nChatId: ${msg.chat.id}`)
	bot.sendMessage(msg.chat.id, "Message received!")
})

I generated the self-signed certificate with this (from core.telegram.org/bots/webhooks):
openssl req -newkey rsa:2048 -sha256 -nodes -keyout YOURPRIVATE.key -x509 -days 365 -out YOURPUBLIC.pem -subj "/C=US/ST=New York/L=Brooklyn/O=Example Brooklyn Company/CN=<SAME_IP_ADDRESS_IN_SETWEBHOOK>"

Follwowing this (core.telegram.org/bots/webhooks), I've added firewall rules to accept 443 (https) and 8443 ports. AND YET.... STILL NOTHING!!! I get no error messages, no crashes, absolutely nothing! Any help will be GREATLY appreciated.

I did an npm install github:yagop/node-telegram-bot-api (which is essentially install the MASTER BRANCH) and now everything works great! Please publish latest on npm so no one goes through this pain.


PLAN-934 Hide session selection and rank based on person status

I don't think this is the right approach - i might go a different route with it


Support for Swift 4.1 or greater

Is there a plan to support Swift 4.0 or greater?
It's possible if I have time that I create a pull request into this project and fix things, if needed.

I'll add support to swift 4.0 over this week
If there's new features or fixes you wish to add, feel free to create a pull request

Support is added, feel free to try it out and let me know if there are any issues


[FLINK-16560] Forward Configuration in PackagedProgramUtils#getPipelineFromProgram

Before, when using PackagedProgramUtils (for example in the standalone cluster entrypoint or the web ui) the Flink Configuration would not be applied to the execution environment.
This also adds a test that verifies that we forward configuration.
Brief change log

change OptimizerPlanEnvironment and StreamPlanEnvironment constructors to accept a Configuration and forward to the correct super constructor.

Verifying this change

see the newly added test

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: indirectly, because we now correctly forward also deployment options
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no

Thanks for your review! I addressed your comments and merged.


Persist Log string

@qinweigong @Sarkar
Persist the log when for log requests, so they can be restored (potentially causing nil in dictionary).

:+1:

looks good :+1:


chore: update README, add code of conduct

Wave one of doc updates for archivist.
In progress are usage docs for archivistctl and some more tweaks to this structure

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
0

Relevant Lines:
0

💛  - Coveralls


Implement Rewrite Headers support during routing

The final piece to supporting Rewrites is implementing support for overwriting response headers. These overwritten headers need to be collected until we resolve into a final ContentSourceContent::Static (which gets turned into ResolveSourceRequestResult, then GetFromSourceResult) to be returned into the HTTP response.
I chose to implement this as an external list from the headers already present in a ContentSourceContent::Static. It felt wrong to content.await? just to overwrite the returned headers, and then repackage that as a new ContentSourceContentVc. These header_overwrites exist only as a byproduct of route resolution, and specifically the exact path of rewrites we go through to get there. They're not intrinsic to the ContentSourceContent itself.
Fixes WEB-617
Fixes WEB-299

I'm working on the test cases for Rewrite/Redirect/Headers now.

Tests implemented for Rewrite/Redirect, but the Headers support just landed in Next side and I can't test it until we update our canary dep.

I've manually confirmed this works once by duplicating https://github.com/vercel/next.js/commit/185a8921c878fb6fff46d9fc299683ee689d3415.

Merging over some test flake.

Merging over some test flake.

kodiak could handle this now. It won't block on mac or windows CI anymore.


Add the MMDiT model of Stable Diffusion 3

This follows the implementation in ComfyUI.
Smoke test is passed with dumped input and output tensors from a running ComfyUI SD 3 workflow.
Some extra works need to be done to complete a running SD3 example with candle, including a slight modification of the two CLIP encoders used for SDXL. The comfy implemetation of SD3 takes penultimate layer of CLIP L/G as embedding, instead of the final layer output (see sd3_clip.py and sdxl_clip.py). Also, SD3 VAE might need some effort to make/tweak.

Thanks!


Add support for custom styles/themes

It would be nice to match the styling of Overview (font family, colors) with the overall Storybook theme.
API proposal
addDecorator(withOverview({
  typedoc,
  theme: myTheme,
}));

Good idea! Will look into this on my next round of changes 👍

Opened an initial PR for this #10

Hi @South-Paw, will this PR be merged anytime soon?
In the mean time, is there any way to control the look and feel of it?
My application has a dark navy background, the default font color is black which makes quite unreadable. I've been trying to remove the bg color from my inner iframe from storybook without success. Do you have any idea how would I be able to achieve that?
Cheers!

Hi @minuz, I'll try to find some time to merge this in the next few days 👍

If #15 can be finished off and merged first then we just need to add the extra styles for the .signature class to this before merging it.


Move annotation to kotlin

Currently, @InjectMock and @InjectSpy are java annotation. This could be migrated to kotlin for simplicity.

Fixed in #13


[#3] Create base application layout

https://github.com/rosle/react-nimble-survey/issues/3
What happened 👀

Add new route - /sign_in

Add base layout styles

Default Layout: For the main application pages
Auth Layout: For the authentication pages

Set up localization namespaces - shared and auth

Insight 📝
@import
For Styling, I decided to revert back to @import instead of @use and forward. Because when we use @use/@forward, it will apply on that specific file only. So imagine if we want to use variables on file A and file B, we need to @use/@forward on both files.
React Helmet Async
I use React Helmet to modify the layout className. But there are errors with the new version of React. See issue 548 on the Repo

https://reactjs.org/blog/2018/03/27/update-on-async-rendering.html
So now, need to move to use React Helmet Async instead
Proof Of Work 📹

Default Layout: https://62f0ef1bf3c82a5549ad4033--react-nimble-survey-staging.netlify.app

Auth Layout: https://62f0ef1bf3c82a5549ad4033--react-nimble-survey-staging.netlify.app/sign_in

🚀 Deployed on https://62fb000b2013e72cc17b5eed--react-nimble-survey-staging.netlify.app

I cannot access to this preview 🫣

https://62fb000b2013e72cc17b5eed--react-nimble-survey-staging.netlify.app

How does it display on your end, @Lahphim? Could you provide me the screenshot? 😆🙏


PT biases in v3

The usage of the new PT module would be as follows:
cosmo = ccl.CosmologyVanillaLCDM()

# Initialize a perturbation theory calculator
ptc = ccl.nl_pt.EulerianPTCalculator(with_NC=True, cosmo=cosmo)

# If you need to update the PT P(k) kernels to a new cosmology:
ptc.update_ingredients(cosmo)

# Now you can calculate biased power spectra, e.g.:
tg = ccl.nl_pt.PTNumberCountsTracer(b1=1.0, b2=0.5)
tm = ccl.nl_pt.PTMatterTracer()
pk_gg = ptc.get_pk2d_biased(tg, tracer2=tg)
pk_gm = ptc.get_pk2d_biased(tg, tracer2=tm)

# You can also ask for specific P(k) templates.
# E.g. this will get you the <delta * s^2> template
pk_d1_s2 = ptc.get_pk2d_template('b1:bs')

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2807 of 2922 (96.06%)  changed or added relevant lines in 70 files are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-1.2%) to 96.649%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pyccl/halos/concentration/concentration_base.py
42
43
97.67%

pyccl/halos/concentration/constant.py
18
19
94.74%

pyccl/halos/concentration/diemer15.py
42
43
97.67%

pyccl/halos/concentration/ishiyama21.py
131
132
99.24%

pyccl/halos/hbias/bhattacharya11.py
24
25
96.0%

pyccl/halos/hbias/sheth01.py
25
26
96.15%

pyccl/halos/hmfunc/angulo12.py
21
22
95.45%

pyccl/halos/hmfunc/jenkins01.py
20
21
95.24%

pyccl/halos/hmfunc/press74.py
20
21
95.24%

pyccl/base/caching.py
161
163
98.77%

Files with Coverage Reduction
New Missed Lines
%

pyccl/core.py
1
95.74%

pyccl/covariances.py
1
98.59%

Totals

Change from base Build<PHONE_NUMBER>:
-1.2%

Covered Lines:
5942

Relevant Lines:
6148

💛  - Coveralls

Thanks a lot @damonge! I haven't looked much at the IAs as I am not so familiar with it, but pgg ang pgm look good to me! Just to check - this assumes that stochasticity will be handled outside CCL, right?

@anicola Good point. Yes, indeed, I've added a note about stochastic bias in the docstring.

All done, back to you @nikfilippas

All should be covered now.

@damonge you do the honours


Fix overflow error in shuffle colormap for bool and signed integer labels

References and relevant issues
closes #7277
Description
Fix zero_preserving_modulo_numpy by not performing modulo operation when n is bigger than dtype of values

Another option could be to divide n/2 if data are signed integers.

@Czaki this doesn't close #7294 (I've removed that from the description). That PR is still worth merging because it fixes the issue of painting into the bool volume. I also think np.uint8 is preferable anyway, just because it's more commonly used.
And something feels a bit weird about this solution (though I'm still happy to merge it because it fixes the crash): we are relying on negative indexing into the n colours. I had to think about it, and indeed it checks out that min of the dtype, usually np.iinfo(dtype).max - 1, is guaranteed to fit in n, given the checks. Another non-obvious thing: indexing (-k) into an array of size n is the same as indexing with (-k) % n, if precision is not a concern. That's nice because we want the mapping to a color not to change between data types.
So I like this solution :+1: , but it does lead to some mental gymnastics, and I think it's better to avoid them in the simple case of bools. So all that to say I think we should merge this and #7294.

I'll try to write a test and push

Boom. This is ready. I'll merge shortly and include in the release! 🚀

Oh whew, the test failure is just a failure with pinging GH urls 😅
https://github.com/napari/napari/actions/runs/11048379513/job/30692381375?pr=7296#step:13:855
  =================================== FAILURES ===================================
  _________________________ test_help_urls[github_issue] _________________________

I'm going to ignore it. Merging!


Receiving KMails: There was an error processing your greenbox submission.

I have greenbox as part of my post-prism script. For the last 2 weeks or so, I've been getting messages from Jalen Arbuckle telling me There was an error processing your greenbox submission. Sometimes more than one, and they only come in hours after I actually ran the greenbox update. The update itself seems to work fine though.

It'll now only send one message, and I have some error reporting in place to see what is going on.

@gausie just now I received 2 messages again from JA

(times are in CEST)

Now definitively fixed, sorry


Send player back to village after losing AI fight

From Viktor:

hey man, just realised you dont get sent back to village after
losing to AI, i think that would be a good shout no?, iv been
caught 2x after arena battles with 0 hp and its been instant wins
for whoever catches it, i mean i would have lost anyway but just
a food for thought xD

Looked in the comments and this seems to be addressed already, I think Law fixed.


regression: simulacrum failing in 2.12 build since wartremover was added

looks like wartremover adds itself to libraryDependencies which then confuses dbuild?
wartremover was added in mpilquist/simulacrum#68
it looks like the same fundamental problem (with sbt plugins that need access to the project Scala instead of Scala 2.10) as with scalafmt discussed at https://github.com/scala/community-builds/pull/341

I was able to work around the scalafmt issue with:
    check-missing: false
    deps.ignore: [
      "com.geirsson#scalafmt-cli"
      "com.geirsson#scalafmt"
    ]

probably the same trick would work here

it didn't, I don't know why, and another attempt:
    extra.settings: [
      "libraryDependencies in coreJVM := (libraryDependencies in coreJVM).value.filterNot(_.name == \"wartremover\")"
     ]

failed with:
[simulacrum:error] /Users/tisue/community.212/target-0.9.5/extraction/9d8fb794701d979faced9442c9e6afcea96c9b51/projects/bb7380904b8b6c5ff8a3183058262fa514615b31/ÿÿÿÿÿÿÿÿÿÿ~~~~dbuild~defs.sbt:1: error: not found: value coreJVM
[simulacrum:error] libraryDependencies in coreJVM     := (libraryDependencies in coreJVM)     .value.filterNot(_.name == "wartremover")
[simulacrum:error]                        ^

I'm stuck. @cunei @dwijnand any ideas?

Try:
extra.settings: [
  "libraryDependencies in LocalProject(\"coreJVM\") ~= (_.filterNot(_.name == \"wartremover\"))"
]

@dwijnand thanks, that made the compile error go away, but it also didn't make dbuild/sbt stop complaining about the missing dependency, either :-/
I'll see if I can satisfy it by pulling down some wartremover jar using aether — hopefully it won't even really matter much what's in the jar

dammit, that won't work, wartremover is actually getting run:
[simulacrum] java.lang.NoSuchMethodError: scala.Predef$._refArrayOps([Ljava/lang/Object;)[Ljava/lang/Object;
[simulacrum] 	at org.wartremover.WartTraverser.$anonfun$isWartAnnotation$1(WartTraverser.scala:70)
[simulacrum] 	at org.wartremover.WartTraverser.$anonfun$isWartAnnotation$1$adapted(WartTraverser.scala:69)

hmm "the WartRemover SBT plugin sets scalac options - make sure you're not overwriting those by having a scalacOptions := ... setting in your SBT settings. Use scalacOptions ++= ... instead" that sounds like an opening

new, non-simulacrum-specific ticket is #364. I could use community help with this one, I burned too much time on it already


Showing the current file, but as the second option

I want the previously opened file to be selected when using the picker, but the currently opened file to still be available in the menu (as the first, unselected option).
So like this:
third_file
> second_file
current_file

I imagine I could easily do it if I had the access to the picker in the handler. So I think it would be a good idea to return the created picker in pick. There might be a way to access the global state for the currently active picker with one of telescope API's, but it's not easily discoverable, I couldn't figure it out quickly. And, you know, messing with global state is not great.

I've tried doing it, and it's not completely trivial. I mean, I did return the picker, but it doesn't seem to respond to me calling methods on it. Also, we only need to move the selection if there's a file open, which has to be detected somehow, which further complicates the task

Sorry for late reply. But I think I figured out how to do this.
There is an on_complete callback in the picker options. Inside this callback you can perform any action on the picker. Here is an example of how to move the selected item up when the picker is shown:
require('telescope').extensions.recent_files.pick({
  -- Note that on_complete is the array of callbacks
  on_complete = {
    function(picker)
      require("telescope.actions").move_selection_previous(picker.prompt_bufnr)
    end
  }
})


Add docs for the timeout paramter

Add documentation for methods that accept the timeout param. I believe all of these methods accept the param (but I may have miss some?)
Fixes #5055

Could we change the signature to timeout? to make clear it's optional?

Codecov Report

Merging #5057 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #5057      +/-   ##
==========================================
- Coverage   60.74%   60.72%   -0.02%
==========================================
  Files         198      198
  Lines        6605     6605
  Branches        3        4       +1
==========================================
- Hits         4012     4011       -1
- Misses       2593     2594       +1

Impacted Files
Coverage Δ

packages/jest-config/src/normalize.js
90.8% <0%> (-1.15%)
:arrow_down:

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 df9f473...836a248. Read the comment docs.

I mostly just followed how we're currently documenting optional params, eg:
http://facebook.github.io/jest/docs/en/expect.html#tohavepropertykeypath-value
but happy to change if you have a suggestion

Yap, let's just do this for now, and maybe later we can change all of the optional params in a separate PR.


Boundary condition

Hi Dr. Ali,
I have a question about flow rate boundary condition. If I want to use flow rate boundary condition instead of pressure boundary condition to compare the impact of capillary number. What should I assign the boundary condition keywords in the input file(for example, I want to assign a constant flow rate 2cm3/min in the inlet and a constant pressure boundary condition 1 atm in the outlet  )?

Thank you so much!

The viscous pressures are are not used in the displacement process in pnflow. So you can only model low flow rates for which the choice of boundary condition does not matter. In pnflow, only the T/F values are used to set the injection and fluid escape directions.
Sorry the format is taken from gnflow / GNM model for my own convenience in comparing the two models.  I gnflow I did implement and test the viscous pressure drop as a perturbation but even there I rarely use it.  The pnflow model itself has not been changed by me and is more or less compatible with the original 2003 code .   My changes were primarily just refactoring of the code to make its workflow clearer.

Thank you Dr. Ali!


Range bug fix

This fixes the problem with successfully iterating over a Range by raising a StopIteration at the appropriate time. If there is a better way to do this by exposing an iterator on the C++ side, I'm happy to attempt that as well.

Thanks for this @Pshriwise!

Looks good to me. Is this ready to go in @Pshriwise?

As far as I'm concerned, yup!

On Jan 13, 2016, at 8:21 PM, Anthony Scopatz<EMAIL_ADDRESS>wrote:
Looks good to me. Is this ready to go in @Pshriwise?
—
Reply to this email directly or view it on GitHub.

Thanks!


Add mssql.proposed.d.ts

Any extensions using the mssql.d.ts typings currently need to also bring over the azdata.proposed.d.ts typings because of the SqlAssessment definitions. This is generally just an annoyance for extensions, since most of them don't need these typings at all.
So to make this better I'm splitting the MSSQL typings into "stable" and "proposed" like we do with MSSQL. This way anything depending on azdata.proposed.d.ts typings can go into a separate typings file without impacting other extensions.

@sergten FYI, you should be able to just copy over https://github.com/microsoft/azuredatastudio/blob/9f93655d9e9363e0a40bf45e21554de046e042c5/extensions/mssql/src/mssql.d.ts verbatim now.


Images can't be uploaded

Expected behavior
When attaching image(s) in your post in SteemApp, the image should be first uploaded successfully into the Steem blockchain, and then, be represented with a link. Afterwards the post would be posted successfully.
Actual behavior
Upon attaching image(s) in a post, the image gets placed directly into the post instead of a link, and when posting it get's a message of "Images can't be uploaded", which is a indication of an unsuccessful posting.
How to reproduce

Obtain Redeem Code from iTunes and enter it to the Redeem button in TestFlight application.

Download SteemApp 2.0 (7).

Open the application.

Tap the Pencil icon found in the bottom center, which is for the creation of a post, then create a Title.

In the Story tab, create a post and attached an image, the image is shown in the content instead of a link.

Tap Next to proceed in Tags tab.

Tap Next to proceed in the Post tab.

After selection of reward type, tap Post.

After a few seconds bug occurs, and is prompt with a message, Images can't be uploaded.

Browser/App version: SteemApp 2.0 (7)

Device: iPhone 6

Operating system: iOS 11.4

Recording Of The Bug
https://www.youtube.com/watch?v=lHACk9zo0Ok

Hi josephace135, thanks for your bug report. Could you try to reproduce issue not using "</>" button and not typing  tag directly? Because this button only for debugging.

Hi @mby4github ,
I tried posting a picture, still Images can't be uploaded.
Recording
https://www.youtube.com/watch?v=Lurus0uUtVM

Fixed in 08389c80f3614758a0545302bd23109d137a8733


absent second pharyngeal arch artery

absent second pharyngeal arch artery
DEF: failure to develop or absence of the second pharyngeal arch artery
SYN: second pharyngeal arch artery aplasia
SYN: second pharyngeal arch artery agenesis
SYN: absent second aortic arch artery
SYN: absent second branchial arch artery
SYN: absent second PAA

absent second pharyngeal arch artery MP:0030339


[Demo]: Investigate prop documentation options

Investigate options for generating prop type documentation.
Suggestion: use Flow type features from`react-docgen.


Feature: Sandboxed stylesheets?

Our site has 2 distinct, conflicting stylesheets.  Each is used on different pages, and we never have both on a single page.
The default import 'app.css' approach results in both our stylesheets getting used simultaneously in Storybook, which is causing problems.   Any suggestions on better approaches where I might be able to specify per-component styles?

Currently, there's no way to isolate unless you run two storybooks instances with two config files.

Currently we don't have a feature planned for this.
What I suggest it to run two storybooks.


Update fa.json l10n file

Translate some phrases to Persian(fa).

For localization please use transfer:
https://www.transifex.com/traccar/traccar/

Thanks.


[C++] build issue Linux

Tried building latest version /C++11 on linux, faced few issues
gcc version 4.8.5 20150623 (Red Hat 4.8.5-4) (GCC)

It complained of duplicate definition of mmsghdr.
Somehow this section didn't do what it's supposed to do so I commented them in all relevant files
#if !defined(HAVE_STRUCT_MMSGHDR)
struct mmsghdr
{
struct msghdr msg_hdr;
unsigned int msg_len;
};
#endif

Then linker cribbed
/usr/bin/ld: cannot find -luuid
Though system has this lib
/usr/lib64/libuuid.so.1 -> libuuid.so.1.3.0

What is the output of CMake at the beginning? You should see something akin to:
-- Looking for stdint.h
-- Looking for stdint.h - found
-- Looking for stddef.h
-- Looking for stddef.h - found
-- Check size of struct mmsghdr
-- Check size of struct mmsghdr - failed
-- Looking for recvmmsg
-- Looking for recvmmsg - not found
-- Looking for sendmmsg
-- Looking for sendmmsg - not found

Yes, it did give that.  Though I can see sys/socket.h is available and has the definition.

What is the actual output you see?

exactly how you typed.

Here's error for #1
[ 63%] Building C object aeron-driver/src/main/c/CMakeFiles/aeron_driver.dir/aeron_driver_sender.c.o
/home/mb13808/workspace/aeron-master/aeron-driver/src/main/c/aeron_driver_sender.c:26:8: error: redefinition of âstruct mmsghdrâ
struct mmsghdr
^
In file included from /home/mb13808/workspace/aeron-master/aeron-driver/src/main/c/aeron_driver_sender.c:22:0:
/usr/include/sys/socket.h:102:8: note: originally defined here
struct mmsghdr
^
gmake[2]: *** [aeron-driver/src/main/c/CMakeFiles/aeron_driver.dir/aeron_driver_sender.c.o] Error 1
gmake[1]: *** [aeron-driver/src/main/c/CMakeFiles/aeron_driver.dir/all] Error 2
gmake: *** [all] Error 2

Error for #2
[ 49%] Built target Throughput
[ 50%] Built target TimeTests
[ 50%] Linking C shared library ../../../../lib/libaeron_driver.so
/usr/bin/ld: cannot find -luuid
collect2: error: ld returned 1 exit status
gmake[2]: *** [lib/libaeron_driver.so] Error 1
gmake[1]: *** [aeron-driver/src/main/c/CMakeFiles/aeron_driver.dir/all] Error 2
gmake: *** [all] Error 2

The output I showed is from Mac where mmsghdr doesn't exist and thus falls back to defining it. As you mention that sys/socket.h has the definition and it appears to be finding it, I'm not sure how the CMake output is exactly what is shown, but the compilation fails on the struct being already defined.
Linux gcc/glibc versions are a little odd in that the flags vary quite a bit for compat. As gcc 4.8 is a bit behind, it might be better to use a newer gcc.

I have tried with gcc version 6.1.0 but issue persists.
While #1 is not a show stopper,, #2 is . Any suggestion how can we work it around ? Tried playing with CMake files but it's not helping.

It's not clear to me what you are now seeing - does it build with gcc 6.1 but you have some runtime issues?

It's still not able to resolve libuuid issue

@manishbaphna :
workaround:
comment out these lines with ${AERON_LIB_UUID_LIBS} in aeron-driver/src/main/c/CMakeLists.txt
target_link_libraries(
aeron_driver
...
#    ${AERON_LIB_UUID_LIBS}
...
target_link_libraries(
aeronmd
aeron_driver
...
#    ${AERON_LIB_UUID_LIBS}
...
target_link_libraries(
aeron_driver_agent
...
#    ${AERON_LIB_UUID_LIBS}
....

Glad it can be worked around.
@manishbaphna can you paste the output of CMake where it does its configure? It is the start of the build and contains the lines I mentioned as well as compiler, etc.
It's strange that uuid must be found int the CMake find_library, but the final link fails. And would like to see if something is missed here.

Actually, the build tries to find libuuid and the uuid functions/headers separately. I would bet that the functions and headers are not found, but in the process, the library is found. But something else is not allowing it to be linked correctly. Probably a path is not correct for this setup as your original output mentions libuuid is found in a different directory (/usr/lib64).
If uuid can't be used, it falls back to a pure random generation for the receiver id. That is probably what is happening. But because CMake has found the lib, it tries to link the lib anyway. And fails. That is why when you remove the UUID libs, it works.
BTW, you might try installing libuuid and libbsd. But since it looks like you are using a non-standard compiler, things might not be in the correct configuration to make everything work.

we can do somethinh like below ?, where uuid path will always be taken from where cmake found it.
get_filename_component(UUID_LIB_PATH "${LIBUUID_EXISTS}" DIRECTORY)
link_directories(${UUID_LIB_PATH})

Closing this due to no further activity.


[AV2-827]appv2 endpoint where we login a user from a jwt token

Story/Ticket Reference ID from JIRA:
Add link here
Additional Comments:
Add here
Checklist:

[ x ] Unit test completed?: (Y/N)
[ x ] Docs folder up to date?: (Y/N)
[ x ] Add if anything missed: (Y/N)
[ x ] Add if anything missed: (Y/N)

All Checklist are okay.
This PR looks great - it's ready to merge! Please review and let's ship it. :)

Reminder:
@sasangachathumal @ronakamani remember to add this flag to your uncommitted environment file after this PR is merged.


Can I use this to mock localStorage?

I am trying to use this to mock localStorage:
const localStorage = {
  getItem: jest.fn().mockReturnValue(JSON.stringify({
    BUILD_DATE: DATE
  }))
};

jsdom.reconfigure({
  localStorage
});

However it doesn't appear to work. Is mocking localStorage with jest-environment-jsdom-global possible?

Hi @wallzero! Sorry for the late response. Seems like I misconfigured my email notifications for this repo.
It's totally possible to mock localStorage with this library, but it is with Jest on its own, too. Just do something like:
beforeEach(() => {
  Object.defineProperty(window, 'localStorage', {
    value: {
      getItem: jest.fn(() => 'result'),
    }
  });
});

it('should use the mocked local storage', () => {
  expect(window.localStorage.getItem('test')).toBe('result');
  expect(window.localStorage.getItem).toHaveBeenCalledTimes(1);
});

You can see this at work here: https://repl.it/repls/BuzzingIdioticSoftwaresuite


5.5.0 regression - importHelpers do not work with moduleResolution: bundler

🔎 Search Terms
importHelpers
This syntax requires an imported helper but module 'tslib' cannot be found.
TS2354
🕗 Version & Regression Information
fails on 5.5.0-beta

This changed between versions 5.4.5 and 5.5.0

reproduces on 5.5.0-dev.20240425
can't be reproduced on playground as it seems to ignore importHelpers
⏯ Playground Link
https://github.com/Knagis/importHelpers-5.5.0
💻 Code
tsconfig.json:
{
    "compilerOptions": {
        "importHelpers": true,
        "target": "es2017",
        "module": "ESNext",
        "moduleResolution": "Bundler"
    },
    "include": [
        "./src/**/*",
    ],
}

src/index.ts
export function foo(args: any) {
    const { bar, ...extraArgs } = args;
    return extraArgs;
}

🙁 Actual behavior
node_modules\.bin\tsc -b
src/index.ts:2:21 - error TS2354: This syntax requires an imported helper but module 'tslib' cannot be found.

2     const { bar, ...extraArgs } = args;
                      ~~~~~~~~~

Found 1 error.

🙂 Expected behavior
Code compiles without errors.
Additional information about the issue
No response

Bisects to #57896

Duplicate of https://github.com/microsoft/TypeScript/issues/57688


F8 ("next problem") doesn't work correctly when moving into the Settings file

Open your user settings JSON file and make several things invalid
Leave the settings JSON open, but switch to another file
Press F8 - this will jump you to the first problem in the settings JSON file
Press F8 again - nothing happens (expected: go to second problem in file)

This seems to be because the first F8 that takes you to the settings JSON doesn't place focus in there, so pressing F8 again doesn't move you to the next one. If you manually click in the right side of the editor and then press F8, then it works as expected.

Open settings JSON UI editor
Have some errors in the settings
Open another file
F8 should navigate errors in the settings file.

Fix LGTM in insiders! 👍


Include information about the metrics annotations

The current documentation gives no indication of what the metrics annotations do, added some rudimentary documentation to explain the basics

Looks good. Thanks for taking your time to write this.


Automatic broadcasting operations

If two variables are both broadcasted over the same domain, operations should act on the children and then broadcast again.
For example, Broadcast(a, "domain") + Broadcast(b, "domain") should automatically return Broadcast(a+b, "domain"). Similarly for functions.
Currently, this is done manually in some (maybe not all?) of the models, so doing this would also simplify the model structure slightly.
Can be done via the __new__ class constructor

Closing for now: this slows down the building of the tree too much


etcd GPG signature 2.1.1

Can't find the key anywhere that this is signed with.
$ gpg --verify ~/Downloads/etcd-v2.1.1-linux-amd64.tar.gz.gpg
gpg: Signature made Tue Jul 21 10:49:33 2015 PDT using RSA key ID CC22F12F
gpg: requesting key CC22F12F from hkps server hkps.pool.sks-keyservers.net
gpgkeys: key 8556AC44CC22F12F not found on keyserver
gpg: DBG: armor-keys-failed (KEY 0x8556AC44CC22F12F BEGIN
) ->0
gpg: DBG: armor-keys-failed (KEY 0x8556AC44CC22F12F FAILED 6
) ->6
gpg: no valid OpenPGP data found.
gpg: Total number processed: 0
gpg: keyserver communications error: Not found
gpg: keyserver communications error: Bad public key
gpg: Can't check signature: No public key

I have uploaded my key to the sks-keyservers. Please check it again.


Readme: unclear how to run in development

Please provide more information on how to run the project in development:
[13:42:14] Please allow `https://localhost:3000` connections in Google Chrome

[13:42:14] and load unpacked extensions with ./dev

In particular, what does "load unpacked extensions with ./dev" mean?
How to run it without running the whole npm run compress and manually installing it via dragging it to chrome://extensions/?

Refer: https://github.com/jhen0409/react-chrome-extension-boilerplate#development

The injectpageis need https protocol on development mode, but we not have SSL certificate.
It discussed in the (https://github.com/zalmoxisus/browser-redux/issues/5). (Maybe we can change to that solution)

Also, this getstarted can help you, I'll add link to readme and log.

The con of that solution is that you'll still get warnings in the console, which could be annoying. I tried to make Webpack to work on both https and http. The idea was to use automatically the protocol corresponding to the injected page, but didn't figure how to get hmr to work on both protocols.

Hi, you could get the warning removed by this new flag in chrome.

go to the url: chrome://flags/#allow-insecure-localhost
click "enable" and restart chrome.

Having some trouble getting hot reloading to work here. I've set the flag as @kenyeung128  suggested, and my address bar doesn't show any https issues, but getting errors described below.


Any plan for a simple Tree View guide with JSON data as menu in the Activity bar

Hello,
I am looking for a simple tiny example (or tutorial) of a simple TreeView filled by a short JSON data for a very cute simple menu in the activity bar
Sorry but you know, life is too short to try to understand the other TreeProviders :D
Thanks for your help

Here you go - https://github.com/Microsoft/vscode-extension-samples/tree/master/tree-view-sample

So when will we get in vscode production version ( not in vscode.proposed.d.ts ) the TreeView2 class from TestView example ?


How to delete ETH miner?

I did watchdog and stealth mining and ran it for verification, I was stupid, how to remove the miner? Tell me please.

I will quote another post

First stop the process called sihost32.exe and then the miner (that you chose in the 'Mining' tab, by default explorer.exe) and then remove the miners startup entry in regedit at "HKEY_CURRENT_USER\SOFTWARE\Microsoft\Windows\CurrentVersion\Run".****


Feature Suggestion: Sort "My Inventory" Alphabetically

u/Raven on Discord (and many other users in the past) have requested the ability to sort the inventory alphabetically.
Alphabetical sort is especially helpful for new users entering their inventory, or for people going through an update/purge as they clear space and remove outdated equipment.

This can be done.
However, I fail to see how listing alphabetically will help clearing outdated equipment. Wouldn't a "power" sort (e.g. by ATK or DEF...) be easier to identify outdated equipment?

My specific thought was that Testmonkeyalpha on Reddit has been putting together "Should I Sell That" guides, identifying gear by name that can be sold (under various criteria). I've been using those guides as reference points, and the alphabetical sort would help to keep ffbe-equip updated as I sell things in-game.

In that case, the search feature would be quicker for you to find an item, isn't it? :wink:

For one item, sure, but then I would have to type, and that takes effort.
Plus for multiple items, scrolling is nice.
On Mon, Sep 17, 2018, 3:52 PM Guillaume<EMAIL_ADDRESS>wrote:

In that case, the search feature would be quicker for you to find an item,
isn't it? 😉
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/lyrgard/ffbeEquip/issues/239#issuecomment-422147433,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AoFkcQEWfrYMC6wlAwAlJbSj01UwZ0DPks5ub_1lgaJpZM4WsOgN
.

one other note on this,
lyrgard has done quite a bit to try and make the 'type (strength) - descending' order match what is displayed in the game, but there are items that continue to appear out of order. some items are just reversed, some appear in a completely different spot.
a-z sort should match 100%, without having to beat at the code with a cattle prod.


Changing Viewport Viewstate from an empty Sheet to the same Sheet results in an error

Describe the bug
Error appears: "Cannot read property 'transform0' of undefined
To Reproduce
Steps to reproduce the behavior:

Have an empty Sheet open in Viewport
Select to view the same Sheet again
Error modal appears

Expected behavior
No error. Perhaps a transition animation if applicable
Screenshots

Desktop (please complete the applicable information):

OS: MacOS
Browser: Electron
Version: 1.057
iTwin.js Version: 4.7.4

Additional context
Stack Trace:

Screenviewport.changeView

[doAnimate evaluated to true with no opts passed]

Screenviewport.animateFrustrumChange
FrustumAnimation.constructor
SheetViewState.calculateFocusCorner

<map value calculated to be undefined, which is indexed in the following line with 'map.transform0.... which is the source of error>

SheetViewState.computeWorldToNpc
-<worldToSlab returns undefined; value origin:[1e200, 1e200, -1]>

I have an empty sheet. I failed to reproduce your issue. So, something is not captured in the repro steps.
Please supply the iModel and specify what the Id of the view to open, or provide the code that creates the view if it's not a persistent view.
Or, link to a sandbox in which the issue can be reproduced.

code for the view
Doesnt seem to matter what imodel, perhaps its how the sheet is created?
Sheet creation form Sheet creation tool
For reference, an imodel used:
itwinid=32d006af-ed6b-4a6f-aec0-d4e469d3b892
imodelid=b47a2f5b-8441-4443-ab7a-b6f7435bb8b4


TST: fix a typo in the flyer test

Noticed a small typo in the flyer tests.

Thanks, @danielballan.


70 update app navbar rendering

What type of PR is this? (check all applicable)

[x] Feature
[x] Refactor
[ ] Bug Fix
[ ] Optimization
[ ] Documentation Update
[ ] Other (please specify here: )

Description of what this Pull Request does
Updated frontend to handle login routing, some style changes, mostly updated Homepage.jsx with student content
Related Tickets & Documents

Closes #70

This looks very nice, great work, please see my comment about the modal. I realize this is MVP and if we don't get back to it, that's ok too.

I've copied your comment into an Enhancement Issue so we don't lose track of it.  It should be linked properly.  If anything needs to be added to it please do.  I'm going to merge this PR to push up another one.  Thank you for your input!


Pathfinding: pair-based liquidity hints and sync from mission control

This PR synchronizes mission control data from the lnd node and populates liquidity hints, where liquidity hints are now keyed by node pubkey pairs. This way we have an easy way to benefit from the existing data if lnd is used with other tools. Liquidity hints are extended with timestamps for each individual information (can_send/cannot_send in both directions).
Todo:

[ ] deal with number of forwarding trials when mission control data is loaded

@3nprob thank you for the comments, that's really great :+1:! I tried to incorporate your thoughts.

This looks great!


[Text Analytics] Disable checks in tests that fail due to service issues

Resolves: https://github.com/Azure/azure-sdk-for-net/issues/34520

API change check
API changes are not detected in this pull request.


Provide a way to override generic encoding/decoding for certain data types

For encoding Nothing as null for example, or Tuple a b as [a, b], ....
We could include options in Data.Argonaut.Options.Options, called userEncoding, userDecoding:
userEncoding :: GenericSignature -> GenericSpine -> Maybe Json
userDecoding :: GenericSignature -> Json -> Maybe GenericSpine
or something like this. genericEncodeJson'/genericDecodeJson' would then try those functions first on each recursion. Thus, the user can choose to override the encoding/decoding for certain signatures.

WIP.

use purescript-argonaut-generic-codecs for this.


How to implement image placeholders

Provide a customizable image slot during the upload process or if the upload fails

https://prosemirror.net/examples/upload/

https://prosemirror.net/examples/upload/

thks, I have tried it recently, and I don't know what went wrong, so I came here to help


Docs endpoint header basics

closes #1560

Fixed some issues with routing
Improved crumbs ( still needs changes )
Add basic endpoint/method/example components

Example:

Coverage decreased (-0.09%) to 81.289% when pulling baeb4caa3b053cc5b8266d4908b8460beba83020 on mparke:docs-linodes-header into 61d27f77bb03a9dd35c13d408879e7e002f51376 on linode:master.

The breadcrumbs for this page should be the sections it appears under, not the urls. So here it would say "Linodes > Backups".

Maybe add an issue to make the tabbed example component?

Also maybe add an issue to break endpoints out into groups? e.g. Linodes into Instances, Backups, Disks, etc.

Follow ups:
https://github.com/linode/manager/issues/1636
https://github.com/linode/manager/issues/1637


Exe not found

I can't find the exe file in the folder.
so, then I can't run the cmd command that uses a exe.
screen-melter.exe -B "-t 2000 --exit_time=10000"
!I am new to github so i dont know how to close a issue but i found the issue!

Hi, you probably downloaded the repo. It's not advised to store blob files in version control, so you can access them in the release tab.

The zip file listed there contains the exe.

oh yeah i found the issue but i dont know how to close issues


Optimize bottlenecks

These are some bottlenecks that were causing significant performance overhead for me. The changes are rough, but if they look good overall I can clean them up

Is this project still maintained?

Thanks for your work here @michaeleisel. Would you be able to post some timings before and after this PR (after a rebase) for a large project, and also for the performance tests. Thanks!

For carthage dependencies, on my 8 (physical) core machine it reduces the time from 600ms to 200ms for a large project. For the JSON parsing, it goes from 1,500 ms to 300ms. for performance testing, i can take a look. but first, are we good with these at a high level? (i.e., excluding stuff like try vs. try!, which can easily be fixed)

I can't speak for @yonaskolb, but I'm interested in this. From a cursory glance I'm good with it at a high level.

ok i'm happy with it now personally as well. here are the perf test results: https://gist.github.com/michaeleisel/3d1a5316d233571ca5e2ec9531351c82

are we good to merge?

More details on our performance bottleneck:

Seems it's all in project generation and writing. In particular these two functions are hot spots: https://github.com/yonaskolb/XcodeGen/blob/01c6959f487fd67f57d05f1fe82d1263771cda25/Sources/XcodeGenKit/SourceGenerator.swift#L80-L82
(or just  https://github.com/yonaskolb/XcodeGen/blob/master/Sources/XcodeGenKit/PBXProjGenerator.swift#L222)
and
https://github.com/tuist/XcodeProj/blob/master/Sources/XcodeProj/Utils/ReferenceGenerator.swift#L27

Yeah, I've only ensured it improves my own use case. I spent some time with that getAllSourceFiles. Without a deep understanding, it seems somewhat difficult to optimize. Maybe we could open another issue about it.

@brentleyjones ptal


cordova.plugins is undefined

I just added this plugin to my appbuilder web ide project and tried to simply use //cordova.plugins.notification.local.schedule({             //    id: 1,             //    title: 'I will bother you every minute',             //    text: '.. until you cancel all notifications',             //    sound: null,             //    every: 'minute',             //    autoClear: false,             //    at: new Date(new Date().getTime() + 10 * 1000)             //});
and I get the error above.

Did you wait for deviceready to fire?

yes. I added the plugin, then added this code on a button click event, so well after the deviceready fired...

And you're running it in a webbrowser?

Yes

Browsers don't implement the Local Notification API so you really need to use a device / simulator.

Incorrect. I am using the WebBrowser IDE to develop the app but testing Local Notification API on my device as I am well aware of this limitation.  Nevertheless, during debugging I get cordova.plugins is undefined... Please do not close this ticket as I need HELP!

I asked "And you're running it in a webbrowser?" and you said "Yes" so I think my respons was not incorrect. It's just that you thought I meant "IDE" whereas I meant "app".
Share your app with me and I may take a look.

Ok.
Notepad.zip
Here you go...

This code has no method tied to a button and no active cordova.plugins.noti.. code (commented) so it's hard for me to tell what exactly is/was live while you observe the error.
That being said the code looks reasonable to me. Do other plugins (like Toast which you've included as well) work?

Thanks for that second question as Toast gives same error, plugin undefined.  Notice that the cordova.plugins.notification.local.schedule is called in the app.js line 48. Check it out and you will see it is after deviceready... let me know your findings...

@EddyVerbruggen any update on the file I shared with you? (Did you discover anything helpful?)


isWebGLSupported breaking 0.2 change

It looks like https://github.com/phetsims/scenery/commit/f0dbdda2ed30a0fc22c240a7d0e7fca24d3e0fef changed how 0.1 handled Util.isWebGLSupported to a constant (preferred) over a function (which needs to execute more than once), and I missed it when porting commits to 0.2.
Molecule Shapes was using this for detection, and other simulations / libraries may be broken also.
Any objections for me changing it to the new way?

Feel free to proceed as you see fit.  Are you recommending the constant?   It sounds good to me.

Yes, recommending the constant. Do you want to review the changes when I'm done?

I skimmed the change sets--I don't think there's need for a thorough review now but I will keep my eyes open for issues when I am working on related issues.  OK to close if you are done here.

Done, closing. Thanks!


(Big) User avatar should look clickable only when clickable

This PR closes #166091728, by:

Adding disabled property to TouchableOpacity id there is no onPress prop.
Removing unused styles
Updating snapshots

Slideshow

Coverage decreasing seems to be a merging problem since I haven't added any new line


separate_unit for humanize and humanize_bytes

Adds a separate_unit parameter to the humanize and humanize_bytes methods. If selected adds a non-breaking space ('\u00A0') between the value and the unit.
Also includes relevant specs.
Fixes #14349

Maybe a configurable parameter could be better than a boolean flag?
This would give more flexibility and allow use cases where a different separator then a non-breaking space is needed.

Also, the default should use a regular space for compatibility.

What would be the reason for limiting? I don't see why any arbitrary restriction would be useful. If someone want's a particular string as separator, let them have it.

Just pushed the change to allow for custom unit separators. Thought about a Char but that felt overly restrictive as well. 👍

Just a tiny nitpick: specs could use a Char whenever possible to demonstrate the encouraged use, use a String when there are more than one char, and just leave one check that we can pass a 1 char string (or nil, number or whatever).

Good call. I cleaned up the tests and left the ones that I think are meaningful.

Thank you for your patience while we ironed out this feature 🙇

Happy to! Honored to contribute. 🔮✨

Shouldn't this parameter be added to the method documentation?