Mirror logs into Elastic in an background like mode

What feature do you want to see added?
Having an option to send send the logs towards an ElasitcSearch backend in the background - but do not modify the way how Jenkins stores an displays the logs locally.
The problem
If we use ElasticSearch as the main logging backend, Jenkins can not be operate without Elastic. By this Elastic is becoming a critical infrastructure - and if it's offline or crashed, the whole Jenkins infrastructure will not work anymore due to logs are not visible to the user.
Therefore, having the logs duplicated would be awesome:

Staying on the Jenkins and visualized as if there was no change at all
Sending them into ElasticSearch for processing / data mining

Yes, the data would be duplicated - but this is totally fine
Upstream changes
No response
Are you interested in contributing this feature?
No response

the option already exists https://github.com/jenkinsci/opentelemetry-plugin/blob/main/docs/build-logs.md#can-build-logs-be-saved-in-the-build-folder-locally


Use manifest to see if a module should be compiled or not

Using information from manifest in #435, I think we can tell if we will need to compile modules based on if timestamps changed. This issue replaces #310

[ ] Read Manifest
[ ] Get manifest timestamp
[ ] Find removed beam files
[ ] Find changed beam files using path and File.stat
[ ] modules to be compiled will be changed - removed
[ ] Compile modules
[ ] Merge unchanged modules plus new compilation results
[ ] Write to manifest


Spartan Weaponry/Better Combat/just dual wielding in general Compatibility Issues

