Added Dockerfile for 1.9.2+@47bc480 on armv7l (>= Raspberry Pi 2B) on Alpine

Support rpi32 build target in build.sh

Thank you for this PR.
As of VerneMQ 1.10 we changed how the docker images are built (due to the introduction of the EULA). As a consequence this PR can't be applied anymore in that form. But we consider to maintain Docker images for other architectures than we do today in the future.
Best!


configure containers registries in CRI-O

Right now, if you pass an image name unqualified (meaning, with not hostname/port in it, e.g. docker pull fedora), CRI-O assumes docker.io. This was the original behavior introduced with docker back then. To not break existing deployments, we'd like to have a configuration to setup additional and blocked registries in order to assume other registries (not docker.io) and block certain one (think about heavy locked deployments where developers aren't allowed to pull from docker.io forinstance).

This should be moved to containers/image or containers/storage?  It should be in a config file provided by them.

I have a branch for this, it will be done directly in cri-o, containers/image and storage will always be given the fully qualified image name.

Ok, Only issue I would see if whether or not we wanted buildah to have the same support.


ProgressIndicatorOrbsRotator animation get stuck after the first Close

Describe the bug
when using the ProgressIndicatorOrbsRotator prefab, if we show the progress bar several time using the OpenAsync and CloseAsync function, the animation is only shown on the first run.
It seems that this comes from the hasAnimationFinished preoperty which is set to true when CloseAsync function is called, but is not reset to false in OpenAsync.
To reproduce
Add the ProgressIndicatorOrbsRotator prefab to you scene.
Call OpenAsync, and then after some delay CloseAsync.
Call OpenAsync a scond time. The animation doesn't run.
Expected behavior
The animation should run multiple times
Your setup (please complete the following information)

Unity Version 2019.3.1f
MRTK Version 2.3.0

Target platform (please complete the following information)

HoloLens 2

Additional context
Do I miss use the prefab ? If yes what should be the proper way to use it ?
Thanks

Do I miss use the prefab ? If yes what should be the proper way to use it ?

This is a bug, it's been fixed in #7561


[Docs] Update README regarding Traefik access log IPs

Problem
Like many others, #13, #23, #25, #29, #30, I've been trying to use this plugin, in particually with a Crowdsec  instance which uses the Traefik access logs.
While the plugin does exactly what it says it does, fixing the headers, many (like myself) incorrectly assume that the plugin will also fix the access log with the actual user IP instead of the CloudFlare IP, which this plugin does not.
In order to have the user IPs instead of CloudFlare IPs in the logging, the idea is to set trustedIPs under the forwardedHeaders, see docs.
An example would be as follows:
static.yml
entryPoints:
  websecure:
    forwardedHeaders:
      trustedIPs:
        - <IP_ADDRESS>/20
        - <IP_ADDRESS>/22
        - <IP_ADDRESS>/22
        - <IP_ADDRESS>/22
        - <IP_ADDRESS>/18
        - <IP_ADDRESS>/18
        - <IP_ADDRESS>/20
        - <IP_ADDRESS>/20
        - <IP_ADDRESS>/22
        - <IP_ADDRESS>/17
        - <IP_ADDRESS>/15
        - <IP_ADDRESS>/13
        - <IP_ADDRESS>/14
        - <IP_ADDRESS>/13
        - <IP_ADDRESS>/22
        - 2400:cb00::/32
        - 2606:4700::/32
        - 2803:f800::/32
        - 2405:b500::/32
        - 2405:8100::/32
        - 2a06:98c0::/29
        - 2c0f:f248::/32

or if you use environment variables for static configuration:
.env
TRAEFIK_ENTRYPOINTS_WEBSECURE_FORWARDEDHEADERS_TRUSTEDIPS="<IP_ADDRESS>/20,<IP_ADDRESS>/22,<IP_ADDRESS>/22,<IP_ADDRESS>/22,<IP_ADDRESS>/18,<IP_ADDRESS>/18,<IP_ADDRESS>/20,<IP_ADDRESS>/20,<IP_ADDRESS>/22,<IP_ADDRESS>/17,<IP_ADDRESS>/15,<IP_ADDRESS>/13,<IP_ADDRESS>/14,<IP_ADDRESS>/13,<IP_ADDRESS>/22,2400:cb00::/32,2606:4700::/32,2803:f800::/32,2405:b500::/32,2405:8100::/32,2a06:98c0::/29,2c0f:f248::/32"

IPs in the above examples are available from https://www.cloudflare.com/ips/.

Suggestion
Minor addition to the documentation to state that this plugin only fixes the headers and that additional steps have to be taken (like the examples denoted earlier) to get the actual IP changes in the access logs.

https://github.com/traefik/traefik/blob/1ffbffb26a3676b368c4fc52c4fb737de56cd65d/pkg/config/static/entrypoints.go#L19
So, I think the big thing here is the issue is because traefik is using the entrypoint for logging, and not after a middleware (it occurs before).
We also don't want to be doubling config and specifically denoting to trust those IPs because realistically that shouldn't work with CF?
reason: CF do not use the default headers, so the entrypoint shouldn't be able to parse it correctly.
mmmm..... I am going to add some tests with this info later this week and will see what I can find to make it better.

@mrinc have you made any progress on this issue?

We also don't want to be doubling config and specifically denoting to trust those IPs because realistically that shouldn't work with CF?
reason: CF do not use the default headers, so the entrypoint shouldn't be able to parse it correctly.

It works because they do support the default headers as well, although I do not know how Traefik handles multiple addresses in this field.  See https://developers.cloudflare.com/fundamentals/reference/http-request-headers/#x-forwarded-for

@ToshY haven't done anything yet sorry... been a bit busy this side.
Happy to look at a PR if you got any ideas.
@sdwilsh but we also can't trust the x-forwarded-for since it could be manipulated by the client.
So for safety, we should stick with the verification methods provided by CF.
Ideally we want to do it before traefik actually logs the request... somehow

So looks like we can't really make something to intercept ... but we can do this:  https://doc.traefik.io/traefik/observability/access-logs/
So you can add the CF-Connecting-IP - the issue I have right now is I have not done enough research, but if this access log occurs before the middleware, then that is it - however if it occurs after , then we can also include the trusted state.

Not sure if it was one of you guys, but there is some docs on how to do this here:  https://www.reddit.com/r/selfhosted/comments/1dcn19v/standing_up_the_crowdsec_bouncer_plugin_in_traefik/
I will leave this issue open to add some docs on this.

For some reason, when I added CF IPs or trusted all IPs with
forwardedHeaders:
  insecure: true

I got IPv6 addresses in the logs, even though I use only IPv4 with traefik. When I removed the plugin, I got proper CF endpoint IPv4 addresses. So something is definitely not right in the setup and the fact you get incorrect IPs in the logs is just plain user-unfriendly. This should've been fixed long time ago. There's no point in having a plugin like this one without properly logged remote IPs.


A rewrite in Sass

The changes in this PR
So I decided to rewrite OlDesign to refine some stuff and shave off unneeded stuff, and thought a rewrite in Sass would be a good idea.
Pros:

I can split the stylesheet into multiple files, keeping each more focused (for example, a file containing all the variables, making it easier to customise for the end user).
Sass is easier on the eyes, thanks to the use of indentation rather than curly braces.
the Sass rewrite is somehow smaller than the original, but hey I'm not complaining.

Cons:

