Handle AIS data markings as labels

Currently in the elevator, AIS data markings are expressed using the Data Marking object from STIX.  It has been decided to express the AIS data markings as labels.
AIS data marking from STIX 1.x
			<marking:Marking_Structure xsi:type="AIS:AISMarkingStructure">
				<AIS:Not_Proprietary CISA_Proprietary="false">
					<AIS:AISConsent consent="EVERYONE"/>
					<AIS:TLPMarking color="WHITE"/>
				</AIS:Not_Proprietary>
			</marking:Marking_Structure>

Current JSON AIS Data Marking object produced by the elevator for STIX 2.1
    {
        "created": "9999-12-31T00:00:00.000Z",
        "created_by_ref": "identity--b0e91c16-4f0c-48c0-815c-1756a949b994",
        "definition": {
            "consent": "everyone",
            "is_cisa_proprietary": "false",
            "is_proprietary": "false",
            "tlp": "white"
        },
        "definition_type": "ais",
        "id": "marking-definition--f0185abf-3b5d-478c-a72d-60e9f7d53d33",
        "spec_version": "2.1",
        "type": "marking-definition"
    }

Notice that an AIS data marking "names" the TLP color associated with it.
Here is the definition of the AIS data marking labels:

Label Value
Description

cisa-proprietary-true
Consistent   with the Cybersecurity Information Sharing Act (CISA) of 2015 and any other   applicable provision of law, a cyber threat indicator or defensive measure   provided by a non-Federal entity to the Federal Government under this title   shall be considered the commercial, financial, and proprietary information of   such non-Federal entity when so designated by the originating non-Federal   entity or a third party acting in accordance with the written authorization   of the originating non-Federal entity. Other proprietary information that   falls under FOIA or other legal means will be considered PROPIN.

cisa-proprietary-false
The   cyber threat indicator or defensive measure is not considered proprietary   under the Cybersecurity Information Sharing Act (CISA) of 2015.

ais-consent-none
CISA   will not disclose the identity of the submitter and author of any Note or   Opinion SDOs, except as permitted in the AIS Terms of Use). This is the   default if no consent value is selected.

ais-consent-usg
CISA will   only share the identity of the submitter and author of any Note or Opinion   SDOs with Federal Entities.

ais-consent-everyone
CISA will   share the identity of the submitter and author of any Note or Opinion SDOs   with all AIS Participants and with Federal Entities.

Notice that the TLP marking isn't included - since that will reside in the object-marking-ref property.  Here is an example of how it would look:
 {
        "created": "2016-03-29T20:53:29.636Z",
        "description": "Add email information to spam filter",
        "id": "course-of-action--a6ba47ed-71a4-44fb-ae47-c8d1e18c6e5d",
        "labels": [
             "cisa-proprietary-false",
             "ais-consent-everyone"
        ],
        "modified": "2016-03-29T20:53:29.636Z",
        "name": "Malicious Emails",
        "object_marking_refs": [
            "marking-definition--613f2e26-407d-48c7-9eca-b8e91df99dc9"  #  TLP:White ref from the spec
        ],
        "spec_version": "2.1",
        "type": "course-of-action"
 }

@emmanvg: We should start working on this :-)


Added parsing for Recent Significant Weather e.g. REUP

Now handles REUP as request in your issue tracking.

Sweet, thanks!


Calculate nCr

Aim
Calculate nCr
Details
Given two integers n and r, find nCr. Since the answer may be very large, calculate the answer modulo 10^9+7.
Programming language

[ ] C
[x] C++
[ ] Java
[ ] Python

Assign this issue to me, under LGM-SOC
@Amit366 @ravikr126 @Ayush7614


Create a dark mode

Create a dark mode version of the app UI. Should be able to toggle between modes via switcher on profile dropdown.

i would like to work on that

I'm getting some issues to watch the files:
I did:
 ./cmd yarn watch

I get this:
Starting goodwork_redis_1 ... done
ERROR: Cannot create container for service laravel_echo_server: create .: volume name is too short, names should be at least two alphanumeric characters

Note: I ran the dev version with ./install.sh local

try this ./cmd -f docker-compose.dev.yml run --rm -w /var/www laravel_echo_server yarn watch

Works !

Given that the last activity on this was October 2019, I'm gonna tackle this one as I'm trying to improve branding customizability.


[Bug Report] AttributeError: module 'nmmo.lib.material' has no attribute 'Forest'

OS: Ubuntu 16.04
Description:

mkdir neural-mmo && cd neural-mmo
git clone https://github.com/neuralmmo/environment.git
git clone https://github.com/neuralmmo/baselines.git
cd environment && pip install -e .[all]
cd baselines && pip install -r requirements.txt
python train.py
Then, the error occurred:

Traceback (most recent call last):
  File "/home/xxx/neural-mmo/baselines/train.py", line 14, in <module>
    from model.realikun.baseline_agent import BaselineAgent
  File "/home/xxx/neural-mmo/baselines/model/realikun/baseline_agent.py", line 10, in <module>
    from model.realikun.policy import BaselinePolicy
  File "/home/xxx/neural-mmo/baselines/model/realikun/policy.py", line 5, in <module>
    from env.nmmo_team_env import NMMOTeamEnv
  File "/home/xxx/neural-mmo/baselines/env/nmmo_team_env.py", line 8, in <module>
    from feature_extractor.feature_extractor import FeatureExtractor
  File "/home/xxx/neural-mmo/baselines/feature_extractor/feature_extractor.py", line 6, in <module>
    from feature_extractor.map_helper import MapHelper
  File "/home/xxx/neural-mmo/baselines/feature_extractor/map_helper.py", line 9, in <module>
    from nmmo.io import action
ModuleNotFoundError: No module named 'nmmo.io'

I solved the error ModuleNotFoundError: No module named 'nmmo.io' by changing from nmmo.io import action to from nmmo.core import action in /home/xxx/neural-mmo/baselines/feature_extractor/map_helper.py", line 9.
However, another error occurred when I run python train.py:

Traceback (most recent call last):
  File "/home/xxx/neural-mmo/baselines/train.py", line 14, in <module>
    from model.realikun.baseline_agent import BaselineAgent
  File "/home/xxx/neural-mmo/baselines/model/realikun/baseline_agent.py", line 10, in <module>
    from model.realikun.policy import BaselinePolicy
  File "/home/xxx/neural-mmo/baselines/model/realikun/policy.py", line 5, in <module>
    from env.nmmo_team_env import NMMOTeamEnv
  File "/home/xxx/neural-mmo/baselines/env/nmmo_team_env.py", line 8, in <module>
    from feature_extractor.feature_extractor import FeatureExtractor
  File "/home/xxx/neural-mmo/baselines/feature_extractor/feature_extractor.py", line 6, in <module>
    from feature_extractor.map_helper import MapHelper
  File "/home/xxx/neural-mmo/baselines/feature_extractor/map_helper.py", line 34, in <module>
    material.Forest.index: material.Scrub.index,
AttributeError: module 'nmmo.lib.material' has no attribute 'Forest'

9. Thus, there are some bugs, can the baselines repository run successfully?

Hi, the problem I have encountered is the same as yours, and I am trying to solve it. Do you have any solutions now? Thank you.


Reverse for 'password_reset_confirm' not found

Route :/request_password_reset does not seems to work. I got error when send post request
django.urls.exceptions.
NoReverseMatch: Reverse for 'password_reset_confirm' not found. 'password_reset_confirm' is not a valid view function or pattern name.

There was an issue with  template tag in password_reset_email.html. I finally solved it.


Dispatcher macros don't work with template functions

I'm trying to figure out how to generate the macros when using the dispatcher.
First, one need to add -DEMIT_DISPATCHER and the list of supported platforms. But it's not really great for cross platform automated build .
Then, my functions are templated factory builders, and this doesn't work...
I was wondering if using the get compilable macro with cmake could generate a comma separated list (easy to do with CMake) and then use it with Boost preprocessor macros to generate the proper code in a template acceptable way.
Any thoughts on this? Without this 2 features, the SIMD filters I'm trying to build for my library are just unusable :/

Regarding your first observation, I'm not fully sure I understood it correctly. If I misunderstood and my answers don't actually explain the situation, could you clarify your question?
The reason why each supported architecture is supplied as a separate flag is that platforms may be combined one with another. For example, -DSIMDPP_DISPATCH_ARCH1=SIMDPP_ARCH_X86_AVX2,SIMDPP_ARCH_X86_FMA3 is a valid platform which is distinct from both AVX2 and FMA3. I haven't figured out a maintainable way to supply this information as a single flag. I'm open to suggestions of how this could be improved.
Unfortunately, there's no way to avoid this. At least one compilation unit needs to be aware of all architectures that will be dispatched, because there's no cross platform way to statically register them. If we try to rely on constructors of static objects doing this work, then there will be no hard link-time dependency on the compile units the constructors sit in and the linker will strip them out on lots of platforms.
Regarding the template functions, are the required template instantiations known beforehand? That is, if you wrote your template code without using libsimdpp, could you put the definition of the templates into separate .cpp files and force the instantiation explicitly? If yes, then it's possible in theory to support your use case, I just need some time to think about how it could work. If not, unfortunately it's not possible to use libsimdpp dispatcher for this task, because libsimdpp relies on code for different architectures to be put in separate compile units. The compiler needs to know what code to generate for the templates and the only way to do so in such situation is instantiating explicitly in one way or another. All SIMD libraries will have this problem, because only some compilers support generation of code for different instruction sets in the same compile unit.

I do instantiate everything I need beforehand (https://github.com/mbrucher/AudioTK/blob/feature/SIMDBasicFilters/ATK/Core/SIMD/QuaternionConvertFilter.cpp#L92). I already have the functions creating the objects, but how to do the rest...
I wonder if with Boost tuples, we could manage to generate this properly... Does the compilable architecture function retrieves all the combinations, or is it something else that has them?
I do agree that there is an issue for runtime discovery with static libraries, but they should get stripped on shared libraries, shouldn't they? I'm wondering if 2 options, static and runtime dispatches, could solve the problem for complex classes like the one I have. It's also problematic that the CMake files don't seem to support the dispatcher (yet). But once we have a final solution, I can help with the creation of the required functions, no problem.
I could write the dispatcher as std::unique_ptr<> something_float_simd, but it seems a little bit off, as the rest of the library uses template arguments. I could do with if there is really no other way though.

Dispatching via shared libraries is actually quite easy -- compile several libraries with different SIMDPP_ARCH_* flags and then just select correct library according to the cpuid results and dlsym symbols from it.
I think I've figured a way to support template functions in the dispatcher. I'll have some code in a custom branch after several days.
Regarding

CMake files don't seem to support the dispatcher
Did you look into simdpp_multiarch? Is it adequate for your use case?

Obviously I haven't looked enough!
So there is one variable DISPATCHER_FILE for indicating the dispatcher file. What does this mean? Do I still need the full DISPATCH macro in all files, and I set DISPATCHER_FILER to one of them? How do O chose it as I don't know there full names?
Should probably be a list of files actually, instead of just one, as each argument file could require a dispatch file.
Glad to here about your potential solution, that would be great!

OK, the DISPATCHER part is done automatically, I should read the cmake file better! I think I start understanding this :)

