Issue parsing Ahoy::Event properties hash attribute

Ahoy stores Events with 2 basic attributes: name: (e.g. click, view, etc) and properties.  properties is a hash which has varying keys depending on what type of event.  For example, a click event may have keys like class, and page, while view has "url" and "page_title"
For the purpose of creating some site analytics, we need to parse the ahoy events data, but in the default configuration, it is difficult to take the population of events and discern e.g. how many times a particular page was viewed.
So in  the site_analytics branch, we have begun to record a custom "Viewed Article" event with custom attributes.  Ahoy then stores an event with name:  "Viewed Article" and a properties hash with our custom keys.
However, I have still had difficulty querying the data from properties hash.  By default ahoy stores properties as a text data object.  I reached out to @ankane who is the owner of the gem who is offering guidance in a thread that is still ongoing here: https://github.com/ankane/ahoy/issues/120

This issue has been addressed in part by adding metadata to Ahoy Events, which partial solution has been added to the branch site_analytics which will need to be rebased off current master. That branch includes a first draft of an analytics page that graphs pageviews by day, visits by user location, etc.  We do not track clicks, but at this point, I do not remember if we had trouble doing it, or if we just hadn't figured out what metrics we wanted to track.

@campgurus  Have you solved this?

no sir.  Just coming back to take a look again.  It would be nice to have some analytics.

finally got an answer on this.  Based on the slowness of the query, we may need to figure out something asynchronous or not have real time data, but, it does return the desired results from the Events.properties hash.
https://stackoverflow.com/questions/35753631/querying-ahoy-gem-events

relates/dupe to #145

Ok.  Time to rethink this effort.  The above query is so resource intensive as to not make sense for us. Right now, the Ahoy::Events table is taking up the majority of our db capacity but giving us little of the feedback we wanted. Instead of recording every view of every page and every click of every button int he generic way we are doing it now, I would suggest creating custom trackers within the controllers for the events we are interested in. Specifically, remove the code $(document).ready(function(){   ahoy.trackAll(); });
from ahoy.js. and instead basing these events in the controllers where we could specify the properties we want to track like ahoy.track "Viewed book", title: "Hot, Flat, and Crowded"
I believe it would be easy to track article views, follows, comments, which would give us enough to create a relatively robust "trending" list (for lack of a better term). Events would still belong to Visits, which attach to users and contain basic info we could use in parsing events by user, user location, device etc allowing us to build a decent analytics page that could include:

most viewed cases
visitors by geography/time
case viewers by geography

of people following a specific case

follows over time
etc.

With additional work, I think we could record other information that will help shape the product like how many people click on the related case photos, or how many click to follow a case or comment, but don't complete the action. How many people click thru from the recent updates list, how many people search, etc.