Users would have to learn Sass (and many prefer to use SCSS instead since it's quite literally a superset of CSS and a large population of people prefer curly braces instead of indentation)
Users would have to install Sass to compile the output and use it.
multiple files also complicates the project a bit, since one would have to know which file each new property should go in; that is basically a line in the sand at this point, and can cause a lot of future disputes if multiple developers ever pick this codebase up.

Conclusion
I'll keep working on this, and will hopefully come to a decision soon on whether this isa good idea or not.

So I've added a few more features to bring OlDesign to what i would consider feature completeness, and since it's still smaller than the original, I have decided to merge it.


Could support pure js in *.rt like the _.template() way?

That is more the JSX way of doing things.
Can you give an example for how this will be useful?


Don H. is not on the leaderboard

On Jan 3, 2019 3:38 PM, Donald H.  wrote:

I dont see me on leader board. I did all you mentioned. Thanks don.

Looks like Don's Strava ID is 3171830 and he is showing up in the database as being a member of the main team. But
Looks like you haven't logged a qualifying ride yet.
https://www.strava.com/athletes/3171830
I only see:

Indoor / trainer / stationary bike rides don't count. Log an outdoor bike or e-bike ride and you should show up.


Fix bill short title encoding issues

In this PR I've fixed the short title encoding issues that reared their ugly heads in DigiPol.
In loving memory of �€™

Is there a reason why we're not using regex to remove characters that are not alphanumeric and valid white spaces?

Is there a reason why we're not using regex to remove characters that are not alphanumeric or valid white spaces?

Personally it's because I haven't learnt how to use regex properly yet, plus this works, so make of that what you will.

@KipCrossing Could you please investigate why CircleCI is stuck?

@connorhsm found it. I thought it would be in the circleci.yml file but it's in the GUI. May have to push a new commit to run it @king-millez

Also, feel free to use regular expressions. Using .replace() was just a quick fix that blew out of hand

@king-millez magic commit, lol

Nice, everything working now?


[BUG] GpuBroadcastHashJoinExec ArrayIndexOutOfBoundsException

Describe the bug
When performing a join of two tables created with a PARTITIONED BY clause using the following code, a GPU projection fails with ArrayIndexOutOfBoundsException due to receiving a broadcast batch with zero columns. The same code works fine on CPU, indicating that there is a bug in the plugin.
The query works if the PARTITIONED BY clause is removed from the CREATE TABLE statement for t1.
The failure happens with AQE on or off.
Steps/Code to reproduce bug
Run the following unit test.
test("Join partitioned tables") {

  withGpuSparkSession(spark => {
    import spark.implicits._

    val path = new File(TEST_FILES_ROOT, "test.parquet").getAbsolutePath
    (0 until 100)
        .map(i => (i,i*5))
        .toDF("a", "b")
        .write
        .mode(SaveMode.Overwrite)
        .parquet(path)
    spark.read.parquet(path).createOrReplaceTempView("testData")

    spark.sql("DROP TABLE IF EXISTS t1").collect()
    spark.sql("DROP TABLE IF EXISTS t2").collect()

    spark.sql("CREATE TABLE t1 (a INT, b INT) USING parquet PARTITIONED BY (a)").collect()
    spark.sql("CREATE TABLE t2 (a INT, b INT) USING parquet PARTITIONED BY (a)").collect()

    spark.sql("INSERT INTO TABLE t1 SELECT a, b FROM testData").collect()
    spark.sql("INSERT INTO TABLE t2 SELECT a, b FROM testData").collect()

    val df3 = spark.sql(
        "SELECT t1.a, t2.b " +
        "FROM t1 " +
        "JOIN t2 " +
        "ON t1.a = t2.a "
        )
    df3.collect()

  })
}

Stack trace:
Caused by: java.lang.ArrayIndexOutOfBoundsException: 0
	at org.apache.spark.sql.vectorized.ColumnarBatch.column(ColumnarBatch.java:102)
	at com.nvidia.spark.rapids.GpuBoundReference.columnarEval(GpuBoundAttribute.scala:95)
	at com.nvidia.spark.rapids.RapidsPluginImplicits$ReallyAGpuExpression.columnarEval(implicits.scala:35)
	at com.nvidia.spark.rapids.GpuProjectExec$.$anonfun$project$1(basicPhysicalOperators.scala:50)
	at com.nvidia.spark.rapids.RapidsPluginImplicits$MapsSafely.$anonfun$safeMap$1(implicits.scala:160)
	at com.nvidia.spark.rapids.RapidsPluginImplicits$MapsSafely.$anonfun$safeMap$1$adapted(implicits.scala:157)
	at scala.collection.immutable.List.foreach(List.scala:392)
	at com.nvidia.spark.rapids.RapidsPluginImplicits$MapsSafely.safeMap(implicits.scala:157)
	at com.nvidia.spark.rapids.RapidsPluginImplicits$AutoCloseableProducingSeq.safeMap(implicits.scala:192)
	at com.nvidia.spark.rapids.GpuProjectExec$.project(basicPhysicalOperators.scala:49)
	at com.nvidia.spark.rapids.shims.spark301.GpuBroadcastHashJoinExec.builtTable$lzycompute$1(GpuBroadcastHashJoinExec.scala:147)
	at com.nvidia.spark.rapids.shims.spark301.GpuBroadcastHashJoinExec.builtTable$1(GpuBroadcastHashJoinExec.scala:145)
	at com.nvidia.spark.rapids.shims.spark301.GpuBroadcastHashJoinExec.$anonfun$doExecuteColumnar$5(GpuBroadcastHashJoinExec.scala:162)

Expected behavior
The code should work when on GPU. Changing withGpuSparkSession to withCpuSparkSession makes the test pass.
Environment details (please complete the following information)
N/A
Additional context
N/A

@revans2  @sameerz this seems like it could be pretty serious.  From the exception it looks like we tried to address a column index off the end of a batch, implying we're mismatching inputs and outputs between operators in the query.

This looks like it is a bug in the parquet reader for partitioned data.
The plan asks to read only the column that the data is partitioned by (The following is the CPU version).
FileScan parquet default.t1[a#32] Batched: true, DataFilters: [], Format: Parquet, Location: InMemoryFileIndex[file:/tmp/spark-warehouse8378085594252186462/t1/a=235, file:/tmp/spark-warehous..., PartitionFilters: [isnotnull(a#32)], PushedFilters: [], ReadSchema: struct<>

The GPU version is the same.  I'll see what I can do to trace it down in the parquet reader, but I suspect it is the same for all of the readers that we have.

If I set
spark.conf.set("spark.rapids.sql.format.parquet.multiThreadedRead.enabled", "false")

The test now passes. So it appears to be related to the small file work done for parquet.

Big thanks to @tgravescs who pointed me at the right code.  I have a patch that works, but I need to clean it up and add in a test.  Should hopefully not be too long.


[cdc_rsync] Enable local syncing

Adds support for local syncs of files and folders on the same Windows machine, e.g. cdc_rsync C:\source C:\dest. The main changes are

Skip the check whether the port is available remotely with PortManager.
Do not deploy cdc_rsync_server.
Run cdc_rsync_server directly, not through an SSH tunnel.

The current implementation is not optimal as it starts cdc_rsync_server as a separate process and communicates to it via a TCP port.

Thanks!


Have message like Querying [250 - 279] failed from time to time in logs

From time to time I see messages like shown below:
2024-12-04 11:20:14.163 WARNING (MainThread) [homeassistant.helpers.entity] Update of sensor.deye_solarman_pv1_power is taking over 10 seconds
2024-12-04 11:20:20.063 WARNING (SyncWorker_60) [custom_components.solarman.solarman] Querying [250 - 279] failed with exception [Empty: ]
2024-12-04 11:20:20.063 INFO (SyncWorker_60) [custom_components.solarman.solarman] Disconnecting from solarman data logger <IP_ADDRESS>:8899
2024-12-04 11:20:20.064 WARNING (SyncWorker_60) [custom_components.solarman.solarman] Querying [250 - 279] failed, [1] retry attempts left
2024-12-04 11:20:20.064 INFO (SyncWorker_60) [custom_components.solarman.solarman] Connecting to solarman data logger <IP_ADDRESS>:8899

It is ~81 fail
docker logs ea783a24a1c7 2>&1 | grep 'custom_components.solarman.solarman' | grep fail | grep '2024-12-04' | wc -l
81

Could you please provide me any suggestion where to start an investigation?
I checked network connectivity and did not find any packet loss with ping big packets
ping <IP_ADDRESS> -s 1472 -i 0.1 -n

Is it expected when the logger stops responding (data reading, not ping)  from time to time? Or should I replace the logger?

Hi, @sirmax123, what profile it is?

Hi, @davidrapan!
It looks like it is deye_hybrid.yaml

The data looks correct for the first view.
I created my own small script to test and also got exceptions that look like I see in logs
Here is part of code, and it also lost data from time to time with
``
2024-12-04 10:29:08,704 collect_data_therad [read_registers] pysolarmanv5.pysolarmanv5.NoSocketAvailableError: Sleeping for 75 seconds. Registers from 0 to 124, attempt: 9

 for read_number in range(1, self.inverter_registers_reads_number+1):
        read_attempts = self.max_read_attempts
        while read_attempts:
            try:
                if not modbus:
                    modbus = pysolarmanv5.PySolarmanV5(
                        self.stick_logger_ip, self.stick_logger_serial,
                        port=self.port, mb_slave_id=self.mb_slave_id,
                        verbose=self.verbose, logger=self.logger,
                        auto_reconnect=self.auto_reconnect,
                        v5_error_correction=self.v5_error_correction
                    )
                self.logger.info("[read_registers] Read attempt: {}, Read number: {}. Rading registers from {} to {}".format(
                        read_attempts,
                        read_number,
                        read_start_register,
                        read_start_register + self.max_number_of_registers_to_read_in_request -  1
                    )
                )
                inverter_read_raw_result = modbus.read_holding_registers(
                    register_addr=read_start_register,
                    quantity=self.max_number_of_registers_to_read_in_request
                )
                self.logger.debug("[read_registers] Read result (raw): {}".format(inverter_read_raw_result))
                self.inverter_read_raw_result_all_registers = self.inverter_read_raw_result_all_registers + inverter_read_raw_result
                read_start_register = read_start_register + self.max_number_of_registers_to_read_in_request
                self.logger.info("[read_registers] Finished reading registers from {} to {}".format(
                        read_start_register,
                        read_start_register + self.max_number_of_registers_to_read_in_request - 1
                    )
                )

                break
            except pysolarmanv5.pysolarmanv5.NoSocketAvailableError as E:
                random_sleep = random.randint(0, self.sleep_on_inverter_read_error)
                sleep = self.sleep_on_inverter_read_error + random_sleep
                self.logger.error("[read_registers] pysolarmanv5.pysolarmanv5.NoSocketAvailableError: Sleeping for {} seconds. Registers from {} to {}, attempt: {}\n {}".format(
                        sleep,
                        read_start_register,
                        read_start_register + self.max_number_of_registers_to_read_in_request - 1,
                        read_attempts,
                        E
                    )
                    ....

So, can it be something wrong with the wifi stick?
 Is there any reason to use an ethernet (wires, not wireless) stick instead? I can purchase one but not sure is it supported.
As far as I understand, it is possible to use pure Modbus instead of solarmanv5?

Or may be I'm doing read in the completely wrong way?

Hmmm, could you try to run davidrapan/ha-solarman instead, and see if you'll get the same result? (If you decide to do se leave profile selection to auto)


[341] Reverts DB changes

Context
We have decided to scale the DB back to it's pre-start-of-cycle state.
Changes proposed in this pull request
Reverts the change sin this commit.
Guidance to review
To note that there will be some downtime, currently planned for Wednesday morning. We will turn on the service information banner tomorrow morning to let people know it's going to happen (see PR.
Things to check

[ ] If the code removes any existing feature flags, a data migration has also been added to delete the entry from the database
[ ] This code does not rely on migrations in the same Pull Request
[ ] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[ ] If this code adds a column to the DB, decide whether it needs to be in analytics yml file or analytics blocklist
[ ] API release notes have been updated if necessary
[ ] If it adds a significant user-facing change, is it documented in the CHANGELOG?
[ ] Required environment variables have been updated added to the Azure KeyVault
[ ] Inform data insights team due to database changes
[x] Make sure all information from the Trello card is in here
[x] Rebased main
[x] Cleaned commit history
[x] Tested by running locally
[x] Add PR link to Trello card

 Decide whether or not to maintain current database CPU levels.


'kubernetes.io/hostname' Label in worker nodes does not match the actual node name

What happened:
Created EKS cluster using terraform-aws-module  version 2.2.0
All the worker nodes has a different value for 'kubernetes.io/hostname'  label.
$ k describe node ip-10-23-46-23.eu-west-1.compute.internal
....
Labels:   ...
             kubernetes.io/hostname=ip-10-23-46-23.$MY_DOMAIN
....

What you expected to happen:
The label value for 'kubernetes.io/hostname'  should match the name at the output of
kubectl get node
ex: ip-10-23-46-23.eu-west-1.compute.internal
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

AWS Region:
Instance Type(s):
EKS Platform version (use aws eks describe-cluster --name <name> --query cluster.platformVersion):
Kubernetes version (use aws eks describe-cluster --name <name> --query cluster.version):
AMI Version:
Kernel (e.g. uname -a):
Release information (run cat /etc/eks/release on a node):

Same behavior here on the latest ami ami-0392bafc801b7520f.  kubectl get nodes show the name as ip-10-254-100-127.ec2.internal. However the hostname label is kubernetes.io/hostname=ip-10-254-100-239.ec2-internal. Shouldn't these two match?

This is also an issue with the EKS Ubuntu AMI (eg. ami-0cfc8096f1dfd9a2b).
Node has

metadata.name == ip-10-0-3-5.eu-central-1.compute.internal
metadata.labels -> kubernetes.io/hostname: ip-10-0-3-5

Are you still seeing this issue? My guess would be that it's caused by kubelet's in-tree cloud provider, which is feature-frozen in favor of https://github.com/kubernetes/cloud-provider-aws.


Examples of using LLMs for citation verification

Summary by CodeRabbit

New Features

Integrated a new validation system for answers and citations using an external language model to ensure accuracy and relevance.
Introduced a blog post on citation verification with practical examples and code snippets.

Documentation

Updated blog post titles to enhance clarity and focus on advanced topics and Python learning.
Added a new blog post "Validating Citations" to guide users on leveraging language models for data validation.

Refactor

Enhanced existing classes with new validation methods to interact with language models for real-time answer and citation verification.

@jxnl do you ignore the CodeRabbit Suggestions completely or do you read and decide whether it's right or not? Asking because I am considering using this or something like this as well.

its a nice check


Multi users are able to see all the documents from other users

Hello,
I have enabled multi user but all the users are able to see documents uploaded from other users as well which doesn't work well for other user as the new users creates a workspace to see, embed & chat with only their uploaded documents.

Just to be sure, but those other users, are they regular users or admins? In the latter case, that would be the expected behaviour.

Hey,
Other users are regular users. I have a admin user & 2 users as member user1 & user2.
What is uploaded by admin is visible to user1 and user2.
What is uploaded by user1 is also visible to user2, admin & vice versa.
So if all users are able to see all documents it will create confusion for everyone as the user might have to go through a bunch unnecessary items which isn't uploaded by them.

Ok I get it now, I have that behaviour too. Meaning, in the current setup of a-llm all users can see all uploaded files, from which you can then freely choose which ones to include in a workspace. The confusion then potentially stems from the fact that you can only upload files from within a workspace, instead of outside of it. Personally I do not mind that uploaded documents can be viewed by others, but then if would be good if the two were distinct, meaning that you would have a separate Upload documents button in the main page of the UI.

Yes it would be great if every user as "member" uploading files should not be visible to others because sometimes if files are more then for users to select documents in workspace will become a time taking process so if the user is just be able to see there own documents then it would reduce their time to select it in the workspace as well.

I think it is done from the perspective of not incurring additional cost (vectorizing the same document twice). Still, I agree that there is room for improvement. For instance, an option to delete uploaded files, and having the ability to have multiple upload folders (now it is one big custom documents folder for me). @timothycarambat perhaps you can express your view on the question from @MirzaAreebBaig if you have time?

Right now all members can view all files. Some things that are coming that might make more sense in context:

Total UI overhaul and change (includes the file selector and deleting files, folders and making new folders)
New role that is "chat only" and cannot even open the workspace settings

Yes originally showcasing all the files was more of a "Cooperative" model where that would be okay and people would not upload and pay to re-embed documents.
It appears that multi-tenet agencies have moved more towards "I will control docs and workspaces and let you use them". We weren't sure which management style would be used at first so we made an assumption there.

@timothycarambat  Thank you for the kind answer, and great to hear about the upcoming features! Looking forward to it!

@timothycarambat Thank you for your attention over this matter i suppose the things you have mentioned will solve the problem. Do we have any kind of roadmap to follow the upcoming features & can submit new features with information?

Resolved by https://github.com/Mintplex-Labs/anything-llm/pull/351

Yes it would be great if every user as "member" uploading files should not be visible to others because sometimes if files are more then for users to select documents in workspace will become a time taking process so if the user is just be able to see there own documents then it would reduce their time to select it in the workspace as well.

As administrator, I am still bothered by seeing every individual file that was added (by myself, in this case) to My Documents for one of my workspaces. I have to wade through the list to find the few additional files I want to add to another workspace.
It would help a lot to organize the My Documents by folder.  In fact, related to this, it would help a lot to be able to add a whole folder/directory from my local filesystem, recursively applied to subfolders as well, rather than have to add a flat collection of files that all end up in one folder.


Error in tensorflow debugger (tfdbg) while executing session run call in child thread

I ran tensorflow debugger using the command "python -m  --debug" but got the following error (i,e Signal only works in main thread):

The error is thrown when a session run call is executed in a child thread (spawned from main thread). Is tensorflow debugger only supported for single threaded applications?
What related GitHub issues or StackOverflow threads have you found by searching the web for your problem?
None
Environment info
Operating System: CentOS 7.2.1511
Installed version of CUDA and cuDNN:
(please attach the output of ls -l /path/to/cuda/lib/libcud*):

If installed from binary pip package, provide:

A link to the pip package you installed: https://storage.googleapis.com/tensorflow/linux/gpu/tensorflow_gpu-1.0.0-cp35-cp35m-linux_x86_64.whl

The output from python -c "import tensorflow; print(tensorflow.__version__)".

If installed from source, provide

The commit hash (git rev-parse HEAD)
The output of bazel version

If possible, provide a minimal reproducible example (We usually don't have time to read hundreds of lines of your code)
What other attempted solutions have you tried?
Logs or other output that would be helpful
(If logs are large, please upload as attachment or provide link).

@agupta74 tfdbg's CLI class attempts to register signal handlers even when not running on the main threads, which is the reason for this error. We will send a bug fix to the master branch.
As a workaround until the bug is fixed, you can pretend that the child process is a "remote process" and use the approach outlined here:
https://www.tensorflow.org/programmers_guide/debugger#offline_debugging_of_remotely-running_sessions

Thank you for reporting this bug, @agupta74!

@caisq Any updates regarding this issue? I've tried using offline_analyzer per your suggestion; but, keep running into ValueError: Duplicate node name as in #7051. Unlike #7051, I'm running locally on CPU. The problem seems to stem from repeated calls to sess.run(), where sess is an instance of DumpingDebugWrapperSession; but, I'm not sure how to get around this issue when using tf.get_variable() given that a call to an initializer seems necessary. Included below is a code snippet.
# Dependencies
import subprocess
import tensorflow as tf
from tempfile import TemporaryDirectory
from tensorflow.python import debug as tf_debug

dump_dir = TemporaryDirectory() #temp. directory for demo
with tf.Session() as _sess:
  # Create 'DumpingDebugWrapperSession' wrapper
  def watch_fn(fetches, feed_dict):
    node_name_regex_whitelist = '(watch_[0-9]+)'
    debug_ops, op_type_regex_whitelist = None, None
    return debug_ops, node_name_regex_whitelist, op_type_regex_whitelist
  sess = tf_debug.DumpingDebugWrapperSession(_sess,
                  dump_dir.name, watch_fn=watch_fn)

  # Declare TensorFlow variables
  tf_vars = [tf.get_variable('watch_0', shape=[32], dtype='float32'),
             tf.get_variable('ignore_0', shape=[32], dtype='float32')]

  # Calls sess.run()
  init_op = tf.global_variables_initializer()
  sess.run(init_op)
  # <stuff goes here>
  fetches = sess.run(tf_vars, feed_dict={})

# Call to offline_analyzer
argv = ['python3.5', '-m', 'tensorflow.python.debug.cli.offline_analyzer',
        '--ui_type', 'readline', '--dump_dir', dump_dir.name]
cmdline = " ".join(argv)
try:
  proc = subprocess.Popen(cmdline, shell=True)
  proc.wait()
finally:
  dump_dir.cleanup()

@j-wilson With regard to the duplicate node name error you are getting, are you using the same file:// debug URL (i.e., the same directory) for different session.run() calls? If that's the case, can you try using a unique different URL (i.e. ,directory) for each session.run() call?

@caisq The script runs if modified s.t. a new wrapper/directory is used for each call to sess.run(); however, none of the tensors show up inside the debugger CLI, i.e. 0 dumped tensor(s):.
As a separate comment, is there a reason why separate wrappers/directories are needed for each call to sess.run()? The wrapper creates subdirectories run_<run_id>, suggesting that reuse of a single wrapper might be possible --- which would help with usability.

@j-wilson There is already a wrapper along that line, see: https://github.com/tensorflow/tensorflow/blob/master/tensorflow/python/debug/wrappers/dumping_wrapper.py#L31
Usage example:
from tensorflow.python import debug as tf_debug

sess = tf.Session
sess = tf_debug.DumpingWrapperSession(sess, session_root="/tmp/dumps1")

The dumps from different runs will be inside folders such as /tmp/dumps1/run_<epoch_time_stamp>_
See also doc at: https://www.tensorflow.org/programmers_guide/debugger#offline_debugging_of_remotely-running_sessions
As for the reason why you see 0 dumped tensor(s), it might be that ops are constant folded during graph optimization? Does the same 0-dumped-tensor issue happen if you run it on the main thread, and/or on some other graph in a child thread?


Test for continuity after concurrent callers

This PR extends the main test to enforce that once concurrent callers are done, further callers may continue getting results.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
38

Relevant Lines:
38

💛  - Coveralls


I found I could animate alpha from 1 to 0 but not from 0 to 1.

I really really want to thank you for this.  This is exactly what I needed however I can't seem to animate alpha from 0 to 1.
This code can fit directly into your sample to show you the problem.
public void shuffle(View view) {
final AlphaAnimation animation1 = new AlphaAnimation(0.1f, 1.0f);
animation1.setDuration(1000);
animation1.setStartOffset(1000);
    animation1.setAnimationListener(new Animation.AnimationListener() {

        @Override
        public void onAnimationEnd(Animation arg0) {
            blurrHolder.invalidate();
            blurrHolder.setAlpha((float) 1);
        }

        @Override
        public void onAnimationRepeat(Animation arg0) {
            blurrHolder.invalidate();
        }

        @Override
        public void onAnimationStart(Animation arg0) {
            blurrHolder.invalidate();
        }
    });

    mBlurringView.startAnimation(animation1);
}

Hard for me to tell. But maybe what's said here is relevant: http://stackoverflow.com/questions/11386953/alphaanimation-does-not-work ? In other words, maybe we need to ensure that the view's alpha is non zero when animation starts?


Setup CI (NTTVS-3)

Closes NTTVS-3

@scottschmitz thoughts on making squash-and-merge the default?

I will never agree to that haha

I will never agree to that haha

:(


UnboundLocalError: local variable 'eval_features' referenced before assignment

Following through the readme and ran the file
python run_ner.py --data_dir=data/ --bert_model=bert-base-cased --output_dir=out_base --max_seq_length=128 --do_train --num_train_epochs 3 --do_eval --eval_on dev
I get this error. This is blocking,
Traceback (most recent call last):
File "C:\work\code\ai\BERT-NER-TF\run_ner.py", line 512, in
main()
File "C:\work\code\ai\BERT-NER-TF\run_ner.py", line 380, in main
np.asarray([f.label_id for f in eval_features],dtype=np.int32))
UnboundLocalError: local variable 'eval_features' referenced before assignment

You can put the code of definition of eval_features above the line 380


Not enough funds to execute?

Hello,
Its driving me crazy..
The bot starts with "holding on to my but!" :)  and confirming the just balance right.
When it wants to sell coin ("attempting to sell"), it cant,  saying "not enough funds" funds are 0.000, but actually are not. :(
Because Bitfinex and GDAX (modules) have the same issue, i suspect it is somewhere in the code.
It looks like it accurs after some time when the bot is running. At start it executes well.
Same issues as #112 (albertolive) but now with Bitfinex.
Any suggestions?
Thx!

In my case, it seems the bot makes some kind of calculation, for example:
04/18/2017 11:59:03 PM UTC [      action] m24875998 sell {
"type": "sell",
"asset": "ETH",
"currency": "BTC",
"exchange": "gdax",
"price": 0.04157,
"fee": 0.00016032939071584725,
"market": true,
"size": 1.5427413107129877,
"rsi": 65.58469747008522,
"roi": 0.99875,
"roi_delta": -0.0012499999999999734,
"performance": null,
"waited": null,
"balance": {
"ETH": 1.5427413107129877,
"BTC": 0.06397142689562306
},
"end_balance": 0.12810318318196195,
"id": "d7afc48ba80daccd",
"time":<PHONE_NUMBER>000,
"timestamp": "04/18/2017 11:59:00 PM UTC"
}
in my logic i have 50% trading, not the 98%
And I only have BTC, not ETH, and its balance is 0.12810318318196195 which is 3ETH
and the 50% make around-ish 1.5427413107129877 ETH it tries to sell.
So there is something wrong with the logic here

My bot is also configured to use 50% balance and it appears to be trying to sell amounts of coin I do not have. I have several trades which basically bounced as there was not enough funds.
Any solution to this, have you hacked about with the logic? Im going to have a dig about and see what I can see

Cleaning up 3.x issues, as I no longer support it. Try Zenbot 4!


Added resourcs for react

Fixes Issue #1239
Changes proposed
Added resources for react js.
Screenshots

Note to reviewers

will correct it and push the changes again thank you @CBID2

@CBID2 Have pushed the changes again, please check now

@karthiknadar1204 deployment is failing plz check.

should i open another pr?

This PR is closed because #1399 solved it.


Make Validate.checkValid return Message

... similarly to other checkXxx methods from Guava's Preconditions.

Addressed by recent Validation.


Extremely poor performance compared to indexeddb

I've been trying to use this adapter as recommended by Nolan's guide to prebuilt databases on pouchdb. It's horrible! I'm not sure if it's the adapter, cordovasql, or some other problem, but it uses 15%cpu on my app constantly (this might be from building views in the background I'm not sure) and is slow to get a document. Sometimes it fails to return a query promise. It makes my app unusable. Ive had to  replicate the sql database to an standard indexeddb one and now it is snappy with only approximately 2% CPU usage. This is on a galaxy note 4.

Yup, Cordova SQLite has poorer performance than IndexedDB, especially in scenarios like PouchDB's. The goal of this project is to provide native SQLite access to folks who absolutely need it; everyone else should use IndexedDB. I have some thoughts on this here: https://nolanlawson.com/2016/04/10/introducing-the-cordova-sqlite-plugin-2/
I would however be interested to know where you were using Cordova-sqlite-storage or cordova-plugin-sqlite-2 since you didn't specify.

Also yeah if you are doing queries then you will be hit pretty hard by perf issues; keep in mind that Cordova has to send string-based messages back and forth between the WebView and the native context for just about every interaction between PouchDB and the database, and PouchDB builds the index via a lot of reading and writing. (This will be fixed with native secondary indexes, but we're only going to implement that for IDB via the "idb-next" project.)

Hey Nolan, thanks for the insight. I thought I would put this warning here for people so they try to avoid the sqlite adapter. I'm using cordova-plugin-sqlite-2. I assume the high CPU usage (and up to (500 mb of ram usage) was due to the indexing. But it never seemed to finish. I was building 2 different indexes on 6000 documents just emitting a small array.
By the way, thanks so much for all your work on pouchdb. My app would not have been possible without it. The

Thanks for the feedback! I implemented cordova-plugin-sqlite-2 to be as performant as possible, so you're probably just hitting limits of the PouchDB design and limitations in how Cordova WebView communication works.
Another good strategy you can try: you can prebuild indexes as well as the main database. Just copy over the mrview-*.sqlite files. :smiley:

Interesting. I assume the indexes would also be copied over when I replicate to an indexedDB based PouchDB?

Nope, indexes are not copied during replication. But you can copy them using the "prebuilt SQLite" strategy.

Ok thanks. I tried out the sqlite adapter again. It  takes approproximately 2-3minutes to build the index. After its finished it's actually not too bad. However one query with a startkey and endkey on 2 keys (a string and an array) takes just a bit too long for a good user experience. it varies between 2 and 10 seconds. So I'm going to keep my strategy to replicate to and indexeddb pouchdb. I might try pouchdb.load() from a dump file to see if that is faster than the replication, which takes about minute

Yeah I'm sorry, but IndexedDB just naturally has the best performance on Android. Hopefully you can find the best settings for your app. I worked really hard to make it as performant as possible, but some workloads are just very difficult to make performant.
You may want to try using Dexie for your index and PouchDB for your replication. PouchDB is optimized for replication but not for indexing.

Hi Nolan. I just tried db.load() from a dump file as per your prebuilt database guide and it completes in 10seconds, compared to 1 minute for a replication from sqlite db to indexedDB. Nice! That's a better welcome to my app. As an added bonus, the text file is half the size of the sqlite db.
I think at this stage I have to leave everything as sqlite for iPhone with the device storage space issues. I'm yet to test on an iPhone, but I'm getting the vibe that sqlite might be faster there than on Android.


[Custom json read] - Fix string_t concept (it is not based on push_back in practice)

Test PR to attempt to fix this issue.
Basically this PR changes the string_t concept previously based on has_push_back with the methods actually used:
// this concept requires that T is string and copies the string in json
template <class T>
concept string_t = str_t<T> && !std::same_as<std::decay_t<T>, std::string_view> && has_push_back<T>;

becomes:
template <class T>
concept has_resize = requires(T t, typename T::size_type sz) { t.resize(sz); };

template <class T>
concept has_append = requires(T t, typename T::const_iterator it) { t.append(it, it); };

template <class T>
concept has_assign = requires(T t, const typename T::value_type* v, typename T::size_type sz) { t.assign(v, sz); };

template <class T>
concept string_t = str_t<T> && !std::same_as<std::decay_t<T>, std::string_view> &&
                                 (has_assign<T> || has_resize<T> || has_append<T>);

This looks great! Thank you!

template <class T>
concept has_resize = requires(T t, typename T::size_type sz) { t.resize(sz); };

already in <glaze/concepts/container_concepts.hpp> as concept resizable<T>
   template <class T>
   concept resizable = requires(T v) { v.resize(0); };

looks same

Thanks for pointing this out. I'll eliminate the duplication.

It actually also needs .data() with resize based on the code. Created this PR


SSCS-11548-Added the tag version

JIRA link (if applicable)
SSCS-11548
Change description
Preview setup changes.
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ X] No

not needed anymore


docs: Fix JDK and Gradle capitalization

Looks good, thanks for your first contribution to the website @isa-bel!


[Bug] Failed to execute 'setItem' on 'Storage': Setting the value of 'free-chat-gpt' exceeded the quota.

When I try to post after a thread that is ~8000 tokens long, I get this error. And the post cuts off midway through.
"Failed to execute 'setItem' on 'Storage': Setting the value of 'free-chat-gpt' exceeded the quota."

to add to this, I imported a very large history from an OpenAI json and it stopped working as well. had to export and delete conversation history.
For me, the error didn't show up in the UI but only the browser console.


documentation

Hi Jenny,
I am having following errore in the command prompt of node.js
[image: Inline image 1]
Not sure what it is. I went to nonprofit to see the code. user/ c4sg is
similar to the nonprofit one.
Can I take some help from there?
Need help.
Thanks,
Sabrina
On Sun, Feb 5, 2017 at 4:44 PM, Code for Social Good <
<EMAIL_ADDRESS>wrote:

Merged #204 https://github.com/Code4SocialGood/C4SG/pull/204.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Code4SocialGood/C4SG/pull/204#event-949586965, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ATkfjJLl7-eXKf3TY86ariXi4HDmDOU2ks5rZlC1gaJpZM4L3qBM
.


No Google login available only PTC and possible ban + other errors on download

I cannot login with google it only gives me the choice to login with pokemon trainer club. When I open the map it says waiting then goes to possible ban where it normally says operational. And when I downloaded the new update an error message popped up.

I've had tons of issues with the new release. Ultimately boiling down to problems with the PTC servers. I made a new account and cant even login with the game because it says the password is wrong, however I can login to the website. My older account gets the same error although it may of been banned. I know my original PTC account was banned for sure as  Ichecked the email but the last PTC account I had been very conservative with and rarely ever even toggled it on anyways but just hate walking in circles looking for something. I feel like aimless walking is bad design, Niantic may praise walking aimlessly but I like to aim where I am going, I like to know where my destination is and why its my destination. The map app provided that but it sees some more Niantic crap is looming in the background because if the game client itself wont let a new account login via the PTC servers then something is wrong. I don't think its the app, I think some more changes are coming, eventually they will work their way down the list of 3rd party apps they hate to map scanners, theyve gotten pretty good at dealing with some bots and some exploits so my guess is we're next on the list.


Ipv4Network::__constructor bug w/ CIDR Format

If you specify a mask in CIDR notation (<IP_ADDRESS>/8) the parsed mask is always zero. Methods such as Ipv4::isInRange do not work This is due to a bug in the constructor.
Here is a fix:
--- src/Ipv4Network.php	(revision 70f81df2ea1a58d88cd0017e39bbb41c35498117)
+++ src/Ipv4Network.php	(revision )
@@ -77,8 +77,8 @@
 		}
 		if(is_string($ipAddress) && ($dp = strpos($ipAddress, '/')) !== false)
 		{
-			$ipAddress = substr($ipAddress, 0, $dp);
-			$mask = (int) substr($ipAddress, $dp + 1);
+            $mask = (int) substr($ipAddress, $dp + 1);
+            $ipAddress = substr($ipAddress, 0, $dp);
 		}
 		if(is_numeric($bitmask))
 			$mask = (int) $bitmask;

Hi,
Thank you for the report. I effectively understand the bug now, and this should be fixed with commit c0fb322. I changed the Ipv4Network and the Ipv6Network the same way.
You may now download it under the tag 1.1.1.


Intersect Town Hall - 25th June 2024

This is the main tracking issue for the June 2024 Intersect Town Hall
Slides

https://docs.google.com/presentation/d/1RURCxhUYcfNtGjXe2G_N1VXJ53Zifj-Ib9IqRD3EFNg/edit?usp=sharing

Sheet for collecting timestamps

https://docs.google.com/spreadsheets/d/1VCAQDpT3vKrqf9MvwoApw5Z9TMkrtiM5PYaK_rfVolg/edit?usp=sharing

Video

https://www.youtube.com/watch?v=kN17feSQnVQ

Actions
Related Actions include -

https://github.com/Catalyst-Swarm/Intersect-Town-Hall/issues/1
https://github.com/Catalyst-Swarm/Intersect-Town-Hall/issues/2
https://github.com/Catalyst-Swarm/Intersect-Town-Hall/issues/11

Actions arising from Problem Board

https://github.com/Catalyst-Swarm/Intersect-Town-Hall/issues/31

In June, we started requesting permission to tag Intersect Members to amplify our outreach messages.
We also requested that they amplify messages for us if we are not allowed to tag everyone.
As of the due date, it still continues to be an issue where our requests do not get a reply.

In June we had no access to appropriately update Google Calendar and Discord Events.
It was solved in the following month of July


TCVIS layer - false masking for 0 values

valid 0 values are occasionally masked. Might occur in RTS
Solution:

replace single band 0 values with "1"
real NoData values are (0,0,0)

fixed with https://github.com/initze/thaw-slump-segmentation/commit/df4a3f437e71209593ae4414c8070c0cc471f8fd


Fix-tree while upgrading django-cms from 3.3.2 to 3.4.2 takes more than 2 hours

Summary
I'm upgrading django-cms from 3.3.2 to 3.4.2 and running python manage.py cms fix-tree as documented. Running the command however took a long while and I noticed that it has been stuck for around 2 hours on the following query:
SELECT tbn1.path, tbn1.numchild, (SELECT COUNT(1) FROM `cms_cmsplugin` AS tbn2 WHERE tbn2.path LIKE CONCAT(tbn1.path, '____')) AS real_numchild FROM `cms_cmsplugin` AS tbn1 HAVING tbn1.numchild != real_numchild

Seems like this query comes from treebeard.
I got ~151k rows in cms_cmsplugin and I suspect that there are too many plugins for treebeard to process in a timely manner.
Expected behaviour
I expected the cms fix-tree command to take a while but finish within a couple of minutes.
Actual behaviour
I decided to quit the running query after 2 hours because I felt the duration was unacceptable.
Environment

Python version: 2.7.10
Django version: 1.8.17
django CMS version: 3.4.2

Thanks for reporting this.
We're aware of the issue, its part of a limitation imposed by our usage of treebeard.
Can you comment out the lines above and including https://github.com/divio/django-cms/blob/release/3.4.x/cms/models/pluginmodel.py#L443 then re-run the command?
This is not a fix but can help us determine if there's something to be done in the core.

I commented out the lines you specified and the fix-tree command completed successfully (in a few minutes)
fixing page tree
fixing plugin tree
all done

The upgrade seems to be done successfully and the site looks like to be working as expected (including the page tree in the admin).
Any side effects I can expect by skipping the treebeard tree fix? What is the tree fix generally used for?

Hello @bvermeule,
Just realized I said "above"... I meant "below".
This line https://github.com/divio/django-cms/blob/release/3.4.x/cms/models/pluginmodel.py#L442 should still run, all others should be commented out.
If it takes a long time then we need to address the issue in treebeard, otherwise its on our side.
The tree fix does a lot to make sure the tree is consistent, prior to 3.4 there were a lot of "tree corruption bugs" which would put the page or plugin tree in an inconsistent state and that would cause problems when pages or plugins are added, moved or deleted.

Hi @czpython ,
I can easily confirm that the treebeard method takes a really, really long time. I isolated the query (see the original issue post above) and tried to run it directly on the database.
You can easily reproduce this by adding at least as many rows in cms_cmsplugin as me (150.000) en running the query.

@czpython
Any idea what my next step could be?
Skipping the tree fix didn't seem to have any issues. I tested this by just clicking through the site. Any way to verify whether I need the tree fix?

Hello @bvermeule,
You can run https://gist.github.com/czpython/bacfd888e45b6713abb7395acd44ec1d.
This checks the plugin tree for a "numchild" corruption which is the most common.
As this is confirmed to be an issue with treebeard, I'm closing the ticket.
Feel free to reach out via google groups if you need more assistance with this.


Get file reference from URI

Does this library offer users a way to get a reference to a file in GCS from a URI? Suppose I have a uri like the following: gs://mybucket/myvideo.mov.
Ideally I'd like to have an api like the following:
import { Storage } from '@google-cloud/storage';
const storage = new Storage();

const file = storage.getFileFromUri("gs://mybucket/myvideo.mov")

// do stuff with the file like like change it's metadata or download it

Is this possible or will I have to write my own utility function to parse the gs://... uri?

Hey @paymog, this sounds a lot like:

https://github.com/googleapis/nodejs-storage/issues/1898

We think this would be a handy feature! We'll add this functionality in a future release.

closing this as a duplicate of https://github.com/googleapis/nodejs-storage/issues/1898
we'll track updates to this feature request there.


Risk group specific prevalence estimates

Might need another SAE model for this?

Need to disaggregate prevalence but on the log-odds scale:

This from Naomi might help. See also the Odds ratio Wikipedia page.

plhiv = prev * pop = sum(prev_i * pop_i)
prev = sum(prev_i * pop_i) / pop


Add Resources Documentation

Any more guidance available? If I run the app locally, will the nature of "Resources" be obvious enough that I could just start writing documentation for it?

With better use of esdocs in the API/Services area as well as the app as a whole - this would likely be resolved with the integration of github pages and our parsed documentation. Closing for now and will open individual issues related to documenting the app for specific areas of need.


refractor config

This PR is for #19

创建了一个 Settings 模块，并创建了一个抽象类，万一以后还要加配置文件呢
改了其它文件中原本依赖于 GLOBAL_CONFIG 的地方
进行了一些测试，但不能保障没有bug
移除了 Him

Created a Settings module and created an abstract class, in case you need to add a configuration file in the future
Changed other files that originally depended on GLOBAL_CONFIG
Did some tests, but can't guarantee no bugs
Removed Him

@ram6ert Great idea! I'll add them s∞∞∞∞∞∞n!


Storefront improperly displaying unverified moderator badge

A store with verified moderators shows listings with the unverified moderators badge when viewing the storefront. The correct badge displays on the listings from search and when viewing the listing itself.

This turned out to just be due to the store server needing to be updated to the latest release.


Trouble registering custom field types via Blockly.Field.register

Got a report that Blockly.Field.register isn't working for field types that aren't yet defined. Seems to work okay to override existing types, but not new ones.
Needs more investigation.

Moved to LLK/scratch-vm#1647

The registration of custom fields in blocks works okay - the issue is with using custom fields through an extension in the vm.


[2.10] Add the compliance history tests

The cherry-pick for this was missed.

/retest

Sorry @yiraeChristineKim , I wasn't working with the latest changes from main when I copied the test over.

/hold checking on if this can be called multiple times per evaluation

/unhold

The same test keeps failing. I didn't check, but does the downstream gatekeeper test updates also need to be pulled in here?

Let me create a cluster and fix this

I think this entire commit just needs to be pulled in. I think only the compliance history bits got updated in this PR: https://github.com/stolostron/governance-policy-framework/commit/f68bb0b552deee67863f5e0d36bc3b8eec2f0f8c

Of course now that I say that, the most recent failure is the same one: https://redhat-internal.slack.com/archives/CUS1VB8P3/p1711658066131449
So maybe it's a flake this PR just keeps hitting...

Of course now that I say that, the most recent failure is the same one: https://redhat-internal.slack.com/archives/CUS1VB8P3/p1711658066131449
So maybe it's a flake this PR just keeps hitting...

@dhaiducek I pulled in your commit for the downstream Gatekeeper fixes.

@dhaiducek and @yiraeChristineKim it looks like there was a misnamed ConfigurationPolicy since it's supposed to match in policy-gk-prereq.yaml and policy-gk-prereq2.yaml. The latest commit here addresses that and the main branch fix is at #771.

/retest

/retest

@yiraeChristineKim and @dhaiducek I finally got this to pass :laughing:

/hold in case @yiraeChristineKim's PR should be included

#773

/unhold @yiraeChristineKim , you'll just have to cherry-pick your changes.

/unhold


Support SDL schemas in addition to introspection results

Taken from https://spectrum.chat/apollo/apollo-android/can-i-create-a-graphql-sdl-instead-of-using-a-schema-json~e893df98-765f-41d8-8692-583555960ce8
It would be nice to be able to drop in a SDL file instead of using the schema.json format.

It would be a nice feature to have, especially when starting out with Apollo.
Also, if this feature would get implemented, one would be able to use the GraphQL plugin for android studio. JS GraphQL.
The plugin works when you create a schema.graphql using SDL, but the schema needs to be deleted in order for the apollo sources to be generated.

@FernandoH-G I have seen tthee apollo js plugin work with json introspection results. You might have to tweak a bit your .graphqlconfig:
Something like:
{
  "schema": "src/main/graphql/schema.json"
}

That did it. Thank you.

That would be great future, at least SDL more concise than JSON format.
Unfortunately I couldn't find existing Antlr grammar for this, we should write own one, to be able to parse it.


ANDROAPP-5759-mobile-ui-file-input-label-displacement-on-file-insertion

Hi @msasikanth , this PR should go from and to main as it is targeting UI:0.1.1

My bad I forgot to look at the version in the task. Rebasing this PR, need to rebase #160 as well


Hide threads from LHN if they don't have a comment yet

Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01aee66cc771604fe5
Upwork Job ID:<PHONE_NUMBER>086478848
Last Price Increase: 2023-05-18

@chiragsalian some thoughts-
we can 1) simply filter out reports that have no visible actions on the parentReportAction aka this branch: https://github.com/Expensify/App/pull/new/georgia-hideEmptyThreads
or 2) we don't start adding users to the participants list until at least one comment has been made