Just wanted to bring it to your attention (if you don't already know this) that this mod is incompatible with the Better Combat mod, and requires extra mods to even support dual wielding. Nothing too huge, I just want to dual wield dragonbone sabers.

I'm gonna need more information for this...
Which version of the mod are you using?
And which Better Combat mod are you using? There are two different mods with that name now, one for Minecraft 1.12.2 or older, and one for 1.18.2 and higher.
Both versions should have some degree of compatibility with this mod... although since the 1.12.2 version isn't being developed anymore (Better Combat Rebirth, to be clear), there are a few forks which now exist, such as RLCombat, a fork made for RLCraft, which my mod explicitly supports. Other forks aren't compatible sadly.
If you are using the newer Better Combat mod, I added a few datapack files which should add proper compatibility with it, but some features in this mod might not work properly, due to Better Combat taking over how the combat works in totality. If there are any bugs you are experiencing with this version, let me know.

If you are using the newer Better Combat mod, I added a few datapack files which should add proper compatibility with it, but some features in this mod might not work properly, due to Better Combat taking over how the combat works in totality. If there are any bugs you are experiencing with this version, let me know.

This mod supports the newer better combat? like this better combat?
Im gonna have to go test this, i didnt use this mod previously because i assumed it was incompatible due to not listing mod compatibility, ill come back if i find any issues with the mod, thanks for letting me know!


New prefabs workflow support for patcher

The patcher missed feature to edit patches inside prefabs. Since Unity 2018.3 there is a new prefabs workflow that loads prefabs into isolated preview scene which operates almost as play scene. This pull request extends patcher to edit prefabs based on the new workflow. The feuture is conditioned under UNITY_2018_3_OR_NEWER.

I've force-pushed my feature branch with the following update: When the patch is part of prefab instance it is not enough to just Undo.RecordObject() to 100% garanty it will be saved. This code impacts not only prefab editing mode but also usual scene editing mode because scene objects can be prefab instances as well and potintially can lose changes without a call to PrefabUtility.RecordPrefabInstancePropertyModifications() in the end of changes. I mean all changes should be between undo and property modifications recording calls:
Undo.RecordObject(obj, "Changes");

// Perform changes here

if (PrefabUtility.IsPartOfPrefabInstance(obj))
    PrefabUtility.RecordPrefabInstancePropertyModifications(obj);


Update pipeline.py

fixes exception:
UnboundLocalError: cannot access local variable 'attachment' where it is not associated with a value

I thought the script below considers all cases when there are no attachments - Thanks for the contribution!!
            else:
                if submission.is_self:
                    attachment = None

By any chance, do you know the submission_id of the Reddit submission that raised this error?


[Snyk] Upgrade org.apache.tinkerpop:neo4j-gremlin from 3.5.2 to 3.6.0

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade org.apache.tinkerpop:neo4j-gremlin from 3.5.2 to 3.6.0.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 2 versions ahead of your current version.
The recommended version was released 25 days ago, on 2022-04-05.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

this change was already done in https://github.com/tdf/odftoolkit/pull/170


question about weight init

Hi,
Your implementation is great and easy to read.
I just had one question though, from the line:
https://github.com/Khrylx/PyTorch-RL/blob/15b574f5d52f5eeab6917c90c17e8739578f3d96/models/mlp_policy.py#L24
Is there any particular reason why the weights are initialized like that(instead of the normal gaussian/xavier's scheme) with that specific scale?
Thanks!
Gunshi

Hi,
This is just a choice inherited from other reference implementation. I find it working well in practice, although better initialization could yield better results for some problems.
Ye

Thanks for the answer!


Windows 7 support?

I have z170 motherboard and
GTX1070
i7 6700k
24gb
and I get 30fps and I cant use this mod ....
mmh I bought this game for full price pre order :( I dont wanna buy a new ssd just to boot out of bloateddows 10

Microsoft doesn't even support Windows 7 anymore, why should anyone else? Just upgrade...


HTTP Auth for module deployed as a Google Function

Hello,
I am working on moving our extension & notification modules from k8s pods to Google Functions. I've easily deployed them there. However, I do not see a simple way to limit the access. I want only CT to be able to send calls to our Trigger URL because if anyone did it, it would generate big costs in GCP.
I've checked Google documentation and I see it might require an additional request for the auth key. I think it is not really a proper way of doing that in the current case. Do you know how we could protect Trigger URL without code modifications?

Hi @Fifciu
Thank you for your request. Unfortunately in our team we don't have any experience with protecting the Trigger URL. Maybe you can have a look at following links: https://cloud.google.com/load-balancing/docs/negs/internet-neg-concepts#authenticating_requests, https://cloud.google.com/vpc-service-controls/docs/service-perimeters.
I'm closing the ticket as we cannot help you here. Let us know if you have any comments or further questions.

Trigger URL is protected by default and disabling it could generate huge costs in GCP.
It looks like I have to send an additional request for the auth token and put it in Commercetools' extension. However, TTL of auth token in GCP is only 1h, so I would need to update it in CT every hour! That sounds ridiculous to me. That's why I am asking you for detailed instructions on deploying it as Google Function. The current one doesn't contain information about handling tokens that are crucial for Serverless functions.

Hey @Fifciu,
Thanks for the request;

I want only CT to be able to send calls to our Trigger URL because if anyone did it, it would generate big costs in GCP. Trigger URL is protected by default and disabling it could generate huge costs in GCP.

I'm not sure I got the concern related to costs (assuming an attacker who wants to load the functions need to know the trigger url in the commercetools which normally should not because the request/response should be in https so trigger url will be secured) in case with exposing an url to internet. You might check the basic auth mechanism (https://github.com/commercetools/commercetools-adyen-integration/blob/master/extension/docs/HowToRun.md#commercetools) for extension and hmac for the notification which secures the integration from insecure calls.

However, TTL of auth token in GCP is only 1h, so I would need to update it in CT every hour! That sounds ridiculous to me.

Currently, I could not find a mechanism on google functions that covers your scenario by default or sth like an api key that auth info is always same on the trigger url (as you said this might be another security issue). I guess in this case you might check a proxy server approach and do the authentication before going to google function which provides the auth based on the Cloud IAM principles. As said I don't know the best practices here as I haven't check yet but the links above that Lam shared could be useful too.
Let me know how we could help here, next step seems to me understanding the problem in detail and maybe asking it to google support for an approach.
Regards, Ahmet.

I think the most helpful would be the mechanism inside the core of commercetools. It would refresh tokens before requests. I see there is a dedicated approach for AzureFunctions and AWSLambda (https://docs.commercetools.com/tutorials/extensions#setting-up-an-api-extension). Let's add something like that for Google Fn. I know it is not a place for commercetools feature requests. But maybe you could forward it from there.
For now, we've decided to use AWSLambda for this particular task as the Extension configuration allows us to provide accessKey and Secret. And for Adyen -> notification module communication, we can write a simple custom Auth Handler in Node.js for AWSLambda.

Hi @Fifciu Is this issue still a topic or can be closed?

I think the most helpful would be the mechanism inside the core of commercetools. It would refresh tokens before requests. I see there is a dedicated approach for AzureFunctions and AWSLambda (https://docs.commercetools.com/tutorials/extensions#setting-up-an-api-extension). Let's add something like that for Google Fn. I know it is not a place for commercetools feature requests. But maybe you could forward it from there.
For now, we've decided to use AWSLambda for this particular task as the Extension configuration allows us to provide accessKey and Secret. And for Adyen -> notification module communication, we can write a simple custom Auth Handler in Node.js for AWSLambda.

I see that the requested functionality is still not available.

I think the most helpful would be the mechanism inside the core of commercetools. It would refresh tokens before requests. I see there is a dedicated approach for AzureFunctions and AWSLambda

Dedicated approaches exist only because those security features are natively supported by the given cloud providers. It is not the case for GCP functions. One could try to use for example GCP API Gateway and include the API key in the POST google function URL as described here. The generated API key according to the documentation does not expire unless manually revoked.

I know it is not a place for commercetools feature requests. But maybe you could forward it from there.

As you said the right place would be to ask for this feature commercetools through the official support.


go-ipfs 0.4.4 has been released

The "bug" and "find out if you're affected" sections still need a bit of wording polish.
cc @jbenet @whyrusleeping @RichardLitt

Rendered preview: https://ipfs.io/ipfs/QmXJTKBETezj4BeZaoTmDQQQfmEoKqkfTBnVU4taRhxQdj/21-go-ipfs-0-4-4-released/

@lgierth and @whyrusleeping: can you review my changes?

@RichardLitt I added a few more.
@whyrusleeping @Kubuxu either of you have a last look too. Btw we need to fix building ipfs-see-all from source, which is mentioned as an option in the post.
Rendered version: https://ipfs.io/ipfs/QmXarD5t3EDffn4nRtqVTcd6bJq5v5wFTErcLFxtg5MYk2/21-go-ipfs-0-4-4-released/

Ping @Kubuxu @whyrusleeping for a last fact-checking review.

@Kubuxu @whyrusleeping @RichardLitt any updates? We should be able to ship this now.

LGTM

Great. Let's go! @lgierth publish?


CMake - TEST_BIG_ENDIAN

Describe the bug
Adding
include(TestBigEndian)
TEST_BIG_ENDIAN(IS_BIG_ENDIAN)

to the CMakeLists.txtfails. It reports:

-- Check if the system is big endian
-- Searching 16 bit integer
-- Check size of unsigned short
CMake Error at /usr/share/cmake-3.23/Modules/CheckTypeSize.cmake:146 (try_compile):
Cannot copy output executable
''

to destination specified by COPY_FILE:
'zephyr/coaps-zephyr/coaps-zephyr/build/CMakeFiles/CheckTypeSize/CMAKE_SIZEOF_UNSIGNED_SHORT.bin'

Unable to find the executable at any of:
zephyr/coaps-zephyr/coaps-zephyr/build/CMakeFiles/CMakeTmp/cmTC_13b71.elf
zephyr/coaps-zephyr/coaps-zephyr/build/CMakeFiles/CMakeTmp/Debug/cmTC_13b71.elf
zephyr/coaps-zephyr/coaps-zephyr/build/CMakeFiles/CMakeTmp/Development/cmTC_13b71.elf

Call Stack (most recent call first):
/usr/share/cmake-3.23/Modules/CheckTypeSize.cmake:277 (__check_type_size_impl)
/usr/share/cmake-3.23/Modules/TestBigEndian.cmake:55 (CHECK_TYPE_SIZE)
/usr/share/cmake-3.23/Modules/TestBigEndian.cmake:37 (__TEST_BIG_ENDIAN_LEGACY_IMPL)
zephyr/coaps-zephyr/modules/lib/tinydtls/zephyr/AutoConf.cmake:49 (TEST_BIG_ENDIAN)
zephyr/coaps-zephyr/modules/lib/tinydtls/zephyr/CMakeLists.txt:27 (include)
-- Check size of unsigned int
CMake Error at /usr/share/cmake-3.23/Modules/CheckTypeSize.cmake:146 (try_compile):
Cannot copy output executable
''

to destination specified by COPY_FILE:
'zephyr/coaps-zephyr/coaps-zephyr/build/CMakeFiles/CheckTypeSize/CMAKE_SIZEOF_UNSIGNED_INT.bin'

Unable to find the executable at any of:
zephyr/coaps-zephyr/coaps-zephyr/build/CMakeFiles/CMakeTmp/cmTC_6e8ef.elf
zephyr/coaps-zephyr/coaps-zephyr/build/CMakeFiles/CMakeTmp/Debug/cmTC_6e8ef.elf
zephyr/coaps-zephyr/coaps-zephyr/build/CMakeFiles/CMakeTmp/Development/cmTC_6e8ef.elf

Call Stack (most recent call first):
/usr/share/cmake-3.23/Modules/CheckTypeSize.cmake:277 (__check_type_size_impl)
/usr/share/cmake-3.23/Modules/TestBigEndian.cmake:60 (CHECK_TYPE_SIZE)
/usr/share/cmake-3.23/Modules/TestBigEndian.cmake:37 (__TEST_BIG_ENDIAN_LEGACY_IMPL)
zephyr/coaps-zephyr/modules/lib/tinydtls/zephyr/AutoConf.cmake:49 (TEST_BIG_ENDIAN)
zephyr/coaps-zephyr/modules/lib/tinydtls/zephyr/CMakeLists.txt:27 (include)

To Reproduce
Add
include(TestBigEndian)
TEST_BIG_ENDIAN(IS_BIG_ENDIAN)

to the CMakeLists.txt. Execute

$ west build

Expected behavior
TEST_BIG_ENDIAN(IS_BIG_ENDIAN) sets IS_BIG_ENDIAN according the used processor.
Impact
IS_BIG_ENDIAN must be set manually,
Environment (please complete the following information):

Ubuntu 18.04
ncs 1.9.1

This seems to be more of a bug in the CMake itself than the Zephyr build system.
Determining if the system is big endian failed with the following output:
Change Dir: /home/stephanos/Dev/zephyrproject/zephyr/samples/hello_world/build/CMakeFiles/CMakeTmp

Run Build Command(s):/usr/bin/ninja cmTC_5e2a9 && [1/2] Building C object CMakeFiles/cmTC_5e2a9.dir/TestEndianess.c.obj
[2/2] Linking C executable cmTC_5e2a9
FAILED: cmTC_5e2a9
: && /opt/zephyr-sdk-0.14.1/arm-zephyr-eabi/bin/arm-zephyr-eabi-gcc   CMakeFiles/cmTC_5e2a9.dir/TestEndianess.c.obj -o cmTC_5e2a9   && :
/opt/zephyr-sdk-0.14.1/arm-zephyr-eabi/bin/../lib/gcc/arm-zephyr-eabi/10.3.0/../../../../arm-zephyr-eabi/bin/ld: /opt/zephyr-sdk-0.14.1/arm-zephyr-eabi/bin/../lib/gcc/arm-zephyr-eabi/10.3.0/../../../../arm-zephyr-eabi/lib/libc.a(lib_a-exit.o): in function `exit':
exit.c:(.text.exit+0x34): undefined reference to `_exit'
collect2: error: ld returned 1 exit status
ninja: build stopped: subcommand failed

The compilation of TestEndianness.c fails because the toolchain, without any "required arguments," defaults to linking some libraries that expect certain symbols (e.g. _exit in case of arm-zephyr-eabi) to be defined by the application.
Note that the CMake TEST_BIG_ENDIAN implementation calls try_compile without any arguments, causing some required flags like -nostdlib to be not passed when compiling TestEndianness.c:
     try_compile(HAVE_${VARIABLE}
      "${CMAKE_BINARY_DIR}"
      ${_test_file}
      OUTPUT_VARIABLE OUTPUT
      COPY_FILE "${CMAKE_BINARY_DIR}${CMAKE_FILES_DIRECTORY}/TestEndianess.bin" )

For instance, the compilation of CMAKE_SIZEOF_UNSIGNED_SHORT.c, also built as part of the TEST_BIG_ENDIAN function, succeeds because the CHECK_TYPE_SIZE function calls try_compile with the CMAKE_REQUIRED_FLAGS:
  try_compile(HAVE_${var} ${CMAKE_BINARY_DIR} ${src}
    COMPILE_DEFINITIONS ${CMAKE_REQUIRED_DEFINITIONS}
    LINK_OPTIONS ${CMAKE_REQUIRED_LINK_OPTIONS}
    LINK_LIBRARIES ${CMAKE_REQUIRED_LIBRARIES}
    CMAKE_FLAGS
      "-DCOMPILE_DEFINITIONS:STRING=${CMAKE_REQUIRED_FLAGS}"
...

Note that -nostdlib is added to CMAKE_REQUIRED_FLAGS here:
https://github.com/zephyrproject-rtos/zephyr/blob/04836af2a91f04d47431697ee1275ffb09e021df/cmake/compiler/gcc/target.cmake#L121
Applying the following patch to the CMake TestBigEndian.cmake makes this function run successfully:
--- /usr/share/cmake-3.22/Modules/TestBigEndian.cmake   2022-01-25 22:45:06.000000000 +0900
+++ cmaketest/Modules/TestBigEndian.cmake       2022-05-02 17:45:40.429774864 +0900
@@ -91,6 +91,9 @@
      try_compile(HAVE_${VARIABLE}
       "${CMAKE_BINARY_DIR}"
       ${_test_file}
+      CMAKE_FLAGS
+        "-DCOMPILE_DEFINITIONS:STRING=${CMAKE_REQUIRED_FLAGS}"
+        "-DINCLUDE_DIRECTORIES:STRING=${CMAKE_REQUIRED_INCLUDES}"
       OUTPUT_VARIABLE OUTPUT
       COPY_FILE "${CMAKE_BINARY_DIR}${CMAKE_FILES_DIRECTORY}/TestEndianess.bin" )

Output after applying the patch:
-- The C compiler identification is GNU 10.3.0
-- The CXX compiler identification is GNU 10.3.0
-- The ASM compiler identification is GNU
-- Found assembler: /opt/zephyr-sdk-0.14.1/arm-zephyr-eabi/bin/arm-zephyr-eabi-gcc
-- Check if the system is big endian
-- Searching 16 bit integer
-- Looking for sys/types.h
-- Looking for sys/types.h - found
-- Looking for stdint.h
-- Looking for stdint.h - found
-- Looking for stddef.h
-- Looking for stddef.h - found
-- Check size of unsigned short
-- Check size of unsigned short - done
-- Searching 16 bit integer - Using unsigned short
-- Check if the system is big endian - little endian
-- Configuring done
-- Generating done

Of course, the upstream CMake developers may argue that it is the toolchain's fault for not being able to produce an executable as-is without some special arguments (i.e. -nostdlib), which we can certainly address for the Zephyr SDK; but, other third-party GNU toolchains such as the GNU Arm Embedded also has this problem, and it is not without a good reason (i.e. the toolchain is configured such that it uses the newlib by default, which expects _exit to be defined either by the libgloss or the application).
Nevertheless, apart from the questionable toolchain issues, it is true that CMake has an inconsistency in its internal compiler checks that the CMAKE_REQUIRED_INCLUDES flags are used for some checks while they are not for the rest.
When @tejlmand (or @SebastianBoe) comes back, he can further comment on this and maybe file a bug report to the upstream CMake repository.

FMPOV, the interesting part of this issue seems to be, that the tool-chain is usually used for "cross-compiling". So I'm not sure, if that is a working solution at all, to compile & execute the test.
Maybe, it's possible to provide an other approach, where somewhere in the board definition (or target cpu), this could be provided and returned, instead of trying to do the test above.

So I'm not sure, if that is a working solution at all, to compile & execute the test.

It only compiles and does not execute the test, so it can and does (as shown above) work for cross compilers.
Note that it literally looks for a string in a compiled binary, without attempting to execute it:
https://github.com/Kitware/CMake/blob/8dbf38eaaf0d465a7b44e337204c3b763915ebb5/Modules/TestBigEndian.cmake#L99-L103

Thanks, then I was wrong, mislead by the "executable" in the error logs.

I tried to apply your patch. Doing so, I found TestBigEndian.cmake - deprecated:: 3.20

Supserseded by the :variable:CMAKE_<LANG>_BYTE_ORDER variable.

I will try that. Maybe that works without patch.

Thanks for raising this.
There are a couple of reasons why this doesn't work out of the box with Zephyr.
First part is that Zephyr doesn't rely on CMakes infrastructure for testing the compiler due to a couple of different reasons.
Secondly, even if trying to use the new endianess detecting for CMake >=3.20, then the next issue that arises is that CMake default in Zephyr context tries to compile for an executable, which fails as we haven't generated a valid linker script for the compiler.
Therefore one must set STATIC_LIBRARY as compile target type.
I have a working hack that correctly detects the endianess with no patching to CMake, as can be seen here:
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
endianess (test_big_endian)   : 0
endianess (CMAKE_C_BYTE_ORDER): LITTLE_ENDIAN

going to clean up the hack and find a proper location for the code.
So stay tuned for a upcomming PR.
This will mean that both the CMake CMAKE_C_BYTE_ORDER or the test_big_endian can be used to fetch the endianess.


Render issue with @splinetool/runtime

Hello! Thanks for the great work on this project. We recently have run into issues with newer version of @splinetool/runtime. Using versions v0.9.154 or higher causes the scene to not render at all. Previously, we have no issues rendering Spline scenes using React-Spline and Spline Runtime.
Below are reproductions of working and broken demos:
https://codesandbox.io/s/frosty-breeze-t8gtep?file=/package.json  (broken)
https://codesandbox.io/s/thirsty-gwen-fn4y9z?file=/package.json (working)
We are having to pin the NPM package to v0.9.154 or lower to prevent this from happening on our already live production app.
Node - 16.18.1
NPM - 8.19.2
React - 17.0.2
React Dom - 17.0.2

@marcofugaro  thanks for the fast response. In the example you provided (https://codesandbox.io/s/xdvjw1?file=/package.json) , the scene does not animate.
we can try a new export and URL, but trying to understand what caused the issue to begin with. we are using the Spline scene on the homepage of a production app, so ideally we would not ever need to change the export URL. nothing has been changed in the scene itself, so trying to find out what stopped working with newer runtime versions.

@marcofugaro thanks for the fast response. In the example you provided (https://codesandbox.io/s/xdvjw1?file=/package.json) , the scene does not animate.

@grahamd711 the file in the library doesn't have any animation.

we can try a new export and URL, but trying to understand what caused the issue to begin with. we are using the Spline scene on the homepage of a production app, so ideally we would not ever need to change the export URL. nothing has been changed in the scene itself, so trying to find out what stopped working with newer runtime versions.

ok, if you don't need to export again, you can just lock the @splinetool/runtime version and don't upgrade it again.

ok, understood. thanks for feedback @marcofugaro!


fix project website urls

cf. #194
replace http://swimlane.github.io/angular-data-table/ by http://swimlane-contrib.github.io/angular-data-table/

ok already done :p


Delegation fee shelley

Issue Number

#1709
Overview

[x] I have enabled delegation fee in Shelley server
[x] I have ported integration tests from jormungandr
[x] I have adjusted feeMin

Comments

@paweljakubas I don't think we can have such assertions with cardano-node and the way we do fee estimations there. What we can say however is that real fee should be at most what the minimumFee from our estimator return.

bors r+


Add ACF-Extended Fields

ACF-Extended (https://wordpress.org/plugins/acf-extended/) is a nice, large growing extension for ACF wich includes some useful new fields.
THX for your great extension! :)

Currently working on adding these. Will update in a few weeks.

@RostiMelk did they get included in a release?

@RostiMelk did they get included in a release?

Hi, please see https://github.com/RostiMelk/ACF-Tools/issues/37#issuecomment-1122246774


Adding fallbackSVG Input

Closes #165
Using this library within my company project, we had to generate the SVG file based on some data received from the backend.
In case of no corresponding SVG is present we handle it by defining the output onSVGFailed and we just switch the inlineSVG input with our fallbackSVG.
For this reason, I created this PR, in order to have the possibility to provide a fallbackSVG without handle it with the onSVGFailed output.
Have a nice day :)

Hello @arkon ,
Can you please review this one?

Thanks for the contribution!

It was a pleasure :smiley:


GH-161 - Report future engagement by location

Closes NCI-Agency/anet#161

Made a new pull request as this one contained extra commits which didn't have their place in here.


window.history.go / back fire OnNavigating handler on Android

window.history.go / back, fire OnNavigating handler on Android .
This is related on how Webextension works with the onbeforerequestevent.

Same issue here related : https://github.com/piroor/newtabfromlocationbar/issues/18#issuecomment-347074234

Some documentation : https://stackoverflow.com/questions/45139926/how-to-check-if-a-tab-has-been-reloaded-in-background-js


Custom repository is not generated

Hi,
I just had a case where I injected a custom repository into a service. There also is another injected class which also uses this repository, like this:
//repository.ts
@EntityRepository(MyEntity)
export class MyRepository extends Repository<MyEntity> {
//...custom methods
}

//service.ts
export class MyService extends TransactionFor<MyService> {
  constructor(
    private myRepository: MyRepository,
    private myOtherClass: MyOtherClass,
    moduleRef: ModuleRef
  ) {
    super(moduleRef);
  }

  //...methods using myRepository
}

//my-other-class.ts
@Injectable()
export class MyOtherClass {
  constructor(
    private myRepository: MyRepository
  )
}

When nest-transact 1.1.3 creates new repositories, it actually sees that there is a custom repository, but does not generate a new repository with the custom methods:
//line 64
argument = manager.getRepository(entity);

If this line is changed to
argument = isCustomRepository ? manager.getCustomRepository(param as any) : manager.getRepository(entity);

then it seems to actually generate a new repository with the custom method.

Hello! If you can send a pull request to fix this problem - this will be cool! I accept PR if they do work and haven't new bugs. To check this - you can run an example project - it is like a test field. And there are some custom repositories too, which works normally, as I know. Best regards!

@sarathas please, check v.1.1.5, which contains fixes for custom repositories from the community developer, if all is ok - I will close this issue

Works, thanks a lot!


Update photo on LBRY.io/team

Update the banner photo on the Team page

Needs to be updated in two places:
https://github.com/lbryio/lbry.io/blob/master/view/template/page/team.php#L6
https://github.com/lbryio/lbry.io/blob/master/view/template/page/team.php#L2 (when embedded)
Please replace with a spee.ch URL when updating

@Feanor78 / @jsigwart can you find a good team photo it can be updated with?


feat: ion-tab-bar add a clear navigation stack option

Feature Request
Ionic version:
[x] 5.x
Feature Request:
Add a clear navigation stack option in ion-tab-bar or ion-tab-button, currently there is no way to clear pages pushed in stack by an ion-router-outlet if it's used through ion-tab-bar without losing the tab bar.
Preferred Solution:
Adding a property in ion-tab-button that instruct the router outlet to clear stack, I think this could be performed.
Describe Alternatives
An option in ion-tab-bar level to instruct if always clear stack or not.
Related Code
Currently I don't know what changes this code needs to achieve this behavior. It's using the this.navCtrl.navigateRoot and it should clear the stack in every tab change, but it does not.
https://github.com/ionic-team/ionic/blob/master/angular/src/directives/navigation/ion-tabs.ts
Additional Context
a repo with simple ion tabs demostration
https://github.com/ThonyFD/ionic-tabs-stack-erro-reproduce

Thanks for the feature request. Can you please clarify the issue you are running into? There might be a workaround that can be applied to your use case. Each tab holds it own stack, and we are not currently looking to change this behavior as it closely matches the behavior on native iOS and Android.

Hi @liamdebeasi it's not a specific issue. I only want be able to clear the navigation stack when I need it.
Think in this scenario. I loaded a component in one o my tabs that import an service inside of @component decorator.
How can I clear or reset the services data?
should I to crear a method for this ever?
Sometimes we only needs to clear form the DOM from components that it's not in use, or clear memory for those service that we are no using anymore.

If you need to reset the services data, you can create a method that does so. I would caution against trying to manually manage the work the Angular Router does. If you are running into a performance issue with how the Angular Router works, I recommend filing an issue on their repo.
Using navigateRoot should clear all existing pages in the current stack. That being said, each tab holds its own stack so this may or may not work for your use case: https://github.com/ionic-team/ionic/blob/master/angular/src/providers/nav-controller.ts#L97-L114
Another option is you can use the replaceUrl option when using Angular Routing to replace states in your history: https://angular.io/api/router/NavigationExtras#replaceUrl

@liamdebeasi  My motivation for open, this request was, show you that this component could be improved, but you sound like, "we don't want to extend or improve our component functionalities."
It's clear that developers have many alternatives to do it using Angular directly, but if Ionic Framework creates a stack for each tab, why not create a way to manage them through Ionic Framework?
I love what you do with NavController, which is a very smart way to wrap an Angular capability, but the ion-tab-bar is out of this smart concept.
And for last, ion-tab-bar It's very coupled with, Angular Router, maybe like no other of your components, however, it does no expose a way to control it.
I never use ion-tab-bar, cause it's a very limited component, and fits in very few UI cases. You have the opportunity to do it in a better way.
NOTE: You did it great yesterday in Ionic Conf.

@ThonyFD the router and a lot of it's logic is dependent on angular router, so we're not about to go adding methods that go against the flow of how that router works.
Given the flow of how tabs work, clearing the stack when you tap a tab goes 100% against how tabs are supposed to work. They should not clear the stack when you navigate back to a given tab. That is the whole point of how tabs are supposed to work.


[Feat]: An Improved Movie Search Feature

Feature Description
Render the movies accrording to user input
Provide a detailed description of the feature
Currently the Search Feature is displaying results as a dropdown . An improvised Search would be Displaying movies based on user input in the search bar.
Explaination with Example:
If an user searches "dev" then the movies containing it should be rendered on the home page. i.e "Devara".
Put here any screenshots or videos (optional)
No response
By submitting this issue, you agree to

[X] I agree to follow this project's Code of Conduct
[X] I’ve read all the issues in the issue section, and this feature has not been proposed yet.

Other Information

[X] I'm a GSSoC '24 Extended contributor
[X] I'm a Hacktoberfest '24 contributor
[X] I am willing to work on this feature (optional)

@Khizarshah01 I would Like to work in this issue.

@Khizarshah01 please assign this issue so that I can complete it ASAP


Adds support for price rules

Adding support for price rules and discount codes

This looks interesting. Are there any public docs yet on what this refers to?

@jordanliddle I'll take a look tomorrow!

Is this still an active WIP?


Digital miner - "running, ready, to mine: 0", not able to mine anything.

Hello,
For me and the wife I created a private pack for us to play, however I've ran into an issue with the Digital Miner for Mekanism and I'm hoping someone can assist me into solving this issue.
I'm not able to get my digital miner to mine. It always says "running, ready, to mine: 0".
See these images: http://imgur.com/a/XRxKh
So far I haven't been able to solve this problem. As a test, I made a new instance of minecraft with just forge-1.7.10-<IP_ADDRESS>8-1.7.10-universal and mekanism. No other mods are installed and it still has the same issue  I'm running Minecraft 1.7.10 and I've tried both version <IP_ADDRESS>5 and <IP_ADDRESS>1 for Mekanism. Both seem to have the same issue.
Does anyone have any idea how I can solve this problem?

Try changing Min: on the digital miner to 0 instead of 255.

Oh my god...  Mistake of not actually reading well enough I guess.. Thanks for giving me the answer and apologies for my stupidity.

All good.    The only dumb questions are questions not asked imho.


The remote server returned an error: (403) Forbidden

Xamarin.Auth Issue
IF BUG, INCLUDE THIS PART:
Version

[ ] nuget version = <IP_ADDRESS>

Steps to reproduce

Try to do authentication with Twitter

Platform:

[] .NET version =
.NET Core
Runtime: /usr/local/share/dotnet/dotnet
Runtime Versions:
2.0.5
2.0.0
[] mono version =  <IP_ADDRESS> (2017-10/6bf3922f3fd) (64-bit)

Expected behaviour
It should open twitter login page and from there user can enter and get authorized from twitter Oauth.
Actual behaviour
It is displaying error
The remote server returned an error: (403) Forbidden

@krupanshu
Make sure your callback_url already added on https://apps.twitter.com/ > Settings

Thanks for your quick reply. It works like charm. I am closing this issue.


Create a namespaced version of Timber with Mozart

I tried to install the Plugin Slim SEO Schema that uses this namespaced version of twig which causes a conflict with Timbers Twig. Maybe the solution is to generate a namespaced version of Timber with Mozart. I think i managed to do so, but have problems with calling the class. My composer.json looks like this:
{
  "name": "aaronkessler/wordpress-theme",
  "description": "My portfolio website theme",
  "type": "wordpress-theme",
  "minimum-stability": "stable",
  "authors": [
    {
      "email"<EMAIL_ADDRESS>      "name": "jarednova"
    },
    {
      "email"<EMAIL_ADDRESS>      "name": "Aaron Kessler"
    }
  ],
  "repositories": [
    {
      "type": "composer",
      "url": "https://wpackagist.org"
    }
  ],
  "require": {
    "timber/timber": "1.*",
    "vlucas/phpdotenv": "^5.4",
    "mindkomm/timmy": "^0.14.9"
  },
  "require-dev": {
    "phpunit/phpunit": "^9",
    "coenjacobs/mozart": "^0.7.1"
  },
  "config": {
    "allow-plugins": {
      "composer/installers": true
    }
  },
  "scripts": {
    "post-install-cmd": [
      "\"vendor/bin/mozart\" compose",
      "composer dump-autoload"
    ],
    "post-update-cmd": [
      "\"vendor/bin/mozart\" compose",
      "composer dump-autoload"
    ]
  },
  "extra": {
    "mozart": {
      "dep_namespace": "Aaronkessler\\",
      "dep_directory": "/build/",
      "classmap_directory": "/build/",
      "classmap_prefix": "Aaronkessler_",
      "packages": [
        "timber/timber"
      ],
      "excluded_packages": [
        "upstatement/routes"
      ],
      "override_autoload": {
        "twig/twig": {
          "psr-4": {
            "Twig\\": "src/"
          }
        },
        "twig/cache-extension": {
          "psr-4": {
            "Twig\\": "lib/Twig/"
          }
        }
      }
    }
  }
}

inside functions.php i am calling: $timber = new Aaronkessler\Timber\Timber(); but i get an Fatal error: Uncaught Error: Class "Aaronkessler\Timber\Timber" not found

Hello @jstask82!! 👋🏻

dep_directory defines the directory the files of the package will be stored in. Note that the directory needs to correspond to the namespace being used in your autoloader and the namespace defined for the bundled packages. Best results are achieved when your projects are using the PSR-4 autoloader specification.

says the documentation, so you need
  "autoload": {
    "psr-4": {
      "Aaronkessler\\": "build/"
    }
  },

Hey @szepeviktor
Thank you very much 🐬. This worked, through the error with Slim SEO Schema is almost the same. Now it's Fatal error: Uncaught TypeError: twig_get_attribute(): Argument #1 ($env) must be of type Aaronkessler\Twig\Environment, eLightUp\Twig\Environment given, called in [...]/plugins/slim-seo-schema/vendor/elightup/twig/Twig/Environment.php(358) : eval()'d code on line 37 and defined in [...]/themes/aaronkessler/build/Twig/Extension/CoreExtension.php:1400 but I think it's up on eLightUp to fix this.

I can fix all errors. Please hire me. I pay you €1/hour for that!


Add GetDepositMethods/GetDepositAddresses

Thank you, it LGTM. I will merge and then make some minor edits

Thanks. For next time I'm happy to make those minor edits myself if you provide review feedback. That will help me learn what your preferred styles are for this crate to streamline any future PRs :)

Sure! Actually the edits wher SUPER minor.


How To Change the Browser Tab Name from "Flasgger" to Something Else?

Easy:
from flask import Flask, jsonify
from flasgger import Swagger

app = Flask(__name__)

swagger_config = Swagger.DEFAULT_CONFIG
swagger_config['title'] = 'GSP'    # Project name, Same as the Browser Tab Name.
swagger = Swagger(app)

Great. How do I also change this text on the UI?
[Terms of service]
the developer - Website
[Send email to the developer]


Develop Pelican Driver

An initial of the Pelican driver is now finished. It's currently deployed as a container on the 410 testbed.

https://github.com/SoftwareDefinedBuildings/bw2-contrib/issues/8 then we can close this

Issue fixed in https://github.com/SoftwareDefinedBuildings/bw2-contrib/pull/9


Updating cluster state and test app state scripts for slack threads, updating pipeline order to suit

Change description
Updating cluster state and test app state scripts for slack threads
Updating pipeline order to suit
Checklist

[x] commit messages are meaningful and follow good commit message guidelines
[ ] README and other documentation has been updated / added (if needed)
[ ] tests have been updated / new tests has been added (if needed)
[ ] Does this PR introduce a breaking change

example output:


mitmproxy: wsproto and pytest fixes

Motivation for this change
Pull in a few more patches to bring mitmproxy in line with nixpkgs proper until a new release is in.
Needs backport to 19.03
Things done

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

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

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

picked to 19.03 in ac2ceb01b23..d9286030a16


Update JSFiddle Hello World example link

The script tag in this fiddle points to the latest version on npm, which is v1.0.26until v2.0 becomes officially released.
So this should not be added to the docs until then.

Also, we may want to update that example when we update, as the 4-space indentation doesn't comply with the ferross/standard style we're trying to use in examples.

I'm going to close this for now - we'll make a pass to update versions used in all fiddles when we go gold.


Adapt unit tests so as not to fail when run as root

When the phpunit tests are run as root the test Log_test::test_configuration fails because it can write to the /root/ dir while the test expects that /root/ is not writeable. So fix the test so that it doesn't fail when it is run as root.

No, that part of the test should check the situation where the log path is not writeable/not a dir. We shouldn't check for the enabled property differently.
We have vfsStream and we shall use it for this. Please see the below commit.


Add HitBTC to exchanges list

Hello!
Please add a HitBTC exchange on your exchanges list on this page: https://getmonero.org/community/merchants/
Here are the links to monero markets, please use one of them:
https://hitbtc.com/exchange/XMR-to-BTC
https://hitbtc.com/exchange/XMR-to-ETH
https://hitbtc.com/exchange/XMR-to-USDT
Thank you!
You can easilly contact me via email<EMAIL_ADDRESS>

-1, HitBTC is listed as an exchange to avoid on the /r/monero scam list.

This issue has been open for 12 days and it seems the consensus is to not add HitBTC for reasons noted on the Reddit /r/Monero scam list.
@fluffypony Can we please close this issue?

A few have voiced our concerns and there's been no reply from OP. Not to mention, it is listed on the /r/Monero scams list.
Given that community members have also used 👍👎emojis to signal their opinion in this regard, I'm going to assume that the community does not wish to list this exchange at this time.
+merchant
+wontfix

As far as I know, all issues, which were on monero scam page were solved. Our support team is always opened for conversation, they even talk in specified threads, dedicated to those issues on community forums, trying to solve those issues.


Error response from daemon: rpc error: code = 4 desc = context deadline exceeded

was this issue resolved and if yes how ?

Please let us know if this fixed

This error can indicate that the manager lost quorum; which can be (e.g.) if one manager went down in a two-manager setup, but there are other situations where this can happen (such as the host running out of resources (cpu/memory)).
Without context there's not a lot to say about this error, and no context is provided in this issue. Note that the error message has improved in recent versions of docker, providing more information why the context deadline was exceeded.
I'm locking the conversation on this issue; if you run into this issue, and suspect there's a bug (i.e., no clear cause why you lost quorum, or what happened), please open a new issue with details (information requested in the issue template), logs, etc.


Fix default docker command

Precisely what the title says. A follow-up after #272. An omission of mine.

The only test failure is unrelated to this patch. Merging, then looking into it.


M09-Monitoring Application Performance with Application Insights and Azure Load Testing: Numbering and YAML broken

Contact Details
No response
What happened?
Exercise: 01
Task: 01
Step: 12
Description of issue:

Numbering is wrong, for Task 1 it counts to 7 and then starts again at 1. This is regarding step 12 which is currently labelled "5."
The formatting of the yaml snippet is broken, there's some instructional text embedded in the yaml code.
The following numbering is also broken and needs to be fixed.

Lab
Monitoring application performance with Application Insights
Relevant screenshots

Do you want to help us? 👏

[ ] Create a Pull Request by following this project's Contribution Guide

fixed with PR #539


UX issues from 4/15 review

This means it should be fixed for Summit demo , if possible.

Need a title for the tool.
Remove menu icon from the upper left because we don't have a left navigation.
Need the first entry empty state to display by default.
"Storage" section of the listview should be called "Replication repositories"
"Plans" section of the listview should be called "Migration plans"
Plans twistie needs an empty state if no plans have been created.
Create Plan (link and button in the empty state) should be disabled unless two clusters and a repository have been created.
Only one section of the accordion should be able to be opened at one time.
Remove kebabs from the cards at the top.
When a cluster/repository/migration plan is created listview should open to that section when the modal/wizard closes.
Wizard step 1 - Validate that the Name field is not empty before enabling Next button.
Wizard step 2 - Validate that a cluster and at least one project have been selected before enabling Next button.
Wizard steps 2, 3, 4 - Left align table column headers.
Wizard step 3 - All selectable PVs should default to Type = Copy.

On the latest (4/24) code it looks like the following issues have been resolved:

Need a title for the tool.

Remove menu icon from the upper left because we don't have a left navigation.

"Storage" section of the listview should be called "Replication repositories"

"Plans" section of the listview should be called "Migration plans"

Plans twistie needs an empty state if no plans have been created.

Remove kebabs from the cards at the top.

*** (Only seems to have been fixed for Plans).

Wizard step 1 - Validate that the Name field is not empty before enabling Next button.

Wizard step 3 - All selectable PVs should default to Type = Copy.

Thanks for the update Vince, that leaves:
3) Need the first entry empty state to display by default.
7) Create Plan (link and button in the empty state) should be disabled unless two clusters and a repository have been created.
8) Only one section of the accordion should be able to be opened at one time.
10) When a cluster/repository/migration plan is created listview should open to that section when the modal/wizard closes. needs fixing for cluster/storage
12) Wizard step 2 - Validate that a cluster and at least one project have been selected before enabling Next button.
13) Wizard steps 2, 3, 4 - Left align table column headers.
Meeting to discuss these today.

Added individual issues for the still outstanding issues @vconzola


Wallet API error

Hello,
i try using the API Wallet and i creqte the API key in console with domain restriction.
But the code return me an error:
[{"message":"Login Required.","domain":"global","reason":"required","location":"Authorization","locationType":"header"}]
I use the PHP library and i write this code:
$client = new Google\Client();		 $client->setApplicationName('Wallet Objects App'); $client->setDeveloperKey(GOOGLE_WALLET_API_KEY);
How i can fix this?
Thanks

Hello @pws2016 !
The code you've provided looks correct to me - could you provide the rest of your code as well so we can help you debug better?

Closing the issue due inactivity.
Please let us know if we can help you further with a code example @pws2016 :)