As pointed out by @trystant, as of March 24, 2016, the gem documentation outlines a method for querying Events based on elements of the properties hash: https://github.com/ankane/ahoy#querying-properties
However, i am still unable to retrieve and records. I have tried both the jsonb approach (Throws error " "no implicit conversion of Hash to string") and the text datatype (no error but returns 0 results)

Thinking of using json instead of jsonb to see if it works.  Here is  link to how to change the datatype: http://stackoverflow.com/questions/27343231/how-to-use-jsonb-in-rails

Can you provide example queries & the results? I think that would help with answers to those specific questions.

here is one:
~/development/blackops$ heroku run rails c -a blackopswiki           [master]
Running rails c on ⬢ blackopswiki... up, run.1468
Loading production environment (Rails 4.2.6)
irb(main):001:0> Ahoy::Event.where(name: "$view").count
   (108.3ms)  SELECT COUNT(*) FROM "ahoy_events" WHERE "ahoy_events"."name" = $1  [["name", "$view"]]
   (108.3ms)  SELECT COUNT(*) FROM "ahoy_events" WHERE "ahoy_events"."name" = $1  [["name", "$view"]]
=> 69196
irb(main):002:0> Ahoy::Event.where(name: "$view").where("properties LIKE '%\"page\": \"/articles/israel-santos-banos\"%'").count
   (227.2ms)  SELECT COUNT(*) FROM "ahoy_events" WHERE "ahoy_events"."name" = $1 AND (properties LIKE '%"page": "/articles/israel-santos-banos"%')  [["name", "$view"]]
   (227.2ms)  SELECT COUNT(*) FROM "ahoy_events" WHERE "ahoy_events"."name" = $1 AND (properties LIKE '%"page": "/articles/israel-santos-banos"%')  [["name", "$view"]]
=> 0
irb(main):003:0>

attempting to find this result: http://ebwiki.org/admin/ahoy~event/00fc3a49-7245-4a61-a50e-e22554915228

Doing the following provides an answer of sorts. The first line gets all the events where the page property isn't null. The second counts all the events that visit the page "/articles/israel-santos-banos"
events_with_pages = Ahoy::Event.all.select { |event| event.properties['page'] != nil }
events_with_pages.select {|event| event.properties["page"] =~ /israel-rodriguez/}.size

The problem is, searching this way is slow, cumbersome and not too productive. A better solution would be to find a better data store for this Ahoy data. The current solution doesn't pass muster.

BTW, I figured this out from the response to your stackoverflow answer. It might be worth checking with the Ahoy creator about best practices for storing & retrieving Ahoy data.

One final thing; we installed Ahoy with the idea that it would be relatively simple to get that data and review it. If that's not going to be the case, that changes the whole value proposition of Ahoy. In the future, I really do think we need to take a closer look at a feature in advance so that we can accurately judge how much work it's going to be to implement it in a way that works well.

Unless there's something else, I'm closing this thread.

not sure why we closed this issue.  In any case, I think the solution may be related to this SO thread: http://stackoverflow.com/questions/9814622/searching-serialized-data-using-active-record


How do you get 'num_examples_per_epoch'=5.32e4?

Hi Bily,
can I ask how did you obtain the constant 'num_examples_per_epoch' in your config file?
I looked into the training dataset by the following code snippet,
dataset = VID('data/train_imdb.pickle', 100)
print(dataset.epoch_size) # output is 8418

The epoch size is 8418 so the number of examples per epoch (num_examples_per_epoch) should be
8418 * 8 = 67,344?
Can you tell me if I mistook anything?
Best regards,
Yiming

Hi,
It is directly taken from https://github.com/bertinetto/siamese-fc/blob/f586d4e32091184240ab7b496ea4df9cb81dd001/training/experiment.m#L24


Version 4.3 not published to npm

The latest version in this repository is V4.3 while in npm V4.2 is still the latest version (npm debug).
Please publish this version to npm (if possible).

Please search the issues in the future.
https://github.com/visionmedia/debug/issues/785#issuecomment-699666241


Issues with CdsControlAction in v6

Describe the bug
We use the CdsControlAction action="label" component to display an ℹ icon next to the label, with a tooltip to add information. For example:

This worked fine and well in v5, but starting with v6, we are experiencing 2 issues:

The icon does not get vertically centered anymore:
 vs.

The tooltip simply does not appear anymore.

How to reproduce
https://codesandbox.io/s/input-tooltip-issue-v6-qohlfv?file=/src/App.tsx
Steps to reproduce the behavior: Simply hover over the ℹ icon, the tooltip does not appear. It does work fine outside an input.
Expected behavior
For reference: https://codesandbox.io/s/input-tooltip-issue-v5-nrjekz?file=/src/App.tsx
Versions
Clarity project:

[x] Clarity Core
[ ] Clarity Angular/UI

Clarity version:

[ ] v5.x
[x] v6.x

Framework:

[ ] Angular
[x] React
[ ] Vue
[ ] Other:

Framework version:
React 17

In v6, the ControlAction was refactored to be derived from the new ButtonAction element. Part of that work introduced an absolute positioned touch target on top of the action button that seems to be interfering with the tooltip: https://github.com/vmware-clarity/core/blob/main/projects/core/src/button-action/button-action.element.scss#L89
For #2, you can workaround it by moving the ControlAction inside of the TooltipElement: https://codesandbox.io/s/input-tooltip-issue-v6-forked-o97x07?file=/src/App.tsx I'll have to keep investigating a better solution otherwise


Doc: refer to array properly

Woops! Thanks for fixing this!
Would you mind fixing an issue our linter had? https://travis-ci.org/GoogleCloudPlatform/google-cloud-node/jobs/156155832

Should be ok now

Coverage remained the same at 100.0% when pulling 3531e1e1252ddbe64275983ac042e0e42fd204d1 on kevinresol:patch-5 into 353d9f236ee5814667b88fb2fec2f02d6faf2870 on GoogleCloudPlatform:master.

Perfect, thanks!


Drops the Serving-specific stats reporter logic from knative/pkg

This will be breaking downstream, so I'm going to HOLD this while I stage those PRs.
/hold

Serving change is staged: https://github.com/knative/serving/pull/6940

Eventing is here: https://github.com/knative/eventing/pull/2622

/unhold
Paul's changes are in.

/test pull-knative-pkg-unit-tests


Add ability to set custom user agent

It would be nice to have an ability to set custom user agent, for example, if you want to differentiate between different API users in Cachet logs. This change adds such feature.

Looks good to me. Thanks !


Enable specifying multiple config files

This PR adds the ability to use multi-config files handling. This is useful for instance when using a piece of software for many different configurations which can be divided into some sort of parent and children classes.
As an example we could have something like
general_conf_for cars
             |
           /   \
         /       \
       /           \
sports_car    old_cars

With the PR we can now specify something like --my-config general_conf_for_cars sports_car_conf. In terms of priority we have command line > sports_car_conf > general_conf_for_cars > default

The latest commit allows adding config files as arguments inside a config file.
To use the example above, We could now say that we have a Renault Clio Car. This car's config file would then contian the following line:
config=[consumer_cars, renault]

And the consumer_cars config would itself contain a config as an argument:
config=general_conf_for_cars

As such this allows the user to build a full inheritance system.
In terms of evaluation order for the arguments we have the following:

command_line args
renault_clio_car args
renault args
consumer_car args
general_conf_for_cars
default_values

In the general case, for each recursion level, the existing command_line args take precedence, then all the config files are parsed in the reversed order of appearance on the command line, and their args are added to the command line if not there already, and then we start a new recursion round with the newly parsed confs.
Note that we have put some guards to avoid circular dependencies, but the first pass (the config files specified in the original command line arguments) are not registered as already parsed. This will be added in a future commit depending on the complexity.


How to --- Do not store service_principal entries.json When run az login --service principal

az feedback auto-generates most of the information requested below, as of CLI version 2.0.62

Related command

Describe the bug

To Reproduce

Expected behavior

Environment summary

Additional context

Thank you for opening this issue, we will look into it.


upgrade-scale: updates for 8-9 scale testing

This originally targeted upgrade-scale branch but it was not needed anymore (and got deleted).
Re-targeting this to master for documentation purposes.

wrong branch

I didn't look at the added files in hostscripts/upgrade-scale , rest doesn't interfere with anything that is not cloud=u1.


Update golang to latest version

Coverage remained the same at 81.286% when pulling 118130e39b75eacde9a43a6b1f350b6f457f5f56 on up-golang into a68546c68dee8c04f60f50d994e1adf482d43364 on master.

@dotchev I think we should merge this.


Improve Health

Such a good work !
Just click on the green button to Submit :dagger:

Great job @nbeguier. Happy christmas :snowman_with_snow:


raw_src_qc - os.link(self.inputs.src_file, linked_src_file) - PermissionError: [Errno 1] Operation not permitted

Dear Developers,
I currently try to deploy a preconfigured QSIPrep singularity container on our CentOS HPC. Aiming for some test cycles with a single subject I run in a permission problem within the container. The error concerns hard link creation via python’s “os.link()”. All other nodes seem to run flawlessly.
[Node] Setting-up “qsiprep_wf.single_subject_control01_wf.dwi_preproc_wf.pre_hmc_wf.merge_and_denoise_wf.dwi_qc_wf.raw_src_qc” in “/sngl/scratch/qsiprep_wf/single_subject_control01_wf/dwi_preproc_wf/pre_hmc_wf/merge_and_denoise_wf/dwi_qc_wf/raw_src_qc”.
210416-09:42:20,902 nipype.workflow INFO:
[Node] Setting-up “qsiprep_wf.single_subject_control01_wf.dwi_preproc_wf.pre_hmc_wf.dwi_qc_wf.raw_src_qc” in “/sngl/scratch/qsiprep_wf/single_subject_control01_wf/dwi_preproc_wf/pre_hmc_wf/dwi_qc_wf/raw_src_qc”.
210416-09:42:21,205 nipype.workflow INFO:
[Node] Running “raw_src_qc” (“qsiprep.interfaces.dsi_studio.DSIStudioSrcQC”)
210416-09:42:21,205 nipype.workflow INFO:
[Node] Running “raw_src_qc” (“qsiprep.interfaces.dsi_studio.DSIStudioSrcQC”)
210416-09:42:21,217 nipype.workflow WARNING:
Storing result file without outputs
210416-09:42:21,218 nipype.workflow WARNING:
Storing result file without outputs
210416-09:42:21,221 nipype.workflow WARNING:
[Node] Error on “qsiprep_wf.single_subject_control01_wf.dwi_preproc_wf.pre_hmc_wf.dwi_qc_wf.raw_src_qc” (/sngl/scratch/qsiprep_wf/single_subject_control01_wf/dwi_preproc_wf/pre_hmc_wf/dwi_qc_wf/raw_src_qc)
210416-09:42:21,221 nipype.workflow WARNING:
[Node] Error on “qsiprep_wf.single_subject_control01_wf.dwi_preproc_wf.pre_hmc_wf.merge_and_denoise_wf.dwi_qc_wf.raw_src_qc” (/sngl/scratch/qsiprep_wf/single_subject_control01_wf/dwi_preproc_wf/pre_hmc_wf/merge_and_denoise_wf/dwi_qc_wf/raw_src_qc)
210416-09:42:22,505 nipype.workflow ERROR:
Node raw_src_qc failed to run on host node309.
210416-09:42:22,529 nipype.workflow ERROR:
Saving crash info to /sngl/qsiprep-output/qsiprep/sub-control01/log/20210416-094111_6865a7b6-176e-4c40-807f-674909d16ce1/crash-20210416-094222-fatx405-raw_src_qc-8f455aeb-590c-4cc6-b3ab-e71cfb9ca67a.txt
Traceback (most recent call last):
File “/usr/local/miniconda/lib/python3.7/site-packages/nipype/pipeline/plugins/multiproc.py”, line 67, in run_node
result[“result”] = node.run(updatehash=updatehash)
File “/usr/local/miniconda/lib/python3.7/site-packages/nipype/pipeline/engine/nodes.py”, line 516, in run
result = self._run_interface(execute=True)
File “/usr/local/miniconda/lib/python3.7/site-packages/nipype/pipeline/engine/nodes.py”, line 635, in _run_interface
return self._run_command(execute)
File “/usr/local/miniconda/lib/python3.7/site-packages/nipype/pipeline/engine/nodes.py”, line 741, in _run_command
result = self._interface.run(cwd=outdir)
File “/usr/local/miniconda/lib/python3.7/site-packages/nipype/interfaces/base/core.py”, line 434, in run
runtime = self._run_interface(runtime)
File “/usr/local/miniconda/lib/python3.7/site-packages/qsiprep/interfaces/dsi_studio.py”, line 108, in _run_interface
os.link(self.inputs.src_file, linked_src_file)
PermissionError: [Errno 1] Operation not permitted: ‘/sngl/scratch/qsiprep_wf/single_subject_control01_wf/dwi_preproc_wf/pre_hmc_wf/dwi_qc_wf/raw_src_qc/sub-control01_dwi_merged.src.gz’ → ‘/sngl/scratch/qsiprep_wf/single_subject_control01_wf/dwi_preproc_wf/pre_hmc_wf/dwi_qc_wf/raw_src_qc/src_qc/sub-control01_dwi_merged.src.gz’
210416-09:42:22,542 nipype.workflow ERROR:
Node raw_src_qc failed to run on host node309.
210416-09:42:22,549 nipype.workflow ERROR:
Saving crash info to /sngl/qsiprep-output/qsiprep/sub-control01/log/20210416-094111_6865a7b6-176e-4c40-807f-674909d16ce1/crash-20210416-094222-fatx405-raw_src_qc-979b9de1-8ed0-4a3b-94fa-63f0b1ed02e0.txt
Traceback (most recent call last):
File “/usr/local/miniconda/lib/python3.7/site-packages/nipype/pipeline/plugins/multiproc.py”, line 67, in run_node
result[“result”] = node.run(updatehash=updatehash)
File “/usr/local/miniconda/lib/python3.7/site-packages/nipype/pipeline/engine/nodes.py”, line 516, in run
result = self._run_interface(execute=True)
File “/usr/local/miniconda/lib/python3.7/site-packages/nipype/pipeline/engine/nodes.py”, line 635, in _run_interface
return self._run_command(execute)
File “/usr/local/miniconda/lib/python3.7/site-packages/nipype/pipeline/engine/nodes.py”, line 741, in _run_command
result = self._interface.run(cwd=outdir)
File “/usr/local/miniconda/lib/python3.7/site-packages/nipype/interfaces/base/core.py”, line 434, in run
runtime = self._run_interface(runtime)
File “/usr/local/miniconda/lib/python3.7/site-packages/qsiprep/interfaces/dsi_studio.py”, line 108, in _run_interface
os.link(self.inputs.src_file, linked_src_file)
PermissionError: [Errno 1] Operation not permitted: ‘/sngl/scratch/qsiprep_wf/single_subject_control01_wf/dwi_preproc_wf/pre_hmc_wf/merge_and_denoise_wf/dwi_qc_wf/raw_src_qc/sub-control01_dwi_merged.src.gz’ → ‘/sngl/scratch/qsiprep_wf/single_subject_control01_wf/dwi_preproc_wf/pre_hmc_wf/merge_and_denoise_wf/dwi_qc_wf/raw_src_qc/src_qc/sub-control01_dwi_merged.src.gz’

This error always involves os.link(file1,file2) and appears in 3 other instances further down the pipeline which eventually terminates prematurely.
I tried to reproduce the error when shelling into the container and I am indeed not able to write with os.link() in the directories concerned.

Writing with for example touch or mkdir works fine. When trying to write to these directories outside the container there is no problem at all. I presume there is something wrong with my environment influencing Python within the container but I don’t know how to further debug it.
Further information:
Singularity version on HPC
HPC. 3.5.2
Local machine: 3.7.1

QSIPrep versions (tried multiple preconfigured containers from dockerhub)
0.13.0RC2
0.13.0RC1
0.12.2

singularity run --cleanenv --no-home --userns -B /work/fatx405/projects/2104_dwiproc_hchs/02_env:/mnt
-B $raw_bids:/sngl/data
-B $derivatives:/sngl/out
-B $derivatives/qsiprep:/sngl/qsiprep-output
-B $intermediate:/sngl/scratch
-B $env_dir:/sngl/env
/work/fatx405/projects/2104_dwiproc_hchs/02_env/qsiprep-0.13.0RC2
/sngl/data /sngl/qsiprep-output participant
-w /sngl/scratch/
–participant-label ${subjs_subarr[@]} \ #${subjs_subarr[@]} contains the sub ID
–recon_input /sngl/out/qsiprep
–recon_spec mrtrix_singleshell_ss3t
–fs-license-file /sngl/env/freesurfer_license.txt
–output-resolution 1.3
–output-space T1w
–use-syn-sdc \

I also tried to follow some recommendations from NeuroStars and fMRIprep documentation by adding --cleanenv --no-home --userns to singularity and unsetting PYTHONPATH but without avail.
The script and the complete qsiprep output can be found as attached files to a corresponding NeuroStars post (https://neurostars.org/t/qsiprep-raw-src-qc-os-link-self-inputs-src-file-linked-src-file-permissionerror-errno-1-operation-not-permitted/19076).
Thanks in advance for your help. Happy to provide further information.

I contacted the admins of our HPC as well and they had a solution. BeeGFS which underlies the partition I was writing to does not allow hard links between directories. Exchanging the initial $intermed_dir with /scratch (other partition) solved the problem because /scratch allows hard link creation.


Query about “Controlling Access to the Kubernetes API”

In the docs on "Controlling access to K8s" we say: In a typical Kubernetes cluster, the API serves on port 443, protected by TLS, but in "API server ports and IPs" secure port default is 6443
Which statement is correct?

/retitle Query about “Controlling Access to the Kubernetes API”
The true situation is that the API server listens on a nonstandard port, 6443, by default. As a cluster operator you're expected either to override this to the actual port used for HTTP over TLS (443), or to insert a reverse HTTP or TLS SNI proxy between the client and the API server.
I don't think the existing document makes that clear, and we don't have a good page about the API server to add that detail to.
/language en
/sig architecture
I suppose

Hey, I would like to work on this issue. Can I assign it to myself?

/assign @jayesh-srivastava

/remove-lifecycle rotten

As sftim said above, the document could be more clear to avoid confusion.
Hey @jayesh-srivastava, would you like to fix this? 😊
/remove-lifecycle rotten

/triage accepted

I think @jayesh-srivastava is busy for other stuff. You can assign yourself again anytime.
/unassign @jayesh-srivastava
Now, anyone can assign themself using /assign command. It would be nice if someone willing to work on this.

@Sea-n apologies for being inactive. Was acquired a bit. Would be happy to fix this, some help can be great as well.

Content unclear about “Controlling Access to the Kubernetes API” and ports in use
/remove-lifecycle rotten
/lifecycle stale


Using groupBy with date part

Hi.. I cannot find a way to group by my array with a date part (dd/MM/yyyy).
I have to group by day/month/year of a date property, how can I do it?
thanks!

What's your data structure?

Messages = [
{ id: 1, sent: Date(), message: "" },
{ id: 1, sent: Date(), message: "" }
]
It's something like that... 'Sent' is timespam
I want to group messages by MM/dd/yyyy
Thanks

Well, I had a problem with groupBy with dates because my dates were formatted as timestamp, so I changed the format to MM/dd/yyyy before render the view, then it worked. You can format them both server-side as client-side.

To resolve that I use map function to before groupBy, so, on map function I create a new formatted property (MM/dd/yyyy).
It works, but I don't know if that is the better way to do that. (honestly, I didn't like the solution)
ng-repeat="(key, value) in mensagens | map: dateFormatter | groupBy: 'dateFormatted'"

i'm using the same solution but I assume it has performance issues with big lists :(

Above solution proposed by @rafaelss95 can work if I want to group by date. But I have a requirement to group by Date as well as time (complete timestamp), and in such cases groupBy is misbehaving. It would be great if someone would investigate the issue.

@karandesai28 try to change the date format to something (for example) like mysql datetime: YYYY-MM-DD HH:MM:SS so the orderBy compare function will compare the full timestamp instead to compare only the date, I guess.

@codekraft-studio Thanks for reply. Just to confirm, I mentioned about groupBy, not orderBy. Will it work for groupBy? I'll try from my end though.

@karandesai28 yea sorry I answered in early morning and I was still sleepy, I intended groupBy obviously :)

