Build failure on Apple Silicon

/tmp/aspcud-20210111-41893-4psbh5/aspcud-1.9.4/libcudf/tests/critparser.cc:35:5: error: invalid token in expression
    REQUIRE(p.parse(iss));
    ^
/tmp/aspcud-20210111-41893-4psbh5/aspcud-1.9.4/libcudf/tests/catch.hpp:11486:24: note: expanded from macro 'REQUIRE'
#define REQUIRE( ... ) INTERNAL_CATCH_TEST( "REQUIRE", Catch::ResultDisposition::Normal, __VA_ARGS__  )
                       ^
/tmp/aspcud-20210111-41893-4psbh5/aspcud-1.9.4/libcudf/tests/catch.hpp:1296:9: note: expanded from macro 'INTERNAL_CATCH_TEST'
        INTERNAL_CATCH_REACT( catchAssertionHandler ) \
        ^
/tmp/aspcud-20210111-41893-4psbh5/aspcud-1.9.4/libcudf/tests/catch.hpp:1279:38: note: expanded from macro 'INTERNAL_CATCH_REACT'
    if( handler.shouldDebugBreak() ) CATCH_BREAK_INTO_DEBUGGER(); \
                                     ^
/tmp/aspcud-20210111-41893-4psbh5/aspcud-1.9.4/libcudf/tests/catch.hpp:1267:75: note: expanded from macro 'CATCH_BREAK_INTO_DEBUGGER'
    #define CATCH_BREAK_INTO_DEBUGGER() if( Catch::isDebuggerActive() ) { CATCH_TRAP(); }
                                                                          ^
/tmp/aspcud-20210111-41893-4psbh5/aspcud-1.9.4/libcudf/tests/catch.hpp:1246:34: note: expanded from macro 'CATCH_TRAP'
    #define CATCH_TRAP() __asm__("int $3\n" : : ) /* NOLINT */
                                 ^
<inline asm>:1:6: note: instantiated into assembly here
        int $3
            ^

I believe catch.hpp should be updated to use this commit from upstream: https://github.com/catchorg/Catch2/commit/a25c1a24af8bffd35727a888a307ff0280cf9387#diff-ac110b2b11e08efa102d4b37f6a479fb006cdfb9822af039ee07021b349e95bb

@fxcoudert I made a new minor release which hopefully fixes this and other issues.

@fxcoudert I made a new minor release which hopefully fixes this and other issues.


Diffie-Hellman 2048 Bit Parameters should be changed regularly.

Since it is assumed that the NSA uses rainbowtables to break default-DHE-Parameters, one is encouraged to change those parameters periodically.
See:
Paper: Imperfect Forward Secrecy: How Diffie-Hellman Fails in Practice
Consideration: Wie die NSA den Diffie-Hellman Schlüsseltausch knackt(German)

As usual: tests ... work for me! ;)

If we are so concerned about security, why aren't 4096 bit DH parameters used?
Also, isn't daily a bit much ? I mean it doesn't hurt, but I feel like weekly should be enough.

+1 to weekly 4096 bit DH

Even though this has been merged, I can change that daily to weekly if you'd like..
Considering 4096 bit: 1024bit DH is only just getting feasable, 2048 is far in the "cannot break it" zone and bear in mind, that every additional bit doubles the attack space..
This PR was more or less a precautionary thing and common best practice.
Regardless: The last time I tried switching to 4096 bit, around 3 years ago, couple of problems arouse. Clients weren't able to connect because of lacking support and mobile (android) devices took (I guessed) too long to compute the 4096bit(at least I had timeout issues with them)
But I agree, 2048bit would be sufficient to be regenerated every week..

4096 bit DH parameters work great for my nginx setup and all clients I've tried were able to connect to it, but I can imagine that mail clients don't have have up-to-date crypto libraries like browsers do. So you're probably right about that.
I'd recommend 3072 bit, but support for that is worse than for 4096 bit if I remember correctly.
So generating a new 2048 bit param weekly is the way to go imo.

@17Halbe could you add a PR for a weekly update?

I added one #836

Instead of generating your own DH groups with "openssl dhparam", you should use the pre-defined DH groups ffdhe2048, ffdhe3072 or ffdhe4096 recommended by the IETF in [RFC 7919 https://tools.ietf.org/html/rfc7919]. These groups are audited and may be more resistant to attacks than ones randomly generated.
And the Postfix SMTP server EDH parameters file is not secret, these parameters are sent in clear to all remote SMTP clients.

How fast those things change! ;)
So would you like to do a PR? I'm out of time till maybe next week..


ArtPollReplyPacket read EstaCode bug

Please check the ArtPollReplyPacket.cs, method ReadData. EstaCode could be read as EstaCode =  data.ReadInt16();

Are you sure? All the other properties in that packet are read in "network order". Do you have any documentation that shows how it should be read? This is just a fork from an existing package so I haven't authored this particular piece of code.

https://art-net.org.uk/structure/discovery-packets/artpollreply/
"ESTA" field - low byte first like "Port" field

Thanks for the details, make sense, just add that they mixed lo/hi and hi/lo in their spec. But I can see that it's correct in the code for writing a packet. It's been fixed now. BTW, I would probably recommend my other ArtNet library that is a stripped down version of the original ACN, for .NET Standard, and I publish it to NuGet: https://github.com/HakanL/Haukcode.ArtNet

Thanks for your time and effort.


Permission for ACCESS_NETWORK_STATE

Question / Problem
Any suggestions on handling permission for android.permission.ACCESS_NETWORK_STATE?
Summary
I ran into an issue when running the app on Android 13 device where it complained about not having permission for ACCESS_NETWORK_STATE. I initialize the YouTubePlayerView programmatically and I set handleNetworkEvents to true while I set enableAutomaticInitialization to false.
I could not find any reference to handling permissions in the documentation.

Caused by: java.lang.SecurityException: ConnectivityService: Neither user 10171 nor current process has android.permission.ACCESS_NETWORK_STATE.
                                      	at android.os.Parcel.createExceptionOrNull(Parcel.java:3011)
                                      	at android.os.Parcel.createException(Parcel.java:2995)
                                      	at android.os.Parcel.readException(Parcel.java:2978)
                                      	at android.os.Parcel.readException(Parcel.java:2920)
                                      	at android.net.IConnectivityManager$Stub$Proxy.requestNetwork(IConnectivityManager.java:2355)
                                      	at android.net.ConnectivityManager.sendRequestForNetwork(ConnectivityManager.java:4209)
                                      	at android.net.ConnectivityManager.registerDefaultNetworkCallbackForUid(ConnectivityManager.java:4750)
                                      	at android.net.ConnectivityManager.registerDefaultNetworkCallback(ConnectivityManager.java:4717)
                                      	at android.net.ConnectivityManager.registerDefaultNetworkCallback(ConnectivityManager.java:4691)
                                      	at com.pierfrancescosoffritti.androidyoutubeplayer.core.player.utils.NetworkObserver.doObserveNetwork(NetworkObserver.kt:75)
                                      	at com.pierfrancescosoffritti.androidyoutubeplayer.core.player.utils.NetworkObserver.observeNetwork(NetworkObserver.kt:31)
                                      	at com.pierfrancescosoffritti.androidyoutubeplayer.core.player.views.LegacyYouTubePlayerView.initialize(LegacyYouTubePlayerView.kt:102)
                                      	at com.pierfrancescosoffritti.androidyoutubeplayer.core.player.views.YouTubePlayerView.<init>(YouTubePlayerView.kt:90)
                                      	at com.pierfrancescosoffritti.androidyoutubeplayer.core.player.views.YouTubePlayerView.<init>(YouTubePlayerView.kt:33)

What I've tried
Adding the permission android:name="android.permission.ACCESS_NETWORK_STATE fixes the crash issue. However I use this library in a SDK, so I can't force to have this permission always available.  So can you help me understand what would be the consequence of setting  handleNetworkEvents to false if I don't have this permission? Is this just about being able to pause/resume the Player when the internet connection changes or is there anything else to this that I should be aware of?

Default values for both enableAutomaticInitialization and handleNetworkEvents are true so when YouTubePlayerView is initialized via constructor, it runs into RunTimeException when permission is missing.

See the documentation.