Feature: Python API support for Time Dependent models

We want to process existing Inversion Solution to create TD Solutions where events rates are modified according to a selected  MRE resource file.
Done when:

[ ] API use sets input inversion solution
[ ] API user sets MRE resource file
[ ] API user sets other TD related arguments . @voj you know what these are ( eg open window etc)
[ ] Run produces a new TimeDependentInversionSOlution file with modified rates
[ ] the origiginal rates are retained in archived
[ ] write some metadata about the modifications  to a file (details to come)

FY*I @voj for scale3cd solutions we added a file WARNING.md to the solution with the following content ....

@voj Perhaps we can do something simlar - just to make sure the user is aware that the data is not 'as produced' in the inversion.

merged


Force stop refreshing

Hi,
I'm implementing a feature that would display a snackbar if the connection fails.
For that to work, I noticed that the refresh icon is displayed continuously.
I want the refresh icon stop spinning, as I can't refresh again and try to fetch the data again when the connection is restored.

Hello @avishayil
Can you try to replace _updateRows by
  _updateRows(rows = [], options = {}) {
    if (rows !== null) {
      this._setRows(rows);
      if (this.props.withSections === true) {
        this.setState({
          dataSource: this.state.dataSource.cloneWithRowsAndSections(rows),
          refreshStatus: 'waiting',
          isRefreshing: false,
          paginationStatus: (options.allLoaded === true ? 'allLoaded' : 'waiting'),
        });
      } else {
        this.setState({
          dataSource: this.state.dataSource.cloneWithRows(rows),
          refreshStatus: 'waiting',
          isRefreshing: false,
          paginationStatus: (options.allLoaded === true ? 'allLoaded' : 'waiting'),
        });
      }
    } else {
      this.setState({
        refreshStatus: 'waiting',
        isRefreshing: false,
        paginationStatus: (options.allLoaded === true ? 'allLoaded' : 'waiting'),
      });
    }
  },