Well I tried in groupBy, but it didn't work


RHIROS-469 | (Cherry-picked) orderByParam is now max_io instead of io, systems endpoint

Changes:

Updated orderByParam from io to max_io

Reasons:

io is now longer a sortable parameter

merged, thanks @saltgen


Power estimation seems too high

Describe the bug
I ran the sky130_sram_1rw_tiny example and the power report shows hundreds of watts, which seems wrong. Could this be a bug? How can I solve it?

Version
1.2.48
To Reproduce
cd OpenRAM/macros
make sky130_sram_1rw_tiny

It is a known issue that the power numbers have not been calibrated. The cell powers in the tech file are not correct.

Hi, thank you for the response. Is it possible to obtain the power estimation in another way?

You can run a spice simulation.
These are the placeholder values for leakage:
https://github.com/VLSIDA/OpenRAM/blob/stable/technology%2Fsky130%2Ftech%2Ftech.py#L744
Similarly, there are estimates for capacitance used in dynamic power which might be set correctly.

Thank you very much. With this approach, is it possible to obtain the .lib file as well? I would like to use these memories as macros in another design on OpenLane and I would like to have a complete power estimation.

I didn't quite understand this last part.
In this case  how was the power estimated on ? It seems to have more accurate values?

Sorry @mguthaus , in this issue it seems to me that you have the same problem as mine. I have a design on OpenLane that uses OpenRAM macros and I need to estimate the entire system power. However, to do this, it seems from this issue that a gate-level netlist is needed rather than the .lib file. Were you able to solve this problem?

There are no gate-level netlists of SRAM macros. It must use a .lib file.

I understand, but with OpenLane, without the gate-level netlist of a macro, I need to use ///sta-blackbox, which seems to ignore the .lib file.
My question is, with the .lib file but without the gate-level netlist, is it possible to estimate the power of these memories within another design, and if so, how?

I understand, however, it seems that OpenLane is unable to read the .lib files of the macros to calculate power, and it only works when the gate-level netlist is present.
So my question is, based on this issue, how can we include the memory macro in power calculation when we only have the .lib file available?

This would be an openlane issue. Gate level doesn't exist for any memory macro even with other tools. It doesn't use the cells from the library.
Openlane should be using the .lib file for the memory macro to get the power.

Perfect, so you're confirming that it's impossible for OpenLane to estimate the power of these memories since only the .lib file is available?

I'm confirming that openlane should only need the lib file to estimate power.

So, did you manage to solve this issue?


Install az login only

Hi,
Is it possible to install a lightweight version of az login?
We're installing just az login for CI/CD jobs (with pip/pipenv). But installing azure-cli downloads an extreme amount of dependencies for ever sub module.
I have tried to install azure-cli-profile, but it does not provide the cli.
Othwerwise, it would be super useful to be able to install selected command_modules via python "extras".

@jiasli for awareness

Currently, az login is part of azure-cli package and there is no way to install it separately. azure-cli-profile is the old package for az login before all command modules are merged into one monolithic azure-cli package.

Why was this closed?
This feature request is still relevant to us.

We have no way to install az login separately and will track loose weight of Azure CLI package in https://github.com/Azure/azure-cli/issues/7387

@yonzhan, yes i get it, @jiasli already said that.
But that's not how feature requests work... You can't close a feature request by arguing that the feature is not implemented... That's what status "open" is for.
It can be a decision to "won't fix" from the PM/core team (based on e.g. architectural limitations), but that dialogue has not taken place here.
Even though the size of azure-cli is trimmed, it will install a huge amount of unneeded dependencies for users only needing az login (which is a very common scenario). It takes time to install, but also clutters the venv and increases the risk of conflicts/bugs.
Is it possible to implement "extras" python modules?

In a world of lightweight containers this would be great. I can't understand why this was chose to closed. I understand that it would take work, everything does.

Reopened this as #27755


[Android] injectScript not work!

I write bellow code:
function webViewBridgeReady(cb) {
if (window.WebViewBridge) {
cb(window.WebViewBridge);
return;
}
function handler() {
  document.removeEventListener('WebViewBridge', handler, false);
  cb(window.WebViewBridge);
}
document.addEventListener('WebViewBridge', handler, false);

}
function goDetail(id) {
WebViewBridge.send("jump:" + id);
}
webViewBridgeReady(function (webViewBridge) {
webViewBridge.send("message from webview");
});
the html is get from server, html have a button and click to invoke the goDetail function,but in android the
log cat say 'goDetail is not defined'.
can you help me？

@peng051410  you should change your html code

me either, the same code as iOS, but the injected code seems not work.

那你能在安卓上加载出来网页吗？

在 2016年3月18日，下午4:52，quentinzhang<EMAIL_ADDRESS>写道：
me either, the same code as iOS, but the injected code seems not work.
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub https://github.com/alinz/react-native-webview-bridge/issues/69#issuecomment-198261313

你注入的js代码没作用？ 能否QQ上讨论下呢？ 我的QQ413279774

在 2016年3月18日，下午4:52，quentinzhang<EMAIL_ADDRESS>写道：
me either, the same code as iOS, but the injected code seems not work.
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub https://github.com/alinz/react-native-webview-bridge/issues/69#issuecomment-198261313

@quentinzhang so can you load your page in Android?

I solve this in iOS  like #59

@peng051410 I solve this in iOS like #59

@YangYouYong  已加你qq!

能加载出网页，只是注入的代码没有生效。已经在QQ上添加你了。
On Fri, Mar 18, 2016 at 4:54 PM, YangYouYong<EMAIL_ADDRESS>wrote:

那你能在安卓上加载出来网页吗？

在 2016年3月18日，下午4:52，quentinzhang<EMAIL_ADDRESS>写道：
me either, the same code as iOS, but the injected code seems not work.
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub <
https://github.com/alinz/react-native-webview-bridge/issues/69#issuecomment-198261313

—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/alinz/react-native-webview-bridge/issues/69#issuecomment-198262173

got the same issue

I think what write handler() is old. change injectedScript
(function () {
if (WebViewBridge) {
                  WebViewBridge.onMessage = function (message) {
                    if (message === "hello from react-native") {
                      WebViewBridge.send("got the message inside webview");
                    }
                  };

                  WebViewBridge.send("hello from webview");
                }
              }());

If you read example and tried it, you will solve problem

I experienced something interesting.
My script works on iOS but applying it to android did not work.

first as @g6ling suggested watch example
second —that is what is interesting— erase all javascript comments in your scripts!!!

it broke my script on android!!! It was why it did not work...

Illustration:
const scriptNotWorking = `
 // a comment <= erase all comments like this one!!!
 alert('it won\'t work');
`;

const scriptWorking = `
 alert('it will work');
`;

I also am unable to inject JS on android.  If I change the type to plain old WebView with all the same props then it works.

I came up with the same problem.  The code works on iOS buy didn't work on Android even the simplest code as MacKentoch's.

Any updates or eta on a resolution to this related to android injected javascript not working, please?

@weremax, recently @hmedney made a contribution to the code that solves android issues. Have you tried pulling it from master branch?

@alinz , I have pulled from master and still am having the issue I am on react-native v 0.30.0 not the latest version could that be causing an issue.

GEEEEEEEZUZ christ
Thank @MacKentoch for pointing me in the direction of retarded errors. I also had problems executing injected scripts on Android whereas it worked flawless on iOS. After reading your solution I removed all comments, but it still didn't work. However, I then added SEMICOLON's at each and every line ending and now it works. Once again: Thanks! :D


latest game update broke this mod

since version 2024.4.2 the camera has been frozen in the #1 view, and switches are unresponsive to mouse clicks.

fixed


add OnTargetStateChangeListener

if you want to do something after different target turned down, it will be usefull.

I implemented listeners for target. im closing this. check 1.0.3. @zhangruize


Display the proper amount of bytes we will be pruning from the repo.

Closes #846
Closes #847

FYI: you don't need to add the PR number to the comment, the PR will automatically be closed.

Codecov Report

Merging #847 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master     #847   +/-   ##
=======================================
  Coverage   53.22%   53.22%
=======================================
  Files          91       91
  Lines        7500     7500
=======================================
  Hits         3992     3992
  Misses       2912     2912
  Partials      596      596

Impacted Files
Coverage Δ

src/cmds/restic/cmd_prune.go
67.8% <100%> (ø)
:white_check_mark:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7066cc1...1273c6f. Read the comment docs.


Fix demo run, add gradle.properties

Fix demo project's run. Right from README.md:
Run jvm desktop sample:
./scripts/runGradle run

another jvm desktop samples:
./scripts/runGradle run1 run2 run3 run4 runWindowApi runVsync runLayout

Run wasm sample:
./scripts/runGradle runMppJs

Run native macos sample:
./scripts/runGradle runMppMacos

add more memory to gradle.properties

Thanks!
By the way, we should probably move to the androidx-only configuration in the future, because it simpler to use in compare to a composite project + bash scripts

Thanks!
By the way, we should probably move to the androidx-only configuration in the future, because it simpler to use in compare to a composite project + bash scripts

Yeah, I think so


Added No Game No Life emoji

that's nice , is this for hacktoberfest ? should i wait until 1 oct ?

Yes please
On Sun, 29 Sep, 2019, 11:31 PM Iqbal syamil ayasy<EMAIL_ADDRESS>wrote:

that's nice , is this for hacktoberfest ? should i wait until 1 oct ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/2pai-dev/animoji/pull/5?email_source=notifications&email_token=AGMTPWRH7VGHINBVUKEMQEDQMDUQDA5CNFSM4I3TJM32YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7323RY#issuecomment-536325575,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGMTPWTI4MFJIB3NRTPIY7TQMDUQDANCNFSM4I3TJM3Q
.


Use CSS Pre-Processor

Before attempting to do #10 or any more feature that include styling (e.g. add mobile view), there should be some kind of CSS pre-processor.
Advantage:

Reduce code duplication by using function and mixins.
Easily use BEM (Block Element Modifier) convention for class naming.

Disadvantage:

Add more complexity to setup and deploy.
The styling files would still have many code duplication if not used correctly.

I suggest between two options: SASS or LESS.
Both pre-processor have identical syntax but I tend to use SASS (with SCSS syntax) because I have more experience integrate and setup it with React.
Expected development time : 1-2 days.

I'll start working on this once @cl8n agree with this direction.

