Unify minting process experience

Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.
PR type

[ ] Bugfix
[x] Feature
[x] Refactoring

What's new?

[x] PR closes #3703
[ ] Requires deployment <rubick/magick_issue>
[ ] <brief_description_of_what_I've_added>

RMRK mint:

[x] classic mint
[x] simple mint
[x] creative mint

Before submitting Pull Request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality
[x] I've posted a screenshot of demonstrated change in this PR

Optional

[ ] I've tested it at </rmrk/collection/26902bc2f7c20c546a-1FVG7>
[ ] I've tested PR on mobile and everything seems works
[ ] I found edge cases
[ ] I've written some unit tests 🧪

Had issue bounty label?

[x] Fill up your KSM address: Payout

Community participation

[x] Are you at KodaDot Discord?

Screenshot

[x] My fix has changed something on UI; a screenshot is best to understand changes for others.

Hi, I don't see anything as is on your screenshot.

Still don't see the tab as yours...

Still don't see the tab as yours...

Create -> Simple
https://deploy-preview-3711--koda-nuxt.netlify.app/rmrk/mint

I have tried to mint the NFT, and I think this is a little bit strange. Since after some reloading of the page, you can see the NFT minted - which means it worked, but before you see the NFT it shows some error.
https://user-images.githubusercontent.com/104791453/184615157-4283fcee-f786-4338-b734-eaee35db3f77.mov

So I would remove the "error" sign because it could confuse new users. And the NFT you can see only after reloading the page.

So I would remove the "error" sign because it could confuse new users.

Yes, but the warning maybe not be introduced by this PR IMO : )

@Jarsen136 can you check out this one?
trying to mint NFT to existing collection on BSX without uploading file:

also, I've noticed that there are currently no checks once you upload the file?

oh wow ok, apparently we can mint NFTs without name, is that ok @vikiival ?
https://deploy-preview-3711--koda-nuxt.netlify.app/bsx/u/bXkQe5T8pdkGn78sS6viEUs18wf1kAubDTQX1WWaEdMqDNyCN

@Jarsen136 can you check out this one?

There's something wrong with minting validation on BSX in the main branch.
Already fixed.
✅  minting collection on BSX
✅  minting token on BSX

pay 100

😍 Perfect, I’ve sent the payout
💵 $100 @ 54.61 USD/KSM ~ 1.831 $KSM
🧗 Caiv9TbPz68q5dC8EcHu5xKYPRnremimGzqmEejDFNpWWLG
🔗 0x4f9bbfa29173a03933cfa0b693be576f59e0e8750478f6fb33d0050917f30119
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


[BUG] Missing spec for delta_log/00000000000000000000.crc files

Bug
Describe the problem
The PROTOCOL doesn't describe the CRC files in delta_log, is it intentional?
Example:<PHONE_NUMBER>0000000000.crc
{"tableSizeBytes":1234567,"numFiles":123,"numMetadata":1,"numProtocol":1,"protocol":{"minReaderVersion":1,"minWriterVersion":2},"metadata....
Willingness to contribute
The Delta Lake Community encourages bug fix contributions. Would you or another member of your organization be willing to contribute a fix for this bug to the Delta Lake code base?

[ ] Yes. I can contribute a fix for this bug independently.
[ ] Yes. I would be willing to contribute a fix for this bug with guidance from the Delta Lake community.
[X] No. I cannot contribute a bug fix at this time.

Thanks for making this issue! I believe that @vkorukanti will follow up!

Hi @vkorukanti, do you have any comments on this?

@felipepessoto Where did you see this crc file? cc. @prakharjain09.

Currently it is created by Databricks only.

Thanks for bringing this up @felipepessoto . We are working on adding support for checksum file in the Delta spec and implementation.


MDNS: add status indicator

closes #242

We should add this to the validation:
mdns-repeater: error: at least 2 interfaces must be specified

FYI https://github.com/opnsense/plugins/commit/49f860905b398775753ad5c3644af333634b370a

@fichtner what should we do because of the model?
afaik it is not possible to run migrations on data which does not exist

migration on data that not exists should work fine. it is for filling a set of default values.

have you not set a default for interfaces maybe? ;)

https://github.com/opnsense/plugins/blob/master/net/mdns-repeater/src/opnsense/mvc/app/models/OPNsense/MDNSRepeater/MDNSRepeater.xml#L10-L13
It has no default and it should not (I don't know how a user calls his networks - so this would be guessing random values)

Then can't set to required, it's an impossible condition. IDS defaults to lan for that reason.

On 2. Sep 2017, at 18:15, Fabian Franz<EMAIL_ADDRESS>wrote:
https://github.com/opnsense/plugins/blob/master/net/mdns-repeater/src/opnsense/mvc/app/models/OPNsense/MDNSRepeater/MDNSRepeater.xml#L10-L13
It has no default and it should not (I don't know how a user calls his networks - so this would be guessing random values)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

does lan always exist?

no, but it doesn't matter. we should simply make sure the validation works in the enforced defaults.
it's even trickier here: we would have to assume lan,opt1. it's not pretty, but then it would work in the most default cases hitting "enable" and apply :)

the other thing would be to remove required

the required must stay as the configuration is invalid without the interface name. So I will add a default of lan in the hope that it will work.

ok 👍

added it directly to master


Add "busy" status to default emoji status set

As a blocker for #21178, we should add "Busy" to the list of default emoji statuses. As an initial proposal, we can put this status first on the list, and use the "hammer and wrench" emoji to represent it.
If you are working on this issue, you should start a thread in the #design stream on chat.zulip.org to share screenshots and request feedback.

@zulipbot claim


enhancement: Add Cache county as an externally-linked website

Sean found a website for Cache county that can be linked to for points within the county:
http://<IP_ADDRESS>/websites/surveyviewer/
We need to add this to the web pages dictionary so it can be used as a link.
Note: The data will need to be populated in SGID's 'PLSSPoint_AGRC' layer before this enhancement would have any effect.

Do you want to create a draft PR that we can make ready when the data is ready?

Yeah, I can put that together.

This was resolved with the implementation of PR #36


use IS_DEV const

Coverage remained the same at 89.189% when pulling 1c3b8763f7778de9258024db344dc639d886f674 on use-is-dev-const into 97c98f9ae52dffb25c15a49ae97092aa645a5331 on develop.


Ingame wiki / faq / knowledgebase

From @Fohlen on December 5, 2015 20:7
We could render an in-game wiki on the website with markdown.
The files should go into blog-data and could also be available in the game using a NodeJS/markdown-renderer.
On our websites there should be a sort of knowledge-base, searchable with Ajax
Copied from original issue: inexor-game/website#30

From @Croydon on December 8, 2015 20:1
A wiki for these purposes are imo a post-stable job.

From @Croydon on December 5, 2015 22:13
What should be the content of such a wiki?

Could be anything from general tips, tweaking settings, explanation on gamemodes or coop-edit guides..

What do you think about an in-game browser like Steam has?
http://media.steampowered.com/apps/client/manage_screenshots_overlay.jpg

Well, we are integrating CEF and a new UI on top of that. Therefore, we will have an ingame browser. To what extend we will use it, well, that is another story.

This is related to #382

Will this be a separate section of the wiki? @Croydon

In my opinion we should only touch this issue when we starting to work on gameplay and other high level stuff.
I also didn't suggest that @Fohlen. But I would advise strictly against using our main wiki for it. It is already big and will grow bigger. It should stay clearly technically to not mix things up.
@Fohlen You suggested in the issue to use the blog-data repository for it. That would be one way.

blog-data seems inconvenient, rather create a new "knowledge" repository than.

This will be closed via #

via which issue? D: @Fohlen

yeah, I think #526 is related, but it may not be really focused on the aspect to integrate the docs ingame?

As first step we could integrate the user documentation in ui-flex.


Updates in Turkish translation

I improved the Turkish translation a little bit, fixed the links in there and added Turkish to the translation list.

@yamac-kurtulus I just realised that we have another Turkish translation PR https://github.com/Roshanjossey/first-contributions/pull/3353
@karsibali can you guys work together to improve this translation?

Thanks for reminding me @sara-02. I have realised that PR was available just after I had pushed this out, then I put down a review on that :)

So I made the changes that @karsibali suggested in #3353. Added his name as well, as per request Should be good now.

@yamac-kurtulus, this is brilliant. Great job on the discussions and incorporating changes from #3353.
Merging this 🎉


Mod could connect to the database after restart

Minecraft : 1.20.1
Fabric: 0.15.3
Mod Version: 1.1.2
I don't have my database running on the same server as the Minecraft server.
I had to restart the database server for updates but when the database was online again the mod could not connect to the database and the whole log was full of the message:
Caused by: java.io.EOFException: Can not read response from server. Expected to read 4 bytes, read 0 bytes before connection was unexpectedly lost.
at com.mysql.cj.protocol.FullReadInputStream.readFully(FullReadInputStream.java:67) ~[grieflogger-1.1.2-fabric.jar:?]
at com.mysql.cj.protocol.a.SimplePacketReader.readHeaderLocal(SimplePacketReader.java:81) ~[grieflogger-1.1.2-fabric.jar:?]

The only thing that helped was a restart of the server, which is of course annoying for the players.

The mod - at this point on time - only connects to the database at startup and uses this connection for the communication between database and server. I did not add reconnection functionality to the mod because it's very rare for a database to go offline.


RunWith(PowerMockRunner.class) does not work with package annotation

From<EMAIL_ADDRESS>on November 14, 2012 23:23:44
What steps will reproduce the problem? 1.package-info.java // this is for the package annotation
@TestAnnotation(version="1.0")
package com.smin.dummy;
TestAnnotation.java
@Retention(RetentionPolicy.RUNTIME)
@Target(ElementType.PACKAGE)
public @interface TestAnnotation {
String version();
}
A.java
public class A {
private static Package myPackage;
private static TestAnnotation version;
static {
  myPackage = TestAnnotation.class.getPackage();
  version = myPackage.getAnnotation(TestAnnotation.class);
 }

public static String getVersion() {
    return version.version();
}

} 2. MockA.java
@RunWith(PowerMockRunner.class) //comment out this line to see the difference
@PrepareForTest(A.class)
public class MockA {
@Test
public void test_mocked() throws Throwable {
String thisVersion = A.getVersion();
}
} What is the expected output? What do you see instead? in test_mocked(), thisVersion should be not null, but with @RunWith(PowerMockRunner.class), it's null. What version of the product are you using? On what operating system? powermock 1.4.12 mockito 1.9.0 junit 4.8.2 Please provide any additional information below.
Original issue: http://code.google.com/p/powermock/issues/detail?id=410

From<EMAIL_ADDRESS>on November 20, 2014 05:53:00
I also has this problem! :-/

From<EMAIL_ADDRESS>on March 03, 2015 13:30:14
The same issue. It happens when I try to use PowerMock to mock some stuff in HBase.


Allow reuse of client with different credentials

This pull request addresses the use case where you want to setup a single client with your fitbit consumer key and secret, but then use it across multiple users' oauth credentials, rather than creating a new client per user.  The reconnect method provides 99% of this functionality already, but because the access token is ||= 'ed, it doesn't actually get reset, and further API calls all use the first set of credentials.

Thanks!


[Snyk] Upgrade @babel/preset-env from 7.24.3 to 7.24.4

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade @babel/preset-env from 7.24.3 to 7.24.4.
As this is a private repository, Snyk-bot does not have access. Therefore, this PR has been created automatically, but appears to have been created by a real user.
: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 1 version ahead of your current version.
The recommended version was released 22 days ago, on 2024-04-03.

Release notes

Package name: @babel/preset-env

7.24.4 - 2024-04-03v7.24.4 (2024-04-03)
Thanks @ Dunqing, @ luiscubal, and @ samualtnorman for your first PRs!
👓 Spec Compliance

