Line length limit

Is it possible to set line length limit?

This repository has been closed and moved to laminas-api-tools/api-tools-configuration; a new issue has been opened at https://github.com/laminas-api-tools/api-tools-configuration/issues/3.


Vehicle ownership loss

Is it possible to change the distance for vehicle ownership? Sometimes when players go a bit away from their vehicle, it's not theirs anymore

Suggestion:  Use a Decorator: https://runtime.fivem.net/doc/reference.html#_n_DECORATOR

Are they shared/synced between players ?

Yeah, that's the whole idea.
They won't sync if you don't "own" the entity, so you can't set decorators
on a car someone else is driving, or someone else's player ped, but setting
a decorator on your vehicle migrates with it when someone else becomes the
"owner" (network-wise) of the entity.
I use this to sync fuel, for example.
Kind regards,
Fredrik Vold
Evil Overlord at webkonsept.com
On Tue, Apr 3, 2018 at 11:33 AM, Imagic<EMAIL_ADDRESS>wrote:

Are they shared/synced between players ?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ImagicTheCat/vRP/issues/371#issuecomment-378189189,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAs1oevvOchhk0jKJ2QkoMd9An8x2o7jks5tk0HrgaJpZM4RpcAX
.

This is amazing, will implement something ASAP.

Not yet tested for vehicle loss: https://github.com/ImagicTheCat/vRP/commit/1f557b307a7873a580a6ba97124e8eb4630ccaab


[BUG] Profiling Tool does not contain status info for failed event log

Describe the bug
The Profiling tool runs into an exception in the java tool side on an ABFS log. If user does not turn on --verbose, there is no information about this failure in the console/output folder/output log. This could be confusing to the user.
Steps/Code to reproduce bug
spark_rapids profiling -p databricks-azure --eventlogs <my-event-log>  --cluster <my-cluster>
Expected behavior
It will be nice the profiling_status.csv file captures this exception.

This bug is mainly a problem that Failed authentication is tolerated by the Scala code but it is not passed to the status.csv file.
The python code won't be able to tell an error occurred unless the java exits with non-0 value.
The fix to this issue is two folds:

EventLogPath processor should capture the exceptions and pass them to the tool.
Implement a fix to existing issue https://github.com/NVIDIA/spark-rapids-tools/issues/1126


description of the input data ml20m-all

the dataset format:
1  2,1112486027:29,1112484676:32,1112484819:47,1112484727:50,1112484580:112,1094785740:151,1094785734:223,1112485573:253,1112484940:260,1112484826:293,1112484703:296,1112484767:318,1112484798:337,1094785709:367,1112485980:541,1112484603:589,1112485557:593,1112484661:653,1094785691:919,1094785621:924,1094785598:1009,1112486013:1036,1112485480:1079,1094785665:......
1 is the user id
2,1112486027    29,1112484676    32,1112484819:47 ...... is a pair, which 2 is movie id, and<PHONE_NUMBER> stands for time?
it is different from movie lens dataset in http://grouplens.org/datasets/movielens/
no ratings.
Can i say the userid 1 like movie id 2; but  not like movie 3?
Could you interpreter the dataset more detailed？
Thank you!

Thanks for the feedback and what you mentioned is correct
1 is the userId and 2 the movieId and<PHONE_NUMBER> stand for time they watched
DSSTNE Engine supports analog inputs but we have not exposed it in the wrapper . if the Rating comes it could be viewed as an analog signals.
We will update the document on how to support analog Signals so that you can bring in Analog signals(Ratings) as Input to the Network

@buptqitian, did rgeorge@ answer your question? Let me know if you need more info.

@rgeorgej @benitezmatias
thanks a lot!
it really helps!


Disposable/little fixes

Several little changes along the whole app.

Map: Urthecast legend
Map: Search box set a marker on the map whenever you make a search
Global: typo fixes and other similar stuff

Tested locally and all seems to be working :+1:


increasing of max.request.size parameter does not affect on version 3.3.0

From @ahmetbardiz on September 21, 2017 14:9
Like the issue #587 changes on parameter max.request.size through the producer.properties file do not affect on runtime. When I run the producer on console it always get the default value of max.request.size = 1048576 despite of its being a bigger value.  I run the the producer command like below:
sudo ./bin/connect-standalone etc/schema-registry/connect-avro-standalone.properties etc/kafka-connect-jdbc/mssql.properties
Copied from original issue: spring-cloud/spring-cloud-stream#1082

Hi, @ahmetbardiz. It appears that you're using the kafka-connect-jdbc library/project. It is not a Spring Cloud Stream application. Please reach out to Kafka Connect project.


Advices does not work properly with variadic parameters

Uncaught TypeError: Core\Annotations\AnnotationFinder__AopProxied::findInPath(): Argument #2 must be of type string, array given in /app/cache/aop/proxies/src/Annotations/AnnotationFinder.php:28
public function findInPath(string $annotation, string ...$paths)