I've only used LESS but don't mind trying SASS. I went with plain CSS on purpose to keep things simple but somewhere in the create-react-app stack there is CSS processing done anyway...
also not sure how well BEM fits this project, but I won't stop u if u want to try that
i think in general the frameworks used in this project so far are not strong preferences of mine, just whatever allowed me to prototype quickly. same goes for the styles themselves, which had small inspiration from 2000s forums but are otherwise just whatever i came up with on the spot

Got it!
For now just convert the current css into sass.

ended up doing this, and I rewrote a lot of the styles to make use of sass. there are a few odd styles left that have unexpected dependency or !important etc, those can be cleaned up whenever they're visited again


Odd default data/types for many events

Mostly the strings. It's sometimes due to plug not having a consistent API, but sometimes just generally odd as well.

If a user doesn't have a slug, it's set to the empty string
If a media item doesn't exist (eg. an ADVANCE event when the only DJ stops playing), it defaults to a media item with a cid of "" (and some other "zero", but non-null data)

Ideally, an ADVANCE event when no new song starts playing, would just get null (intentional absence) for the next play, because there is no next play. Then developers can simply use !== null to check if the media item exists. Even nicer, you'll just get null from getCurrentMedia, too, so you don't have to do media.cid !== '' afterward.
If a user doesn't have a slug, the property should either be null or just not exist at all. Preferably the latter, because copying properties onto eg. a database model will override empty properties, too. This also goes for USER_UPDATEs especially, because usually only one property of the five possible updates is set.
So the main proposal here is to let plugged report absence, when data is absent. Preferably by returning null as a default for most things (except where defaults really make sense) and sometimes by omitting properties entirely.
(Also, semi-related, return Date instances where appropriate, instead of strings with preformatted dates)
Changes to these would be breaking, so it might be something for a 3.0 release :smile:

since plug was shut down a few days ago this won't happen anymore.


Vision transforms improvements

Extend Zoom transform with zoom out capabilities with min_zoom
RandomRsizedCrop needs at least order==1 to run after PointScaler
Resize.final_size is not used, so I removed it for code clarity

Why not, though I'm very doubtful min_zoom < 1 could be useful (you only add padding). Thanks!

Thanks! Could be useful to train conv kernels to operate on lower scales


Use non-Alpine images for Vizier components?

Describe the Task
We're checking on the feasibility of switching to non-Alpine images for Vizier components: nats, curl, and maybe(?) etcd.
Background on why we want this: we're planning to package Pixie Vizier for installation onto customer K8s clusters. As part of this, we have to get sign-off from our open source compliance and legal team. All of the core Vizier components are based on Debian and are approved. But the third-party images based on Alpine are blockers (specifically, nats and curl).
There's an official NATS image of nats:2.8.4-scratch that may be a drop-in replacement, and I imagine there's a Debian-based curl alternative for the initContainers.
I'm not sure if the etcd image is based on Alpine – it's not a present concern since we deploy Vizier without etcd for the metadata store.
Subtasks

[ ] Switch gcr.io/pixie-oss/pixie-prod/vizier-deps/nats:multiarch-2.8.4-alpine3.15 to nats:2.8.4-scratch.
[ ] Replace gcr.io/pixie-oss/pixie-dev-public/curl:multiarch-7.87.0 with a Debian, Ubuntu or scratch alternative.

Hey @oppegard ! Just wanted to check in and see if https://github.com/chainguard-images/images/tree/main/images/curl for curl works as a good alternative for you all.

Thank you for looking into this @aimichelle. I've run our compliance scanner against cgr.dev/chainguard/curl, and am asking the internal Open Source team to review the results. 🤞
I'm surprised there isn't a minimal and maintained curl image out there based on debian, ubuntu, or rhel. But I haven't found anything promising yet. If there isn't anything, perhaps there's a curl alternative we can find to serve the same purpose in the initContainers.

Hi folks, just checking if it's feasible to replace gcr.io/pixie-oss/pixie-dev-public/curl:multiarch-7.87.0 with the image at https://hub.docker.com/r/bitnami/bitnami-shell? If so, then I think this issue could be resolved.


How can I get a resolved json?

I want to validate and get a json with all $ref resolved, how can I get this?

I want to do something similar: part of my openapi is sprend around multiple files, and the are referenced in the main file with "$ref", which is fine.
But I'd like to generate a single file representing that openapi, aka by replacing the content of each "$ref" by the content of the referenced file.
Is that possible with that lib? I couldn't find a way to make it happen.

It isnt possible with this validator afaics.  You may be able to achieve that by creating a script that uses https://github.com/p1c2u/openapi-core .  If you want a CLI tool, I found https://github.com/kstasik/schema-tools (Rust) which can do this and quite a few other neat tricks - you will need my draft PR https://github.com/kstasik/schema-tools/pull/61 to get it building.


Get VerneMQ running on Erlang/OTP 20

To make VerneMQ run on Erlang/OTP 20 a few things need to be done.

[x] cuttlefish (export_all warning): https://github.com/tsloughter/cuttlefish/pull/1
[x] plumtree (gen_fsm deprecated warning) (gen_statem is not available in OTP 18)
[x] vmq_plugin (smerl.erl filename:find_src/2: deprecated; use filelib:find_source/3 instead) (filelib:find_source/3 is not in OTP 18)
[x] vmq_server (vmq_queue.erl, gen_fsm deprecated warning)
[x] vmq_diversity (export_all warning in vmq_diversity_utils)
[x] cuttlefish -smp disable not supported on OTP 20
[x] crypto:rand_bytes does not exist on OTP 20 (used in bcrypt)

I've assigned this to 1.3.0 - I believe we'll have to drop Erlang 18 support unless we want to run through a bunch of hoops to keep it compatible and my thinking was we'd announce in 1.2.0 that it will be the last version with erlang 18 support.

Whooha, i promise i'll work on this one :P, i think we can keep it compatible if needed, will be a bit more of work but not a big deal

@codeadict I'm not sure it's worth the trouble to keep VerneMQ erlang 18 compatible - by far the most users won't know anything changed and those who do have only to change trivial things in their plugins. Do you have a feeling people would get very upset if we drop erlang 18?

I don't think is a big deal at all, keeping the compatibility is doable, perhaps in a previous release make an announcement about 18 deprecation

Hey,
Have closely related issue, I'm unable to compile VerneMQ on Linux (Ubuntu 14.04LTS)
getting this error:
===> Compiling _build/default/lib/cuttlefish/src/cuttlefish_unit.erl failed
_build/default/lib/cuttlefish/src/cuttlefish_unit.erl:4: export_all flag enabled - all functions will be exported
I have Erlang 20 with OTP, already running on the server.

@syberkitten
There's ongoing work in PR here for OTP 20 support: https://github.com/erlio/vernemq/pull/431
Although it probably does not the cuttlefish issue specifically.
Thanks for your report!

correct, there is a PR to cuttlefish also but downgrading erlang to 19 will make it work

If Erlang 20 still does not work properly with VMQ
and Erlang 18 will be deprecated, it leaves only one slice
of good version, that is 19.
Does it not leave the project in a very fragile state?
It's like saying i need Python 2.74, but python 2.75 will not work.
Well, in Python we have VirtualEnv which allows us to have
multiple versions of Python installed with their own dependencies.
Is there something similar for Erlang?
I cannot be installing and un-installing versions of Erlang all
the time, just to check different versions....
Please advise. :)

https://github.com/kerl/kerl is the usual way to test/use different OTP versions.
Nothing is decided here. If we support 19 and 20, then we think we can move forward and drop 18... :)

Changes for this is in the #450 PR.

Putting the patched version of plumtree into checkouts #450 can now compile and build a release, but starting the release fails:
$ _build/default/rel/vernemq/bin/vernemq console
Argument '-smp disable' not supported. Use "+S 1" instead.
Usage: erl [-version] [-sname NAME | -name NAME] [-noshell] [-noinput] [-env VAR VALUE] [-compile file ...] [-smp [auto|enable]] [-make] [-man [manopts] MANPAGE] [-x] [-emu_args] [-start_epmd BOOLEAN] [-args_file FILENAME] [+A THREADS] [+a SIZE] [+B[c|d|i]] [+c [BOOLEAN]] [+C MODE] [+h HEAP_SIZE_OPTION] [+K BOOLEAN] [+l] [+M<SUBSWITCH> <ARGUMENT>] [+P MAX_PROCS] [+Q MAX_PORTS] [+R COMPAT_REL] [+r] [+rg READER_GROUPS_LIMIT] [+s SCHEDULER_OPTION] [+S NO_SCHEDULERS:NO_SCHEDULERS_ONLINE] [+SP PERCENTAGE_SCHEDULERS:PERCENTAGE_SCHEDULERS_ONLINE] [+T LEVEL] [+V] [+v] [+W<i|w|e>] [+z MISC_OPTION] [args ...]
Error generating config with cuttlefish
  run `vernemq config generate -l debug` for more information.

I could hack cuttlefish to work around that (but have to make sure this works on PRE20 version as well) and got the release up and running. Further I found we use crypto:rand_bytes/1 in various places which does not exist on OTP 20 as well...

Switch the bcrypt dependency for a newer version which is OTP20 compatible

I've opened a PR for the -smp disable issue Kyorai/cuttlefish#2

I moved this to the 1.1.1 milestone as it looks like we can keep erlang 18 compatibility after all.

Plumtree changes were merged to master and we're using a custom patched version of cuttlefish which contains Kyorai/cuttlefish#2 until that fix has been merged.

It now compiles and the release seems to be working :)  /cc @codeadict

We merged #450 yesterday

Niice

Thanks for the help @codeadict  :)


Control covers?

Hi @fredck ,
I have four smart blinds in my basement that all share a top position, but one of the four is shorter than the other three. (It sounds a little odd but actually makes sense in the room.)
I found myself wishing I could control them more naturally as a group. I currently have them grouped in HA, but when setting them to a position other than fully open or fully closed, it's a little unfortunate that they don't visually line up. As a current lightener user, I immediately wondered whether lightener could create a cover group that allowed me more sophisticated control of the cover ranges (so that the oddball shade would line up with the others at arbitrary percentages).
I see that you mentioned this in #26 as originally part of the scope of the integration but eventually set aside. I was curious if your thinking remains the same there? Assuming you want lightener to remain light-specific, do you have any thoughts on what would be involved in hacking up a version that works for shades, before I dive into it?
Thanks!

Yes, my comment in https://github.com/fredck/lightener/issues/26#issuecomment-1583603259 is conclusive. There is no plans for that as it involves a lot of tests, issues and new use cases (e.g. covers don't have on/off state) that are out of the scope of the integration.
But you can hack it in a different way, by using ha features.
You could create helper "lights" using the the Template Light integration that would sync the lights brightness to the curtains position.
Then, the other way around. You create a helper "cover" using the Template Cover integration that syncs with the Lightener entry that controls the above "lights".
All this would work behind the scenes, transparent to the end user.
So well... you wanted to hack this up... here you have it :)


chore: update pygitguardian dependency

This is required to properly release a new version of ggshield.

Codecov Report

Merging #419 (0f21f05) into main (c13552e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #419   +/-   ##
=======================================
  Coverage   94.27%   94.27%
