ERROR:root:module 'warp_drive.numba_includes.env_runner' has no attribute 'NumbaCustomEnvStep'

Hi thank your help setting up this,
Following the same tutorial  and this examples I have set up the cpu env in custom_env.py where CustomEnv class is and the numba env in custom_env_step_numba.py where NumbaCustomEnvStep function is.
The environment I register is the first custom_en.py this way :
env_registrar.add_cuda_env_src_path(CustomEnv.name, "custom_env", env_backend="numba")

So I have been able to charge my cpu environment without any problem.
env_wrapper = EnvWrapper(
    env_obj=CustomEnv(**run_config["env"]),
    env_name=CustomEnv.name,
    num_envs=run_config["trainer"]["num_envs"],
    env_backend="cpu",
    env_registrar=env_registrar
)

However when I try to set env_backend="numba" I get the following error ERROR:root:module 'warp_drive.numba_includes.env_runner' has no attribute 'NumbaCustomEnvStep'
Not sure where is this error coming from. Warp_drive should have find the NumbaCustomEnvStep class in custom_env_step_numba.py but it obviously did not.
What am I missing again ?

I should precise that this error message is prompted when self.env.initialize_step_function_context is called in env_wrapper.py

You can simply test from <<YOUR_ENV_NUMBA_LIB>> import * and see if your NumbaCustomEnvStep can be loaded successfully. And this <<YOUR_ENV_NUMBA_LIB>> is what you shall use here env_registrar.add_cuda_env_src_path(CustomEnvironment.name, "<<YOUR_ENV_NUMBA_LIB>>", env_backend="numba")
There is no magic here, here we just secretly integrate your numba module into the entire ecosystem, and essentially if your from <<YOUR_ENV_NUMBA_LIB>> import * works, then it should work. If not, please report to me, because we have multiple custom CUDA environments loaded by other users but not Numba, so I can imagine if there is some hiccup.

So I guess I wasn't using the proper <<YOUR_ENV_NUMBA_LIB>>. I was using the custom_env file (so the cpu versions with the Class that call the numba step version) instead of custom_env_step_numba (the numba file with only functions). This wasn't to clear from the tutorials I guess especially because the tutorial says that you need to register the environment. One question remains though : how does warpdrive knows about that cpu file 'custom_env' and the CustomEnv class ? Is this done secretly as well ?
Anyway after changing that I tried what you said
from custom_env_step_numba import *
NumbaCustomEnvStep

loads the custom_env_step_numba and outputs correctly
CUDADispatcher(<function NumbaCustomEnvStep at 0x7fe0a451ee50>)

BUT I still get the same error
ERROR:root:module 'warp_drive.numba_includes.env_runner' has no attribute 'NumbaCustomEnvStep'

The cpu model class is there when you call import custom_env, this is just a Python class, this is not magic but just a Python import as you did for any Python codes else.
The specialty is that since you have a Numba step(), and this step() is actually outside of your Python class , WarpDrive needs to know where is the source code of the Numba step() and then integrate the compiled Numba step() to your Python class. This is done by the EnvWrapper, that is why you see the error from there.

It seems there is still something wrong with my implementation as I get the following stacktrace 👍
---------------------------------------------------------------------------
AssertionError                            Traceback (most recent call last)
Cell In[13], line 1
----> 1 trainer.train()

File /project_ghent/warp-drive/warp_drive/training/trainer.py:422, in Trainer.train(self)
    419 start_time = time.time()
    421 # Generate a batched rollout for every CUDA environment.
--> 422 self._generate_rollout_batch()
    424 # Train / update model parameters.
    425 metrics = self._update_model_params(iteration)

File /project_ghent/warp-drive/warp_drive/training/trainer.py:469, in Trainer._generate_rollout_batch(self)
    467 # Step through all the environments
    468 start_event.record()
--> 469 self.cuda_envs.step_all_envs()
    471 # Bookkeeping rewards and done flags
    472 _, done_flags = self._bookkeep_rewards_and_done_flags(batch_index=batch_index)

File /project_ghent/warp-drive/warp_drive/env_wrapper.py:355, in EnvWrapper.step_all_envs(self, actions)
    351 """
    352 Step through all the environments
    353 """
    354 if not self.env_backend == "cpu":
--> 355     self.env.step()
    356     result = None  # Do not return anything
    357 else:

File /project_ghent/collective_MARL/custom_env.py:598, in CustomEnv.step(self, actions)
    595     if self.env_backend == "numba":
    596         print("Try calling numba step function")
    597         self.cuda_step[self.cuda_function_manager.grid, self.cuda_function_manager.block](
--> 598             *self.cuda_step_function_feed(args)
    599         )
    600     result = None  # do not return anything
    602 # CPU version of step()
    603 else:

File /project_ghent/warp-drive/warp_drive/managers/function_manager.py:120, in CUDAFunctionFeed.__call__(self, arguments)
    118 for arg in arguments:
    119     if isinstance(arg, str):
--> 120         data_pointers.append(self.data_manager.device_data(arg))
    121     elif isinstance(arg, tuple):
    122         key = arg[0]

File /project_ghent/warp-drive/warp_drive/managers/data_manager.py:395, in CUDADataManager.device_data(self, name)
    393     assert name in self._host_data
    394     return self._host_data[name]
--> 395 assert name in self._device_data_pointer
    396 return self._device_data_pointer[name]

AssertionError:

Here self.cuda_step_function_feed(args) is called in my custom_env.py file but I don't understand what fails after...

You have some data array not registered in the data manager


