Bug: structures from other mods spawn on the "floor" (y=-63)

What happened?
First of all, I think this bug may be on their part, but I am not sure. I have already created an issue on their github repo (see https://github.com/finallion/The_Graveyard_-FABRIC-/issues/29), which reads as follows:
Structures from The Graveyard spawn on the "foloor" of the Aether dimension from "Paradise Lost - the Aether Reborn" (y=-63). I am not sure if this is a problem on your part or on theirs, but this seems very similar to this issue from Mo' Structures, which happened in the Aether too:
https://github.com/frqnny/mostructures/issues/62
The differences are that:

volcanic vents from Mo' Structures also spawn in the End and in Eden Ring (from the "Eden Ring" mod),
volcanic vents from Mo' Structures spawn at y=0.
I suppose that The Graveyard structures would spawn also in the End and in Eden Ring, but those dimension do not allow building under y=0.

Using the configuration to disable the structures in all the biomes used in the Aether works:
"blacklistedBiomes": [
"aether_highlands",
"aether_highlands_forest",
"aether_highlands_thicket",
"aether_wisteria_woods",
"autumnal_tundra",
"continental_plateau",
"highlands_shield"
]
(these biomes were added to the blacklist of all structures), but there may be name overlaps in biome names I think (for example an hypothetic modname:plains would overlap with minecraft:plains).

Mod Version
1.6.5 - 1.18
Fabric API Version
0.64.4
Relevant log output
No response
Other mods

geckolib 3.0.32
paradise lost <IP_ADDRESS>
the graveyard 1.5 (without biomes)

Additional Information
No response

Thanks for reporting this. This is indeed an issue on their end. The note about configuration is also interesting. They really should use identifiers instead of namespaces. But I'll let them know that. Thanks for reporting.

Update: this also seems to happen with awesomedungeon:abandoned_oak_place structure from Awesome Dungeon, but not all of the structures from that mod

Ooops, this actually happens with all the structures from said mod, it does not happen with structures from Awesome Dungeons Ocean Edition (presumably because there are no oceans)

Fixed for graveyard mod

Fixed for Awesome Dungeon mod. Have yet to hear back from mo' structures

Thank you for the updates!


DNM - 21051 - Fix sorting for remediation column governance policy table

Fixes sorting for remediation column in governance policy table
Related issue: https://github.com/stolostron/backlog/issues/21051

FYI @crizzo71 - would like to get this in for 2.4.5

/hold

/lgtm

/unhold


Add Metadata Groups

Several projects (including the 4CeeD materials science project) have requested the ability to add user defined metadata as a group of keys with values. 4CeeD implemented a new object class for this, but since it was not actually metadata it was not searchable.
The goal of this feature is adding this into clowder, and improving it by enabling sharing via spaces and not just 'public' and 'private' (the 2 settings in 4ceed.)

How is this different that asserting present usr-metadata?


Fixes an improper markup

Hello, this PR fixes  an improper markup.
Signed-off-by: Hirotaka Wakabayashi<EMAIL_ADDRESS>

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

@hiwakaba Thank you for signing the Contributor License Agreement!


set securityContext on each driver and pod

May i know is it possible to customize security Context for each driver and pod spun up by lighter? i use below on lighter session config, but does not work
"spark.kubernetes.driver.podTemplateFile": "s3a://buckert/pod_template.yml",
"spark.kubernetes.executor.podTemplateFile": "s3a://bucket/pod_template.yml",

We always override those to
"spark.kubernetes.driver.podTemplateFile": "/home/app/k8s/driver_pod_template.yaml",
"spark.kubernetes.executor.podTemplateFile": "/home/app/k8s/executor_pod_template.yaml",

We'll try make those configurable on next Lighter release.
As a workaround you can create a configmap for those files and mount them to the /home/app/k8s/ path on your lighter pods.


About the test dataset accuracy

Hi! Thanks for your work! However, when I load your model and test it on the validation test and test dataset, the main task accuracy is extremely low (0.58%), but the training accuracy is proper (89.7%) for resnet-18 with 200 rounds. Do you have any clues?
I have checked the evaluation functions and all transforms. It's really weird.

Hi, I test your setting but can't reproduce your error case. Could you let me know if you followed the steps below?

download checkpoint:

wget https://huggingface.co/zeyuanyin/tiny-imagenet/resolve/main/rn18_200ep/checkpoint.pth

evaluation script:

torchrun --nproc_per_node=1 classification/train.py \
        --model 'resnet18' \
        --batch-size 256 \
        --test-only \
        --resume ./save/rn50_200ep/checkpoint.pth

output:

Test:   [ 0/40]  eta: 0:01:52  loss: 1.8256 (1.8256)  acc1: 57.8125 (57.8125)  acc5: 82.4219 (82.4219)  time: 2.8022  data: 0.9022  max mem: 1122
Test:  Total time: 0:00:04
Test:  Acc@1 60.470 Acc@5 81.070

Hi, I test your setting but can't reproduce your error case. Could you let me know if you followed the steps below?

download checkpoint:

wget https://huggingface.co/zeyuanyin/tiny-imagenet/resolve/main/rn18_200ep/checkpoint.pth

evaluation script:

torchrun --nproc_per_node=1 classification/train.py \
        --model 'resnet18' \
        --batch-size 256 \
        --test-only \
        --resume ./save/rn50_200ep/checkpoint.pth

output:

Test:   [ 0/40]  eta: 0:01:52  loss: 1.8256 (1.8256)  acc1: 57.8125 (57.8125)  acc5: 82.4219 (82.4219)  time: 2.8022  data: 0.9022  max mem: 1122
Test:  Total time: 0:00:04
Test:  Acc@1 60.470 Acc@5 81.070

thanks! It's solved. It turns out there's something wrong with my data source. After downloading from your link. It works well. You can close this issue. Thanks again for your prompt response!


Opencre not yet secure (I understood we have something in place but do not see the results)

can you please elaborate? what results are you referring to?

I mean no https. Firefox and Chrome both say not secure

fixed in #54


DSD-3581: update required provider for deprecated object

BREAKING CHANGE: No longer uses deprecated object

@creynolds15 What's the status of this PR?

@creynolds15 What's the status of this PR?

Good to merge once it gets approval.

@souza-dan Mind reviewing the PR and approving it if it looks good. Thanks!

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


Optimization flags for Cython

Our Conda packages are currently broken at least for some users (#320), most probably because the build server creates the precompiled Cython module (for the direct_C method) with a more advanced instruction set than users' CPUs have. So let's try to remove the -march=native optimization flag to avoid this.
It also makes sense to replace -O2 with -Ofast to make the code faster by sacrificing strict IEEE compliance (which we don't need).
And let's try to do similar optimizations for Windows by adding corresponding MSVC flags.

This passes all unit tests on Travis and AppVeyor, including test_direct.py::test_direct_c_python_correspondence, which probably means that the Cython module is built correctly. Or not? The logs mention cython but are very terse about building/installing pyabel. Is it possible to see more information about this?

I think that if it passes that test that you mention then that confirms that the Direct-C implementation is working okay.
I think that you can see as much as I can see on Travis. But, I'm happy to give you full access to the Travis and Appveyor accounts if we can figure out how to do that. I log into travis through my GitHub account, so I wonder if you do the same if you will have access to the PyAbel organization account on Travis...

This builds with -O2 and -Ofast, with the last optimization overriding the others :
gcc -pthread -B /home/stg121/anaconda3/compiler_compat -Wno-unused-result -Wsign-compare -DNDEBUG -O2 -Wall -fPIC -O2 -isystem /home/stg121/anaconda3/include -I/home/stg121/anaconda3/include -fPIC -O2 -isystem /home/stg121/anaconda3/include -fPIC -I/home/stg121/anaconda3/lib/python3.9/site-packages/numpy/core/include -I/home/stg121/anaconda3/lib/python3.9/site-packages/numpy/core/include -I/home/stg121/anaconda3/include/python3.9 -c abel/lib/direct.c -o build/temp.linux-x86_64-3.9/abel/lib/direct.o -Ofast

Timing corresponds to -Ofast:
=== Forward Abel transform ====================================

Method              n = 257     n = 513    n = 1025    n = 2047
---------------------------------------------------------------
direct_C                2.5        17.8       118.8       873.6

=== Inverse Abel transform ====================================

Method              n = 257     n = 513    n = 1025    n = 2047
---------------------------------------------------------------
direct_C                2.9        19.5       131.1       909.1

Q: Is there a way to check a "direct" conda install?

@stggh, could you please rerun full timing benchmarks for direct_C on your i7-9700 and i7-6700 to update the documentation? Redoing the slow direct_Python is probably unnecessary, since it shouldn't be affected.

@DanHickstein, I've just checked and now see that Travis indeed lets me sign in using my GitHub account. However, it was giving access to full raw logs even without that. So I'm basically wondering how to increase the verbosity level to something like what @stggh showed for local manual installation.

rerun full timing benchmarks for direct_C on your i7-9700 and i7-6700

Sorry for the delay, there were some power outages in Canberra due to summer storms.
i7-9700 direct_C.dat
# direct_C        iabel(ms)
     5             0.16358
    9             0.17085
   17             0.18188
   33             0.20043
   65             0.27702
  129             0.68181
  257             2.87535
  513            16.77957
 1025           115.07356
 2049           845.21606
 4097          8275.82846
 8193         64248.76927
16385<PHONE_NUMBER>8

i7-6700:
# direct_C        iabel(ms)
    5             0.12384
    9             0.12569
   17             0.13269
   33             0.14659
   65             0.21511
  129             1.34027
  257             6.28215
  513            40.42908
 1025           195.81982
 2049          1271.44220
 4097         11299.37075
 8193         87618.95179
16385<PHONE_NUMBER>6

n=32769  to follow.

Thanks! This is not urgent. :–)
The new data, however, look somewhat strange compared to what we had (excerpts for i7-9700):
new:
  513            16.77957
 1025           115.07356
 2049           845.21606

old:
  513            17.88147
 1025           141.61429
 2049           998.25061

So it's in fact not much faster, and the old numbers differ significantly from what you've shown for -O2 -march=native previously. Has anything changed since these old benchmarks?..

anything changed since these old benchmarks

Quite a lot, with the Ubuntu upgrades.  I will rerun the original -O2 -march=native to compare.

Quite a lot, with the Ubuntu upgrades.

I guess, in this particular case only the GCC behavior matters (unless Cython started to produce something very odd), and it's hard to believe that GCC has unlearned how to do “native” optimization properly. :–) Maybe these old benchmarks were done with “cpu-boost” that you've mentioned?

done with “cpu-boost”

Here are 3 cases: -O2 -march=native, -Ofast, and -Ofast with turbo-boost, which allows the cpu to clock above 3GHz, upto 4.7GHz, but the CPUs are thermally throttledif they overheat.
Daun should not depend on Cython flags, so this makes sense. And I guess that turbo boost has little effect on it probably because matrix operations even at “normal” speed already load the CPU to the limit (I've read somewhere that CPUs can even reduce its clock rate when doing such heavy calculations to avoid overheating, so if this indeed happens, then “allowing” higher clock doesn't really change the actual clock).
I've updated the benchmarks using your first data as the most complete. It's still faster than before, and probably not very important anyway...

The logs mention cython but are very terse about building/installing pyabel. Is it possible to see more information about this?

OK, I've added the -v option to pip install on both Travis and AppVeyor, so now they report the details, and it can be seen that Cython indeed works properly in all cases.
Unless somebody knows how to test this on Conda before merging, I think, we can just release a new version in the near future and see what happens. It shouldn't get worse at least. :–)
@DanHickstein, I've noticed that AppVeyor has tests for Python 3.7 and 3.8, but they both in fact reached their end of support, and 3.10 is already available. Maybe we should switch these 3.7 and 3.8 to 3.9 and 3.10? Travis doesn't have 3.10 yet, so its current configuration with 3.8 and 3.9 looks fine.

It's fine with me to bump the Travis versions. It should just be a simple matter of changing the version numbers in .travis.yml (but probably some bug will pop up! :) )

OK. It seems that Python 3.10 is available by itself but is not compatible with their current Miniconda. So I've just updated the versions to 3.8 and 3.9.

Yeah, this has been my experience with new versions of Python on the CI platforms. You can only use one version behind the state-of-the-art. 3.10 will only work once 3.11 is released :)


v0.6.5

Changes

drop Python 3.8
run actions on new default Python version 3.10
update minimum scipy and networkx versions

create release PR

[x] create a PR that finalizes the code for the next version

[x] name the PR according to the version vX.Y.Z and add the release
tag (example here)
[x] make sure CHANGELOG.md is up-to-date and enter current date to the release version
[x] add summarized release highlights where appropriate
[x] update the CITATION.cff version number and date
[x] search the project for deprecated and remove deprecated code

[ ] wait for review and the CI jobs to finish
[ ] check the readthedocs PR build

Merge the Pull Request

[ ] merge normally and wait for all CI actions to finish on the main branch

add Git(hub) tag

[ ] tag and release the current master version on GitHub using the Releases feature

[ ] name the release git tag according to the version released (e.g. v0.3.3)
[ ] name the GitHub release accordingly, omitting the v prefix (this can be change later so don't worry, in
doubt use vX.Y.Z everywhere)
[ ] copy the changes/release notes of the current version into the description and change the GitHub PR links to GitHub markdown

[ ] wait for all Github Actions to finish

ReadTheDocs updates

[ ] check the build processes for latest, stable and vX.Y.Z get triggered on RTD (the tag build can get
triggered twice, resulting in a failed/duplicated build, no need to worry)

pypi release

[ ] check the automatic release to pypi after the build action completes here

conda-forge release

pypi release should get picked up by the conda-forge bot and create the new
pull-request here
[ ] carefully check the meta.yaml in the pull request, manually update all changes in the build and run dependencies
[ ] merge with 2 or more approved reviews

redone to keep Python 3.9 for now @marscher
I have set the testing pipeline to use 3.10 as the main Version from now on until we can look deeper into the fs issue

The cached environment seems a little messed up for today, but without caching Python 3.9 passes: https://github.com/BAMWelDX/weldx/actions/runs/4629167902/jobs/8189100689


AYR-737/refactor mobile views

Changes in this PR

Remove mobile specific code for browse / record routes
Update CSS / Tests
refactor summary-list component for record route

JIRA ticket
https://national-archives.atlassian.net/browse/AYR-737
Screenshots of UI changes
Before
After

[ ] Requires env variable(s) to be updated

verified search terms are not getting overwritten when user enters multiple search terms, apart from them it looks good


Lenovo Brands (Frontend)

This is the Front End code for the Lenovo brand page of the project

Only Low fi design was provided for the Lenovo page and it has been coded.
Kindly view attached links for source codes, pull requests and live preview

https://github.com/ifylyd/zuri-project-frontend/tree/main/Lenevo%20Brands  - Source codes
https://ifylyd.github.io/Lenevo-brands/- Live preview
https://github.com/Dev-180Memes/zuri-project-frontend/pull/9 - pull requests


Setup CIM 61850 mapping

Pull Request containing basic CIM 61850 mapping:

Convert incoming CIM file to RDF4J model
Using IEC 61850-6 XSD schemas to generate SCL Java classes
Using JAXB to fill and create outgoing SCL file based on Java classes.

@Flurb
https://github.com/com-pas/compas-core/pull/4#pullrequestreview-637155224

can we just use the XSD's from IEC-61850-6 without copyright notice or link to an end-user license agreement or something? (like OpenSCD did in their README https://github.com/openscd/open-scd). Did you guys investigate this?

It looks like your latest XSD contains copyright, isn't -:)  ?

@Flurb
com-pas/compas-core#4 (review)

can we just use the XSD's from IEC-61850-6 without copyright notice or link to an end-user license agreement or something? (like OpenSCD did in their README https://github.com/openscd/open-scd). Did you guys investigate this?

It looks like your latest XSD contains copyright, isn't -:) ?

I'm not sure what you mean, but we are using the latest XSD schemas right now, coming from https://www.iec.ch/dyn/www/f?p=103:227:4631044159258::::FSP_ORG_ID,FSP_LANG_ID:1273,25.
It looks like we can use these XSD files in our own repository (like in compas-core), but it's still under investigating.
Besides, I think it's better to remove these schemas from the compas-cim-mapping repository, and use the compas-core functionality instead :)


怎么插入到resnet中，

他的输入high 和 low 部分是怎么划分的

@jiafengshen
I do not understand Chinese well, but I think I'm asking about the structure of ResNet with OctConv, so I will explain.
First, let's look back on the operations of OctConv. This is from the paper, and we need high and low inputs and outputs respectively.

And, this is a traditional ResNet structure. Let's call it ResBlock.

ResBlock with OctConv is as follows. Just replace Conv with OctConv and make it parallel.

How to make high and low in the first ResBlock, we apply average pooling to the input of the model to make it low. High uses the input of the model as it is.

And in the last ResBlock, concat high and low, apply global average pooling, and pass it to the output layer activation function such as softmax.

It is possible to apply global average pooling each of high and low followed by concat, but it is less successible (It is observed that the error does not go down and stagnates, or the accuracy suddenly drops during training) .

@koshian2 excellent explanation. I have one question though. If I am correct, the high and low parts are chosen from the channel dimension with alpha and 1-alpha respectively. If I am correct in the above visualization, you are taking alpha==1.0. Is that a correct assumption of mine?

@AakashKumarNain
In the above figure, ResBlock with OctConv is a general case where alpha is 0 to 1.
If alpha is 0 or 1, it will be equal to the traditional ResBlock. Because if alpha is 0, it can be considered that there is no low sequence. Also, if alpha is 1, it can be considered that there is no high sequence.
To understand what OctConv is doing, the figure written in the paper is useful.

In the traditional convolution (e.g. Conv2D), the kernel has a shape of  (k, k, c_in, c_out)(This shape order changes depending on the framework). OctConv just assigns the Conv2D kernel to high or low depending on  alpha value. That is, if alpha is 0 or 1, 3 out of 4 cubes in OctConv's kernel will be meaningless (size = 0). Under these conditions, OctConv's kernel looks the same as Conv2D's kernel.
In other words, the traditional convolution can also interpret as a special condition (alpha = 0) in OctConv. Thus, ResBlock with OctConv is a general case where alpha is 0 to 1, right?

Thanks for the explanation!

@koshian2 Thanks for the explanation. That is much clear now. Also, can I use your implementation as a reference implementation for my work?

@AakashKumarNain
Sorry, my reply was late.
This project is a MIT license. You can use freely within it.

I changed the model of the last layer because I slightly misunderstood (issue #7)
The last layer in upper figure I wrote is different now.


Request for new connect method

As stated in websockets documents, the legacy implementation is deprecated, while this library only provided supports for legacy implementation. It would be much appreciated if proxy implementation for websockets.asyncio.client is provided.

noted, I'll be seeing about that
prs are welcome

My patch is as follows:
from typing import Optional, Union
from urllib.parse import urlparse

from websockets.legacy.client import Connect, WebSocketClientProtocol
from websockets.asyncio.client import connect, ClientConnection
from python_socks.async_.asyncio import Proxy

class ProxyConnect(connect):
    def __init__(  # noqa
        self,
        uri: str,
        *,
        proxy: Optional[Proxy],
        proxy_conn_timeout: Optional[Union[int, float]] = None,
        **kwargs,
    ) -> None:
        self.uri = uri
        # This looks strange, but
        if "sock" in kwargs:
            raise ValueError(
                "do not supply your own 'sock' kwarg - it's used internally by the wrapper"
            )
        kwargs.pop("host", None)
        kwargs.pop("port", None)
        u = urlparse(uri)
        host = u.hostname
        if u.port:
            port = u.port
        else:
            if u.scheme == "ws":
                port = 80
            elif u.scheme == "wss":
                port = 443
                # setting for ssl (because we specify sock instead of host, port)
                kwargs["server_hostname"] = host
            else:
                raise ValueError("unknown scheme")
        self.__proxy: Proxy = proxy
        self.__proxy_conn_timeout: Optional[Union[int, float]] = (
            proxy_conn_timeout
        )
        self.__host: str = host
        self.__port: int = port
        # pass it to the super().__init__ call
        self.__kwargs = kwargs
        # HACK: We deliberately don't call
        # the super().__init__ constructor method YET!

    def set_proxy(self, proxy: Proxy) -> None:
        self.__proxy = proxy

    async def __await_impl_proxy__(self) -> ClientConnection:
        # creating patched socket
        sock = await self.__proxy.connect(
            dest_host=self.__host,
            dest_port=self.__port,
            timeout=self.__proxy_conn_timeout,  # type: ignore
        )
        self.__kwargs["sock"] = sock
        # HACK: THE super().__init__ IS DELIBERATELY CALLED HERE!
        # It is because we need an already
        # connected socket object inside the constructor,
        # but we've only just got it inside of this method
        super().__init__(self.uri, **self.__kwargs)  # noqa
        try:
            await_impl = getattr(self, "__await_impl_timeout__")
        except AttributeError:
            # newer versions removed the __await_impl_timeout__ method
            await_impl = getattr(self, "__await_impl__")
        proto = await await_impl()
        return proto

    def __await__(self):
        return self.__await_impl_proxy__().__await__()

proxy_connect = ProxyConnect

__all__ = ["proxy_connect", "Proxy"]

cool


Scrape db.etree.org for setlist info

db.etree.org is probably a great source for setlist info. Any knowledge of how complete/accurate it is?
We might even be able to match up sources with torrent which would be cool.
@switz thoughts?

It's pretty darn accurate. The only issue is consistency w/ respect to tracking (multiple tracks in one file) and naming convention.
Tapers are a detail oriented bunch.
Matching up torrent w/ source would be awesome.

So maybe a good fallback for artists on archive but have no other set list info?


allow version to be set as option

Context: AWS Lambda runs Node.js v4, but locally we tend to run the latest.
Given this mismatch, it would be nice to set the target Node.js version as an option to this babel plugin, with the default value of process.version. We could then say we always want babel to transpile to Node.js-v4-compatible JS.

Any ETA on this?


Test Strategy: Create a test plan for the migration of integration test cases

Create a test plan for the migration of integration test cases. All aspects covered by current Build Manager tests must be documented and equivalent go test cases must be implemented

Added a Test Migration plans


[gym]support relative path like "~/file"

The "~/file" return false and the "/Users/name/file" return true.

:+1:

Thanks @yyq !! This is great! 🚀


Allow direct call of helpers: shortcut to execute

(cherry picked from commit 3ab770036e03c58cc88a058cd87d1cf4d98d4d4e)

Pull Request Test Coverage Report for Build 351

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.6%) to 99.403%

Totals

Change from base Build 342:
0.6%

Covered Lines:
999

Relevant Lines:
1005

💛  - Coveralls


WMIC Deprecation

I was hoping you would consider updating the Windows binaries so that it doesn't use WMIC.exe (Windows Management Instrumentation Command line, and not to be confused with the base "WMI").
WMIC has been deprecated since Windows 10 21H1 and will likely be removed in the future.
Also, wmic.exe is on a list of applications that Microsoft recommends be blocked for security reasons if possible, for those using certain security policies. It says, "Unless your use scenarios explicitly require them, Microsoft recommends that you block the following applications." For anyone (including myself) using Windows 11's Smart App Control, which blocks all the files on that list and cannot be overridden, this causes a slight issue.
When WMIC is blocked from running, OpenSpeedTest Server still works, but the GUI window is unable to show the local IP address of the  system, so the user would have to figure that out themselves if they don't already know.
Here's what it shows, noting the last line in particular:
OpenSpeedTest-Server : Checking port 3000 status.
OpenSpeedTest-Server : Port 3000 is available.
OpenSpeedTest-Server : Starting...
OpenSpeedTest-Server : Started.
OpenSpeedTest-Server : Failed to get IP Address using port 3000

Also interestingly, the "Start Server" button doesn't change to "Stop Server", but the server is still definitely running and works.

OpenSpeedTest-Server is an Electron app that utilizes the "internal-ip" package to ascertain the system's IP address. This "internal-ip" package, in turn, relies on a dependency known as "default-gateway," which employs "wmic."
A discussion regarding this issue is underway within the "default-gateway" repository. A proposed solution using PowerShell has been submitted in the form of a pull request, but it has yet to be merged.
Should "default-gateway" rectify this issue, we can proceed with updating the apps accordingly. However, if the issue persists, we shall address it in the subsequent major release by either forking "internal-ip" and implementing a fix ourselves or opting for an alternative package that functions without "wmic."
To reproduce this issue, we need to enable Smart App Control, correct? I am going to replicate this and explore other options that can be used to fix this issue as soon as possible.

Ah ok I see. Yea again not a huge deal since the program still works and probably doesn't affect many users, but figured I'd mention it.
To reproduce you wouldn't necessarily need to enable smart app control or Windows Defender Application Control (SAC is basically a special made policy using WDAC). Any method that 'blocks' it from running could work.
The easier way I'm aware of would be using AppLocker which is a feature built into Windows 10/11 Pro (because it uses Group Policy Editor). AppLocker is a whitelisting feature that is the predecessor to WDAC but a lot easier to set up. It basically just lets you restrict what can run on the system by letting you create rules where something can only run if you create a specific rule to allow it, but you can also create rules to block specific stuff too.
Here's a simple walkthrough which should be the fastest way to reproduce:

Open Group Policy Editor (Just search the start menu for gpedit.msc )

In the group policy editor to go: Computer Configuration > Windows Settings > Security Settings > Application Control Policies > AppLocker > Executable Rules

Then in the right side there won't be any executable rules yet, so AppLocker won't be active until there's at least one. Before creating a rule to block wmic.exe you'll want to create a rule to just allow all exe's so it doesn't interfere with the rest of your system. Basically with applocker active (meaning there's at least one rule), a program can only run if there is a rule explicitly allowing it. So if you were to just go and create a wmic.exe blocking rule, actually it wouldn't allow any other exes to run without making a rule to allow them, so we can just do that first.

In the right hand side with executable rules selected, right click and hit 'create new rule' > Keep 'Allow' selected and hit 'Next' > Then select 'Path' and hit next > In the path box, just simply put an asterisk ( * ) and hit next > on exceptions page just hit next > Then you can name the rule whatever you want and hit Create.

See here as an example:
https://github.com/openspeedtest/Speed-Test/assets/12518330/98208d1e-64ae-419e-8488-e3f66c3d1b63
That basically creates a rule that just allows any exe to run, which in your case where AppLocker wasn't enabled before, will actually mean it will make no difference, since without AppLocker everything was allowed by default anyway.

Next right click to create another rule, but this time on the permissions page select "Deny" action > Then select 'Path' type rule like before > then in the path box this time put *\wmic.exe . (Note it has to be a backslash, it wont let you put a forward slash) then hit next > Again on the exceptions page hit next without changing anything > Then name the rule whatever you want and hit Create.

Now you'll have 2 rules, one allowing all exe's, but one blocking wmic.exe, but Deny rules always overrule the Allow rules so now it effectively only blocks wmic.exe from running. If you delete all the rules it should just revert to disabling AppLocker altogether.

One possible thing you might have to do, is back on the left side of the window with the tree (where Executable Rules was selected), right click "AppLocker", click "Properties", then where it has the Executable Rules "Configured" check box, check that and in the dropdown hit "Enforce Rules" and hit OK.

This step might not even be necessary but worth knowing about. In the dropdown you can also select "Audit Only" if you want to disable AppLocker without having to delete the rules.
Also note that there are other sections like 'Windows Installer Rules' and 'Script Rules'. Those all get activated independently, so as long as you don't create any rules in there they won't be affected so you don't have to worry about those.

@ThioJoe I reproduced this issue; I enabled SmartAppControl, and everything worked fine. However, after a Windows update, I encountered the same error as you did. I modified the default-gateway package using the changes from the deleted pull request, and it is working on my machine. I need to test this on Windows 10, etc. I have not bumped up the version number. As I mentioned, I need more testing.
https://openspeedtest.com/downloads/OpenSpeedTest-Server 2.1.8-PowerShell.exe

Maybe it would explain then why it doesn't work and is stuck on "Start request received"?
Tried the Microsoft store one, then a simple binary, run as admin, use different port, no success...

Your Must have a default gateway and IP addresses
wmic issue is very rare. If that is the problem try https://openspeedtest.com/downloads/OpenSpeedTest-Server 2.1.8-PowerShell.exe

@dw5 Also check firewall rules.

FYI, wmic.exe is no longer available on a clean install of Windows 11 version 24H2. (Not sure if it remains and/or is functional on an upgrade in-place.)
The PowerShell test version does seem to work.

The test version also seems to work for me on Windows 10 version 22H2.

FYI, I upgraded one of my systems in place to the 24H2 Release Preview (26100.1000) and wmic.exe is still there and functional.

Im having this issue from both "OpenSpeedTest-Server 2.1.8-PowerShell.exe" and "OpenSpeedTest-Server_2.1.8.exe", im on Windows 11 with this build number

Ditto here on a brand new Windows 11 machine.


lock: 1.3.0 -> 1.3.4

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for lock is: Process data with GnuPG
meta.homepage for lock is: https://konstantintutsch.com/Lock
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.3.4 with grep in /nix/store/mmwwl58c1wfjyn1q5darllywg6v1ydjs-lock-1.3.4
found 1.3.4 in filename of file in /nix/store/mmwwl58c1wfjyn1q5darllywg6v1ydjs-lock-1.3.4

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

lock

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/7351e6666d107a68e9bde2d92fd1e013ceb4f2cc#lock

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

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

x86_64-linux

:white_check_mark: 1 package built:

lock

Maintainer pings
cc @getchoo for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

@ofborg build lock lock.passthru.tests


Creating a provider for web pages

I'm not sure if elementum would support this, but it would be cool to be able to write a provider that searches non-torrent based websites (1movies, alluc etc) that can then be used with the urlresolver addon for kodi. This is similar how the popular addon exodus worked. A separate provider addon would need to be written to search these sites, but the core needs to be able to support these.

Hi guys! @elgatito I want to help with this task becourse I already thinking about this feature.
Not sure if this feature related to Pulsar/Quasar/Elementum. But if we talking about streaming addon, I think we can apply http/rtsp/etc streaming to Elementum.
How you think about it?
Also I think it no huge and difficult task. Becourse I looking to another addons with video streaming support and they just provide link to stream source like plugin.play(linkToRemoteSource)
As I see in Elementum we have something like
XBMC_PLAYER.play(item)
And I think we can just send link to this without buffering by Elementum (just by using only kodi streaming features).
Only one problem I see: How to present streaming sources to Elemenum select source dialog
Also, I plain implement streaming sources as different from Brust provider for split logic. I think this feature will be cool for Elementum

It's a bigger task for Burst or other plugin that would query websites and form search results. Elementum can only give queries, formed with TMDB database.
The bad thing - Elementum won't control all these website based videos.

But if we talking about streaming addon, I think we can apply http/rtsp/etc streaming to Elementum.

You don't have to do anything with streams, you get a stream url from the search plugin (like youtube), and give it to elementum, elementum will pass it, AS IS, to Kodi, and Kodi is playing the stream, because for Kodi, it looks like a usual stream from the web (and it is it, really).
If you want to make own searcher - you can use Burst code as a start, to leave parsing incoming payload and return results back to elementum, when you have some results - I can make further changes in Elementum to properly pass the stream to Kodi.

I can make further changes in Elementum to properly pass the stream to Kodi.

Thats what I talking about.
I can do it. I already have code base for some resources.

The bad thing - Elementum won't control all these website based videos

What you mean? Why need control?

What you mean? Why need control?

When you start a playback with a torrent file you start in internal player, which reads your playback progress, does scrobble on trakt, provides file readers to read torrent info.
When you have an external stream url - you just pass it to Kodi and that is it, it opens a player and it's not linked to plugin anymore.

Understood. What we can do for keep tracking? Or it is super hard at this moment?
PS. Kodi also have trakt plugin for track what played in kodi itself. May be we can get code base from trakt plugin for kodi and hook all played in kodi and pass to elementum trakt?
It is can be OverKill for elementum, but we can pass it as additional addon in elementum repo (and pass it as dependency for new provider plugin)

May be we can get code base from trakt plugin for kodi and hook all played in kodi and pass to elementum trakt?

There is no need to do all of that.
I will try different ways to handle correctly and see how it goes, but only after there is a plugin ready for testing as a source for web streams.

Great! I notify you when make something ready for use


FlameGraph Issues in FX UI ?

I just read @nitsanw 's great blog post about Flamegraphs, and after having another look at the flamegraph rendering in the honest profiler JavaFX front-end, it seems to me that there's an issue : the stacks do not seem to be aggregated together properly.
Fundamentally, if I understand it correctly, a flamegraph is "just" a graphic rendition of  the tree view (aggregated by method id), with the roots at the bottom, and rectangles for frames, the width being in proportion to the sample count % in their parent.
If I read the code correctly though, the flame graph collector starts a new FlameTrace whenever the next received stack isn't identical to the previous one. This is then rendered as such without further aggregation.
As a consequence, if only 2 stacks appear in the sample, but they alternate, the result will be highly fragmented.
To put it more explicitly, say that we see : a.b.c() (stack 1) and d.e.f() being called alternatingly 3 times :

Log events (simplified) :
---   Trace Start
---   Frames for a.b.c
---   Trace Start
---   Frames for d.e.f
---   (repeat the above two more times)

My understanding is that this should be rendered as 2 vertical columns, each with 3 layers, and every block has a sample count of 3.
However, from the code, it seems that this will be rendered as 6 columns, with every block having a sample count of 1.
I'm about to try and "fix" this, but before checking anything in I'd like to get some confirmation that this is indeed an issue, and not based on my misunderstanding the principles.

@PhRX The issue you refer to is not a problem for the command line tool producing collapsed stacks file (I think). At least from what I have seen. The JavaFX rendering is broken. Please hold fire until your MEGA PR is accepted.

@nitsanw OK I'll fix it on a new branch.

Fixed in https://github.com/PhRX/honest-profiler-fx-ui/commit/459331053e357d895fa47b50f984d29740917ee2
Will close after #184 is accepted + new PR created for this fix + accepted.

@nitsanw you might be interested in my latest commit on the flame branch since it adds FlameGraph diffs. Also, the flamegraphs are filterable and groupable, so you can switch between aggregate-by-thread or aggregate-all-threads at the flick of a, well, switch... :) The coloring logic might not necessarily be to your taste but it is easily tweakable.

I'm excited for this fix.

Can be tested in PR #192

Fixed independently in #210


[rmlui] Add Rmlui library

Describe the pull request

What does your PR fix?
Adds a new port: RmlUi

Which triplets are supported/not supported? Have you updated the CI baseline?
Hopefully all platforms supported, tested on x64-windows, x64-windows-static, x86-windows.

Does your PR follow the maintainer guide?

Yes

I noticed some issues when consuming the library with static builds. I'm fixing things upstream so that the necessary define RMLUI_STATIC_LIB is properly exported in CMake.
My (unpublished) changes seem to work well for CMake builds now. However, if I may ask, is it correct that MSBuild does not make use of the exported defines from CMake, so they need to be added manually by users? What is the best course of action in vcpkg ports to make sure macros are included, should we patch the library source to add the macro definition in-source?
Please don't merge before this is resolved, thanks!

My (unpublished) changes seem to work well for CMake builds now. However, if I may ask, is it correct that MSBuild does not make use of the exported defines from CMake, so they need to be added manually by users? What is the best course of action in vcpkg ports to make sure macros are included, should we patch the library source to add the macro definition in-source?

If the macro exists in the header file, you should use vcpkg_replace_string to replace this macro in the header file with #if 1 according to the conditions. See example:
https://github.com/microsoft/vcpkg/blob/030cfaa24de9ea1bbf0a4d9c615ce7312ba77af1/ports/libflac/portfile.cmake#L56-L74

When building rmlui:x64-linux:
CMake Error at scripts/cmake/z_vcpkg_apply_patches.cmake:61 (message):
  Applying patch failed: Checking patch CMakeLists.txt...

  Checking patch cmake/FileList.cmake...

  error: cmake/FileList.cmake: No such file or directory

  Checking patch include/RmlUi/Config/Config.h...

  error: include/RmlUi/Config/Config.h: No such file or directory

Call Stack (most recent call first):
  scripts/cmake/vcpkg_extract_source_archive_ex.cmake:144 (z_vcpkg_apply_patches)
  scripts/cmake/vcpkg_from_github.cmake:154 (vcpkg_extract_source_archive_ex)
  ports/rmlui/portfile.cmake:1 (vcpkg_from_github)
  scripts/ports.cmake:139 (include)

This LGreatTM, thanks for the awesome library and making a PR into vcpkg!

Thank you!


[MM-28235] Hide UseNewSamlLibrary feature from system console

Needs to be cherry-picked into 5.25 / 5.26 / 5.27
Summary

Since we are not going to be using the new SAML library for the time being, we are hiding this feature from the system console.
Ticket Link

https://mattermost.atlassian.net/browse/MM-28235
Related Pull Requests

Has server changes https://github.com/mattermost/mattermost-server/pull/15371
Has enterprise changes https://github.com/mattermost/enterprise/pull/753

/update-branch

/update-branch

/update-branch

@hahmadia
Error trying doing the automated Cherry picking. Please do this manually
+++ Updating remotes...
Fetching upstream
Fetching origin
+++ Updating remotes done...
+++ Creating local branch automated-cherry-pick-of-mattermost-webapp-#6350-upstream-release-5.27-1599056943
Branch 'automated-cherry-pick-of-mattermost-webapp-#6350-upstream-release-5.27-1599056943' set up to track remote branch 'release-5.27' from 'upstream'.

+++ About to attempt cherry pick of PR #6350 with merge commit 3e2f76ab10b2ca9de0620cba7d7268df0353715d.

+++ Conflicts detected:

UU components/admin_console/admin_definition.jsx

+++ Aborting in-progress git cherry-pick.

+++ Returning you to the master branch and cleaning up.


Use v-form in playground instead of form-validator

In vuetify newer version we are allowed to pass an async method as validator https://github.com/threefoldtech/tfgrid-sdk-ts/issues/1219

after some investing v-form api doesn't provide the same powerful api as validator-form


Add git-ftp

Tool information

Tool name: git-ftp
Tool license: gpl3
Add or update? Add
Desired version: latest stable
Approximate size: 70kB (informed by apt-get during installation on Ubuntu 18.04 LTS)
If this is an add request:

Brief description of tool: Git-ftp -- uploads to FTP servers the Git way
URL for tool's homepage: https://github.com/git-ftp/git-ftp

Area for Triage:
Deployment/Release
Question, Bug, or Feature?:
Feature
Virtual environments affected

[ ] macOS 10.15
[x] Ubuntu 16.04 LTS
[x] Ubuntu 18.04 LTS
[ ] Windows Server 2016 R2
[ ] Windows Server 2019

Can this tool be installed during the build?
apt-get install git-ftp
Tool installation time in runtime
26s
Are you willing to submit a PR?
Sure.

Hi @fabriciomurta , we have added git-ftp to Ubuntu image and it will be rolled out next week.
In case of any questions, please let us know.


I want to get a warning if I try to open a channel with 0.0 tokens

User Story
As Chris, I want to get a warning if I try to open a channel with 0.0 tokens so that I am able to use the channel when I have opened it.
Acceptance criteria
Tasks

@christianbrb what does this mean technically, that you are still allowed to open a channel without deposit but you get a warning that the channels will be unusable until you deposit?

Yes :-)