execution:
findInPath('any, 'path1', 'path2');

Hello @jakublabno,
I have fixed this with okapi-aop: 1.2.3 and also added a test case (https://github.com/okapi-web/php-aop/pull/27)


Remove MetadataWrapper class

I find it a bit paradox that MetadataWrapper is another container on top of Metadata, adding another level of nesting, when most of the methods/properties are actually shortcuts to attributes contained somewhere in Metadata. If we want a client-variant of Metadata, I'd maybe use inheritance instead of composition. 🤷
But maybe we don't even need it. Let's quickly go through the individual methods/properties of here:
On MetadataWrapper:

[x] from_json_file -> available as Metadata.from_file with json as default (#1279)
[ ] from_json_object -> could be a format-agnostic method on Metadata just like from_file.
[x] persist ->  available as Metadata.to_file with json as default (#1279)
[ ] expires -> could be a method on Signed, I'd call it 'is_expired' #1305
[ ] verify -> This is definitely needed for the repository tool as well. So it should be somewhere where both client and repository tool can access it. Maybe even on Metadata in addition to the vanilla  one-signature verify we currently have. See https://github.com/theupdateframework/tuf/pull/1060#issuecomment-660056376 for thoughts about that distinction. #1306

All the other methods here are either pure shortcuts or filters of attributes contained in Metadata objects (or contained objects).

[ ] Re shortcuts: https://github.com/theupdateframework/tuf/pull/1060/files#r452906629 has arguments against shortcuts.
[ ] Re filters: Maybe it will become easier to access individual properties once we have classes for all complex attributes as planned in https://github.com/theupdateframework/tuf/issues/1139? But similarly to the arguments against shortcuts, I'd be careful when implementing behavior that relates to contained classes on container classes.

Originally posted by @lukpueh in https://github.com/theupdateframework/tuf/pull/1291#discussion_r590391070

Step that I suggest should be followed for completing this issue:

[ ] Implement from_object method in Metadata class #1336
[ ] Implement 'expired' method in Signed class #1305
[ ] Implement verification by a threshold of keys #1306
[ ] Replace calls to MetadataWrapper.from_json_file with the available Metadata.from_file
[ ] Replace calls to MetadataWrapper.from_json_object with  Metadata.from_object
[ ] Replace calls to MetadataWrapper.persist with the available Metadata.to_file
[ ] Replace calls to MetadataWrapper.expires with the newly implemented 'expired'
[ ] Replace calls to  MetadataWrapper.verify with the newly implemented verification
[ ] Remove shortcuts and filters (see the issue description)
[ ] Remove the MetadataWrapper class itself

All the steps above will be done withe the merge of #1355 and #1396.

Done with #1355 and #1396.


update go v1.21.3

Go 1.21.3 released, including a security fix, do we need to update go version?
Reference:
Go v1.21.0 was release, we need to upgrade the CI/CD container image to adjust for pingcap/tidb repo.
https://github.com/PingCAP-QE/ci/pull/2341

It's upgraded, thanks @shawn0915


Theming example does not work

Issue type
I'm submitting a ...  (check one with "x")

[x] bug report
[ ] feature request

Issue description
I am trying to create a label with a custom background color. Like the one described here: https://akveo.github.io/react-native-ui-kitten/docs/design-system/use-theme-variables#declare-custom-component
Current behavior:
The background color is not blue
Expected behavior:
The background color should be blue
Steps to reproduce:
Copy the lines below
Related code:

Given I have a file in src/components/shared/Label.tsx
import * as React from "react";
import { View } from "react-native";
import { withStyles } from "react-native-ui-kitten";

const AwesomeView = props => {
  const { themedStyle, style, ...restProps } = props;

  return <View {...restProps} style={[themedStyle, style]} />;
};

export default withStyles(AwesomeView, theme => ({
  backgroundColor: "blue", // theme["color-primary-500"],
}));

When I use it like this:
                      <Label>
                        <Text>How do you identify?</Text>
                      </Label>

Then it should render render
And have a different background color
Other information:
OS, device, package version
insert information here

omg, that's a quite strange solution :)
sorry for late reply. I guess checking this issue can solve your trouble. Please leave feedback if so


owner(tier4_autoware_utils): add maintainers

Signed-off-by: Takayuki Murooka<EMAIL_ADDRESS>Description
added @takayuki5168 and @purewater0901, who contribute to tier4_autoware_utils a lot, to maintainers of tier4_autoware_utils

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@kenji-miyake @TakaHoribe Could you review this.


PEX 2.0 API help needed

It looks like the API of PermissionsEx 2  is completely different from PermissionsEx 1, and with no documentation available I'm pretty much stuck as of right now.
Can't find where to start with in eclipse by browsing the methods either, is an API even implemented yet?
What do I need to call in order to get access to the permissions manager?

Correct.  PEX 2.x is still in development.  Zml can tell you more about the API and programming aspects.

That'd be great, since we have to use PEX 2.0 for compatibility with Sponge, and there don't seem to be any Sync Plugins for Sponge (to update a member rank when registered on a website).
So I planned to make my own, but for that it would be useful to know how to get started with the current build haha :)

Use the Sponge permissions API. PEX2.0 currently does not have any exposed API.

yeah that's not an option, I guess I'll just have to run the commands then.


Enhance documentation and samples for monitoring

Configuring moitoring is not staright forward. Involves OpenShift configuration, RCO CR config among others. Review the documentation and identify gaps and enhance them.

@arturdzm please add overall steps/artifacts from your testing for now, so we can revisit this later. Thanks.


Can't play back downloaded YouTube videos after using -f bestvideo+bestaudio parameter?

I did a "youtube-dl -f bestvideo+bestaudio https://www.youtube.com/watch?v=AjSr8RvAzJE", but playing its local download back seems to be incomplete like after a quarter in it? I did not have this problem with other videos. Is anyone else seeing this problem too?
Thank you in advance. :)

In the next version it won't use the -shortest option.


Ready for production use?

I came across this great repo wanting to start writing X-Plane plugins in Rust instead of C++. I've read the readme, and the notes about drawing and display SDK parts needing 'some more work'. Is it realistic to start down the Rust path? Or will I find myself in a dead end because of missing SDK wrappers? I don't have much Rust knowledge, so I don't know how easy or hard it would be to contribute any missing pieces down the road.
What would be your advice at this point in the development?

If you want to make a plugin that works with datarefs and commands, using Rust and this library should work without too much trouble.
Graphics and GUI are the main areas where Rust and the plugin APIs don't easily work together, and the Rust bindings are incomplete/missing. For those, C++ and C are better options.
Because the documentation for this library is currently not very detailed, it's good to have some experience with X-Plane plugins in C or C++ before using Rust.

If you want to make a plugin that works with datarefs and commands, using Rust and this library should work without too much trouble.
Graphics and GUI are the main areas where Rust and the plugin APIs don't easily work together, and the Rust bindings are incomplete/missing. For those, C++ and C are better options.
Because the documentation for this library is currently not very detailed, it's good to have some experience with X-Plane plugins in C or C++ before using Rust.

Would you recommend writing plugin code in Rust and then the GUI portion of a plugin in C++ using something like this? https://github.com/ocornut/imgui

Yes, I haven't done that myself but it might be a good approach. If you can get that GUI library to work in the X-Plane environment. it should be fairly easy to use.


docs: optimize the syntactic expression of graph usage

I have already used langgraph in some production projects. When dealing with particularly complex logic, it is easy to misspell node names by simply using string to declare node names. We can optimize the way of node declaration and use enumeration to declare node names. Ref: https://github.com/langchain-ai/langgraph/pull/255
I have only changed some of the previous examples. If you agree with this way of writing, I think I can help modify the writing of node declarations in all documents.
from langgraph.graph import StateGraph, END
from enum import Enum

class NodeType(str, Enum):
    PLAN = "planner"
    EXECUTE = "agent"
    REPLAN = "replan"

workflow = StateGraph(PlanExecute)

# Add the plan node
workflow.add_node(NodeType.PLAN, plan_step)

# Add the execution step
workflow.add_node(NodeType.EXECUTE, execute_step)

# Add a replan node
workflow.add_node(NodeType.REPLAN, replan_step)

workflow.set_entry_point(NodeType.PLAN)

# From plan we go to agent
workflow.add_edge(NodeType.PLAN, NodeType.EXECUTE)

# From agent, we replan
workflow.add_edge(NodeType.EXECUTE, NodeType.REPLAN)

workflow.add_conditional_edges(
    NodeType.REPLAN,
    # Next, we pass in the function that will determine which node is called next.
    should_end,
    {
        # If `tools`, then we call the tool node.
        True: END,
        False: NodeType.EXECUTE,
    },
)

# Finally, we compile it!
# This compiles it into a LangChain Runnable,
# meaning you can use it as you would any other runnable
app = workflow.compile()

Recently, I was developing an Agent communication framework, studied its design pattern, and put forward a suggestion on syntax optimization. Could you give me some suggestions?
Link: https://github.com/langchain-ai/langchain/discussions/20420

I think rather than updating all the existing notebooks, we could add a "how-to" showing how to use enums for node names to make it easier to manage

this looked cute so I tried it, but it breaks serialization in LangServe streaming events (RemoteRunnable.astream_events) because the NodeType's value object is set as a key along the data object of on_chain_stream and WellKnownLCSerializer.dumps doesn't like that

I think rather than updating all the existing notebooks, we could add a "how-to" showing how to use enums for node names to make it easier to manage

Agree with you, so that we do not have to change all the notebooks, can also be singled out to emphasize this point, I will make further optimization.

this looked cute so I tried it, but it breaks serialization in LangServe streaming events (RemoteRunnable.astream_events) because the NodeType's value object is set as a key along the data object of on_chain_stream and WellKnownLCSerializer.dumps doesn't like that

Thanks for the feedback, it was something I hadn't noticed, could you provide more detailed code to help me understand the problem more clearly, it will help us to think more fully.

this looked cute so I tried it, but it breaks serialization in LangServe streaming events (RemoteRunnable.astream_events) because the NodeType's value object is set as a key along the data object of on_chain_stream and WellKnownLCSerializer.dumps doesn't like that

Thanks for the feedback, it was something I hadn't noticed, could you provide more detailed code to help me understand the problem more clearly, it will help us to think more fully.

it's basically as you had shared; compile the graph to a Runnable and provide to LangServe add_routes to expose as a route:

demo_server.py
from fastapi import FastAPI, HTTPException, Request
from langserve import add_routes

# your example compiled workflow
from example import app as workflow

app = FastAPI(
    title="LangChain Server",
    version="1.0",
    description="Spin up a simple api server using Langchain's Runnable interfaces",
)

add_routes(app, workflow, path="/example")

if __name__ == "__main__":
    import uvicorn

    uvicorn.run(app, host="localhost", port=8000)

then on client side I put together this little utility REPL:

demo_remote_runnable.py
import pprint
import asyncio
from enum import StrEnum
from uuid import uuid4

from langserve import RemoteRunnable
from langchain_core.messages import AIMessage

class Commands(StrEnum):
    NEW_THREAD = "new thread"
    OLD_THREAD = "prev thread"
    DEBUG_ENABLE = "debug"

class Modes(StrEnum):
    STREAM = "stream_async"
    GENERATE_EVENTS = "generate_events"

async def prompt():
    while True:
        try:
            _prompt = input("Human: ")
        except EOFError:
            break
        if _prompt == "q":
            print("Exiting...")
            break

        yield _prompt

async def run():
    runner = RemoteRunnable(
        url=f"http://localhost:8000/example",
    )

    # NOTE: default ON for purpose of example and also I haven't finished this next part
    debug: bool = True

    user_id = str(uuid4())
    thread_id = str(uuid4())
    prev_thread: str = ""

    async for user_input in prompt():
        if user_input == Commands.NEW_THREAD:
            prev_thread = thread_id
            thread_id = str(uuid4())
            continue
        elif user_input == Commands.OLD_THREAD:
            prev_thread, thread_id = (thread_id, prev_thread)
            continue
        elif user_input == Commands.DEBUG_ENABLE:
            debug = not debug
            continue

        print("...")

        try:
            async for event in runner.astream_events(
                input={"input": user_input},
                config={
                    "configurable": {
                        "user_id": user_id,
                        "thread_id": thread_id,
                    },
                    "run_name": "Test Run",
                    "metadata": {"remote_meta": "test"},
                },
                version="v1",
                include_names=["/example"],
            ):
                if debug:
                    pprint.pp(event)

                # TODO: format the event stream into something more fun and exciting to look at in a demo video :/

        except Exception as e:
            print(f"Whoopsy doodle, something broke :(\n{e}")

        print("\n---")

if __name__ == "__main__":
    asyncio.run(run())

Interesting. cc @eyurtsev - seems like something that could be fixable in langserve?


Port the app on mobile phone

Tested the app on an Android phone:
Observations:

The balls escape the screen, collide with the virus outside the screen frame and return as viruses.

Line drawing feature works well, as of now.

#21

Both tested on ios/android devices. Done


Add semver check to CI

Draft because I currently cannot get cargo semver to run locally so I'm going to abuse CI to run it.
Add a CI job to check if we have correctly set the crate version based on Semantic Versioning rules.
This job was copied from: https://github.com/libp2p/rust-libp2p/pull/2647
And uses this tool: https://github.com/rust-lang/rust-semverver
Thanks to @thomaseizinger for bringing this to my attention.

Lol, internal compiler error 🙃

Damn, I guess I'll have to make this work locally ... another day.

One year has passed since I started this and now the tool is deprecated :(

Thanks man!


Add support for GYB in benchmarks

As discussed on swift-dev, the repetitive nature of some performance tests lends itself very well to templataing. In order to reduce the chance of copy paste errors when adding new test cases, this PR adds support to generate tests using gyb (Generate Your Boilerplate) script that is employed in other parts of the project. This is done as a part of generating the test harness in benchmark/scripts/generate_harness/generate_harness.py.
When adding new performance tests to the benchmark suit, contributors sometimes forget to regenerate the test harness using generate_harness.py and instead manually edit the benchmark/CMakeLists.txt and benchmark/utils/main.swift files. To ensure this mistake is caught by CI bots, this also adds validation-test that guards against this issue.

Resolves SR-4533 and  SR-4534.
GYB is then used to refactor tests for sequence methods DropLast.swift and Suffix.swift:

changing [DropLast/Suffix][Any?]Sequence tests to use UnfoldSequence in line with equivalent tests in MapReduce.swift...
...while preserving the equivalents to previous test version under AnySeqCntRange and AnySeqCRangeIter
adding a test for AnyCollection wrapping CountableRange
adding performance tests for lazy variants of all tested sequences

This also introduces new performance tests for sequence method DropFirst.swift, DropWhile.swift, Prefix.swift, PrefixWhile.swift with all the same variants as described above.

Can I get a review, please? CC @gottesmm @lplarson @airspeedswift

Should I split the new performance test (DropFirst, DropWhile, Prefix, PrefixWhile) into separate PR?
(i.e. keep here only the harness modifications and DropLast and Suffix refactoring)

@swift-ci benchmark

@palimondo I think one pull request is fine.

I would also like to adjust the constants used in these tests. So far I have been using those introduced in Suffix and DropLast: sequenceCount = 4098 and [dropCount|suffixCount] = 1024. Give that these test are partly focused on exposing the performance deficits in AnySequence, I think it might be better to always split the sequence in half. That way DropFirst processes first half of the sequence internally and then has to wend the second half of the sequence through AnySequence. So I’d like to change [drop|preffix|suffix]Counts to 2048. Are you OK with getting new baselines for DropLast and Suffix while we change that?

I'd prefer to change the other tests in a different pull request.

In the new pull request, please ask therealbnut for a review of changing the count to 2048 for the other benchmarks.

Build comment file:
Build failed before running benchmark.

Build failed due to failure running SortSortedStrings?!? -- I didn’t touch that.
But I need help with the lit test: I have run --validation-test and my new test for freshly generated harness is failing on my machine (in addition to failure in TestCharacterSet.test_AnyHashableContainingCharacterSettypeTypeRef - should I report this somewhere?).
It looks like when we are trying to get to %swift_src_root variable, the %swift part gets substituted for swift invocation. How does one access that variable?

Looks like we need to modify lit.cfg and add a new substitution like this one? config.substitutions.append( ('%swift_obj_root', config.swift_obj_root) )

@swift-cni test

@swift-ci benchmark

@swift-ci test

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - 1c9cc3815a4e707d28984a4060a59b71b41b5dfa
Test requested by - @lplarson

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - 1c9cc3815a4e707d28984a4060a59b71b41b5dfa
Test requested by - @lplarson

@swift-ci benchmark

Build comment file:
Optimized (O)

Luke, please re-run the test and benchmark, I have fixed the python-lint issues.

@swift-ci test

@swift-ci benchmark

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - e9995fa83a37617dd29369f1400892d95e7e0d00
Test requested by - @gottesmm

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - e9995fa83a37617dd29369f1400892d95e7e0d00
Test requested by - @gottesmm

@swift-ci test

@swift-ci benchmark

@palimondo Got you covered man!

@palimondo Can you add a large comment to the top of:
benchmark/scripts/generate_harness/test_generate_harness.sh
explaining what it is doing (i.e. its purpose) and how it is invoked. I am imagining something that basically says that the script is meant to ensure that the checked in template files in the benchmark suite always match what would be regenerated if one re-ran the relevant scripts. I would also mention that the reason to use a lit test here is that we want to ensure that it runs on all smoke test runs so that the code checked into tree is always correct.

I would actually wait until the benchmark/test run finishes. Otherwise, it will reset since you added another commit.
Assuming that everything works out from the full test run/benchmark run and we get that nice comment in, I will quickly run a smoke test merge afterwards and we can get this work in! = ).

I’ll write that comment, but I also noticed I had accidentally named the directory under validation-test in plural - benchmarks instead of benchmark (that matches the one under swift). So I’ll push that rename in the same commit, OK?

I looked at the console log of the benchmark in progress, and I don’t see it running any of my new tests… am I missing something?

Build comment file:
Optimized (O)

@palimondo I think it is b/c it is doing a before/after comparison. If you look at the first group of runs, it is there. Can you add the changes that I requested and then I will do a smoke test and merge

@gottesmm I think we’re ready to smoke and merge.

@swift-ci smoke test and merge

@swift-ci smoke test and merge

Thank you @gottesmm,  @lplarson,  @dabrahams !

@eeckstein Re:

Yeah, CheckResults should not be inside the for 1...N loop.
I've patterned my additions after #7420 Benchmarks for dropLast and suffix by @therealbnut
They check the correctness of the operation, too. I found it quite useful when crafting variations on the original tests. Some of my sequence versions were buggy (of by iteration) on first attempt.

I'm not sure how hoisting them out of that loop will check the result - for different Ns. I think as long as the overhead between comparisons of the same method on different Sequences is the same, the test still works.

@palimondo

I'm not sure how hoisting them out of that loop will check the result - for different Ns.

You could just add the results for all iterations and compare against the total sum after the loop. I think the chance that this will miss a "compensated" wrong computation is quite low.

@eeckstein I agree as the error tends to zero as N→∞, and the cost of CheckResults is negligible.
However, it's a pattern that could easily cause mistakes. This would not work if the cost of CheckResults(...) is non-linear with the inner loop (n), so the cost grows quickly as N increases. For example checking a binary tree with CheckResults being O(n log(n)).
Anyway, I think in an ideal world we would do something that complete removes the verification when calculating times (out of scope of this PR):
func CheckResults(_ test: @autoclosure () -> Bool, _ message: String) {
   guard VERIFY else { return }
   assert(test(), message)
}

Run like this:
run-benchmarks -N=1 -verify=true
run-benchmarks -N=10 -verify=false

The documentation states that N=1 should take a minimum amount of time (to minimise the impact of error like CheckResults), it would be nice to assert this in the first run as well.

A closure is bad because it involves memory allocation which is much more expensive than the benchmark body itself for some benchmarks (That's what @dabrahams found).

I'm not so sure that's the only/correct way to interpret it… see discussion in #9298.

Note that the benchmark body is meant to take a minimum amount of time to run, from the README:

The benchmark driver will measure the time taken for N = 1 and automatically calculate
the necessary number of iterations N to run each benchmark in approximately one second,
so the test should ideally run in a few milliseconds for N = 1. If the test contains
any setup code before the loop, ensure the time spent on setup is insignificant compared to
the time spent inside the loop (for N = 1) -- otherwise the automatic calculation of N might be
significantly off and any performance gains/regressions will be masked by the fixed setup time.

I believe the intent is that the time of the setup/teardown/verification should be negligible by comparison to the work being measured. So if the time for a closure is significant then the body should be iterated enough times that it isn't.


Little QN tests

Codecov Report

Merging #212 into master will increase coverage by 0.12%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #212      +/-   ##
==========================================
+ Coverage   87.02%   87.14%   +0.12%
==========================================
  Files          44       44
  Lines        4153     4153
==========================================
+ Hits         3614     3619       +5
+ Misses        539      534       -5

Impacted Files
Coverage Δ

src/qnindex.jl
75.92% <0%> (+0.92%)
:arrow_up:

src/iterativesolvers.jl
93.05% <0%> (+2.77%)
:arrow_up:

src/qnitensor.jl
92% <0%> (+8%)
:arrow_up:

Continue to review full report at Codecov.

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


bootstrap_sdk: override seed path after building stage1

use built stage1 to build stage2
Logs show that currently the seed tarball is used for building stage1 and then stage2 - so the output of stage1 is discarded. This PR fixes that and uses stage1 results for stage2.
How to use
./bootstrap_sdk
Testing done
./bootstrap_sdk
Before:
26 Jul 2021 10:33:52 -00: NOTICE  : Emptying directory: /mnt/host/source/src/build/catalyst/tmp/flatcar-sdk/stage1-amd64-2021.07.26+dev-main-nightly-3124
26 Jul 2021 10:33:52 -00: NOTICE  : Starting auto from /mnt/host/source/src/build/catalyst/builds/seed/flatcar-sdk-amd64-2920.0.0.tar.bz2
26 Jul 2021 10:33:52 -00: NOTICE  : to /mnt/host/source/src/build/catalyst/tmp/flatcar-sdk/stage1-amd64-2021.07.26+dev-main-nightly-3124 (this may take some time) ..
...
26 Jul 2021 11:13:54 -00: NOTICE  : Emptying directory: /mnt/host/source/src/build/catalyst/tmp/flatcar-sdk/stage2-amd64-2021.07.26+dev-main-nightly-3124
26 Jul 2021 11:13:54 -00: NOTICE  : Starting auto from /mnt/host/source/src/build/catalyst/builds/seed/flatcar-sdk-amd64-2920.0.0.tar.bz2
26 Jul 2021 11:13:54 -00: NOTICE  : to /mnt/host/source/src/build/catalyst/tmp/flatcar-sdk/stage2-amd64-2021.07.26+dev-main-nightly-3124 (this may take some time) ..

After:
27 Jul 2021 09:28:41 -00: NOTICE  : Emptying directory: /mnt/host/source/src/build/catalyst/tmp/flatcar-sdk/stage1-amd64-2021.07.27+dev-flatcar-master-3136
27 Jul 2021 09:28:41 -00: NOTICE  : Starting auto from /mnt/host/source/src/build/catalyst/builds/seed/flatcar-sdk-amd64-2920.0.0.tar.bz2
27 Jul 2021 09:28:41 -00: NOTICE  : to /mnt/host/source/src/build/catalyst/tmp/flatcar-sdk/stage1-amd64-2021.07.27+dev-flatcar-master-3136 (this may take some time) ..
...
27 Jul 2021 10:10:40 -00: NOTICE  : Emptying directory: /mnt/host/source/src/build/catalyst/tmp/flatcar-sdk/stage2-amd64-2021.07.27+dev-flatcar-master-3136
27 Jul 2021 10:10:40 -00: NOTICE  : Starting auto from /mnt/host/source/src/build/catalyst/builds/flatcar-sdk/stage1-amd64-latest.tar.bz2
27 Jul 2021 10:10:40 -00: NOTICE  : to /mnt/host/source/src/build/catalyst/tmp/flatcar-sdk/stage2-amd64-2021.07.27+dev-flatcar-master-3136 (this may take some time) ..

Need to run this through CI, but this explains why some of the nightly SDKs had broken packages - stage2 was built with old stage3 with old pax-utils that segfaults on seeing glibc 2.33.
This might also uncover some other issues, eg.: old sdk has readline-7, update_seed tries to update to readline-8 but fails because cross/gdb depends on readline-7 and seed doesn't have the ebuilds for cross packages. Do you think it would make sense use stage3 as the seed tarball instead of the SDK? Do we store them somewhere?

CI passed, ready to merge as soon as someone reviews.


Fix false positive for NoUnnecessaryCollectionCallRule

[x] Added or updated tests
[ ] Documented user facing changes
[x] Updated CHANGELOG.md

Resolves https://github.com/nunomaduro/larastan/issues/607
Changes
Fix false positive for NoUnnecessaryCollectionCallRule when statically calling hydrate on a Model class.

Thank you @Daanra

@canvural are you ok with this?

@canvural are you ok with this?

Yeah, looks good :+1: Thanks @Daanra


Mark workflow run green after deletion

When actually deleting a branch, I think it makes sense to mark the run green. I see that this was changed in https://github.com/jessfraz/branch-cleanup-action/pull/8 and my guess is that this case was just missed.

PTAL @jessfraz


Refactor file descriptor handling in openFileDescriptors()

Motivation:
The motivation behind this change is to enhance the readability and maintainability of the codebase by refactoring the file descriptor handling in the openFileDescriptors method. By extracting the logic for closing file descriptors into a separate method, closeFileDescriptor,
Modification:
In this commit, I have refactored the openFileDescriptors method by extracting the code responsible for closing file descriptors into a separate method, closeFileDescriptor.
Result:
This PR introduces improved code organization and readability by separating the concerns of opening and closing file descriptors in the openFileDescriptors method. It does not introduce any functional changes.

Did you sign the ICLA? https://netty.io/wiki/developer-guide.html

Yes, I have completed the ICLA process.

Looks like you messed up the rebase and entangled your changes with the merge of #13942. Please fix.

Looks like you messed up the rebase and entangled your changes with the merge of #13942. Please fix.

Fixed.
Thanks.

Thanks!


AssertionError: target columns not found

After features are generated, there's AssertionError: target columns not found

same issue here, i think the problem comes from here
https://github.com/Featuretools/predict-next-purchase/blob/main/utils.py#L79

X = fm_encoded.reset_index().merge(label_times,on="user_id")
X.drop(["user_id", "time"], axis=1, inplace=True)
X = X.fillna(0)
y = X.pop("bought_product_y").astype('bool')
these changes work for me


Kaprekar Number

Title
kaprekar Number calculation
🛠 Issue #1541

Issue no #1541
👨💻 Changes proposed
Added Kaprekar Number program files in c , c++ , python , java

✔ Check List (Check all the applicable boxes)

[ ✔] My code follows the code style of this project.
[ ✔] This PR does not contain plagiarized content.
[ ✔] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers

📷 Screenshots
C  Program Output

CPP Program Output

Java Program Output

Python Program Output

The code seems to be copied from https://www.geeksforgeeks.org/kaprekar-number/
Any explanation for this?
@Kumar-laxmi

No sir this is not copied code I have took the refrence of that article and done some of the changes according to my knowledge and perspective , I have used again two three resources like  Wikipedia , learningcode etc , this is my early stage of learning and making  a practical use of it . There's no any bad intention to gain a marks in wrong way and benifit me . I had a pure intention to learn new through this source and upskill my self at the beginning of my career , but sir please TRUST me I have note copied code as it is , to mentor my self I have just used the refrence , I request you to please check code one more time .

You should refer the algorithm and not the code, sorry @pra9patil but I have to close the PR since the code is exactly same


How to install TobiiEyeTracker module?

When I run python MicEye.py after following the installation tutorial, I get ModuleNotFoundError: No module named 'TobiiEyeTracker'
I tried pip install TobiiEyeTracker, but it didn't work, where can I download this module?

Hi, I also encountered this problem, did you solve it and how did you solve it?

@Fortogether Yes, I've solved the problem and it's working fine now. You can download it from this link corresponding DLL library: https://github.com/JamesQFreeman/PyEyetracker

Thanks, I will link it down below the README.


[API] Paginate Patterns, Apps, Filters

Current Behavior
A fully loaded Meshery Server has thousands of pattern components.
Implementation
Enhance /api/oam/workload to offer pagination
Acceptance Tests

Logs
https://user-images.githubusercontent.com/7570704/132809417-d0b19b1e-06d7-4906-8c12-4338e2513289.mov
Acceptance Criteria

All three resources should have pagination available on their queries.

Contributor Guides and Resources

🛠 Meshery Build & Release Strategy
📚 Instructions for contributing to documentation

Meshery documentation site and source

🎨 Wireframes and designs for Meshery UI in Figma
🙋🏾🙋🏼 Questions: Layer5 Discussion Forum and Layer5 Community Slack

@Abhishek-kumar09 Is this something that you're able to do?

@dhruv0000 @bariqhibat @utkarsh-pro  do we have a standard approach here, yet?

Need investigation but what I can infer from the logs here, it is not due to the overflowing of patterns, but certainly due to continuous re-rendering and fetching, maybe there is data leak issue here(maybe due to wrong call to useEffect), it is also possible that the schema is unparsable thus not rendered.

The api is throwing endless workloads :(

@Abhishek-kumar09 I'd be addressing this issue in a coming PR. Thanks for looking into it.

Can we close this? @leecalcote

I think yes.

The patterns, filters, and applications support paginations.
The schema coming from the server for adapter configuration doesn't contains pagination, actually that can't  be paginated either. But it strongly requires enhancement in terms of fetching only those resources which are visible on UI.

Currently around 8 Mib of data is fetched and processed on frontend for the RJSF forms, which is actually not required since frontend is only consuming 5-10% of coming data. 90% of processing and server resources is going waste.

@Abhishek-kumar09 ?trim=true with the lazy loading in the UI not helping here?

forgot to add trim=true
trim removes oam_ref from all the schema right, it helps still the size is vast, but at least UI is not breaking. Thanks a lot.

@leecalcote, do we still need this? We already have trimming in place. Do you foresee any usage for paginating the OAM components?

@leecalcote, do we still need this? We already have trimming in place. Do you foresee any usage for paginating the OAM components?

@utkarsh-pro no, I think you're right. Let's shut this one down.

@utkarsh-pro no, I think you're right. Let's shut this one down.


more bagdes, bump deps

cc @rust-embedded/tools

In the absence of a Cargo.lock file, the package bumps won't make any difference because the resolver will pick the highest one anyway. Technically we should specify the lowest we can get away with, but usually it's whatever was the latest at the point the dep was added.
So I would either add the Cargo.lock file (and I'll want a second opinion from @rust-embedded/tools on that), or I wouldn't bump the dependencies and I wouldn't show the dependency check badge.
I lean towards adding the lock file myself.

Bumping the Cargo.toml will still ensure everyone has at least that version, but yea since this is typically installed as a binary with cargo install you'd expect everyone to get the latest version anyway.
Unless someone runs cargo install --locked svd2rust I don't think Cargo will pay attention to the lockfile if we did add one, so I'm not sure there's much point adding it. It might make some difference to people working on it from a Git checkout, but I think that's just developers. I'd expect mostly it's best to use the latest available version of dependencies anyway, and the CI will be checking that too.
Still it shouldn't hurt to bump the deps here to whatever the latest version we want to use is. I wouldn't bother with the dependency check badge myself but I don't mind having it. I think @burrbull wants to do a release ASAP, so I think let's merge this now and we can revisit a lockfile after the release.

Just adding on to the last comment, Cargo does recommend binaries like svd2rust publish a lockfile so someone could use --locked if they wanted to, so perhaps we should be including it here too, even though it won't be used by default.

Yeah I think we should. Further I think we spiff encourage the use of --locked in the docs/README and also ship binaries to use with cargo binstall which were built with locked dependencies.

I'm happily surprised about the inclusion of a Cargo.lock, we've had discussions about this before and ended up not doing it.
cc https://github.com/rust-embedded/svd2rust/issues/399


Survival analysis

I'm submitting a

[ ] bug report.
[ ] improvement.
[X] feature request.

What is survival analysis?

The objective in survival analysis — also referred to as reliability analysis in engineering — is to establish a connection between covariates and the time of an event. The name survival analysis originates from clinical research, where predicting the time to death, i.e., survival, is often the main objective. Survival analysis is a type of regression problem (one wants to predict a continuous value), but with a twist. It differs from traditional regression by the fact that parts of the training data can only be partially observed – they are censored.
re: scikit-survival

In view of great popularity of time series data and time series databases, survival analysis provides a straightforward way to reason about time-to-event predictions. We learnt about Smartcore from PostgresML— machine learning for Postgres, which recently made the switch to use Smartcore for performance reasons. We're trying to estimate how hard it would be to perform time-to-event prediction in our Postgres; it would seem that doing so would require a Smartcore implementation for survival analysis (originally implemented as sckit_learn.linear_model)
Exactly how hard would it be, how much would have to be done from scratch, and where would we even begin to approach this if we don't have the expertise in the subject? Honestly, we would love to pay somebody to do this but where would we look to hire for this?
Best regards

Hi, thanks for using smartcore. We really like to hear from organizations leveraging the library, the engineers at PostgresML have been very kind to contribute useful code recently.
I have never heard of scikit-survival, I will take a look; though I have worked a little with SCM (structural causal models, you can read some examples in my blogposts), at first sight it looks like they try to tackle similar problems but I don't know of any formal relation between the two approaches. New day, new things to learn.
As usual scikit provides a nice API and we can always try to backtrack, having support of a scientist that knows the the math in depth is sometimes hard to find, fortunately we can usually collect a nice number of tests from existing libraries and experiments or papers to feel confident enough of being numberically accurate. Becoming an expert of a particular framework takes indeed years, the same to make a good library. In my experience it depends mostly on how many layers of nested functions are needed. In very simple words, for example, implementing a pipeline with multiple pre-processing and processing steps takes longer (usually months even in presence of existing code, many "classes" and methods that call each other) than a simple regression that usually means implementing some methods (in smartcore most of the classifiers fit in one file, multiple files usually means compounded effort).


feat(config): add Qubino Shades Remote Controller

This should close #5846 . There are no parameters and no association groups.

PR description here
Qubino_Shades_Remote.pdf
memo: How this device appears to work 1) Includes in network (gets ID) 2) Includes Qubino blinds during operation.  Probably picks up zwave transmissions. 3) operate blinds with remote. Unclear how many blinds it can control.

Hi @apella12,
The Qubino Shades Remote controller has no parameters and it is included (added) to the gateway - hub as a controller type device.
The associations are done automatically by the remote depending on the device that is included in the remote.
It can control up to 9 rollers or in case of blinds it can control 4 of them as one channel is used for up/down movement the second for the tilt, total 8 channels one remain free.
Link to pairing video: https://youtu.be/P4aeIGOV7aA
Pairing the Qubino Shades Remote Controller.pdf


Populate all references to the same resource

I've got a call that will return multiple entities. I've configured the SpringDataRestAdapter to recursively resolve multiple links.
This actually works fine, but my problem is the filter that ensures that each URL is called only once which leads to unpopulated fields if one of the entities has a link to an resource that was already fetched for another entity. This leaves all references after the first one empty.
While I do appreciate that each URL is only called once (I've already got 29 requests with only one call of SpringDataRestAdapter), it would be extremely useful if other entities that reference the same links would get populated aswell.

Hi @ahatius,
can you please provide me an example of what you are doing with the filter and without the filter it works properly right? You can also use the caching of the $http service described here such that you do not call the back end multiple times with the same request: https://docs.angularjs.org/api/ng/service/$http#caching
Thanks and regards,
Guy

Hi Guy
Thanks for the fast reply :)
Actually my wording was not quite correct when I said filter. I'm not using an angular filter, I was actually referencing the comment on line 261 in angular-spring-data-rest-provider.js. According to that every resource will be fetched only once.
However, I tried to copy my JSON responses and noticed that they are way too big to copy in here, so I created a small, simple project which provides an example of my problem. Maybe I'm doing something wrong and that's the reason for my issue.
You can find the example project here. The project just needs to be checked out and started, everything else should be automatically initialized so you just need to navigate to http://localhost:8080/index. I have also added a screenshot of how it looks, what it's supposed to look like and what the final JSON object looks like.
I'd be really glad if you could have a look at it if you have a moment.

Hi @ahatius,
sorry for my late reply but I hadn't the time to look into it. This nice little project runs and I analyzed your domain model and one issue is that you have cyclic dependencies as shown in this diagram:

Therefore the problem is that you call the SpringDataRestAdapter recursively and then it will never stop because of this cyclic dependencies.
I will check to add a flag to disable this check that the link was already fetched as this is an issue but then you must not have cyclic dependencies.
I would propose that you remove these dependencies and then update the project and I can check again for the flag such that you get the proper responses.
Would that be ok for you?
Thanks and regards,
Guy

Hi Guy
Thank you very much for that analysis.
I think that should be feasible. I have removed the circular dependencies for the Product, ProductRate and ProductCategory entities in the test project. I have also removed them for the Job and JobDetail entities, although those didn't seem to be problematic.
Cheers,
Ahatius

Hey Ahatius,
nice I am glad to hear that. I will try to integrate the new flag as soon as I have more time and then I will do a new release.
Thanks and regards,
Guy

I have the same problem here. I have a directive which loads a tree recursivly from my server (using the 'recursive=true' option for 'SpringDataRestAdapter.process' and given links to follow) . If I add the directive twice on a side, one tree is fully populated, the other one has only its root nodes.
I can see that the request for the "root nodes" is made twice, but the following requests which dive deeper are done only once and the resolved response is in one case only filled with the root nodes, the other one has all childs that were fetched.
It is very complicated to build up a standalone testcase, but maybe you can have a look at the screenshot. It is a console.log from the two responses. The first is with empty 'childDiagnosisGroups', the second includes them as object. You can see that the link is for both the same, the initial requests are identical. If I make another call to it, it results everytime the full tree. I think this two requests get grouped at some point and the links are followed only once (good for performance), but the resulting data isn't distributed to every call.

Hope that helps to explain the problem and make further investigation possible

Hi
the flag fetchMultiple flag has been added. Read more here in the documentation: https://github.com/guylabs/angular-spring-data-rest#how-to-automatically-fetch-links (nead the end of the section)
I will now release 0.4.5 and then you can update and adapt your projects.
Regards,
Guy

Thank you very much, will try it out ;)


[List/Button] vertical alignment issue of icon buttons inside horizontal list

There is an incorrect alignment of ui button next to ui icon button inside a horizontal list as can bee seen here: http://jsfiddle.net/fchz8nkx/

ping

We finally fixed this in Fomantic-UI by https://github.com/fomantic/Fomantic-UI/pull/1568


new function to find the point where two geodesics are closest

function description is as follows:
''' returns LatLon point that is closest point between two geodesics,
ab and cd, where each geodesic is defined by two points (in the tuple for
that line). If the lines intersect, the returned point is the intersection.
'res' gives the resolution in meters of the geodesics and so determines the
accuracy of the point found compared to an analytical solution '''

feedback welcome! :)