Fix logging in started release (close #361)

Thanks! I've been using foreground myself, so I never noticed that the logs were messed up.


Update love calculator

Describe the Feature you want to add
Want to change the frontend part of the website.
Expected behavior
Will be adding he background image and aliging he contents of the page.
Screenshots (optional)

Additional context (optional )

@pranjay-poddar will you please assign me this issue

@pranjay-poddar this issue isnt closed?

or do I need to make any changes?


[ENG-567] migrating default branch to main

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Sample for .ContainerInstances

Hi!
I'm migrating from the old management SDK and struggling with creating container instances with a lot of options. Example would be appreciated.
How do I set exposed UDP ports, and command line options? With the old Fluent SDK it was done like this:
.DefineContainerInstance(containerId + "-1")
                .WithExternalUdpPort(51234)
                .WithStartingCommandLine(_containerConfig.ImageExecutable, arguments)
                .Attach()

Hi @yacuzo , Thank you very much for your feedback. I am not sure if the method DefineContainerInstance you mentioned is included in Microsoft.Azure.Management.ContainerInstance.Fluent.ContainerGroup. However, I could not find any sample related to Azure.ResourceManager.ContainerInstance in this sample. Therefore, I can only answer your question based on the information I found. Based on the latest Azure.ResourceManager.ContainerInstance, you can refer to the following example to achieve the same functionality:
var containers = new ContainerInstanceContainer[]
{
new Models.ContainerInstanceContainer(
name: containerGroupName,
image: "alpine",
resources: new ContainerResourceRequirements(
new ContainerResourceRequestsContent(
memoryInGB: 1.5,
cpu: 1.0))
)
{
Ports =
{
new ContainerPort(80)
},
Command =
{
"/bin/sh", "-c", "while true; do sleep 10; done"
},
}
};


[Fix] the tests for custom image import

https://issues.redhat.com/browse/RHOAIENG-1192 has been resolved, so we can update that test accordingly.
Plust this fixes another one test in the custom image import and fixes some common keywords from the ODHDashboard.robot file.

CI:

I think you could set some of the xpath variables as globals and reuse them across keywords, but this works too

You are right, Luca - I was considering it for the custom_image_name_record but it needs the image name to be resolved properly. At the same time - shortening it so it doesn't contain the image name looked a bit ugly for me. Thus I left it defined in each function... maybe we can restructure it in the future


scalafmt makes treehugger code hard to read

especially the fact that chained method calls get indented like here:
https://github.com/unicredit/sbt-swagger-codegen/blob/master/lib/src/main/scala/eu/unicredit/swagger/generators/DefaultClientGenerator.scala#L191
can this indent feature be switched off?
an alternative would be to use // format: off for treehugger-code, but that kind of defeats the purpose of an autoformatter as treehugger AST makes up quite a big part of the code this project has.

I'd go for // format: off for now. What do @andreaTP and @fralken think?

Scalafmt now is not automatic during compilation, you must explicitly invoke scalafmt to format code.
The idea was to help formatting treehugger AST code, which is intrinsically hard to read, but without imposing it, that it could be quite annoying sometimes.
Do you think it would be better to get rid of it completely?
Scalafmt reason was to help formatting AST code, disabling just for it doen't make much sense to me.

Now Scalameta is used for code generation


Add some command like NpmInstall for package.json

It looks like npm install . and npm install --dev . doesn't work as expected, and tries to install the package itself (maybe because we are installing with npm install --global).
Anyway we should support package.json as good as python's requirements.txt. Fortunately it's easy to do because file is JSON, so we don't have to implement a parser.
Other question is whether dev-dependencies should be installed by default or dependencies? And should it be single command or two separate ones.

docs


Added external IoC / Dependency Injection support

Minimal changes to BsonMapper's constructor, now it takes an optionnal
Func<Type,object> parameter, to let the user specify a custom type instanciation function.
implements #325

Well I needed structure support as well ( #320 ), so I added it, still using Emit


[PE-D][Tester A] Wrong error message shown for invalid claim amount

When a claim amount is invalid due to input appending at the back of a number, the following error message is shown.

This is inconsistent with the usual error message despite both being invalid inputs.

Labels: type.FunctionalityBug severity.Low
original: Oceankoh/ped#11

any characters after the "." is considered a digit unless its a space
put inside the UG.

Defense: both are meaningful error messages and similar meaning. Would not really affect user experience


Task API: TaskExecution contain a TaskResult

I think it would be useful if a TaskExecution provided a way to get the result of the Task being executed which would include the exit code, stdout, and stderr.  This would allow for things like providing a different notification when a Task completes depending on success or failure, or scanning the output and reporting back certain details.
I also think it would be useful if a cached TaskExecution could be polled to see when it has a TaskResult, so that if an extension kicks off a Task, it can monitor it for completion, instead of having to subscribe to onDidEndTask when it's only concerned with a particular Task.

I still don't see why polling is preferred over eventing especially in a system like NodeJS. Can you let me know in which case polling is preferred.

I don't know NodeJS well enough to directly address that, so it could very well be unnecessary.  My perspective is that I would expect to be able to track the life of a Task without having to subscribe to a global event and compare each ending Task with the one I'm interested in.
For comparison, creating a process in Windows/Linux/MacOS returns a process ID which can be used query the state of that process, terminate it, etc.  Your terminateTask() API fits that pattern (using the TaskExecution as the identifier).  For completeness I'd expect to be able to get the TaskResult from a similar API, or a future to the TaskResult which could then be waited on.

Since NodeJS is an async event based system (single threaded) polling can only be done using a setInterval which at the end results in the same as adding an event handler to the task end event. I will look into how to best allow for query (which does make sense) but not to encourage polling.


Notification, minicontroller and expandedControllsActivity do not get updated when next item in queue starts

Here're the steps that produce the issue:

Play a video
Add another video into the queue
Turn off the phone display and throw it away
When chromecast plays the second video in queue, I turn on the display, but the notification, minicontroller and ExpandedControllsActivity all stay at the end of first video,
There's no way to get them updated without disconnecting media cast session.

By the way this only happens after putting the display off for quite long time, let's say 20-30 minutes

File a bug against the Cast SDK and provide a complete log: https://code.google.com/p/google-cast-sdk/issues/list


ClosestPointDist: calculation of bounding spheres of nodes may not be correct

Hello.
The formula at
https://github.com/AcademySoftwareFoundation/openvdb/blob/7a1c1764be8cc8d1295f2a02e1ca8a1364021947/openvdb/openvdb/tools/VolumeToSpheres.h#L313
computing radius of a bounding sphere of a node around bounding spheres of leaves does not seem right.
        for (size_t i = mLeafRanges[n].first; i < mLeafRanges[n].second; ++i) {
            pos[0] = mLeafBoundingSpheres[i][0];
            pos[1] = mLeafBoundingSpheres[i][1];
            pos[2] = mLeafBoundingSpheres[i][2];
            const auto radiusSqr = mLeafBoundingSpheres[i][3];

            double tmpDist = (pos - avg).lengthSqr() + radiusSqr;
            if (tmpDist > maxDist) maxDist = tmpDist;
        }

Here a square of a new bounding radius is calculated by summing square radius of a sphere with square distance of centers connecting the bounding sphere with the sphere to be enclosed. The resulting tmpDist is always smaller than it should be. That does not break any unit test, but because the bounding sphere is smaller than it should be, the search may not be optimal.
I am not sure whether the code above is an optimization (approximation to avoid square roots) or whether it is a bug.

I see, the result is corrected for at
https://github.com/AcademySoftwareFoundation/openvdb/blob/7a1c1764be8cc8d1295f2a02e1ca8a1364021947/openvdb/openvdb/tools/VolumeToSpheres.h#L322
Sorry for the noise.


Display the type of ward (i.e. contagious, non-contagious)

As a user I want to know the type of ward (i.e. contagious, non-contagious) so that I know if I need to don PPE before attending to them

Closed via #245, #249 as this is classified under Tags.


Question: How to add catagory in LNReader

How to add catagory in LNReader

Currently you can't create categories in the app. It is a planned feature but there's no ETA for it.
Duplicate of #45


Nvidia Deep Pose - Testing

Hello,
Is it possible to test the Nvidia deep pose models same as the eval_rotating_cup?
Thank you!

Hello, to evaluate 6D object pose estimation methods (such as Nvidia deep pose), please use the BOP Toolkit or the BOP online evaluation system. The eval_rotating_cup (and the associated ECCVW'16 paper) was one of the first steps leading to the BOP benchmark.

Hi,
I am trying to evaluate Dope obj pose estimation. Like calculating ADD metrics, but I can't find any example or implementation steps? any help will be appreciated

Hello, the obj_pose_eval repository is no longer maintained. Instead, please use the BOP Toolkit which includes all of the functionality. Please refer to this issue to see how to use ADD/ADI for the evaluation using the BOP Toolkit.


docs:  Restructure key sections for better discoverability

Lift contents out of the usage/contrib chapter into appropriately labelled top-level chapters so users can easily discover them
Break up the "The Litestar App" into top-level chapters
Move "routing", "route handlers" and "parameters" as sections into a "routing" chapter
Improve consistency of some headline wording
Fix a few minor spelling and content error I came across

Very nice 👍.


Dense or TimeDistributedDense for lstm_text_generation.py example??

Dear all,
I just found good LSTM example (keras)- https://github.com/fchollet/keras/blob/master/examples/lstm_text_generation.py
In the code, author used 'Dense' layer for fully connection LSTM layers.
I wonder why not used 'TimeDistributedDense' instead of 'Dense'?
any idea?

The last LSTM layer does not return a sequence, but a single vector (return_sequences=False). Hence Dense instead of TimeDistributedDense.


De-deprecate @MongoId

@MongoId is actually pretty useful in cases when a class is stored in mongo as well as used as a response in REST API. At the moment kmongo forces id field to be named _id which makes it impossible to use with existing model classes that already use kotlinx.serialization and can't have id field be renamed to _id.

nvm, I missed that it's doable with a custom IdController


UI bug? SmoothProductImage with different sizes

In https://github.com/openfoodfacts/smooth-app/commit/8d6f045cc0d8a6c8b491af43dca6f438c33af863 (about "responsive design") some code changed: width and height were replaced with maxWidth and maxHeight in SmoothProductImage.
I don't know if it was the original goal, but now the products don't share the same dimensions.
Given that we have no control on the product pictures, it means we can have a very inconsistent display.
The previous display contained the picture into the same rectangle for all products, and that seemed more appropriate.
I'm not a big fan of the current display:

Ohh did not noticed that, I dont like it so either, but providing a fixed size will probably cause issues on different screen sizes. Ill try to fix it

I'm not talking about a fixed size. I'm talking about the same size for all displayed items.

Just width or also height

The human brain puts a premium on regularity.
We should

[ ]  take the whole width of the screen for the suggestions
[ ] create a grid (vertical line for all the text, which start at the same point) to create regularity
[ ] Center the images on the left column

it should be the same size (width and height) for all the cells containing the images ?

@teolemon/ @monsieurtanuki, this is my first attempt. I am not cinvinced by the blur effect behind the images but I cant think of a better alternative,

I like the fact that the suggestions take almost the full width. But why not take all the full width? Instead of having the suggestions hovering over the rest of the screen, they could just push the rest of the screen down.

For the record, what @M123-dev implemented is very similar to the behavior of google search (which can be considered as a reference, can't it?): not full screen width, and hovering over the rest of the screen.
I also assume that the width of the suggestions depends on (and cannot be greater than) the width of the text field.
The initial expressed requirement ("text field with automatic suggestions") is (and was already) fulfilled. It's currently improved, with the larger width. Obviously there's still some fine tuning needed about the ratio and the padding of the pictures, which was precisely the purpose of the current issue.
The topic of text field with suggestions is interesting enough to be worth several discussions and issues, about

the UI
the data sources (local database and/or API?)
the product fields to perform the search on (product name? in which language?)
the order of the results (newer products first?)
the possible action(s) when a suggested product is clicked (add to a list? alway go to the product?)

My personal preference: I'd prefer the pictures to be centered in a square instead of a rectangle. I would also remove the blur and the rounded corners. And add a bit of vertical spacing.

The UI can of course still be fine-tuned, what I wonder, @stephanegigandet , are the suggestions only for the search suggestions or also for the photos in the cards. There the images still have different heights.

I like the fact that the suggestions take almost the full width. But why not take all the full width? Instead of having the suggestions hovering over the rest of the screen, they could just push the rest of the screen down.

As @monsieurtanuki correctly assummed, the size of the suggestion popup depends on the size of the text field. To get the size to full width we would have to remove all padding and a small border does not hurt in my opinion.

the data sources (local database and/or API?)

Of course the API and before its response we can display local products. We could probably add lists pantrys to it, but the app is not that big as Microsoft Teams for example it is rather a nice to have feature, but it is not really needed.

the product fields to perform the search on (product name? in which language?)

Of course product name, other things rather not. For this there is the advanced search on the website. (We can add this later, but we don't really need it).

the order of the results (newer products first?)

What do you mean by "newer"?

the possible action(s) when a suggested product is clicked (add to a list? alway go to the product?)

For the main search bar just opening it, but for the add action block we can add it directly

I finetuned a bit in #311
I left the corners and the blur, the rounded corners because everything in the app has rounded corners, and without the blur the images would have different sizes again (when we dont want to crop them)

@M123-dev Looks good to me!
It fixes the issue (product image size and ratio) and answers an additional requirement by @teolemon about width.

I think we are ready to close this. The main problem of this issue the difference of the picture size in the searchbar and most the other mentioned things are now updated in #311


Update style.css

I have center aligned and improved the participants column for any screen as well as fixed the overlapping of participant's name and messages in chat by changing position  attribute. This is my first pull request concerned with the issue #61  👍.
Before
The chat messages were overlapping with the participants column and column was not properly aligned.

After
Both the issues are fixed and now the chat message being displayed  at top of screen  slides downwards on opening the participant's column.

Can you put a before and after screenshot?

yeah sure just give me an hour after quiz i ll surely do it.

@Anmol2001 in the 1st screenshot of After, it looks like the message is going below the input box. Can you fix that and update?

@Anmol2001 in the 1st screenshot of After, it looks like the message is going below the input box. Can you fix that and update?

@abhigyank  sure ,i will try.

@Anmol2001 your branch has so many conflicts, either you reset this branch and then rebase with upstream or open another PR after closing this one.

closing in favor of fa4c0c369d52be122dd1cc649cb93c3aee099510


Compiler experiments halt around 11th cycle

Bug that needs to be investigated, as I haven't the foggiest what is causing it.  Effectively, we're getting to the compile phase and all feedback from the machine node is stopping, while the rest of the director/dashboard remains responsive.

Hopefully fixed (there was a nasty aliasing/concurrency bug, how ironic!), we'll see.

Hopefully fixed (there was a nasty aliasing/concurrency bug, how ironic!), we'll see.

Seems to be ok now.


The transitive dependency which has a version variable won't be generated

For example, the properties and dependencies in the pom.xml of artifact com.google.inject:guice:3.0 are:
<properties>
    <cglib.version>2.2.1-v20090111</cglib.version>
</properties>
<dependencies>
    <dependency>
      <groupId>javax.inject</groupId>
      <artifactId>javax.inject</artifactId>
      <version>1</version>
    </dependency>
    <dependency>
      <groupId>aopalliance</groupId>
      <artifactId>aopalliance</artifactId>
      <version>1.0</version>
    </dependency>
    <!--
     | Replace with official CGLIB artifact when it's released
    -->
    <dependency>
      <groupId>org.sonatype.sisu.inject</groupId>
      <artifactId>cglib</artifactId>
      <version>${cglib.version}</version>
    </dependency>
    <dependency>
      <groupId>javax.inject</groupId>
      <artifactId>javax.inject-tck</artifactId>
      <version>1</version>
      <scope>test</scope>
    </dependency>
    <dependency>
      <groupId>org.springframework</groupId>
      <artifactId>spring-beans</artifactId>
      <version>3.0.5.RELEASE</version>
      <scope>test</scope>
    </dependency>
    <dependency>
      <groupId>biz.aQute</groupId>
      <artifactId>bnd</artifactId>
      <version>0.0.384</version>
      <scope>test</scope>
    </dependency>
    <dependency>
      <groupId>org.apache.felix</groupId>
      <artifactId>org.apache.felix.framework</artifactId>
      <version>3.0.5</version>
      <scope>test</scope>
    </dependency>
  </dependencies>

Then only add the com.google.inject:guice:3.0 to the dependencies.yaml and run bazel-deps. However, it will only generate three dependencies without artifact org.sonatype.sisu.inject:cglib:2.2.1-v20090111.
def maven_dependencies(callback = declare_maven):
    callback({"artifact": "aopalliance:aopalliance:1.0", "lang": "java", "sha1": "0235ba8b489512805ac13a8f9ea77a1ca5ebe3e8", "repository": "http://szxy1.artifactory.cd-cloud-artifact.tools.huawei.com/artifactory/third-party-maven/", "name": "aopalliance_aopalliance", "actual": "@aopalliance_aopalliance//jar", "bind": "jar/aopalliance/aopalliance"})
    callback({"artifact": "com.google.inject:guice:3.0", "lang": "java", "sha1": "9d84f15fe35e2c716a02979fb62f50a29f38aefa", "repository": "http://szxy1.artifactory.cd-cloud-artifact.tools.huawei.com/artifactory/third-party-maven/", "name": "com_google_inject_guice", "actual": "@com_google_inject_guice//jar", "bind": "jar/com/google/inject/guice"})
    callback({"artifact": "javax.inject:javax.inject:1", "lang": "java", "sha1": "6975da39a7040257bd51d21a231b76c915872d38", "repository": "http://szxy1.artifactory.cd-cloud-artifact.tools.huawei.com/artifactory/third-party-maven/", "name": "javax_inject_javax_inject", "actual": "@javax_inject_javax_inject//jar", "bind": "jar/javax/inject/javax_inject"})

We do not add test scope dependencies.
On Thu, Oct 26, 2017 at 16:38 GinFungYJF<EMAIL_ADDRESS>wrote:

For example, the properties and dependencies in the pom.xml of artifact
com.google.inject:guice:3.0 are:

2.2.1-v20090111

javax.inject
javax.inject
1

aopalliance
aopalliance
1.0

org.sonatype.sisu.inject
cglib
${cglib.version}

javax.inject
javax.inject-tck
1
test

org.springframework
spring-beans
3.0.5.RELEASE
test

biz.aQute
bnd
0.0.384
test

org.apache.felix
org.apache.felix.framework
3.0.5
test

Then only add the com.google.inject:guice:3.0 to the dependencies.yaml
and run bazel-deps. However, it will only generate three dependencies
without artifact org.sonatype.sisu.inject:cglib:2.2.1-v20090111.
def maven_dependencies(callback = declare_maven):
callback({"artifact": "aopalliance:aopalliance:1.0", "lang": "java", "sha1": "0235ba8b489512805ac13a8f9ea77a1ca5ebe3e8", "repository": "http://szxy1.artifactory.cd-cloud-artifact.tools.huawei.com/artifactory/third-party-maven/", "name": "aopalliance_aopalliance", "actual": "@aopalliance_aopalliance//jar", "bind": "jar/aopalliance/aopalliance"})
callback({"artifact": "com.google.inject:guice:3.0", "lang": "java", "sha1": "9d84f15fe35e2c716a02979fb62f50a29f38aefa", "repository": "http://szxy1.artifactory.cd-cloud-artifact.tools.huawei.com/artifactory/third-party-maven/", "name": "com_google_inject_guice", "actual": "@com_google_inject_guice//jar", "bind": "jar/com/google/inject/guice"})
callback({"artifact": "javax.inject:javax.inject:1", "lang": "java", "sha1": "6975da39a7040257bd51d21a231b76c915872d38", "repository": "http://szxy1.artifactory.cd-cloud-artifact.tools.huawei.com/artifactory/third-party-maven/", "name": "javax_inject_javax_inject", "actual": "@javax_inject_javax_inject//jar", "bind": "jar/javax/inject/javax_inject"})
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/johnynek/bazel-deps/issues/91, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEJdjovHa_TpkQAVLZ0gW6XVn82GqOeks5swUIOgaJpZM4QIeJD
.

--
P. Oscar Boykin, Ph.D. | http://twitter.com/posco | http://pobox.com/~boykin

I mean artifact org.sonatype.sisu.inject:cglib:2.2.1-v20090111 doesn't be generated, and it isn't a test scope dependency.

I wonder if it is the variable substitution that is failing. It would be nice to make a test case and work on it. Honestly the maven resolution is not tested very well sadly.

After debugging, I found that the maven resolution thinks that the artifact org.sonatype.sisu.inject:cglib:2.2.1-v20090111 is an optional dependency because it is declared as an optional dependency in an activated profile in the pom.xml. As a result the tool doesn't generated it. So close this issue.


feat(python): fastapi plugins for UT #540

rename lib name for naming collision
testcase for ut

close #540

test case https://github.com/pinpoint-apm/pinpoint-c-agent/pull/541/files#diff-357c1ac1d734f0dfdc2fa029c9e9df13d8b1973ca549931615e8ae0c5607c2d0


Failed to negotiate substream: Protocol negotiation failed

I see heaps of logs about this. Does not sound normal to me:
taker: 76d3bd18018a0087aea761fd560f49ddd08a5e4e
2022-08-28 11:14:31 DEBUG xtra_libp2p::endpoint: Failed to negotiate substream: Protocol negotiation failed.
2022-08-28 11:14:32 DEBUG xtra_libp2p::endpoint: Failed to negotiate substream: Protocol negotiation failed.

Adding it to this iteration. But if it's not critical, feel free to postpone.

I'm removing the bug label given what I mentioned in my previous comment. I'll assign myself to this ticket and repurpose it to describe the current behaviour and how I think we should solve it.

I'll assign myself to this ticket and repurpose it to describe the current behaviour and how I think we should solve it.


[Request] Load title names from external file

I would like you to add support for loading the names of games from an external file, such as the 3dsdb.com XML list. This would make it a lot easier to find the title in a list of games from the sd card, instead of having to find what titleid coresssponds to what game.
If implemented, it would also be nice to be able to load names of system titles/homebrew from an xml as well, which would make it a lot easier to find the title that you needed.

The rewrite already has support for showing titles from the SMDH, whenever it ends up finished.


Build creates strange folder structure lib/lib

Is your feature request related to a problem? Please describe.
I noticed that when installing this module, react-calendar-timeline has the following folder structure:

Describe the solution you'd like
I think it would be great if we could change the build command to write to a dist folder as it's arguably more conventional.
https://github.com/namespace-ee/react-calendar-timeline/blob/b94cd37e196ca64b59079dbea62d9b795ef41221/package.json#L9
Happy to submit a PR for this in the next days!

that would help thanks

Hey @tobiasbueschel and @Ilaiwi, mind seeing if the above PR would do what you are thinking of?
I agree the lib/lib output is a little odd. My only question is whether there's much to be gained by shipping the breaking change.


ADX import

These look like good references:

https://stackoverflow.com/questions/52055212/convert-xsd-to-typescript
https://github.com/FranklinChen/cxsd/tree/upgrade-dependencies


Text Extraction / How to get font information like italic or bold

Hi, I'm sorry to misuse this issue reporting for a question. But I guess I get the most usefull information from here.
If I use
let content = yield page.getTextContent();
content.items[0]...

How can I tell that item 0 is italic or bold?

As far as I know this information is not exposed through the API. Marking as a feature for now.

In general, I don't think that it's actually possible to obtain this information with any certainty, considering that there's a number of ways a font could be made to look bold/italic.
For example, in non-embedded standard fonts, you can often tell by the font name (as is done here).
For embedded fonts, it's usually a lot more difficult to tell, since the font name may be (and often is) somewhat arbitrary. In this case, whether or not the text looks bold/italic depends on the embedded font data itself.
Then there are the font flags, but based on experience I wouldn't really trust them, since PDF files unfortunately often provides incomplete/inconsistent/broken font information.
Furthermore, even though (in my experience) a majority of PDF files uses different fonts to represent regular/bold/italic text, there are exceptions. One such example is #7297, where the Text rendering mode is modified in order to simulate bold text.
So, to summarize, there's unfortunately no way that you can consistently obtain the information you're asking for. And as https://github.com/mozilla/pdf.js/issues/7297#issuecomment-217631260 said, this seems out-of-scope for text-selection/text-extraction use case.

@Snuffleupagus thank you for the insights. I'l have a look at the font names used, and hardcode it


Can't pull Saves From xbox Error.

Xbox Game Pass for PC savefile extractor
Traceback (most recent call last):
File "main.py", line 303, in
main()
File "main.py", line 256, in main
temp_dir = tempfile.TemporaryDirectory(ignore_cleanup_errors=True)
TypeError: init() got an unexpected keyword argument 'ignore_cleanup_errors'
Any help?  I keep getting "Caution!  errors and I want to abandon Xbox.

Your Python is too old, update it: https://www.python.org/downloads/
Alternatively you can just remove the ignore_cleanup_errors=True part from the indicated line, it shouldn't be absolutely necessary.


HDDS-2481. Close streams in TarContainerPacker

What changes were proposed in this pull request?
Make sure all streams are closed in TarContainerPacker (571fc9003b490db324ccd601893a18a6b5bcace3).  Plus various refactorings, see list of commits.
https://issues.apache.org/jira/browse/HDDS-2481
How was this patch tested?
Ran TestTarContainerPacker.

Thanks @anuengineer for reviewing and committing it.


Using lightbody with jetty-server:jar:9.0.6.v20130930

I'm trying to use the proxy code with a newer jetty version but it gives me this error
java.lang.NoSuchMethodError: org.eclipse.jetty.util.UrlEncoded.decodeTo(Ljava/lang/String;Lorg/eclipse/jetty/util/MultiMap;Ljava/lang/String;)
It seems that the function decodeTo in the new version has an additional parameter so it raises this exception. Could you please make a new release which depends on a newer jetty version and fix this issue?
Thanks.

I'm running into same issue, could anyon help ? I'm using :

net.lightbody.bmp
browsermob-proxy
2.0.0

Unfortunately, 2.0.0 is not compatible with Jetty 9. It has a hard dependency on Servlet API 2.5, and using Servlet API 3.0+ causes several issues.
In the 2.1.0 branch in master, it should theoretically be possible to produce a browsermob-core shaded artifact that you could embed in a Jetty 8 or 9 application. That would take some tinkering with the maven-shade-plugin in the browsermob-core pom file, but I would welcome any PR that does so.
In 2.1.x the Jetty 5 code will eventually be optional, and in 3.0 we are planning to remove the Jetty 5 code entirely, so that will eliminate the compatibility issues with Jetty 8 and 9. But that's still a ways down the line, unfortunately.

This issue should now be resolved when using the LittleProxy implemenation. Make sure to not include browsermob-core in your pom, only browsermob-core-littleproxy. Let me know if you still run into these problems.

What pom dependency should we be using instead of
net.lightbody.bmp
browsermob-proxy
2.0.0
We can not upgrade to browsermob 3.0 at this point

If you want to avoid the Jetty 9 conflict, your only option is to use the LittleProxy implementation. Version 2.1.0 was just released. The legacy implementation is not being actively developed, and no further enhancements/fixes/etc. are planned for it.

Thanks for quick response. I added below dependancy but i still see same issue. I tried to exclude "browsermob-core" but that did not help either .. any further pointers?
net.lightbody.bmp
browsermob-proxy
2.1.0 ..


Properties on η

I will finish the work started in #355 to specify η.
I will mainly focus on proving postponements and perhaps bisimulations.

Small remark: we can't eta-expand under lets with your definition.
In PCUICReduction, Matthieu did an almost full type of terms with hole / stack. One day we should unify ...

Oh. I didn't see #354 sorry.

Well my definition is complete (except for CoFix but that would be a copy-paste of Fix stuff) and has already properties proven on it so it felt appropriate.

Well what I'm worried about is the apparent duplication with term_context and fill_context in PCUICReduction.v.

This can be merged now…


Redis ingest is growing faster than pulse:work can keep up with

Pulse Version
1.3.2
Laravel Version
11.36.1
PHP Version
8.3.13
Livewire Version
latest
Database Driver & Version
MySql 8.0.33
Description
Hi, I'm using the Redis ingest stream for my project. I'm observing that the ingest stream in Redis appears to be growing slightly quicker than what the worker can keep up with.
What can be done to prevent the stream growing excessively and killing a Redis instance by running out of memory whilst still retaining data accuracy?
Steps To Reproduce
What can be done?

Is this a very high-traffic application?
There are a few options:

Determine if there's a particularly "hot" metric and enable sampling on it, or add an ignore pattern if there's a particularly hot key that isn't worth recording. For most applications, the most active recorder would probably be UserRequests. The more records you have, the more aggressively you can sample without sacrificing too much accuracy.
Determine if the server running the pulse:work command or the Redis server are consistently under high load and consider increasing their resources. You may want to run the pulse:work command on a dedicated server.
Tweak the pulse.ingest.redis.chunk configuration. By default, this is set to 1,000. The pulse:work command will continually ingest this many records from the stream until it gets fewer than this amount from the stream, and then it waits for 1 second before starting again. If you increase the value, then the work:command will use more memory but will be able to ingest more records in between each read from the stream, which should increase throughput provided you have enough memory to handle it.

Can there be multiple 'pulse:work' instances running? I wonder how this would affect database locking on rows

It's not designed to have multiple pulse:work commands running simultaneously. This would likely insert duplicates and, as you mentioned, potentially lock rows. Redis stream consumer groups would need to be used to solve the duplication issue.

Is that something Pulse might add in the future? In my application I've got a tonne of requests, events and jobs. My use case I cannot enable sampling because this doesn't accurately represent the true numbers, without another pulse worker my pulse entries are delayed, which from a business point of view makes it hard to make decisions quickly


[OOZIE-3599] Upgade to Jetty 9.4.36.

Upgrade Jetty to 9.4.36.

No longer needed due to https://issues.apache.org/jira/browse/OOZIE-3599.


[FLINK-8423] OperatorChain#pushToOperator catch block may fail with NPE

What is the purpose of the change
Fix the NPE when outputTag is null.
Brief change log
Add when outputTag is not null, then do the catch block work.
Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): ( no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): ( no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no )
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (/ no)
If yes, how is the feature documented? ( not documented)

@zentol I have corrected the code, please helps to review. Thank you very much!

merging.


Mount common JS config files into /assets/_jsconfig and add these filenames to the JS environment

This issue talks about using PostCSS with the TailwindCSS plugin in different situations in Hugo, from the main project, from the theme in /themes, from modules, from vendored modules ... But this is also a general issue which is mostly relevant to these JS tools living outside of Hugo.
I created this local "test project" to illustrate the problem for myself:
├── mod
│   ├── babel.config.js
│   ├── package-lock.json
│   ├── package.json
│   ├── postcss.config.js
│   └── tailwind.config.js
└── project
    ├── out.css
    ├── package-lock.json
    ├── package.json
    ├── run.sh
    └── styles.css

So, when running postcss from project with the configuration living in mod, I can do:
postcss  --config ../mod/postcss.config.js -o out.css styles.css;

This is in essence what Hugo does today, and it breaks in, at least, the Tailwind setup above. There are some other "gotchas" to the current situation (one being vendored modules), but I'm not going into details about those.
The main problem with the above is that it fails to find the TailwindCSS configuration. So I could do:
module.exports = {
        plugins: [ require('tailwindcss')('/Users/bep/mod/tailwind.config.js') ]
};

Which works, but how do I, running in Hugo, know where to find that file?
I have thought long and hard about this, and jumping to my conclusion, I thought we could do something ala this:
console.warn('In postcss.config.js, root.');

let tailwindConfig = process.env.HUGO_FILE_TAILWIND_CONFIG_JS || './tailwind.config.js';

module.exports = {
	plugins: [ require('tailwindcss')(tailwindConfig) ]
};

We allow mounting of single files in the root of a project/module. For the project I think we can mount the *.config.js files by default (if no other root files are mounted)
We add these filenames to the OS (and Node) environment mapping them to their absolute filename, normalized on the form tailwind.config.js => HUGO_FILE_TAILWIND_CONFIG_JS. In the example above, we could be even more specific doing:

let tailwindConfig = process.env.HUGO_MY_TAILWIND_CONFIG_JS || './tailwind.config.js';

Thoughts? Any better ideas?
/cc @regisphilibert @onedrawingperday @digitalcraftsman @jmooring @coliff  and gang.
Also see #7635 and some others ....

Question: is this using ESBuild or the normal JS pipeline. Some of this seems to work in ESBuild which is why I am asking.

@richtera Not relevant for ESBuild. Main issue I want to solve here is PostCSS/Tailwind living in a Hugo Module.
What I would have loved to solve on the ESBuild side is to allow some custom "import resolver", so Hugo could have a way to say react => '/some/path/react` etc. This is how we handle imports that live outside of the current tree in the SCSS (using LIbSass) function. That concept works pretty fantastic, where you can mount different SCSS/SASS libraries into the same tree structure.
ESBuild have some internal abstractions that would enable this, it's just a matter of convincing the author that it's a fantastic idea, and seeing how busy he has been, I have not wanted to push that idea, yet ...

I think I was able to actually get this to work. I am using two theme modules layered and I was able to put a jsconfig.json on each layer to optional search for js files in the other theme layer. Although it's a bit tricky to setup.
NOTE: Add an identical matching entry to just "react" only if "react/index.js" is not the right file to import by default.
{
  "compilerOptions": {
     "paths": {
         "react/*": ["/some/path/react/*"],
         "react": ["/some/path/react/main"]
     }
  }
}

This should work although it was not quite obvious where to put the jsconfig.json file. I put it parallel to the file I used as source for the resource.Get path. I did find that if I had the file in theme/*/assets/js and there was a assets/js folder it would no longer find the file even though there was no assets/js/index.js file. So the ESBuild will assume an index.js file exists as long as it finds the folder. But that was a minor issue for me.

Although the js.Build PR solves the resolution of js files within the js.Build transform in the pipeline, it would need to dynamically render the js from assets in order to inject the dependency into the postcss pipeline, so it's related. I am not sure how easy it would be to launch the postcss transform from the output of a js.Build pipeline.

@richtera I have modified the description of this issue somewhat, and I don't think this and ESBuild overlaps. This is about running an external program (PostCSS) and how that program resolves its configuration, there is no "Javascript building" involved.

Ok, true. I was just thinking in theory if we were to "build" the tailwinds config using js.Build and then execute the output file from postcss it would resolve the correct paths.

OK, so my initial testing of this seems to have been flawed, so I got to get back to the drawing board. These JS/Node path issues are fairly frustrating.

OK, I think I figured it out ... And that was about to ruin my Saturday evening ... Now, a glass of wine.

@richtera oh, and your PR/research around jsconfig vs ESBuild looks very promising, I will look into it in a couple of days.

Cool, thanks. I have been using it locally and if I find anything I'll amend the PR. Have a great night!

I just thought of a crazy idea. why not setup NODE_PATH and PATH environment variable inside postcss.go to be something like.
PATH=./node_modules/.bin:./themes/THEMEMOD1/node_modules/.bin:./themes/THEMEMOD2/node_modules/.bin:$PATH
NODE_PATH=./node_modules:./themes/THEMEMOD1/node_modules:./themes/THEMEMOD2/node_modules

Sounds crazy, but in theory, it should do what is required for postcss to find the main modules. You'd then only need to resolve the config file by looking up the same paths. Hmmm.

So, the "I think I got it" comment above is rooted in setting the NODE_PATH (to at least) include the main dir (e.g. the Hugo project).
One could think that one could expand that idea to the different submodules, and that would probably work if you could (or wanted) do a npm install in all of them (or they had node_modules in Git)
For the problem in the previous bullet I have made a hugo mod npm pack command that makes a top level package.json -- which should also little "in the face" what you bring in from the JS world.

... also, the NODE_PATH does not solve the primary problem in this issue: That Tailwind resolves its config file relatively to where it lives.

I like the hugo mod idea to install modules, but if those modules are pulled automatically then I think npm install should be run automatically if there is a package.json within them. It should use yarn is there is a yarn.lock or npm if there is a package-lock.json. It's a bit tricky for sure and this would cause things to be a little fragile. There would be a lot of moving parts and if there was babel going on as well, then this is a lot of things. Hmmm.

I like the hugo mod idea to install modules, but if those modules are pulled automatically then I think npm install should be run automatically

So, I'm complicating this further at this moment. What I add now works seamlessly with Netlify and other build tools. In your local setup you will currently still have to do a npm install.


Adds a zoom palette to Color My World Activity

Fixes #1593
Added a zoom palette to the Color My World Activity similar to that in the Caligra Activity.
Demo -
screen-capture (19).webm

Good job.
Miss the update of the tutorial to add explanation of the new palette.

@llaske, I have added translations for the languages that had translated tutorial strings, the languages that did not have translations for the tutorial strings, I have left them in english for now.

Nice thanks.

I have added translations for the languages that had translated tutorial strings, the languages that did not have translations for the tutorial strings, I have left them in english for now.

That's what should be done. We're expecting localization by native people instead of using Google translate.
If slightly update the string: the word "palette" is not necessarily known by users and it's better to talk about "map".


set interval to undefined in Feedback.prototype.cancel

Since clearInterval would not change value of this.interval, we should set this.interval to undefined  manually.

Could you be a little more specific about the problem here?

if someone call Feedback.prototype.cancel N times, clearInterval will be called N times. because we never set this.interval to undefined, so this.interval !== undefined will always be true.