i would suggest 1, because if we go with 2 then that's changing the participant list data to work drastically different for threads. Plus with solution 2, then once the user is in the chat and they click members they won't see themselves which is incorrect.
So option 1 sounds correct to me.

draft PR up, but need to fix the optimistic data

still in progress

I need to update my PR to fix the optimistic data

Isn't this a duplicate of this older issue?

Isn't this a duplicate of this https://github.com/Expensify/App/issues/14523?

Hey @JmillsExpensify since this issue is specific to Threads I don't think it is a duplicate! I checked the draft P.R. logic and compared it with the one you mentioned and since I'm adding another clause and param to shouldReportBeInOptionList I don't think we have any conflicts in logic either : )

Still need to update unit tests

Coming back to this today

@MitchExpensify added BZ label to help with C+ $1000 PR Review Payment for this PR - https://github.com/Expensify/App/pull/19375

Upwork job created for C+ payment and hire offer sent to @mollfpr https://www.upwork.com/jobs/~01393c0b271fdfe3bc

Added a reminder to pay on the 10th. Friendly reminder to take a look at the BZ checklist above @mollfpr 👍

Hey, just curious how this should work for tasks? Currently uncommented tasks disappear which doesn't seem expected

Paid and contract ended! @mollfpr

Are their any tests we should add for this change? https://github.com/Expensify/App/issues/19205#issuecomment-1618405750