then do callback(null); inside your onFetch method when the connection fails

Hi,
Here's my fetch code:
  _onFetch(page = 1, callback, options) {
    // console.log(options);
    fetch(API_URL + '?page=' + page)
      .then((response) => response.json())
      .then((responseData) => {
        if (connError == 1) {
          Snackbar.show("Connection Restored!",{
            duration: Snackbar.UNTIL_CLICK,
            actionColor: '#FFCA00',
            actionLabel: 'Success'
          });
          connError = 0;
        }
        this.data = responseData.posts;
        page++;
        // console.log(page);
        // if (page === 3) {
          // callback(this.data, {
            // allLoaded: true, // the end of the list is reached
          // });
        // } else {
          callback(this.data);
        // }
      })
      .catch((error) => {
        console.log(error);
        connError = 1;
        Snackbar.show("Problem...",{
          duration: Snackbar.UNTIL_CLICK,
          actionColor: '#FFCA00',
          actionLabel: 'Error'
        });
        callback(null);
      })
    .done();
  },

It says "cannot convert undefined or null to object".

Thanks! Your patch worked!

Ok, I will publish a new version.
Is your connection fails snackbar implementable inside the GiftedListView component ? I think other people can be interested

I think so, I took the snackbar from here:
https://www.npmjs.com/package/react-native-android-snackbar
Should be very simple to implement, but some updates are required to MainActivity.java, build.gradle, and settings.gradle as the instructions say there.