Closed via #647


Possibility to get undefined values in objects using wildcard path

Hey,
Here is an example:
import * as dot from 'dot-wild-tiny'

const data = {
  pets: [
    {},
    { name: 'Rex' }
  ]
}

dot.forEach(data, 'pets.*.name', (value, _, __, path) => {
  console.log(`${path} - ${value}`)
})

This will return pets.1.name - Rex. However, there is another object in this path which doesn't have name property and I can't get it as undefined. I would like the result to be:
pets.0.name - undefined
pets.1.name - Rex

Is it somehow possible to achieve? I'm using dot-wild-tiny, but opened issue here as it's more feature rich.

Hi @VladShcherbin, Thank you for Issues. And sorry late response :sob:
I am busy recently and I can not secure enough time for work. Therefore, it may take some time to respond to the problem...
If you can fix it, it will be a great help if you send me a PR :smiley:

I'm in agreement to fix this problem :+1:

@tsuyoshiwada I kinda tried to fix it today but unfortunately this deep object enumeration operations are a bit hard for me :( This is actually why I decided to use external package instead of my own implementation.
Since you are busy now, I'll add skip to nested tests for now and wait for this feature to be added :)

@VladShcherbin
Thanks ... :sob: !!!!
We are sorry for the inconvenience, but please wait for the fix version :bow:


add config for loading openai model.

Objective
Fix the issue: #90 .
I remove deprecated GPT-3.5-turbo-0613, and set GPT-4o as default.
Because GPT-4o is cheaper and has higher accuracy than GPT-4.
I added config to load GPT model name, because in the future GPT-4o will be deprecated.

I tested and confirmed that it works fine.

I could successfully display the graphical image by using Neo4j Desktop.


[Angular] Add account with authentication JWT

Fix #898

@qmonmert : you need to add the new API in generator.sh, otherwise, it won't be tested in CI

@pascalgrimaud where is this file? I don't see yet I rebase main branch

@qmonmert : here is the file https://github.com/jhipster/jhipster-lite/blob/main/tests-ci/generate.sh#L183
You need to call your new API here

@pascalgrimaud okay it's generate.sh not generator.sh :p

My bad, I'm sorry for this typo :-p

I started to review it.
Globally, the authent works, but I think we need to reorganize all these files, as it does not totally respect Hexagonal Architecture.
I'll propose you how we can reorganize it, but it would be easier to propose it directly in a sample project, rather than the generated part.

@pascalgrimaud ok

Here the repo I use for testing and reviewing:
https://github.com/pascalgrimaud/tmp-jhlite-angularapp/pull/1

After studying the architecture, I don't have enough skill to propose a good folder to totally respect Hexagonal Architecture with Angular. So I think we can have features by package, a little bit similar to what we have in generator-jhipster.
Can you refactorize and use this plz?

In another PR, we could refactorize "common" to use another like "app", as it's not necessary to have 'primary' if we can't split correctly in primary/domain/secondary
Another comment:

I use angular styled, followed by angular+jwt and I lost all styled in app.component.html

@pascalgrimaud ok I will do that!

@pascalgrimaud refacto done + fix angular styled jwt


Repair Simple Metrics data displayed on /metrics/simple route

Hi !
repost because i failed by posting it on master
Metrics on /metrics/simple only return values from the day instead of the month (or the chosen month).
So i added in the djangoo orm filter the precision of a month instead of taking only one day.
I may recommend also to change the name of the variable "now" to "date_asked" or something like that, but it is your code :)
Very nice project !!!!