We need the regression step here.
Web/Desktop/Large screen

Create a thread from a message
Verify that the thread report appears in the LHN
Begin Typing a message; note that the thread still appears in LHN with the edit icon.
Click another report
Verify that the threaded message with no replies no longer appears in the LHN
Create a thread from a message, and comment
Click another report
Verify that the threaded message with no replies remains in the LHN

mWeb/Mobile/Small screen

Create a thread, don't comment
Go back to LHN/chat list page
Verify that the newly created thread does not appear

Issue to add new test here


GET Image Error on the Home Page and Store Page (Header Image)

When I fired up the client on the home view, I get the following error:

When I went to the Art Store (seed server), I received the following error:

As you can see, for some reason it is trying to make an API call with <IP_ADDRESS> instead of localhost. What's even weirder about this, is that the address is my other node hosted remotely and not even the seed server's IP address.
When I replaced the address with local host and search for the hash of the image (on the Art store page), this is what I get:

^ So this is clearly supposed to be the header image of the seed server. I also couldn't pull out the image it is trying to load on the home page, even with localhost.

Also, that image that won't load is from @hoffmabc 's node.


Create dependent jobs

Hi!
I have a necessity to have several jobs which related to each other.
So some job will start executing after children jobs are executed.
Is it possible right now? Or how is it hard to add this ability?