=======================================
  Files          80       80
  Lines        3425     3425
=======================================
  Hits         3229     3229
  Misses        196      196

Flag
Coverage Δ

unittests
94.27% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


[rancher/charts] Add support for repositories without trailing slashes in dependencies

What kind of request is this (question/bug/enhancement/feature request):
bug
Steps to reproduce (least amount of steps as possible):
Pull in rancher/charts branch dev-v2.5 at current head (noticed in commit 2bb14d65395cba9f0d93ec48b53adae74b2e7229)
Run make CHART=rancher-monitoring prepare; make CHART=rancher-monitoring patch
Result:
Patch has a diff, specifically in dependencies; this is not expected
Other details that may be helpful:
https://github.com/rancher/charts/blob/2bb14d65395cba9f0d93ec48b53adae74b2e7229/packages/rancher-monitoring/rancher-monitoring.patch#L411-L427
Happens because the scripts expect repository to end with a trailing slash here but it does not.
The behavior of the scripts also needs to be modified to notify us if a download_url cannot be found since this was currently silently succeeding.

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by performing another checkout.

If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -b with the checkout command again. Example:

  git checkout -b <new-branch-name>

HEAD is now at 2bb14d6 Generated changes
arvindiyengar: ~/Rancher/charts/src/github.com/rancher/charts
$ make CHART=rancher-monitoring prepare > /dev/null; (make CHART=rancher-monitoring patch > /dev/null); make CHART=rancher-monitoring clean > /dev/null
{note: omitted tons of logs printed by make patch}
arvindiyengar: ~/Rancher/charts/src/github.com/rancher/charts
$ git status
HEAD detached at 2bb14d6
Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git checkout -- <file>..." to discard changes in working directory)

	modified:   packages/rancher-monitoring/rancher-monitoring.patch

no changes added to commit (use "git add" and/or "git commit -a")

On dev-v2.5 head:
arvindiyengar: ~/Rancher/charts/src/github.com/rancher/charts
$ make CHART=rancher-monitoring prepare > /dev/null; (make CHART=rancher-monitoring patch > /dev/null); make CHART=rancher-monitoring clean > /dev/null
{note: omitted tons of logs printed by make patch}
arvindiyengar: ~/Rancher/charts/src/github.com/rancher/charts
$ git status
On branch dev-v2.5
Your branch is up to date with 'upstream/dev-v2.5'.

nothing to commit, working tree clean


Fix for log/inverse/sqrt color scale in map plot

Fixed broken interaction with color axis limits
Added warnings for user selecting properties that have some values <= 0
Ensured that the user cannot select a property with only values <= 0 with the log/sqrt scale.

OK, I'm testing this. Two first observations:

it looks like proper logarithmic colorscale labels can be generated see
https://plotly.com/python/colorscales/#customizing-tick-text-on-logarithmic-color-bars
the error message says that negative values are mapped to gray, but actually they are shown with the color corresponding to the minimum of the scale

it looks like proper logarithmic colorscale labels can be generated see
https://plotly.com/python/colorscales/#customizing-tick-text-on-logarithmic-color-bars

This is possible, but mainly works if the data spans multiple orders of magnitude. If all values are between 1 and 3, creating log-spaced ticks is a bit harder.

it looks like proper logarithmic colorscale labels can be generated see
https://plotly.com/python/colorscales/#customizing-tick-text-on-logarithmic-color-bars

I did have a look at this. While it is possible, the ticktest argument requires that the tickmode is set to array, such that the tickvals are set in the layout.  Also, the alternative implementation with the labelalias argument allows for any tickmode, but the tickvals are still required to set the aliases. Since we set the min and max for the colorbar, Plotly automatically sets the tickvalues and extracting them is non-trivial.


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_1e3o4ks2g4/clone/recipe/meta.yaml:30: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_1e3o4ks2g4/clone/recipe/meta.yaml:30: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_1e3o4ks2g4/clone/recipe/meta.yaml:38: missing_pip_check: For pypi packages, pip check should be present in the test commands
WARNING: /tmp/abs_1e3o4ks2g4/clone/recipe/meta.yaml:47: missing_description: The recipe is missing a description
Errors were found


Fixed issues in Graph.validateSeries

Was debugging with Chester and we found a few issues with validation of series data.  Incorrect operator precedence as well as a lack of checks on the data per series.  This fixes those issues.

Thanks!


Dropdown bug

Tested versions
Reproductible in Godot v4.3 and later.
System information
Godot v4.4.dev4.mono - Windows 10.0.19045 - Multi-window, 2 monitors - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 3060 (NVIDIA; <IP_ADDRESS>14) - AMD Ryzen 7 5800X 8-Core Processor (16 threads)
Issue description
When I click on a dropdown arrow in the inspector to open it, and click again on the arrow to close it, sometimes the dropdown box stays, but all informations inside disappear. The dropdown box will close if I click a third time on the arrow.
Here is a video that showcase that issue:
https://github.com/user-attachments/assets/0fde6689-e48d-4e28-8276-7b453c6dbc20
Steps to reproduce
You can reproduce what is shown in the video.
Minimal reproduction project (MRP)
No project needed.

So on windows 10 dev5 im getting a similar Issue but not the same. When i quickly click the dropdown button again, the Content of the drop down menu, does not get removed, but it actually stays arround as a ghost. you can click through the Menu Items onto the buttons below, and can not access or highlight the menu Items. seems to be within an about 500 ms range

Radio Menus seem not to be affected

On Godot 4.4 dev5, I get the same issue.
https://github.com/user-attachments/assets/4c56ae4b-1365-4c19-917d-3b4d70af8212

May be related to https://github.com/godotengine/godot/issues/99203

May be related to Context menu empty or corrupted when closing with RMB #99203

You're right. I've managed to reproduce the bug showcased in #99203.


update blocklist and config loading

untested paths and changes
further work pending on config and abstractions

[ ] places/blocklist.txt.local is not supposed to be committed
[ ] have the blocklists based on regex / preferrably a bloom filter (similar to what chrome uses for spam/malacious links detection
[ ] include malacious links detection
[ ] throttling functionalities and integration with hashing (prefer diskcache ingestion / plugin integration, as opposed to using raw lists/sets/native data structures)
[ ] ...

Looks promising! for the blocklist, I wonder if we can also add regexps.
Thinking about catching all pages that have a known authentication pattern, like oauth2 dances etc,

Typical pattern for oauth:
https://<server>auth?response_type=<type>&client_id=<id>&redirect_uri=<url>

so maybe:
^(?=.*\bresponse_type\b)(?=.*\bclient_id\b).*$

what about ad servers? maybe we could use https://github.com/anudeepND/blacklist/blob/master/adservers.txt

also: URL_SKIPLIST should be added into the /index web API so we drop URLs there and that will include calls from the extension, not just the places db loader

Right yeah I feel like we should discuss this. Seems like it needs a lot more thought into it as there are so many patterns and URLs. That regex will be unmanageable after a while (let alone searchable).
I'm thinking something that works browser's cache/cookies/url parsers or maybe pattern lists from adblockers.

Right yeah I feel like we should discuss this. Seems like it needs a lot more thought into it as there are so many patterns and URLs. That regex will be unmanageable after a while (let alone searchable).
I'm thinking something that works browser's cache/cookies/url parsers or maybe pattern lists from adblockers.

sounds good let's do this as a follow up

I would drop 3.8 and 3.9 support unless you think there's a specific use case to run against those python versions

will cover the rest here https://github.com/myinternets/places/issues/27


Add Regression Tests for Mold Breaker

Probably doesn't have complete coverage quite yet... tagging @Marty-D for advice on what else I should check
In preparation for when I refactor Mold Breaker away from the ModifyPokemon event.

IMO Mold Breaker tests should be added to each ability, as it's currently the case with Magic Bounce, as the coverage is highly dependant on implementation details.

Noted. I'll wait for other people to make comments on this request, and if enough people want Mold Breaker to be in the other ability checks, I'll close this and work on something else that uses ModifyPokemon first.

In other news, I still have no idea how to get Mocha to catch event handlers like onHit. Does anyone have ideas on how it'd work?

Why do you need to catch event handlers like onHit? What does that even mean?
It's a test. Do things, make sure the result is the one you wanted. Event handlers have nothing to do with it. o_O

Bleh, you're right. I was just thinking it'd be easy for the weight tests with Grass Knot, snagging the Base Power straight off the event. But damage calcs like what I'm doing here are probably fine.
Either way, put the things Mold Breaker is involved in within other tests. Yes or no?

I'm just going to take that as a "yes" and put it in other regression tests instead.

Sorry for being late on this, but yeah I agree with Slayer95. A test that might be worth having in Mold Breaker itself would be Roar/Dragon Tail or similar into Levitate (which is ignored) or Magic Guard (which is not).


index villager house -1

By mistake i placed 11 villager houses and now house 1's index is -1, the lot is empty as if i was villager hunting but the villager is still in my island(??. Ive tried to move the house somewhere else but it doesn't let me. Plus no one can visit my island

I fixed!! I changed the villager and now he has a house :)


(docs) displaying-data: use npm, not cdn (oops) and upgraded to alpha46

Also no line numbers for single-line examples.

done

@wardbell any reason this is closed?

Learning the process. I just said "done". Should say which commit ... b82e229


[SUPPORT] Low performance with upserts on S3 storage

Problem
I'm testing the ability of Apache Hudi to make upserts faster than the current functions on Spark.
Each record contains 40 fields. The partitioning key is country_iso (a string field). There are 200 different values for this field. The partitions are quite unbalanced (US, China have much records).
The problem is that I'm getting very slow performance with small datasets (~1Gb)
I'm updating a string field which is not the partitioning key and the record key.
The ratio of updates in my upsert dataset : 100%.
This could come from the way of partitioning my Parquet file, the unbalanced partitioning, choose another partitioning key ...
Environment Description

Hudi version : 0.11.1

Spark version : 3.1.2-amzn-1

Hive version :

Hadoop version : 3.2.1 (Amazon)

Storage (HDFS/S3/GCS..) : S3

Running on Docker? (yes/no) : no

AWS EMR : emr-6.5.0, 1 master (r5.xlarge), 2 cores (r5d.2xlarge)

Additional context
Add any other context about the problem here.
Hudi Config
hoodie.index.type = BLOOM/SIMPLE
hoodie.bloom.index.prune.by.ranges = false
hoodie.metadata.enable = true
hoodie.enable.data.skipping = true
hoodie.metadata.index.column.stats.enable = true
hoodie.bloom.index.use.metadata = true

sample
src parquet   (nb in millions)  (size in Gb)
Updates   (nb in millions)   (size in Gb)
Upsert S3 - Simple index  (time in mins)
Upsert S3 - Bloom index  (time in mins)

1
8.7 M records  (0.9 Gb)
0.35 M records  (0.05 Gb)
1.80
1.88

10
87 M records  (7.9 Gb)
3.5 M records  (0.55 Gb)
10.5
21.5

25
217M records  (18.7 Gb)
8.7 M records  (1.1 Gb)
27.05
110.5

For example, the sample_10, I've got the following results :

index_type
2 most costly tasks