From what I understand from the documentation(please correct me if I'm wrong), you need to define YouTubePlayerView in XML even if you want to initialize it programmatically. This is not how I'm doing it. I create the instance programmatically and add the YouTubePlayerView to a ViewGroup programmatically. Here's a snippet to explain what I'm trying to say.
            val container = FrameLayout(context)
            val playerView = YouTubePlayerView(context) // this line fails if there is no android.permission.ACCESS_NETWORK_STATE
            container.addView(playerView)

            playerView.enableAutomaticInitialization = false
            playerView.initialize(youtubeListener,
                true,
                IFramePlayerOptions.default
            )

I have copy-pasted the code snipped you provided and it works for me. Both with handleNetworkEvents set to true and `false.
class MainActivity : AppCompatActivity() {
  override fun onCreate(savedInstanceState: Bundle?) {
    super.onCreate(savedInstanceState)
    setContentView(R.layout.activity_main)

    val playerView = YouTubePlayerView(this)

    val root = findViewById<ViewGroup>(R.id.root)
    root.addView(playerView)

    playerView.enableAutomaticInitialization = false
    playerView.initialize(object : AbstractYouTubePlayerListener() {
      override fun onReady(youTubePlayer: YouTubePlayer) {
        youTubePlayer.loadOrCueVideo(lifecycle, "6JYIGclVQdw", 0f)
      }
    },
      handleNetworkEvents = true,
      IFramePlayerOptions.default
    )
  }
}

<?xml version="1.0" encoding="utf-8"?>
<androidx.appcompat.widget.LinearLayoutCompat
    xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:tools="http://schemas.android.com/tools"
    android:id="@+id/root"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    tools:context=".MainActivity">

</androidx.appcompat.widget.LinearLayoutCompat>

Thanks for looking into it. Appreciate your time.
The issue is reproducible with your code if you add the following to the Manifest.

 <uses-permission
        android:name="android.permission.ACCESS_NETWORK_STATE"
        tools:node="remove" />

While this is trivial, I have to be able to respect the choice of client app using the SDK I'm working on(to reiterate, I use this SDK in a library project that's distributed via Maven). For now, I am checking for permission before I initialize the YouTubePlayerView.  I was wondering if there is a way android-youtube-player SDK could handle this internally.
BTW, I didn't realize there was AbstractYouTubePlayerListener. I very much needed this. Thanks for showing the snippet 🙏

Thanks for clarifying, I understand now :)
We need to check if it's possible to define this permission programmatically instead of in the manifest. If so we could define it only if needed.
If it's not possible, maybe we could wrap the code that requires the permission and catch the exception.
Are you going to be able to look into it?

Yes, I will try adding in a PR. I'm occupied with some work at the moment, so give me a couple of weeks.


Proposal - Incorrect information about fee #2881

https://github.com/bitshares/bitshares-ui/issues/2881
Please check if that is still happening with your new component

Same issue is actual for HTLC creation:

I believe it is an issue in TransactionConfirm component

@sschiessl-bcp , same seems true for new component, even worse - it seems like new component state is not reflected now in actual transaction created.
When I choose CNY asset to pay fee, in transaction confirm I still see TEST as asset for fee payment.
One more question - should I be able to specify arbitrary fee asset for propose operation? if so - there is another issue here since in propose dialog on https://develop.bitshares.org/#/account/twat123 when I change fee asset - default one is used for fee displaying in TransactionConfirm dialog:
https://gyazo.com/5f83191bb11fdecd6604d267409aef96

@sschiessl-bcp , after some initial research I believe that cause of this problem lies in checkFeeStatusAsync function of trxHelper. This function is used in lots of places and is not designed to handle propose operation. All the components, which do use it for fee calculation do not handle this situation in any proper way - so they just use it with specific operation type.
If we take a look onto, for example, transfer method of ApplicationApi it checks if propose_account is set and in this case creates proposal_create operation.
I think implementing the same handling in checkFeeStatusAsync will resolve the problem, but I am not sure how to verify suggested concept, can you give me a hint on this?


omit result id from package version summary

Perhaps just show the packages and the package versions? (And only say something about consistency if they are inconsistent?)

Maybe all this information could instead be in the info results on the top right. Not sure the typical user will know what to make of it


Added slideTo, slidePrev and slideNext public methods

Hi gilbitron,
I've added support to public function as slideTo, slidePrev and slideNext.
The functions works using your native implementation, the only difference is that I made the internal function triggered by namespaced events (slideto.nivoslider, slideprev.nivoslider, slidenext.nivoslider).
Than I exposed those events triggering through public functions.
Feel free to pull or comment,
Regards,
Fabio

Excellent slider!
Please download a working example. Button - slideTo.
For version 3.2.
Thank you.


added Project.toml, removed REQUIRE

I am currently using CBOR.jl for a commercial application.
It would be great to have a tag + release, thank you!

Thank you!


'第三次第一、二题'

 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.


Markdown code blocks unreadable in dark mode

When switching to dark mode, text in code blocks are difficult to read.

Try to give a language name after code block, like ```rust.

Thanks for the tip! However I think it would be great if the code blocks support light and dark mode

However I think it would be great if the code blocks support light and dark mode

LGTM. I will add it in the future.


Fixed to execute Pushy.listen() after registration has been completed

Here is a correction on the implementation example. 🙏
I think Pushy.listen() should be executed after Pushy.register() is completed, since it seems to be implemented assuming that registration is complete.
https://github.com/pushy/pushy-electron/blob/cc979fea1563219a52b4b695df719b27870a88ce/util/mqtt.js#L8-L10

Hi @y-mimura,
Thanks so much for creating this PR and for your contribution.
As of today, Pushy.listen() is already automatically executed after Pushy.register() completes, as can be seen on this line:
https://github.com/pushy/pushy-electron/blob/cc979fea1563219a52b4b695df719b27870a88ce/lib/Pushy.js#L111-L112
Also, there is already a check inside Pushy.listen() to ensure the device is registered before any listening takes place:
https://github.com/pushy/pushy-electron/blob/cc979fea1563219a52b4b695df719b27870a88ce/util/mqtt.js#L7-L10
Therefore, there is no need to move Pushy.listen() inside Pushy.register() as it already gets invoked accordingly.
Please let us know if this makes sense, and if you have any further questions.


Ci/add cicd

Task of story #5.
NOTE that the master branch already has a first version of .travis.yml cicd file, which is a mistake. The introduction of CICD should be done in this PR, but oh well, too late.
TODO: add on: master branch deploy protection

What we need to do now is to add the deploy script or travis CI deploy configurations to deploy to our cloud environment of choice.
For AWS EB (Elastic Beanstalk), follow this tutorial  to change the .travis.yml file.
This task is blocked until story #7  is finished (cloud environment set up). We need to weigh in pros and cons of using EB, EC2, or even other providers like DigitalOcean.
I would prefer using EB since the travis configuration seems quite doable, rather than manually configuring ssh-ing the bundle into the server, restarting the node process, etc.

What we need to do now is to add the deploy script or travis CI deploy configurations to deploy to our cloud environment of choice.
For AWS EB (Elastic Beanstalk), follow this tutorial  to change the .travis.yml file.
This task is blocked until story #7  is finished (cloud environment set up). We need to weigh in pros and cons of using EB, EC2, or even other providers like DigitalOcean.
I would prefer using EB since the travis configuration seems quite doable, rather than manually configuring ssh-ing the bundle into the server, restarting the node process, etc.

Thanks Wilson for getting this started. I will work on unblocking #7 :)

Thanks Wilson for getting this started. I will work on unblocking #7 :)

I tried doing deployments on my EB instance but couldnt get it to work xD
My encrypted aws keys apparently not found

I tried doing deployments on my EB instance but couldnt get it to work xD
My encrypted aws keys apparently not found

Closing in favor of new PR #11

Closing in favor of new PR #11


audio send receive examples corrupt audio

testing on OFX  0.10.1, osx 10.13.6, xcode 9.2, NDI 4.6.2
The left hand channel gets corrupted when sending audio. the right hand channel is ok.
some of the right hand audio gets added to the left channel.
I added a waveform render of the audio buffers to show what happens -  see picture below, i am putting
a sinewave into the right hand channel only  of my audio USB audio interface , but the application is sending both channels
(48khz, 2 channels setup) if I just send the left channel, the receiver shows a line(0) and a little bit of a waveform.

thanks for reporting! I will check it.

@steeley Hi, there was a bug with sending audio stream.
fixed by this commit. b90a675
please check it!

Thank you!  Yes, I can confirm that this has fixed the problem.


[Request] Improve logging and handling of MQTT disconnect

Describe the solution you'd like
I have had 3 situations when the remote MQTT broker has been offline for a period and unfortunately I have not realised that the LnxLink instances have not reconnected when the broker is back up.  To recover from this I have to log into each LnxLink endpoint and restart the service, because I have lost remote control and can't use LnxLink to fix things for me :)
I assume that the MQTT broker connection is reattempted, and that eventually LnxLink gives up. But I see no log entries to help understand what is happening.
I turned on debug logging and shut down my MQTT broker, here's the log output:
Nov 16 19:39:55 stuff systemd[1]: Started LNXlink.
Nov 16 19:39:56 stuff lnxlink[717354]: INFO:lnxlink:LNXlink 2024.11.0, Python 3.10.12
Nov 16 19:39:56 stuff lnxlink[717354]: DEBUG:lnxlink:Path=/usr/local/lib/python3.10/dist-packages
Nov 16 19:39:56 stuff lnxlink[717354]: DEBUG:lnxlink:Found addons: bash, cpu, memory, mounts, restart, required_restart, statistics, sys_updates, update
Nov 16 19:39:56 stuff lnxlink[717354]: INFO:lnxlink:Loaded addons: bash, cpu, memory, mounts, required_restart, restart, statistics, sys_updates, update
Nov 16 19:39:56 stuff lnxlink[717354]: INFO:lnxlink:MQTT connection: Connection Accepted.
Nov 16 19:39:56 stuff lnxlink[717354]: DEBUG:urllib3.connectionpool:Starting new HTTPS connection (1): api.github.com:443
Nov 16 19:39:57 stuff lnxlink[717354]: DEBUG:urllib3.connectionpool:https://api.github.com:443 "GET /repos/bkbilly/lnxlink/releases/latest HTTP/1.1" 200 822
Nov 16 19:54:58 stuff lnxlink[717354]: INFO:lnxlink:Sending statistics data: {"uuid": "c55c416261984ee982cd6a6ac871b920", "version": "2024.11.0"}
Nov 16 19:54:58 stuff lnxlink[717354]: DEBUG:urllib3.connectionpool:Starting new HTTPS connection (1): analyzer.bkbilly.workers.dev:443
Nov 16 19:55:00 stuff lnxlink[717354]: DEBUG:urllib3.connectionpool:https://analyzer.bkbilly.workers.dev:443 "POST /api/lnxlink HTTP/1.1" 200 0
Nov 16 20:12:06 stuff lnxlink[717354]: INFO:lnxlink:MQTT connection: Connection Accepted.

I shut down the MQTT broker at 19:52 and started it back up again at 20:10 (just over 17 minutes with no MQTT broker available).  A few things of note:

no log entry to indicate MQTT broker connection has been lost
no log entry to indicate reconnection attempts
when the MQTT broker was started back up, LnxLink reconnected automatically as expected (after around 2 minutes)

How long/how many times is the MQTT connection attempted before giving up?
Can we get some log entries to indicate broker connection attempts, and broker connection loss?
Additional context
As a side note, I would like the log level setting to be in the config file rather than having to edit the service definition.

Thank you for taking the time to investigate the issue!
I've also made some tests, but I couldn't get the MQTT to timeout.
I disabled the MQTT Broker for about 2 hours, but my LNXlink devices connected without any issue.
The 2 minutes connection time is the default after some time, so it is normal to connect with a bit of a delay.
The MQTT client used by LNXlink should never give up connecting after a connection drop.
I could add a log message when it loses connection which will make it more clear when this happens, but I don't know what else I could check for the issue you are having.

Adding a log entry when connection is lost would be helpful as there is already a "Connection Accepted" log message, so a disconnect message would be appropriate.

The MQTT client used by LNXlink should never give up connecting after a connection drop.

Interesting.. that means the behaviour I have experienced is probably related to my environment.
If you are able to add a DEBUG log entry for MQTT connection attempts then I can attempt to diagnose my issue further, and confirm whether the MQTT client is indeed giving up or if something else is happening.

No change since previous comment;

If you are able to add a DEBUG log entry for MQTT connection attempts then I can attempt to diagnose my issue further, and confirm whether the MQTT client is indeed giving up or if something else is happening.

But also, no further occurrences of the issue that led me to log this request.


[Extracting images] No error message when video duration is less than the passed number of seconds.

I was trying to execute a bulk image extraction from a list of videos :
 $ffmpeg->open('/path/to/video.mp4')
                    ->frame(TimeCode::fromSeconds(10))
                    ->save('/path/to/output.jpg');

The code snippet above aborts the execution without throwing any erreur when the video duration is less than the passed number of seconds to the timecode.

Thanks for the report. I can reproduce it with any video tested.
The problem I see is that as soon as we fix this bug, we need to introduce a breaking change, thus I would say that we delay this to the 1.0 milestone.

This is now fixed in the v1 branch.


Only monsters visible

I can only see monsters, no blocks seem to be visible.
I this a known issue ?

Running on windows.
PS C:\Users\jimmy\PycharmProjects\mineflayer> npm list
<EMAIL_ADDRESS>C:\Users\jimmy\PycharmProjects\mineflayer
+--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>`--<EMAIL_ADDRESS>

Are you using a supported Minecraft version?
On Tue, Aug 29, 2023, 21:03 Jimmy Engelbrecht @.***>
wrote:

I can only see monsters, no blocks seem to be visible.
I this a known issue ?
[image: image]
https://user-images.githubusercontent.com/3243901/264126262-ede7d90f-987e-4dc3-8662-a9e4635b9377.png
Running on windows.
npm list
@.*** C:\Users\jimmy\PycharmProjects\mineflayer
+-- @.***
+-- @.***
`-- @.***
—
Reply to this email directly, view it on GitHub
https://github.com/PrismarineJS/prismarine-viewer/issues/407, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAR437VNA3625KLSI2PLWU3XXY4JXANCNFSM6AAAAAA4DMZXGE
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Fixes 36

what
Added a working example.
Added the ability to specify the number of public or private subnets.
Added the ability to specify the format of the public/private subnet tag.
Updated the label module to allow the use of context.
Updated the default cidr_range to allow for a default value so it automatically can pull it from the vpc
why
Fixes https://github.com/cloudposse/terraform-aws-dynamic-subnets/issues/36
Also allows users to use the module to create just a public OR a private subnet, or have different numbers of each.
i.e:
module "dynamic_subnets" {
  source                  = "./.."
  context                 = "${module.label.context}"
  tags                    = "${merge(module.label.tags, local.subnet_tags)}"
  region                  = "${data.aws_region.current.name}"
  availability_zones      = ["${data.aws_region.current.name}a", "${data.aws_region.current.name}b"] // Optional list of AZ's to restrict it to
  vpc_id                  = "${module.vpc.vpc_id}"
  igw_id                  = "${module.vpc.igw_id}"
  public_subnet_count     = "2"                                                                      // Two public zones for the load balancers
  private_subnet_count    = "4"                                                                      // Four private zones for the
}

If it does’t have a default then context doesn’t nest correctly.
Regards
Jamie Nelson
-/-
On 4/06/2019, at 2:47 PM, Andriy Knysh<EMAIL_ADDRESS><EMAIL_ADDRESS>> wrote:
@aknysh commented on this pull request.

In label_variables.tf:

+variable "namespace" {

type        = "string"
default     = "global"
description = "Namespace, which could be your organization name, e.g. 'cp' or 'cloudposse'"
+}

+variable "stage" {

type        = "string"
default     = "global"
description = "Stage, e.g. 'prod', 'staging', 'dev', or 'test'"
+}

+variable "name" {

type        = "string"
default     = "codebuild"

remove default, make name required
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@aknysh Reviewed. Thank you.


Implicitly watch and track anonymous pipeline and PCollections

When pipelines are defined in local scopes, they need to be explicitly watched so that Interactive Beam can cache for them. The change makes it implicit for PCollections and their pipelines queried by show and collect even if they are locally defined and anonymous to Interactive Beam.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @y1chi
PTAL


Add root folder qualification to Webpack resolve.

Howdy,
What do you think about adding the root folder as a qualified search location for imports?  That way we could write something like:
import myComponent from 'src/components/myComponent';
instead of
import myComponent from '../../../../../../components/myComponent';
I believe all you would need to do is add the root property to the resolve section of the Webpack config.
...
resolve: {
    root: path.resolve('.'), // <--- This line here.
    fallback: paths.nodePaths,
    extensions: ['.js', '.json', '.jsx', ''],
   ...
}
...

This seems like a feature request for the original create-react-app library as well @tgoyer ..

I'm sure a lot of people are clamoring for it.  It's a no-brainer, really.  Their proposed solution (of using a node_modules folder under the src directory just seems.. odd to me.

:+1: Doesn't seem like much to ask for, does it?


Warning/Error for incorrect endpoint paramaters or parameter values

Hi there! The plugin rocks! It would be very nice feature to inform user about incorrect endpoint parameters or parameter values. The natural way of doing this could be using Warnings/Errors in IDEA.

Yeah its related to #10 but maybe we can also have a way of turning on / off showing validation errors in the source code editor by somehow flagging the error at the endpoint urls.

solved in PR #127


LSM Multi-column domain and pond/soil test

Goals for this PR:

define an LSM MultiColumnDomain - Hybrid box + plane or hybrid box + hybrid box with one element
redefine interaction methods for soil/pond to broadcast correctly between the field of surface variables and the field of the top soil layer, for both single column and multi column domains.
check that the output for a simulation with no variation in the horizontal leads to the same output in each column, and matches what we expect based on a single column run, or that the RHS eval gives the same result (unit test)
add unit tests for the new domain type.
we could even design the RHS for the LSM as looping over columns and computing horizontal terms afterwards; saving this for a later PR is fine!

Closing in lieu of https://github.com/CliMA/ClimaLSM.jl/pull/85


Compile error: internal compiler error: Segmentation fault

When I include mapmap into my project, it report internal compiler error: Segmentation fault.
The detail error information:
"
In file included from /usr/local/include/opencv2/core.hpp:3281:0,
from /usr/local/include/opencv2/opencv.hpp:52,
from /mnt/dev/myproject/src/test_mapmap.h:11,
from /mnt/dev/myproject/src/test_mapmap.cc:13:
/usr/local/include/opencv2/core/utility.hpp: In instantiation of 'class cv::TLSDatacv::instr::NodeDataTls':
/usr/local/include/opencv2/core/utility.hpp:1247:26:   required from here
/usr/local/include/opencv2/core/utility.hpp:764:17: internal compiler error: Segmentation fault
    inline void gather(std::vector<T*> &data) const
             ^

Please submit a full bug report,
with preprocessed source if appropriate.
See <file:///usr/share/doc/gcc-5/README.Bugs> for instructions.
"
While the error missing when I comment the reference to mapmap.
I try to change the opencv version to 4.1, unfortunately it report the same error!
The gcc version:
gcc (Ubuntu 5.5.0-12ubuntu1) 5.5.0 20171010
opencv: 3.4.0
Looking forward to your reply!

Wow, I have never seen a segfault in the compiler. Could you please try another compiler, e.g., clang?


Drawing fixes

Fixed #600
Fixed #517

Download:
Environment: QT=5.12, ARCH=64, PYTHON_PATH=C:\Python37-x64
Environment: QT=5.12, ARCH=32, PYTHON_PATH=C:\Python37

Download:
Environment: QT=5.12, ARCH=64, PYTHON_PATH=C:\Python37-x64
Environment: QT=5.12, ARCH=32, PYTHON_PATH=C:\Python37


Noobish question: where does the data /actually/ live in the OVF deployment?

So, I spun up Cortx on my ESXi host...but having read basically-everything I could find, I'm still missing an a-HA! moment somewhere...
For a bit of context, I have an ESXi host that lives on my lab LAN, along with a FreeNAS build. I've got Minio running on the FreeNAS via its S3 communications option. The ESXi doesn't have a meaningful amount of storage available to it, so the ultimate goal is to compare Cortx to Minio and see how it fares. I'm not a super huge enterprise user or anything, but I am interested in seeing Cortx work...I'm just missing a thing or two and I might not be the only one...

The WebUI doesn't seem to allow the ability to define or monitor the storage on the backend. While I was able to figure out how to define buckets, assigning them to a storage volume didn't seem to be readily apparent in the WebUI.

I'm assuming that it's intended to have the capacity to point data to something other than the included hard disks, as 20/50/50GB disk volumes do not a massive storage server make. It'd make a bit more sense that I would be expected to put the VMDK volumes on an NFS or iSCSI share if one of them was a thin-provisioned 16EB volume or something, but that does not seem to be the case.

The WebUI frontend (nor any of the documentation I have read thus far) does not seem to indicate the ability to create an iSCSI/NFS connection.

Really, what I'm missing here is that if Cortx is supposed to send data to hard disks (ideally Seagate ones, obviously), then I would assume that there would be some amount of that function that's handled by Cortx instead of letting VMWare do it. I readily admit to not-being super well versed in this sort of storage functionality, but it may well be worth addressing in the FAQ if this concern is 100% a PEBKAC issue.

@voyager529 , thank you for your interest in CORTX.
The VM is provided just to demonstrate basic CORTX functionality but is not intended for production use. It doesn't provide high availability, performance or capacity and there is no guarantee for long term data preservation.
CORTX is an object storage layer which doesn't care much about the underlying storage. We focus on making it shine with particular hardware for mass capacity use cases (we call it Lyve Rack), but CORTX may work with any general purpose local drives. If a customer decides to use an FC or an iSCSI target instead of local drives - it should work too. However, as these options are practically unlimited we're not planning to provide special integrations for them. If you're interested in a particular configuration, you're welcome to test it and provide an installation documentation which we'll gladly publish in CORTX git repository. See an example in https://github.com/Seagate/cortx/tree/main/doc/scaleout


When a user requests to kill an execution a check that it's still running is made several times however the execution may finish before the kill action is processed after those checks causing it to stay in KILLING status forever.

Pull Request Test Coverage Report for Build 5736

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
7 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.004%) to 43.428%

Files with Coverage Reduction
New Missed Lines
%

azkaban-common/src/main/java/azkaban/jobExecutor/AbstractProcessJob.java
2
55.67%

azkaban-common/src/main/java/azkaban/executor/container/ContainerizedDispatchManager.java
5
50.88%

Totals

Change from base Build 5727:
-0.004%

Covered Lines:
14998

Relevant Lines:
34535

💛  - Coveralls


jupyter lose all progress

Issue Type: Bug
i have auto-save selected. however, more than 5 times in the past 7 days, i run some .ipynb scripts and it got stuck, and once i reopen the scripts only to find that most of progress was lost.
VS Code version: Code 1.46.1 (cd9ea6488829f560dc949a8b2fb789f3cdc05f5d, 2020-06-17T21:13:20.174Z)
OS version: Windows_NT x64 10.0.17763

System Info

Item
Value

CPUs
Intel(R) Xeon(R) W-2133 CPU @ 3.60GHz (12 x 3600)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.66GB (23.22GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (3)

Extension
Author (truncated)
Version

gc-excelviewer
Gra
3.0.36

latex-workshop
Jam
8.10.0

python
ms-
2020.6.89148

The Jupyter notebooks come from the Python extension. Please file the issue there.


Implement threads pin offset for PerClusterPools

In my scenario, I use a multi-process application to handle user requests, and each process has limited threads to limit the computing resources each request uses. After adapting to the new PerClusterPools APIs, I found that threads in different processes were pinned to the same few processors, greatly hurting performance. So I added this parameter to the constructor of PerClusterPools to provide a way to avoid this issue.

PR #343 fixes the issue this PR was trying to fix so that this PR can be closed.


CRS behaviour in leaflet 0.7.7 and 1.0.1

[X] I'm reporting a bug, not asking for help
[X] I've looked at the documentation to make sure the behaviour is documented and expected
[X] I'm sure this is a Leaflet code issue, not an issue with my own code nor with the framework I'm using (Cordova, Ionic, Angular, React…)
[X] I've searched through the issues to make sure it's not yet reported

How to reproduce

Leaflet version I'm using: 1.0.1 and 0.7.7
Browser (with version) I'm using: Chromium Version 53.0.2785.143 (64-bit)
OS/Platform (with version) I'm using: Linux-64

I have a few example example, like these two below, to show CRS distortion to students using leaflet. However, I noticed that the behavior changed from leaflet 0.7.7 to leaflet 1.0.1. Is that expected? I could not find any docs on the change.
Leaflet 0.7.7
https://bl.ocks.org/ocefpaf/ef8b8c558b40f024a67080d876e2a617
Leaflet 1.0.1
https://bl.ocks.org/ocefpaf/c01a2777e78550b8a8dfa3cbc88703cd
Thanks for the attention!

You're not reporting a bug, you're asking for help.
Yes.
It's noted in the changelog.

It's noted in the changelog.

Where?

You're not reporting a bug, you're asking for help.

Not sure that is true.

https://github.com/Leaflet/Leaflet/blob/master/CHANGELOG.md#projections-api-improvements

You're not reporting a bug, you're asking for help.

Not sure that is true.

Putting EPSG:3857 tiles in a Leaflet map with a EPSG:4326 CRS is a PEBKAC, not a Leaflet bug. See
https://playground-leaflet.rhcloud.com/xus/edit?html,output

Putting EPSG:3857 tiles in a Leaflet map with a EPSG:4326 CRS is a PEBKAC,

A very useful error for teaching distortion. I will just go back to Python libraries to demonstrate this as the behavior there still works  Thank you for the clarification.

So are you actually asking to port a buggy behaviour from 0.7.x into 1.0.x, just so you can show how it fails?

So are you actually asking to port a buggy behaviour from 0.7.x into 1.0.x, just so you can show how it fails?

Thanks, but there is no need to change that only for me. It is not a https://xkcd.com/1172/ case. As I mentioned in https://github.com/Leaflet/Leaflet/issues/5029#issuecomment-255349684 there are many other libraries I can use to demonstrate that behavior. Leaflet was just an eye candy to explore the distortion interactively.


Check_node_active calculation incorrect in certain scenario.

Simultaneous removal of an existing nodepool and addition of new nodepool messes up the check_node_active calculation.

@orisvscs Could you please brief the use case and share the terraform logs .

Resolved for 5.x in #640


Added ability to save preprocessed files with data augmentation

Fixes #316
Proposed Changes

using applyaugs as the cli parameter, a user can save preprocessed files with augmentation

Checklist

[x] I have read the CONTRIBUTING guide
[x] My PR is based from the current GaNDLF master
[x] Non-breaking change (would not break existing functionality): please provide as many details as possible for any breaking change
[x] Function/class source code documentation added/updated
[x] Code has been blacked for style consistency
[x] If applicable, version information has been updated in GANDLF/version.py
[x] If adding a git submodule, add to list of exceptions for black styling in pyproject.toml file
[x] Usage documentation has been updated, if appropriate
[x] History has been updated, if appropriate
[x] Tests added or modified to cover the changes; if coverage is reduced, please give explanation

Is this a usecase?


dev: run gas snapshot compare python script in the CI and comment results in PR

### Tasks
- [ ] Create a CI job that runs on PR
- [ ] The job runs the compare python script
- [ ] Comments the result on the PR as a report

see also https://github.com/ClementWalter/madara/blob/8aa5efa51847cdc06ecb0d9585531607689777c2/.github/workflows/benchmarks.yml#L36
https://github.com/benchmark-action/github-action-benchmark/tree/v1/
      - name: Compare result
        uses: benchmark-action/github-action-benchmark@v1
        with:
          tool: "customBiggerIsBetter"
          output-file-path: ./benchmarking/reports/metrics.json
          alert-threshold: "120%"
          github-token: ${{ secrets.GITHUB_TOKEN }}
          fail-on-alert: true
          summary-always: ${{ github.ref == 'refs/heads/main' }}
          comment-always: false
          comment-on-alert: false
          auto-push: ${{ github.ref == 'refs/heads/main' }}


Sync data to my server in background state every hour

I want to sync data even when app is terminated, i want to upload the data to my server.
How is that possible for me ?

Ops put the wrong label, its a question

Check the background observers

@mmubeennisar were you able to make it work?

@Gytjarek @mmubeennisar have you tried background observers? I am having issue so asking if it is running for you if you have tried?


Update dmlc-core submodule

This introduces a number of bug fixes and enhancements:

Handle dangling symlinks in dmlc::ListDirectory (dmlc/dmlc-core#463)
Enable specifying stacktrace size at runtime. (dmlc/dmlc-core#470)
In-house implementation of float parsing, regardless of system locale (dmlc/dmlc-core#455, dmlc/dmlc-core#481)

Closes #3230

Added dmlc/dmlc-core#482 to handle f suffix correctly.

@trivialfis Cool. Let me know when #3795 is ready for review.

Submitted dmlc/dmlc-core#484 to fix compilation error on 32-bit Windows.


ValidationDefaults Validators() assumes the underlying type returned from a DI container resulting in an exception for a custom DI container

The problematic line: https://github.com/appany/AppAny.HotChocolate.FluentValidation/blob/770b7ad4f5e7d6238b8a5fe2f99d5ff77a86665d/src/ValidationDefaults.cs#L219
The call to GetServices() returns an  IEnumerable<object?>, It would not be correct to assume that the underlying type is an array (object[]). While the default DI container indeed returns an array, this may not hold in all cases.
Real-life example: I was looking into Wolverine for messaging, which requires you to use Lamar. Lamar returns a List<T> instead of an array which results in a runtime InvalidCastException.

@WORMrus Could you please provide a pr for this fix?

@sergeyshaykhullin Sorry, took me some time to notice a notification. Sure, I will. Seems like an easy enough fix.

@sergeyshaykhullin please see https://github.com/appany/AppAny.HotChocolate.FluentValidation/pull/112

@WORMrus https://www.nuget.org/packages/AppAny.HotChocolate.FluentValidation/0.11.2


[BUG] import_weekly_rosters function thowing ValueError

Is there an existing issue for this?

[X] I have searched the existing issues

Have you installed the latest development version of the package(s) in question?

[X] I have installed the latest development version of the package.

If this is a data issue, have you tried clearing your nflverse cache?
I am not using an nflverse R package (nflreadr, nflfastR) to access this data.
What version of the package do you have?
0.3.2
Describe the bug
when running this function on a single season there is no error, but when putting multiple years in, it throws an error
Reprex
roster = nfl_new.import_weekly_rosters([i for i in range(2022,2024)])

error:
Traceback (most recent call last):
  File "...interactiveshell.py", line 3553, in run_code
    exec(code_obj, self.user_global_ns, self.user_ns)
  File "<ipython-input-68-4f57e1e7a23a>", line 1, in <module>
    roster = nfl_new.import_weekly_rosters([i for i in range(2022,2024)])
  File "...lib\site-packages\nfl_data_py\__init__.py", line 474, in import_weekly_rosters
    rosters["age"] = ((roster_dates - rosters.birth_date).dt.days / 365.25).round(3)
  File "...\lib\site-packages\pandas\core\frame.py", line 4299, in __setitem__
    self._set_item(key, value)
  File "...\lib\site-packages\pandas\core\frame.py", line 4512, in _set_item
    value, refs = self._sanitize_column(value)
  File "...\lib\site-packages\pandas\core\frame.py", line 5250, in _sanitize_column
    return _reindex_for_setitem(value, self.index)
  File "...\lib\site-packages\pandas\core\frame.py", line 12679, in _reindex_for_setitem
    raise err
  File "...\lib\site-packages\pandas\core\frame.py", line 12674, in _reindex_for_setitem
    reindexed_value = value.reindex(index)._values
  File "...\lib\site-packages\pandas\core\series.py", line 5144, in reindex
    return super().reindex(
  File "...\lib\site-packages\pandas\core\generic.py", line 5607, in reindex
    return self._reindex_axes(
  File "...\lib\site-packages\pandas\core\generic.py", line 5630, in _reindex_axes
    new_index, indexer = ax.reindex(
  File "...\lib\site-packages\pandas\core\indexes\base.py", line 4429, in reindex
    raise ValueError("cannot reindex on an axis with duplicate labels")
ValueError: cannot reindex on an axis with duplicate labels

Expected Behavior
function to return combine df for all years in list
nflverse_sitrep
NA

Screenshots
No response
Additional context
No response

Duplicate of https://github.com/nflverse/nfl_data_py/issues/75, fixed in 0.3.3 dev release.
@nateb7722 run the command below in your environment and that should fix it.
pip install --force-reinstall nfl-data-py<EMAIL_ADDRESS>


Python Basic Data-Structure (List)

Python Basic Data-Structure (List) #1659

Description 📜
I added the File which Describe the List in Python
Issue Number : #1659

Type of change 📝

[ ] Audio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)
[ ] Video (Animations, screen-recordings, presentations and regular explanatory films are all possibilties etc)
[x] Documentation (Content Creation in the form of codes or tutorials)
[ ] Other (If you choose other, Please mention changes below)

Domain of Contribution 📊

[ ] Android Dev (Flutter, Java, Kotlin)
[ ] Backend Dev (Java, .NET, Python)
[ ] C/CPP
[ ] Competitive Programming
[ ] Cyber Security
[ ] DSA
[ ] Database
[ ] Datascience with Python
[ ] Datascience with R
[ ] Frontend Dev HTML/CSS/JS
[ ] Frontend Dev React/Angular/Vue
[ ] Golang
[ ] Interview Prep
[ ] Java_Domain
[ ] JavaScript
[ ] MERN
[ ] Machine Learning
[ ] Open Source
[x] Python_Domain
[ ] Rust
[ ] Statistics
[ ] UI/UX

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

Kindly follow the project structure given and you had pushed the contribution to main repo, instead it should be in branch domain @bhumitbedse

Please assign me this issue. Documentation @prathimacode-hub


[silabs] Copy of EFR32 folder in SiWx917 folder

Copy of EFR32 folder in SiWx917 folder

It is just a duplicate of EFR32 folder
In the next PR the platform changes will be merged which will be the diff for EFR32 files for easy review

NOTE : !!! no need to review the files, it's just a copy of EFR32 files. Just verifying the path is enough
NOTE : In the next PR we will include SiWx917 platform specific changes so that It will be easier to check the diff done in order to make the new platform modifications.

@andy31415 The point of have the clone is just to get the easy review done on the next PR by diff between the 2 platforms. Since reviewing the complete file will be time taking. It will be easier to check the diff done inorder to make the new platform modifications.

@andy31415 The point of have the clone is just to get the easy review done on the next PR by diff between the 2 platforms. Since reviewing the complete file will be time taking. It will be easier to check the diff done inorder to make the new platform modifications.

Please add details to the PR summary.

@andy31415 The point of have the clone is just to get the easy review done on the next PR by diff between the 2 platforms. Since reviewing the complete file will be time taking. It will be easier to check the diff done inorder to make the new platform modifications.

Please add details to the PR summary.

I would also say that if you want the diff to be manageable, you can also create a PR with several commits:

one copy
one with deltas

And let the reviewer know that the first commit is just the copy and the commits can be then reviewed individually.

@andy31415 The point of have the clone is just to get the easy review done on the next PR by diff between the 2 platforms. Since reviewing the complete file will be time taking. It will be easier to check the diff done inorder to make the new platform modifications.

Please add details to the PR summary.

I would also say that if you want the diff to be manageable, you can also create a PR with several commits:

one copy
one with deltas

And let the reviewer know that the first commit is just the copy and the commits can be then reviewed individually.

@andy31415   #24207  this will be a follow up PR

The PR  (https://github.com/project-chip/connectedhomeip/pull/24207)  already have these changes, so closing this PR


Unicode chars incorrectly read with driver 2.0

Driver version
<IP_ADDRESS>
Redshift version
Redshift 8.0.2
PostgreSQL 8.0.2 on i686-pc-linux-gnu, compiled by GCC gcc (GCC) 3.4.2 20041017 (Red Hat 3.4.2-6.fc3), Redshift 1.0.44126
Client Operating System
Win 10 21H2 (19044.2251)
Table schema
Any varchar column
Problem description

Expected behaviour: Non-printable Unicode characters shouldn't be shown.
Actual behaviour: Non-printable Unicode characters are shown
Error message/stack trace: N/A
Any other details that can be helpful: Problem is easy to reproduce. Store a concat of CHR(8203) and any string. Unicode 8203 is "zero width space". JDBC driver and ODBC v1 properly output this as empty space to clients.

ODBC trace logs
N/A
Reproduction code
Any table with a varchar column. Insert data containing char(8203), and try to consume in clients. In my case, the client is PowerBI. Messy values with the new driver

old driver, same dataset, showing the right values

Hi @RenatoMartins-tomtom, thank you for reporting this issue. I've forwarded this to our ODBC development team for investigation and will update when available.

@RenatoMartins-tomtom Thanks for reaching out.
A unicode conversion fix has been included in the upcoming release. I just added chr(8203) to some fields (including VARCHAR as well as column names) in one of the manual tests and things worked.
The change will be included in the upcoming release.
#!/usr/bin/env python
# coding: utf-8
''' This test aims to verify unicode conversion of cell values and column names.
    It verifies column names using a VIEW
'''
import pyodbc, sys,traceback
from array import *
try:
    T = [['我的数@据1111','我的数@@据1111','我的数@@@据1111'],
        ['我的数`据2222','我的数``据2222','我的数```据2222'],
        ['我的数†据3333','我的数††据3333','我的数†††据3333'],
        ['我的数µ据4444','我的数µµ据4444','我的数µµµ据4444' + chr(8203)]]

    C = ["col编号","col名称",chr(8203) + "col地i" + chr(8203) + "址" + chr(8203)]

    cnxn = pyodbc.connect('Driver={Amazon Redshift ODBC Driver (x64)};Server=MYSERVER;Port=5439;Database=dev;UID=auser;PWD={Pdtst;123}')
    cursor = cnxn.cursor()

    cursor.execute('START TRANSACTION')
    cursor.execute('DROP TABLE IF EXISTS testtb001 CASCADE')
    cursor.commit()
    cursor.execute('''
    CREATE TABLE IF NOT EXISTS testtb001
    (
    p_partkey     INTEGER NOT NULL,
    p_cx      VARCHAR(100) NOT NULL,
    p_c0      VARCHAR(100) NOT NULL,
    p_c1      VARCHAR(100) NOT NULL,
    p_c2      VARCHAR(100) NOT NULL
    );'''
    )
    cursor.execute('COMMIT')
    #insert unicode values
    cursor.execute(f"INSERT into  testtb001 VALUES(111,'t001','{T[0][0]}','{T[0][1]}','{T[0][2]}');")
    cursor.execute(f"INSERT into  testtb001 VALUES(222,'t002','{T[1][0]}','{T[1][1]}','{T[1][2]}');")
    cursor.execute(f"INSERT into  testtb001 VALUES(333,'t003','{T[2][0]}','{T[2][1]}','{T[2][2]}');")
    cursor.execute(f"INSERT into  testtb001 VALUES(444,'t004','{T[3][0]}','{T[3][1]}','{T[3][2]}');")
    #create a view with unicode column names
    t = '''CREATE OR REPLACE VIEW testview02 ({0},{1},{2}) AS select case when p_cx is null then '未知' else p_cx end, p_partkey , p_c0 from testtb001;
    '''.format(C[0], C[1], C[2])
    cursor.execute(t)
    cursor.commit()
    cursor.execute('SELECT * FROM testtb001')

    rows = cursor.fetchall()

    print('test table\'s unicode cell values....', end = '')
    for i in range(len(rows)):
        row = rows[i]
        #print(row)
        assert row.p_c0.encode('utf-8') == T[i][0].encode('utf-8'), f"{i},{j}: row fetch failed! Expected {T[i][0].encode('utf-8')}, Actual {row.p_c0.encode('utf-8')}"
        assert row.p_c1.encode('utf-8') == T[i][1].encode('utf-8'), f"{i},{j}: row fetch failed! Expected {T[i][1].encode('utf-8')}, Actual {row.p_c1.encode('utf-8')}"
        assert row.p_c2.encode('utf-8') == T[i][2].encode('utf-8'), f"{i},{j}: row fetch failed! Expected {T[i][2].encode('utf-8')}, Actual {row.p_c2.encode('utf-8')}"
    print('pass')

    print('test view\'s unicode cell values...', end = '')
    cursor.execute('SELECT * FROM testview02')
    rows = cursor.fetchall()
    for i in range(len(rows)):
        row = rows[i]
        #print(row)
        assert row[2].encode('utf-8') == T[i][0].encode('utf-8'), f"{i}: view row fetch failed!Expected {T[i][0].encode('utf-8')}, Actual {row[2].encode('utf-8')}"
    print('pass')

    print('test view\'s unicode column name....', end = '')
    columns = [column[0] for column in cursor.description]
    for i in range(len(C)):
        # print(columns[i], len(columns[i]))
        assert columns[i].encode('utf-8') == C[i].encode('utf-8'), f"{i}: View Column error: Expected {C[i].encode('utf-8')}, Actual {columns[i].encode('utf-8')}"
    print('pass')
except AssertionError:
    _,msg, tb = sys.exc_info()
    traceback.print_tb(tb) # Fixed format
    tb_info = traceback.extract_tb(tb)
    filename, line, func, text = tb_info[-1]
    print(f'Error: {msg}')
    sys.exit(1)
finally:
    cnxn.close()

sys.exit(0)

Hi Vahid.
Thanks for the reply. I didn’t see this comment in the issue, oddly enough..
Another question is I see you don’t publish releases in a standard way in the driver repo. Do you have an ETA for this <IP_ADDRESS> version? How can I know a new release came? By looking at the readme edits, or the aws doc here https://docs.aws.amazon.com/redshift/latest/mgmt/odbc20-install-config-win.html ?
<EMAIL_ADDRESS>sorry for looping you on this also. Still puzzled as of how to know about new versions, as the repo doesn’t follow a standard way of releases in there, as they are dumped into an S3 bucket.
Regards,

Renato Martins  /  Cloud Solutions Architect CCoE /  TomTom Service Platform /<EMAIL_ADDRESS>/  www.tomtom.comhttp://www.tomtom.com/

From: vahid @.>
Sent: Friday, 3 February 2023 01:38
To: aws/amazon-redshift-odbc-driver @.>
Cc: Renato Martins @.>; Mention @.>
Subject: Re: [aws/amazon-redshift-odbc-driver] Unicode chars incorrectly read with driver 2.0 (Issue #6)
@RenatoMartins-tomtomhttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FRenatoMartins-tomtom&data=05|01|renato.martins%40tomtom.com|dadec4beef674931881b08db057eece1|374f80267b544a3ab87d328fa26ec10d|0|0|638109814925817849|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=7PA454h1anpdREPaVZ2jkCDVDnD11tOdHMHseLtx%2F1A%3D&reserved=0 Thanks for reaching out.
A unicode conversion fix has been included in the upcoming release. I just added chr(8203) to some fields (including VARCHAR as well as column names) in one of the manual tests and things worked.
The change will be included in the upcoming release.
#!/usr/bin/env python
coding: utf-8
''' This test aims to verify unicode conversion of cell values and column names.
It verifies column names using a VIEW

'''
import pyodbc, sys,traceback
from array import *
try:
T = [['我的数@据1111','我的数@@据1111','我的数@@@据1111'],

    ['我的数`据2222','我的数``据2222','我的数```据2222'],

    ['我的数†据3333','我的数††据3333','我的数†††据3333'],

    ['我的数µ据4444','我的数µµ据4444','我的数µµµ据4444' + chr(8203)]]

C = ["col编号","col名称",chr(8203) + "col地i" + chr(8203) + "址" + chr(8203)]

cnxn = pyodbc.connect('Driver={Amazon Redshift ODBC Driver (x64)};Server=MYSERVER;Port=5439;Database=dev;UID=auser;PWD={Pdtst;123}')

cursor = cnxn.cursor()

cursor.execute('START TRANSACTION')

cursor.execute('DROP TABLE IF EXISTS testtb001 CASCADE')

cursor.commit()

cursor.execute('''

CREATE TABLE IF NOT EXISTS testtb001

(

p_partkey     INTEGER NOT NULL,

p_cx      VARCHAR(100) NOT NULL,

p_c0      VARCHAR(100) NOT NULL,

p_c1      VARCHAR(100) NOT NULL,

p_c2      VARCHAR(100) NOT NULL

);'''

)

cursor.execute('COMMIT')

#insert unicode values

cursor.execute(f"INSERT into  testtb001 VALUES(111,'t001','{T[0][0]}','{T[0][1]}','{T[0][2]}');")

cursor.execute(f"INSERT into  testtb001 VALUES(222,'t002','{T[1][0]}','{T[1][1]}','{T[1][2]}');")

cursor.execute(f"INSERT into  testtb001 VALUES(333,'t003','{T[2][0]}','{T[2][1]}','{T[2][2]}');")

cursor.execute(f"INSERT into  testtb001 VALUES(444,'t004','{T[3][0]}','{T[3][1]}','{T[3][2]}');")

#create a view with unicode column names

t = '''CREATE OR REPLACE VIEW testview02 ({0},{1},{2}) AS select case when p_cx is null then '未知' else p_cx end, p_partkey , p_c0 from testtb001;

'''.format(C[0], C[1], C[2])

cursor.execute(t)

cursor.commit()

cursor.execute('SELECT * FROM testtb001')

rows = cursor.fetchall()

print('test table\'s unicode cell values....', end = '')

for i in range(len(rows)):

    row = rows[i]

    #print(row)

    assert row.p_c0.encode('utf-8') == T[i][0].encode('utf-8'), f"{i},{j}: row fetch failed! Expected {T[i][0].encode('utf-8')}, Actual {row.p_c0.encode('utf-8')}"

    assert row.p_c1.encode('utf-8') == T[i][1].encode('utf-8'), f"{i},{j}: row fetch failed! Expected {T[i][1].encode('utf-8')}, Actual {row.p_c1.encode('utf-8')}"

    assert row.p_c2.encode('utf-8') == T[i][2].encode('utf-8'), f"{i},{j}: row fetch failed! Expected {T[i][2].encode('utf-8')}, Actual {row.p_c2.encode('utf-8')}"

print('pass')

print('test view\'s unicode cell values...', end = '')

cursor.execute('SELECT * FROM testview02')

rows = cursor.fetchall()

for i in range(len(rows)):

    row = rows[i]

    #print(row)

    assert row[2].encode('utf-8') == T[i][0].encode('utf-8'), f"{i}: view row fetch failed!Expected {T[i][0].encode('utf-8')}, Actual {row[2].encode('utf-8')}"

print('pass')

print('test view\'s unicode column name....', end = '')

columns = [column[0] for column in cursor.description]

for i in range(len(C)):

    # print(columns[i], len(columns[i]))

    assert columns[i].encode('utf-8') == C[i].encode('utf-8'), f"{i}: View Column error: Expected {C[i].encode('utf-8')}, Actual {columns[i].encode('utf-8')}"

print('pass')

except AssertionError:
_,msg, tb = sys.exc_info()

traceback.print_tb(tb) # Fixed format

tb_info = traceback.extract_tb(tb)

filename, line, func, text = tb_info[-1]

print(f'Error: {msg}')

sys.exit(1)

finally:
cnxn.close()

sys.exit(0)
—
Reply to this email directly, view it on GitHubhttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Faws%2Famazon-redshift-odbc-driver%2Fissues%2F6%23issuecomment-1414555759&data=05|01|renato.martins%40tomtom.com|dadec4beef674931881b08db057eece1|374f80267b544a3ab87d328fa26ec10d|0|0|638109814925817849|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=8L3WQxZZM0GG8J0UFqZxnktR3NBA%2FIBRo3WDCmJ93vI%3D&reserved=0, or unsubscribehttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FASBMEDKX22S2MRDS24VPDEDWVRHPBANCNFSM6AAAAAAUNWOYRI&data=05|01|renato.martins%40tomtom.com|dadec4beef674931881b08db057eece1|374f80267b544a3ab87d328fa26ec10d|0|0|638109814925817849|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=DQw5q8fwkk81Kp4YBTclLqpZGtYBAqTaEyCAW5eFn5o%3D&reserved=0.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

Hi @RenatoMartins-tomtom ,
After posting a message I realised that I have posted it after following some standard regulations. So, please stay tuned. I'll be back to you shortly.

Hi @RenatoMartins-tomtom
And sorry for the inconvinience caused by the long delay.
Kindly aloow me to be more specific about the publish and installation guidelines once I discussed this internally.
At this moment, I can say that we fortunately have made some progress in this issue and a release will be published soon.
By the way,
we are considering to upgrade our Windows build requirements to Visual Studio 2022 and C++17. Appreciate your comments on that.

Closing the issue as a fix was provided in v<IP_ADDRESS>.


Hotfix - Top blocked not showing up in FF

Reviewer: @jdorweiler

Description:
A last minute change during the previous release resulted in the top blocked stats being always hidden in FF.
This PR fixes that.
Since it's a hotfix I also bumped the manifest pin so we can release to FF immediately.
Steps to test this PR:

build and reload FF extension
initially no top blocked line is shown
after 30 page loads top blocked should show up

Automated tests:

[ ] Unit tests
[ ] Integration tests

Reviewer Checklist:

[ ] Ensure the PR solves the problem
[ ] Review every line of code
[ ] Ensure the PR does no harm by testing the changes thoroughly
[ ] Get help if you're uncomfortable with any of the above!
[ ] Determine if there are any quick wins that improve the implementation

PR Author Checklist:

[ ] Get advice or leverage existing code
[ ] Agree on technical approach with reviewer (if the changes are nuanced)
[ ] Ensure that there is a testing strategy (and documented non-automated tests)
[ ] Ensure there is a documented monitoring strategy (if necessary)
[ ] Consider systems implications

👍


docker pull returns manifest unknown

Hi,
Is there something specific that needs to be done in order for this to work?
docker pull ghcr.io/alex3025/ilo-fans-controller:latest
latest: Pulling from alex3025/ilo-fans-controller
manifest unknown

Thanks

Had to manually recreate the docker images, now it's working.

Ah, great! Thank you


soc: stm32: update soc config flag to reflect flash size variant

In order to to be able to distinguish flash size variants of SoCs when needed (for instance in dts/arm/st/mem.h file), rename some CONFIG_SOC_SMT32XXX flags

Actually I meant stm32f429zi since this is the exact soc in use in stm32f4-disco board.
Same for stm32l476rg used in nucleo_l476rg board in next commit.
For stm32f407xg, I kept the x since soc is not actually used in any board right now, it could be any package variant and I kept the mystery.

Ok, so little confused if stm32f429zi than why use stm32f429xi in code?

In STM32 SoCs naming convention, first letter after chip ('z' in stm32f429zi case) is the pin package variant (32 pin, 64 72, 81, 100, 132, 144, 169....).
Since first stm32 porting this information was removed. I don't think zephyr cares about this right now, and keeping the information will add complexity in configuration, so I think we can  keep it that way.


[notification] sendNotification undefined is not a function

Hi, I get this error when doing the sendNotification function
TypeError: undefined is not a function (near '...window.__TAURI__.invoke...')
sendNotification
(anonymous function)
sendNotification — index.ts:356
(anonymous function) — Nav.tsx:100

this is my code
          <button onClick$={async () => {
              const perm = await isPermissionGranted();
              if (!perm) await requestPermission();
              try {
                sendNotification({
                  title: 'SimplyMC',
                  body: 'Hello Tauri!',
                });
              }
              catch (e) {
                console.error(e);
              }
          }} class="group transition ease-in-out hover:bg-gray-900 hover:text-white px-4 py-2 rounded-lg items-center hidden sm:flex">
            v{version} ({tauriVersion})
          </button>

Thanks for the report! That should be a fairly easy fix, but until then you can enable withGlobalTauri in tauri.conf.json and it should work until a proper fix lands :)

fixed by https://github.com/tauri-apps/plugins-workspace/pull/414


useNativeDriver required

Error: "Animated: useNativeDriver was not specified. This is a required option and must be explicitly set to true or false"

For anyone that wants to fix this, go to the Animator and add the useNativeDriver: true then fix the styling for the Animated.View from {...this.position.getLayout(), left: 0} to {transform: [{translateY: this.position.y}]}

After setting {transform: [{translateY: this.position.y}]}, useNativeDriver: true, I still got this error.
Error: AnimatedValue: Attempting to set value to undefined

This fix no longer works with the update deprecating super. I plan on working on a fix when I have time.

Is there already a solution for this bug? I am currently facing the same issue.

I redid it with a functional component, I didn't use all the features but it should without faults: https://github.com/NTakemori/react-native-bottom-drawer


A road to bunker through tunnel

Issue summary
Hi,
i dunno if its a bug or a feature tbh ^^
Description of issue:

No ramparts on road from tunnels to behind first line of ramparts
Steps to reproduce:

You can see the problem here : https://screeps.com/a/#!/room/shard0/W28S88
Look at the bottom of the room, i had to manually add the 3 ramparts at the tunnel end.

This is a bug -- the rampart placement code was written before tunnels were a thing and I haven't updated them yet. I'll try to patch in the near future, but since it's pretty rare, it's not very high priority.


Feat/submit mutation

db3js build configuration

This PR exceeds the recommended size of 1200 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.


Interpolate default changed for VectorTile source

Describe the bug
VectorTile sources have interpolate disabled by default since v6.10.0
To Reproduce
Steps to reproduce the behavior:
Interpolate enabled: https://openlayers.org/en/v6.9.0/examples/osm-vector-tiles.html
Interpolate disabled: https://openlayers.org/en/v6.10.0/examples/osm-vector-tiles.html
Interpolate is set to false by default here:
https://github.com/openlayers/openlayers/blob/e98607e3448338c08e6e4e11ec963a622c1cef4b/src/ol/Tile.js#L132
Expected behavior
Vector tiles should be rendered with image smoothing enabled by default.

The setting is only used for raster and data sources: ol/source/TileImage, ol/source/DataTile, ol/source/Image and their subclasses.

Which does not change the fact that the behavior has changed for VectorTile layers, which use the tile image renderer for rendering lines and polygons in renderMode: 'hybrid'.

You are right, @mike-000. I updated the linked source.

It seems both interpolate options are used and the one for the Tile does not matter, unless e. g. an icon would be drawn to the Tile. For the VectorTile source it should only draw lines and polygons, so it is irrelevant.
Adding interpolate: true to the VectorTile source seems to behave as in v6.9.0 and will draw the rendered tiles with imageSmoothing.


Naming Conventions for attribute names in ArangoDB

Hi,
I found one limitation in the documentation.
Attribute names should not start with the at-mark (@). The at-mark at the start of attribute names is reserved in ArangoDB for future use cases.

I need to make sure whether could we use @ in the attribute names. Certainly, it's supported now in the db. However, if it won't in the future version, that would be a problem.
Why can not use other character for future use cases? As I know @ in JSON is much more important than other reserve keywords. e.g., JSON-LD uses always attribute names starting with @.
Bg,
Kevin

I'm not aware of any plans to give special semantics to attributes starting with @. I'm not sure whether that line in the documentation is still accurate. Maybe @fceller or @jsteemann could chime in with a more normative opinion?

I remember we discouraged the usage of @ in the docs, and we planned to use @-prefixed attributes in future versions for special purposes.
That was long ago already, and we did not come up with any concrete plans about such attributes yet. From my point of view, this point of the documentation is outdated and can be changed so any @-prefixed attributes can be used, and will not have special meanings in the future.
However, I'd like @fceller or @neunhoef to confirm before removing this from the docs and encouraging the use of @-prefixed attributes.

Thx a lot for your feedback. Then I can use that now.

I cannot think of a good usage for these attribute names. One question: Would the existence of these attributes pose difficulties in AQL? For example in
FOR x IN coll
FILTER x.@name == "Max"
RETURN x
Would this work? Or would AQL take @name as a query parameter?

Attribute names can be used outside of AQL, e.g.:
{"foo":"bar","@name":"bla"}. In this case, @name would be a regular attribute name.
In AQL, a literal @ can still be used inside an attribute name if the attribute name is enclosed in backticks.

I would not encourage the use of '@'

To clear the confusion that may have arisen, I just discussed that with @fceller:

attribute names starting with @ are supported by ArangoDB and should work, so they can be used by end users if required
the section about them being reserved for future use cases has been removed from the manual (at least in the source code, the online manual will be updated eventually).
we still do not encourage the usage of @-prefixed attribute names because in AQL, identifiers starting with the at-mark will be interpreted as bind parameters. To use an at-mark-prefixed attribute name in AQL, it will need to be enclosed in backticks. The same is true for other attribute names by the way, e.g. attribute names including spaces.

Long story short: at-prefixed attributes can be used, though they will need some attention when used in AQL queries.
I think this should provide everyone with enough information to make a good decision about whether it is worth to use such attribute names in their applications.


Questions about VideoIntro & initializing game

I've just been studying the use of the engine for 2 days,I found this engine pretty good.
But I have 2 questions to ask about the author of the engine:
1.When I learned "VideoIntroSample":
    public static class MySceneFactory extends SceneFactory {         public IntroScene newIntro() {             return new VideoIntroScene("1.mp4");         }     }
I don't know which path the MP4 resource is required for "VideoIntroScene"
2.Every time you load a game scene, there will be a progress bar(Black interface),I want to add a picture to the Loading interface(initializing game),what should I do?
Thanks to the handsome and kind-hearted author( ͡° ͜ʖ ͡°)

No worries, I appreciate your interest and your questions will help me write a better documentation on how to use FXGL.

The video file should be placed under assets/video/, so in your case it would be assets/video/1.mp4.

Have a look at this directory. There are 2 files, one shows to how to design your custom loading scene, the other shows how to use your custom scene.

Ah,I got it! It's amazing
Thank you very much

Hello,handsome author,i am finding this directory,but it is disappear.can u tell me where can i get?and i want to learn how to load video into my game start for introducing the game of backstory,what should i do?

@AlmasB

@lll-hue Hi, I've just added the functionality. Here's an example:
public class VideoSample extends GameApplication {
    @Override
    protected void initSettings(GameSettings settings) {
        settings.setWidth(1280);
        settings.setHeight(720);
    }

    @Override
    protected void initUI() {
        // testvideo.mp4 must be located in /assets/videos/
        var video = getAssetLoader().loadVideo("testvideo.mp4");
        video.setFitWidth(getAppWidth());
        video.setFitHeight(getAppHeight());
        video.getMediaPlayer().play();

        addUINode(video);
    }

    public static void main(String[] args) {
        launch(args);
    }
}

You will need the latest dev build to use this:
<dependencies>
    <dependency>
        <groupId>com.github.almasb</groupId>
        <artifactId>fxgl</artifactId>
        <version>17+dev-SNAPSHOT</version>
    </dependency>
</dependencies>

<repositories>
    <repository>
        <id>oss.sonatype.org-snapshot</id>
        <url>https://oss.sonatype.org/content/repositories/snapshots</url>
    </repository>
</repositories>

Thanks for your reply，this is helpful to me，i am learning this engine。
Hope you have a good day


Placeholder: Hotfix for CRA/SIN File Exchange

SIN Validation Request

[x] FIX IN #3841: Invalid header format<PHONE_NUMBER>120241023BC. There’s 8 characters, but should be 6 characters only.

[x] Update batch number to "6" from "8"
https://github.com/bcgov/SIMS/blob/main/sources/packages/backend/libs/integrations/src/esdc-integration/sin-validation/sin-validation-files/sin-validation-file-header.ts#L38

[x] Fix e2e tests

Income Verification Request

[x] FIX in HOTFIX 1.14.1 DEPLOY: CCRA request file that is in production does not have the correct header/trailer record – it contains a “BCSLx” identifier – should be “BCSAx” instead.

[x] Update cra-program-area-code env variable to "BCSA" from "BCSL" - says "BCSL" in federal documentation

[x] Fix to archive file with new expected name

SIN Validation Response

File successfully processed - Oct 31, 2024

Income Verification Response

[x] FIX in HOTFIX 1.114.1 DEPLOY: Missing income record from CRA due to invalid program code- investigate if we are inspecting cra-program-area-code in header
[x] Error when archiving file - review archiving rules.

Closing as completed in other tickets.


math/big: (*Rat).SetString("1234") not the same as big.NewRat(1234, 1)

What version of Go are you using (go version)?
1.11.2
Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GOHOSTARCH="amd64"
GOHOSTOS="linux"

What did you do?
func main() {
	var x, y big.Rat

	x = *big.NewRat(123456789, 1)
	y.SetString("123456789")

	fmt.Printf("%#v\n", x)
	fmt.Printf("%#v\n", y)
}

https://play.golang.org/p/yFKM2k6djNA
What did you expect to see?
I would expect to see both structures be identical.
big.Rat{a:big.Int{neg:false, abs:big.nat{0x75bcd15}}, b:big.Int{neg:false, abs:big.nat{}}}
big.Rat{a:big.Int{neg:false, abs:big.nat{0x75bcd15}}, b:big.Int{neg:false, abs:big.nat{}}}

What did you see instead?
While the same numerical value, they are different for the purposes of test assertion.
big.Rat{a:big.Int{neg:false, abs:big.nat{0x75bcd15}}, b:big.Int{neg:false, abs:big.nat{}}}
big.Rat{a:big.Int{neg:false, abs:big.nat{0x75bcd15}}, b:big.Int{neg:false, abs:big.nat(nil)}}

As a side note, you should not write x := *big.NewRat(...).  Shallow copies of *big.Rat values are not supported.  This will be more clearly documented in the upcoming 1.12 release (https://tip.golang.org/pkg/math/big/#Rat).
To address the main issue, you should compare *big.Rat values using the Cmp method.  The package makes no promise that values that compare as equal use the same internal representation.


Task Status Exporter

Hi,
Use case: Task status is exported on task completion.
Is there a way to export Task completion status with the following payload:
{
workFlowId:"",
taskId:"",
taskRefName:"",
payload:"${$workflow.input.*}"
}

One way is to maintain a registry of the tasks running and poll Conductor for their statuses. Since this is not the best approach, what other options do we have?

IIUC, your use-case is to be notified using eventing when a task has moved to a terminal status? Is that correct?
This is a use-case that we are looking to add in a future release and the implementation should be similar to that of the WorkflowStatusListener.

IIUC, your use-case is to be notified using eventing when a task has moved to a terminal status? Is that correct?
You got it right. Future Release, okay.

Hi,
We are planning to implement this feature if no one else is working on this. Basically, the design approach would be similar to WorkflowStatusListener. We are also considering adding a onWorkflowStarted method to the WorkflowStatusListener to help notify kick-off of workflow execution.
Please let us know if this is ok with the community. If so, we should be able to get a PR up hopefully.

@optimal-rhythm I believe this is not being actively worked on. Please feel free to create a PR for this. Please also add documentation for this feature as part of the PR.

thanks @apanicker-nflx ! will do


Update CODEOWNERS to align with new MR review process

Migrated from GitLab: https://gitlab.com/meltano/sdk/-/issues/337
Originally created by @pandemicsyn on 2022-02-28 18:20:56

Per https://gitlab.com/meltano/meta/-/issues/212 , @edgarrmondragon will join @aaronsteers as one of the two primary reviewer/approvers. Since the primary owner can't approve their own work, they'll use the secondary owner as default reviewers for their own MRs. @edgarrmondragon will require approval from @aaronsteers and vice-versa.

View 4 previous comments from the original issue on GitLab


octoeverywhere integration

hi =)
this is not an issue but more a question.
Octo everywhere has a cool feature called "Share Connection" that allows you to create a link that allows apps to connect to Octoprint via Octoeverywhere, in order for example, to check your printer while you are not at home, using your smartphone.
Is there a chance to implement a way to let the printer monitor connect to octoprint using the Share Connection feature?
This feature would let the monitor be able to show the printing infos even if it isn't connected to the same wifi that's octoprint is connected to.
I tried to paste the shared link into the Octoprint Address, and the API from my octoprint API.
Clicking on the test button it seems that the web interface is able to retrieve the informations, but on the main page, in the status field, there is a 301 server error message.

Looking at the plug-in you would need a https connection which I don’t think printer-monitor has programmed into it.


Value is not updated on duplicate write

Influx Version: 0.12
OS: Ubuntu 14.04
In some situations, overwriting a value for the same "point" does not work.
I recognized this behaviour in a real situation when I tried to correct some values on a downsampled measurement. When I tried to override a value on a plain fresh new measurement, everything worked fine, but doing so on a measurement that was in daily usage, it did NOT work.
As an example, i picked a single record in an existing measurement:
This is a kind of a dump what the PHP client lib was really doing:
HTTP Write URL: /write?db=user&precision=s&rp=INF
Payload (Represents lines in Line Protocol)
Array
(
[0] => user_action-1d-#country#gender#client,country=US,gender=2,client=232 match=10000i,value=1i<PHONE_NUMBER>
)
BEFORE:

SELECT * FROM "user"."INF"."user_action-1d-#country#gender#client" WHERE country='US' AND gender='2' AND client='232' AND time > '2016-04-30'
name: user_action-1d-#country#gender#client

time			became_active	became_inactive	block_received	block_sent	client	country	dislike_received	dislike_repeat	dislike_sent	gender	like_received	like_sent	magic_moment	match	moment_like	moment_upload	photo_upload	visit_received	visit_sent
1462060800000000000	164		281		375		864		232	US	206497			5735		102586		2	184335		23892		131		3522	6555		2072		1052		81332		56839
AFTER:

SELECT * FROM "user"."INF"."user_action-1d-#country#gender#client" WHERE country='US' AND gender='2' AND client='232' AND time > '2016-04-30'
name: user_action-1d-#country#gender#client

time			became_active	became_inactive	block_received	block_sent	client	country	dislike_received	dislike_repeat	dislike_sent	gender	like_received	like_sent	magic_moment	match	moment_like	moment_upload	photo_upload	value	visit_received	visit_sent
1462060800000000000	164		281		375		864		232	US	206497			5735		102586		2	184335		23892		131		3522	6555		2072		1052		1	81332		56839
Expected:
Field "match" should be "10000". Actually it did not change from 3522 to 10000. The query itself was executed as can be observed by changing the value for field "value".
This example is not bound to a special field nor to a special time-range. It happens on all points on all kind of measurements in our production system. In contrast, just to repeat, it worked on a plain, fresh measurement with only a few test points.

Dup of #6439


HelperErrorText and its aria-* props

Component in question:

HelperErrorText

Description of issue:

HelperErrorText should read its ariaLive and ariaAtomic props and also allow assertive as an option.

Fixed in #275. Closing.


"Error: Can't find interface/stlink.cfg in procedure 'script'"

Hai, I am working with ST Nucleo-H743ZI board and Zephyr micro-ROS. While trying to flash the firmware an error occurred. The terminal displays an error message:
"Error: Can't find interface/stlink.cfg in procedure 'script' "

How can I solve this?
Thank you.
Sachin

Hello @sachinjairam, could you try to flash the device removing this line. I do not have this board available here, so I cannot try how to fix this.
@CLansdaleBEC wrote the support for this board, any idea on that?

Hai @pablogs9 Thanks for your reply. I tried it, and the old error gone and new one arise,
Flashing firmware for zephyr platform nucleo_h743zi
Bus 002 Device 020: ID 0483:374e STMicroelectronics STLINK-V3
Open On-Chip Debugger 0.10.0
Licensed under GNU GPL v2
For bug reports, read
http://openocd.org/doc/doxygen/bugs.html
embedded:startup.tcl:60: Error: Can't find target/stm32h7x.cfg
in procedure 'script'
at file "embedded:startup.tcl", line 60

You can check if it works without openocd, set this to false and let see if west flash does the work.

@pablogs9 Its worked. Thank you.
But what will be the problem with openocd?
Can you tell me please?
Thank you
Sachin


Copilot Chat: Document Ingestion

Motivation and Context

To demonstrate the ability of the Copilot Chat App to ingest documents to memory and query against those documents.
Description

Create DocumentSkill that can be invoked to ingest documents to memory and process queries coming from the chat.
Create a simple console app called the document upload app to ingest documents to the app.

Contribution Checklist

[ ] The code builds clean without any errors or warnings
[ ] The PR follows SK Contribution Guidelines (https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md)
[ ] The code follows the .NET coding conventions (https://learn.microsoft.com/dotnet/csharp/fundamentals/coding-style/coding-conventions) verified with dotnet format
[ ] All unit tests pass, and I have added new tests where possible
[ ] I didn't break anyone :smile:

Align casing for "DocumentUploadApp" with "webapp" and "webapi"

I'm not to happy with this Config class. The name should be more descriptive, like ImportDocumentConfiguration.
Further, the Config.GetConfig() method should not hardcode where it gets the configuration from. It should use an IConfigurationSection instead.

I'm not to happy with this Config class. The name should be more descriptive, like ImportDocumentConfiguration.
Further, the Config.GetConfig() method should not hardcode where it gets the configuration from. It should use an IConfigurationSection instead.
The configuration should also be validated using Options Validation

We're in the midst of refactoring CopilotChat's configuration to follow the Options pattern. Stay tuned!


[v5] Consistent renaming: options & definition

Superseded by https://github.com/statelyai/xstate/pull/4080


istiod not running

Hi, I'm testing cilium with istio but I have some problem when install istio. When I run cilium-istioctl, I see istiod not running, I checked logs of istiod. Below:

How to fix this problem ?
Thanks for watching

@jrajahalme can you take a look?

@thanhlong9x Please update this issue with steps to reproduce if you still have this problem.

@jrajahalme I'm sorry, I can't reproduce this issue. I guess cause from network k8s, I often meet error time-out when get k8s service (<IP_ADDRESS>). I often reset network to resolve althought I don't know reason of problem.

I'm closing this issue, please open a new one if the problem happens again.


M3-1875 Create volume for Linode.

Purpose
Use the global drawer for creation of a volume and immediate attachment to a Linode.
Notes
This is only creation. The next PR will add the attach to Linode flow back.
Goldplatting
I'll be submitting a final clean-up PR to bring this component in-line with our current patterns at the end of the review. If anything jumps out, let me know so I can include it in the final.

Coverage decreased (-0.006%) to 30.871% when pulling ed5126e041166de5f6f03d70ce169a290948e282 on rjstires:M3-1875-unified-volume-creation into 5f0fc6fd6e81fd3521e37126f94aa534326578f0 on linode:feature/unified-volumes.


Migrate packages to by-name

Description of changes

joshuto: migrate to by-name
just: migrate to by-name
cargo-make: migrate to by-name
wit-bindgen: migrate to by-name
sleek: migrate to by-name
sniffglue: migrate to by-name
eksctl: migrate to by-name
api-linter: migrate to by-name

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Please read the by-name README which says that packaged should not be migrated manually unless there's refactoring at the same time. All the eligible package will be migrated automatically later.

In particular here. Note that there's nothing inherently wrong with doing this, it's just that we don't want everybody to think this is required, thanks @Artturin :)


sql: CASE expressions

From Gitter:
Nathan Johnson @nathanejohnson Oct 26 21:19
another thing I notice is that case statements appear to work, even though I couldn't find any documentation
I'm doing a poc retrofitting an existing application to use cockroach
Andrei Matei @andreimatei Oct 26 21:25
looks like our SELECT documentation, for example, is a bit light on the expressions that can be used.
CASE is only mentioned in our grammar dump
https://www.cockroachlabs.com/docs/sql-grammar.html#case_expr

Covered in @knz's expressions docs:

https://www.cockroachlabs.com/docs/sql-expressions.html#simple-case-expressions
https://www.cockroachlabs.com/docs/sql-expressions.html#searched-case-expressions


Make SigmaR take array(s) as arguments

I want to be able to do something like this
ccl.sigmaR(C,8/0.7,np.array([1,0.5,0.3]))

Also for the R parameter or both (in which case it only makes sense if values are associated pairwise.

This is an example of a broader issue in the library FWIW. Numpy's API typically likes anything that can be broadcasted together, but IDK if we want to support that.

It takes R as an array, just not the scale factor. I actually think we don't want to fix this given the current API.

Are we okay to close this? @beckermr @slosar

i'm fine but @slosar should weigh in

If it is OK with becker it is ok with me.  I'm closing this.


NIFI-9607: Honor Update Keys when Quoting Identifiers in PutDatabaseRecord

Thank you for submitting a contribution to Apache NiFi.
Please provide a short description of the PR here:
Description of PR
Update Keys weren't being used when Quoted Identifiers were set. Refactored and added unit test.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[x] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[x] Has your PR been rebased against the latest commit within the target branch (typically main)?

[x] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[ ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[x] Have you written or updated unit tests to verify your changes?
[ ] Have you verified that the full build is successful on JDK 8?
[x] Have you verified that the full build is successful on JDK 11?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check GitHub Actions CI for build issues and submit an update to your PR as soon as possible.

@mattyb149 Hi, we have hit the same problem recently and were also going to think about PR, so many thanks for your work.
I would like to clarify one thing if you mind and maybe suggest some additional fixes.
We had the problem with generateUpdate function.
And... I am not sure it is addressed here.
According to the code ( @@ 1135 in your version) there's a step when all keys are being normalized by normalizeKeyColumnNamesAndCheckForValues which also performs quotation if it is set by parameters.
However, on  @@ 1170 we have
                // Check if this column is an Update Key. If so, skip it for now. We will come
                // back to it after we finish the SET clause
                if (!normalizedKeyColumnNames.contains(normalizedColName)) {

I.e. normalizedKeyColumnNames contains quoted identifiers, while normalizedColName is obtained by normalizeColumnName here, which does no quotation.
Thus, during this step Key columns will still be included into SET block of  UPDATE statement, which is syntactically wrong.
What do you think?

Will review

I tested out these changes and they look good to me. I wasn't sure if we want to include the changes mi-volodin suggested as well.

@mi-volodin I didn't see the same behavior you describe, but did find a bug. I debugged the unit test and normalizeKeyColumnNamesAndCheckForValues doesn't quote the column names, just normalizes it by uppercasing it and removing underscores and such. Also the normalized column name it matches against isn't quoted, it should only be quoted when generating the SQL. However it was using normalizedColumnName as the column name in the WHERE clause when it should be using desc.getColumnName() like the SET clause does. That can cause issues, I will issue another commit to update the unit test to illustrate this, along with the fix to get the test to pass :)

+1 will merge. If we can get more information/a specific case where the issue mentioned by mi-volodin we can create another Jira to resolve it.