Hello, I published a version with the patch


Report XSpec version at command line

It would be helpful if the version of XSpec could be report as part of the synopsis given at the command line, and probably also in the messages emitted at runtime.
(Many thanks as ever for your work on this tool, and apologies if I have missed a pre-existing issue or means to enable this.)

Agreed, it will be helpful.
I once implemented it and merged it into master (#1146), only to find later that I had to revert it due to its incompatibility with our release process.
Maybe we need some automation of updating the version number and/or releasing Oxygen add-on.


How can I change tootip style ?

On IONIC 4, how can I change the tooltip style (background-color, padding, text style, etc...) ?
I've searched but I don't find anything...
Thanks.

Hey @lagriffedigitale
As of now I can't give you an answer since I haven't tried to style it externally.
I will update the Ionic 4 implementation to have its own SASS file that uses CSS variables to make customisation easy and more aligned with the current styling methods for Ionic 4.
Please keep in mind the current Ionic 4 version is experimental and might change soon. I've discovered a few issues with it due to usage of shadow DOM in latest Ionic. Hopefully I will find a solution to that soon.

Add this to global.css
tooltip-box {
background-color: var(--ion-color-primary) !important;
color: var(--ion-color-primary-contrast) !important;
border-radius: 5px !important;
}
But i can't style the arrow.. please if u find how post here...

@naolembro the arrow would be tooltip-box:after. See here: https://github.com/zyra/ionic-tooltips/blob/master/projects/ionic-tooltips/src/lib/tooltip-box.component.ts#L46

hey guys
If you are using Ionic 3, you can to add a style:
.tooltip {
border-radius: 4px;
border: none !important;
background: YOUR-COLOR !important;
color: #fff;
font-size: 1.2rem;
}
maybe globally or in you local.

hey guys
If you are using Ionic 3, you can to add a style:
.tooltip {
border-radius: 4px;
border: none !important;
background: YOUR-COLOR !important;
color: #fff;
font-size: 1.2rem;
}
maybe globally or in you local.

this doesn't work for me :(

we can modify arrow style also as following
.has-arrow.arrow-bottom:before {
border-top: 5px solid blueviolet !important;
}

Add this to global.css
tooltip-box {
	background-color: var(--ion-color-primary) !important;
	color: var(--ion-color-primary-contrast) !important;
	border-radius: 5px !important;
}

But i can't style the arrow.. please if u find how post here...

.has-arrow.arrow-bottom:before {
border-top: 5px solid blueviolet !important;
}


Negated value is not recognized as a return statement

fn run() {
    let n = 5;
    -n
}

print(run())

EXCEPTION: Unexpected '-' (line 3, position 5)

Again, this is a bug.  Will fix.

Closing this for now.  The fix will appear in the next release.


PDBs are not working

It looks like symbols we publish are broken. See https://github.com/Microsoft/ApplicationInsights-Home/issues/152 Potentially they are broken because of GitLink

We fixed this as part of the SDL review.


Update Step Functions service endpoint from sfn to states

When importing the SFNClient from @aws-sdk/client-sfn, the runtime will throw an error: client error (Connect).
This was occurring because the addAwsSdkInitTask function in the bootstrap was trying to connect to
https://sfn.<region>.amazonaws.com rather than https://states.<region>.amazonaws.com.
This change updates the service endpoint subdomain from sfn to states.

Thanks for the PR 👏


Cant Access Admin Panel (No API key provided)

Describe the bug
I can't access the Django Admin Panel because it is being locked saying that I need an API Key. This error happens when the drf_simple_apikey.analytics package is added. Is there a way to access the admin panel without an API key?
To Reproduce
Steps to reproduce the behavior:

Fresly installed the library
Try to access Admin Panel

Screenshots
If applicable, add screenshots to help explain your problem.

Is this a bug or is it the desired behavior?

I believe this might be a bug. Let me give a quick look at this.

Any news on this or what may be triggering the issue?


No audio cue when Terminal Command Failed set to on

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.76.0
OS Version: Windows_NT x64 10.0.19045

Steps to Reproduce:

Go to settings, set Audio Cues: Terminal Command Failed to on instead of auto.
(optional) restart VS Code. (I don't think this is necessary but I thought I'd try it anyway when it didn't work immediately).
Open terminal (CMD) and type ping a.xf.wef.wef.f (or any other similarly invalid domain)

No audio cue is heard.
Note that I also tried ping a.xf.wef.wef.f || echo FAILED to ensure that ping does return a non-zero result (it echo'd "FAILED" to the console).
Maybe I'm missing something else?
[EDIT]
Forgot to mention that I did go to Help: List Audio Cues in the command palette and could hear the audio cues when I navigated between them. So it's not an issue with my speakers being muted or anything like that.

I'm not sure why this wouldn't work for you as it's working for me. You should hear the error in line audio cue - are you able to hear that in the List Audio Cues command quick pick?

Yep, in the List Audio cues I can hear it. I don't have any screenreader attached. I assume you don't either in your testing.. Weird.. I'll try on another machine.

Confirmed not working on another computer with 1.76.0 installed:
Version: 1.76.0 (user setup)
Commit: 92da9481c0904c6adfe372c12da3b7748d74bdcb
Date: 2023-03-01T10:22:44.506Z
Electron: 19.1.11
Chromium: 102.0.5005.196
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19044
Sandboxed: Yes

Strange. If you hover the command decoration, does it say the exit code?

Sorry @meganrogge what is the "command decoration"?

Hey @meganrogge so I decided to try the powershell terminal today, and it worked there!
Is it possible that the cmd terminal is simply not supported yet? In your testing were you maybe running the powershell terminal?

Yes we don't have shell integration support for command prompt


Update to fo-dicom v4.0.0

Expected Behavior
DICOMcloud uses latest version of fo-dicom v4.0.0
Actual Behavior
Currently it is still on version v<IP_ADDRESS>

I think this is becoming important, in fact the 4.0.2 version contains some breaking changes so would be nice to update to the latest version.

I have forked and will make a pull request soon.  Many warnings to fix but following the recommendations in the obsolete method makes the fixes quick.  I have fixed the warnings witht the name changes of certain methods.  Therare a few obesolete messages associated with the updateing of image data.  These require a bit more care to fix, but the answer over here https://stackoverflow.com/questions/53096584/how-to-change-the-pixel-data-of-dicom-file-using-fo-dicom
shows the solution.

@shelbypereira you are welcome to submit a PR and it is very much appreciated! I meant to do this for a while now but unfortunately didn't have the time.
I can help with testing and if specific questions comes up in fo-dicom I'll be happy to answer to my knowledge.
Thank you!

Great thanks for your feedback.  I have 3 more warnings to fix,  related to the image array.  I just wanted to highlight one change where there was an ambiguitiy In the file JpegMediaWriter.cs line 52:
  -      var bitmap = dicomImage.RenderImage (frameIndex).AsBitmap();
   +     var bitmap = dicomImage.RenderImage (frameIndex).AsSharedBitmap();
        var stream = new MemoryStream ( ) ;

        bitmap.Save (stream, System.Drawing.Imaging.ImageFormat.Jpeg );

I used AsSharedBitmap here (another choice was AsClonedBitmap)  As the image is being saved immediately and the function returns after the same,  I think the clone is not needed here and may be slow for large images.  However I mam not sure whether the original AsBitmap makes a clone or not.
All other changes up to now had noambiguity there was only one logical fix to remove the obsolete warning.

@shelbypereira AsSharedBitmap seems to be the right choice here. However not sure if there are any complications of using it over the AsClonedBitmap. I'll try to do some digging.

I have been testing more.  I installed DICOMCloud/DICOMWebjs  and I see that exceptions are being thrown (not sure they were there before the migration,  but I am assuming no.  I any case to prove things are working I think at least the following tests are needed:
1 test for each of the supported web apis (QidoRS, StowRS, WadoRS, WadoUri)
2. 1 test with an 8 bit image, another with a 16 bit image.  to confirm VR is ok, but  I suspect it works as expected based on feedback from the fo-dicom team.

@shelbypereira I tried your update and I see the error when you query. I remember hitting this same error when I first tried to upgrade but didn't figure it out at the time.
The query initializes a default empty dataset as a template.
studyDs.Add<object>(fo.DicomTag.SpecificCharacterSet,null) ;
in v4 I get an exception when trying something similar:
studyDs.Add<object>(fo.DicomTag.SpecificCharacterSet,(object)null) ;
with the exception message:

InvalidOperationException: Unable to create DICOM element of type CS with values of type System.Object

I posted this to the fo-dicom group and hope to get some help.

@shelbypereira I think this commit here fixes our problem, please check and let me know if you find other issues:
https://github.com/DICOMcloud/DICOMcloud/commit/edf4ae9ef519b9cf63d34785c1e4444d1bbc5d12

This is now merged in development


dotnetfx Package not installable: 'Install-DotNetFramework' is not recognized as the name of a cmdlet

On Windows Server 2022 I can't upgrade from <IP_ADDRESS>90930 to <IP_ADDRESS>20524. I get the message 'Install-DotNetFramework' is not recognized as the name of a cmdlet.
Help is appreciated! :)
Terminal:
You have dotnetfx v<IP_ADDRESS>90930 installed. Version <IP_ADDRESS>20524 is available based on your source(s).
Progress: Downloading dotnetfx <IP_ADDRESS>20524... 100%
dotnetfx v<IP_ADDRESS>20524
dotnetfx package files upgrade completed. Performing other installation steps.
ERROR: The term 'Install-DotNetFramework' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again.
The upgrade of dotnetfx was NOT successful.
Error while running 'C:\ProgramData\chocolatey\lib\dotnetfx\tools\ChocolateyInstall.ps1'.
See log for details.

The dotnetfx package depends on the chocolatey-dotnetfx.extension to provide this helper function. I'm guessing the extension package did not install successfully. Normally it should be done automatically by choco. Unless - did you by any chance use the --ignore-dependencies switch when calling choco?

Hi! Actually the chocolatey-dotnetfx.extension was installed but somehow broken – reinstalling it did the trick. In my case it was necessary to first uninstall chocolatygui and dotnetfx because they depend on it. Thanks for pointing me to the right direction.


Prevent a user from creating a new account if a user is logged in

Users can currently create an account even when logged in.

This was closed in e27266d3ef94f88c98a61ebaf74497f8c2aef058


batch_max_steps and batch_max_steps_valid in melgan and multiband_melgan

I had tried training multiband_melgan in my own dataset and found that batch_max_steps_valid is 10 times than batch_max_steps, but why?
https://github.com/TensorSpeech/TensorflowTTS/blob/db692cbbee3fcfa257bcaece6298905ef3fe25d9/examples/multiband_melgan/conf/multiband_melgan.v1.yaml#L69-#L70
And I also found that batch_max_steps is used to filter some short audios, a little puzzled
https://github.com/TensorSpeech/TensorflowTTS/blob/21a8f3687df27a89ee993e59eaa16ae872a5e529/examples/melgan/train_melgan.py#L520-#L523
https://github.com/TensorSpeech/TensorflowTTS/blob/21a8f3687df27a89ee993e59eaa16ae872a5e529/examples/melgan/audio_mel_dataset.py#L74-#L87

@ZhaoZeqing hi, in training we use batch_max_steps=8192 ->> len mel = (8192//256) = 32 frames. This audio segment is around 0.3-0.5s, but on inference, we want to hear longer audio to check the performance so we use 81920 ~ 4s audio :D. We also remove all short audio <= 32 frames (assumed it's noise audio :D).

@ZhaoZeqing hi, in training we use batch_max_steps=8192 ->> len mel = (8192//256) = 32 frames. This audio segment is around 0.3-0.5s, but on inference, we want to hear longer audio to check the performance so we use 81920 ~ 4s audio :D. We also remove all short audio <= 32 frames (assumed it's noise audio :D).

@dathudeptrai Thanks for your prompt reply! The shortest audio in my valid datasets has 111 frams, when I set batch_max_steps->> 32 frams, batch_max_steps_valid ->>100 frams, this error occurred:
2020-07-09 14:43:58,812 (train_melgan:221) INFO: (Steps: 5) Start evaluation.
Traceback (most recent call last):
File "anaconda3/lib/python3.7/site-packages/tensorflow/python/eager/context.py", line 1986, in execution_mode
yield
File "anaconda3/lib/python3.7/site-packages/tensorflow/python/data/ops/iterator_ops.py", line 655, in _next_internal
output_shapes=self._flat_output_shapes)
File "anaconda3/lib/python3.7/site-packages/tensorflow/python/ops/gen_dataset_ops.py", line 2363, in iterator_get_next
_ops.raise_from_not_ok_status(e, name)
File "anaconda3/lib/python3.7/site-packages/tensorflow/python/framework/ops.py", line 6653, in raise_from_not_ok_status
six.raise_from(core._status_to_exception(e.code, message), None)
File "", line 3, in raise_from
tensorflow.python.framework.errors_impl.InvalidArgumentError: Paddings must be non-negative: 0 -49800
[[{{node cond_4/else/_38/Pad}}]] [Op:IteratorGetNext]
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "examples/multiband_melgan/train_multiband_melgan.py", line 548, in
main()
File "examples/multiband_melgan/train_multiband_melgan.py", line 540, in main
resume=args.resume,
File "./examples/melgan/train_melgan.py", line 377, in fit
self.run()
File "lib/python3.7/site-packages/tensorflow_tts/trainers/base_trainer.py", line 67, in run
self._train_epoch()
File "anaconda3/lib/python3.7/site-packages/tensorflow_tts/trainers/base_trainer.py", line 93, in _train_epoch
self._check_eval_interval()
File "anaconda3/lib/python3.7/site-packages/tensorflow_tts/trainers/base_trainer.py", line 135, in _check_eval_interval
self._eval_epoch()
File "./examples/melgan/train_melgan.py", line 225, in _eval_epoch
self.eval_data_loader, 1
File "anaconda3/lib/python3.7/site-packages/tensorflow/python/data/ops/iterator_ops.py", line 631, in next
return self.next()
File "anaconda3/lib/python3.7/site-packages/tensorflow/python/data/ops/iterator_ops.py", line 670, in next
return self._next_internal()
File "anaconda3/lib/python3.7/site-packages/tensorflow/python/data/ops/iterator_ops.py", line 661, in _next_internal
return structure.from_compatible_tensor_list(self._element_spec, ret)
File "anaconda3/lib/python3.7/contextlib.py", line 130, in exit
self.gen.throw(type, value, traceback)
File "anaconda3/lib/python3.7/site-packages/tensorflow/python/eager/context.py", line 1989, in execution_mode
executor_new.wait()
File "anaconda3/lib/python3.7/site-packages/tensorflow/python/eager/executor.py", line 67, in wait
pywrap_tfe.TFE_ExecutorWaitForAllPendingNodes(self._handle)
tensorflow.python.framework.errors_impl.InvalidArgumentError: Paddings must be non-negative: 0 -49800

@ZhaoZeqing is everything fine with batch_max_step_valid = 81920 ?

@ZhaoZeqing is everything fine with batch_max_step_valid = 81920 ?

Our sample rate is 48k, hop_size=600, when batch_max_step_valid=batch_max_steps=19200, everything is ok, but when batch_max_steps=19200, batch_max_step_valid=192000 or 60000, the error occurred

@ZhaoZeqing check  this https://github.com/TensorSpeech/TensorflowTTS/blob/master/examples/multiband_melgan/train_multiband_melgan.py#L465-L471. Maybe because we forgot pass hop_size parameter into the function so the hop_size is still 256

@ZhaoZeqing check this https://github.com/TensorSpeech/TensorflowTTS/blob/master/examples/multiband_melgan/train_multiband_melgan.py#L465-L471. Maybe because we forgot pass hop_size parameter into the function so the hop_size is still 256

Oh!!! I noticed this problem, but modified in melgan/train_melgan.py by mistake, many thanks!
But after I modified multiband_melgan/train_meltiband_melgan.py, a new error occurred:
Traceback (most recent call last):
File "examples/multiband_melgan/train_multiband_melgan.py", line 544, in  main()
File "examples/multiband_melgan/train_multiband_melgan.py", line 536, in main resume=args.resume,
File "./examples/melgan/train_melgan.py", line 377, in fit self.run()
File "anaconda3/lib/python3.7/site-packages/tensorflow_tts/trainers/base_trainer.py", line 67, in run self._train_epoch()
File "anaconda3/lib/python3.7/site-packages/tensorflow_tts/trainers/base_trainer.py", line 89, in _train_epoch self._train_step(batch)
File "examples/multiband_melgan/train_multiband_melgan.py", line 110, in _train_step y, y_hat = self._one_step_generator(y, mels)
File "anaconda3/lib/python3.7/site-packages/tensorflow/python/eager/def_function.py", line 580, in call result = self._call(*args, **kwds)
File "anaconda3/lib/python3.7/site-packages/tensorflow/python/eager/def_function.py", line 627, in _call  self._initialize(args, kwds, add_initializers_to=initializers)
File "anaconda3/lib/python3.7/site-packages/tensorflow/python/eager/def_function.py", line 506, in _initialize *args, **kwds))
File "anaconda3/lib/python3.7/site-packages/tensorflow/python/eager/function.py", line 2446, in _get_concrete_function_internal_garbage_collected graph_function, _, _ = self._maybe_define_function(args, kwargs)
File "anaconda3/lib/python3.7/site-packages/tensorflow/python/eager/function.py", line 2777, in _maybe_define_function graph_function = self._create_graph_function(args, kwargs)
File "anaconda3/lib/python3.7/site-packages/tensorflow/python/eager/function.py", line 2667, in _create_graph_function capture_by_value=self._capture_by_value),
File "anaconda3/lib/python3.7/site-packages/tensorflow/python/framework/func_graph.py", line 981, in func_graph_from_py_func func_outputs = python_func(*func_args, **func_kwargs)
File "anaconda3/lib/python3.7/site-packages/tensorflow/python/eager/def_function.py", line 441, in wrapped_fn return weak_wrapped_fn().wrapped(*args, **kwds)
File "anaconda3/lib/python3.7/site-packages/tensorflow/python/eager/function.py", line 3299, in bound_method_wrapper  return wrapped_fn(*args, **kwargs)
File "anaconda3/lib/python3.7/site-packages/tensorflow/python/framework/func_graph.py", line 968, in wrapper raise e.ag_error_metadata.to_exception(e)
ValueError: in user code:
examples/multiband_melgan/train_multiband_melgan.py:139 _one_step_generator  * sub_sc_loss, sub_mag_loss = self.sub_band_stft_loss(y_mb, y_mb_hat)
anaconda3/lib/python3.7/site-packages/tensorflow_tts/losses/stft.py:142 call  * sc_l, mag_l = f(y, x)
anaconda3/lib/python3.7/site-packages/tensorflow_tts/losses/stft.py:101 call  * sc_loss = self.spectral_convergenge_loss(y_mag, x_mag)
anaconda3/lib/python3.7/site-packages/tensorflow_tts/losses/stft.py:36 call  * y_mag, ord="fro", axis=(-2, -1)

anaconda3/lib/python3.7/site-packages/tensorflow/python/ops/math_ops.py:984 binary_op_wrapper return func(x, y, name=name)
anaconda3/lib/python3.7/site-packages/tensorflow/python/ops/gen_math_ops.py:10103 sub "Sub", x=x, y=y, name=name)
anaconda3/lib/python3.7/site-packages/tensorflow/python/framework/op_def_library.py:744 _apply_op_helper attrs=attr_protos, op_def=op_def)
anaconda3/lib/python3.7/site-packages/tensorflow/python/framework/func_graph.py:595 _create_op_internal compute_device)
anaconda3/lib/python3.7/site-packages/tensorflow/python/framework/ops.py:3327 _create_op_internal op_def=op_def)
anaconda3/lib/python3.7/site-packages/tensorflow/python/framework/ops.py:1817 init control_input_ops, op_def)
anaconda3/lib/python3.7/site-packages/tensorflow/python/framework/ops.py:1657 _create_c_op raise ValueError(str(e))
ValueError: Dimensions must be equal, but are 156 and 64 for '{{node tf_multi_resolution_stft/tfstft/tf_spectral_convergence/sub}} = Sub[T=DT_FLOAT](tf_multi_resolution_stft/tfstft/Sqrt_1, tf_multi_resolution_stft/tfstft/Sqrt)' with input shapes: [256,156,193], [256,64,193].