babel-parser

#16403 Forbid initializerless using (@ JLHwung)

babel-helpers, babel-plugin-proposal-decorators, babel-runtime-corejs3

#16388 Ensure decorators are callable (@ JLHwung)

🐛 Bug Fix

babel-generator

#16402 fix: Correctly prints { [key in Bar]? } (@ liuxingbaoyu)
#16394 fix: Correctly generate TSMappedType (@ liuxingbaoyu)

babel-compat-data, babel-plugin-bugfix-firefox-class-in-computed-class-key, babel-preset-env

#16390 Create bugfix plugin for classes in computed keys in Firefox (@ nicolo-ribaudo)

babel-helper-create-class-features-plugin, babel-plugin-proposal-decorators

#16387 fix: support mutated outer decorated class binding (@ JLHwung)
#16385 fix: Decorators when super() exists and protoInit is not needed (@ liuxingbaoyu)

babel-plugin-transform-block-scoping

#16384 fix: Transform scoping for for X in loop (@ liuxingbaoyu)
#16368 fix: Capture let when the for body is not a block (@ liuxingbaoyu)

babel-core, babel-plugin-transform-block-scoped-functions, babel-plugin-transform-block-scoping

#16363 Fix incorrect function hoisting in some case statements (@ luiscubal)

Committers: 7

Babel Bot (@ babel-bot)
Dunqing (@ Dunqing)
Huáng Jùnliàng (@ JLHwung)
Luís Reis (@ luiscubal)
Nicolò Ribaudo (@ nicolo-ribaudo)
Samual Norman (@ samualtnorman)
@ liuxingbaoyu

7.24.3 - 2024-03-20v7.24.3 (2024-03-20)
Thanks @ ota-meshi for your first PR!
🐛 Bug Fix

babel-helper-module-imports

#16370 fix: do not inject the same imported identifier multiple times (@ ota-meshi)

Committers: 2

Nicolò Ribaudo (@ nicolo-ribaudo)
Yosuke Ota (@ ota-meshi)

    from @babel/preset-env GitHub release notes

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

Just a patch version bump. Change log looks minor.


The project should use the dependabot to keep its dependencies up to date.

Any news on an update that works with RestSharp 110.2.0? Latest SDK throws System.MissingMethodException when calling CreateAssetAsync method.

I did the work in #7 and that was merged. It's just that none of the PRs that have been created have been merged.
@ModSimon


Integration with HA

Hi !
Thanks for your work on that package !
I'm interested to integrate SOMweb with HA but I'm not sure where to find the extension.
Kind regards,
WM

Hi,
I've yet to send a pull request to the HA-team. Still some work and testing required on the config flow. If you want to test my current version as a custom component do this:

Download the latest version of the component.
Extract the file contents under custom_components
EITHER test the config flow by restarting the server and add the SOMweb integration through the user interface (not finished so bugs adding the integration should be expected).
OR add the following to configuration.yaml with your somweb UDI and login details before restarting the server:

cover:
  - platform: somweb
    id: <SOMweb UDI>
    username: <username>
    password: <pwd>

Feedback on the experience with my integration is appreciated.
-Trond

Hi !
Thanks for the reply. I'm still waiting for the SOMweb to arrive, then I'll test the integration and give a feedback.
Regards,
WM

I was thinking about creating this myself as a first phyton component for HA.. I'll for sure try this out :) Hope it get's published! I've seen so many people using somWeb via IFTTT, this component would solve that "problem"..


WIP:  Ensure default asset configuration is extensible

🚧 WIP 🚧
see commit messages for details

I kinda doubt this will still be merged, so ... 💀


Enhanced Navbar

Fixes Issue🛠

Closes #1042
Description👨💻

Modified media queries to enhance navbar from 769 to 1060 width. The distorted lines arise due to the display of contents in 2 lines as it couldn't fit. So changed the font-size and padding of the navbar items to fit the items in a single line.
Type of Change📄

[x] Bug fix (non-breaking change which fixes a bug)
[x] Style (non-breaking change which improves website style or formatting)
[x] Responsiveness (non-breaking change which improves UI/UX on different screen sizes)

Checklist✅

[x] I am an Open Source contributor
[x] I have performed a self-review of my code
[x] My code follows the style guidelines of this project

Screenshots/GIF📷

Before:

After:

@Rakesh9100 Everything looks good.


NodeJSEnv in 1.x does not tolerate source-map-support to be absent anymore

I discovered that while trying to test the satellite libraries with 1.0.0-M4 before publishing them. For example see here a failed job for scalajs-java-time: https://travis-ci.org/scala-js/scala-js-java-time/jobs/398785844#L1354
This seems to have been introduced in 1acffde2e96877bc6d7b280e3dc35ded7242129e, particularly here:
https://github.com/scala-js/scala-js/commit/1acffde2e96877bc6d7b280e3dc35ded7242129e#diff-f111e3e5727bbd6e0ed3d457a32af359R74
which changed compared to
https://github.com/scala-js/scala-js/commit/1acffde2e96877bc6d7b280e3dc35ded7242129e#diff-29501c0410c2a5694836cf35a3e227c6L61
where require('source-map-support') was in a try.
@gzm0 Given the parent commit 891e58330f6484f6f828c685092daa1316221a7a, this appears to have been "intentional". But it seems that it is a big usability issue.
I'm considering declaring this a blocker issue, and by-pass 1.0.0-M4. We should promptly fix it and republish 1.0.0-M5, pretending M4 did not exist.
WDYT?

Hum, I'd rather think about this as WAI: If you do not want source map support, you need to disable it. The reason I dropped the try is because otherwise there is no other way to make sure source map support is enabled; that's not good.
Before we had the ability to detect it so one could use this if one needed to be sure (and we did in our build).

But it seems that it is a big usability issue.

What is the usability issue? That one has to explicitly disable source map support if one does not want it / it isn't installed?

Yes that's basically the usability issue. It is particularly annoying for developers of cross-compiling libraries who don't want to install anything JS-related. It was hard enough, for some of them, to convince them that Node.js was needed in the forest place. And those are the people we really don't want to annoy, because it can make the difference between keeping support for Scala.js and dropping it.

The reason I dropped the try is because otherwise there is no other way to make sure source map support is enabled; that's not good.

Does anyone ever need to do this? There could be other JS envs for which it is impossible to tell anyway. If the user really wants to know, they would have to know which kind of JS env anyway, and then they could perform themselves the test that we did before.

Another point is that, in the same context of the cross-compiling library (but also other projects), needing source map support is not a property of the project per se, but of the local environment. It's very possible that many developers on a cross-compiling project never directly work on the JS version--they only work on the JVM version and let the CI validate JS. Those developers wouldn't want to install source-map-support, and neither the CI. But one developer working on JS-specific stuff, or debugging a JS bug, would like to get the benefit of source maps.
So that means that recommending to cross-compiling libraries that they configure NodeJSEnv with .withSourceMap(false) in their build is not a good idea.
Perhaps there is a middle ground using a triple-state configuration rather than just true/false:
sealed abstract class SourceMaps

object SourceMaps {
  case object Disable extends SourceMaps
  case object EnableIfAvailable extends SourceMaps
  case object Enable extends SourceMaps
}

def withSourceMaps(sourceMaps: SourceMaps) ...

// and for source compat and convenience
def withSourceMaps(sourceMaps: Boolean) =
  withSourceMaps(if (sourceMaps) SourceMaps.Enable else SourceMaps.Disable)

Does anyone ever need to do this?

The Scala.js project does. Otherwise source map tests will fail strangely.

There could be other JS envs for which it is impossible to tell anyway.

This is a Node.js specific issue.

needing source map support is not a property of the project per se, but of the local environment.

That is a very fair point indeed, so probably we should go with tri-state config with the default being EnableIfAvailable. It seems that this is the most reasonable thing to do for most projects. (This will also give us a good location to document the trade-offs discussed in this issue).


Docker configs

Capture Docker compose files to share how I have configured my Docker containers.

I am capturing my Docker configuration such as my compose files in this Wiki and will use separate repos to maintain any custom images.
https://github.com/jcallaghan/home-assistant-config/wiki/Docker


[SPARK-43274][SPARK-43275][PYTHON][CONNECT] Introduce PySparkNotImplementedError

What changes were proposed in this pull request?
This PR proposes to introduce PySparkNotImplementedError. This PR also includes migrating Spark Connect GroupedData errors into error class as well.
Why are the changes needed?
To cover user-facing Python built-in error by PySpark error framework for improving error message usability.
Does this PR introduce any user-facing change?
No API changes.
How was this patch tested?
Fixed UT, the existing CI should pass.

merged to master


Missing includes CL and lvr2

By cloning manually HighFive in ext/HightFive I might have introduced the following problem:
When running catkin build, I have:

 #include <CL/cl2.hpp>
          ^~~~~~~~~~~~
compilation terminated.
make[2]: *** [CMakeFiles/rviz_map_plugin.dir/src/ClusterLabelPanel.cpp.o] Error 1
In file included from ...

and also
fatal error: lvr2/io/PlutoMapIO.hpp: No such file or directory
 #include <lvr2/io/PlutoMapIO.hpp>
          ^~~~~~~~~~~~~~~~~~~~~~~~

What dependencies am I missing? Thanks

Hi @ToniRV  I'll have a look today, and try to fix that issue. Thanks for reporting the bugs!

Hi @ToniRV, could you check again? The bug should be fixed now.

Hi @spuetz,
I still can't find the lvr2 dependency when building the package.
Are there any instructions to install it ?

Hi @pedroperrusi Please see the LVR2 installation steps in the README here: https://github.com/uos/lvr2

Hi,
I'm getting the same compilation error:
Errors     << rviz_map_plugin:make ws/logs/rviz_map_plugin/build.make.002.log
In file included from ws/src/cleansquare_ros/utils/uos_mesh_tools/rviz_map_plugin/src/ClusterLabelTool.cpp:49:
ws/src/cleansquare_ros/utils/uos_mesh_tools/rviz_map_plugin/include/ClusterLabelTool.hpp:57:10: fatal error: CL/cl2.hpp: No such file or directory
 #include <CL/cl2.hpp>
          ^~~~~~~~~~~~
compilation terminated.
make[2]: *** [CMakeFiles/rviz_map_plugin.dir/src/ClusterLabelTool.cpp.o] Error 1
make[2]: *** Waiting for unfinished jobs....
In file included from ws/src/cleansquare_ros/utils/uos_mesh_tools/rviz_map_plugin/include/ClusterLabelPanel.hpp:56,
                 from ws/src/cleansquare_ros/utils/uos_mesh_tools/rviz_map_plugin/src/ClusterLabelPanel.cpp:49:
ws/src/cleansquare_ros/utils/uos_mesh_tools/rviz_map_plugin/include/ClusterLabelTool.hpp:57:10: fatal error: CL/cl2.hpp: No such file or directory
 #include <CL/cl2.hpp>
          ^~~~~~~~~~~~
compilation terminated.
make[2]: *** [CMakeFiles/rviz_map_plugin.dir/src/ClusterLabelPanel.cpp.o] Error 1
In file included from ws/build/rviz_map_plugin/rviz_map_plugin_autogen/6YEA5652QU/../../../../src/cleansquare_ros/external_modules/uos_mesh_tools/rviz_map_plugin/include/ClusterLabelPanel.hpp:56,
                 from ws/build/rviz_map_plugin/rviz_map_plugin_autogen/6YEA5652QU/moc_ClusterLabelPanel.cpp:9,
                 from ws/build/rviz_map_plugin/rviz_map_plugin_autogen/mocs_compilation.cpp:3:
/opt/enway/ws/src/cleansquare_ros/utils/uos_mesh_tools/rviz_map_plugin/include/ClusterLabelTool.hpp:57:10: fatal error: CL/cl2.hpp: No such file or directory
 #include <CL/cl2.hpp>
          ^~~~~~~~~~~~