This looks simple enough, and with actual metrics showing too. Thanks for your contribution!


Add doc on incremental Checkmarx scans, also re-adding fortify STO re…

…f per DOC-2446
Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[ ] Issue
[ ] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

Preview environment: https://ci.pr.harness.io/pr-345

Preview environment: https://ci.pr.harness.io/pr-345


How to cancel the upload operation if its take too long too make connection due to slow internet ?

I used this document to upload images :Blob uploading
But when my internet is too slow. It keeps on trying to connect. Is there any way to set timeout exception while making a connection to a container.

Hi @coderbhupendra, there are two options for timeouts. MaximumExecutionTime and TimeoutInterval – the former is the total time across all retries, the latter is passed to the service as the server timeout for each request. Please take a look at https://github.com/Azure/azure-storage-java/issues/78 for more information

@jofriedm-msft thanks for the help. And sorry for late reply.


python3Packages.pyparsing: 2.4.6 -> 2.4.7

Motivation for this change
Update to latest upstream release 2.4.7
Changelog: https://github.com/pyparsing/pyparsing/blob/master/CHANGES
Things done

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

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

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

Duplicate of #107791


Ensure that notifications are received properly

Reported in #10
Fixed, and added a test.
fixes #10

@untergeek I don't think the changes in "lib" fix anything related to the new specs. I tried the new specs on master and they still all pass.