@ZhaoZeqing weird, it should be the same shape. Can you make the fork of this repo and update what you changed then i can debug ?. It seem that len(audio_gts) != len(audio_preds). Thanks

@ZhaoZeqing weird, it should be the same shape. Can you make the fork of this repo and update what you changed then i can debug ?. It seem that len(audio_gts) != len(audio_preds). Thanks

@dathudeptrai I forked this repo and update my code on this branch: https://github.com/ZhaoZeqing/TensorflowTTS/tree/sr_48k
Many thanks!

@ZhaoZeqing https://github.com/ZhaoZeqing/TensorflowTTS/blob/sr_48k/examples/multiband_melgan/conf/multiband_melgan.v1.yaml#L22 here, the upsamples scales is wrong. we set it to [8, 4, 2] because 8 * 4 * 2 * (n_bands) = 8 * 4 * 2 * 4 = 256 (hop_size). Also in https://github.com/ZhaoZeqing/TensorflowTTS/blob/sr_48k/examples/multiband_melgan/conf/multiband_melgan.v1.yaml#L40 is the same logic as above :D. 4 * 4 * 4 * 4 = 256

@ZhaoZeqing https://github.com/ZhaoZeqing/TensorflowTTS/blob/sr_48k/examples/multiband_melgan/conf/multiband_melgan.v1.yaml#L22 here, the upsamples scales is wrong. we set it to [8, 4, 2] because 8 * 4 * 2 * (n_bands) = 8 * 4 * 2 * 4 = 256 (hop_size). Also in https://github.com/ZhaoZeqing/TensorflowTTS/blob/sr_48k/examples/multiband_melgan/conf/multiband_melgan.v1.yaml#L40 is the same logic as above :D. 4 * 4 * 4 * 4 = 256. BTW, seem 48k is too large, i recommend you sampling it into 24k :))) and use hop_size = 300 :D or sampling it into 22050 hz as ljspeech :'(