@JuliaWindmiller1987 @d70-t


Inline media in WhatsApp

Is your feature request related to a problem? Please describe.
This would improve the user experience when integrating Telegram & WhatsApp
Describe the solution you'd like
I would like to be able to have inline/playable media instead of attached files.
Describe alternatives you've considered
Downloading files and playing with external players
Additional context
Tested with Android native clientes for sending (Telegram) & receiving (WhatsApp). This seems to work the other way around (receiving in Telegram looks inline).
Receiving audio notes and videos in WhatsApp from Telegram, I received them as file attachments:
file_1.mp4.mp4 for fideo
file_3.oga.ogg.oga for audio notes
animation.gif.mp4.mp4 for GIF animation
Tested with official container: "Running version 1.22.2-dev a0bca42a"

Also, if there is text in the same message with the media, the text is omitted and only the file lands in WhatsApp.

Also, if there is text in the same message with the media, the text is omitted and only the file lands in WhatsApp.

Opened #1925 for this.

#1928 fixes this, so this can be closed.


Onboarding of few task types to throttling

Signed-off-by: Dhwanil Patel<EMAIL_ADDRESS>Description
Added few of the Cx driven tasks to cluster manager task throttling framework. These all tasks onboarded on this PR, extends the TransportClusterManagerNodeAction, so they will by default have the retry mechanism on the throttling from cluster manager node.
Issues Resolved
#479
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff
[ ] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #4542 (58824e2) into feature/master-task-throttling (1212afd) will decrease coverage by 0.12%.
The diff coverage is 82.85%.