I think I know why.  I'm only sending a single packet, and that would work.  It breaks when a string of packets go through.
I'll have to see if I can string a bunch of packets together to force the test. The message and severity field don't propagate otherwise.

Ran the tests and they pass :+1:
small note: only the change 91 is required to make specs pass.
after talking to @untergeek understood they target other issues with collectd decoding and are also necessary.
LGTM


minor improvement on documentation "Loading Iris Cubes.rst"

🚀 Pull Request
Description

Made some of the wording clearer
Modified the first example, on iris.load() function

Consult Iris pull request check list

 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.


DRILL-7729: Use java.time in column accessors

DRILL-7729: Use java.time in column accessors
Description
Uses java.time classes in the column accessors. Leaves Joda time for Interval, as it has no java.time equivalent.
This change allows the column accessors to work with Drill's JSON writer implementation. This PR includes a new JsonWriter based on a RowSetReader.
Documentation
The change is transparent to users except in one particular case: a use of the "provided schema" feature in which the user has provided a format for a DATE, TIME or TIMESTAMP column. The java.time formats are slightly different than their Joda predecessors.
Testing
Modified all tests which used Joda formats. Reran the full unit test suite.

@paul-rogers, did you have a chance to take a look at @cgivre comments?

@paul-rogers Hello, Is this PR still necessary to merge? Is it replaced by #2149?