@dathudeptrai Thank you very mucn! I will try 24k :)

@ZhaoZeqing maybe you want follow https://github.com/TensorSpeech/TensorflowTTS/pull/105. He finished training mb-melgan for chinese dataset (24k) :))

@ZhaoZeqing maybe you want follow #105. He finished training mb-melgan for chinese dataset (24k) :))

@dathudeptrai Cool~ :D


Enable Graph Explorer to support sovereign/national clouds

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.
AB#8314

Related to #614


Move Datacenter to a constraint instead of a label

Datacenter is currently implemented by setting a label with the stack file, this should be moved to a constraint.

@nicholasjackson Mind if I try taking this one. It shouldn't be a hard change.

@nicholasjackson My thought here is to create a constraints parsing function. Right now its only job will be to iterate over the string slice and find the datacenters attribute. It will take that attribute and pass it to the current createDatabase function. We can then have a starting place for issue #18 to build out the parsing logic.
I am still a little slow on the type system in Go but want to get something out there by EOW.


global shader cache file missing

CARLA version: 0.9.12
Platform/OS: Ubuntu 18.04.5
Problem you have experienced:
When executing launch script (CarlaUE4.sh), it gives error that global shader cache file (GlobalShaderCache-GLSL_150_ES31.bin) is missing.
The terminal output:

4.26.2-0+++UE4+Release-4.26 522 0
Disabling core dumps.

What you expected to happen: Town launches in spectator view
Steps to reproduce:

Download and extract CARLA 0.9.12 release (CARLA_0.9.12.tar.gz)
Download assets (AdditionalMaps_0.9.12.tar.gz) to Import directory
Run ImportAssets.sh script
Install client library by making Python virtual environment and running pip install carla
Run CarlaUE4.sh

Other information (documentation you consulted, workarounds you tried):
I am using Quadro RTX 4000 graphics card
The Engine directory contains this file: GlobalShaderCache-SF_VULKAN_SM5.bin

try sudo apt install vulkan-utils

try sudo apt install vulkan-utils

That fixed the issue, thanks!


Laravel 10.37.3 Shift

This pull request includes updates for the recent minor version release of Laravel as well as updates to your package dependencies. Feel free to commit any additional changes to the shift-ci-v10.37.3 branch.
Before merging, you need to:

Checkout the shift-ci-v10.37.3 branch
Review all pull request comments for additional changes
Run composer update
Thoroughly test your application (no tests?, no CI?)

:x: Shift was unable to run composer update. Please be sure to run composer update before merging to ensure your dependency changes are fully compatible and reflected in your composer.lock file.


feat: add cursor includeCount option

Hi !
This is my first contribution to a TypeScript project, let me know if there is anything wrong.
This PR adds the ability to opt-out for count in cursor pagination.
In the tests I wanted to add an assertion to make sure that "find" is called and "findAndCount" is not called, but I could not find any equivalent to $mock->expects($this->never()->method('findAndCount'). Is there a way to do that easily with mocks in JS?
PS: I don't really know why the lock files are part of the PR, I did not update any dependencies:/

This should be good!

I don't really understand the tests failure. When I try locally I indeed have some errors with mysql and mongo (most likely because I don't have them installed on my machine?), but SQLite works properly. However it seems that this may due to the tests not running locally, that the snapshot are not properly generated.
What is the best way to solve such issues?

Having the tests pass locally is the very first step. Have you followed the contributing guide? It should be just about docker compose up nowadays.

After doing the compose up, all the tests are now passing locally! Thanks, this PR should be good to go :)


.main-wrap minxin question..

I don't think this is a bug, not sure, but little things like this drive me nuts and I am wondering if anybody else has run into this....  For the full-width template shouldn't the output of the  mixin FP uses equal the same as the below?
However, there are extra margins on our usage as a mixin on the grid. I tried to add grid-margin-x , but the same.

cell

Looks like by default the xy-cell mixin has margin gutters. That's kind of confusing since when not using the mixin you have to add the grid-margin-x class in order to get margins. I guess it has to be setup this way in order to allow for the option to have padding or margin gutters.
/// Creates a cell for your grid.
///
/// @param {Keyword|Number} $size [full] - The size of your cell. Can be `full` (default) for 100% width, `auto` to use up available space and `shrink` to use up only required space.
/// @param {Boolean} $gutter-output [true] - Whether or not to output gutters
/// @param {Number|Map} $gutters [$grid-margin-gutters] - Map or single value for gutters.
/// @param {Keyword} $gutter-type [margin] - Map or single value for gutters.
/// @param {List} $gutter-position [right left] - The position to apply gutters to. Accepts `top`, `bottom`, `left`, `right` in any combination.
/// @param {String} $breakpoint [null] - The name of the breakpoint size in your gutters map to get the size from. If using with the `breakpoint()` mixin this will be set automatically unless manually entered.
/// @param {Boolean} $vertical [false] - Set to true to output vertical (height) styles rather than widths.
@mixin xy-cell(
  $size: full,
  $gutter-output: true,
  $gutters: $grid-margin-gutters,
  $gutter-type: margin,
  $gutter-position: right left,
  $breakpoint: null,
  $vertical: false
)

In FP, even if you set $gutter-output to false for .full-width, you'll still get gutters (larger ones) because it falls back to the margins from the non full-width layout because both templates use the .main-content class.
We should probably just have a .main-content-full-width class and a .main-content class instead of putting a .full width class on the .main-wrap to prevent overrides.

@colin-marshall Thanks. At least I know it's not just my tired eyes. Little stuff like this drives me nuts and I end up spending a lot of time finding out the "whys." I actually tried to use "grid-margin-x" and it didn't do anything. I'll need to look at a solution for this or it will continue to drive me crazy as I do like to write grid inline sometimes.

@JPOak what about the solution I proposed in the last sentence of my previous post? If you want to get rid of the margins just do what I suggested and change the mixin in .main-content-full-width to:
@include xy-cell(12, false);

@colin-marshall Ah, that would work. Thanks!

@JPOak no problem. I'm reopening this until I can put a PR in to change the class for full-width.


Creation shouldn't start instance

If there are no instances, I can create one and the creation takes some time before it returns because it automatically starts as well. If I stop the instance, it is 'off' and I can delete it.
In all other providers you can create an instance and it exists in an off/stopped state, then it is a separate action to start it. This state must exist for OpenShift Local (since it's there if I stop), and for consistency I think OpenShift Local should do the same - creating the instance should prime the preset but not start yet.

:+1 I agree...
this is especially one that helps with OpenShift as preset.

I'm not sure how to achieve that, on crc cli there are no create function.
I could detect that there are not VM and allow to 'create' new(just start), so if create will be fully on extension side, it is impossible to determine that connection was 'created' or not. As that 'created' state is not stored on crc cli.

It could be completely handled by the extension... or, we have an issue from way back to details this issue for CRC. We might be able to pick this up from the discussion around the preset-config.

We could do that in extension, but we don’t have a way to store 'created' connection, so if you create a connection, then restart podman-desktop(with out starting crc), your connection will be lost, and you need to create another one.
The problem is that we unable to differentiate crc state with no VM (no connection) and 'created' connection(VM) from extension, as that create will happen only after crc start.
So we couldn’t solve properly this issue without changing crc behaviour(new command crc create?)


Add Doxygen check

Description
This PR adds check for Doxygen (to check whether the code is properly commented or not).
Few changes in this PR:

Update .github/workflows/ci.yml file to add the check
Add the config files necessary to run the check in the docs  directory.
Modify the source files to adhere to doxygen standards.

Test Steps

Related Issue

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Closing this as a new comprehensive PR shall be opened soon.


Fake initial players

If there isn't an opponent, create a fake one instead of waiting. This'll fix the issue where once the server restarts it can take awhile to find opponents

Compromised by showing them the more recent player on the previous turn


Lack of disk space on default AWS Batch ComputeEnvironment

Running training jobs with AWS Batch often will result in a lack of disk space as the default EBS volume size is quite low (8 GB). Would it be possible to add an input parameter to increase the root EBS volume of the EC2 instances ? I have tested it out and it requires adding an EC2 Launch Template to the ComputeEnvironment setup. Happy to create a PR if required.

Hey @mattmcclean ,
I think this is a really good idea! We ended up experiencing this issue pretty early on as well and solved this in our infrastructure by creating a new launch template that dictates a 100 GB disk. While our solution is in Terraform, you might find it useful as a guide if you wanted to do this yourself in Cloud Formation. link to the specific code


How to use transparency correctly maya2gltf

Hi all! I need assistance with using the transparency feature in the glsl shader in Maya from the maya2gltf plugin.
I want to create a glass with a logo on it so I have the glsl shader in transparency mode. I have placed the jpeg texture in the base color slot then have the base color factor to white and alpha to 0. In maya preview it looks fine with a transparent glass and the logo on it. The issue is now that when I test it in glTF viewer the the glass has disappeared. I have a suspicion it's the alpha but when I plug in a transparent PNG with an alpha of 1 everything is white. What am I missing in the process?
Thanks!

Could you try exporting one of the transparent scenes in this repository, and see if that works for you?

Apologies I am not familiar with what you are asking. I am not a coder. I just use your plugin/addon to export to glTF/glb. I can up load my files if that helps?

Sorry, I forgot about this issue.
What happens if you load and expert the transparent scenes in
https://github.com/iimachines/Maya2glTF/tree/main/maya/scenes

Hi Ziriax
I have tried what you have requested and it looks like the transparent files work. See attached.
So what am I doing incorrectly?
The logo on the glass needs to be white. But from your examples. White is the alpha. See attached for my example.

I had a closer look at the PNG file with the blue circle. I think I know what what to do but I would like to hear your info of it thanks.

Hard to tell just looking at the output.
Can you share your Maya scene with me?

Yes I can.

I am not sure how to share it with you but here is a wetransfer links.
https://we.tl/t-oG6fDJqXhw

Sorry, I missed this issue, I was on vacation in august.
The link is expired, can you upload this again, if still needed?

I see. I was wondering why there was no response and I didn't want push.
I have figured out some of the transparency issues. I made some car lights work but I don't know how to have a label on a glass container work.
In your old versions of the plugin when I put alpha to 0 and the color to black the export shows a transparent object with full reflection on it gltf viewer etc.... Now in your new plugin when I apply the same settings the object is completely gone.
Here is my file.
Thank you.
Breville_Glass.zip

You can try to abuse the emissive texture I guess, so that the color of the text is added to the transparent bottle. I am not sure if this will work, but I don't think anything is wrong with the exporter in this regard, the output looks correct at first sight.

No that doesn't seem to work... I'm afraid that your mesh can't work without OIT (order independent transparency). This has nothing to do with Maya2glTF, but the render engine.
Best to ask in the general GLTF rendering forums or rendering engine specific forums for how to this with GLTF and the engine.
PS: I couldn't get it any better than this, using the Babylon viewer:

See also
https://doc.babylonjs.com/features/featuresDeepDive/materials/advanced/transparent_rendering
https://doc.babylonjs.com/features/featuresDeepDive/materials/advanced/transparent_rendering#order-independent-transparency

My apologies for such a late reply. Thank you. I will have a look at the links provide. I really appreciate your help.


Is it possible to use train_on_responses_only with the Mistral template?

train_on_responses_only expects instruction_part and response_part, which seems to not work with the Mistral chat template.
Whenever I try some kind of [INST] combination, the spaced decode is always just a new-line for me. Perhaps I'm doing it wrong.
Is it possible to train on responses only with the Mistral chat template? If so, could you kindly provide a example? :)