Short answer: No. You can write your own dependency manager by listening to job events and managing related job ids.
if you search issue list, you can find related discussions

Thanks. I did not find anything related to it.


Adding CaratLane app to react-native showcase

CaratLane app is using react-native in HomePage. We recently went live in react-native and happy to share this news with the community.
CaratLane is one of the leading online jewellery company.

But still in website "CaratLane" is not reflecting in the react-native showcase.

@anishakd4  website will be deployed again in some time.


update input-label letter spacing to 1px

feedback from design

You'll need to increment the version number of Juice in package.json.
After that, :ship:


Campaign id cannot be less than 1 Error

Getting the following error:
2023-09-18 19:07:56.1702 FATAL Fatal error, application will be closed: UniversalDownloaderPlatform.Common.Exceptions.UniversalDownloaderException: Unable to retrieve crawl target info: Campaign id cannot be less than 1 (Parameter 'campaignId')  ---> System.ArgumentOutOfRangeException: Campaign id cannot be less than 1 (Parameter 'campaignId')    at PatreonDownloader.Implementation.PatreonCrawlTargetInfoRetriever.GetCrawlTargetInfo(Int64 campaignId) in F:\Sources\BigProjects\PatreonDownloader\PatreonDownloader.Implementation\PatreonCrawlTargetInfoRetriever.cs:line 58    --- End of inner exception stack trace ---    at PatreonDownloader.Implementation.PatreonCrawlTargetInfoRetriever.GetCrawlTargetInfo(Int64 campaignId) in F:\Sources\BigProjects\PatreonDownloader\PatreonDownloader.Implementation\PatreonCrawlTargetInfoRetriever.cs:line 75    at PatreonDownloader.Implementation.PatreonCrawlTargetInfoRetriever.RetrieveCrawlTargetInfo(String url) in F:\Sources\BigProjects\PatreonDownloader\PatreonDownloader.Implementation\PatreonCrawlTargetInfoRetriever.cs:line 26    at UniversalDownloaderPlatform.Engine.UniversalDownloader.Download(String url, IUniversalDownloaderPlatformSettings settings) in F:\Sources\BigProjects\PatreonDownloader\submodules\UniversalDownloaderPlatform\UniversalDownloaderPlatform.Engine\UniversalDownloader.cs:line 183    at PatreonDownloader.App.Program.RunPatreonDownloader(CommandLineOptions commandLineOptions) in F:\Sources\BigProjects\PatreonDownloader\PatreonDownloader.App\Program.cs:line 128    at PatreonDownloader.App.Program.Main(String[] args) in F:\Sources\BigProjects\PatreonDownloader\PatreonDownloader.App\Program.cs:line 68