SIMPLE
Building workload profile: SIMPLE_hudi_sample_10 (countByKey at HoodieJavaPairRDD.java:104) -- 1.5 minDoing partition and writing data: SIMPLE_hudi_sample_10 (count at HoodieSparkSqlWriter.scala:643) -- 8.1 min

BLOOM
Building workload profile: BLOOM_hudi_sample_10 (countByKey at HoodieJavaPairRDD.java:104) -- 13 min -- IMAGE 1Doing partition and writing data: BLOOM_hudi_sample_10 (count at HoodieSparkSqlWriter.scala:643) -- 8.0 min -- IMAGE 2

The image below show the partition /BN, with very small parquet files.

Here is the Spark trace of an upsert with Bloom index (sample_10)

IMAGE 1. Building workload profile: BLOOM_hudi_sample_10 (duration : 13 min),

IMAGE 2. Doing partition and writing data: BLOOM_hudi_sample_10, (duration : ~8mins) :

@alexeykudinkin : can you take a look at this.

@ns

Hey, @floriandaniel! Thanks for taking the time to file very detailed description.
First of all i believe the crux of the problem is likely lying in the realms of using Bloom Index of the Metadata table: we've recently identified a performance gap in there and @yihua is currently working on addressing that (there's already a PR in progress).
Second, i'd recommend you to do following in your evaluation:

Try Hudi 0.12 that has been recently released (we've done a lot of performance benchmarking/optimizations during last release cycle specifically to make sure Hudi's performance is top of the line)
Disable hoodie.bloom.index.use.metadata for now (until above fix lands)
Any particular reason you switching off hoodie.bloom.index.prune.by.ranges? It's very crucial aspect of using the Bloom Index that allows to prune the search space considerably for update-heavy workloads only checking the files that could contain the target records (and eliminating ones that couldn't)


Incomplete Text Rendering Across Vector Tiles in OpenLayers 6.9

When rendering point data using vector tiles in OpenLayers 6.9, there is an observed problem where text labels, extending beyond the span of individual vector tiles, get truncated across tile boundaries.

This will be fixed with #15466.

这将通过 #15466 修复。

Does this mean this issue will be fixed in 9.0?

yes


bug: KubeSteller setup scripts do not wait for KubeFlex control plane sync

Describe the bug
A KubeFlex ControlPlane is like a Kubernetes Pod or CustomResourceDefinition --- merely creating the API object is not enough, you generally have to wait for the asynchronous processing to finish. For example, kflex create ... -p SomePCH will complete successfully before any attempt is even made to find or apply the named PCH. See https://github.com/kubestellar/kubeflex/issues/235 .
Steps To Reproduce
This is from code examination.
Expected Behavior
Waiting where completion of asynchronous processing is needed.
Additional Context
No response

/cc @pdettori
/cc @francostellari
/cc @dumb0002
/cc @effi-ofer

That's necessary but not sufficient. "SYNCED" will only tell if the templates have been applied without errors, but not if the objects encoded in those templates are up and running. While kflex can tell if a control plane is up checking the status of the API server, it cannot tell if  arbitrary objects part of a PCH are up and running. Consider the case of the OCM PCH for example, there is no general way to know that OCM is up and running unless we write use-case specific logic for it.

Note that even helm (as an example) will not tell you if all objects in the release are in a READY state. It will only tell if all objects have been applied without errors.

@effi-ofer Any update on this ?  (CC: @pdettori )


feature(secretstores): add context to api and tencent/ssm

https://github.com/dapr/components-contrib/pull/1973
/cc @berndverst @ItalyPaleAle

This PR needs to wait for another MR  to be merged,  before it can be changed back in the go.mod file, a temporarily used fork repo github.com/1046102779/components-contrib

@1046102779 we also need a docs PR for this component.

@1046102779 we also need a docs PR for this component.

docs PR: https://github.com/dapr/docs/pull/2732

/cc @daixiang0 @yaron2 @artursouza

This PR needs to wait for another PR to be merged, before it can be changed back in the go.mod file, a temporarily used fork repo github.com/1046102779/components-contrib

Done. /cc @yaron2 @berndverst @daixiang0 @ItalyPaleAle @artursouza

fixed:

make lint

pkg/components/state/pluggable_test.go:617:5  typecheck  unknown field `Filters` in struct literal

/cc @daixiang0 @artursouza @yaron2

@halspang - the pluggable components change is due to https://github.com/dapr/components-contrib/pull/2038


Update Cake.Graph version

Cake.Graph 0.7.1 is targeted netstandard2.0, which allow run Cake.Recipe with Cake.CoreCLR

@vhatsura If you delete the whole tools folder you made sure that you used the correct version of Cake.Recipe including these changes? I've deleted all addins and tools in tools folder and there were the issues mentioned above.