compilation terminated.
make[2]: *** [CMakeFiles/rviz_map_plugin.dir/rviz_map_plugin_autogen/mocs_compilation.cpp.o] Error 1
In file included from ws/src/cleansquare_ros/utils/uos_mesh_tools/rviz_map_plugin/src/ClusterLabelDisplay.cpp:51:
ws/src/cleansquare_ros/utils/uos_mesh_tools/rviz_map_plugin/include/ClusterLabelTool.hpp:57:10: fatal error: CL/cl2.hpp: No such file or directory
 #include <CL/cl2.hpp>
          ^~~~~~~~~~~~
compilation terminated.
In file included from ws/src/cleansquare_ros/utils/uos_mesh_tools/rviz_map_plugin/src/MapDisplay.cpp:51:
ws/src/cleansquare_ros/utils/uos_mesh_tools/rviz_map_plugin/include/ClusterLabelTool.hpp:57:10: fatal error: CL/cl2.hpp: No such file or directory
 #include <CL/cl2.hpp>
          ^~~~~~~~~~~~
compilation terminated.
make[2]: *** [CMakeFiles/rviz_map_plugin.dir/src/ClusterLabelDisplay.cpp.o] Error 1
make[2]: *** [CMakeFiles/rviz_map_plugin.dir/src/MapDisplay.cpp.o] Error 1
make[1]: *** [CMakeFiles/rviz_map_plugin.dir/all] Error 2
make: *** [all] Error 2

I have installed lvr2 and also manually cloned HighFive into the expected subdirectory because git submodule update --init --recursive doesn't manage to clone the required recursive sub modules. The required opencl dependencies are also installed. Any idea what's wrong?
Thanks

Hi,
I'm getting the same compilation error:
Errors     << rviz_map_plugin:make ws/logs/rviz_map_plugin/build.make.002.log
In file included from ws/src/cleansquare_ros/utils/uos_mesh_tools/rviz_map_plugin/src/ClusterLabelTool.cpp:49:
ws/src/cleansquare_ros/utils/uos_mesh_tools/rviz_map_plugin/include/ClusterLabelTool.hpp:57:10: fatal error: CL/cl2.hpp: No such file or directory
 #include <CL/cl2.hpp>
          ^~~~~~~~~~~~
compilation terminated.
make[2]: *** [CMakeFiles/rviz_map_plugin.dir/src/ClusterLabelTool.cpp.o] Error 1
make[2]: *** Waiting for unfinished jobs....
In file included from ws/src/cleansquare_ros/utils/uos_mesh_tools/rviz_map_plugin/include/ClusterLabelPanel.hpp:56,
                 from ws/src/cleansquare_ros/utils/uos_mesh_tools/rviz_map_plugin/src/ClusterLabelPanel.cpp:49:
ws/src/cleansquare_ros/utils/uos_mesh_tools/rviz_map_plugin/include/ClusterLabelTool.hpp:57:10: fatal error: CL/cl2.hpp: No such file or directory
 #include <CL/cl2.hpp>
          ^~~~~~~~~~~~
compilation terminated.
make[2]: *** [CMakeFiles/rviz_map_plugin.dir/src/ClusterLabelPanel.cpp.o] Error 1
In file included from ws/build/rviz_map_plugin/rviz_map_plugin_autogen/6YEA5652QU/../../../../src/cleansquare_ros/external_modules/uos_mesh_tools/rviz_map_plugin/include/ClusterLabelPanel.hpp:56,
                 from ws/build/rviz_map_plugin/rviz_map_plugin_autogen/6YEA5652QU/moc_ClusterLabelPanel.cpp:9,
                 from ws/build/rviz_map_plugin/rviz_map_plugin_autogen/mocs_compilation.cpp:3:
/opt/enway/ws/src/cleansquare_ros/utils/uos_mesh_tools/rviz_map_plugin/include/ClusterLabelTool.hpp:57:10: fatal error: CL/cl2.hpp: No such file or directory
 #include <CL/cl2.hpp>
          ^~~~~~~~~~~~
compilation terminated.
make[2]: *** [CMakeFiles/rviz_map_plugin.dir/rviz_map_plugin_autogen/mocs_compilation.cpp.o] Error 1
In file included from ws/src/cleansquare_ros/utils/uos_mesh_tools/rviz_map_plugin/src/ClusterLabelDisplay.cpp:51:
ws/src/cleansquare_ros/utils/uos_mesh_tools/rviz_map_plugin/include/ClusterLabelTool.hpp:57:10: fatal error: CL/cl2.hpp: No such file or directory
 #include <CL/cl2.hpp>
          ^~~~~~~~~~~~
compilation terminated.
In file included from ws/src/cleansquare_ros/utils/uos_mesh_tools/rviz_map_plugin/src/MapDisplay.cpp:51:
ws/src/cleansquare_ros/utils/uos_mesh_tools/rviz_map_plugin/include/ClusterLabelTool.hpp:57:10: fatal error: CL/cl2.hpp: No such file or directory
 #include <CL/cl2.hpp>
          ^~~~~~~~~~~~
compilation terminated.
make[2]: *** [CMakeFiles/rviz_map_plugin.dir/src/ClusterLabelDisplay.cpp.o] Error 1
make[2]: *** [CMakeFiles/rviz_map_plugin.dir/src/MapDisplay.cpp.o] Error 1
make[1]: *** [CMakeFiles/rviz_map_plugin.dir/all] Error 2
make: *** [all] Error 2

I have installed lvr2 and also manually cloned HighFive into the expected subdirectory because git submodule update --init --recursive doesn't manage to clone the required recursive sub modules. The required opencl dependencies are also installed. Any idea what's wrong?
Thanks

Found the problem. In addition to the OpenCL dependencies specified in the lvr2 installation, I installed opencl2 headers available via synaptic (for ubuntu 18.04). Now it compiles.

https://packages.ubuntu.com/bionic/opencl-clhpp-headers is required. I'll add this to the dependencies soon. Thank you for reporting the bug.

Just run rosdep install --from-paths src --ignore-src -r -y in the workspace folder.

When I issue the command, I get following:
rosdep install --from-paths src --ignore-src
executing command [sudo -H apt-get install ros-melodic-lvr2]
[sudo] password for mfc:
Reading package lists... Done
Building dependency tree
Reading state information... Done
E: Unable to locate package ros-melodic-lvr2
ERROR: the following rosdeps failed to install
  apt: command [sudo -H apt-get install ros-melodic-lvr2] failed

It is working for me.


Resize image

Can we have a image selector resize feature ?

i dont have it either, and i actually use the demo thing


Facts: Identify services and their current states

Current services and the state of those services would be very useful as facts

merged with #35


add env options and disable negative weights check

What does this PR do?

Adds a file to encapsulate environment variable based options.
Critical path, make negative weight check more tolerant

Testing:
pytest tests/test_config.py
We can also see the options.
>>> from hta.configs.env_options import get_options
>>> print(get_options())

disable_ns_rounding=False, HTA_DISABLE_NS_ROUNDING_ENV=unset
disable_call_graph_depth=False, HTA_DISABLE_CG_DEPTH_ENV=unset
critical_path_add_zero_weight_launch_edges=False, CP_LAUNCH_EDGE_ENV=unset
critical_path_show_zero_weight_launch_edges=False, CP_LAUNCH_EDGE_SHOW_ENV=unset
critical_path_strict_negative_weight_check=False, CP_STRICT_NEG_WEIGHT_CHECK_ENV=unset

Before submitting

[ ] Was this discussed/approved via a Github issue? (no need for typos, doc improvements)

[x] N/A

[x] Did you write any new necessary tests?

[ ] N/A

[ ] Did you make sure to update the docs?

[x] N/A

[ ] Did you update the changelog?

[x] N/A

Thanks for adding the support for configuring with env variables. One of my suggestion is to also support the customization through local configuration, i.e., command line option > env vars > local configuration file > default configuration file.


No usage by users report for some servers

Host operating system: output of uname -a
Linux localhost.localdomain 3.10.0-693.21.1.el7.x86_64 #1 SMP Wed Mar 7 19:03:37 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux
flexlm_exporter version: output of flexlm_exporter --version
flexlm_exporter, version 0.0.1 (branch: master, revision: df8de569d92350b4cf04e1a79990b07f781cb524)
build user<EMAIL_ADDRESS>build date:       20180222-17:37:07
go version:       go1.8.3
flexlm_exporter command line flags and relevant YAML configuration file information
Command line
/usr/bin/flexlm_exporter --path.lmutil=/usr/bin/lmutil --path.config=/etc/prometheus/flexlm_exporter.yaml
yaml file
FlexLM Licenses to be monitored.
licenses:

name: abbrcs_Trappes
license_server<EMAIL_ADDRESS>monitor_users: True
monitor_reservations: True

- name: kukarcs_Castres
license_server<EMAIL_ADDRESS>monitor_users: True
monitor_reservations: True

- name: Tecno_Castres
license_server<EMAIL_ADDRESS>monitor_users: True
monitor_reservations: True

name: Xelec_Castres
license_server<EMAIL_ADDRESS>monitor_users: True
monitor_reservations: True

name: PS_Castres
license_server<EMAIL_ADDRESS>monitor_users: True
monitor_reservations: True

name: PS_Trappes
license_server<EMAIL_ADDRESS>monitor_users: True
monitor_reservations: True

name: kuka_Trappes
license_server<EMAIL_ADDRESS>monitor_users: True
monitor_reservations: True

prometheus.yaml flex_lm part
job_name: 'flexlm'
scrape_interval: '15s'
static_configs:
targets: ['localhost:9319']
Are you running flexlm_exporter in Docker?
No on a VM
What did you do that produced an error?
the metric "flexlm_feature_used_users" is empty for some servers
the metric "flexlm_feature_used" is ok for all the servers
What did you expect to see?
the full list of users
What did you see instead?
nothing
Possible cause:

The servers reporting correctly the used users list

lmstat -c output:
License server status<EMAIL_ADDRESS>License file(s) on UFRTRP01LMRP060.COMAUGROUP.COM: C:\kuka_rcs\COMAU FRANCE_00155DBB9E8B_6955.lic:
UFRTRP01LMRP060.COMAUGROUP.COM: license server UP v11.13.1
Vendor daemon status (on Ufrtrp01lmrp060):
KUKAROB: UP v9.2
lmstat -a output :
Users of KUKAROB_RCS_8:  (Total of 2 licenses issued;  Total of 1 license in use)
"KUKAROB_RCS_8" v5.0, vendor: KUKAROB, expiry: 18-may-2019
nodelocked license locked to NOTHING (hostid=ANY)
cmfy211 UFRTR1LT0087375 UFRTR1LT0087375 (v2.0) (UFRTRP01LMRP060.COMAUGROUP.COM/27001 101), start Fri 5/18 2:46

The servers NOT  reporting the used users list

lmstat -c output:
License server status<EMAIL_ADDRESS>License file(s) on UFRTRP01LMRP060.COMAUGROUP.COM: C:\Program Files\Siemens\PLMLicenseServer\License_Ugslmd.lic:
UFRTRP01LMRP060.COMAUGROUP.COM: license server UP v11.13.1
Vendor daemon status (on UFRTRP01LMRP060):
ugslmd: UP v11.13.1

lmstat -a output :
Users of prd_em_workplace_nt:  (Total of 8 licenses issued;  Total of 1 license in use)
"prd_em_workplace_nt" v14.0, vendor: ugslmd, expiry: permanent(no expiration date)
floating license
cmfy211 UFRTR1LT0087375 UFRTR1LT00873750.0 (v12.0) (UFRTRP01LMRP060.COMAUGROUP.COM/28000 101), start Fri 5/18 2:00