Did you start working on the implementation? Or if you have an idea, I can participate in the implementation. I will definitely try it as soon as you have a first version ;)

Sorry for a long delay in responding. Yes, I did start working on the implementation and it's almost finished. It turned out to be more complex than I anticipated, so I decided not to push anything unfinished, as it would very likely waste your time. It's hard to predict when I have something for you to use, but it's likely that during weekend I'll have more time to wrap things up.

@mbrucher I've pushed an experimental implementation on the dispatch-templates branch. Some docs are in the source an several examples can be seen in tests. It would be great to hear any issues, suggestions or ideas that you might have once you start using the new functionality.
I've only tested the implementation on GCC, so the implementation could still contain bugs. More testing results will be available soon.

Excellent, thanks a lot!
I'll try it with clang and VS 2017 as soon as I have time playing with it (might take some days, I'm stuck with another project to finish first!).

Currently trying to implement this for the following function:
template std::unique_ptr createRealToQuaternionFilter<float,simdpp::float32<4> >(std::size_t);
Seems like it tries to to deduce the arguments from the following declaration:
SIMDPP_MAKE_DISPATCHER((template<typename DataType_, typename SIMDType>) (std::unique_ptr) (createRealToQuaternionFilter)
((std::size_t) nb_channels))
and can't figure out DataType_ (and probably SIMDType).
I thought the MAKE_DISPATCHER call would actually return calls to createRealToQuaternionFilter<DataType_, SIMDType>, as it seemed from the doc that it should have worked? (https://github.com/p12tic/libsimdpp/blob/dispatch-templates/simdpp/dispatch/make_dispatcher.h#L212)

Thanks for feedback. It looks like I made an error when writing documentation and the implementation indeed does not work with non-deducible arguments. I've merged your PR with one modification - the new macro parameter was moved to the second position in the argument list. Having template-related arguments together reduces the chance of user errors which in this case result in quite nasty compiler output.

Excellent, thanks a lot!
And really great work with the library. It's really great to have a way of not writing intrinsics
and still support all these different platforms ;) I like portable code.

FYI, I saw errors when compiling dispatch tests on MSVC on the dispatch-templates branch, so I think we can keep the issue open for the time being :-)

OK, I'll have a look as well.

Seems to be building on VS2017 for the template dispatchers I use.

I noticed something strange. On AVX, if I do a vector<>.assign() with the zero value in some conditions, it triggers a segmentation fault. At least, that's what happens on Linux, not on macOS or VS (that I noticed). Is there something special about AVX registers compared to the others?

Could you give more information about the circumstances of the crash? Does this happen in dispatched code? What instruction is being executed when the crash happens?

Seems like it might be an alignment issue in my code (error signal: SIGSEGV, si_code: 0), so I'm investigating this lead more.

OK, was my problem with a bad understanding of std::align...

I think I managed to reproduce the compilation issue on Windows, I'll have a look at it.

Seems like all the sequence stuff doesn't work with VS2017, but I don't know much about preprocessor :/
It's just as if it would process the macro SIMDPP_DETAIL_EXTRACT_PARENS_IGNORE_REST keeps the rest of the call instead of ditching it. So even the simple instantiation fails :/

Seems like all the sequence stuff doesn't work with VS2017, but I don't know much about preprocessor :/
It's just as if it would process the macro SIMDPP_DETAIL_EXTRACT_PARENS_IGNORE_REST keeps the rest of the call instead of ditching it. So even the simple instantiation fails :/

I think I've fixed the issues on MSVC in 2822508bd1fb6459d and merged the branch to master. Could you please check whether the code on master branch fixes your issues?

Compiles properly, thx a lot!


m_cellularNoiseLookup ownership

FastNoise has a member m_cellularNoiseLookup which is initially a null pointer, and can be set with SetCellularNoiseLookup. The problem is that FastNoise has a destructor defined as follows:
~FastNoise() { delete m_cellularNoiseLookup; }

This leads to a few API design bugs:

If you make a copy of a FastNoise object with a cellular noise lookup, it'll call delete twice on the same pointer. This will likely crash the program.
If you call SetCellularNoiseLookup twice, it'll leak the memory passed to the first call.
Pointer to a FastNoise object allocated on the stack will cause a crash (since it wasn't allocated with new).
The user may not expect that FastNoise will delete their pointer, and the user might delete it a second time themselves by accident.

I have recommend two alternative designs to suggest to solve this problem:
Design 1: Use a smart pointer
Change the type of m_cellularNoiseLookup to std::unique_ptr<FastNoise> or std::shared_ptr<FastNoise>. This will solve all the problems with double-deletion and memory leaks, but will effectively force users to allocate the FastNoise object on the heap. It is not idiomatic to create a smart pointer that points to an object on the stack, so users will probably avoid it.
Changes required:

Include <memory>
Remove the line ~FastNoise() { delete m_cellularNoiseLookup; }
Remove the line void SetCellularNoiseLookup(FastNoise* noise) { m_cellularNoiseLookup = noise; }
Add the line void SetCellularNoiseLookup(std::unique_ptr<FastNoise> noise) { m_cellularNoiseLookup = move(noise); }
Remove the line FastNoise* GetCellularNoiseLookup() const { return m_cellularNoiseLookup; }
Add the line FastNoise* GetCellularNoiseLookup() const { return m_cellularNoiseLookup.get(); }

Design 2: Use a non-owning pointer
To implement this design, simply remove the destructor.
This makes the FastNoise object not have ownership of the lifetime of its cellular noise lookup member, so it becomes the API user's responsibility to ensure that the pointer is valid throughout its usage.
I think this would be the simpler solution. It would allow users to pass in a pointer to a stack object. It would also allow FastNoise objects to be copied while retaining the same reference to cellular noise lookup. However, it would break existing code that assumes that FastNoise will automatically delete its cellular noise member.

Design 2 makes sense now that you added the GetCellularNoiseLookup(). So I've pushed this change.


Add the ability to stream (send messages) between distributed actors

This issue will track the work to add a pub/sub event messaging capability. The minimal api required would be:
    public <T> void publish(TopicId<T> id, T message);

    public Subscription subscribe(TopicId<T> id, Consumer<T> messageConsumer);

A TopicId would need to be created, which could be exactly like PropertyId.
Perhaps the PropertyService could be enhanced to add these methods. If added to the PropertyService, the PropertyService would need to be renamed, perhaps to Remote or TigerBus.
Additionally, a new Stream type should be added, TopicStream, which would inherit from EventStream.

Added an EventBus and a HazelcastEventBus with the api detailed above. This was pushed in PR #4, which wrongly reference Issue #2. This completes the implementation of this Issue.


Create a directive to convert a pull-request into a CL

This is being done internally, but wanted to track the feature externally.
It should look something of the form:
moe github_pull --pull_request http://github.com/google/guava/pull/5 --config /path/to/config

The directive should validate that that one of the repositories is a github repository that matches the pull request, and it will attempt to stage the commits on that branch into a client/workspace in the other repository.
This might be more generalizable to something of the form:
moe stage_branch --branch repo_name:foo --root <commit> (if needed) --config /path/to/config

Really a github pull request is just fluff around a branch.  But github is by far the more common use-case, so supporting that is the priority.

So - this sort of exists.  github_pull and merge_branch both exists.  I need to sync out the docs, and update things, and there's a rather stark bug in github_pull that relates to the Migrator's logic being way too linear. I'm working on that. For now I'll leave this open.


Addon registers too many CVars

You are using CVars for a lot of things that could be SavedVariables... This makes WoW alert it and refuses to create them.
You should really just use Saved Variables :)
good idea tho

Hey, yea I only started messing with CVar's as an experiment. I've not run into the error with to many CVars but I dont use many other addons so that's logical...
I will go through all the variables and make those who doesnt need to be CVar's just simple saved variables soon. Thanks for reporting!!

This should be fixed in v1.0


”Găgești”, ”Giurcani”, ”Peicani”, ”Popeni”, ”Tupilați” are listed in the drop-down menu

Description:
When user writes ”Găgești”, ”Giurcani”, ”Peicani”, ”Popeni”, ”Tupilați” the locations appear in the drop-down menu next to the comune and the county which is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Găgești"

Expected results
The location appears in the drop-down menu as "Sat Găgești, VASLUI (Găgești)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Giurcani"

Expected results
The location appears in the drop-down menu as "Sat Giurcani, VASLUI (Găgești)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Write in the search bar "Peicani"

Expected results
The location appears in the drop-down menu as "Sat Peicani, VASLUI (Găgești)".

Step 6
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 7
Write in the search bar "Popeni"

Expected results
The location appears in the drop-down menu as "Sat Popeni, VASLUI (Găgești)".

Step 8
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 9
Write in the search bar "Tupilați"

Expected results
The location appears in the drop-down menu as "Sat Tupilați, VASLUI (Găgești)".


FLAKE: c_macos_opt_native.cmake/build/fork_test

Lidi, even though this is a problem in core, I'm assigning this to you, since I believe python is the only language that depends on fork support.
https://source.cloud.google.com/results/invocations/8fa5ae2d-97cd-4d30-9880-dd4d8373a80f/targets/github%2Fgrpc%2Frun_tests%2Fc_macos_opt_native/tests
D0730 08:55:57.985197000<PHONE_NUMBER> test_config.cc:161]                test slowdown factor: sanitizer=1, fixture=1, poller=1, total=1
E0730 08:56:06.001663000<PHONE_NUMBER> fork_test.cc:86]                   assertion failed: gpr_time_similar(end_time, est_end_time, tolerance)
*** SIGABRT received at time=1627660566 ***
PC: @     0x7fff60e852c2  (unknown)  __pthread_kill

Assigning to @ericgribkoff who lead the Python fork support effort.

Closing since we decided not to track test flakes on github


pandoc export failed: TypeError: Cannot read properties of underlined (reading 'path')

I get this error when I try to export.

The same here

You need to add more details if you want someone to actually look at this problem.
What were you doing?
What did you expect to happen?
What actually happened?
What operating system are you running?
What version of obsidian are you running?
What plugin version are you running?
Can you post the entire output of the error?
Is there anything in the developer tools that provides more information (ctrl+shift+i/cmd+option+i)
Do you have a screenshot?
Put an example of the file you're trying to export.


Log test node metadata: depends_on_nodes

Describe the feature
Currently the extra field from the structured log record is providing unique_id and run_state for test nodes.
{"timestamp": "2020-04-13T08:01:21.445113Z", "message": "10:01:21 | 2 of 385 START test assert_nnull_refid............................... [RUN]", "channel": "dbt", "level": 11, "levelname": "INFO", "thread_name": "Thread-6", "process": 13649, "extra": {"unique_id": "test.pmt_curated.assert_nnull_refid", "run_state": "running"}}