Hmmm. It strange for me. First of all I tried to use Cake.Graph without any other dependencies and all works without errors.
test.zip
In addition I tried to remove separate reference to RazorLight. Don't ask me about specifying version and prerelease parameters in package=Cake.Graph&version=0.7.1&prerelease&loaddependencies=true.
It seems that Cake cannot find RazorLight without specifying prerelease flag and throw the following exception:
Error: System.AggregateException: One or more errors occurred. ---> NuGet.Resolver.NuGetResolverConstraintException:
Unable to find a version of 'RazorLight' that is compatible with 'Cake.Graph 0.7.1 constraint: RazorLight (>= 2.0.0-beta1)'.
   at NuGet.Resolver.PackageResolver.Resolve(PackageResolverContext context, CancellationToken token)
   at Cake.NuGet.Install.Extensions.NuGetPackageManagerExtensions.GetPackagesToInstall(PackageIdentity packageIdentity, ResolutionContext resolutionContext, INuGetProjectContext nuGetProjectContext, IReadOnlyCollection`1 availablePackages, ISet`1 sources, CancellationToken token)
   at Cake.NuGet.Install.Extensions.NuGetPackageManagerExtensions.<GetPackageAndDependenciesToInstall>d__3.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Cake.NuGet.Install.Extensions.NuGetPackageManagerExtensions.<GetInstallProjectActionsAsync>d__0.MoveNext()
   --- End of inner exception stack trace ---
   at System.Threading.Tasks.Task`1.GetResultCore(Boolean waitCompletionNotification)
   at Cake.NuGet.Install.NuGetPackageInstaller.Install(PackageReference package, PackageType type, DirectoryPath path)
   at Cake.Core.Scripting.ScriptProcessor.InstallAddins(IReadOnlyCollection`1 addins, DirectoryPath installPath)
   at Cake.Core.Scripting.ScriptRunner.Run(IScriptHost host, FilePath scriptPath, IDictionary`2 arguments)
   at Cake.Commands.BuildCommand.Execute(CakeOptions options)
   at Cake.CakeApplication.Run(CakeOptions options)
   at Cake.Program.Main()
---> (Inner Exception #0) NuGet.Resolver.NuGetResolverConstraintException: Unable to find a version of 'RazorLight' that is compatible with 'Cake.Graph 0.7.1 constraint: RazorLight (>= 2.0.0-beta1)'.
   at NuGet.Resolver.PackageResolver.Resolve(PackageResolverContext context, CancellationToken token)
   at Cake.NuGet.Install.Extensions.NuGetPackageManagerExtensions.GetPackagesToInstall(PackageIdentity packageIdentity, ResolutionContext resolutionContext, INuGetProjectContext nuGetProjectContext, IReadOnlyCollection`1 availablePackages, ISet`1 sources, CancellationToken token)
   at Cake.NuGet.Install.Extensions.NuGetPackageManagerExtensions.<GetPackageAndDependenciesToInstall>d__3.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Cake.NuGet.Install.Extensions.NuGetPackageManagerExtensions.<GetInstallProjectActionsAsync>d__0.MoveNext()<---

Also I tried to check which exactly version will be downloaded if prerelease and version parameters are specified. When I specified #addin nuget:?package=Cake.Npm&version=0.15.0&prerelease&loaddependencies=true (the latest version in nuget.org is 0.16.0) 0.15.0 version was installed.

@pascalberger, did you have a chance to look into the last comment?


Update Travis test matrix

Travis test matrix needs to be updated to properly validate against proper webpack versions and supported node versions ( 4.7 || > 6.x && !5.x).

Resolved with 91b8cb82f3c48af2caa76d61b02a07b6129fee67


Timeline UI - time entries list

💻 Environment
Platform: macOS
📒 Description
Vertical list that displays time entries in a timeline

Every full hour is shown as a label (8:00 AM etc)
Time entries are the color of their project
Time entry item height is based on it's duration.
If time entries overlap they are horizontally positioned side by side
Empty slots or gaps in the day area marked with an element with a dotted line border.
Hovering items will show the info about the item (https://zpl.io/VQv3jz4)
Clicking on items will show edit popup for the item

Close by https://github.com/toggl/toggldesktop/pull/3050


Revert "sending the appropriate query string parameters in deprovision"

Reverts kubernetes-incubator/service-catalog#885
I LGTM3'ed 885 and merged it and realized afterwards that there is actually a problem with it. I'm reverting 885 and then we can discuss further.

Rather than revert the previous one, any reason you don't just use this PR to remove the hard-coded query string params and use @arschles's code instead?

@duglin because this PR is already merged. His branch could be amended and a new PR could be opened. If that gets done before revert passes CI, that's cool and we can move in that direction instead, but reverting to a previously known state seemed like a safe action I didn't mind taking unilaterally after performing a merge that I wish I hadn't.

@duglin this just passed CI as I finished typing the previous comment. I'm moving forward with this revert so that we're at least back to a known good state.

fyi to all... above merge was done in good faith to revert a merge that shouldn't have happened.


python311Packages.alexapy: 1.27.6 -> 1.27.7

Diff: https://gitlab.com/keatontaylor/alexapy/-/compare/refs/tags/v1.27.6...v1.27.7
Changelog: https://gitlab.com/keatontaylor/alexapy/-/blob/v1.27.7/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 265624 run on x86_64-linux 1

9 packages built:

home-assistant-component-tests.roborock
python310Packages.alexapy
python310Packages.alexapy.dist
python310Packages.python-roborock
python310Packages.python-roborock.dist
python311Packages.alexapy
python311Packages.alexapy.dist
python311Packages.python-roborock
python311Packages.python-roborock.dist


Do we really need this dependency when it's in pixl core? Let's find

out.

Interesting, we do. That surprises me but 🤷


The problem of the first expansion of the expression panel

When the expression panel is first expanded, there will be a Caton animation from left to right. I don't know if this is a problem, but it's really uncomfortable
https://user-images.githubusercontent.com/103582888/163195817-92e3fad1-8206-4808-8940-b7e5e4ff2ce8.mp4

Yes I noticed that also, it's not intended, I will try to fix it soon. Don't hesitate to send a PR if you can fix the issue.

Fixed in https://github.com/antoine92190/vue-advanced-chat/commit/731839c94600ac5761520a98a656f35703a3e300


Update Sidebar's Form import path for compatibility with Filament 3

Extended parameter and return types for Compatibility with Filament 3 which now uses \Filament\Forms\Form instead of \Filament\Resources\Form

Closes #19

Thanks Sam! Released in 1.3.1.


fixes wrong code after merge

@shiffman This code parts are messed up by the merge. This should work correct now.

Thanks!!


Automate Project Account (Unity-Sips-Test) SPS S3 Bucket Deployment

The SPS S3 bucket deployment/creation needs to be automated (if not done already in terraform).  The SPS team will manage the creation of this, as part of their terraform code in the SPS github repo(s).  The U-CS deployment infrastructure will run the SPS terraform, to create the bucket at project deployment time.
Acceptance Criteria:

[ ] Inspection into both the Github Action logs, and the AWS console prove that the project deployment led to the automatic creation of the SPS S3 bucket.

@LucaCinquini this may already be implemented, but if not, I'll leave it up to you, to assign this to a member of the SPS team.


Fix method name not start with 'get' following kubeclient change

and bump kubeclient version

@Fryguy I tried to look at the travis failure message, if I get it right it fails with "missing database.yml" which sounds irrelevant to this patch? what am I missing?

Something is up with that statement...not sure what...I think it's related to #1661

I resubmitted again and get the same errors.
This does not yet have the changes from 1661:
While travis.yml is missing the cp statements,
preinitializer.rb still has "copy if no database.yml exists.
Maybe rebase master and re-push?

I tried again and it didn't go through, but others seem to be compiling fine. could you rebase?

@Fryguy @kbrock  I rebased and pushed again though I still don't understand why are those issues related.

@Fryguy @kbrock I guess that can be safely merged now?


After merging #216, neil add produces super long lines for certain aliases

Problem
After merging #216 (which I wrote), certain aliases produce some super long lines, for example neil add build.
$ # per latest stable
$ neil add build
$ cat deps.edn
{:deps {}
 :aliases
 {:build ;; added by neil
  {:deps {io.github.clojure/tools.build {:git/tag "v0.10.3" :git/sha "15ead66"}
          slipset/deps-deploy {:mvn/version "0.2.2"}}
   :ns-default build}}}
$ # on master, `:deps` keys for aliases are on the same line.
$ rm deps.edn
$ rm build.clj
$ neil-dev add build
$ cat deps.edn
{:deps {}
 :aliases {:build {:deps {io.github.clojure/tools.build {:git/tag "v0.10.3", :git/sha "15ead66"}, slipset/deps-deploy {:mvn/version "0.2.2"}}
                   :ns-default build}}}

Proposed solution:
An alias is a map. If the alias map values themselves are maps, print them on separate lines.
Making this change, should produce the following deps.edn:
{:deps {}
 :aliases {:build {:deps {io.github.clojure/tools.build {:git/tag "v0.10.3", :git/sha "15ead66"},
                          slipset/deps-deploy {:mvn/version "0.2.2"}}
                   :ns-default build}}}

, where io.github.clojure/tools.build and slipset/deps-deploy are on their own lines.

sounds good. maybe also add a test for it? I assumed such a test was already there

Yes, good idea.

After merging #225, this is the is the current behavior:
$ neil-dev add test && neil-dev add build && neil-dev add nrepl && neil-dev add kaocha
If you wish to create a `bin/kaocha` file, copy and run the following:

mkdir -p bin && \
echo '#!/usr/bin/env bash
clojure -M:kaocha "$@"' > bin/kaocha && \
chmod +x bin/kaocha
$ cat deps.edn
{:deps {}
 :aliases {:test {:extra-paths ["test"]
                  :extra-deps {io.github.cognitect-labs/test-runner {:git/tag "v0.5.0"
                                                                     :git/sha "b3fd0d2"}}
                  :main-opts ["-m" "cognitect.test-runner"]
                  :exec-fn cognitect.test-runner.api/test}
           :build {:deps {io.github.clojure/tools.build {:git/tag "v0.10.3"
                                                         :git/sha "15ead66"}
                          slipset/deps-deploy {:mvn/version "0.2.2"}}
                   :ns-default build}
           :nrepl {:extra-deps {nrepl/nrepl {:mvn/version "1.1.2"}
                                cider/cider-nrepl {:mvn/version "0.49.0"}
                                refactor-nrepl/refactor-nrepl {:mvn/version "3.10.0"}}
                   :main-opts ["-m" "nrepl.cmdline" "--interactive" "--color" "--middleware" "[cider.nrepl/cider-middleware,refactor-nrepl.middleware/wrap-refactor]"]}
           :kaocha {:extra-deps {lambdaisland/kaocha {:mvn/version "1.91.1392"}}
                    :main-opts ["-m" "kaocha.runner"]}}}

I think the :main-opts line is a bit long, but I don't have any ideas on how to make it nicer. Perhaps the user may just edit the formatting if they don't like it later.
Do we consider this issue closed then?


image/png: makeslice: len out of range in Decode with insane image size

What version of Go are you using (go version)?

$ go version
go version go1.17.6 darwin/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/catena/Library/Caches/go-build"
GOENV="/Users/catena/Library/Application Support/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/catena/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/catena/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_amd64"
GOVCS=""
GOVERSION="go1.17.6"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/catena/go/src/github.com/catenacyber/go/src/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch x86_64 -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/pp/dc1dtf9x2js3v0jx_m010nqr0000gn/T/go-build4237848497=/tmp/go-build -gno-record-gcc-switches -fno-common"
GOROOT/bin/go version: go version go1.17.6 darwin/amd64
GOROOT/bin/go tool compile -V: compile version go1.17.6
uname -v: Darwin Kernel Version 21.3.0: Wed Jan  5 21:37:58 PST 2022; root:xnu-8019.80.24~20/RELEASE_X86_64
ProductName:	macOS
ProductVersion:	12.2.1
BuildVersion:	21D62
lldb --version: lldb-13<IP_ADDRESS>
Apple Swift version 5.6 (swiftlang-<IP_ADDRESS>3.62 clang-13<IP_ADDRESS>)
gdb --version: GNU gdb (GDB) 9.1

What did you do?
Run https://go.dev/play/p/JNfieKSukyI?v=gotip
What did you expect to see?
The program finishing and printing Hello with an appropriate error
What did you see instead?
Hello, %!s(<nil>) image.Config{ColorModel:(*color.modelFunc)(0xc000012060), Width:1347179589, Height:623657023}
panic: runtime error: makeslice: len out of range

goroutine 1 [running]:
image.NewGray16({{0xc000074b80?, 0x48432e?}, {0x100c000115088?, 0x4d3668?}})
	/usr/local/go-faketime/src/image/image.go:1019 +0x66
image/png.(*decoder).readImagePass(0x4d36e8?, {0x7fd864c280e0, 0xc00006c050}, 0x0?, 0x0)
	/usr/local/go-faketime/src/image/png/reader.go:477 +0x612
image/png.(*decoder).decode(0xc000045400)
	/usr/local/go-faketime/src/image/png/reader.go:374 +0x1ae
image/png.(*decoder).parseIDAT(0xc000045400, 0x45478?)
	/usr/local/go-faketime/src/image/png/reader.go:859 +0x25
image/png.(*decoder).parseChunk(0xc000045400)
	/usr/local/go-faketime/src/image/png/reader.go:918 +0x129
image/png.Decode({0x4d3688?, 0xc000016210})
	/usr/local/go-faketime/src/image/png/reader.go:977 +0x11b
main.main()
	/tmp/sandbox1166748030/prog.go:15 +0x205

Program exited.

Found by https://github.com/catenacyber/ngolo-fuzzing on oss-fuzz
https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=50234
I know that the result of DecodeConfig should be checked before calling Decode, but I think that in this case, the error should be a bit nicer

@nigeltao @robpike
image/png is choking on fuzz, not sure what's the best plan here.

"Decoding some (possibly valid) PNGs can require a lot of memory" is a duplicate of #53778 (which is itself arguably a duplicate of #5050).
If we wanted image/png (in the standard library, obviously) to catch out-of-memory panics and convert them to a nicer error, then we'd probably want a number of stdlib packages to do similarly, which sounds like a broader discussion (maybe on the golang-dev mailing list??).

The problem is not a out of memory here, it is panic: runtime error: makeslice: len out of range ;-)


Translations for Spanish and Asturian

Hi,
I open this pull request with the translations for Spanish and Asturian languages. I have done them for our own LodView instance so I hope that you can find them useful too. For the Asturian language, which might be a bit more tricky to test, I have performed my tests using this language extension on Firefox (https://addons.mozilla.org/en-US/firefox/addon/asturianu-language-pack/).
Thanks for taking this contribution under consideration.
Best regards,
Herminio García

Thank you @herminiogg !
Let me know if are going to use LodView in one of your projects

@dvcama Even though it is still under testing you can find LodView used for Holocaust archival data here: https://lod.ehri-project-test.eu/
Thanks for merging the pull request and congratulations on such an amazing project!
Best,
Herminio


Record has subjects that aren't in the bib-record or Figgy, and a refresh isn't removing them

Record: https://dpul.princeton.edu/eastasian/catalog/xg94hs896

dpul_subject_mismatches.xlsx
Attached is a file of examples of this problem.  The tab "eastasian" contains 92 examples from the DPUL site https://dpul.princeton.edu/eastasian with the (incorrect) subjects
Medicine, Chinese—Early works to 1800
Medicine, Chinese—Formulae, receipts, prescriptions
The tab "ealtoc" contains 36 examples from https://dpul.princeton.edu/ealtoc with the incorrect subjects
Chinese poetry—19th century
Chinese poetry—20th century
Chinese poetry—Societies, etc—History—Sources

A staff member has reported what appears to be a related problem.  Please note that more than just subjects are involved now.
All nine Korean records still pull up the wrong information in edition, abstracts, contents, editors, etc.
https://dpul.princeton.edu/ealtoc/catalog?f[readonly_language_ssim][]=Korean&q=

This looks like the chinese record those korean records are pulling data from
https://figgy.princeton.edu/catalog/b1e79dcc-9ff5-4786-9bc5-f863291defcf

Here's a pair:
https://dpul.princeton.edu/msstreasures/catalog/zc77sr50z "the book of enoch"
has metadata from https://figgy.princeton.edu/catalog/87c05bba-8946-4773-bace-5630ef141c7f "Dettingen Te Deum"

@jrgriffiniii and I looked at some of these examples and tried to identify any common factors between "source" and "corrupted" records. We looked at last modified dates and did not find a pattern (in one case the "source" had an earlier modified date and in the other case it had a later modified date).
We note that the problem is definitely at the pomegranate layer; the corrupted records look right in figgy.
Note also that one source record will provide data to many (sometimes very many) corrupted records. It looks like the source and corrupted records are members of the same collection.
In the case of the Handel "source" record, it seems to be corrupting maybe the entire "Treasures of the Manuscripts" collection. Is it possible that the Handel record is being used to supply exhibit-level metadata? I checked its create / update dates against other members of that collection and it's not the first or last of either.

https://github.com/pulibrary/pomegranate/pull/586 will potentially fix this after it's deployed and some objects are reindexed.

This issue has been fixed, at least in the East Asian examples I initially reported.

Fixed by #586
Thanks for confirming @tventimi !


Close stream used in FileTreeView.nio

We created a stream without closing it. This came up because I was
reading the release notes for metals 0.7.2 and they mentioned fixing a
similar bug in the release notes.
In general, the nio FileTreeView should not really be in use in sbt
1.3.0. It would only be used on 32 bit platforms running sbt. Because
FileTreeView is a new api, and I rolled back the default path finder
implementations, none of the downstream consumers should be affected.

To validate this pull request against other sbt modules, please visit this link.


authorization-server 1.0RC1  客户端模式  /oauth2/token 接口返回值少字段

authorization-server 1.0RC1  客户端模式  /oauth2/token 接口返回值 无scopes字段了，请问如何配置才会有scopes字段？

@wangxianlong0408 Please comment in english as I do not understand the issue.

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.


org.dmfs.tasks.QuickAddDialogFragment org.dmfs.tasks.TaskListActivity org.dmfs.tasks.TaskListFragment may leak

I recently found that the following components leaked:
org.dmfs.tasks.QuickAddDialogFragment
org.dmfs.tasks.TaskListActivity
org.dmfs.tasks.TaskListFragment

However, I cannot easily found the causes from the GC paths. So I attach the .hropf file here for your reference.
heap_0.txt(in .rar format)
Hope it helpful!

Thanks for the report. At this time I have no clue yet, but I can see the leaked instances too. I need to look into that.


Use plain fetch to get source_types.

There's something broken with the javascript API client.
Using plain fetch for now.

I would probably leave the API clients for now until we have some stable version. I am having troubles with it myself in catalog UI. I am constantly switching between the client and direct calls.


Will this be updated to support the new pushshift api credentials?

Pushshift is back online (https://old.reddit.com/r/pushshift/comments/14ei799/pushshift_live_again_and_how_moderators_can/) and has been backfilled with content. The new update requires authentication, (https://api.pushshift.io/docs#), through cURL  headers.
One tool, https://adhesivecheese.github.io/chearch/, has already been updated with the access token functionality. Will unedit-for-reddit be updated as well?

I don't have access to the API so I can't fully test the code, though I think it should work.
Are you able to load the code from the PR by any chance to test it?

Reddit itself is down right now, but I will when I can, thanks! Also, the new authentication expires every 24 hours, so I'd be fine with sharing a temporary key with you so you can test.

Tested and is working! Confirmed with loading unpacked chrome extension on chromium 114.0.5735.133. My offer of sharing an api key for testing purposes still stands.

Thanks for sharing the API key, it was useful for testing and fixing some issues. It was working well in Chrome and the Userscript version but there were some issues with the headers not getting passed in Firefox. I was able to make a workaround for Firefox (though it is a bit hacky).
Thanks for your help getting this back to being usable. :+1:


Remove the need to set up permissions for courses you run AND ratify

Screenshots: https://trello.com/c/VFw3VKTj/2112-design-dont-ask-users-to-setup-permissions-for-an-organisation-that-ratifies-and-runs-their-own-courses

Good catch


selecting multiple keys

From @arnottcr on February 5, 2015 7:54
I find myself wanting to use the shift key to select a run of keys and the ctrl key to select additional keys. Both yours and this are valid approaches, I wonder which most users would prefer based on previous UI/UX training?
Copied from original issue: winhowes/grd.me#19

From @winhowes on February 5, 2015 8:22
For the shift key case do you mean holding down the shift key and dragging over an area of keys to select vs ctrl selecting individual keys?

From @winhowes on February 5, 2015 8:22
For the shift key case do you mean holding down the shift key and dragging over an area of keys to select vs ctrl selecting individual keys?

so, I am thinking shift click will select keys from the currently selected option to the clicked option
and ctrl clicking will add the clicked option to the subset of currently selected options
side effect: shift will need to pick an origin to start from [first click or most recent click] and clicking can deselect some entries
it might just be simpler to stick with the current implementation and see if people are used to or want shift or ctrl [or use both as the same]
walking through this it has turned into a pita to implement, so thoughts on both?

so, I am thinking shift click will select keys from the currently selected option to the clicked option
and ctrl clicking will add the clicked option to the subset of currently selected options
side effect: shift will need to pick an origin to start from [first click or most recent click] and clicking can deselect some entries
it might just be simpler to stick with the current implementation and see if people are used to or want shift or ctrl [or use both as the same]
walking through this it has turned into a pita to implement, so thoughts on both?

Yeah, I think google drive works like that; I can confirm that message
selection in thunderbird works that this:
Ctrl toggles the selection of the clicked item.
Shift selects everything from the most recently clicked item [note this
can be unselected, think select cell then ctrl+click, then shift to
another cell] to the currently clicked item
I think it could be more intuitive to some users, but I am not really
sure what "most" people consider "standard" on the internet.

From @winhowes on February 5, 2015 8:51
sorta like the implementation on google drive? Definitely could be a pain to implement, but if it's better it's worth implementing

From @winhowes on February 5, 2015 8:51
sorta like the implementation on google drive? Definitely could be a pain to implement, but if it's better it's worth implementing

Yeah, I think google drive works like that; I can confirm that message
selection in thunderbird works that this:
Ctrl toggles the selection of the clicked item.
Shift selects everything from the most recently clicked item [note this
can be unselected, think select cell then ctrl+click, then shift to
another cell] to the currently clicked item
I think it could be more intuitive to some users, but I am not really
sure what "most" people consider "standard" on the internet.


Documentation for AS3

Where can I find a copy of the documentation for AS3?
I see a very good explanation of the library for Unity and I'd like to have something similar for AS3 so I can know with more detail how I can use this library
Thanks

Unfortunately only from examples (https://github.com/TouchScript/TouchScript.as3/tree/master/Examples). This library has been discontinued long time ago when I stopped developing with AS3.

Thank you @valyard I already took a look at the examples and they're working fine for me.
I think it's time to move away from AS3 to Unity :P

Join the Dark Side!


Tree Shaking problem with Swiper

Swiper Version: 6.4.10
Platform/Target and Browser Versions: Windows CHROME (I use webpack for bundling)

What You Did
I use Swiper like this (dynamic importing in webpack)
const {Swiper, Navigation} = await import("swiper");
Swiper.use([Navigation]);
const mySwiper = new Swiper(".swiper-container", {...});
Expected Behavior
I expect the smaller size in the vendor/swiper (by webpack tree shaking) when I build my project
Actual Behavior
All of Swiper components are imported to vendor/swiper

Change const {Swiper, Navigation} = await import("swiper"); to import { Swiper, Navigation } from 'swiper'

And make sure tree shaking configured properly in Webpack config

Same Problem in vite.
As a workaround I created a vite plugin to strip out all the modules, I'm not using:
{
  name: 'swiper-remove-unused',
  transform(src, id) {
    if (/swiper\.esm\.js$/.test(id)) {
      const code = src.match(/export .* as (Navigation|Swiper|FreeMode).*;/g)?.join('\n') ?? src
      return {
        code,
        map: null
      }
    }
  },
},

before:
dist/app-vite/chunks/swiper.esm.63281de0.js   137.86 KiB / gzip: 39.63 KiB
after:
dist/app-vite/chunks/swiper.esm.cb5f9dac.js   68.86 KiB / gzip: 20.75 KiB
I think setting:
exports: { ".": "./"}

to allow importing modules directly would help.


Implementing Point Data similar to what exists in the JSON-API

It would be good to have the Point data functionality existing in the JSON-API, e.g. by overloading the Hourly constructor e.g. via something like: Hourly(lat, lon, alt, start, end, tz)

This feature was implemented in v1.1.0.


Updating /etc/hosts on docker

Docker has had a long standing issue preventing writing of /etc/hosts.
root@www:~# mv -f /tmp/hosts /etc/hosts
mv: cannot move '/tmp/hosts' to '/etc/hosts': Device or resource busy

However, this does work:
cat /tmp/hosts > /etc/hosts
rm /tmp/hosts

Could this be changed in the plugin?

Hello, I just picked up this project, are you still able to reproduce this issue?

I'm experiencing this problem as well:
==> login3: Creating the container...
    login3:   Name: vagrant_login3_1438670100
    login3:  Image: bashtoni/centos7-vagrant
    login3: Volume: /opt/local_repo:/opt/local_repo
    login3: Volume: /vagrant/puppet:/srv/puppet
    login3: Volume: /vagrant/puppet/keys:/etc/puppet/keys
    login3: Volume: /vagrant:/vagrant
    login3: Volume: /vagrant/puppet/manifests:/tmp/vagrant-puppet/manifests-846018e2aa141a5eb79a64b4015fc5f3
    login3: Volume: /vagrant/puppet/modules:/tmp/vagrant-puppet/modules-49b3b11000988f27c566b5d91d584b1f
    login3: Volume: /vagrant/puppet/localmodules:/tmp/vagrant-puppet/modules-dbe66f79eff7ee1c95f21e72d4d705a2
    login3:   Port: <IP_ADDRESS>:2200:22
    login3:
    login3: Container created: 679688f6f271a1d2
==> login3: Starting container...
==> login3: Waiting for machine to boot. This may take a few minutes...
The following SSH command responded with a non-zero exit status.
Vagrant assumes that this means the command failed!

mv -f /tmp/hosts /etc/hosts

Stdout from the command:

Stderr from the command:

mv: cannot move ‘/tmp/hosts’ to ‘/etc/hosts’: Device or resource busy

Let me know if you need any more info, or need debug output

:+1:

It looks like this issue can be closed?


Raster Tile Rendering

We want to be able to render raster tiles. This is required for rendering data sets like "Natural Earth". Raster tiles can be of any web-native image format like PNG or WEBP.
🤔 Expected Behavior
Raster tiles need to be scaled and transformed like vector tiles.
😯 Current Behavior
No raster tiles are supported.
💁 Possible Solution
The WebGPU spec offers a way to load bitmaps to textures: https://toji.github.io/webgpu-best-practices/img-textures.html
In the browser this requires the createImageBitmap() API which offloads decoding of images to the browser.
🔦 Context

💻 Examples

Any ideas about implementation of this feature?

Got it. Thanks for update. I'll try to implement it in wgpu.

Just as a note: I have a personal interest in this feature! At the moment, I'm using MapLibre (JavaScript) on my personal site to overlay .webp images of environmental drone imagery I've collected. The site's backend is actually written in Rust, and I'd love the opportunity to re-write this particular application with maplibre-rs as well, which is dependent on raster imagery being available (and symbols). Really excited about this project, so just wanted to mention a use case I'd personally be delighted to have support for at some point. I don't have a ton of capacity for new contributions at the moment, but if you need support for something like testing for either this project or the wgpu feature, just let me know!

Hey. I just created a PR to wgpu. It probably take a time to merge it. But it's not a blocker any more for start this feature implementation.

@ybiletskyi, @Quillasp might also be interested in working on this as part of his bachelor thesis. I wondered whether collaboration could make sense. If not it shouldn't be hard to find two distinct areas to work on in maplibre-rs.

@Drabble can you check with @Quillasp?

@Drabble gave me an overview of the project today. But the project is still quite big (for me at least), do you have any idea where I should start looking in the code to see where I could implement the basics of this feature ? (sorry for the noob question)

Probably you should add new layer with type raster link that represent raster tiles on a map. Than you add logic for loading data (io module I supose) and processing data.  After that I think you can start to render data.


Fix test_softmax, add test_log_softmax, fix Float generation

Passes on both cosim and F1

Other than the license the code looks great!