there is a ".0" append to the second client computer name on the second server type. Could it be the problem?
Thanks for your feedback.
BR

@nwies Thanks for reporting! This should be fixed now. Please tell me if it works as intended.

hello, I still ahve the problem. PLease find a complete lmstat -a from the server .
lmstat_trp_pb_a.txt
lmstat_trp_pb.txt
thanks for our help.

hello, I compare a working output to the not working .
there is some datas after on the example below : "(linger: 1140944 / 1340160)"
cmfy211 UFRTR1LT0087375 UFRTR1LT00873750.0 (v1.00) (UFRTRP01LMRP060.COMAUGROUP.COM/28000 1401), start Tue 5/29 9:30  (linger: 1140944 / 1340160)

hope it's help.
BR

@nwies yes, you are right. I added support for this now. Could you please check if this work as intended now? Thanks!

hello,
I just let it run for the day and the lic user name was correctly reported.
thanks for your support.
Br
Nicolas
Le jeu. 31 mai 2018 20:03, Mario Trangoni<EMAIL_ADDRESS>a
écrit :

@nwies https://github.com/nwies yes, you are right. I added support for
this now. Could you please check if this work as intended now? Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mjtrangoni/flexlm_exporter/issues/8#issuecomment-393621694,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJ6I9QoR9UQPBPX8HNB0e89yvpsAXQLeks5t4DBxgaJpZM4UEEOi
.


Introduce a set of official OpenNebula Ansible roles (POC)

Description
Create a set of Ansible roles/playbooks that can be used by the Community to deploy OpenNebula clusters.
Use case
Given a set of VMs / bare-metal machines deploy OpenNebula in common scenarios, including EE features.
Interface Changes
N/A
Additional Context
N/A

Progress Status

[x] Code committed
[x] Testing - QA
[x] Documentation (Release notes - resolved issues, compatibility, known issues)

https://github.com/OpenNebula/one-deploy :hugs:


Chainer: Simplify n_step_gru test

jenkins, test this please

Jenkins CI test (for commit d3a1ee18ae95fe71fdea209e5f582d177c4c6686, target branch master) failed with status FAILURE.

Is this failure related to this PR?

Jenkins, test this please

Jenkins CI test (for commit d3a1ee18ae95fe71fdea209e5f582d177c4c6686, target branch master) failed with status FAILURE.

CI unrelated, merging


Removed unnecessary break

@espadrine while you are around, what do you think of this one?  Noticed it while I was doing last PR.  Thanks!

Nicely spotted, is what I think. Thanks!

@espadrine not a problem at all :smile:


Make the SFFCorrector bspline more robust

This PR makes the SFFCorrector more robust by fitting its smoothing bspline against an arbitrary index instead of time.  This helps in the event of a big time gaps in the data, which make the bspline fit fail due to the knots being linearly spaced in time.
This PR very slightly changes the outcome of the SFF algorithm and, as a result, makes on of the unit tests fail.  This needs to be investigated.

My attempt to improve SFFCorrector was not effective.  Closing for now until we revisit improvements.


Optionally create the _origional files

Right now the _origional files are always created.
Instead there should be a checkbox (default false) for the creation of these duplicate _origional files.
(if exiftool abstraction allows)
Using exiftool directly does allow this option, so I imagine the abstraction also does.

From https://exiftool.org/exiftool_pod.html#item_n

-overwrite_original
Overwrite the original FILE (instead of preserving it by adding _original to the file name) when writing information to an image. Caution: This option should only be used if you already have separate backup copies of your image files. The overwrite is implemented by renaming a temporary file to replace the original. This deletes the original file and replaces it with the edited version in a single operation. When combined with -o, this option causes the original file to be deleted if the output file was successfully written (ie. the file is moved instead of copied).

https://github.com/addshore/facebook-data-image-exif/releases/tag/0.10


password field was not hidden, The issue is fixed [GSSoC'23]

Description
The password field was not hidden so people could see your password. I fixes the issue
Fixes #233
Screenshots
Before:

After

Checklist

[x] Tests have been added or updated to cover the changes
[x] Documentation has been updated to reflect the changes
[x] Code follows the established coding style guidelines
[x] All tests are passing

Thank you @SurajPratap10


[PRODUCT]: Additional Florida Extents map

Product Source (URL Preferred)
Extent Addition for the state of Florida - SAJ would like this added as an option for extents in Cumulus
Product Description
FloridaExtents.zip
Product Format

[ ] Geotiff
[ ] Grib2
[ ] NetCDF
[X ] Other (Please provide description below)

Format Description
Shapefile

Hey @katfeingold, I wanted to give you an assist on this watershed add.
Here's the sql that you will need to add as a new migration:
--(xmin ymax, xmax ymax, xmax ymin, xmin ymin, xmin ymax)

-- notes
/**********************
Buffered extends from qgis

xmin =<PHONE_NUMBER>116814
xmax = 1671778.6007457851
ymax =<PHONE_NUMBER>769654
ymin =<PHONE_NUMBER>058009

After rounding by hand

xmin = 989500
xmax = 1671800
ymax = 978700
ymin = 340000

*********************/

-- add new watershed
INSERT INTO watershed (id, slug, "name", geometry, office_id, output_srid) VALUES
	 ('bfbd5b34-ff07-41d4-87cf-d67c80f98be1','florida','Florida',ST_GeomFromText('POLYGON ((989500 978700, 1671800 978700, 1671800 340000,<PHONE_NUMBER>00,<PHONE_NUMBER>00))',5070),'4142c26c-0407-41ad-b660-8657ddb2be69', 5070);

This should provide the following coverage:

@Enovotny should be able to help with any further technical details.


STAC Client pagination support

Overview
This PR refactors StacClient. All methods that returned Lists return fs2.Streams now. These changes allowed SttpStacClient to support pagination 🎉 🎉 🎉
All changes are done to keep the interface as simple as it is possible, see how it can be combined via Apply2K here
Checklist

[x] New tests have been added or existing tests have been modified
[x] Changelog updated (please use chan)

Other info
// that's how we can use tofu.Mid to add tracing both on F and G
// it is really sad that we have A in our interfaces which was done to avoid copy pasting in JVM and JS clients :/
implicit class MidOps3[U[_[_], _[_], _], F[_], G[_], A](val u: U[F, G, A]) extends AnyVal {
  def attach2(mf: U[Mid[F, *], G, A])(mg: U[F, Mid[G, *], A])(implicit af: ApplyK[U[F, *[_], A]], ag: ApplyK[U[*[_], G, A]]): U[F, G, A] =
    Mid.attach[U[*[_], G, A], F](mf)(Mid.attach[U[F, *[_], A], G](mg)(u))
}

// usage
client.attach2(midF)(midG)

or a more traditional attach semantics:
implicit class MidOps3Tuple[U[_[_], _[_], _], F[_], G[_], A](val mfg: (U[Mid[F, *], G, A], U[F, Mid[G, *], A])) extends AnyVal {
  def attach(u: U[F, G, A])(implicit af: ApplyK[U[F, *[_], A]], ag: ApplyK[U[*[_], G, A]]): U[F, G, A] =
    Mid.attach[U[*[_], G, A], F](mfg._1)(Mid.attach[U[F, *[_], A], G](mfg._2)(u))
}

// usage
(midF, midG) attach client

Closes #198
Closes #324
Closes #326
Closes #316
Closes #314

@jisantuc you mean testing insturctions that are different from tests? Sure:
// to run it you'd need the following dep
// "com.softwaremill.sttp.client3" %% "async-http-client-backend-cats-ce2" % "3.3.4"

import com.azavea.stac4s.api.client.SttpStacClient
import cats.effect.IO
import sttp.client3.asynchttpclient.cats.AsyncHttpClientCatsBackend
import sttp.client3.UriContext

import scala.concurrent.ExecutionContext

object Test {
  implicit val executionContext = ExecutionContext.global
  implicit val contextShift     = IO.contextShift(executionContext)
  implicit val timer            = IO.timer(executionContext)

  def main(args: Array[String]): Unit = {
    AsyncHttpClientCatsBackend
      .resource[IO]()
      .use { backend =>
        val client = SttpStacClient(backend, uri"https://franklin.nasa-hsi.azavea.com/")
        // any number here, or don't use take at all
        // try to pass params, fetch collections, items, etc
        client.search.take(95).compile.toList
      }
      .map(list => println(s"items count: ${list.map(_.id).distinct.length}"))
      .unsafeRunSync()
  }
}

Yeah pretty much exactly that -- it's helpful to me to have a sense of the real use that the test is a tiny version of. Maybe that's not a great attitude about tests. Anyway I threw your example into this ammonite script and it worked great:
import $ivy.`com.azavea.stac4s::core:0.4.0-13-ge85ee2f-SNAPSHOT`
import $ivy.`com.azavea.stac4s::client:0.4.0-13-ge85ee2f-SNAPSHOT`
import $ivy.`com.softwaremill.sttp.client3::async-http-client-backend-cats-ce2:3.3.4`

import com.azavea.stac4s.api.client.SttpStacClient
import cats.effect.IO
import sttp.client3.asynchttpclient.cats.AsyncHttpClientCatsBackend
import sttp.client3.UriContext

import scala.concurrent.ExecutionContext

object Test {
  implicit val executionContext = ExecutionContext.global
  implicit val contextShift     = IO.contextShift(executionContext)
  implicit val timer            = IO.timer(executionContext)

  def main(args: Array[String]): Unit = {
    AsyncHttpClientCatsBackend
      .resource[IO]()
      .use { backend =>
        val client = SttpStacClient(backend, uri"https://franklin.nasa-hsi.azavea.com/")
        // any number here, or don't use take at all
        // try to pass params, fetch collections, items, etc
        client.search.take(95).compile.toList
      }
      .map(list => println(s"items count: ${list.map(_.id).distinct.length}"))
      .unsafeRunSync()
  }
}

Test.main(Array.empty[String])


pre-PR discussion. Breaking changes in Start-RSJob parameter handling

Before I make Pull request with my changes I want to discuss it with community.
NewParam branch of my fork contains several fixes and changes that lead to behaviour change of Start-RSJob function
here is the main article https://github.com/MVKozlov/PoshRSJob/wiki/NewParam-branch-changes
Danger: long text inside :)
Small summary: Start-RSJob can change the way it deals with parametrized scriptblocks, somethimes it stops add $_ into param(), sometimes contrariwise
I think that these changes are good, but you can call them evil :)
Tell me what you think !

The description is very long and complex. I don't think it affects me because I always declare param() in my scriptblock.
I always assumed -ArgumentList should not work with Pipeline. There are no good reasons to allow both. People should use one or the other.
My goal is for PoshRSJob to become simpler. The code should be shorter. It has too much functionality that isn't used and causes lots of bugs.
Does this do that?

Yes, the new code is shorter, it combine Start-RSJob and $data | Start-RSJob cases in one code branch. It also remove bug when first parameter always added as array even it not array
new Start-RSJob never insert  $_   if called like  Start-RSJob -ArgumentList $a, $b, $c  as it do now.
but I can't agree with

There are no good reasons to allow both at the same time. People should use one or the other.

people expects that  $data | Start-RSJob{param($par1) } -ArgumentList $arg1 will work so $par1 = $arg1 (and $data go to $_)
and $data | Start-RSJob{param($par1,$par2) } -ArgumentList $arg1,$arg2 also get $par1 = $arg1, $par2=$arg2 ...
but what we should do if user call it like $data | Start-RSJob{param($par1,$par2) } -ArgumenLlist $arg1 or $data | Start-RSJob{param($par1) } -ArgumentList $arg1, $arg2, $arg3, $rag4 ? ?
And here comes the moment when nuances occur ...
Maybe we should just throw ??
So I created this topic to get answers.
Anyway, I think new code work with parameters more transparent way than old
And it can be modified as we like in a simpler way.
But how we like ? :)