@@                         Coverage Diff                          @@
##             feature/master-task-throttling    #4542      +/-   ##
====================================================================
- Coverage                             70.71%   70.59%   -0.13%
+ Complexity                            57644    57562      -82
====================================================================
  Files                                  4665     4666       +1
  Lines                                276614   276702      +88
  Branches                              40297    40300       +3
====================================================================
- Hits                                 195621   195350     -271
- Misses                                64737    65090     +353
- Partials                              16256    16262       +6

Impacted Files
Coverage Δ

...search/cluster/service/ClusterManagerTaskKeys.java
0.00% <0.00%> (ø)

...main/java/org/opensearch/script/ScriptService.java
71.65% <0.00%> (-0.45%)
:arrow_down:

...oredscripts/TransportDeleteStoredScriptAction.java
55.55% <50.00%> (+5.55%)
:arrow_up:

.../storedscripts/TransportPutStoredScriptAction.java
55.55% <50.00%> (+5.55%)
:arrow_up:

...ing/delete/TransportDeleteDanglingIndexAction.java
9.58% <50.00%> (+1.13%)
:arrow_up:

...dmin/indices/rollover/TransportRolloverAction.java
51.21% <50.00%> (-0.04%)
:arrow_down:

...ster/metadata/MetadataCreateDataStreamService.java
61.53% <50.00%> (-0.31%)
:arrow_down:

...arch/persistent/PersistentTasksClusterService.java
45.55% <50.00%> (-2.71%)
:arrow_down:

...cluster/metadata/MetadataIndexTemplateService.java
82.84% <83.33%> (+1.00%)
:arrow_up:

...h/cluster/service/ClusterManagerTaskThrottler.java
87.67% <87.09%> (-3.24%)
:arrow_down:

... and 498 more

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


Now copy failed, docker context

I'm running a docker deployment using now in the root of my project and I get an error:
COPY failed: stat /var/lib/docker/tmp/docker-builder896428753/build/package.json: no such file or directory. (also applie to ADD)
I can run docker build . and it builds no problem, but when I run now in the same directory the following error occurs. The file is clearly there since docker build works fine.
Lots of searching has resulted in solutions running it from the wrong folder but there isn't another folder to run it from. Is there a way to set the docker context?

here is my dockerfile:

# Set a working directory
WORKDIR /usr/src/app

COPY ./build/package.json .
COPY ./build/yarn.lock .

# Install Node.js dependencies
RUN yarn install --production --no-progress

# Copy application files
COPY ./build .

# Run the container under "node" user by common
USER node

# Set NODE_ENV env variable to "production" for faster expressjs
ENV NODE_ENV production

CMD [ "node", "server.js" ]

EDIT: I am also not using a .dockerignore file

I was able to resolve this by removing the build folder from .gitignore which according to this restricts files being uploaded even on a docker build


Update README.md

Description
Updated readme
Fixes #502
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Test Configuration:

Firmware version:
Hardware:
Toolchain:
SDK:

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

Screenshots
Before
After
Include video if needed

Hello @Shikhar9425!
Thank you for submitting this pull request Your pull request will be reviewed by maintainers shortly. Meanwhile make sure you have referenced the respected issue in this pr Your pr will not be merged if it does not have reference to respected issue
please read our Contributing Guidelines https://github.com/UBA-GCOEN/StichHub/blob/main/CONTRIBUTING.md And Code of Conduct https://github.com/UBA-GCOEN/StichHub/blob/main/CODE_OF_CONDUCT.md

@UBA-GCOEN I have updated the readme

@UBA-GCOEN please merge this pull request


Revert index.html to custom version that sets up ports

This reverts index.html to the custom version that sets up the ports, fixing https://github.com/evancz/elm-todomvc/issues/33.

Thanks!


Uncaught ReferenceError: defineNuxtPlugin is not defined

I think you have to explicit import defineNuxtPlugin like you import Logrocket in the plugin
import { defineNuxtPlugin } from '#app'

Experiencing the same thing.

Same thing, seems to be an issue.

For those who curious the error is:

After executing
npm i nuxt-logrocket

and adding config
// nuxt.config.js

export default defineNuxtConfig({
  modules: [
    'nuxt-logrocket',
    //...
  ],
  logRocket: {
    id: 'xxx',
    dev: true,
    config: {}
  },
  //...
});

Nuxt info:
- Operating System: `Darwin`
- Node Version:     `v20.6.1`
- Nuxt Version:     `3.8.0`
- CLI Version:      `3.9.1`
- Nitro Version:    `2.7.2`
- Package Manager<EMAIL_ADDRESS>- Builder:          `-`
- User Config:      `ssr`, `devtools`, `typescript`, `app`, `build`, `buildModules`, `modules`, `compilerOptions`, `vite`, `hooks`, `runtimeConfig`, `inlineSSRStyles`
- Runtime Modules<EMAIL_ADDRESS>`<EMAIL_ADDRESS>`<EMAIL_ADDRESS><EMAIL_ADDRESS>`<EMAIL_ADDRESS>- Build Modules:    `@nuxtjs/svg`

Just adding a comment here just so it's known that it's still an issue. I have set enablePinia to false and still get the error

I'm getting the error as well.

@armenr
Just a note to watch for filenames that contain log and are excluded by a .gitignore rule. Caught me when I pushed to production and the plugin wasn't there at all :)
Thanks for the help. It's working well.
Only thing I added was LogRocket identity tracking.
      // track user in LogRocket
      LogRocket.identify(user.id.toString(), {
        name: user.name,
        email: user.email,
        // Add your own custom user variables here, ie:
        // subscriptionType: 'pro'
      })
      ```

getting the same issue

hi @richard-edwards, can you show me how you implemented it in you nuxt.config file please?