Include additional metadata field depends_on_nodes from the test node
{"timestamp": "2020-04-13T08:01:21.445113Z", "message": "10:01:21 | 2 of 385 START test assert_nnull_refid............................... [RUN]", "channel": "dbt", "level": 11, "levelname": "INFO", "thread_name": "Thread-6", "process": 13649, "extra": {"unique_id": "test.pmt_curated.assert_nnull_refid", "run_state": "running", "depends_on_models": ["modelA", "modelB"]}}

Who will this benefit?
People analyzing the logs for getting pipeline metrics can relate a failed test to a model or set of models.
Also asked in in Slack.

I'd be happy to help with this if you think it's worth it.

Sure, I think this is a good idea! I would just call it depends_on instead of depends_on_models, since the test could conceivably be run against a seed, source, or snapshot too.
We'd love a PR for this one - let us know if there's anything we can help out with when you get to implementing :)

Hey! I saw the depends_on_models attribute can be easily logged with a minimal change (check #2341 ). This will log the dependencies in the Began running node stage, but not in the test report as it was pictured in the issue description.
{"timestamp":"2020-04-18T07:54:45.832149Z","message":"Began running node test.pmt_curated.unique_request_ref_id","channel":"dbt","level":10,"levelname":"DEBUG","thread_name":"Thread-1","process":58947,"extra":{"node_alias":"unique_request_ref_id","node_schema":"None","node_database":"dwh","node_path":"models/marts/pmt_analytics/request/request.yml","node_name":"unique_request_ref_id","resource_type":"test","depends_on":["model.pmt_curated.request"],"node_index":28,"node_materialized":"view","node_started_at":"2020-04-18T07:54:45.832192","unique_id":"test.pmt_curated.unique_request_ref_id","run_state":"running"}}

I am wondering if I should leave it like this or I should add the depends_on in the test report line as well. If that's the case, could you maybe point me at which level you see this should be resolved? I was thinking somewhere in the call_runner function
https://github.com/fishtown-analytics/dbt/blob/c05b45b241b6394222675e63b98dfb19d4633023/core/dbt/task/runnable.py#L155
or in the TestRunner
https://github.com/fishtown-analytics/dbt/blob/c05b45b241b6394222675e63b98dfb19d4633023/core/dbt/node_runners.py#L544
but I can definitely some guidance here.

I think we'd probably want to make a new log context manager (similar to this one called DependsOn (or something like it). This handler would need to implement a mapping_keys method which contains something like:
('depends_on_nodes', 'depends_on'),

So, pretty similar to what you've implemented in #2341
Once the DependsOn handler exists, you'd want to use it as a context manager in the GraphRunnableTask wrapping the end-of-test debug log line.
I think we could alternatively just use extended_metadata as an additional context manager here in conjunction with the change you've already made in #2341.
@beckjake do you have any further thoughts / preferences / ideas on this one?

I don't really have strong feelings about it. That ModelMetadata is only used once, at the beginning of running a model (rather than the end), which seems acceptable to me. I was pretty satisfied with #2341. Is the concern around logging this information when we're not running a test? Because that doesn't seem so awful to me.

I think the goal here is to get depends_on in the same log line as the test failure. Right now, the node metadata is going to show up in the log line for the node start, but not the log line for the node end. I imagine pushing the depends_on list in the same log line as the ultimate status info is the desired goal here

If it helps with the discussion around the scope, logging at the beginning of running a model already solves my use case. If logging at the test report line as well is preferable, I have no problem to add the change there too. Using the extended_metadata sounds good to me in this case.


feat: Pasting Markdown Support

Describe the feature you'd like to request
As in gitbook.io when we paste the markdown it will be auto formatted to the rich text.
When we paste normal markdown to novel editor it doesn't shows the rich text
Describe the solution you'd like to see
Try copying the below or any sample md file and pasting on th eeditor it will be pasted as normal text or markdown....
Additional information
## Adding Plyr to Custom HTML5 Video with Additional Features

#### Introduction
Plyr is a lightweight, accessible, and customizable media player for the web that supports HTML5 video, audio, and YouTube. This article will guide you through the process of integrating Plyr into your custom HTML5 video setup, including additional features like customizing the skin and adding a download button.

{% youtube https://www.youtube.com/watch?v=SR8pFHpsC8c&ab_channel=ShadeTech %}

> https://github.com/techshade/how-to/blob/main/plyr/adding-plyr-to-html5-video-player.md

#### Prerequisites
Before starting, ensure you have basic knowledge of HTML, CSS, and JavaScript. You'll also need a code editor and a web browser for testing.

#### Step-by-Step Guide

##### Step 1: Set Up Your Project
Create a project directory and set up your HTML file.

```plaintext
project-directory/
    ├── index.html
    └── css/
        └── styles.css
    └── js/
        └── scripts.js

Step 2: Include Plyr CSS and JavaScript
Plyr can be included via a CDN for ease of use. Add the following lines to the <head> section of your index.html file.
<!DOCTYPE html>
<html lang="en">
<head>
    <meta charset="UTF-8">
    <meta name="viewport" content="width=device-width, initial-scale=1.0">
    <title>Custom HTML5 Video with Plyr</title>
    <!-- Plyr CSS -->
    <link rel="stylesheet" href="https://cdn.plyr.io/3.7.2/plyr.css" />
    <!-- Your custom CSS -->
    <link rel="stylesheet" href="css/styles.css">
</head>
<body>

Add the Plyr JavaScript at the end of the <body> section, along with your custom JavaScript file.
    <!-- Your custom JavaScript -->
    <script src="js/scripts.js"></script>
    <!-- Plyr JavaScript -->
    <script src="https://cdn.plyr.io/3.7.2/plyr.js"></script>
</body>
</html>

Step 3: Add HTML5 Video Element
In the body of your HTML file, add a video element with custom attributes.
<body>
    <video id="player" playsinline controls>
        <source src="path/to/your/video.mp4" type="video/mp4" />
        <!-- Add more sources if needed -->
    </video>
</body>

Step 4: Initialize Plyr
Now, initialize Plyr in your scripts.js file. This will replace the standard video player with Plyr's player.
document.addEventListener('DOMContentLoaded', () => {
    const player = new Plyr('#player');
});

Step 5: Customize Plyr Controls
Plyr offers extensive customization options. For instance, you can customize the controls by passing an options object when initializing Plyr.
document.addEventListener('DOMContentLoaded', () => {
    const player = new Plyr('#player', {
        controls: [
            'play-large', // The large play button in the center
            'restart', // Restart playback
            'rewind', // Rewind by the seek time (default 10 seconds)
            'play', // Play/pause playback
            'fast-forward', // Fast forward by the seek time (default 10 seconds)
            'progress', // The progress bar and scrubber for playback and buffering
            'current-time', // The current time of playback
            'duration', // The full duration of the media
            'mute', // Toggle mute
            'volume', // Volume control
            'captions', // Toggle captions
            'settings', // Settings menu
            'pip', // Picture-in-picture (currently Safari only)
            'airplay', // Airplay (currently Safari only)
            'fullscreen' // Toggle fullscreen
        ],
        settings: ['captions', 'quality', 'speed', 'loop'],
    });
});

Step 6: Add Custom CSS (Optional)
To style the Plyr player or adjust your page layout, you can add custom CSS in styles.css.
body {
    font-family: Arial, sans-serif;
    margin: 0;
    padding: 0;
    display: flex;
    justify-content: center;
    align-items: center;
    height: 100vh;
    background-color: #f0f0f0;
}

#player {
    width: 80%;
    max-width: 800px;
}

Step 7: Adding a Download Button
To add a custom download button, we need to extend the Plyr controls and add some custom JavaScript. Modify your Plyr initialization code in scripts.js:
document.addEventListener('DOMContentLoaded', () => {
    const player = new Plyr('#player', {
        controls: [
            'play-large', // The large play button in the center
            'restart', // Restart playback
            'rewind', // Rewind by the seek time (default 10 seconds)
            'play', // Play/pause playback
            'fast-forward', // Fast forward by the seek time (default 10 seconds)
            'progress', // The progress bar and scrubber for playback and buffering
            'current-time', // The current time of playback
            'duration', // The full duration of the media
            'mute', // Toggle mute
            'volume', // Volume control
            'captions', // Toggle captions
            'settings', // Settings menu
            'pip', // Picture-in-picture (currently Safari only)
            'airplay', // Airplay (currently Safari only)
            'download', // Custom download button
            'fullscreen' // Toggle fullscreen
        ],
        settings: ['captions', 'quality', 'speed', 'loop'],
    });

    // Adding event listener to the custom download button
    const downloadButton = document.createElement('button');
    downloadButton.classList.add('plyr__control');
    downloadButton.innerHTML = '<svg role="presentation" focusable="false"><use xlink:href="#plyr-download"></use></svg>';
    downloadButton.addEventListener('click', () => {
        const videoSrc = document.querySelector('#player source').src;
        const a = document.createElement('a');
        a.href = videoSrc;
        a.download = 'video.mp4';
        document.body.appendChild(a);
        a.click();
        document.body.removeChild(a);
    });

    // Insert the download button before the fullscreen button
    const controls = player.elements.controls;
    const fullscreenButton = controls.querySelector('.plyr__control--fullscreen');
    controls.insertBefore(downloadButton, fullscreenButton);
});

Step 8: Customize the Skin
To customize the skin of Plyr, you can override the default CSS rules in styles.css.
/* Change the color of the play button */
.plyr--full-ui .plyr__control--overlaid {
    background: #1e90ff;
}

/* Customize the progress bar */
.plyr--full-ui .plyr__progress__container {
    background: #d3d3d3;
}

.plyr--full-ui .plyr__progress__buffer {
    background: #a9a9a9;
}

.plyr--full-ui .plyr__progress__played {
    background: #1e90ff;
}

Conclusion
You have successfully integrated Plyr into your custom HTML5 video setup, customized the controls, added a download button, and customized the skin. Plyr provides a sleek, consistent, and feature-rich media player that enhances the user experience. For more advanced configurations and updates, refer to the official Plyr documentation.
By following this guide, you can provide a highly customizable and user-friendly video playback experience on your website.

Useful Links

https://codexdindia.blogspot.com/2021/05/plyrio-video-player-integration-skin-Customizing-and-Adding-Download-Button-to-plyr.html
https://dev.to/sh20raj/sample-videos-for-testing-4a30

would love to have this feature

+1


Consider pre-commit-ci/lite

If I understand correctly, we could use https://github.com/pre-commit-ci/lite-action instead of the autocommit action to apply changes to PRs. This may work better than the current approach which creates a separate commit but doesn't run CI on it.
We'd still have to run pre-commit ourselves.

I just installed it for essnmx.

Here is what I tested in essnmx: https://github.com/scipp/essnmx/pull/12.
Applied wrong format on purpose and pre-commit fixed them

Pre-commit CI made a commit with fixes

Final CI tests passes!

I think it's very neat and we should use this one instead.
First we need to install them for all repositories I guess,
and then I can make the change here.

And it actually ran the CI pipeline on the new commit!
I'm happy with it, any objections @scipp/scipp-maintainers ?


Improve comparison operators type checking to disallow unions containing numbers as an operand

fixes https://github.com/microsoft/TypeScript/issues/52036
at this moment this is just a spike to check if this is even a desired change

@typescript-bot test this
@typescript-bot test top100
@typescript-bot user test this
@typescript-bot user test tsserver
@typescript-bot test tsserver top100
@typescript-bot run dt
@typescript-bot perf test this

Breaks from an internal codebase (an ancient snapshot FWIW):
                animateOverlayPanel: (deltaY?: Number) => {
                    if (typeof(deltaY) !== 'undefined' && deltaY < 0 && !!this.state.currentItem.url) {
                                                          ~~~~~~~~~~
!!! error TS2365: Operator '<' cannot be applied to types 'Number' and 'number'.

        if (item && item.redacted && item.redacted > 0) {
                                            ~~~~~~~~~~~~~~~~~~~~~~~~
!!! error TS2365: Operator '>' cannot be applied to types 'string | number' and 'number'.

                } else if (latin1ToGsm[codePoint] < -1) {
                           ~~~~~~~~~~~~~~~~~~~~~~~~~~~
!!! error TS2365: Operator '<' cannot be applied to types 'string | number' and 'number'.
                    scalarGsmEncodedLength.push(2);
                } else {

                    urlPreviewInfo.content_type === FileTypeHelper.imageContentTypes.gif
                        && urlPreviewInfo.size
                        && urlPreviewInfo.size > this._maxGifFileSizeForIOs;
                           ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
!!! error TS2365: Operator '>' cannot be applied to types 'string | number' and 'number'.

Most of the breaks here look very desirable to me. The Number vs number one should probably be corrected though.

@RyanCavanaugh the union cases that you mentioned are conceptually the same as the issue that I'm trying to fix here, so I guess that if we decide to move this PR forward (in some shape) then that would be a breaking change for those situations. I don't see any additional heuristics that would keep those cases working while erroring on the case linked to this PR.
The Number case is an edge case but well, conceptually it's a valid one - so this could be fixed here.
Cases discovered by the top are also related to unions and one is related to unknown. There are also some LSP issues reported here. It's surprising that this has affected LSP anyhow so I wonder if those are not false positives somehow but I could investigate this further and fix the issues if they are legit.
All in all, it kinda boils down to making a decision about the desired behavior here and only your team can do that. I'm more than happy to work on an adjusted algorithm for this but I'd have to know what that is (if it would be any different from what I have in this PR).
Let me know what you think, if you want to act on this anyhow or if you prefer to close the PR. From my PoV, most of the new errors are desired - those cases won't throw, so they are "type-safe" but they rely on some implicit coercions~ that are often super easy to miss. This change would require adding some additional typeof x === 'number' to codebases but I'd consider the RoI worth it.

I see in the latest design notes that:

Conclusion: Feels like this PR is a good start. Want to bring this in for 5.0.

What are the next steps here then? Which cases should I fix and which cases do we want to keep as "broken"? cc @RyanCavanaugh @DanielRosenwasser

There might be some more corner cases to explore here, but I think this is effectively what we want to "try" in the 5.0 beta to gauge reaction and figure out what to tweak (or if it's just going to have to roll back entirely). I'm leaning toward merge as-is. @DanielRosenwasser good to go?

While working on this, I initially made a different change but then discovered that TS currently allows this:
class A {
    foo() {}
}

class B extends A {
    bar() {}
}

declare const a: A
declare const b: B

a < b // ok?

So I went with a different route, trying to keep this behavior as I thought that it was desired. But given the discussion and an attempt to change the base behavior - should this case (and similar ones) be considered an error? How do we define what TS treats as a valid comparison?
From my PoV, allowing the comparison to be OK based on both types being comparable is not that useful and is a footgun. I'm specifically calling out this part of the check:
!leftAssignableToNumber && !rightAssignableToNumber && areTypesComparable(left, right);

@Andarist
FYI, Regarding this issue with implicit coercion I wrote a library https://github.com/redradist/SafeScript that completly exclude implicit coersion


Fix find_z2_symmetries when resulting object is empty

Summary
Fix #9418 .
This PR extends the input typing of the Z2Symmetries class in quantum_info.analysis from PauliList to list[Pauli] | PauliList.
With this we can fix the errors of find_z2_symmetries(operator) when the operator is None or when it does not have symmetries.
Test were added to cover these exact two cases.

We need this fix in the 0.23 release right, since this corrects a bug in RC1 in the Z2 Symmetries functionality that will be new in this release.

Iterable[Pauli] is better for the __init__ since PauliList is also Iterable[Pauli] and each property should be list[Pauli]. (If what I am saying is too difficult, shall I commit directly?)

I had a small discussion with @woodsp-ibm and we concluded that Iterable[Pauli] would complexify the eq. More precisely, list equality would need to be replaced with element wise equality.

Iterable[Pauli] can be used on the constructor. Its would be simple to create a list from that in the init and internally store as a list. This way eq remains the same. If we really want/need to do this.
@ikkoham this was my comment to @Anthony-Gandon in a DM yesterday

I think I would go with list[Pauli] - at least for the returns. If you want the input on constructor to be more flexible then you could use Iterable{Pauli] there and create a list from it. I imagine the Z2Symmetry logic will go away at some point since on large systems it makes so little difference overall. As such it maybe simpler just to use list all round. I doubt any one will use it outside of the find symmetries method we have. If someone wanted Iterable on the constructor it could always be added later I think. As such I would just go with list[Pauli] for now - keep it simpler

I'm not sure we really want it if we don't use the features of having PauliList in the attributes

I agree, as I stated in the DM I would keep it simple as list type.

How about this? I don't think there is any essential difference because it immediately consumes the Iterables and makes a list.
https://github.com/Anthony-Gandon/qiskit-terra/pull/1
You can reject this PR and make it as is. In that case, however, the test must be rewritten. There is a type violation.

Looks good to me, I merged your suggestion @ikkoham. It is true that I had forgotten a PauliList in the tests, this is fixed by this Iterable PR, thanks


DisplayUnlockCaptcha,  This website is no longer available  To better protect your account, Google no longer allows account access with this website.

hi bro.
i have 2 problems.

DisplayUnlockCaptcha no longer available
i printed $http_response_header. it says:

Array ( [0] => HTTP/1.1 403 Forbidden [1] => Content-Type: text/plain; charset=utf-8 [2] => Cache-Control: no-cache, no-store, max-age=0, must-revalidate [3] => Pragma: no-cache [4] => Expires: Mon, 01 Jan 1990 00:00:00 GMT [5] => Date: Sat, 27 Jan 2024 11:18:18 GMT [6] => Cross-Origin-Opener-Policy: same-origin [7] => X-Content-Type-Options: nosniff [8] => X-Frame-Options: SAMEORIGIN [9] => Content-Security-Policy: frame-ancestors 'self' [10] => X-XSS-Protection: 1; mode=block [11] => Server: GSE [12] => Alt-Svc: h3=":443"; ma=2592000,h3-29=":443"; ma=2592000 [13] => Accept-Ranges: none [14] => Vary: Sec-Fetch-Dest, Sec-Fetch-Mode, Sec-Fetch-Site,Accept-Encoding [15] => Connection: close )
what is the problem?

google changed their request ciphers and ssl algorithms, I will look & find solution and then will update codes. So wait...


x/tools/godoc: a function returning multiple types from its package is assigned to a wrong type

Example (from http://tip.golang.org/pkg/io/): func Pipe() (*PipeReader, *PipeWriter) is placed under type PipeReader however logically it is related to both PipeWriter and PipeReader.
I propose to add the following rule to the godoc heuristic for assigning function to types: if a function returns multiple types that belong to the same package as the function then do not assign this function to any type.
E.g. for the example above the function index should look like:
...
func Pipe() (*PipeReader, *PipeWriter)
type PipeReader
    ...
type PipeWriter
    ...

I just noticed this bug in my own code's doc as well.
Out of curiosity, why is a function that is not a method assigned to any type at all?  i.e., why is it indented under a type in the generated "Index" section of the doc?  I've often found this confusing when looking at, for example, the "time" docs. I guess I see the idea behind it (to group related functions together?) but it's counter-intuitive insofar as the function doesn't really "belong" to that type so when I go looking for time.Parse I ddidn't expect to have to look under the time.Time section.
And of course it has the drawback expressed by this bug, where choosing one of several types or moving it to top-level will be inconsistent.

@cgay This is a heuristic of course, but it seemed to work well in many cases: The thinking was that a function returning a T is a factory function for T, i.e., a mechanism to create a new T.

@griesemer Thanks for the reply. I would like to add that in my case the function was returning a struct called "Stats", which is just an aggregation of various stats and errors occurring during parsing of a very large file, so the function in no way belonged to Stats. (Quite the opposite.)  Anyway, it sounds like you are aware of the issue. Carry on. :-)


enhancement Abort merging rowsets when the tablet is no more active

Proposed changes
Merge rowsets is relatively long-term. Abort this stage if tablets are no more active (not ready is an active state for compaction).

run buildall

run buildall


Node2D position icon shown at offset position after reparenting

Godot version: v3.2.3.stable.official & v4.0.dev.calinou.160ad23e5
OS/device including version: Not graphics related
Issue description:
The godot editor shows a position icon  for each Node2D node.
After reparenting a Node2D to another Node2D which is not at position (0, 0), its icon will be shown at an offset position. The icon will be shown at the correct position after reselecting the node.

Steps to reproduce:

Create a new 2D scene
Add a new Node2D named A to root, and move it to (200, 200)
Add a new Node2D named B to root, and move it to (100, 100)
Reparent B to A

In 4.0 version, you have to move the mouse into the 2D viewport after reparenting.
In some rear cases the position icon will be updated correctly, just press undo.
Minimal reproduction project: N/A

CC @groud. I think there are other related issues in the tracker too.
Might need a refactor of how position gizmos are updated to fix all issues.

The behavior changed in 4.3. In 4.2.2 and before I could reproduce as shown.
In 4.3 and master I can only reproduce when undoing a reparent.
https://github.com/user-attachments/assets/07d81438-765e-463f-8154-394abda9a6b9


Moving from absolute paths to relative paths

We are still setting absolute path as the value for the path attribute. But in future, as the app grows it will become tedious and will be more nested making it unmaintainable. So, I think that moving towards relative paths will make it better.
Please assign me this under hacktoberfest thank you.

Hi @Anandsg my PR for issue #165 has not been merged yet. And I want to resolve the above issue in the code changes that I made in PR for #165 issue. Can you please merge it. If there's a problem with it do let me know.

your PR has been merged


chore: update tailwindcss and @tailwindcss/language-service

Updated TailwindCSS and the language service to latest versions.
discovered that size-x and text-balance didn't autocomplete.

Thanks trying to help out, but this doesn’t update anything. This only makes package.json and package-lock.json out of sync. Run npm install to resolve that first. Since we rely on internal APIs, the real update will likely require a lot more changes.

Sorry, forgot to commit the package-lock. Updated it!

For me upgrading these packages fixes the missing autocompletes.


Mod not working for geode 2.0

Tried to build it myself... Needs some modification :|

the mod isn't updated for 2.2 yet - see #3 for my updating progress

the pr has been merged 🎉


[NEW ICON]: PyTorch Lightning ⚡, Pytorch Ignite, Pytorch Fastai and Mojo

Skill Name
Lightning, Ignite, Fastai, Mojo
Why?
3 Deep learning frameworks that provides high level of abstraction (similar to keras for Tensorflow)
While Mojo is new programming language has faster computational speed than python, rust
Reference Image

Code of Conduct Acknowledgment

[X] I have read and agree to the Code of Conduct.

Hi @LuluW8071 I've been working on these icons, but I have a little issue for the naming, should they all include pytorch in it ? or can it be shorten to lightning, ignite ?

Hi @LuluW8071 I've been working on these icons, but I have a little issue for the naming, should they all include pytorch in it ? or can it be shorten to lightning, ignite ?

No you don't need to put pytorch
Naming:

Lightning
Fastai
Ignite
Mojo


CARLA x Autoware test on Twizy

This partially resolved issue is pretty much a tutorial for setting up Carla x Autoware (i.e. set a path in Rviz and Autoware Ackermann Control Messages are published to ROS, and thus, can control the Twizy).
If you want to run the Carla x Autoware in one machine, and SD-VehicleInterface in another, you'll need to run the following commands on both machines' terminals:
export ROS_LOCALHOST_ONLY=2 # This can be any number
export ROS_DOMAIN_ID=2      # This can be any number, but must match the

is there a repo for this?

You'll need  3 repos by hatem-darweesh. I managed to get it working on the Beast.

It would be good if this could all be packaged in some way into one repository so that it's reproducible.
I've just made a new repo for running HIL demos with the Streetdrone vehicles as I need to set up a CARLA HIL demo for a unit.
Would be neat if (when you get time) make a branch for carla-autoware-zenoh bridge HIL demo and could probably add those 3 repos ^ as submodules?

Also great work!!

Also Rviz + Carla should be done on a machine with at least 32 gb ram (i.e. not the SD Alienware Laptop). Maybe we could use one of the hives or something. Then we could publish to ROS topics via LAN.

Rviz (on ~8 fps) + Carla running on the SD Alienware: https://drive.google.com/file/d/1ywcLC-7fMzx-sukkRjqpWB_ePQO9rQ5b/view?usp=share_link


Improve CloudWatch Agent configuration file

🗣 Description
This pull request makes some improvements to the CloudWatch Agent configuration file.  It also modernizes the Ansible code by using FQCNs everywhere.
💭 Motivation and context
The CloudWatch Agent configuration file changes are the meat of this pull request, and these changes are being made for several reasons:

To support cisagov/instance-cw-alarms-tf-module#5, which in turn supports cisagov/cool-sharedservices-freeipa#54
To capture instance log files that were previously missing from the configuration file and hence not being forwarded to CloudWatch
To make sure any CloudWatch Agent metrics that may be useful for debugging instance failures or improving instance performance are being sent to CloudWatch

The Ansible moderinization is just something I noticed while I was here.
This pull request also just happens to resolve cisagov/cool-system-internal#20.
🧪 Testing
I have deployed these configuration file changes to all of our staging COOL FreeIPA and OpenVPN instances and verified that they function as expected.
✅ Pre-approval checklist

[x] This PR has an informative and human-readable title.
[x] Changes are limited to a single goal - eschew scope creep!
[x] All relevant type-of-change labels have been added.
[x] I have read the CONTRIBUTING document.
[x] These code changes follow cisagov code standards.
[x] All new and existing tests pass.

This makes sense. I had one thought though: Would it make sense to define "timezone": "UTC" for any logs that have a timestamp_format field? This would ensure that regardless of the configuration of the instance the log events are in UTC.

That's a good point.  I will take another pass and try to identify whether the timestamps are in local or UTC for each log.

This makes sense. I had one thought though: Would it make sense to define "timezone": "UTC" for any logs that have a timestamp_format field? This would ensure that regardless of the configuration of the instance the log events are in UTC.

I misread your comment last night.  I think timezone defines the timezone to assume when parsing the timestamp from the log entry.  I think it is intended to be used in cases where the timezone to use cannot be parsed from the timestamp.  I think it does need to be added in some cases, so that is what I am doing.

This makes sense. I had one thought though: Would it make sense to define "timezone": "UTC" for any logs that have a timestamp_format field? This would ensure that regardless of the configuration of the instance the log events are in UTC.

I misread your comment last night. I think timezone defines the timezone to assume when parsing the timestamp from the log entry. I think it is intended to be used in cases where the timezone to use cannot be parsed from the timestamp. I think it does need to be added in some cases, so that is what I am doing.

By the way, all of our Linux instances are set to use UTC as local time.

This makes sense. I had one thought though: Would it make sense to define "timezone": "UTC" for any logs that have a timestamp_format field? This would ensure that regardless of the configuration of the instance the log events are in UTC.

I misread your comment last night. I think timezone defines the timezone to assume when parsing the timestamp from the log entry. I think it is intended to be used in cases where the timezone to use cannot be parsed from the timestamp. I think it does need to be added in some cases, so that is what I am doing.

The configuration file documentation reads like it's for the events logged in AWS:

timezone – Optional. Specifies the time zone to use when putting timestamps on log events. The valid values are UTC and Local. The default value is Local.

The complete example seems to support that interpretation:
{
  "file_path": "/opt/aws/amazon-cloudwatch-agent/logs/amazon-cloudwatch-agent.log",
  "log_group_name": "amazon-cloudwatch-agent.log",
  "log_stream_name": "amazon-cloudwatch-agent.log",
  "timezone": "UTC"
},
{
  "file_path": "/opt/aws/amazon-cloudwatch-agent/logs/test.log",
  "log_group_name": "test.log",
  "log_stream_name": "test.log",
  "timezone": "Local"
}

I know we configure our instances to run under UTC, but since this is an independent Ansible role I thought it worth mentioning for configuration that isn't reliant on how the systems it's run on are configured.

The configuration file documentation reads like it's for the events logged in AWS:

I don't understand how this could possibly work. There are quite a few timestamp formats for which there is no timezone information whatsoever, so how could AWS determine what to do in those cases? I don't see how the example supports your hypothesis; it doesn't seem to support or refute it.

I believe I misled myself because of how poorly the AWS documentation explains the timezone field. Looking at the older documentation in the AWS CloudWatch Logs user guide it seems to indicate that the time_zone (assuming it's analogous to the timezone field here) is used to set the timezone when one cannot be inferred from the log event's timestamp.
So in the end this is a log format specific setting and not something to blanket apply. Mea culpa.

So in the end this is a log format specific setting and not something to blanket apply. Mea culpa.

No worries.  The documentation is not great, as you already pointed out.  I blame society.


Add CONTRIBUTING.md file to Repository

@roannav, I think your repository should contain CONTRIBUTING.md file (as seen in many projects participating in Hacktoberfest) in order to get recognized in Hacktoberfest.
You can either add it by yourself or assign this issue to me to add the File 🙂
Also please label this issue as "hacktoberfest".

Thank you for contributing the CONTRIBUTING.md !  :smile:


Switch to hard tabs for indentation

Which Cloudflare product(s) does this pertain to?
All
Subject Matter
Switch to hard tabs for indentation
Content Location
https://developers.cloudflare.com/*
Additional information
(I couldn't find a more appropriate issue form type, so apologies if this is not accurate)
Continuing from https://github.com/cloudflare/wrangler2/issues/1298 and https://github.com/cloudflare/templates/pull/76, it would be great to update this repo to also use hard tabs for indentation, both in code examples and any other TS/JS.
With the merge of https://github.com/cloudflare/cloudflare-docs/pull/5013 recently, there wouldn't be any visual change to how code samples are displayed, but when copied out to editors, would include tabs for better default accessibility.
I would very much encourage folks to read through https://github.com/cloudflare/wrangler2/issues/1298 for all of the reasons why switching to hard tabs for indentation on this repo would be a good thing.
Given the complexity and fast moving nature of this repo, I didn't want to simply submit a PR for this myself (since the diff would be massive), but am happy to do so if requested.

Closing this out b/c it's tracked in the other associated issue.


bug/API SDK multiple small issues

Describe the bug
Missing runtime arguments and checks create runtime errors.
In api.py https://github.com/Unstructured-IO/unstructured/blob/main/unstructured/partition/api.py
To Reproduce

Function partition_via_api:

parameter api_url is not used at all and not forwarded to s = UnstructuredClient(api_key_auth=api_key) as serverUrl.
with on-prem docker image the service always tries to call "https://api.unstructured.io/general/v0/general"

Function partition_multiple_via_api:

In this part:
if response.status_code == 200:
documents = []
response_list = response.json()
# NOTE(robinson) - this check is because if only one filename is passed, the return
# type from the API is a list of objects instead of a list of lists
if not isinstance(response_list[0], list):
response_list = [response_list]
response_list can be empty (send HTML with no text parts in it, response is empty string "") to unstructured on-prem docker.
response_list[0] creates Index Error -> missing check
Expected behavior
Better code...

Going to close this one out since we've switched partition_via_api over to using our python SDK. If you discover issues with the Python SDK, you can open issues there.


Fix badelf and add tests

There are a number of bugs in the current badelf code. Some are old and I didn't notice them until I tested the algorithm on a large number of structures. The others are new and probably from small changes made while adding the code to Simmate.

[x] Fix bugs when running bader_dev
[x] Update naming conventions to _w_empty_atoms
[x] Fix regrid bug for voronelf algo
[ ] Fix index error when running badelf on some electrides
[x] Add quick test workflow for badelf algorithm
[ ] Fix error when running zero-flux algorithm
[ ] Fix issue #569
[ ] Fix issue #570
[ ] Fix issue #556
[ ] Fix issue #560
[ ] Fix issue #564

can I close #586 and #585 as they are no longer needed?
 You can make them check boxes for things that aren't complete yet

[ ] closes #123
[x] make progress on #123
[x] fixes bug that ...

can I close #586 and #585 as they are no longer needed?
Yep. Those will both be included in this.

@jacksund  At this point I believe I have a working test for BadELF that doesn't take more than 5-6 minutes. I ended up making a new badelf workflow with much lower dft settings that's currently in the warren_lab app. Let me know if you think there's a better way to do this.
I'm also going to solve several of the current issues we have (which I've added to a list in the post above) and make sure the regular pytests run cleanly but after that I'm hopeful that we'll be close to ready for release. Let me know if I should work on anything else to get things ready to go.

Sounds good. I already made the v0.15.0 release, so there's no rush on this. Once you're ready and we merge this, I can make a v0.15.1 release that includes your updates.

@jacksund this PR is basically ready on my end. Let me know if you need me to do anything else.

everything looks good to me. Flag this PR as ready for review once you're done making edits & I'll merge it

@SWeav02 if you want issues to automatically close when something is merged, you need to follow these rules: https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue#linking-a-pull-request-to-an-issue-using-a-keyword
so closes #123 will work but closes issue #123 (like you had here) won't 🤷


escape json in structured llm

if there was text in brackets {, } then it would get mistakenly treated as a template variable
@logan-markewich is there a centralized utils method to escape json? seems like a genreally helpful function

It's been left up to the user until now to safely handle their inputs. Not using f-string prompt templates eventually will be the solution here


question about bgl dataset

Hi, thank you for making this amazing project.
I have some question when I use the BGL dataset to train and test. In the logdeep/dataset/sample.py you use 'hdfs/event2semantic_vec.json', I have no idea what is the function of this file. And when I use the BGL dataset, how I generate this file? Or I needn't this file? If it is not needed, how should I modify sample.py?
looking forward to your reply.

https://github.com/donglee-afar/logdeep/issues/3#issuecomment-750028771


Likert scale styling

We have received feedback that completing the likert scale is not intuitive. This relates to the colour bar (going from the left extreme to the selected point), it often has no relation to the question (as they are typically not numerical values). In addition, the markers along the scale are too small.

Remove the coloured line to the left of the selected point
Increase the size of the markers

 General refinement


[GLUTEN-2443][CH] Add new features for reading excelformat

What changes were proposed in this pull request?
realize the features that issue 2443 described
(Fixes: #2443)
How was this patch tested?
UT in GlutenClickHouseFileFormatSuite

Run Gluten Clickhouse CI


Fix the incorrect fix for 'Cannot redeclare class Gedmo\Mapping\Annotation\Reference' error

The previous fix removes the 'Cannot redeclare class Gedmo\Mapping\Annotation\Reference'
error but does not fix the real bug. Sorting the list of files does not guarantee that
the same error will not show up again when new annotations are added. Instead, using an
include_once instead of include will not only prevent this bug happening again, but also
remove the requirement that the file must be loaded with require_once. This is useful
because the autoload/files section of composer.json loads files with require instead
of require_once. Adding this file to the composer.json of a library that uses
DoctrineExtensions will allow a developer to use that library without having to worry
about manually loading this file (since composer will include it in it's autoload).

the assort is needed because the file order matters. centos lists files by default not in ascending order and the bug was related to that. include_once is ok, but revert the asort please.


[feature req] align compiler settings with wiiu game compiler settings to fix alignment so accessing game memory is more useful.

Feature Request
[ If any section does not apply, replace its contents with "N/A". ]
[ If you do not have the information needed for a section, replace its contents with "Unknown". ]
[ Lines between [ ] (square brackets) are to be removed before posting. ]
[ Please search for existing feature requests before you make your own request. ]
[ Duplicate requests will be marked as such and you will be referred to the original request. ]
What feature are you suggesting?
I'm suggesting changing the compiler rules to make alignment match with WiiU SDK games will make it a much more useful tool for modding.
Overview:

[ Include the basic, high-level concepts for this feature here. ]
N/A. Described above.

Smaller Details:

[ These may include specific methods of implementation etc. ]
Possibly an option to enable/disable these settings?

Nature of Request:
[ Remove all that do not apply to your request. ]

Change
Changing alignment will break compatibility with libraries built before the change.

Why would this feature be useful?
[ What does it add to devkitPro tooling that did not already exist? ]
A much richer ability to modify games and collect/modify data
[ What benefit does it provide for users of devkitPro tooling? ]
Removal of manual padding applied by developers to match games.

There isn't enough detail here for a meaningful response. Can you give some actual example of the problem you're trying to address here?


Calling bayesflow from R

Issue:
R stores all numbers as float64 where as tensorflow in python wants to have a float32. I tried changing the forward model function so that it would explicitly generate float32, but when it is called later by another function to sample training data, the training data which is stored in R, is in the default number type float64.
Tried:
history <- approximator$fit(   epochs = 3L,   dataset = training_dataset,   validation_data = validation_dataset )
Full Error:
tensorflow.python.framework.errors_impl.InvalidArgumentError: cannot compute Mul as input #1(zero-based) was expected to be a float tensor but is a double tensor [Op:Mul] name:
Run `reticulate::py_last_error()` for details.

9.
stop(structure(list(message = "tensorflow.python.framework.errors_impl.InvalidArgumentError: cannot compute Mul as input #1(zero-based) was expected to be a float tensor but is a double tensor [Op:Mul] name: \n\033[90mRun \033]8;;rstudio:run:reticulate::py_last_error()\a`reticulate::py_last_error()`\033]8;;\a for details.\033[39m",
call = py_call_impl(callable, call_args$unnamed, call_args$named)), class = c("tensorflow.python.framework.errors_impl.InvalidArgumentError",
"tensorflow.python.framework.errors_impl.OpError", "python.builtin.Exception",
"python.builtin.BaseException", "python.builtin.object", "error", ...

8.
raise_from_not_ok_status at ops.py#5983

7.
error_handler at traceback_utils.py#153

6.
compute_metrics at flow_matching.py#117

5.
compute_metrics at continuous_approximator.py#95

4.
build_from_data at approximator.py#23

3.
fit at approximator.py#82

2.
fit at continuous_approximator.py#109

1.
approximator$fit(epochs = 3L, dataset = training_dataset, validation_data = validation_dataset)```


Can't Subscribe to a topic

Hi, Everyone.
I've tried to implement this plugin. It has been able to get the registration ID.
But unfortunately it can't subscribe to a topic.
Event hardcoded this function right after register token is doesn't work:
FirebaseMessaging.getInstance().subscribeToTopic("news");
Maybe you guys have any idea about this.
Any help would be really appricated.
I just can't figure it out why.

To verify your topic is active, on the Firebase Console website:

select your APP
Go to Notifications Page (under GROW on left)
Select "NEW MESSAGE" button
Change Target to "Topic"
Type in "n" in "Enter a Topic" to see all registered topics that start with the letter "n" in your APP.

Ohh, okay.
You gave me an idea.
There is a one day delay before a topic can be targeted as target message and of couse one delay also before a new topic creation.
Thanks


Review parentage of adult refsum disease (MONDO:0009958) and added de…

…finition to 'non-Zellweger spectrum disorder' (MONDO:0100322)
Fix #3306
Note:

I “excluded subclass of” ‘autosomal recessive congenital ichthyosis’ for 'adult refsum disease' (MONDO:0009958)BUT I still see it as an inferred parent class.
need to add point 4 from #3306 when we confirmed the action item.

I “excluded subclass of” ‘autosomal recessive congenital ichthyosis’ for 'adult refsum disease' (MONDO:0009958)BUT I still see it as an inferred parent class.

I excluded 'autosomal ichthyosis syndrome with prominent neurologics signs' too. I don't think this is a ichthyosis syndrome, icthysosis is just a phenotype.

I changed 'fix' to 'address' above so I can merge this PR without closing the issue.


Add possibility to have different notifications in a single service

Hi there!
We like to have a possibility, to add a notification for only one service for one host.
Something like that:
`
object Host "myhost1.com" {
address="myhost1.com"
vars.os="Linux"
vars.memory[""] = {
}
vars.users[""] = {
notify_users = [ "myuser2" ]
notification.teams = true
}
vars.load[""] = {
load_wload1 = 1.6
load_wload5 = 1.6
load_wload15 = 1.6
load_cload1 = 2
load_cload5 = 2
load_cload15 = 2

}
vars.notify_users = [ "myuser1" ]
vars.notification.teams = true
`

Can you please elaborate your requirement? The title and context are confusing each other.

Do you need to add multiple type of notifications (Teams, Slack, etc) for a single service?
Do you need to enable the Teams notification for a specific service in a specific host only?

sure thing :)
sorry for the confusion though.
It's the second.
In my example i want the notifications for load, memory and host (up|down) sent to the teams user "myuser1" but the notifications for the users check should be send to "myuser2".
I hope it's bit more clear now. :)


Only allow token authentication with 2FA enabled

Fixes #1394
Previously, when a user had 2FA enabled they were still able to authenticate via HTTP(S) using their regular username and password. This PR makes token authentication mandatory for accounts with 2FA enabled.

@lafriks I refactored the routine according to your suggestions :+1:

@moritzheiber few more optimizations needed and than form me it would be LG-TM

@lafriks Should be all there now

LGTM

@moritzheiber I have tested and works as expected only thing that I found that does not match GitHub behaviour is that when 2FA is enabled it returns Invalid username or password even for valid password and from security perspective it is right thing to do. So I would recommend changing also Uers with two-factor authentication enabled cannot... to invalid credentials.

@lafriks The only grievance I'd have with that is that within Gitea that's completely undocumented. Nothing on the two-factor authentication page tells you that you need to create an application token in order to login while having 2FA enabled:

This is probably going to confuse a lot of people.
I know the response is probably the wrong time and place of conveying this, but as of right now it's the only one.

Maybe than add todo comment that later for security reasons response should be changed to invalid credentials than it would be LG-TM from me

@lafriks Done.

LGTM

LGTM

make CI work


Js2 combat

Megan drove, George and JP navigated. Added turn swap, hit randomizer, hit/miss effects.

them's fightin' words


Investigate the possibility of implementing Salesforce PKCE

Description
Salesforce changed the checked fields on their end when creating a Connected App.

Since Require Proof Key for Code Exchange (PKCE) Extension for Supported Authorization Flows is a default setting now, investigate if it is possible to implement it on our component. Currently, this field is not necessary and can be manually disabled during the app creation process.
These two fields are also default:

Require Secret for Web Server Flow
Require Secret for Refresh Token Flow

While the required field Enable Authorization Code and Credentials Flow which is not the default, should be enabled manually.

Component use built in to platform OAuth 2.0 feature (Faceless) for authorization, and it doesn't support PKCE.
To implement this, we will need to make our own realization, and it's really not as easy to do with this component.
I've tested other default options (without PKCE) and it works fine:


Introduce dt_passthrough node support for domcfg

Add dt_passthrough support to domcfg as analogue to dt_passthrough nodes in xl.cfg
This will allow us to pass nodes, that should be copied to the Domain device tree without need to put them to /passthrough node.

Acked-by: Mykola Kvach<EMAIL_ADDRESS>

Updated

This will allow us to pass nodes, that should be copied to the Domain device tree without need to put them to /passthrough node.

Do we really need this feature? It is considered  as ill-defined by the Xen community. This is why we can't upstream it to the Xen Tools.

Reviewed-by: Sergiy Kibrik<EMAIL_ADDRESS>for patch domain.h: Add dt_passthrough parameter to domain config


Improvement for k8s.io/docs/concepts/overview/working-with-objects/common-labels/

With the MySQL StatefulSet and Service you'll notice information about both MySQL and Wordpress, the broader application, are included.

Last instance of "WordPress" is spelled without a capital "P" as it is spelled on the rest of the page within the context of wordpress.com

/assign

@valetarton I had a look at it, I found only one line which has been included in the issue where WordPress was mentioned as Wordpress. I've sent in a pull request for this.

/good-first-issue
addressed in #25868
/close

/triage accepted
too


Please add Equals() implementation to AttributeValue

Tried to write a unit test today and needed to compare two AtttributeValues for equality, however it doesn't implement .Equals().  AttributeValue seems like the kind of class that should implement .Equals() given that it does represent a simple value.

Added to https://github.com/aws/aws-sdk-net/blob/master/FEATURE_REQUESTS.md


Edge model: remove architecture-specific items

The Edge Model includes entities from the GSMA Operator Platform, namely USER_CLIENT, OPERATOR_PLATFORM, and EDGE_CLOUD.
Suggest to remove USER_CLIENT because it is an architecture detail for operators deploying the GSMA operator platform but not part of the API model. It can still be included in the model but as an abstraction that covers both direct communication or communication via a user client.
Suggest to rename 'OPERATOR_PLATFORM' to NETWORK_OPERATOR_API_GATEWAY.
EDGE_CLOUD is GSMA Operator Platform specific, but it could be represented in the model as an optional way to access the MEC_PLATFORM

I agree with the issue. The Edge Model follows the entities listed in https://github.com/camaraproject/EdgeCloud/blob/main/documentation/SupportingDocuments/edge_terminology.md. I agree we should converge here towards abstract definitions independent of a specific architecture or heritage. I'd therefore suggest to first fix the edge terminology to list all entities in an architecture independent way.
In matter i would not agree with renaming OP to API Gateway as the OP covers substantially more roles than an API Gateway.


Initialize eigh by previous iteration.

Every iteration of DFT computes an eigendecomposition at (line 49). Since we compute the eigendecomposition at every iteration, we may be able to initialize the eigensolver at iteration i by the eigenvector solution from iteration i-1.
Notes:

The eigensolver is implemented in tesselate.
tesselate has two eigensolvers: the jacobi algorithm and the QR algorithm.
The matrix may change a lot iteration 0 to iteration 1 and less onwards.

Looks like we would add a all_AV_Cols argument to
def ipu_jacobi_eigh(x: Array, num_iters: int = 1) -> Tuple[Array, Array]:


fix(app, react-api-client): add run creation spinner and handle errors in slideouts during run creation

Overview
During run creation on the slideout we show the spinner icon, and we now throw errors from the useCreateProtocolMutation just as we do in the useCreateRunMutation in api-client. previously we were logging the error in the former so the error did not bubble up to the hooks that were using it in the app and the app was not able to handle that error path.
Changelog

CreateRun buttons on slideouts now watch for isCreatingRun and change the state for this boolean in parent component to display spinner

Review requests
confirm that spinner appear on both slideouts when proceed to run button is clicked and run creation is in process

Risk assessment

Low

@b-cooper Was the issue with the 500 error not being rendered some kind of unnecessary re-render that was setting the runCreationError back to null?
I'm also curious when and why we use this host higher up in the context tree? I imagine it is always in theory possible given the ApiClientProvider rendered in src/index.tsx?

This large error is formatted a bit awkwardly due to some long commands. Maybe you can add an overflow-wrap: anywhere; or something

@b-cooper perhapos we should use the default error message if the error is length is longer than a certain number of characters? displaying an error like this seems unnecessary

This large error is formatted a bit awkwardly due to some long commands. Maybe you can add an overflow-wrap: anywhere; or something

@b-cooper perhaps we should use the default error message if the error length is longer than a certain number of characters? displaying an error like this seems unnecessary

Yeah that sounds like a good idea,  if that is the case, we should make sure that that full message at least makes it to the JS console as a log.

I see spinner a now when I proceed to run.


NANNA'S OATS cause crash

i like your mod but some it make my game crash.

2023-10-08 10-30-45.zip

hey, sorry for late reply. Can you join our modding discord and upload your log in #troubleshooting there? Bot will check if everything is up to date. You can ping me there (@Waltz) so I can take a look: https://discord.gg/x3aynBJX

sup


Is the command to connect to cluster wrong?

According with this document, the command to connect to the cluster is:
az acs kubernetes get-credentials --resource-group=myResourceGroup --name=myK8sCluster
But, for some reason, this doesn't work for me.
I used the following command and it worked well:
az aks get-credentials --resource-group=myResourceGroup --name=myK8sCluster

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

ID: a172fdbe-fd3e-1ad5-d37e-fe393a241c5a
Version Independent ID: 516ee51f-d68c-685e-7f9d-8442bb977a6e
Content: Quickstart - Azure Kubernetes cluster for Linux
Content Source: articles/container-service/kubernetes/container-service-kubernetes-walkthrough.md
Service: container-service
GitHub Login: @neilpeterson
Microsoft Alias: nepeters

Returned error is:
"The Resource 'Microsoft.ContainerService/containerServices/myK8sCluster' under resource group 'myResourceGroup' was not found."
My kubernetes cluster was created using the Portal.

Thanks for the feedback! We are currently investigating and will update you shortly.

@jmpichardo to confirm, you have the correct Resource Group Name and Cluster name when running the command?
The command is correct.
You can also try running rm ~/.kube/config to clear any past credentials
Then try the command again

Yes, resource group and cluster names were OK.
I tried removing config file from .kube and then running "az acs kubernetes get-credentials...." command, and I got the same error again.
Then, I tried with "az aks get-credentials..." and worked well.
Used machine info:
Windows 10 Pro 10.0.17134 Build 17134
Azure-cli version 2.0.46
Kubernetes Client v1.10.3
Kubernetes Server v1.11.2
The Kubernetes cluster was created in Azure Portal.
By the way, I found this other documentation where they use "az aks get-credentials".

There's two different things here. This doc was for a Kubernetes created with Azure Container Service (ACS), which is largely deprecated and replaced by Azure Kubernetes Service (AKS). az acs kubernetes get-credentials should be correct for Kubernetes clusters created with ACS.
For clusters created with AKS, then yes, az aks get-credentials is correct.

Thanks @iainfoulds, that makes sense because I used AKS.
I will close the issue then.


populate the _data/allChanges.json file with standardized file names

Problem Summary
Currently the file paths for the pdf are generated from the Jupyter Notebook that reads the file names, this needs to be done after the rename script is run.

A script was added to package.json to run the rename.js file.  It can be called using npm run rename and it should be run before running the Jupyter notebook.
I think we can try to figure out a way to translate this into Python and incorporate it directly into the notebook itself.

that sounds good @matikin9! although i'm not sure what rename.js does 🤔

Looks like it tries to normalize the filenames into something consistent for the notebook to process

@matikin9  oh ok, that shouldnt be too hard!


[move-prover] move-lang name resolution overrides spec-lang shadowing

If there is a struct type T in a module and a type parameter T for a function as well, then the T type parameter should shadow the global struct T everywhere in the function. However, this does not happen today, since move-lang name resolution will resolve T to M::T in spec blocks, and the subsequent analysis in move-model cannot see any longer that T (without M::) was meant.

@tnowacki Not sure whether the issue is really in move-lang. If so, we would need to deactivate perhaps type resolution in spec blocks, similar as (I think) we do with names in expressions.


Update image-size library

svg  cause the packager to fail with invalid svg. This is caused by an old version of the image-size library. This issue is outlined here and was fixed in a more recent version. Just gotta do an npm i -S image-size :)