Try using the [/INST] as the response part, that is what I am currently doing. Although I am not sure if this would work for multi-turn chats.
trainer = train_on_responses_only(
    base_trainer,
    instruction_part='[INST] ',
    response_part='[/INST] '
)

tokenizer.decode(trainer.train_dataset[0]["input_ids"])
[INST] Very long content… [/INST] Assistant answer…'

space = tokenizer(' ', add_special_tokens = False).input_ids[0]
tokenizer.decode([space if x == -100 else x for x in trainer.train_dataset[0]["labels"]])

'                        Assistant answer…'

Thanks a lot for your input! :)
I'll give it another shot, but I think I tried this combination before. It's crucial that it works with a multi-turn dataset.


analyze.py error: "max() arg is an empty sequence"

The recent updates resolved the errors I was getting in two repos!
It unearthed a new error in one repo. Until yesterday I was getting this error:
Parse data files, perform aggregation and analysis, generate Markdown report and render as HTML
210412-23:45:08.034 INFO: Remove output directory: latest-report
210412-23:45:08.035 INFO: Create output directory: latest-report
210412-23:45:08.869 INFO: generated new fontManager
210412-23:45:09.041 INFO: fetch stargazer time series for repo olets/nitro-zsh-completions
210412-23:45:09.264 INFO: GH request limit before fetch operation: 4849
210412-23:45:09.421 INFO: GH request limit after fetch operation: 4848
210412-23:45:09.421 INFO: http requests made (approximately): 1
210412-23:45:09.422 INFO: stargazer count: 1
210412-23:45:09.467 INFO: stars_cumulative, raw data: time
2021-03-28 22:03:09+00:00    1
Name: stars_cumulative, dtype: int64
210412-23:45:09.468 INFO: len(series): 1
210412-23:45:09.468 INFO: resample series into 1d bins
210412-23:45:09.472 INFO: len(series): 1
210412-23:45:09.472 INFO: stars_cumulative, for CSV file (resampled):                            stars_cumulative
time
2021-03-28 00:00:00+00:00                 1
210412-23:45:09.477 INFO: write aggregate to ghrs-data/views_clones_aggregate.csv
210412-23:45:09.480 INFO: fetch fork time series for repo olets/nitro-zsh-completions
210412-23:45:09.701 INFO: GH request limit before fetch operation: 4847
210412-23:45:09.827 INFO: GH request limit after fetch operation: 4846
210412-23:45:09.827 INFO: http requests made (approximately): 1
210412-23:45:09.827 INFO: current fork count: 0
210412-23:45:09.835 INFO: len(series): 0
210412-23:45:09.835 INFO: resample series into 1d bins
210412-23:45:09.836 INFO: len(series): 0
210412-23:45:09.837 INFO: forks_cumulative, for CSV file (resampled): Empty DataFrame
Columns: [forks_cumulative]
Index: []
210412-23:45:09.837 INFO: write aggregate to ghrs-data/forks.csv
210412-23:45:09.839 INFO: read views/clones time series fragments (CSV docs)
210412-23:45:09.840 INFO: number of CSV files discovered for *_views_clones_series_fragment.csv: 1
210412-23:45:09.840 INFO: attempt to parse ghrs-data/snapshots/2021-04-12_234505_views_clones_series_fragment.csv
210412-23:45:09.841 INFO: parsed timestamp from path: 2021-04-12 23:45:05+00:00
Traceback (most recent call last):
  File "/analyze.py", line 1398, in <module>
    main()
  File "/analyze.py", line 82, in main
    analyse_view_clones_ts_fragments()
  File "/analyze.py", line 691, in analyse_view_clones_ts_fragments
    if df.index.max() > snapshot_time:
TypeError: '>' not supported between instances of 'float' and 'datetime.datetime'
+ ANALYZE_ECODE=1
error: analyze.py returned with code 1 -- exit.
+ set -e
+ set +x

Now instead I get
Parse data files, perform aggregation and analysis, generate Markdown report and render as HTML
+ python /analyze.py --resources-directory /resources --output-directory latest-report --outfile-prefix '' --stargazer-ts-resampled-outpath ghrs-data/stargazers.csv --fork-ts-resampled-outpath ghrs-data/forks.csv --views-clones-aggregate-outpath ghrs-data/views_clones_aggregate.csv --views-clones-aggregate-inpath ghrs-data/views_clones_aggregate.csv --delete-ts-fragments olets/nitro-zsh-completions ghrs-data/snapshots
210414-23:42:48.927 INFO: Remove output directory: latest-report
210414-23:42:48.927 INFO: Create output directory: latest-report
210414-23:42:49.691 INFO: generated new fontManager
210414-23:42:49.847 INFO: fetch stargazer time series for repo olets/nitro-zsh-completions
210414-23:42:49.969 INFO: GH request limit before fetch operation: 4900
210414-23:42:50.095 INFO: GH request limit after fetch operation: 4899
210414-23:42:50.096 INFO: http requests made (approximately): 1
210414-23:42:50.096 INFO: stargazer count: 1
210414-23:42:50.102 INFO: stars_cumulative, raw data: time
2021-03-28 22:03:09+00:00    1
Name: stars_cumulative, dtype: int64
210414-23:42:50.103 INFO: len(series): 1
210414-23:42:50.103 INFO: resample series into 1d bins
210414-23:42:50.107 INFO: len(series): 1
210414-23:42:50.108 INFO: stars_cumulative, for CSV file (resampled):                            stars_cumulative
time
2021-03-28 00:00:00+00:00                 1
210414-23:42:50.112 INFO: write aggregate to ghrs-data/views_clones_aggregate.csv
210414-23:42:50.115 INFO: fetch fork time series for repo olets/nitro-zsh-completions
210414-23:42:50.743 INFO: GH request limit before fetch operation: 4898
210414-23:42:51.145 INFO: GH request limit after fetch operation: 4897
210414-23:42:51.145 INFO: http requests made (approximately): 1
210414-23:42:51.145 INFO: current fork count: 0
210414-23:42:51.148 INFO: len(series): 0
210414-23:42:51.148 INFO: resample series into 1d bins
210414-23:42:51.149 INFO: len(series): 0
210414-23:42:51.150 INFO: forks_cumulative, for CSV file (resampled): Empty DataFrame
Columns: [forks_cumulative]
Index: []
210414-23:42:51.150 INFO: write aggregate to ghrs-data/forks.csv
210414-23:42:51.154 INFO: read views/clones time series fragments (CSV docs)
210414-23:42:51.154 INFO: number of CSV files discovered for *_views_clones_series_fragment.csv: 1
210414-23:42:51.154 INFO: attempt to parse ghrs-data/snapshots/2021-04-14_234245_views_clones_series_fragment.csv
210414-23:42:51.155 INFO: parsed timestamp from path: 2021-04-14 23:42:45+00:00
210414-23:42:51.158 WARNING: empty dataframe parsed from ghrs-data/snapshots/2021-04-14_234245_views_clones_series_fragment.csv, skip
210414-23:42:51.158 INFO: total sample count: 0
Traceback (most recent call last):
  File "/analyze.py", line 1409, in <module>
    main()
  File "/analyze.py", line 82, in main
    analyse_view_clones_ts_fragments()
  File "/analyze.py", line 717, in analyse_view_clones_ts_fragments
    newest_snapshot_time = max(df.attrs["snapshot_time"] for df in dfs)
ValueError: max() arg is an empty sequence
+ ANALYZE_ECODE=1
+ set -e
+ set +x
error: analyze.py returned with code 1 -- exit.

I don't use Python much so I haven't looked for the bug :)
I can make you a repo collaborator if that's helpful — I think that would let you run the action on the repo?

Should have really waited for your feedback before making that release haha. Will look into that soon, probably not today anymore!

I think that would let you run the action on the repo?

Would appreciate that if that's OK for you! I'll still have to see if I'm courageous enough to actually run that in your repo, but for the testing work of course it saves you and me both some time!

Landed https://github.com/jgehrcke/github-repo-stats/pull/16 which should address the problem shown.

Btw @olets if you like to, you can remove empty CSV files from your data repository by just pushing a corresponding commit manually. In your case this is ghrs-data/snapshots/2021-04-14_234245_views_clones_series_fragment.csv. I understand though if you wouldn't care, and you can also leave it in there : ).
What I like here is the concept of being able to manually fix/change/tweak the state of data via a simple git workflow, leaving transparent history of changes. It think that's a conceptual strength of this solution (compared to e.g. having to 'fix' the state of an object in S3 or in a database).

should address the problem shown

It works! 🙌

Would appreciate that if that's OK for you!

I'll set it up if there's a problem again 👍 That data repo's project is deprecated. Now that we don't need it for debuggin, I'm going to freeze the repo anyway!

Sorry for the annoyances

Not annoying at all!

Thank you for the feedback @olets as always!!


[dvsim] Documentation

Signed-off-by: Srikrishna Iyer<EMAIL_ADDRESS>

I like the partition into user and reference guides. I wonder if the design doc can be the same util/dvsim/doc/_index.md. I think the codelab may be completed after the other two, so I would not mention it until there is something to show: it can be discouraging to be told of something like a codelab and when you click you see it is yet to be developed.
The design doc text needs whittling down.


FR: Device battery

Would like to start off with saying I really love this UI!
I am running it on an dedicated Android tablet. It would be nice to be able to see the tablet's battery level in the UI. Something that could be achieved by creating a dedicated card that interacts with the https://developer.mozilla.org/en-US/docs/Web/API/Battery_Status_API .
What do you think?

No, the browser support is too limited. However, there are alternative methods to achieve this, such as using the Fully Kiosk app and integrating its sensors with HA

Oh, that's a great alternate solution, I will investigate, thank you!