@paul-rogers Hello, Is this PR still necessary to merge? Is it replaced by #2149?

@luocooong, thanks for the reminder. Looks like I originally factored out the Java time stuff for its own review. Got no takers, it seems, and I forgot about it. So, when @cgivre asked me to finish the streaming JSON stuff, I just included the time stuff as well.
If the team prefers, I can pull the Java time stuff out of the streaming JSON PR and replace the code here with the updated version. Or, if the team is OK with the single big PR, we can keep it in the streaming JSON PR.
How would you like to proceed?

@luocooong, thanks for the reminder. Looks like I originally factored out the Java time stuff for its own review. Got no takers, it seems, and I forgot about it. So, when @cgivre asked me to finish the streaming JSON stuff, I just included the time stuff as well.
If the team prefers, I can pull the Java time stuff out of the streaming JSON PR and replace the code here with the updated version. Or, if the team is OK with the single big PR, we can keep it in the streaming JSON PR.
How would you like to proceed?

I propose to leave changes in bigger PR, to avoid additional work on splitting it and rebasing this one onto the master.

I propose to leave changes in bigger PR, to avoid additional work on splitting it and rebasing this one onto the master.

@vvysotskyi, thanks for the suggestion. Given that, I'll go ahead and close this PR.

@vvysotskyi, thanks for the suggestion. Given that, I'll go ahead and close this PR.


ポーリングの仕方を変更

Codecov Report

Merging #36 (4bb6f32) into main (db6f961) will increase coverage by 0.05%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #36      +/-   ##
==========================================
+ Coverage   97.24%   97.29%   +0.05%
==========================================
  Files           6        6
  Lines         145      148       +3
  Branches       21       21
==========================================
+ Hits          141      144       +3
  Misses          4        4

Impacted Files
Coverage Δ

src/queue.ts
100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


marketplace guide

please review! thank you!

Please Review and let me know if this is good to go. - Bren


docs: Mendix Buildpack is now automatically updated on resume for Free Tier

@MarkvanMents It's released :+1:


#175

Merge checklist

[ ] TravisCI tests passed
[ ] Documentation

Codecov Report

Merging #182 (a45c030) into master (8673721) will decrease coverage by 0.175%.
The diff coverage is 0.000%.

@@              Coverage Diff              @@
##            master      #182       +/-   ##
=============================================
- Coverage   25.035%   24.859%   -0.176%
=============================================
  Files            4         4
  Lines          711       712        +1
=============================================
- Hits           178       177        -1
- Misses         533       535        +2


[BUG] - TraceMempool (and perhaps others?) config setting is ignored

Internal/External
External
Area
Other  cardano-node tracing config
Summary
It seems TraceMempool configuration is ignored in cardano-node 1.34.0
Steps to reproduce

Upgrade to 1.34.0
Set TraceMepool to false in config

Expected behavior
I do not see mempool trace logging
System info (please complete the following information):
Ubuntu 20.04 x64, cardano-node 1.34.0 built from source
Additional context
I see a lot of changes regarding tracing. Should we configure this differently? Anyway, this seems to be a regression for 1.34.0 vs 1.33.0 at least.

https://github.com/input-output-hk/cardano-node/blob/master/configuration/cardano/mainnet-config-new-tracing.yaml relevant? Should we list this is in the changelog maybe? Seems like a heavy change if everyone needs to edit config to alter tracing.

The release notes (or QA process - maybe it does not include testing EKG metrics against configuration for hydra builds) are a bit rushed.
The new tracing infra is summarised with a reference to a PR in changelogs, the hydra config templates referred in release notes are not updated either🙁

The release notes (or QA process - maybe it does not include testing comparison of EKG metrics against previous version given configuration for hydra builds) are a bit rushed.
The new tracing infra is summarised with a reference to a PR in changelogs, the hydra config templates referred in release notes are not updated eitherslightly_frowning_face

Quite a steep learning curve to understand how to configure tracing again. :( I'll try to figure out how to disable most of the trace logging which is pretty useless for day-to-day operation.

This should be fixed by #3668

Resolved in #3668


Click on image to pop-out!

The image is so small and barely readable. Provide the option to 'Click on image to pop-out!"

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

ID: e017b080-4692-bb90-8fd4-8ce1f65f4b4c
Version Independent ID: 980eb134-f777-4f09-db6a-a744038e061c
Content: End-to-end security in Azure
Content Source: articles/security/fundamentals/end-to-end.md
Service: security
Sub-service: security-fundamentals
GitHub Login: @TerryLanfear
Microsoft Alias: terrylan

Hello @savirod211 , thanks for taking time to share your valuable feedback! We are taking a look into this and will get back to you soon.

Hello!
Just checking to see if you have had a chance to see the previous response. Kindly let us know if it answers your question and if you need further assistance on this issue.

Hello!
Just checking to see if you have had a chance to see the previous response. Kindly let us know if it answers your question and if you need further assistance on this issue.

We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Issue 291 doc params

Description
Tutorial and notebook for parameter values. To be merged after #299 as there may be a clash
Fixes #291
Type of change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)