Hi there! This issue is being closed because it has been inactive for a while.
But don't worry, it will live on with ProductPains! Check out its new home: https://productpains.com/post/react-native/update-image-size-library
ProductPains helps the community prioritize the most important issues thanks to its voting feature.
It is easy to use - just login with GitHub. GitHub issues have voting too, nevertheless
Product Pains has been very useful in highlighting the top bugs and feature requests:
https://productpains.com/product/react-native?tab=top
Also, if this issue is a bug, please consider sending a pull request with a fix.
We're a small team and rely on the community for bug fixes of issues that don't affect fb apps.


Add reverse links to rancher metadata

It would be useful to list all the reverse links to a service instead of searching these relationships.

With the release of Rancher 2.0, development on v1.6 is only limited to critical bug fixes and security patches.


res.dataset() is not a function when no factory is provided

I think this is confusing. Since this detects whether a factory is provided or not, calling .dataset() without a factory should throw something more explicit than TypeError: res.dataset is not a function

see #5


Seamless-immutable should support AMD modules

I'm happy to make a PR expanding the UMD to accept an AMD request, but it's a hassle to shim seamless-immutable in my RequireJS project. Please let me know if there's a reason this hasn't been implemented, and if you'd like me to add it in :)

No reason except that I don't use AMD. Happy to accept a PR! :smile:


Support exporting icons/images from a single figma file

Adds two configuration options to the common.icons and common.images portion of the yaml file.

Thanks for the PR. I will check it on weekend.

Thanks, @alexey1312. I've published a new FigmaExport version.


Installed polybar but doesn’t run(i3wm)

I am using Manjaro Linux with i3-gaps as wm.
Installed polybar and did put exec_always --no-startup-id ~/.config/polybar/launch.sh to ~/.config/i3/config
But nothing happens. I even tried bash ~/.config/polybar/launch.sh but got the error as:
error: Failed to set foreground (reason: X resource "color7" not found (no fallback set))
Bars launched...
But again nothing happens. Rebooting doesn’t have any effect.
edit: tried installing from aur too but nothing happens.

There is an issue in your config file: reason: X resource "color7" not found (no fallback set) most likely you have a ${xrdb:color7} somewhere and have not set color7 in .Xresources

And what am I supposed to do here?

I know I am dumb. I got it finally. Defined a bunch of colours in .Xresources and is cool now.


SonarQube integrations

Is there a possibility to integrate this tool with SonarQube ?

Hi, I found someone's solution https://github.com/nacho4d/XCLogParser2Sonar/tree/main
but it would be great if XCLogParser would add new reporter sonarqube that would generate output in sonarqube format: https://docs.sonarsource.com/sonarqube/9.9/analyzing-source-code/importing-external-issues/generic-issue-import-format/