@codykonior  I rewritten long text into other long text ...
in addition I published test script. With it you can clearly see that you too can be affected by these changes if you use Start-RSJob {param($params) } -ArgumentList $args scenario
And in latest commit I included code variants that can mimic current behaviour for different scenarios


Fix minor typo in index.rst (s/mqsqrt.h/mysqrt.h/

Thanks for your interest in contributing to CMake!  The GitHub repository
is a mirror provided for convenience, but CMake does not use GitHub pull
requests for contribution.  Please see
https://gitlab.kitware.com/cmake/cmake/-/tree/master/CONTRIBUTING.rst
for contribution instructions.  GitHub OAuth may be used to sign in.

@fazl thanks.  I migrated this to GitLab MR 5662.
For future contributions, please see CONTRIBUTING.rst and open a merge request over in our gitlab.

@fazl thanks.  I migrated this to GitLab MR 5662.
For future contributions, please see CONTRIBUTING.rst and open a merge request over in our gitlab.


Changes the behavior of block_num(<chain_id>)

Summary
This PR changes the behavior of the custom function block_num(<chain_id>).
Context
That function was returning the latest block that contained a Tableland event, and it should return the latest block seen from the network provider's perspective.
Implementation overview

It implements a new data structure called SharedMemory to make it easy to exchange data between different threads. In this particular case, we want to exchange the last block seen from each network thread and make it available to the gateway. This was already a component I have been discussing with @avichalp about adding. Can be useful for other stuff.
The SharedMemory is passed to the EventFeed that has awareness of the blocks, and also passed to the Gateway's resolver, responsible for resolving the block_num(<chain_id>) function.

This was closed in favor of #527


[twilight-http] unable to add options to a command

What unexpected behavior are you experiencing?
Error trying to create global commands
What are you running your code on?
Win11, rust nightly
What feature flags are you using?
None
How can we reproduce the issue?
Code as below

The following code produces an error, however omitting the command_options works:
        let option = CommandOption::String(ChoiceCommandOptionData { autocomplete: false,
                                                                     choices:      vec![],
                                                                     description:  "Message To Send".to_string(),
                                                                     name:         "Message".to_string(),
                                                                     required:     false });
        self.http
            .create_global_command("echo1")
            .unwrap()
            .chat_input("echo")
            .unwrap()
            .command_options(&[option])
            .unwrap()
            .exec()
            .await
            .expect("set command");

Error
thread 'main' panicked at 'set command: Error { source: None, kind: Response { body: [123, 34, 99, 111, 100, 101, 34, 58, 32, 53, 48, 48, 51, 53, 44, 32, 34, 101, 114, 114, 111, 114, 115, 34, 58, 32, 123, 34, 111, 112, 116, 105, 111, 110, 115, 34, 58, 32, 123, 34, 48, 34, 58, 32, 123, 34, 110, 97, 109, 101, 34, 58, 32, 123, 34, 95, 101, 114, 114, 111, 114, 115, 34, 58, 32, 91, 123, 34, 99, 111, 100, 101, 34, 58, 32, 34, 65, 80, 80, 76, 73, 67, 65, 84, 73, 79, 78, 95, 67, 79, 77, 77, 65, 78, 68, 95, 73, 78, 86, 65, 76, 73, 68, 95, 78, 65, 77, 69, 34, 44, 32, 34, 109, 101, 115, 115, 97, 103, 101, 34, 58, 32, 34, 67, 111, 109, 109, 97, 110, 100, 32, 110, 97, 109, 101, 32, 105, 115, 32, 105, 110, 118, 97, 108, 105, 100, 34, 125, 93, 125, 125, 125, 125, 44, 32, 34, 109, 101, 115, 115, 97, 103, 101, 34, 58, 32, 34, 73, 110, 118, 97, 108, 105, 100, 32, 70, 111, 114, 109, 32, 66, 111, 100, 121, 34, 125],
error: General(GeneralApiError { code: InvalidFormBodyOrContentType, message: "Invalid Form Body" }), status: StatusCode(400) } }'

Deps:
twilight-model = "0.8"
twilight-http = "0.8"
twilight-gateway = "0.8"

The error you're getting (decoded from error) is:
{"code": 50035, "errors": {"options": {"0": {"name": {"_errors": [{"code": "APPLICATION_COMMAND_INVALID_NAME", "message": "Command name is invalid"}]}}}}, "message": "Invalid Form Body"}

Command names cannot contain numbers, see https://discord.com/developers/docs/interactions/application-commands#application-command-object-application-command-naming

@vilgotf Thankyou, how can I decode that text myself?

Figured that out, just for future reference incase anyone come across this, the ChoiceCommandOptionData name field also has to abide by these rules, and capital M was throwing it off.


On latest Firefox scroll wheel zooms map two zoom levels

Steps to reproduce

go to any leaflet map (e.g. the default one in the plunker or any of the examples in the documentation) using Firefox 99.0
use the scroll wheel to zoom
the map jumps two zoom levels each time instead of one.

Environment

Leaflet version: 1.7.1 (also 1.8.0-beta.1)
Browser: Firefox 99.0 (64 bit)
OS/Platform (with version): Linux (Ubuntu 18.04) or Windows 10

This appears to be the only browser exhibiting this behaviour. It was also present in Firefox 98.0.2 but wasn't in the last update we did before that so this is new.

BTW I'm raising this on Leaflet rather than on Mozilla as our other apps, using OpenLayers are not exhibiting this behaviour so it seems to be related to the way Leaflet is servicing scroll wheel events from Firefox.

Can confirm.
We need to double-check the getWheelDelta behaviour at https://github.com/Leaflet/Leaflet/blob/7847cc6913c296fb7e409e086a57366237c42728/src/dom/DomEvent.js#L252-L266

Related / potentially duplicate of https://github.com/Leaflet/Leaflet/issues/7403.

If anyone wants a quick and dirty fix for now then this (where mapObj is your map):
mapObj.on('zoomstart',  function(evt) { evt.target.options.wheelPxPerZoomLevel = 120; });
seems to make the problem go away without breaking anything else.
Tested on Win10/Edge, Win10/Firefox, Linux/Chrome, Linux/Firefox, Linux/Waterfox, MacOS/Safari, ChromeOS/Chrome

We have the same issue.
Can we check the zoomDelta in the following method of leaflet.js?
_performZoom: function () {
	...

    if (!delta) { return; }

    // Fix the issue - https://github.com/Leaflet/Leaflet/issues/7403
    if (Math.abs(delta) > map.options.zoomDelta) {
        delta = delta > 0 ? map.options.zoomDelta : -map.options.zoomDelta;
    }

	if (map.options.scrollWheelZoom === 'center') {
		map.setZoom(zoom + delta);
	} else {
		map.setZoomAround(this._lastMousePos, zoom + delta);
	}
}

I closing this because of duplicate of #7403


Dont Upload If GIST and settings content are the same.

When user hit upload.
Extension needs to check either the settings content are updated compared to existing GIST.

If yes then upload
If no, it should not upload the settings again.

forceUpload can provided to user like forceDownload in order to forcefully hit upload in Gist.
Original suggestion from @borekb on https://github.com/shanalikhan/code-settings-sync/issues/232#issuecomment-299833676

@arnohovhannisyan and this one


[3.x] Deprecate NOTIFICATION_MOVED_IN_PARENT

NOTIFICATION_MOVED_IN_PARENT makes node children management very inefficient.
Replaced by a NOTIFICATION_CHILD_ORDER_CHANGED (and children_changed signal).
Most of the previous tasks carried out by NOTIFICATION_MOVED_IN_PARENT are now done not more than a single time per frame by SceneTree.

This PR breaks compatibility (although this notification was very rarely used, even within the engine), but provides an alternate way to do the same.
Alternative to #74672
Backport of #75701
Example Project
Run before and after PR:
ChildOrderChangedBenchmark.zip
before 9069 milliseconds, after 894 milliseconds (approx 10x faster)
The example project stress tests moving sprite children, which causes large numbers of NOTIFICATION_MOVED_IN_PARENT. After the PR, these are reduced to once per frame, and the notification is replaced with a cheaper direct function call.
Explanation
The exponential notification problem remains one of the biggest inefficiencies / achilles heel in the engine in 3.x, so it makes sense to solve it properly, even if there may require some beta testing to iron out any regressions.
This is not a straight backport of #75701 (as 3.x works in a slightly different way) but is inspired to work in a similar(ish) way. There are some significant differences:

The storage of the dirty parent nodes is on the SceneTree (as in #74672) rather than the Viewport (as in #75701). This is to avoid an extra complication of keeping track of all viewports to flush the dirty parent nodes once per frame.
As with #74672, the dirty nodes store a minimum and maximum child to update. This makes the processing significantly more efficient when only a small subset of the children are moved. This optimization is not so readily available in master, due to the new way the children are stored.

Notes

Unlike #74672, this is compatibility breaking, because it avoids sending the NOTIFICATION_MOVED_IN_PARENT completely. This notification is no longer required in core, but there is the possibility that third parties may rely on it, and need to switch to the new NOTIFICATION_CHILD_ORDER_CHANGED.
We could still send the old notification if breaking compatibility was a major concern, however this would lose a lot of efficiency. This could for instance be switched on by means of a project setting. However this may be best to avoid unless there are regressions requiring it.
Inspired by #75701 this uses simple casting during iteration to determine whether a child needs an update, rather than marking nodes with a flag (the approach used in #74672).
Still requires more checking / testing as the changes here are quite fundamental.
This potentially makes the CanvasLayer updates more efficient because it ensures only a single canvas layer order update per frame.

Follow up
After we decide whether an approach such as this or the alternative PR is desired, there is another significant optimization which can be made in a followup PR, and possibly also applied in master. Much of the remaining inefficiency is in multiple calls to VisualServer::canvas_item_set_draw_index(), where in some cases hundreds or thousands of calls may take place. It seems to make sense to replace this by a single call to an extended version of the function which can change the draw indices of all the items in a single batch.
Discussion
Personally overall I would be tempted to go with this approach over #74672 . This is now closer to master, and is more efficient because it calls update_draw_order() directly instead of going through the heavyweight notifications.
Although it technically breaks compatibility, this is likely to be a rare case, which can either be worked around with the new notification, or we can add an optional legacy mode to call the old notification.
Additionally it should be significantly faster than the version in master because it only updates the range of children changed, rather than all the children every time.

Have assigned Juan or Pedro for review, am struggling here because there's not a lot of us who are familiar with this part of core (and less so in 3.x). But anyone else is welcome to review.

I did a final pass last night / this morning (as I had a rebase disaster and had to look over it minutely anyway :grin: ), and spent a while doing as much testing as possible. I spotted a couple of minor bugs which are now fixed.
I confirmed this gives a major speed up in some node operations, but it is a significant change so will be watching for regressions particularly in controls. Hopefully the next beta will give this some decent testing in the wild.


Intergration no longer working

After the .12 update it appears the intergration no longer works at all.
I can not even remove and set it up again as I get an error that the intergration does not support configuration via the UI.

Hi @Apadgett879 ,
Can you share the debug logs? The overall change was very minor so I'm not sure what the issue is.
Thanks!

@sbabcock23 I must apologize, it seems I managed to get a corrupted download, after updating back to .12 from .10 it all seems to work.
Please feel free to close this issue.

No worries thanks @Apadgett879 for confirming.

I sure will when I get home tonight, I rolled back to .10 for now, and it
is working.
On Mon, May 2, 2022, 11:15 AM sbabcock23 @.***> wrote:

Hi @Apadgett879 https://github.com/Apadgett879 ,
Can you share the debug logs? The overall change was very minor so I'm not
sure what the issue is.
Thanks!
—
Reply to this email directly, view it on GitHub
https://github.com/sbabcock23/hass-tryfi/issues/44#issuecomment-1115021718,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE4XUMPTKD6GY75Y5MA66GLVH7WP7ANCNFSM5U4EMFSQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


Camera in escape menu v4

Acknowledgement of preliminary instructions

[X] I have read and acknowledged the preliminary instructions.

What idea do you have?
Can you add a way to make the camera usable in other menu versions?

No.
https://github.com/pizzaboxer/bloxstrap/wiki/A-guide-to-FastFlags#escape-menu-version

"Keep in mind that Roblox does not maintain any menu versions older than 2023 anymore, so some of the buttons and controls may not work properly. Minor visual or usability problems with them are to be expected."

No.
https://github.com/pizzaboxer/bloxstrap/wiki/A-guide-to-FastFlags#escape-menu-version

"Keep in mind that Roblox does not maintain any menu versions older than 2023 anymore, so some of the buttons and controls may not work properly. Minor visual or usability problems with them are to be expected."

Isn't V4 the latest?

...I didn't see that
Okay, what?

wait sorry i meant v2

wait sorry i meant v2
Im sorry


[APIM] Add API Schema operations

Description
This PR contains code that adds api graphql schema operations to the az apim command module. The following operations were added:
apim api schema create
apim api schema delete
apim api schema list
apim api schema show
apim api schema entity
Testing Guide
I have extended the testing scenario contained in the apim command module folder. For testing execute azdev test apim in the virtual environment. The testing sequence I implemented is described here:
Steps Followed:

First I created an graphql api
Then I created a graphql schema for the api
Validated the schema uploaded by retrieving schema using show command
Validated the list of schemas for an api returned using list command
Validated entity tag using entity tag
Deleted the api schema and validated the count using list command

History Notes
[APIM] Add general az apim api schema commands

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

APIM

Please refer to this doc submitting-pull-requests to specify the information of those new commands in the PR description

/azp run


[Bug]: New Headless Ignores: --disable-features=AutoupgradeMixedContent

Minimal, reproducible example
const puppeteer = require("puppeteer");

(async () => {
	const browser = await puppeteer.launch({
		headless: "new",
		// headless: true, // <-- using the old version fixes the problem
		args: ["--disable-features=AutoupgradeMixedContent"], // I have also tried using --disable-web-security and --allow-running-insecure-content
	});

	const page = await browser.newPage();
	await page.goto("http://neverssl.com");

	console.log(page.url());

	if (page.url().startsWith("https://")) {
		throw new Error("This shouldn't redirect to HTTPS!");
	}
})();

Error string
This shouldn't redirect to HTTPS!
Bug behavior

[ ] Flaky
[ ] PDF

Background
I've been trying to measure redirect timings and noticed that in the new headless mode that I can't disable the autoupgrade to HTTPS.
Expectation
I'm expecting this to not redirect to HTTPS since I've disabled the feature, the domain is not in any HSTS preload lists and doesn't specify any HSTS headers.
Reality
For some reason it redirects (307) to HTTPS. It also includes the Non-Authoritative-Reason header set as  HttpsUpgrades.
Puppeteer configuration file (if used)
No response
Puppeteer version
21.4.0
Node version
18.17.1
Package manager
yarn
Package manager version
1.22.19
Operating system
Linux

Does the behaviour match the headless: false behaviour?
cc @thiagowfx it appears that the args are not correctly merged although it does not affect this issue most likely.

I checked it locally, the args are being correctly merged:
[
  'Translate',
  'AcceptCHFrame',
  'MediaRouter',
  'OptimizationHints',
  'ProcessPerSiteUpToMainFrameThreshold',
  'AutoupgradeMixedContent'
]

Also, even if we remove all other features, the same error is thrown.

Does the behaviour match the headless: false behaviour?

Just gave it a go, strangely I can only get it to work with headless: true. Is there any other way to prevent auto HTTPS upgrades?

@JAD3N it does not look so. Could you try filing a feature request at crbug.com but the new headless matches the headful mode in behaviour and there is no bug from Puppeteer's perspective? I am pretty sure it would reproduce without Puppeteer as Puppeteer just passes the arguments down to the browser.

I believe this is the recent change in Chrome https://blog.chromium.org/2023/08/towards-https-by-default.html and it didn't affect the old headless as that is a technically separate implementation of the browser (see https://developer.chrome.com/articles/new-headless/).


Long term running builds

These actions seem to be frozen and consuming CPU time.
Should be cancelled
https://github.com/dotnet/Docker.DotNet/actions/runs/523150747
https://github.com/dotnet/Docker.DotNet/actions/runs/523182693

Tasks are getting cancelled after 360 minutes

Tasks are getting cancelled after 360 minutes


Removed one youtube channel link

Thanks for your contribution, kindly review the contributions guidelines.Youtube channel are hard to check in terms of provided content. No more channels will be added thanks.


Failed to start server. Error: Unknown: 1 servable(s) did not become available

docker run -p 8501:8501 -p 8500:8500 --mount type=bind,source=/model/test,target=/models/test -e MODEL_NAME=test -t tensorflow/serving:2.1.0
2023-05-24 10:00:07.041263: I tensorflow_serving/util/retrier.cc:33] Retrying of Loading servable: {name: test version: 1} retry: 5
2023-05-24 10:00:07.049557: I external/org_tensorflow/tensorflow/cc/saved_model/reader.cc:31] Reading SavedModel from: /models/test/1
2023-05-24 10:00:07.071548: I external/org_tensorflow/tensorflow/cc/saved_model/reader.cc:54] Reading meta graph with tags { serve }
2023-05-24 10:00:07.071613: I external/org_tensorflow/tensorflow/cc/saved_model/loader.cc:264] Reading SavedModel debug info (if present) from: /models/test/1
2023-05-24 10:00:07.085314: I external/org_tensorflow/tensorflow/cc/saved_model/loader.cc:333] SavedModel load for tags { serve }; Status: fail: Not found: Op type not registered 'StatelessRandomGetKeyCounter' in binary running on 3e5ec75a2724. Make sure the Op and Kernel are registered in the binary running in this process. Note that if you are loading a saved graph which used ops from tf.contrib, accessing (e.g.) tf.contrib.resampler should be done before importing the graph, as contrib ops are lazily registered when the module is first accessed.. Took 35764 microseconds.
2023-05-24 10:00:07.085387: E tensorflow_serving/util/retrier.cc:37] Loading servable: {name: test version: 1} failed: Not found: Op type not registered 'StatelessRandomGetKeyCounter' in binary running on 3e5ec75a2724. Make sure the Op and Kernel are registered in the binary running in this process. Note that if you are loading a saved graph which used ops from tf.contrib, accessing (e.g.) tf.contrib.resampler should be done before importing the graph, as contrib ops are lazily registered when the module is first accessed.
2023-05-24 10:00:07.085449: I tensorflow_serving/util/retrier.cc:46] Retrying of Loading servable: {name: test version: 1} exhausted max_num_retries: 5
2023-05-24 10:00:07.085536: I tensorflow_serving/core/loader_harness.cc:155] Encountered an error for servable version {name: test version: 1}: Not found: Op type not registered 'StatelessRandomGetKeyCounter' in binary running on 3e5ec75a2724. Make sure the Op and Kernel are registered in the binary running in this process. Note that if you are loading a saved graph which used ops from tf.contrib, accessing (e.g.) tf.contrib.resampler should be done before importing the graph, as contrib ops are lazily registered when the module is first accessed.
2023-05-24 10:00:07.085573: E tensorflow_serving/core/aspired_versions_manager.cc:359] Servable {name: test version: 1} cannot be loaded: Not found: Op type not registered 'StatelessRandomGetKeyCounter' in binary running on 3e5ec75a2724. Make sure the Op and Kernel are registered in the binary running in this process. Note that if you are loading a saved graph which used ops from tf.contrib, accessing (e.g.) tf.contrib.resampler should be done before importing the graph, as contrib ops are lazily registered when the module is first accessed.
Failed to start server. Error: Unknown: 1 servable(s) did not become available: {{{name: test version: 1} due to error: Not found: Op type not registered 'StatelessRandomGetKeyCounter' in binary running on 3e5ec75a2724. Make sure the Op and Kernel are registered in the binary running in this process. Note that if you are loading a saved graph which used ops from tf.contrib, accessing (e.g.) tf.contrib.resampler should be done before importing the graph, as contrib ops are lazily registered when the module is first accessed.}, }
2023-05-24 10:00:07.768893: I tensorflow_serving/core/basic_manager.cc:280] Unload all remaining servables in the manager.

could not find the op StatelessRandomGetKeyCounter in official tensorflow, could you show the code for your model?


Small changes to README.md

Description
This PR makes 3 tweaks to the documentation for setting up this repo:

Quick Set Up Section: Added make build_tools to in the list of commands in the Quick Setup Section since it was missing from the list and mentioned along with the other commands in order to get set up.

Direnv Section: Added a link to the Set Up AWS section within the Direnv section since AWS has to be set up in order to use Chamber

Set Up AWS (Optional) Section: Added an updated link for instructions to finish setting up AWS. The previous link went to a 404 page

So glad you're helping us keep the docs updated! 😃 But I noticed the checks didn't run and then realized this is a branch in a forked mymove repo under your github account. I'm not sure if the checks can run that way -- and the checks must pass for you to merge. Our normal flow is to branch in the mymove repo itself rather than fork the repo like you might do with an open-source project where you don't have commit/merge privileges.

@reggieriser Ooohh sorry about that! I just created (or recreated) this PR there: https://github.com/transcom/mymove/pull/4333. I'll go ahead and delete this


The first location shown is very zoomed in

Its not easy to see (without changing zoom) where the device is when first entering the page.

What do you suggest the first zoom level to be?

Would be nice to see what city / part of the city you are in. I don't have a reference to zoom level, but googles 14z is nice:
https://www.google.com/maps/@63.4249306,10.4301998,14z?entry=ttu&g_ep=EgoyMDI0MDkyNC4wIKXMDSoASAFQAw%3D%3D


Commits missing from nuget version 1.0.3

I noticed that I cant override ValidateUser from MembershipProviderOAuthUserService when using nuget version 1.0.3, even though github "Release page" says it should be in the 1.0.3 tag in this commit: https://github.com/mattbrailsford/umbraco-authu/commit/dd746b8d365b67716f70a00fece737a59f08b2af
So I checked the nuget content and this change among a couple of others commits are missing in the files 😱
Maybe the wrong tag was selected for the build service? Could you release a fixed 1.0.4? 😄

Unfortunately I cant use 2.0 since I'm on umbraco 7 :/

See the code in my comment.

Yes but you can use 1.0.3 as me that I'm on 7.
If you want I can pass you the dll.


feat(tests): add support for unregistering a client

Change Summary
As suggested in https://github.com/RobertCraigie/prisma-client-py/issues/243, this adds a unregister_client function that unregisters/deregisters the prisma client.
Closes https://github.com/RobertCraigie/prisma-client-py/issues/243
Checklist

[x] Unit tests for the changes exist
[x] Tests pass without significant drop in coverage
[x] Documentation reflects changes where applicable
[x] Test snapshots have been updated if applicable

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

Thank you so much again 💜


Add Logging Module

NB: Would be nice to use the logging module here for some additional flexibility in output format, location, and configuration.
Originally posted by @lparsons in https://github.com/Princeton-LSI-ResearchComputing/tracebase/pull/442#discussion_r890496237

Dupe of #384

Duplicate of #384, closing.


Vehicles - Lane select by vehicle transport type

Private vehicles are entering public lanes. Fix it.

Fixed!


An Insect Catching game

A Web Game where players can choose a particular Insect and try and catch them on the screen.

Solved the issues.

Fixed the https issue
On Mon, Oct 24, 2022 at 11:08 AM VIDHI BHATT @.***>
wrote:

Assigned #78 https://github.com/VidhiBhatt01/The-Frontend-Store/pull/78
to @Samadeen https://github.com/Samadeen.
—
Reply to this email directly, view it on GitHub
https://github.com/VidhiBhatt01/The-Frontend-Store/pull/78#event-7652487626,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUMCO2WEP6S37GPKCIZHIVLWEZNZXANCNFSM6AAAAAARMYPRLI
.
You are receiving this because you were assigned.Message ID:
@.***
com>


auth refactor?

idea: refactor auth-related stuff to be a 'thing you can request a token from' trait, rather than the current setup which hardcodes our specific oauth implementation. the Client would then probably
be generic over the auth implementation i guess
the current implementation of the auth stuff is in an akward midpoint where it's somewhat written with multiple backend/implementation support in mind, but in practice doesn't really support it:

https://github.com/adrianmgg/tumblr_api/blob/06804b586e9fc148da4e5874ad1205e38d38a8e9/tumblr_api/src/client.rs#L72-L75
https://github.com/adrianmgg/tumblr_api/blob/06804b586e9fc148da4e5874ad1205e38d38a8e9/tumblr_api/src/client.rs#L200-L202

some work started in branch auth-refactor-wip

(ended up going about this slightly differently than what the first paragraph of the original issue proposed, but i'm pretty happy with the current redesign for now and from what sketching out of it i did i'm not sure going the trait way would be the right call anyways)


graylog: 3.2.2 -> 3.2.4

Motivation for this change
New minor release available. Fixes some bugs and adds back functionality that was removed before:
https://docs.graylog.org/en/3.2/pages/changelog.html#graylog-3-2-3
https://docs.graylog.org/en/3.2/pages/changelog.html#graylog-3-2-4
Things done

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

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

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

@fadenb should be backport this?

@ma27 yes, I can create a 20.03 backport PR tonight.
Do we need one for 19.09, too?

Backported to 20.03: 182f229ba7cc197768488972347e47044cd793fb

Do we need one for 19.09, too?

Do you think that's needed? Please note that we haven't backported graylog to 19.09 for a while already (currently there's 3.1.3).


Graphite emitter - add plaintext protocol

This patch adds two configuration options to graphite-emitter extension:

protocol - with addtional plaintext protocol, up to now there was only pickle
replaceSlashWithDot - posibility of sending metrics without slashes by replacing them with dots(#3537)

ex.
Now metrics structure looks like this:
druid.broker.query/cache/delta/hit
             query/cache/delta/miss

After:
druid.broker.query.cache.delta.
                               hit
                               miss

@blugowski thanks for the contribution, i am on board with the change but if my understanding is correct the plain text one you are adding will not support batching of multiple events. If my claim is correct we need to make sure to update the docs to make sure that users understand the eventual difference in terms of operation and performance as well.

@b-slim yes, you're correct, batching is supported only by pickle protocol. I've added appropriate comments in Introduction and description of druid.emitter.graphite.batchSize. Is it good enough?

@blugowski can fill the CLA http://druid.io/community/cla.html

@b-slim done.

👍

Thank you for this PR, both of these features would be very useful for our use case; in which release will it be included?

will run test one more time and merge it afterwords, expect this to be in the next release.

@blugowski can you please fix the style issues ?

[ERROR] /home/travis/build/druid-io/druid/extensions-contrib/graphite-emitter/src/main/java/io/druid/emitter/graphite/GraphiteEmitter.java:143:31: '{' at column 31 should be on a new line. [LeftCurly]

@b-slim fixed.


feat(connector): [Netseasy] added redirection flow

Type of Change

[ ] Bugfix
[ ] New feature
[x] Enhancement
[ ] Refactoring
[ ] Dependency updates

Description

Additional Changes

[ ] This PR modifies the database schema
[ ] This PR modifies application configuration/environment variables

Motivation and Context

How did you test it?

Checklist

[x] I formatted the code cargo +nightly fmt --all
[x] I addressed lints thrown by cargo clippy
[x] I reviewed submitted code
[ ] I added unit tests for my changes where possible
[ ] I added a CHANGELOG entry if applicable

@Vs2720  Thanks for your contribution, Unfortunately we couldn't proceed with this PR as it is a duplicate of #898.
Closing in favour of #898


Force tenants databases to use default user to access the database

I need to enable the package to use a single user to access all the tenants databases. I checked out the tenancy config but couldn't see how to tweak this.
Every Assistance I can get will be appreciated.

Hi there,
I don't think this is actually possible at this moment unless you use the "prefix" division mode.
Which will simply adjust the prefix of the connection in order to separate databases.
You can adjust the division mode in the tenancy.php config.
I hope this helps you out.

Hi @oluomotoso
Here is how I did it, might be a workaround for you too:

I created a custom model for my websites.
The website model has a user_id column
When the user connects through the main domain.com, I can list all websites he has access to (a.domain.com, b.domain.com, ...)
I then created a Gate implemented into a middleware to check if the user has indeed access to the website he is going on:

Gate::define(
    'has-access-website',
    function () {
        // Get current Website
        $currenWebsite = app(Environment::class)->tenant();

        $authUserWebsites = Auth::user()->websites;

        return $authUserWebsites->firstWhere('id', $currenWebsite->id);
    }
);

Hope this helps

Hi, I did exactly that, multi-tenant use a lot of events that could be modified trough listeners. I explained how I did it here
https://github.com/hyn/multi-tenant/issues/722

Listen to the ConfigurationLoaded event and update the username and password of the configuration array.
see https://github.com/hyn/multi-tenant/blob/5.x/src/Database/Connection.php#L333

Thanks @Punksolid and @luceos . I was able to achieve this using the ConfigurationLoaded event.


Increase max skip count to MaxInt64

Codecov Report
Base: 70.61% // Head: 70.61% // No change to project coverage :thumbsup:

Coverage data is based on head (ac02f3a) compared to base (16aea0a).
Patch coverage: 100.00% of modified lines in pull request are covered.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff           @@
##           master     #272   +/-   ##
=======================================
  Coverage   70.61%   70.61%
=======================================
  Files          27       27
  Lines         895      895
=======================================
  Hits          632      632
  Misses        234      234
  Partials       29       29

Impacted Files
Coverage Δ

http/server.go
100.00% <100.00%> (ø)

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


how to call stored functions from laravel

i have enabled auth from mongodb config file. so i have to login using  id and pwd , which is written in env. and i have all set to fetch and insert in mongoDB.
but i have written a function
db.system.js.save(
      {
       _id: "echoFunction",
       value : function(x) { return x; }
     }
   )

but how to call it from my laravel controller  ?
i have tried this :
$cursor = DB::connection('mongodb')->command(array('eval' => 'echoFunction("4")'));
 $data = $cursor->toArray();
 dd($data);

but it show me an error

not authorized on xyz_db to execute command { eval: "echoFunction("4")", $db: "xyz_db", $readPreference: { mode: "primaryPreferred" }, lsid: { id: UUID("91e20240-dcd8-4d90-b6c0-c5bc97c95b6f") } }

it would be better if i can trigger this function from the driver.
thank you in advance.

Hello,
You need to give permission to user, so it'll be authorized to run such commands.
Thanks.


Use etcd proxy for clustermesh

Hi!
I was playing with etcd-proxy and here an small summary about the things that
can be done.
To install etcd-proxy we only need to apply the following deployment, that it's
easy part:
apiVersion: extensions/v1beta1
kind: Deployment
metadata:
  name: etcd-proxy
  namespace: kube-system
spec:
  replicas: 3
  template:
    metadata:
      labels:
        app: etcd-proxy
        etcd_cluster: cilium-etcd
        io.cilium/app: etcd-proxy
    spec:
      automountServiceAccountToken: false
      containers:
      - command:
        - /usr/local/bin/etcd
        - grpc-proxy
        - start
        - --advertise-client-url=https://cilium-etcd-proxy.kube-system.svc:2379
        - --listen-addr=<IP_ADDRESS>:2379
        - --resolver-prefix=https://cilium-etcd-proxy.kube-system.svc:2379
        - --resolver-ttl=60
        - --endpoints=https://cilium-etcd-client.kube-system.svc:2379
        - --data-dir=/var/etcd/data
        - --cacert=/etc/etcdtls/member/peer-tls/peer-ca.crt
        - --cert=/etc/etcdtls/member/peer-tls/peer.crt
        - --key=/etc/etcdtls/member/peer-tls/peer.key
        - --trusted-ca-file=/etc/etcdtls/member/server-tls/server-ca.crt
        - --cert-file=/etc/etcdtls/member/server-tls/server.crt
        - --key-file=/etc/etcdtls/member/server-tls/server.key
        - --debug=true
        image: quay.io/coreos/etcd:v3.3.11
        imagePullPolicy: IfNotPresent
        name: etcd
        ports:
        - containerPort: 2380
          name: server
          protocol: TCP
        - containerPort: 2379
          name: client
          protocol: TCP
        resources: {}
        terminationMessagePath: /dev/termination-log
        terminationMessagePolicy: File
        volumeMounts:
        - mountPath: /etc/etcdtls/member/peer-tls
          name: member-peer-tls
        - mountPath: /etc/etcdtls/member/server-tls
          name: member-server-tls
        - mountPath: /etc/etcdtls/operator/etcd-tls
          name: etcd-client-tls
      volumes:
      - name: member-peer-tls
        secret:
          defaultMode: 420
          secretName: cilium-etcd-peer-tls
      - name: member-server-tls
        secret:
          defaultMode: 420
          secretName: cilium-etcd-server-tls
      - name: etcd-client-tls
        secret:
          defaultMode: 420
          secretName: cilium-etcd-client-tls
---
apiVersion: v1
kind: Service
metadata:
  labels:
    app: etcd-proxy
    etcd_cluster: cilium-etcd-proxy
  name: cilium-etcd-proxy
  namespace: kube-system
spec:
  clusterIP: None
  ports:
  - name: client
    port: 2379
    protocol: TCP
    targetPort: 2379
  selector:
    app: etcd-proxy
    etcd_cluster: cilium-etcd
  sessionAffinity: None
  type: ClusterIP

This set-up has two things that need to be changed:

Well-know identities in Cilium.

The main reason is if we want to use the proxy from operator we need to change
the configmap, so Cilium DS will be affected. I think that can be ok to have
the etcd-proxy as well-know identity.

Secure connections between endpoints and certificates handling:

a) Use the same certicates as etcd-operator and add a new domain in
the following lines:
https://github.com/cilium/cilium-etcd-operator/blob/master/certs/certs.go#L131-L136
b) Install a new certificate for the proxy and use the new certificate from
cilium-config configMap, this is more trouble for users, but I think that is
the best way.
Around the issues that can happen:

Lack of monitoring in the proxy. I didn't found a way (yet) to know:

cache usage and metrics.
The number of backends endpoints that the proxy has.

Using the endpoints as the cilium-etcd-client service works for me, deleted
the endpoints and all works as I expected. Maybe there is any corner case that
I didn't catch yet.

Delay on events. Issue https://github.com/etcd-io/etcd/issues/9988

Deployment:
For me there is two situations here:

Etcd-operator:

I think that this deployment should be part of etcd-operator, an env variable
that defines if etcd-proxy should be deployed. (Obviously the livenessProbe,
etc.. need to be adjust)
This way will help use to scale the proxy to the right numbers, 3 ETCD nodes-1
proxy.

Docs:

For some users this will be an addition for scale their ETCD databases and be
able to scale to more nodes, so a doc about how to install and how to monitor
will be helpful for them.
I'm not sure about next steps here, do we want to test this in a real-scale
cluster to see the benefits and make a decision about next steps?