@aguirremac assuming you mean the user identification part .. the plugin part is the same as @armenr shows above.
At the top of my app.vue I added the following. I'm not 100% sure that it's the correct way to do it but it seems to be getting me some of the sessions attached to real users. Still in alpha stage so only have a few people testing it. Important part here is that this has to be run on client only thus the if (process.client) { check.
<script setup lang="ts">
  import LogRocket from 'logrocket'
  const { user } = useUserStore()
  if (process.client) {
    // track user in LogRocket
    if (user) {
      LogRocket.identify(user.id.toString(), {
        name: user.name,
        email: user.email
        // Add your own custom user variables here, ie:
        // subscriptionType: 'pro'
      })
    }
  }
</script>

import LogRocket from 'logrocket'
const { user } = useUserStore()
if (process.client) {
// track user in LogRocket
if (user) {
LogRocket.identify(user.id.toString(), {
name: user.name,
email: user.email
// Add your own custom user variables here, ie:
// subscriptionType: 'pro'
})
}
}

thank you.

The same happens here when adding the LogRocket module in nuxt.config.ts
export default defineNuxtConfig({
  devtools: { enabled: true },
  modules: [
    'nuxt-feather-icons',
    'nuxt-gtag',
    'nuxt-icon',
    'nuxt-simple-robots',
    'nuxt-logrocket',
    '@nuxtjs/sitemap'
  ],
  logRocket: {
    id: process.env.LOGROCKET_APP_ID,
    dev: process.env.NODE_ENV !== 'production',
    config: {}
  },
  ......
})

I am running into this issue as well. It seems like the only solution currently is to rewrite defineNuxtPlugin() ? I'm running Nuxt3. Is this plugin generally still supported / active?

For anyone else that's stuck, this seems to work, for now. I did this quick & dirty, but if you're running Nuxt3, with the latest pinia and the latest logrocket npm packages, you should be ok, I think.
plugins/00.logRocket.client.ts
import LogRocket from 'logrocket'
import type { Plugin as NuxtPlugin } from '#app'

const plugin: NuxtPlugin = defineNuxtPlugin(() => {
  const { $pinia } = useNuxtApp()
  const opts = useRuntimeConfig()?.public?.logRocket

  if (!opts?.id || (!opts?.dev && !(process.env.NODE_ENV === 'production')))
    return

  LogRocket.init(opts?.id, opts?.config)

  if ($pinia && opts?.enablePinia) {
    $pinia.use(({ store }) => {
      store.$subscribe(m => LogRocket.log('mutation', m))
    })
  }

  return { provide: { LogRocket } }
})

export default plugin

And in your nuxt.config.ts, you should have this:
  runtimeConfig: {
    public: {
      // all options can be found here: https://www.npmjs.com/package/logrocket?activeTab=code
      // dist/types.d.ts --> interface IOptions
      logRocket: {
        id: 'som_id/your_logrocket_appName',
        dev: false, // or true if you want
        enablePinia: true,
        config: {},
      },
    },
  },

Hope this helps. It's pretty much the full re-implementation of this repo's plugin...except up-to-date and properly typed.

not working still same error


Does not work with latest Astro & Solid

Reproduction: https://github.com/marvin-j97/solid-1.7-icons
Icon is invisible.
Changing client:load to client:only fixes it.

Probably related to https://github.com/x64Bits/solid-icons/issues/41 ?
Maybe https://github.com/x64Bits/solid-icons/issues/45 as well

the svg path seems to be in quotes in the html causing the svg to not render.

Hello, I am very aware of the problem that solid-icons are having when used in astro.
I hadn't given an answer because I wanted to reply with the solution, but it's taking me longer to figure out why this is happening, I think it's a problem in the build because in solid you can't just do <svg>{iconSrc.c}< /svg>, I'm concentrating on fixing this, I'll bring news over the weekend, thanks!

Hi everyone, i comment here about it https://github.com/x64Bits/solid-icons/issues/41#issuecomment-1620939575

Working for me @x64Bits :+1:


Remove FetchFailed conditions from OCI

Signed-off-by: Soule BA<EMAIL_ADDRESS>If implemented the Helmrepository_oci reconciler will have a new
negative polarity condition addFailedCondition.
This implementation is also adding a deletion of all conditions when switching type. Previously not doing so was resulting in negative polarity condition set in previous generation to be taken into account when inferring the ready condition at the end of the reconciliation, resulting in a wrong reconciliation state.
tests
We have tested the following in a live cluster

switching url from a type: "oci" from oci://ghcr.io/souleb/charts to https://ghcr.io/souleb/charts => resulted in AddFailed = True, Ready= False
switching back url from a type: "oci" from https://ghcr.io/souleb/charts to oci://ghcr.io/souleb/charts => resulted in Ready = True
unsetting type from type: "oci"  => resulted in Reconciling= True, Ready= False, FetchFailed= True
setting back type: "oci"  => resulted in Ready= True

Here some of the test results
Wrong URL Scheme
Spec:
  Interval:  1m0s
  Timeout:   60s
  Type:      oci
  URL:       https://ghcr.io/stefanprodan/charts
Status:
  Conditions:
    Last Transition Time:  2022-05-27T20:21:28Z
    Message:               the URL scheme is not supported: https://ghcr.io/stefanprodan/charts
    Observed Generation:   8
    Reason:                URLInvalid
    Status:                True
    Type:                  Stalled
    Last Transition Time:  2022-05-27T20:21:28Z
    Message:               the URL scheme is not supported: https://ghcr.io/stefanprodan/charts
    Observed Generation:   8
    Reason:                URLInvalid
    Status:                False
    Type:                  Ready
    Last Transition Time:  2022-05-27T20:21:28Z
    Message:               the URL scheme is not supported: https://ghcr.io/stefanprodan/charts
    Observed Generation:   8
    Reason:                URLInvalid
    Status:                True
    Type:                  AddFailed
  Observed Generation:     8
Events:
  Type     Reason      Age                  From               Message
  ----     ------      ----                 ----               -------
  Normal   Succeeded   23m                  source-controller  Helm repository "podinfo" has been successfully reconciled
  Warning  Failed      2m5s (x14 over 23m)  source-controller  failed to fetch Helm repository index: failed to cache index to temporary file: object required
  Warning  URLInvalid  3s (x2 over 23m)     source-controller  the URL scheme is not supported: https://ghcr.io/stefanprodan/charts

missing secret
Spec:
  Interval:  1m0s
  Secret Ref:
    Name:   oci-creds
  Timeout:  60s
  Type:     oci
  URL:      oci://ghcr.io/souleb/charts
Status:
  Conditions:
    Last Transition Time:  2022-05-27T20:23:09Z
    Message:               failed to get secret 'default/oci-creds': Secret "oci-creds" not found
    Observed Generation:   1
    Reason:                AuthenticationFailed
    Status:                False
    Type:                  Ready
    Last Transition Time:  2022-05-27T20:23:09Z
    Message:               failed to get secret 'default/oci-creds': Secret "oci-creds" not found
    Observed Generation:   1
    Reason:                AuthenticationFailed
    Status:                True
    Type:                  AddFailed
  Observed Generation:     1
Events:
  Type     Reason                Age                From               Message
  ----     ------                ----               ----               -------
  Normal   Succeeded             20m (x2 over 22m)  source-controller  Helm repository "podinfo-private" has been successfully reconciled
  Warning  AuthenticationFailed  1s (x12 over 23m)  source-controller  failed to get secret 'default/oci-creds': Secret "oci-creds" not found

I was testing this, it leads us into more complicated conditions related issue. For example, we usually have extra conditions that have higher priority than Reconciling and Stalled conditions. But now, Reconciling and Stalled conditions have the highest priority. Due to this, an actual failure value (reason + message) set in Ready condition directly during reconciliation is overwritten by the condition summarizer with the value of Reconciling. When a referred secret is not found, instead of the Ready condition showing the error that the secret wasn't found, it shows the Reconciling message about reconciling new generation.
status:
  conditions:
  - lastTransitionTime: "2022-05-26T11:45:04Z"
    message: reconciling new object generation (2)
    observedGeneration: 2
    reason: NewGeneration
    status: "True"
    type: Reconciling
  - lastTransitionTime: "2022-05-26T11:45:04Z"
    message: reconciling new object generation (2)
    observedGeneration: 2
    reason: NewGeneration
    status: "False"
    type: Ready
  observedGeneration: 1

The events and logs show the right reason
Event:
19m         Warning   AuthenticationFailed   helmrepository/gitlab-repo   failed to get secret 'default/gitlab-registry2': Secret "gitlab-registry2" not found

Log:
{"level":"error","ts":"2022-05-26T17:42:43.267+0530","logger":"controller.helmrepository","msg":"Reconciler error","reconciler group":"source.toolkit.fluxcd.io","reconciler kind":"HelmRepository","name":"gitlab-repo","namespace":"default","error":"failed to get secret 'default/gitlab-registry2': Secret \"gitlab-registry2\" not found"}

Usually we set things like login failure in a separate condition which has higher priority than Reconciling and Stalled conditions, so they still exist on the status, but don't overwrite the Ready condition, which is overwritten by another condition that contains the actual reason for the failure, like a FetchFailed condition we used to have.
Setting both Ready and Reconciling condition with the same value doesn't seem to be right. Reconciling condition shows the reason something why a reconciliation is taking place and not why something failed. Failures should have their own conditions.
This makes me conclude that it'd be better to keep a negative condition like FetchFailed, but rename it to something appropriate as this reconciler no longer fetches anything. Maybe RegistryAuthFailed or LoginFailed, or even RegistryAddFailed with an assumption that not all the registries would need authentication/login, based on how we add helm repositories using helm repo add command.
This will solve the problem of Reconciling overwriting Ready condition. I don't think we need a positive polarity condition yet, just setting the Ready condition directly should be fine. The above issue arise only because of the negative polarity conditions.

I was testing this, it leads us into more complicated conditions related issue. For example, we usually have extra conditions that have higher priority than Reconciling and Stalled conditions. But now, Reconciling and Stalled conditions have the highest priority. Due to this, an actual failure value (reason + message) set in Ready condition directly during reconciliation is overwritten by the condition summarizer with the value of Reconciling. When a referred secret is not found, instead of the Ready condition showing the error that the secret wasn't found, it shows the Reconciling message about reconciling new generation.
status:
  conditions:
  - lastTransitionTime: "2022-05-26T11:45:04Z"
    message: reconciling new object generation (2)
    observedGeneration: 2
    reason: NewGeneration
    status: "True"
    type: Reconciling
  - lastTransitionTime: "2022-05-26T11:45:04Z"
    message: reconciling new object generation (2)
    observedGeneration: 2
    reason: NewGeneration
    status: "False"
    type: Ready
  observedGeneration: 1

The events and logs show the right reason
Event:
19m         Warning   AuthenticationFailed   helmrepository/gitlab-repo   failed to get secret 'default/gitlab-registry2': Secret "gitlab-registry2" not found

Log:
{"level":"error","ts":"2022-05-26T17:42:43.267+0530","logger":"controller.helmrepository","msg":"Reconciler error","reconciler group":"source.toolkit.fluxcd.io","reconciler kind":"HelmRepository","name":"gitlab-repo","namespace":"default","error":"failed to get secret 'default/gitlab-registry2': Secret \"gitlab-registry2\" not found"}

Usually we set things like login failure in a separate condition which has higher priority than Reconciling and Stalled conditions, so they still exist on the status, but don't overwrite the Ready condition, which is overwritten by another condition that contains the actual reason for the failure, like a FetchFailed condition we used to have.
Setting both Ready and Reconciling condition with the same value doesn't seem to be right. Reconciling condition shows the reason something why a reconciliation is taking place and not why something failed. Failures should have their own conditions.
This makes me conclude that it'd be better to keep a negative condition like FetchFailed, but rename it to something appropriate as this reconciler no longer fetches anything. Maybe RegistryAuthFailed or LoginFailed, or even RegistryAddFailed with an assumption that not all the registries would need authentication/login, based on how we add helm repositories using helm repo add command.
This will solve the problem of Reconciling overwriting Ready condition. I don't think we need a positive polarity condition yet, just setting the Ready condition directly should be fine. The above issue arise only because of the negative polarity conditions.

Superseded by #748


Non privileged users - account expiry and payment

Whenever a non-admin attempts to run a build in an account that has had its free trial period expire, the message is
"You're not allowed to change billing details for this organization"
I think it should be different - saying that theres no monies

Totally, good catch!

This should be all fixed now! Thanks for the issue.


[WFLY-12675] Upgrade WildFly Core 11.0.0.Beta1

JIRA: https://issues.jboss.org/browse/WFLY-12675

Release Notes - WildFly Core - Version 11.0.0.Beta1
        Component Upgrade

[WFCORE-4678] -         Use a more recent version of guava

[WFCORE-4682] -         Upgrade Jandex to 2.1.1.Final

[WFCORE-4689] -         Upgrade jboss-remoting to 5.0.16.Final

[WFCORE-4691] -         Upgrade XNIO to 3.7.5.Final

[WFCORE-4692] -         Upgrade WildFly Common to 1.5.2.Final

[WFCORE-4705] -         Upgrade WildFly Elytron 1.11.0.CR1

[WFCORE-4706] -         Upgrade Elytron Web to 1.7.0.CR1

[WFCORE-4708] -         Upgrade XNIO to 3.7.6.Final

[WFCORE-4709] -         Upgrade Undertow to 2.0.27.Final

[WFCORE-4714] -         Upgrade galleon plugins to 4.0.4.Final

        Enhancement

[WFCORE-4505] -         use default charset for process file via jboss-cli  --properties

[WFCORE-4594] -         Expose CoreProcessStateService functionality used by subsystems via a capability

[WFCORE-4595] -         ControlledProcessState.State should expose whether a state means a running server

[WFCORE-4680] -         Redact passwords in the sun.java.command logging

[WFCORE-4710] -         Wrong BufferPoolMXBean name used in lookup

        Feature Request

[WFCORE-4681] -         Add Phase entries for Microprofile OpenAPI Extension

        Bug

[WFCORE-4338] -         Race condition on PID file allows for termination of arbitrary processes by local users

[WFCORE-4656] -         Write Empty ALLOWED_ORIGINS list cause ConfigurationPersistenceException on reload

[WFCORE-4683] -         User with Monitor role can stop the servers in domain mode

[WFCORE-4684] -         The legacy core-feature-pack doesn't install the .well-known/acme-challenge dir

[WFCORE-4687] -         StandaloneScriptTestCase fails on Eclipse OpenJ9

[WFCORE-4699] -         preferIPv6Addresses and preferIPv4Stack System Properties are Mishandled in the Config

        Task

[WFCORE-4652] -         Ignore tests that expect default configuration during Elytron testing

[WFCORE-4679] -         Update the mock-server-netty dep; specify the logback dep

[WFCORE-4686] -         Ensure the log manager is set for tests for Eclipse OpenJ9

[WFCORE-4690] -         Fix test suite tests to work with Eclipse OpenJ9

[WFCORE-4696] -         Bump the Elytron subsystem version

[WFCORE-4702] -         Add deployment phases for MicroProfile Fault Tolerance

Dependency Tree Analyzer Output:
Major Version Changes:

org.wildfly.core:wildfly-cli:jar:client:10.0.0.Final:compile -> 11.0.0.Beta1
org.wildfly.core:wildfly-core-feature-pack:zip:10.0.0.Final:compile -> 11.0.0.Beta1
org.wildfly.core:wildfly-elytron-integration:jar:10.0.0.Final:compile -> 11.0.0.Beta1
org.wildfly.core:wildfly-event-logger:jar:10.0.0.Final:compile -> 11.0.0.Beta1
org.wildfly.core:wildfly-version:jar:10.0.0.Final:compile -> 11.0.0.Beta1

CC @wildfly/prod


[TF 2.0] keras model.train_on_batch allocates extremely large CPU memory

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 10 x64 1809
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary): pip
TensorFlow version (use command below): 2.0.0-dev20190504
Python version: 3.6.7
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version: 10
GPU model and memory: Geforce GTX 1070 8GB