Write top-level success status into trace

Right now, no status is written into top-level sample, which misses some failures

sorry for waiting)


Big thanks

Not an issue at all but I just wanted to say thanks for taking the time to upload such useful libs.

Thanks @looxonline
Hope you find them useful


Fix ErrorMessage_CouldNotMapToAction usage

Summary
While working on another feature I came across a bug where we don't use ErrorMessage_CouldNotMapToAction correctly - we weren't supplying the necessary argument to it.

Release Notes Entry
Fix the error log message shown when an action could not be registered.

Do you think this needs to go into release notes? Also, please backport where applicable (presumably all in-support versions).

/backport to release/6.x

/backport to release/8.0


vector::ops::transformations::tests::test_simplify_preserve_topology macOS test regression

The test vector::ops::transformations::tests::test_simplify_preserve_topology is failing on macOS under GDAL 3.7.1.
Output
---- vector::ops::transformations::tests::test_simplify_preserve_topology stdout ----
thread 'vector::ops::transformations::tests::test_simplify_preserve_topology' panicked at 'assertion failed: `(left == right)`
  left: `POLYGON ((45 20,10 10,30 5,45 20),(30 20,20 15,20 25,30 20))`,
 right: `POLYGON ((20 35,10 10,30 5,45 20,20 35),(30 20,20 15,20 25,30 20))`', src/vector/ops/transformations.rs:258:9