+1 here...

Same problem:
2023-09-19 10:43:24.9759 FATAL Fatal error, application will be closed: UniversalDownloaderPlatform.Common.Exceptions.UniversalDownloaderException: Unable to retrieve crawl target info: Campaign id cannot be less than 1 (Parameter 'campaignId')

same here

Created a PR that I believe fixes this issue
https://github.com/AlexCSDev/PatreonDownloader/pull/197

Also confirmed that it works when built from source code per build instructions with that PR implemented

Also confirmed that it works when built from source code per build instructions with that PR implemented

a big thanks!

Created a PR that I believe fixes this issue
#197

With the PR it is working now.
Thanks

Sorry i'm new to this. I have forked my own copy and modified the code as directed by Skyl3lazer. How do I compile everything in this fork into an executable to be able to run/use it?

Read the build instructions as directed by the readme. It's another
document in the docs folder.
On Tue, Sep 19, 2023, 9:13 PM IceySnowman @.***> wrote:

Sorry i'm new to this. I have forked my own copy and modified the code as
directed by Skyl3lazer. How do I compile everything in this fork into an
executable to be able to run/use it?
—
Reply to this email directly, view it on GitHub
https://github.com/AlexCSDev/PatreonDownloader/issues/196#issuecomment-1726740399,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABQYWEQFOSYAYCIHEPPJO5DX3I7MNANCNFSM6AAAAAA45NQMKY
.
You are receiving this because you authored the thread.Message ID:
@.***>

I've spent a few hours trying to get this to work & keep failing in various different ways. Until AlexCSDev is able to return, could anyone please consider providing me with the amended "PatreonDownloader.Implementation.pdb" file or entire amended net6.0-win-x64-release.zip?

I also tried compiling it according to the instructions but only got errors, I'm a complete noob mind you.
Would appreciate if someone could share it.

I also tried compiling it according to the instructions but only got errors, I'm a complete noob mind you. Would appreciate if someone could share it.

Sure thing.
debug-net6.0.zip

Thank you!

I've spent a few hours trying to get this to work & keep failing in various different ways. Until AlexCSDev is able to return, could anyone please consider providing me with the amended "PatreonDownloader.Implementation.pdb" file or entire amended net6.0-win-x64-release.zip?

https://github.com/AlexCSDev/PatreonDownloader/files/12675516/debug-net6.0.zip

Thank you, lifesaver!

for everyone who wants to compile it from source while it isnt fixed use this to clone the repo: git clone https://github.com/Skyl3lazer/PatreonDownloader -b patch-1 --recursive

Thank you

anyone can share a fixed build for linux, thx a lot!

@jefflaunmg just build it yourself build -c release

What am I supposed to do with the debug-net6.0 file?

@jdragoon nothing, download @Skyl3lazer fox and follow the instructions in the readme

Should be fixed in release 26/27, thanks to Skyl3lazer for the fix!


First Feedback and idea for improvement