I've updated the yaml above with what I needed to do to get this working. The diff was:
27,29c27,29
<         - --cacert=/etc/etcdtls/member/peer-tls/peer-ca.crt
<         - --cert=/etc/etcdtls/member/peer-tls/peer.crt
<         - --key=/etc/etcdtls/member/peer-tls/peer.key
---
>         - --cacert=/etc/etcdtls/operator/etcd-tls/etcd-client-ca.crt
>         - --cert=/etc/etcdtls/operator/etcd-tls/etcd-client.crt
>         - --key=/etc/etcdtls/operator/etcd-tls/etcd-client.key

I also had to patch cilium and cilium-etcd-operator. The images are raycovalent/cilium:etcd-proxy and raycovalent/cilium-etcd-operator:etcd-proxy.
I added a known identity to cilium for the etcd-proxy:
$ git diff
diff --git a/pkg/identity/numericidentity.go b/pkg/identity/numericidentity.go
index 90870a5f3..15ecd2214 100644
--- a/pkg/identity/numericidentity.go
+++ b/pkg/identity/numericidentity.go
@@ -102,6 +102,9 @@ const (

        // ReservedCiliumEtcdOperator is the reserved identity used for the Cilium etcd operator
        ReservedCiliumEtcdOperator NumericIdentity = 107
+
+       // ReservedCiliumEtcdOperator is the reserved identity used for the etcd proxy.
+       ReservedCiliumEtcdProxy NumericIdentity = 108
 )

 type wellKnownIdentities map[NumericIdentity]wellKnownIdentity
@@ -258,6 +261,15 @@ func InitWellKnownIdentities() {
                fmt.Sprintf("k8s:%s=cilium-etcd-operator", api.PolicyLabelServiceAccount),
                fmt.Sprintf("k8s:%s=%s", api.PolicyLabelCluster, option.Config.ClusterName),
        })
+
+       WellKnown.add(ReservedCiliumEtcdProxy, []string{
+               "k8s:app=etcd-proxy",
+               "k8s:etcd_cluster=cilium-etcd",
+               "k8s:io.cilium/app=etcd-proxy",
+               fmt.Sprintf("k8s:%s=%s", api.PodNamespaceLabel, namespace),
+               fmt.Sprintf("k8s:%s=default", api.PolicyLabelServiceAccount),
+               fmt.Sprintf("k8s:%s=%s", api.PolicyLabelCluster, option.Config.ClusterName),
+       })
 }

 var (

and I added the service name to the certificates generated by cilium-etcd-operator. I'm not sure if there's any way to do a clean upgrade without regenerating certificates (and restarting cilium).
$ git diff
diff --git a/Makefile b/Makefile
index f328e89..a12ee63 100644
--- a/Makefile
+++ b/Makefile
@@ -3,5 +3,5 @@ VERSION=latest
 endif

 all:
-       docker build -t cilium/cilium-etcd-operator:${VERSION} .
-       @echo "\nTo push to the registry:\ndocker push cilium/cilium-etcd-operator:${VERSION}"
+       docker build -t raycovalent/cilium-etcd-operator:${VERSION} .
+       @echo "\nTo push to the registry:\ndocker push raycovalent/cilium-etcd-operator:${VERSION}"
diff --git a/certs/certs.go b/certs/certs.go
index 0536cc6..f2b2e0f 100644
--- a/certs/certs.go
+++ b/certs/certs.go
@@ -131,6 +131,7 @@ func getServerCertReq(namespace string) *csr.CertificateRequest {
                Hosts: []string{
                        "*.cilium-etcd." + namespace + ".svc",
                        "cilium-etcd-client." + namespace + ".svc",
+                       "cilium-etcd-proxy." + namespace + ".svc",
                        "*.mesh.cilium.io",
                        "localhost",
                },

Looks good. Did you test the propagation delay already? I got a bit spooked by upstream reports about added latency of several seconds to a minute.


docker pull images to update them from docker-hub in boot event

Fixed : https://github.com/WLCG-Lightweight-Sites/simple_grid_puppet_module/blob/1.x/manifests/components/component_repository.pp#L254-L266


Application crash when take picture from camera and accept this image

Bug Report
Problem
When I take a picture with the tablet and accept, the application crashes.
What is expected to happen?
The image is successfully added and the alert is displayed.
What does actually happen?
The application is crashed.
Information

When select image from gallery it working fine
When change option to allowEdit to true, no crashes but showed this alert: You can't edit this image.
Command or Code

function setOptions(srcType) {                 var options = {                     // Some common settings are 20, 50, and 100                     quality: 75,                     destinationType: Camera.DestinationType.FILE_URI,                     // In this app, dynamically set the picture source, Camera or photo gallery                     sourceType: srcType,                     encodingType: Camera.EncodingType.JPEG,                     mediaType: Camera.MediaType.PICTURE,                     allowEdit: false,                     correctOrientation: true                 }                 return options;             }
                    var srcType = Camera.PictureSourceType.CAMERA;
                    var options = setOptions(srcType);

                    navigator.camera.getPicture(function cameraSuccess(imageUri) {

                        alert("get picture");

                    }, function cameraError(error) {
                        console.debug("Unable to obtain picture: " + error, "app");

                    }, options);

Environment, Platform, Device

Lenovo tab m10 hd
Android: 11
Version information

Cordova 11.0.0
Cordova platform: Android 10.1.2
Cordova plugins:

Android studio: 2021.1
Android sdk-tools:

Java Jdk:

Gradle:

Operating System: Windows 10 Home
Checklist

[x] I searched for existing GitHub issues
[x] I updated all Cordova tooling to most recent version
[x] I included all the necessary information above

I have the same issue, have you solved it?

Hi @FZampetti,
I change the plugin. Now I use https://github.com/apache/cordova-plugin-media-capture.
I hope this helps to you.

Presumably the same issue as #679

have something similar, any recommended solution

Closing as stale.
If the "crash" involves the webview restarting, then it isn't a crash. It's the android activity that got killed while the camera is operating, which is normal android lifecycle. Your camera response will be in the resume event. Android Quirk
If the app is fatally crashing (completely closes, potentially with "App stopped working" dialog, then it is likely related to RecoverableSecurityException error which is indirectly fixed by https://github.com/apache/cordova-plugin-camera/pull/907


Fix deleteDefaultPlugins default value

In https://github.com/Oteemo/charts/blob/f4f684482b7a4f569482635a10851d7b99669a91/charts/sonarqube/templates/copy-plugins.yaml#L13 the deleteDefaultPlugins is used as a boolean as in values.yaml example.

Please increment the version in Chart.yaml

Please increment the version in Chart.yaml

I did it.


Fails to build with Clang due to -Werror=unused-but-set-variable

Regressed by #47
$ export CC=clang
$ cmake -G Ninja
$ cmake --build .
[...]
src/conf/grammar.c:1243:9: error: variable 'yynerrs' set but not used [-Werror,-Wunused-but-set-variable]
 1243 |     int yynerrs = 0;
      |         ^

https://mail.gnu.org/archive/html/bison-patches/2022-08/msg00006.html

Above #54 suppresses the warning temporarily, as it's coming from bison and reported by recent clang. That should fix the build breakage.
(Will have to revert that eventually, once bison versions catch up over stable distros)


Increasing the pop up definition text size or maybe add the possibility to increase it manually

Is your feature request related to a problem? Please describe.
Hi jz, maybe this is specific to my computer only but the text on the pop up is a bit small so it's difficult for me to read the text that I have added.
Describe the solution you'd like
Increasing the default pop up text size a little bit or maybe add the possibility to adjust it manually
Add any other context or screenshots about the feature request here.

Hi @emanuelps2708 , thanks for the note.
You could try a custom style.  For example, here's the default hover:

If I add the custom style div.ui-tooltip {  font-size: 150%; } to the settings (home page - settings - settings), it looks like this:

Give that a shot, and if it works for you, I'll add a note to the styling page in the manual.  (It's hard to know how to change things unless you already know CSS etc -- maybe some day that will change, but for now this hacky approach works).
Thanks!

It works perfectly, is just what I needed, thank you so much for helping me ;)

Nice hack, @jzohrab!
I'll start going blind in my 40s, so this will be helpful. I'll increase 5% every few years! :D

Yeah it was a bit tiny, wasn't it!  Super that this works.  I've added it to the manual, https://jzohrab.github.io/lute-manual/usage/themes/custom-styles.html.  It will be there in a few minutes.  Cheers and thx for the issue!


Azbox - Sci device is not created

To do it
mknod /dev/sci0 c 253 0

For azbox Me
mknod /dev/sc0 c 182 0
mknod /dev/sci0 c 252 0

mknod /dev/sc0 c 182 0 is not required


Can material-ui/core 4.12.4 work with React 18?

I have to upgrade all the libraries that an application uses. However, I read some articles and I found that material-ui/core 4.12.4 does not work with React 18.
Do I have to use the force option or upgrade to v5?

Sorry for the late response. For some reason, this issue wasn't automatically labeled and got lost.
Answering your question - it likely won't work. We haven't been testing v4 with React 18 specifically, but only v5 contains changes made for React 18. You may be able to make it work (possibly with some hacks), but you'll be on your own, as we don't provide any guidance in this case.


Modified Samples for Local Registry Entry Definitions

Split the given sample to its constituent parts
Introduced a proxy instead of a main Sequence
Changed the stockquote url to reflect new proxy

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


IPP Everywhere PPD generator: Enumeration of media not consistent when generating options PageSize, PageRegion, ImageableArea, and PaperDimension

PPD files have the required options/entries PageSize, PageRegion, ImageableArea, and PaperDimension to describe the paper sizes supported by a printer and their unprintable margins. Each of them treats the same set of paper sizes so when generating them in the PPD generator the list of paper sizes needs to be generated the same way for each of them.
In the IPP Everywhere PPD generator of CUPS (function _ppdCreateFromIPP() in file cups/ppd-cache.c) PageSize uses another method to obtain the list of supported media than PageRegion and ImageableArea/Paperdimension. It seems that you have added support for both the "media-size-supported" and "media-supported" attributes and have taken into account the different internal structures of them, but made this change only for PageSize, not for PageRegion and ImageableArea/Paperdimension, leading to broken PPDs for some printers, as my HP OfficeJet Pro 8500.
I have passed your changes over to PageRegion and ImageableArea/Paperdimension now and this way the printer works perfectly now.
Patch attached.
ppd-generator-fix-pagesize-pageregion.patch.txt

[master 3be8e02] Fix some more IPP Everywhere PPD generator problems (Issue #4921)


exit-code ignored in cmd-blossom

BUG-issue
What you wanted to do
Running a command with the cmd-blossom.
Description of the bug you run into
The bash-command failed, but the process was continued, even the exit-code of the command was not 0.

Added ignore_errors-flag to cmd-blossom to optionall ignore the exit-code. The old implementation had always ignored the exit-code. No the user can decide it.


Escape variables capability

Submission Checklist

[x] Have you installed and run this change against pre-commit? (pre-commit install)
[x] Are all tests passing? (make tests)
[ ] Have you written unit tests to cover this change?
[x] Is unit test coverage still above 80%?
[x] Have you updated all applicable documentation?
[x] Have you followed the guidelines in our Contributing document?

/gcbrun

@heyealex  ready for reviewing. I resolved the comments that doesn't make sense anymore.

/gcbrun

/gcbrun

This all looks good! I kicked off one full integration test that includes a packer test and a couple terraform tests. Assuming that succeeds, this will be ready to merge.


Added "GPL2+" and "BSD License" to the Free Licenses list.

Hello, I added GPL2+ and BSD License to the free licenses list, because packages such as carla-git and javafx-scenebuilder use those.

Sorry for taking so long, somehow this notification slipped past me (this seems to be a recurring theme with vrms-arch maintainers...).
FYI you don't need to add AUR packages to the "unambiguous DB", that's only for packages whose license is marked as "custom".

Thanks for the feedback. I have now removed the packages from unambiguous_db.py