Build Details
GDALVersionInfo {
    RELEASE_NAME: "3.7.1"
    RELEASE_DATE: "20230706"
    VERSION_NUM: "3070100"
    BUILD_INFO {
        CURL_ENABLED: "YES"
        PAM_ENABLED: "YES"
        GEOS_ENABLED: "YES"
        OGR_ENABLED: "YES"
        GEOS_VERSION: "3.12.0-CAPI-1.18.0"
        CURL_VERSION: "7.79.1"
        PROJ_RUNTIME_VERSION: "9.2.1"
        PROJ_BUILD_VERSION: "9.2.1"
        COMPILER: "clang 14.0.0 (clang-14<IP_ADDRESS>)"
    }
}

$ uname -a
Darwin  21.6.0 Darwin Kernel Version 21.6.0: Thu Jun  8 23:56:13 PDT 2023; root:xnu-8<IP_ADDRESS>1.6~1/RELEASE_ARM64_T6000 arm64

I can only conclude that your MacOS GDAL is cursed :smiling_face_with_tear:.

I'll try rolling back to an earlier version.

So if I'm not mistaken, this is the original polygon:

This is the simplified version:

And this is what your GDAL outputs:

CC @rouault does this look like an expected output for OGR_G_SimplifyPreserveTopology? GEOS 3.10 "works", but 3.12 appears (unconfirmed, I don't have it on hand) to produce the output at the bottom.

According to @dr-jts, this is a know but in GEOS. We should drop the hole or use another polygon (from JTS?). But keeping the outer ring is probably enough.

@lnicola Thanks for the fantastic research on this.
This is the only test I can find around this functionality in OSGeo/gdal. Give validating GEOS or GDAL is a non-goal here, should we just replicate this simpler test, and consider that enough to verify the binding works?

Yup, that sounds good.


added man pages for feathercoin, modified icons

I noted there is still a BITCOIN.CONF,  is it worth checking for other capitalisations?

For sure.
There is still a lot of 'bitcoin' stuff in the sources, starting with a lot of
png files with bitcoin logos, but I'm not sure if it is worth the effort to
remove/rename all of them, as we probably have to do taht with every version
Lizhi pulls over from Bitcoin.
I hope, that he will manually add the changes for the different versions of
code comming from bitcoin, but I think, that he simly will pull the upstream
repository.
It's a pity that he is not online often anymore and that he does his own
thing, without talking to others
Wolfgang
On Monday, November 09, 2015 02:33:41 wrapperband wrote:

I noted there is still a BITCOIN.CONF,  is it worth checking for other
capitalisations?

Reply to this email directly or view it on GitHub:
https://github.com/FeatherCoin/Feathercoin/pull/81#issuecomment-155024032


feat: using action for signing application

As title says

It cannot be used since it is a private action and this repo is public. Closing PR


Hello, I am just starting out with julia and i seem to be having trouble using IJulia. It may be my fault though. I installed julia a few weeks ago and started playing around with it for fun. Then i decided to take the julia course on corsera. The first week of that course has you instal Atom and then the pkg uber-juno to be able to work offline.

I am having a slight issue with IJulia. I am pretty sure it's my fault though. I downloaded and installed Julia a couple of weeks ago to see what it was like. I enjoyed it so much I decided to take a coursera class on it. The first week of the class had me set up Atom with the pkg uber-juno. I did that finished the lesson and thought all was well.
However, I opened up the command line Julia that I set up a few weeks ago and tried using to open a jupyter notebook and I got an error message.
julia> notebook() ERROR: UndefVarError: notebook not defined Stacktrace:  [1] top-level scope at none:0
I am not sure, but I thought that I should enter using IJulia before I try to open a notebook, but when I did that I got the error messages below.
julia> using IJulia [ Info: Precompiling IJulia [7073ff75-c697-5162-941a-fcdaad2a7d2a] ERROR: LoadError: ArgumentError: Path to conda environment is not valid: C:\Users\showe\.julia\conda\3 Stacktrace:  [1] prefix(::String) at C:\Users\showe\.julia\packages\Conda\CpuvI\src\Conda.jl:43  [2] top-level scope at none:0  [3] include at .\boot.jl:326 [inlined]  [4] include_relative(::Module, ::String) at .\loading.jl:1038  [5] include(::Module, ::String) at .\sysimg.jl:29  [6] top-level scope at none:2  [7] eval at .\boot.jl:328 [inlined]  [8] eval(::Expr) at .\client.jl:404  [9] top-level scope at .\none:3 in expression starting at C:\Users\showe\.julia\packages\Conda\CpuvI\src\Conda.jl:48 ERROR: LoadError: LoadError: Failed to precompile Conda [8f4d0f93-b110-5947-807f-2305c1781a2d] to C:\Users\showe\.julia\compiled\v1.1\Conda\WZE3U.ji. Stacktrace:  [1] error(::String) at .\error.jl:33  [2] compilecache(::Base.PkgId, ::String) at .\loading.jl:1197  [3] _require(::Base.PkgId) at .\loading.jl:960  [4] require(::Base.PkgId) at .\loading.jl:858  [5] require(::Module, ::Symbol) at .\loading.jl:853  [6] include at .\boot.jl:326 [inlined]  [7] include_relative(::Module, ::String) at .\loading.jl:1038  [8] include at .\sysimg.jl:29 [inlined]  [9] include(::String) at C:\Users\showe\.julia\packages\IJulia\4UizY\src\IJulia.jl:33  [10] top-level scope at none:0  [11] include at .\boot.jl:326 [inlined]  [12] include_relative(::Module, ::String) at .\loading.jl:1038  [13] include(::Module, ::String) at .\sysimg.jl:29  [14] top-level scope at none:2  [15] eval at .\boot.jl:328 [inlined]  [16] eval(::Expr) at .\client.jl:404  [17] top-level scope at .\none:3 in expression starting at C:\Users\showe\.julia\packages\IJulia\4UizY\src\jupyter.jl:8 in expression starting at C:\Users\showe\.julia\packages\IJulia\4UizY\src\IJulia.jl:89 ERROR: Failed to precompile IJulia [7073ff75-c697-5162-941a-fcdaad2a7d2a] to C:\Users\showe\.julia\compiled\v1.1\IJulia\nfu7T.ji. Stacktrace:  [1] error(::String) at .\error.jl:33  [2] compilecache(::Base.PkgId, ::String) at .\loading.jl:1197  [3] _require(::Base.PkgId) at .\loading.jl:960  [4] require(::Base.PkgId) at .\loading.jl:858  [5] require(::Module, ::Symbol) at .\loading.jl:853
I kinda feel like Atoms installation or uber-juno's installation may have messed something up. Can someone have a look at the error message and suggest a way to repair. For what it is worth, I have pasted the location of where Atom says the installation of Julia is.
C:\Users\showe\AppData\Local\Julia-1.1.0\bin\julia.exe
I don't actually need it to work since I can use Atom or juliabox, but I am a bit OCD and hate knowing something on my PC isn't working right. I googled the error message and found similar issues but I did not find an answer that applied to my situation. I would appreciate any help given, and I am sorry if the answer is somewhere else.

accidentally submitted this before I was ready. I submitted a different one and then noticed that this one got published, The other request for help is formatted better so I closed this one. Sorry for any inconvenience!


feat: add stopword checker + iterable generate function

This PR adds two sets of functionality.

It adds an iterable generate function that is useful for streaming usecases where the upstream caller may want to stop execution before the model is ready
Adds a stopwords argument to the pipeline args, this mimics the OpenAI complete endpoint and is useful halting execution when certain strings are produced, eg bob: could be used to yield control when the model outputs bob which would indicate it is the user turn to speak.

Let me know what you think :)
eg. usage
args = PIPELINE_ARGS(temperature=1e-1,top_p=1e-1,
                     stop_words=['bob:']
                     )
instr = """
bob: hello, how are you today?

alice: I'm fine, thanks.

bob: that's good. Write me a function in python to calculate pi please

alice:"""

for i in pipeline.igenerate(instr, args=args):
    print(i, end='', flush=True)

Nice :) The better method is to "recover" the state when you see Bob/Alice, as in https://github.com/BlinkDL/ChatRWKV/pull/87