Describe the current behavior
When I try to train_on_batch, TF 2.0 allcaotes memory endless and finally crashes. Using tf.keras.backend.set_learning_phase(1) can be temporary solution, but this trick doesn work with lateset build (dev20190511).
Describe the expected behavior
The model can be trained regardless of tf.keras.backend.set_learning_phase(1) because I used model.train_on_batch(). It should handle learning phase internally.
Code to reproduce the issue
import tensorflow as tf
import numpy as np

def conv(x, filters, kernel_size, strides=(1, 1), padding='same', initializer='he_normal'):
    c = tf.keras.layers.Conv2D(
        filters=filters, kernel_size=kernel_size, strides=strides, padding=padding, kernel_initializer=initializer, use_bias=False)(x)

    return c

def conv_bn(x, filters, kernel_size, strides=(1, 1), padding='same', initializer='he_normal', bn_gamma_initializer='ones'):
    c = conv(x, filters=filters, kernel_size=kernel_size,
             strides=strides, padding=padding, initializer=initializer)

    c_bn = tf.keras.layers.BatchNormalization(
        gamma_initializer=bn_gamma_initializer)(c)

    return c_bn

def conv_bn_relu(x, filters, kernel_size, strides=(1, 1), padding='same', initializer='he_normal', bn_gamma_initializer='ones'):
    c_bn = conv_bn(x, filters=filters, kernel_size=kernel_size, strides=strides, padding=padding,
                   initializer=initializer, bn_gamma_initializer=bn_gamma_initializer)

    return tf.keras.layers.Activation('relu')(c_bn)

def conv_gap(x, output_filters, kernel_size=(1, 1)):
    x = conv(x, filters=output_filters, kernel_size=kernel_size)
    x = tf.keras.layers.GlobalAveragePooling2D()(x)

    return x

def my_block(x, output_filters, inter_filters):
    c1 = conv_bn_relu(x, inter_filters, (1, 1))
    c2 = conv_bn_relu(c1, inter_filters, (3, 3))
    c3 = conv_bn(
        c2, output_filters, (1, 1), bn_gamma_initializer='zeros')

    p = tf.keras.layers.add([c3, x])

    return tf.keras.layers.Activation('relu')(p)

def my_block_inc(x, output_filters, inter_filters, strides1x1=(1, 1), strides2x2=(2, 2)):
    c1 = conv_bn_relu(
        x, inter_filters, (1, 1), strides=strides1x1)
    c2 = conv_bn_relu(
        c1, inter_filters, (3, 3), strides=strides2x2)
    c3 = conv_bn(
        c2, output_filters, (1, 1), bn_gamma_initializer='zeros')

    strides = np.multiply(strides1x1, strides2x2)
    s = conv_bn(
        x, output_filters, (1, 1), strides=strides)  # shortcut

    p = tf.keras.layers.add([c3, s])

    return tf.keras.layers.Activation('relu')(p)

def repeat_blocks(x, block_delegate, count, **kwargs):
    assert count >= 0

    for _ in range(count):
        x = block_delegate(x, **kwargs)
    return x

# This line makes trick!
# tf.keras.backend.set_learning_phase(1)
shape = (299, 299, 3)