3 Phase is working well. My first proposal for improvement wuld be to add a unique ID (could be the same as entity, this would help to add them into dashboards. Additionally, the ability to add all (or more than one a a time) entity to a dashboard would help

Add to dashboard:

Hello and thank you for opening this issue.
I'm not sure why the unique id is not being set for you.
In my own production setup (running the latest released version) every entity (including my grid energy forward total entity) has a unique id.

Is the unique id not being set (at your end) occuring for every entity provided by this integration?

I reinstalled the integration and now it works from this point. This time the IP Address (without http:// before) gave me success. But now, I have another Issue. Will open a new one and close this one, thanks!


Add Array.find_last() and Array.count()

Added Array.find_last(var) and Array.count(var), with documentation. I think this is a nice feature to have available in engine/in gdscript. find_last(var) is inspired by String.find_last(var) for consistency.  See #5020.

@J08nY Would be nice if we support an optional from parameter, which would behave like in the String class (but maybe thats for another PR).


Use groupified apiVersion

Non-groupified objects were deprecated in OCP 4.7.

Coverage remained the same at 84.868% when pulling e091ec036ba668e68a79907526dac34108df6715 on multi-arch:samples into 96ef167640f8584da8b6acab9a97154dafd9f936 on nodeshift-starters:main.

@yselkowitz  thanks!


Docker seems to leak memory, on Tutum/Ubuntu 15.04 x64, for reasons unknown

The docker process is using about 635 MB of swap memory, on a machine with 1 GB of physical memory and 2 GB of swap memory. It rises to this level after about one day, whereupon the machine is borderline unresponsive from swapping. The total usage of swap memory is about 1482 MB at that point.
This is on a Tutum managed Ubuntu 15.04 x64 Digital Ocean VM.
docker version:
Client:
 Version:      1.9.1-cs2
 API version:  1.21
 Go version:   go1.4.3
 Git commit:   4ade326
 Built:        Mon Nov 30 21:56:07 UTC 2015
 OS/Arch:      linux/amd64

Server:
 Version:      1.9.1-cs2
 API version:  1.21
 Go version:   go1.4.3
 Git commit:   4ade326
 Built:        Mon Nov 30 21:56:07 UTC 2015
 OS/Arch:      linux/amd64

docker info:
Containers: 44
Images: 189
Server Version: 1.9.1-cs2
Storage Driver: aufs
 Root Dir: /var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 277
 Dirperm1 Supported: true
Execution Driver: native-0.2
Logging Driver: json-file
Kernel Version: 3.19.0-22-generic
Operating System: Ubuntu 15.04
CPUs: 1
Total Memory: 993.1 MiB
Name: muzhack.com2
ID: IC6S:RE62:77W2:424Q:7OKZ:JDT4:RETI:WQ4A:D23X:B7O5:EWVX:XAQY
Username: aknudsen
Registry: https://index.docker.io/v1/

uname -a: Linux muzhack.com2 3.19.0-22-generic #22-Ubuntu SMP Tue Jun 16 17:15:15 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux
How to reproduce:

Install MuzHack stack on Tutum, with corresponding 1 GB Ubuntu 15.04 x64 DO nodes having 2 GB of swap memory
Let marinate for a day
Log on to muzhack-green node
Inspect memory usage, where much swap memory should now be consumed

You can see my findings regarding memory usage, system wise and by individual processes in the below screenshot:

If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
For more information about reporting issues, see CONTRIBUTING.md.
You don't have to include this information if this is a feature request
(This is an automated, informational response)

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

@aknuds1 Based on your version string, it seems you're running a commercially supported version of the Docker binaries.
The community can't help you with that version of Docker because it's not something which is available out in the open as far as I know. You might want to get in touch with Docker Inc. to get support for that.

@unclejack Ah, great :(

@aknuds1 I'll close this issue, but happy to hear if you've been able to get it resolved through<EMAIL_ADDRESS>


Travis kitchen tests

To get around those gem install issues, first thing id try is updating the version of rubygems. You could do that in the travis script with /opt/chef/embedded/bin/gem update --system.  Im not sure what adverse effects that would have, but another option to try is not use latest version of chef (try 12.12.15 or something).

👍 :1st_place_medal:

@ARentz07 @ChiefAlexander think you can take a look at this and get it merged?

lgtm, 👍


Question about constraint updating in API

I have finally gotten around to converting my workflow to popupcad after having manually modified my last set of dxf layers. One thing that I want to do now is to be able to programmatically vary a constraint parameter (I could just programmatically change the points but would like to modify a constraint to get that working) in one of my sketches and then rebuild the drawing and output the cut DXFs. My use case is to make hinges with set stop angles over a fine grade of a single parameter  to dial in a hinge design.
I have spent ~1.5 days in the popupcad code just seeing how everything is laid out, I wanted to do this so that I can contribute when applicable to the project and don't just have to ask you for everything.
The problem I am having is how to programmatically update sketch geometry after a constraint value is changed. This notebook shows my attempt here,
In the notebook I can locate the constraint in the sketch that I want to change. And I can update the constraint value, but I am not sure how best to then update the geometry. I thought the constraintsystem.update() routine would do this. Can you advise how best to update geometry based on changes to the constraints? If we can work this out I am happy to write this up for the API examples.

I will take a look soon and get back to you.  Yes, you're looking in the
right place.  It may take a small piece of code to glue some functionality
together.
On Tue, Mar 1, 2016 at 9:32 AM, Nick Gravish<EMAIL_ADDRESS>wrote:

I have finally gotten around to converting my workflow to popupcad after
having manually modified my last set of dxf layers. One thing that I want
to do now is to be able to programmatically vary a constraint parameter (I
could just programmatically change the points but would like to modify a
constraint to get that working) in one of my sketches and then rebuild the
drawing and output the cut DXFs. My use case is to make hinges with set
stop angles over a fine grade of a single parameter to dial in a hinge
design.
I have spent ~1.5 days in the popupcad code just seeing how everything is
laid out, I wanted to do this so that I can contribute when applicable to
the project and don't just have to ask you for everything.
The problem I am having is how to programmatically update sketch geometry
after a constraint value is changed. This notebook shows my attempt here
https://github.com/nickgravish/popupcad/blob/master/api_examples/Popupcad_API_tutorial.ipynb,
In the notebook I can locate the constraint in the sketch that I want to
change. And I can update the constraint value, but I am not sure how best
to then update the geometry. I thought the constraintsystem.update()
routine would do this. Can you advise how best to update geometry based on
changes to the constraints? If we can work this out I am happy to write
this up for the API examples.
—
Reply to this email directly or view it on GitHub
https://github.com/popupcad/popupcad/issues/92.

Thanks, I was able to figure out what I needed to do to get this functionality working. The end result is in this notebook.
When you get some time we can discuss how to best implements something like this for use.
Thanks!
Nick

thanks!


packit/rpm-build - Only collaborators can trigger Packit-as-a-Service

We want all PRs to trigger the rebuild. Why we are receiving this error on PRs from non-collaborators?
packit/rpm-build - Only collaborators can trigger Packit-as-a-Service
I have to trigger the rebuild manually. It happened e.g. in:
https://github.com/redhat-performance/tuned/pull/210

Design decision:

Utilize check from https://github.com/packit-service/packit-service/pull/237 whether p-s is allowed to be used
If a PR by a non-contributor is created: the upstream project needs to trigger p-s for the first time

Create an issue in the notification repo to whitelist the person

Once the check is green, keep on buildin'
Whitelist checking: check if the person is whitelisted, not just the namespace of the repo

Output from the today's architecture meeting:

\packit whitelist to whitelist author of the PR in the PR/project

more transparent, allows removing from the project whitelist
onetime action needed from the maintainers

I am adding EPIC so we don't receive messages from the stale bot. (Multiple things needs to be done here.)

The issue is here for some time so it's probably a time to make some conclusion. There were multiple solutions mentioned already:

Globally whitelist users -- user needs to be approved on our side for the first time.
Comment of the allowed user will unlock the PR for forever.
Provide some config option and/or comment command to enable the user across the project.

Do we have anything else? Which one(s) do we prefer?  (The solutions can be combined...)

The issue is here for some time so it's probably a time to make some conclusion. There were multiple solutions mentioned already:
* Globally whitelist users -- user needs to be approved on our side for the first time.

I really like this one - once a user is approved we can trust the person.

* Comment of the allowed user will unlock the PR for forever.

Sounds good as well, though you'd need to store this in DB probably which may be complicated to implement.

* Provide some config option and/or comment command to enable the user across the project.

Can be a great way in combination with 1)

Some update on this topic. Copr has reached to us that we don't need to require write access for the pull-request authors to start the build.
I am going to close this issue in favour of #920 since it contains the more up-to-date info and contains the solution for this problem.
The implementation is not hard and we want to start working on this soon.

Some update on this topic. Copr has reached to us that we don't need to require write access for the pull-request authors to start the build.
I am going to close this issue in favour of #920 since it contains the more up-to-date info and contains the solution for this problem.
The implementation is not hard and we want to start working on this soon.


Your character get's desynced when trying to quickly drop backpack.

Hello, I've been testing deleting some of my mods and installing them again and I came to the conclusion that this mod has a bug which doesn't let you quickly drop your backpack in raid.
Whenever you do it, your backpack will just desync and it's Icon will start glowing in and on not letting you drop it. Also, with your backpack desynced, you can't pickup items, shoot, or aim.
This mod is amazing! please fix this bug ASAP! It's unplayable...

Sorry I didn't realize a hotfix already released lol


Collect rakeDb lazy run logs

If I have a lazy rakeDb, how do I collect logs from each run?
const { change, run } = rakeDb.lazy()

await run(['up']) // I want logs of this call separately
await run(['down']) // from logs of this

The only option I see now is a global logger singleton passed to lazy({}, { logger }), a wrapper around run which resets its, and a semaphore preventing parallel runs, which is very cumbersome.
What I suggest is to allow run to accept partial RakeDbConfig as a second argument, which will be merged with the defaults that have been provided to the original rakeDb.lazy call.

Done, now it can accept a partial config via a second optional arg: rakeDb.lazy


Interaction with SPO external sharing settings

Please add some information about how these settings integrate with the external sharing settings in SharePoint Online. Specifically, what needs to be done to ensure that no conflicts occur and to ensure that external users are granted access in accordance with an organizations requirements.

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

ID: 59900927-5e5f-77a4-04be-e05a91330fca
Version Independent ID: 1e4d6b87-f57c-fa45-1630-3c32b821d93c
Content: Communicate with Teams users in another organization
Content Source: Teams/let-your-teams-users-communicate-with-other-people.md
Service: msteams
GitHub Login: @tonysmit
Microsoft Alias: tonysmit

@DeanGross Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

@officedocsbot assign @scanum

@DeanGross Thanks for reaching out Dean.  Is there anything specific you could suggest to add to the article? Alternatively please have a look at these articles and let me know if adding the relevant links in this article would help clarify the above - https://docs.microsoft.com/sharepoint/sharing-permissions-modern-experience or https://docs.microsoft.com/sharepoint/external-sharing-overview.

It would be good if the prereqs stated what External Sharing option needs to be enabled in the associated SPO site. A matrix that shows Teams External Users and Guest vs SPO External Sharing settings with the enabled/disabled functionality for each option would be very useful

Hi @DeanGross - So, I would like to talk about this a bit more. So are you recommending to add a third column with SPO settings? I think this is a good idea but I would like for you to explain more in detail if you can. Keep in mind I have worked with SP since 2013. :)

@DeanGross We appreciate any feedback that improves the content in the Microsoft docs but we haven't had any response from you so we are going to close this issue.

Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


How to experience the whiteboard feature？

I learned that jitsi has added the whiteboard function,
but after using the react-sdk 1.3.0 version on my app,
there is still no whiteboard button on the web side,
what should I do?
react-sdk version : 1.3.0
Thanks

Hello, thank you for taking an interest in that feature! At the moment the whiteboard is released only for the stage environment, so in order to experiment with it you need to use the JaaSMeeting component with useStaging set to true. You'll need a JaaS pilot account too for the appId. However I don't recommend using that in production, as the staging environment is used by the dev team and you might experience inconsistencies. We're planning on making it available for JitsiMeeting (i.e. https://meet.jit.si/) very soon.

Thanks for the efficient reply.
The server we deployed with the new version of jitsi-meeting can already experience this function, and the whiteboard switch button can be displayed using the electron api.
When is it expected to be available in JitsiMeeting?


Can't use Parsers with Vue3/Vite

I can copy/paste the code shown in the examples for AsyncParser, but they will not work when run using NodeJs/Vite/Vue3.  I see the following in the logs:
2.224 vite v4.4.10 building for production...
2.618 transforming...
10.70 [plugin:vite:resolve] Module "stream" has been externalized for browser compatibility, imported by<EMAIL_ADDRESS>See http://vitejs.dev/guide/troubleshooting.html#module-externalized-for-browser-compatibility for more details.
10.70 [plugin:vite:resolve] Module "os" has been externalized for browser compatibility, imported by<EMAIL_ADDRESS>See http://vitejs.dev/guide/troubleshooting.html#module-externalized-for-browser-compatibility for more details.
10.70 [plugin:vite:resolve] Module "stream" has been externalized for browser compatibility, imported by<EMAIL_ADDRESS>See http://vitejs.dev/guide/troubleshooting.html#module-externalized-for-browser-compatibility for more details.
12.59 ✓ 1268 modules transformed.
12.59 ✓ built in 10.37s
12.60 "Transform" is not exported by "__vite-browser-external", imported by<EMAIL_ADDRESS>12.60 file: /app/node_modules/@json2csv/node/dist/mjs/Transform.js:2:9
12.60 1: import os from 'os';
12.60 2: import { Transform, } from 'stream';
12.60             ^
12.60 3: import { StreamParser, } from '@json2csv/plainjs';
12.60 4: export default class JSON2CSVNodeTransform extends Transform {
12.60 error during build:
12.60 RollupError: "Transform" is not exported by "__vite-browser-external", imported by<EMAIL_ADDRESS>12

Clicking on the link in the error (https://vitejs.dev/guide/troubleshooting.html#module-externalized-for-browser-compatibility) says to contact the developer.

If you are using Vue, you are on the front.
You should use the plainjs flavour of json2csv instead of the node one, so you don't depend on node internals and don't need to polyfill.
You error says that whatever polyfill Vite is using is missing the Transform class from node.

Thanks, that let me get past the issue and I can use the Parser object as expected.  Maybe it's just me but how do you pass myData to StreamParser?  The examples don't show how and there is no streamParser.parse(myData) method that I can see

You can call the write method.
streamParser.write(myData) where myData can be a a string, and array or a binary array.
Also, if you are getting your data from an endpoint, I recommend that you look into the whatwg interface as it's a better abstraction.

Thanks, but it's still not clear to me.  For example, how do I pass it a file name to output the csv to?  It would be great if the examples in the README file were updated to show the write method and its proper usage.  Thanks!

btw I should not that the JSON mentioned above works fine with regular parser
      let outputAsObject = [{ some valid json data in object form}];
      const parser = new Parser(opts);
      const csv = parser.parse(outputAsObject);

Hi,
StreamParser is designed to handle string or bytearrays representing a string.
If you object is a JSON object, means that you already have it in memory and that it's not huuuuge.
So, I'd recommend that you use the sync parser as it is the fastest
      let outputAsObject = [{ some valid json data in object form}];
      const parser = new Parser(opts);
      const csv = parser.parse(outputAsObject);

If you want to use the async approach to send the json in chunks and don't block the event loop, you can use StreamParser in object mode:
const parser = new StreamParser(opts, { objectMode: true });
// ...
outputAsObject.forEach(obj => parser.write(obj));

The StreamParser in binary mode (default) is useful when you are getting your object as a byte stream. For example, from an HTTP request.
Although, the WHATWG interface is better to handle requests.
The problem with your code is that you have set incorrectly the onData callback.
You have:ç
parser.onData = (chunk) => () => csv += chunk.toString();

and it should be
parser.onData = (chunk) => csv += chunk.toString();

Then, the problem with the second write is that StreamParser auto-end when the full array is parsed.
This is because a second array might have different headers and be very different.
If you want to process more than one array, you need to specify what the separator should be. If you have them just one after the other without any space, new line or any other separator, then you can use the empty string as separator.
This is currently not working due to a bug that you just uncovered. I'll fix it asap.
Then this should allow you to parse many consecutive json arrays separated by nothing:
const parser = new StreamParser(opts, { separator: '' });

The docs are in the repo, so feel free to contribute any improvements that you think that might help other users 🙂


running the certmonger_certificate more than once always resubmits

It seems that we are doing always resubmission of the certificate. This is because we are checking wronly the force_resubmit.

Addressed by #12 ... still gotta test it though.

Tried #12 and it works for me.

Nice work, I'm still using the verify shell script so I didn't get hit with this bug.


failed to initialize

12:20:30.936 - mycroft.skills.core:load_skill:144 - ERROR - Failed to load skill: skill_easter_eggs.jarbasal
Traceback (most recent call last):
File "/home/sid/assistant/mycroft-core/mycroft/skills/core.py", line 115, in load_skill
('.py', 'rb', imp.PY_SOURCE)
File "/usr/lib/python3.6/imp.py", line 235, in load_module
return load_source(name, filename, file)
File "/usr/lib/python3.6/imp.py", line 172, in load_source
module = _load(spec)
File "", line 684, in _load
File "", line 665, in _load_unlocked
File "", line 674, in exec_module
File "", line 781, in get_code
File "", line 741, in source_to_code
File "", line 219, in _call_with_frames_removed
File "/opt/mycroft/skills/skill_easter_eggs.jarbasal/init.py", line 330
print "Invalid stardate format"
^
SyntaxError: Missing parentheses in call to 'print'. Did you mean print("Invalid stardate format")?
I'll trust you to do the poking that is needed.

Made some quick updates and mostly works on my install (picroft)

Update:  I forked it and merged my new Diehard themed holiday skill here: https://github.com/jshep321/skill_easter_eggs

hello @jshep321 , would you like to be the new maintainer of this skill? i would be happy to archive this repo and link to yours, i only request you submit it to the marketplace!

Hmmm, just playing around now as a side hobby but since the interest seems low, sure.  I'll not touch your German locale however as High School Deutsche is quite useless.  I probably won't submit to the marketplace as including these mp3's might cause DMCA bots to wake up, even though these are most likely covered under US fair use laws.

Hi there, Jarbas asked for my thoughts on this from Mycroft's perspective.
Assuming all the files are short audio snippets and not used for commercial profit, I think this fits within the terms of fair use under US copyright law. As such we would accept it into the Mycroft Skills Marketplace if you chose to submit it. In the unlikely event that a copyright holder disagreed with this, we would have that conversation with them.

Skill has been ported to Python 3 and the latest OVOS standards, closing this issue


File -> Load... To load a local model and show it in the viewer.

@aymanhab, since I was not sure about where to put a File -> Load... menu in the viewer, I just allow users to drop the gltf file, and it is opened in the viewer in this PR #61.
I think what we should do is to add a + button at top, that allows users to directly create a model that will be stored in the database in the backend. This would imply that we have also to implement a "My models" page, to see existing models.
What are your thoughts?


feat: added event routing support

Description
Added activity tracker event routing support.
Types of changes in this PR
No release required

[ ] Examples or tests (addition or updates of examples or tests)
[ ] Documentation update
[ ] CI-related update (pipeline, etc.)
[ ] Other changes that don't affect Terraform code

Release required

[ ] Bug fix (patch release (x.x.X): Change that fixes an issue and is compatible with earlier versions)
[x] New feature (minor release (x.X.x): Change that adds functionality and is compatible with earlier versions)
[ ] Breaking change (major release (X.x.x): Change that is likely incompatible with previous versions)

Release notes content

Checklist for reviewers

[ ] The PR references a GitHub issue.
[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

Merge actions for mergers

Merge by using "Squash and merge".

Use a relevant conventional commit message that is based on the PR contents and any release notes provided by the PR author.
The commit message determines whether a new version of the module is needed, and if so, which semver increment to use (major, minor, or patch).

@Soaib024 @ocofaigh - What is the outlook on this PR?

@Soaib024 @ocofaigh - What is the outlook on this PR?
PR was raised against issue #3433 created in github.ibm.com
It's parked for now until a new logging agent  is rolled out
hence closing it for now


Avoid tag wrapping in phrasing context

This is a bit of an addition to supporting VSCs html.format options.
By default VSCode will avoid formatting tags listed in "html.format.unformatted". By default it uses the tags defined in HTML spec for Phrasing content. This is helpful for avoiding issues caused by whitespace.
Antlers tags inside those rules should probably also not be wrapped. When it doesn't the formatting results are quite strange and could possibly cause whitespace issues in the output.

   {{ author:name }} ‐

    {{ event_date ?? date }}

The unformatted settings will be respected in 1.0.22
Going to spend some time with #8 before pushing the update.


Add change log file.

Hi, as I can see, you are carefully fill tags and labels for issues in your repo.
For such cases I create a github_changelog_generator, that generate change log file based on tags, issues and merged pull requests from :octocat: Issue Tracker.
This PR add change log file to your repo (generated by this script).
You can check, how it is look like here: Change Log
Some essential features, that has this script:

it exclude not-related to changelog issues (any issue, that has label question duplicate invalid wontfix )
Distinguish issues according labels:

Merged pull requests (all merged pull-requests)
Bug fixes (by label bug in issue)
Enhancements (by label enhancement in issue)
Issues (closed issues w/o any labels)

Generate neat Change Log file according basic change log guidelines.

You can easily update this file in future by simply run script: github_changelog_generator danielgindi/ios-charts in your repo folder and it make your Change Log file up-to-date again!
Hope you find this commit as useful. :wink:

Ok let's try this, see what it produces

@skywinder @danielgindi  I notice the automatic generated change log includes closed issues, which feels verbose and not useful? Could we not include it?


Stability level of sumologic extension is not defined

Seeing an error message logged out when starting otelcol-sumo version 0.56.0-sumo-0 with info-level logging enabled:
2022-07-27T22:57:32.063Z        info    components/components.go:30     Stability level of component is undefined       {"kind": "exporter", "data_type": "logs", "name": "sumologic", "stability": "undefined"}

I suspect we're missing an attribute in our extension to set the stability level. Somehow, seeing "undefined" here is even worse than "alpha" or "experimental". Given that we've announced the general availability of our OTC distribution and we are offering commercial support for our distribution (and exporter extension) to our customers, it would be great if we could set this to "beta" or better. 😊
I got this message on an amd64 linux host running Debian Linux.

Done in https://github.com/SumoLogic/sumologic-otel-collector/pull/769


Add x stop for drive brake

TODO: Test and implement after balancing

What is the purpose of this set off changes? I don't understand from just the title.


Link broken in readme.md

This link doesn't exist anymore: https://ui-wedeploy.wedeploy.io/docs/deploy/deploying-java-maven.html
Sounds like maven was replaced by gradle: https://ui-wedeploy.wedeploy.io/docs/deploy/deploying-java.
I don't know if I should have created a issue for this.
I just try to help you.

Thanks @murillocg!


Package dependencies are too strict

In 0985cb26d902aaf201561b0878d0febf5fc4d4d2 the install_requires of the package was updated to require specific versions of all dependencies, recursively.
For testing this package itself, locking that way is great. But for downstream packages this at best prevents me from updating any of these packages, at worst causes conflicts with other packages.

the only package with a version requirement is an inequality  - "pyjwt>=2.0.0" - versions earlier than 2.0 are not compatible and are a source of endless bug tickets, the solution to each of course is update pyjwt.

Thanks for pointing it out! Have released the dependencies from exact versions and set it only where it was needed (for PyJWT). Have updated PyPI.


HDDS-9227. Recon: OverView Page Summary End Point Correction

What changes were proposed in this pull request?
Correction

/api/v1/keys/open?limit=0 Correct to /api/v1/keys/open/summary in Overview Page.
/api/v1/keys/deletePending?limit=1 correct to /api/v1/keys/deletePending/summary

What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-9227
How was this patch tested?
Manually

could you provide more details why is this change needed @smitajoshi12? I don't see much either in the jira description or here, is there a change that broke the current behaviour? when did the response change?

could you provide more details why is this change needed @smitajoshi12? I don't see much either in the jira description or here, is there a change that broke the current behaviour? when did the response change?

@dombizita The modifications introduced in the mentioned PR inadvertently caused a reversal to the previous API endpoint calls. Unfortunatly, we overlooked including this particular alteration in the set of changes. As a result, the summaries on the overview page could function properly.

The best thing to do right now is to merge this first and then merge the OM-DB Insights PR

Thanks @smitajoshi12 LGTM +1

thanks for the patch @smitajoshi12! thanks for the review @ArafatKhan2198 and @devmadhuu!


download fails because repository changed atom/atom-shell to atom/electon

download fails because repository changed atom/atom-shell to atom/electon, maybe.
$ grunt download-atom-shell
Running "download-atom-shell" task
>> Cannot find atom-shell v0.23.0 from GitHub [Error: Not Found]
Warning: Task "download-atom-shell" failed. Use --force to continue.

I think L129 and L190 of tasks/download-atom-shell-task.coffee need to be updated to reflect the new name.

This is being handled in #23. Thanks for the helpful report :bow:

Another solution in #24, in a way that allows old builds of atom-shell to be used as well.

Fixed by #23


Include the databaseId (GUID) in the verbose logs when starting a migration

Right now, when starting a migration, we only print the GraphQL ID (beginning RM_ to the logs).
This improves the logs by also outputting the GUID (aka databaseId), which is referred to in certain places (e.g. the Migration Log issue and text file).
We don't actually write any new logging code to do this - we just request the databaseId field in our GraphQL queries, and rely on existing behaviour which logs GraphQL responses to the verbose log.

[x] Did you write/update appropriate tests
[x] Release notes updated (if appropriate)
[x] Appropriate logging output
[ ] Issue linked
[ ] Docs updated (or issue created)
[ ] New package licenses are added to ThirdPartyNotices.txt (if applicable)

@hfishback01 Could you please give this another quick 👍🏻 ? I just had to resolve the merge conflicts.