hmm, can't really grasp the purpose of that PR after reviewing, the out state doesn't get used? eg here and I don't think it's solving the same problem. Also looks to use some global state load_all_stat which is better for me not to use as I would like to run this in a production (in a very loose sense) environment
The target use case here supports arbitrary custom stopwords, this can be used in langchain to stop the LLM when it needs to. eg
Task: Find a list of cheeses
Action: search google for cheese
Observation:

in such a chain you might put Observation: as a stop word in order to stop there and inject your own observation based on the search results.

I replace all \n\n in ChatGPT generations by \n

yes! are you thinking of including an <end of turn> token or something? Is it even possible to finetune in new tokens?

I use \n\n for now, because I replace all \n\n in ChatGPT generations by \n, so whenever you see \n\n it must be endoftext

Sometimes the text generation stop even it doesn’t finish the text because it creates \n\n for example if i ask the bot to write code, it starts with the sentence “here is the python code to write X algorithm” and then “\n\n”. Actually it will write the code, but the chatrwkv will stop the generation because it sees “\n\n”

Oh actually I did notice something, based on this section of the HF article; do you use str.replace('\n\n', '\n') or re.replace('\n+', '\n')? I don't think the former is actually removing all double new lines? Might explain the behaviour @cahya-wirawan is seeing since you would might not expect to see that if the model was not trained on it?
Big GZ on the HF release by the way!


Update to 23w42a

Thanks a lot for making the PR! I'm not much acclimated with PRs and stuff like this yet!


Warden loadout

Loadouts arent technically completed because they still need access levels, and also likely won't be spawning with their armor equipped iirc


Add bundling of service worker entry

cc @angus-c

Seems good but I'm still uncertain where to get the assets from w/r/t to ctx.
In WIP service worker plugin PR — I'm currently using ctx.chunkUrlMap for both precachePaths and cacheablePaths.
I'm assuming precachePaths (i.e. core bundles—vendor, main, runtime) and cacheablePaths (all JS bundles) can be derived at build time (with base asset path modified at runtime)

Yeah, JS asset names could made into a virtual module (and imported in the sw entry). Alternatively, this information could be registered under a well-known token by fusion-cli and depended on via the DI system. Or possibly something else depending on how https://github.com/fusionjs/rfcs/pull/7 shakes out. But whatever this ends up being shouldn't have any impact on anything this PR is concerned with.