Key checklist:

[x] No style issues: $ flake8
[x] All tests pass: $ python run-tests.py --unit
[x] The documentation builds: $ cd docs and then $ make clean; make html

You can run all three at once, using $ python run-tests.py --quick.
Further checks:

[x] Code is commented, particularly in hard-to-understand areas
[x] Tests added that prove fix is effective or that feature works
[x] Any dependent changes have been merged and published in downstream modules

Codecov Report

Merging #303 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #303   +/-   ##
=======================================
  Coverage   97.63%   97.63%
=======================================
  Files          53       53
  Lines        3093     3093
=======================================
  Hits         3020     3020
  Misses         73       73

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 b573057...ef64ec1. Read the comment docs.

Thanks, updated


Support installation of specific versions of Mesos frameworks

Similarly to the existing support of installing Mesos from a specific version of Ubuntu package or from sources, enable users of this module to specify which versions of Marathon, Chronos etc. they want to install on the cluster.

Implemented in containers branch. Marathon can be run in any of versions listed here: https://registry.hub.docker.com/u/mesosphere/marathon/


Mechanism for deploying developed tools to Core OU accounts

Describe the issue

We need to develop a standard deployment approach for tools developed internally by the CPF team
One of the POCs reviewed was developed around the billing utility
It will be beneficial for the team to have a general solution that is not tied to a single utility

Additional context
There are two approaches being considered:
Distributed Services Model

In this model, Terraform state file, OIDC Identity Provider and associated role exists on each AWS ECF Core OU account.
This model provides multiple entry points for deployment to the AWS ECF -- one for each Core OU account. - Depending on the permissions model used by the OIDC assumed role, complexity on this approach would increase with added Core OU accounts and deployed resources.
An illustration of this model with the Core OU accounts: Operations and Management is presented below

Shared Services Model

For this model, Terraform state file, OIDC Identity Provider and associated role exists on a single AWS ECF Core OU account (eg: operations account).
This model provides a single entry point for deployment to the AWS ECF -- deployments to other account will be using cross account role permissions.one for each Core OU account.
Terraform state files for all Core OU deployments will be saved in a single account
The IAM role associated with the OIDC Identity Provider will need cross account permissions in order to deploy resources to accounts other than the one hosting the OIDC Identity Provider and role.
An illustration of the shared services model with the Core OU accounts: Operations and Management is presented below

Definition of done

Identity which model we'd adopt
Ideally, review and identify how tradeoffs on each model affects the team. Few areas that come to mind:

Permission boundary for deployment roles in target Core OU accounts
Implications with troubleshooting failed deployment
Kill switch option (easier with the shared services model)

Implementation of model agreed on

Nice to have, not urgent at the moment

not relevant


Add client ENV vars

Add missing CLIENT_* vars to docker-compose.yaml. CRA requires a separate .env from that in the parent directory.
Signed-off-by: PGobz<EMAIL_ADDRESS>

Removing myself from this PR so someone else can close it as I will be unavailable to review new changes.


fix: improper javadoc preventing release

Signed-off-by: Chris Laprun<EMAIL_ADDRESS>

Fast-tracked to get a next branch snapshot ASAP.


feat(STONE-841): remove Environment field from ITS CRD

Maintainers will complete the following section

[ ] Commit messages are descriptive enough (hints)
[ ] Code coverage from testing does not decrease and new code is covered (check the PR coverage on codecov)
[ ] Controllers diagrams are updated when PR changes controllers code  (if applicable)

As part of stopping provisioning ephemeral environments, we need to update our IntegrationTestScenario by removing environment field.

Add webhook that will convert v1beta1 version to v1beta2
Update unit test affected by this change
Remove all the code that will become redundat after applying this change
note: second and third point will be substracted from this PR after This PR gets merged

This PR duplicates a lot of work done in https://github.com/redhat-appstudio/integration-service/pull/572 why we don't merge 572 first and then just rebase, instead of reviewing the same code change twice?

Changes were tested following way:

Deployed integration-service latest version by infra-deployments.
Created v1beta1 integrationtest scenario with environment specified.
Deployed changes in this PR on the same cluster
Checked and confirmed the format of migrated v1beta2 scenario.
Example of migrated scenario:

/test integration-service-e2e

No comment from me. Will approve when all conversion are closed. Thanks!


Remove version: eTeks.SweetHome3D version 7.4

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://sourceforge.net/projects/sweethome3d/files/SweetHome3D/SweetHome3D-7.4/SweetHome3D-7.4-windows.exe/download
Status Code: 200
(Automated message - build 897)

URL: https://sourceforge.net/projects/sweethome3d/files/SweetHome3D/SweetHome3D-7.4/SweetHome3D-7.4-windows.exe/download
OK
(Automated message - build 923)

URL: https://sourceforge.net/projects/sweethome3d/files/SweetHome3D/SweetHome3D-7.4/SweetHome3D-7.4-windows.exe/download
OK
(Automated message - build 923)

Close with reason: Package still available;


Fetch tail previous logs when Cilium is in CrashLoopBackoff

During Cilium install with my own PR, I hit:
...
⌛ Waiting for Cilium to be installed and ready...

    /¯¯\
 /¯¯\__/¯¯\    Cilium:         3 errors
 \__/¯¯\__/    Operator:       OK
 /¯¯\__/¯¯\    Hubble:         disabled
 \__/¯¯\__/    ClusterMesh:    disabled
    \__/

DaemonSet         cilium             Desired: 2, Unavailable: 2/2
Deployment        cilium-operator    Desired: 1, Ready: 1/1, Available: 1/1
Containers:       cilium             Running: 2
                  cilium-operator    Running: 1