inputs = tf.keras.Input(shape, dtype='float32')
outputs = conv_bn_relu(inputs, 256 // 4, (7, 7), strides=(2, 2))
outputs = tf.keras.layers.MaxPool2D(
    (3, 3), strides=(2, 2), padding='same')(outputs)
outputs = my_block_inc(outputs, 256, 256 // 4, strides2x2=(1, 1))
outputs = repeat_blocks(outputs, block_delegate=my_block,
                        count=2,
                        output_filters=256,
                        inter_filters=256 // 4)

outputs = my_block_inc(outputs, 512, 512 // 4, strides2x2=(2, 2))
outputs = repeat_blocks(outputs, block_delegate=my_block,
                        count=7,
                        output_filters=512,
                        inter_filters=512 // 4)

outputs = my_block_inc(outputs, 1024, 1024 // 4, strides2x2=(2, 2))
outputs = repeat_blocks(outputs, block_delegate=my_block,
                        count=40,
                        output_filters=1024,
                        inter_filters=1024 // 4)

outputs = my_block_inc(outputs, 1024, 1024 // 4, strides2x2=(2, 2))
outputs = repeat_blocks(outputs, block_delegate=my_block,
                        count=16,
                        output_filters=1024,
                        inter_filters=1024 // 4)

outputs = my_block_inc(outputs, 1024, 1024 // 4, strides2x2=(2, 2))
outputs = repeat_blocks(outputs, block_delegate=my_block,
                        count=16,
                        output_filters=1024,
                        inter_filters=1024 // 4)

outputs = my_block_inc(outputs, 2048, 2048 // 4, strides2x2=(2, 2))
outputs = repeat_blocks(outputs, block_delegate=my_block,
                        count=6,
                        output_filters=2048,
                        inter_filters=2048 // 4)

outputs = conv_gap(outputs, 1024)
outputs = tf.keras.layers.Activation('sigmoid')(outputs)
model = tf.keras.models.Model(
    inputs=inputs, outputs=outputs, name='resnet_custom_v2')

optimizer = tf.optimizers.Adam(0.001)
model.compile(optimizer=optimizer, loss='binary_crossentropy',
              metrics=['mean_absolute_error'])

def make_batch(_):
    x = np.ones((shape[0], shape[1], shape[2]), dtype='float32')
    y = np.ones((1024), dtype='float32')

    return (x, y)

dataset = tf.data.Dataset.from_tensor_slices(
    ['0', '1', '2', '3', '4', '5', '6', '7', '8', '9'])
dataset = dataset.map(lambda x: tf.py_function(
    make_batch, (x,), (tf.float32, tf.float32)))
dataset = dataset.batch(10)

for x, y in dataset:
    step_results = model.train_on_batch(x=x, y=y)
    print(f'loss={step_results[0]}')

If you uncomment tf.keras.backend.set_learning_phase(1), the model may be successully trained. But with TF 2.0 dev20190511, the traininig is always failed regardless of tf.keras.backend.set_learning_phase(1).

Related comment:
https://github.com/tensorflow/tensorflow/issues/28110#issuecomment-490082449

@KichangKim Able to reproduce the issue using build (dev20190511) allocates endless memory and finally crashes.

I found that TensorFlow 2.0.0-beta1 still have this issue.

Same problem here. Exact same code was running fine on tf.keras 1.14 causes OOM with tf.keras 2.0. Will update code snippet.

Same problem here. Exact same code was running fine on tf.keras 1.14 causes OOM with tf.keras 2.0. Will update code snippet.

I think OOM issue is related with #31871 too. Also it is not fixed yet.

I found that tf-nightly-gpu==2.1.0.dev20191112 does not have this issue anymore. Thanks.


Update Test-ExchAVExclusions.ps1

Included l3codecp.acm from UM role in E16 and System.IdentityModel.Tokens.jwt.ni.dll that is sometimes in use by Exchange.
Issue:
Describe what the issue is you are addressing
Reason:
l3codecp.acm is used by UM in E16 and System.IdentityModel.Tokens.jwt.ni.dll in E16 and E19
Fix:
Just Add both to the list of modules in Exchange
Validation:
Test In lab

@iserrano76 This is approved but the branch needs to be rebased. Ping me on Teams if you need help with the rebase.


Provide option for setting an environment variable across an entire session

Is your feature request related to a problem? Please describe.
I like to set a $PROJECT environment variable when in tmux sessions. The value depends on the session. For example, when I'm in my ensemble session, the value is /home/openjck/dev/ensemble. When I'm in my glam session, the value is /home/openjck/dev/glam. That way, I can move around the file system as needed and simply type cd $PROJECT when I want to return to the root of the project.
I can imagine other use cases for session-wide environment variables like this.
Before I started using tmuxinator, I achieved this with some manually-written tmux commands. It worked some of the time, but it was pretty unreliable. Since then, tmux added an -e flag to new-session which seems to solve this exact problem. If I understand the documentation correctly, it allows an environment variable to be set across an entire session, no matter how many windows or panes are opened or closed.
Describe the solution you'd like
I'd like if the tmuxinator session config file allowed the user to specify an environment variable that would be active across the entire session. That environment variable should be readable from any window or pane, including new windows and panes.
Describe alternatives you've considered
A hook like pre_window could be used, but it wouldn't affect new windows.
Additional context
tmux recently added an optional -e flag to new-session which looks like it's meant for exactly this kind of thing.

This may be related to #494, but I'm having a hard time understanding exactly what that issue is about.

This is a good idea and I, for one, would be open to it.
In the meantime, I think you could mimic this behavior with some clever use of tmux hooks in a tmuxinator hook (I wish we'd used a better name, as hook is overloaded and could be cause for confusion), like on_project_start.
I haven't had time to test this, but something like:
on_project_start: tmux set-hook -a "window-add" "run-shell \"tmux set-environment foo bar\""

Is the environment variable foo used for something different in this context?
If I run tmux set-environment foo bar, it shows up when I run tmux show-environment, but now when I run echo $foo.
I've never quite understood what the show-environment environment is all about, but perhaps that's what the new flag to new-session appends to. I was hoping for something that could be used ala. echo $foo or cd $foo.

As mentioned, that temporary solution wasn't tested. It looks like you're right in that set-environment doesn't update existing windows/panes. There are kludgy solutions out there which will query show-environment and use its contents to set a variable on demand. I'm not sure I'd go to the trouble, though.
Another, even simpler temporary solution would be to set the project variable when starting tmuxinator: project=/home/openjck/dev/ensemble tmuxinator start ensemble

Ah. After experimenting for a while, I think part of the problem is that the hook would be named after-new-window. I appreciate the suggestion for a workaround; I hadn't used hooks before! :smiley:

I am using the following:
pre_window: export ABC=123 && tmux setenv ABC $ABC

The export ABC=123 part sets the environment variable for the first window, and tmux setenv ABC $ABC updates tmux's state so that later windows will have the same setting.

I am using the following:
pre_window: export ABC=123 && tmux setenv ABC $ABC

The export ABC=123 part sets the environment variable for the first window, and tmux setenv ABC $ABC updates tmux's state so that later windows will have the same setting.

The pre_window trick does the job if one somehow has access to the value within the tmuxinator YAML configuration.
What it doesn't seem to cover is when the user has an env variable set in their env and they want to pass it to tmuxinator for it to set it to all the windows.
I think it would be nice to be able to achieve that, what do people think?

@damdo I need exactly that, because I face a dynamic configuration that needs to be adjusted depending on some environment variables. Any attempt to access a variable in pre_window failed so far, but I desperately need this ...


job is running indefinitely, again!

Reference to #123.
Everything worked fine with version 3.2 . When I upgraded to 4.1, Some builds work, but those builds with a long log just run indefinitely. Nginx log:
<IP_ADDRESS> - - [18/Jan/2014:20:47:57 +0000] "PUT /api/v1/builds/50.json HTTP/1.1" 200 4 "-" "-"
<IP_ADDRESS> - - [18/Jan/2014:20:48:02 +0000] "PUT /api/v1/builds/50.json HTTP/1.1" 200 4 "-" "-"
<IP_ADDRESS> - - [18/Jan/2014:20:48:08 +0000] "PUT /api/v1/builds/50.json HTTP/1.1" 200 4 "-" "-"
<IP_ADDRESS> - - [18/Jan/2014:20:48:14 +0000] "PUT /api/v1/builds/50.json HTTP/1.1" 200 4 "-" "-"
<IP_ADDRESS> - - [18/Jan/2014:20:48:19 +0000] "PUT /api/v1/builds/50.json HTTP/1.1" 500 40 "-" "-"
<IP_ADDRESS> - - [18/Jan/2014:20:48:25 +0000] "PUT /api/v1/builds/50.json HTTP/1.1" 500 40 "-" "-"
<IP_ADDRESS> - - [18/Jan/2014:20:48:31 +0000] "PUT /api/v1/builds/50.json HTTP/1.1" 500 40 "-" "-"

Evidently, something to do with PUT request max body size

The server should probably return code 204 (no content) instead of 404 (not found).


Enable admins to set mentor profile images

Is your feature request related to a problem? Please describe.
Currently, there is no mechanism for administrators to manage or update mentor profile images, which limits their ability to ensure that profiles are up-to-date and accurately represent the mentors. This lack of functionality can lead to outdated or incorrect profile images, impacting the professionalism and effectiveness of mentor profiles.
Describe the solution you'd like

[ ] Update admin mentor profile view for managing mentor profile image
[ ] Implement image upload functionality
[ ] Implement image cropping/resizing if necessary
[ ] Add confirmation step before overwriting existing images

Describe alternatives you've considered
Additional context
Backend issue: https://github.com/sef-global/scholarx-backend/issues/158

Can you assign me for this ? I am looking to contribute in hactober fest. If you can assign me to do this and if my pr is getting accepted then it will be really helpful for me.
thanks

@P-yiush07 Sure! Go ahead

@P-yiush07 Sure! Go ahead

Is there any pre built api for updating mentor profile picture?


Use null datetime instead of 0000-00-00 to represent no dates

Part of a larger task of making the database follow a strict MySQL definition to prepare/allow for easier upgrading to new versions which as strict mode on by default.
Gazelle uses the '0000-00-00' date to represent that something doesn't have a date. That value is not a valid datetime as specified by the MySQL spec. While we could try and change all of these instances to use 1000-01-01 instead, given that these are being used to non-existent values, we should just use nulls instead.

been done


[Question] What Max Height and Width?

First! Congratulations !!!
What max size height and width do Bitmap decode?
Att.
Fernando

Thank you dear Fernando,
Max of height and width its little complicated, because all devices have different paper size.
I think no limit paper size but if your devices accepted your size(width or height).
Best Regards.


Error in page

I noticed that the example command is "RegistryEvents | limit 10", but that appears to be an invalid request.  As near as I can tell, the correct one is "DeviceRegistryEvents | limit 10".

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

ID: e93614ec-1755-75cc-7672-ccdfc50d2946
Version Independent ID: 9b867028-f6eb-fe80-ca88-2aa860a123bf
Content: Advanced Hunting with Python API Guide
Content Source: microsoft-365/security/defender-endpoint/run-advanced-query-sample-python.md
Product: m365-security
GitHub Login: @mjcaparas
Microsoft Alias: macapara

@gamory Thank you for your feedback. We have made changes to the document via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/8321
Once the author approves, the changes will be live.
Thanks
Sri

@msbemba Thank you very much for the contribution and updating the documentation with PR. @gamory Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Allow flux_bias_offset, anneal_offset, et al to be specified in terms of source variables

Similar to:
https://github.com/dwavesystems/dwave-system/blob/f7926eccf117c3a9959a0b8dc86f2c46902ff344/dwave/system/composites/embedding.py#L252
See also #217

Since the offsets play a huge role when dealing with long chains, I think it would make sense to reflect this logic in the API (maybe a utility accepting an embedding and some mapping function?).
Also, is it possible, that some embeddings might not allow for setting all physical qubits in the same chain to the same offset? If so, there should be a way for the embedding algorithm to take this information into account.


Got wired inference results when running example in Neural Machine Translation (seq2seq) Tutorial.

Sorry, I should open this issue in NMT.


Create a docker image for this project or a docker-compose config

Cool project!
A docker image would allow people to spin this up quickly.
A docker-compose config would also allow separate containers for each role, eg nginx vs elasticsearch.
You could use the official Elastic image for the Elasticsearch too.

So I tried this out, it spins up the containers ok and the web ui can see into the Elasticsearch container just fine to show a list of indices. I've not used Diskover before, so unsure if I'm doing something wrong along the way, but I wasn't able to log into the UI after picking the index to browse.
Do I need to wait for at least one successful processing run to complete first or should it allow logging in while a dir is still being crawled?

I think the issue is in the Dockerfile there is a CMD to create a symlink
for index.php, it's pointing to selectindices.php, it should be pointing to
dashboard.php.
In diskover-web, you can view indices which the crawls are still running as
well as finished indices.
On Fri, Jun 8, 2018 at 10:03 AM Peter Dyson<EMAIL_ADDRESS>wrote:

So I tried this out, it spins up the containers ok and the web ui can see
into the Elasticsearch container just fine to show a list of indices. I've
not used Diskover before, so unsure if I'm doing something wrong along the
way, but I wasn't able to log into the UI after picking the index to browse.
Do I need to wait for at least one successful processing run to complete
first or should it allow logging in while a dir is still being crawled?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/shirosaidev/diskover-web/issues/6#issuecomment-395613018,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQomf_D0BTsLi4UP-ZE4yaEG4Rkqwa9Qks5t6c1agaJpZM4USbEV
.

Ok cool, I'll test again when I can and provide outcome.

I fixed the issues described by @shirosaidev. I tested and it should work now.
You need to at least start a crawling using diskover to use the ui.

I've merged the docker additions into master, thanks!