Cluster Pods:     0/3 managed by Cilium
Image versions    cilium             quay.io/cilium/cilium-ci:3378013ead0dca0263c1598ef8ede3dbe4f[25](https://github.com/cilium/cilium/runs/7735517531?check_suite_focus=true#step:7:26)318: 2
                  cilium-operator    quay.io/cilium/operator-generic-ci:3378013ead0dca0[26](https://github.com/cilium/cilium/runs/7735517531?check_suite_focus=true#step:7:27)3c1598ef8ede3dbe4f25318: 1
Errors:           cilium             cilium          2 pods of DaemonSet cilium are not ready
                  cilium             cilium-4j4r4    unable to retrieve cilium status: unable to upgrade connection: container not found ("cilium-agent")
                  cilium             cilium-nmzps    unable to retrieve cilium status: unable to upgrade connection: container not found ("cilium-agent")
ℹ  Rollback disabled with '--rollback=false', leaving installed resources behind
Error: Unable to install Cilium: timeout while waiting for status to become successful: context deadline exceeded

Given unable to retrieve cilium status: unable to upgrade connection: container not found ("cilium-agent"), it may be worthwhile to attempt to gather the --previous logs via kubectl and print the tail. This way there should be a much more specific error condition to look deeper into.

Taking a look at this.

I also filed https://github.com/cilium/cilium/pull/21012 to set the TerminationMessagePolicy: FallbackToLogsOnError for most pods in the manifests; this will mean we capture the "dying gasp" in the Status object. It persists across node reboots.

Aha; this isn't running kubectl logs, but rather kubectl exec ... -- cilium status. It's not going to work if Cilium isn't running; I'll make the result better regardless.


Allow making inherited options hidden on subcommands

This depends on https://github.com/remkop/picocli/issues/649.
Once inherited/global options are implemented, there will be cases where the top-level command defines many shared options, and each subcommands defines only the options applicable for that specific command. Applications may want to define some or all of the top-level options as "global" or "inherited" options, so they can be specified on the top-level command as well as on the subcommand.
However, they may not want the inherited options to clutter the usage help message of the subcommand, especially if there are many inherited options.
Programmatic Solution
Provide API to allow subcommands to hide inherited options. For example:
CommandLine::setHideInheritedOptions(boolean) // on the full hierarchy
CommandSpec::hideInheritedOptions(boolean) // on a specific command

Declarative Solution
Add an enumeration value to hide inherited options. For example:
public enum ScopeType {
    /** The element only exists in the current command. */
    LOCAL,
    /** The element exists in the command where the element is defined
     * and all descendents (subcommands, sub-subcommands, etc.). */
    INHERIT,
    /** The element exists in the command where the element is defined
     * and all descendents (subcommands, sub-subcommands, etc.),
     * but is not shown in the usage help for the descendents. */
    INHERIT_AS_HIDDEN,
}

TBD should this apply to inherited positional parameters?

Some work is in progress to address this: the tickets below make it easier to customize the option list in the usage help message. However, it is not yet possible to customize the synopsis...

[#1090] API: Add methods Help::optionListExcludingGroups to return a String with the rendered section of the usage help containing only the specified options, including hidden ones.
[#1092] API: Add method Help::parameterList(List<PositionalParamSpec>) to return a String with the rendered section of the usage help containing only the specified positional parameters, including hidden ones.
[#1091] API: Add method Help::optionListGroupSections to return a String with the rendered section of the usage help containing only the option groups.
[#1087] API: Add methods ArgumentGroupSpec::allOptionsNested and ArgumentGroupSpec::allPositionalParametersNested.
[#1086] API: add methods Help.Layout::addAllOptions and Help.Layout::addAllPositionals, to show all specified options, including hidden ones.

This is now implemented in master.
See these examples for how to accomplish this:

CustomOptionListAndSynopsis
CustomOptionListAndSynopsisWithInheritance


What happened?
I am following your just installation process on Ubuntu.  It looks like a file is missing in  pgai/projects/extension/sql/ai--0.6.1-dev.sql?
pgai extension affected
0.6.1
pgai library affected
No response
PostgreSQL version used
16
What operating system did you use?
Ubuntu 24.04
What installation method did you use?
Source
What platform did you run on?
On prem/Self-hosted
Relevant log output and stack trace
required sql file is missing: /home/usr/pgai/projects/extension/sql/ai--0.6.1-dev.sql

How can we reproduce the bug?
git clone https://github.com/timescale/pgai.git
cd pgai
just ext install

Are you going to work on the bugfix?
None

can you try just ext build; just ext install?

You are installing a development version: 0.6.1-dev. Are you sure that's what you want?

If you want the latest production version, check out the repo at the extension-0.6.0 tag. Then, just ext install will "just work".
https://github.com/timescale/pgai/releases/tag/extension-0.6.0

Thank you, pulling the 0.6.0 extension branch solved the error.  I was following instructions from claude.ai.


✏Editorial : Added error handling in index.js.

title: " Error Handling in index.js"

I am new to this open source contribution community. It would be helpful to know what were the problems with my branch.

@Manush-2005 hello - welcome!
Clicking through to https://github.com/asyncapi/spec/actions/runs/9412605514/job/25927798347?pr=1058 I can see that the title for your PR doesn't start with one of the keywords that the automation relies on

You can see this in the comment that the bot posted above


Support for files generated by Racc.

This changeset includes a sample racc file from this auto-generated
file
(MIT-licensed).
Racc Racc is an LALR(1) parser
generator. It is written in Ruby itself, and generates ruby programs.

:+1: thanks @lucky. This will be live in the next release of Linguist (later this week)


True north drag calibration only works on iPad and with camera smoothing disabled

Do you want to request a feature or report a bug?
Bug
What is the current behavior?
Using the following attributes on your a-camera to try and smooth it:
      look-controls-enabled='true'
      arjs-device-orientation-controls='smoothingFactor: 0.1'

Disables the ability to drag over the bottom of your screen to adjust true north
If the current behavior is a bug, please provide the steps to reproduce.
Just did
Edit: oh yeah, make sure location services are turned off too, that also seems to disable true north drag calibration (which is very annoying)
Please mention other relevant information such as the browser version, Operating System and Device Name
I can only get the drag to calibrate true north to work on an iPad, but it stops working when smoothing is enabled
What is the expected behavior?
Drag calibration on true north works on all devices and works when smoothing is enabled
I'll add, no idea what I've been calling 'drag calibration' is actually called, couldn't find anything on it in the docs.

@Platform-Group not sure what this "true north drag calibration" is, is it something enabled by default?
I don't believe it's part of AR.js, I haven't seen anything like this anywhere in the code so guessing it's part of A-Frame.
Note that arjs-device-orientation-controls is a replacement for the default DeviceOrientationControls component present by default in A-Frame, which fixes a few issues. So it's conceivable that this "true north drag calibration" feature is present in the default A-Frame code and not the replacement.
Do you still get the problem if you add the arjs-device-orientation-controls without the smoothingFactor?
i.e.
  arjs-device-orientation-controls

@Platform-Group not sure what this "true north drag calibration" is, is it something enabled by default?

@nickw1 Basically on iPad's I've found that the compass doesn't work, literally no idea why. Instead north is always the direction you're pointing the device, what you can do to adjust that is drag on the screen to rotate entities around you.
I assumed it was part of https://aframe.io/docs/1.4.0/components/look-controls.html as this mentions dragging.
No it doesn't work with only arjs-device-orientation-controls, my bad in mentioning the smoothing.

So from modifying look-controls further: on the iPad, look controls's updateOrientation function is being reached every tick, but it isn't on android. updateOrientation just isn't ever called, which doesn't seem right. So I looked into the tick function. And it simply seems that despite having look controls enabled in my html attributes, data.enabled is false within tick().
look-controls-enabled='true'
look-controls='enabled: true; magicWindowTrackingEnabled: true; touchEnabled: true; mouseEnabled: true'

Pretty weird that android decides to selectively change enabled to false somewhere. Anyway, looking into inspect element I see that
look-controls-enabled="false" despite the fact it's explicitly set to true. So something in ar.js is disabling it.

I think this is the only place in the codebase which disables this attribute:
    const mobile = this._isMobile();
    this.el.setAttribute("look-controls-enabled", !mobile);
    if (mobile) {
      this.el.setAttribute("arjs-device-orientation-controls", true);
    }

So my guess is that _isMobile is flawed. But even then I think that whole 4 line snippet should be removed as it's overriding control from the developer.

Thanks for that. Note that arjs-device-orientation-controls allows the smoothing; it won't work without it.
Again I'll put this on the todo list, but again feel free to submit a PR if you want it done quickly :-)


CLI: appsettings.json contents is not being loaded?

I'm debugging a Bot Service instance, running locally in the functions cli. I downloaded the appsettings.json file from the online instance, which was neatly copied into appsettings.json locally. When I run "func settings", it shows the correct output. Here is the output with a very simple (test) appsettings.json:

E:\Sources\github\bots-test>func settings list
-> Name: AD_CLIENT_ID
Value: *****

But now when I run the function (debug) in VS, and I have this line:
var test = System.Configuration.ConfigurationManager.AppSettings["AD_CLIENT_ID"];
that gives null. This seems so trivial that I'm completely lost on what is happening here. I've used the app settings in other functions before in the exact same way, never had any issues. Also tried decrypting the file first, nothing. So what could I be overlooking?

@jsiegmund As far as I'm aware, the contents of the "appsettings.json" file is not actually loaded into ConfigurationManager.AppSettings. This is somewhat counter-intuitive. I would use the environment variables API. See here for a quick example.

Note: we've recently moved the CLI to another repo https://github.com/Azure/azure-webjobs-sdk/issues/804. This issue should be moved there.


Pointers?

Hi,
Just discovered your code.
Seems super interesting, in particular the TK model.
I am wondering if there is a paper on TK model?
May be some scores and info on speed? (I have found nothing on the MsMarco leaderboard).
My main question: is it much more powerful than CKNRM and light enough to be usable in real scenario (not taking minutes to rerank candidates).

Hi,
Thank you for the fast interest!
I will present a tech report at TREC next week and make it available then. (Including updating the readme of this repo)
As a teaser: TK is 50 times faster than BERT-base and substantially better than CONV-KNRM across multiple collections. And TK is analyzable by design.
Best,
Sebastian

great teaser indeed!
Can't wait to check.
Just a last question before waiting for 1 week for more info, 50 times faster than Bert still mean 1-2 secs per query, right?
Just an advice btw FWIW, conv-knrm is supposed to be on par with Bert on ad hoc search (a different task by nature than msmarco where contextual understanding of words makes sense). May be in the future you want to add a benchmark on ad hoc search too.
Here for more info on Bert Vs CKNRM:
https://arxiv.org/pdf/1904.07531.pdf

Hi, want to let you know that I am very impatient, I tried TK V1 on my ad hoc search logs (I work for a legal publisher). Results are similar to those I get from CKNRM (almost the same score). I waited for 2 epochs (no more improvements).
I have seen in your config file that you use TK V6, so I will wait to check :-) (the code of TK V6 is not yet available in the repo)

That's very interesting - are the neural models better than bm25? in general there are many things we can do to tune the models, so for example how long are the documents/passages you are using?

Yep they are both 10 points better than a simple Bm25 on P@5. However I rerank only top 20 results (1 page of SERP), and we think our in prod system is far from being optimized (lots of SOLR boosters on some words not needed, etc.).
So BM25 score is BM25 applied to the 20 docs of each SERP (it gives better results that our prod sys).
I can also say that other reranker don't reach those results, they are not even on par with Bm25.
I use raw clicks. I added a little modif to both models to modelize bias position. It gives a little boost to both models but is of no need on MsMarco (AFAI it s already debiased by a click model).

Hi,
Sorry for answering so late. I uploaded our TREC tech report on arxiv now (https://arxiv.org/abs/1912.01385).
I would try a bigger re-ranking depth than 20 (in core_metrics.py there are methods to automatically evaluate all possible re-ranking depths at once with numpy) and also maybe you could try full documents instead of snippets (there is a tk_v2 in tk.py which uses windowed kernel pooling for longer document inputs, and it did pretty well in the TREC document ranking task)


add fsolve docs it doesn't handle over- or under-determined problems

fixes fsolve docs should say it doesn't handle over- or under-determined problems #7291

yeah sure, will follow those guidelines. Thanks.

Squashed and merged as https://github.com/scipy/scipy/commit/b643769138a7134cbb6098784e440be1044cc229
, also edited the commit message. Thanks @sudheerachary @tylerjereddy


How to read and write int？

Hello， How to read and write MSSQL int ?

int is i32 in Rust, so take a look how the test does it.
This probably needs some documentation I see, but you can read other mappings from the column_data.rs.

Documented in https://github.com/prisma/tiberius/pull/60


Question about the context

Why is the FeatureHost needing a context? Is it just for convenience?
If I'm right, it's save to pass the application context as context (if I don't need the context in my class).

As for now, this is just a convenience for features. Instead of giving features context via constructor features can use context of feature host. It's safe to pass application context in there.
Now I am working on an update where context type will be a parameter, meaning you will be able to use any type and not just android.content.Context. Since new update context type will be a part of Feature contract. It means your features can define which type of context they can work with. This will make feature contract even more strict.

So the @NonNull  of the context is there by accident, because it does not really need a context currently?
I just found this problem because I'm trying out https://github.com/bluelinelabs/Conductor and I wanted to log lifecycle events and constructor events via a feature...

It will thing of making it optional. For now it is required because this is the only what a feature can access context and you can give it application instance.

That answers my question. Actually, you can pass null as a context as well...


Is it ok to run linutil in the Linux subsystem for Windows?

I have watched CTT's recent YouTube video on linutil, and it sounds that is is designed to run on many distribution of Linux. Does that include the Lunux subsstem for Windows?

Sure, why not.

As long it passes the checks, I don't see why not.

Yes it should work

This issue has been answered.

/close


Import commits from apache/arrow

We need only commits for https://github.com/apache/arrow/tree/main/go . Other commits such as commits for .github/ aren't needed. We need to adjust other files for this repository.
We can do this by the following command lines:
git clone git@github.com:apache/arrow-go.git
cd arrow-go
git remote add arrow https://github.com/apache/arrow.git
git fetch arrow
git switch -c arrow-main-go arrow/main
git filter-branch --subdirectory-filter go
git switch main
git rebase arrow-main-go
git push origin --force main

I've tested the commands locally (without the push obviously) and the repository looks good on a first check. Should we run some testing to validate functionality before pushing?

Hmm. It seems that we misses some files. (There are some "[no test files]".)

This doesn't necessarily mean we missed files, not all directories contain test files. I've confirmed I get the same test output running on apache/arrow:main.

Thanks. I didn't know about it.

This LGTM, we can do a follow-up PR that will update all of the import paths across the code-base to the new URL

Thanks for confirming this approach. I'll do it soon.
FYI: Here are related issues:

BTW we should track if there is any new commit being pushed to the main arrow repo on the go side to manually cherry-pick it in the interim.

https://github.com/apache/arrow/issues/43881  [Go] Notify apache/arrow-go migration to all open PRs

we can do a follow-up PR that will update all of the import paths across the code-base to the new URL

#4 Update package name

Sorry. I haven't done this yet.
I'll do this now.

We need to remove the protected branch configuration in .asf.yaml:
$ git push origin --force-with-lease main
Enumerating objects: 13921, done.
Counting objects: 100% (13921/13921), done.
Delta compression using up to 24 threads
Compressing objects: 100% (3993/3993), done.
Writing objects: 100% (13921/13921), 13.79 MiB | 2.75 MiB/s, done.
Total 13921 (delta 9809), reused 13212 (delta 9721), pack-reused 0 (from 0)
remote: Resolving deltas: 100% (9809/9809), done.
remote: error: GH006: Protected branch update failed for refs/heads/main.
remote: error: Cannot force-push to this branch
To github.com:apache/arrow-go.git
 ! [remote rejected]     main -> main (protected branch hook declined)
error: failed to push some refs to 'github.com:apache/arrow-go.git'

I tried git cherry-pick arrow-main-go instead of git merge to avoid force push. But it causes conflicts...
Can we enable force push temporary? #80

Oh, (git log --format=%H --reverse arrow-main-go | while read commit; do git cherry-pick $commit; done) works without conflict. I'll push it without #80.

Pushed!


many strong mode errors are not correctly classified in Analyzer

I'm going through StaticWarningCode. Almost all of these should be errors in strong mode. There are a handful of exceptions that are bona fide warnings.
I'll send out a CL shortly working around this in DDC, but naturally, that is just a short term fix.

Perhaps the StaticWarningCode constructor should take an optional parameter indicating that it is a strong-mode error, which DDC (and other tools) could then use to promote warnings when running in strong mode. Just a thought.

https://codereview.chromium.org/1840203002/diff/20001/lib/src/report.dart for my attempt to workaround it on DDC side.

Yep. Hence my suggestion to mark them in the error codes.

CC @vsmenon @leafpetersen @munificent if any of you have a chance to take a look at this?
IMO it would be good to fix sooner rather than later, since lots of folks are in the process of getting packages strong mode clean.

BTW one option: we could probably just treat all StaticWarningCodes as errors.
The only categories that DDC allows are: uninitiated final, non-exhaustive enum, mismatched getter/setter types, and various incorrect use of "void" and/or return types.
That would leave our "down cast composite" as the only remaining warning.

This:
const StaticWarningCode ARGUMENT_TYPE_NOT_ASSIGNABLE_STATIC_WARNING = const StaticWarningCode(
is one of the most noticeable misclassified ones - since this is not a StaticTypeWarning, it doesn't get caught by our current heuristic to upgrade severity.

@leafpetersen -- I thought @munificent 's ErrorProcessor was already upgrading these to errors

Specifically https://codereview.chromium.org/1840203002/diff/20001/lib/src/report.dart around line 180 should hit https://github.com/dart-lang/sdk/blob/0c1e85ebadaa0680d4ff30acc1f3ac823c53756e/pkg/analyzer/lib/source/error_processor.dart#L126. Maybe the error type isn't ErrorType.STATIC_TYPE_WARNING even though it's a StaticWarningCode?

Didn't I just say that... ?  :)
Yes, I don't know why, but that warning is not a STATIC_TYPE_WARNING, it's a STATIC_WARNING, and so it doesn't get upgraded.

doh, yeah I misunderstood that :). But yeah, at least we're upgrading it on the DDC side now. Analyzer would be better though, for sure!

https://codereview.chromium.org/1861363002/ adds the hooks necessary to fix this issue. To finish, someone who knows which warning codes should not be errors should change the default value of the optional parameter in StaticWarningCode's constructor and add an argument of false where appropriate.

https://codereview.chromium.org/2108823002/ -- fixes this. From that CL I also link to the DDC side.


disabeling (global-display-fill-column-indicator-mode -1) does not work

Hello
I tried to disable (global-display-fill-column-indicator-mode -1), it works only with current session like M-x global-display-fill-column-indicator-mode,
where is it defined so that I can disable it ?

It’s not enabled globally, it’s added to prog-mode-hook, see init-editing-utils.el.


Preventing creation of users in an oauth scenario

We have a situation where we want to use our Google apps accounts to authenticate against an internal rails app.  We are using devise_token_auth and redux_auth to accomplish this.  It works great so far.
To perfect this, we need to prevent the creation of new user accounts upon a successful omniauth authentication.  We want to pre-register our users as the app is internal and the default behavior means that anyone can log in to the system with a google account.  Maybe there is not an option for this ATM because the omniauth_success method in the omniauth_callbacks_controller.rb saves the resource after sign_in?  Or, is there a way I'm not thinking of?

Hi there @chriskozlowski    ,
In an effort to cleanup this project and prioritize a bit, we're marking issues that haven't had any activity in a while with a "close-in-7-days" label. If we don't hear from you in about a week, we'll be closing this issue. Obviously feel free to re-open it at any time if it's the right time or this was done in error!
If you are still having the issue (especially if it's a bug report) please refer to our new Issue Template to provide some more details to help us solve it.
Hope all is well.


NIFI-6567 HandleHttpRequest does not shutdown HTTP server in some cir…

Description of PR
I added the @OnPrimaryNodeStateChange into HandleHttpRequest.class, it used to trigger shutdown web service when the Primary Node role revoked and also the processor is set to "Primary Only"
I also keep a boolean for processor which run on all node or primary node only because the @OnPrimaryNodeStateChange can't get the processor context.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[X] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[X] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[ ] Has your PR been rebased against the latest commit within the target branch (typically master)?

[ ] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[ ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

+1 LG


Responsive Error in login page

Login page shifts to left when width of screen decreases
The Login div shifts to left of the screen when width of screen decreases to 767px or less, which according to me stay at centre.
You can check your self
Goto this url ~ https://6.demo.plone.org/login?return_url=
reduce your screen width and see yourself.
Screenshots

Hey, I have a way to fix this, Just by placing the login Container in a grid.

@suyashrastogi7 I just submitted a different fix, which I think is better.

I can solve it easily, Assign it to me.

Is the issue still open ?? if yes i would like to work on it

@SaiRev0 Yes, it's still open.
One of the core principles in Plone (Volto's backend, and by extension also Volto) is that our software is a do-ocracy. Things will get done only if we start working on them. So, really, there's no task in Volto that's off-limits. There are, though, limits to the things that can be approached, realistically, by new contributors. There's back history, context, understanding of the overall vision, etc, that all come with experience.

Also, most important to read: https://6.dev-docs.plone.org/volto/developer-guidelines/contributing.html

@tiberiuichim Thanks, I m actually facing some issues in setting the repo in my local system so can you please help me out? is there any other way of communication other than here?

@SaiRev0 see the plone discord. https://plone.org/news-and-events/news/2021/join-plone-chat-now-at-discord

@tiberiuichim I have fixed the issue, So now I have to make a pull request, So is there some specific way of making a pull request?

@tiberiuichim please check my pull request and review it.

Is this issue still open? i think I can contribute

is the issue is still open ??.i want to work on this

@lord2anil and @varun-s22 please read the history of this issue to answer your own question and learn how to contribute to this repository.

Hi, I am still able to see this issue but have solved it and would like to contribute. Please assign me this issue.
Solved:-

I can solve it easily, Assign it to me.

The following pull requests are proposed solutions to this issue.

https://github.com/plone/volto/pull/3252
https://github.com/plone/volto/pull/4291
https://github.com/plone/volto/pull/4309
https://github.com/plone/volto/pull/4335
https://github.com/plone/volto/pull/4437
https://github.com/plone/volto/pull/4631

Before any pull requests can be considered by the Volto team, all the following requirements must be satisfied by all contributors.

Sign the Plone Contributor Agreement, and return it.
Open a pull request from your fork, not from the volto repository. This is a new requirement, for security purposes. Close any pull request made from the volto repository.
Follow contributing guidelines for Volto.

@alpha2207 @dobri1408 @suyashrastogi7 @VedantRaut-sudo @HyperDanisH @Shahbaz898414 @Monu01123

I have created a PR regarding this issue
Please check this out #4637


backgroundColor don't work in sceneStyle (Scene, Modal, Router)

Version

react-native-router-flux v4.0.0-beta.24
react-native v0.49.3
I use: sceneStyle={{backgroundColor: '#000'}}  , but the background don't change in Router, Modal, Scene

It does work with:
-react-native: 0.50.4
-"react-native-router-flux": "^4.0.0-beta.24"
<Router sceneStyle={{ backgroundColor: '#fff' }}>

thanks @tevatahiti ; now, how i can change the default background or styles on scenes or router, have some documentation for this version

I believe the doc for v4 is not yet ready as version is still in beta


Could you release the prepocess code for custom dataset?

Hi,
Thanks for sharing this interesting work. In the code of 'core/load_XXX.py', the loaded dataset '*.h5' has already been preprocessed. Could you release the preprocess code for the custom dataset？

Hi @tlok666 , we have provided the reference code/function process_[dataset_name].py for how we process the data in each of the load_[dataset_name].py data.

Hi,
Thanks for sharing your excellent work!
When I execute the core/load_h36m.py
https://github.com/LemonATsu/A-NeRF/blob/main/core/load_h36m.py#L134
I find it raised error as follows:
FileNotFoundError
[Errno 2] No such file or directory: 'data/h36m/S1_clean_bkgds.npy'
  File "/home/dx/projects/A-NeRF/core/load_h36m.py", line 134, in process_h36m_data
    bkgds = np.load(os.path.join(data_path, f"{subject.replace('s', '')}_clean_bkgds.npy"))
  File "/home/dx/projects/A-NeRF/core/load_h36m.py", line 453, in <module>
    data = process_h36m_data(data_path, subject, camera_name=camera, extend_iter=extend_iter)

Could you please provide these missing files?
Thanks a lot!


Add valgrind test

This is primarily to make it easy to generate a valgrind log file for manual checking at this moment.