Control of the styles of the chevrons

I would like to set different colors on the chevrons in the tree for each node, depending on a special condition. Is there an easy way to do this?

Chevrons ? Expanded and collapse to have a different colour ?

Not different colors between the expanded and collapsed chevrons, but different colors between the chevrons in front of the different nodes. Something like this:

@SciManJD
Thanks for attaching this, and apologies for the delayed response.
A couple of observations:

From the image, it is not exactly clear if the colored boxes are checkboxes or expanders used to expand and or collapse the node.

If they are checkboxes, does it mean the checkboxes only apply to certain levels and not others?

I assume it is possible to support custom expander elements. In this case, it could be these colored boxes that can be styled in whatever way you deem fit.
If this assumption is validated, I think I can work to support something like
<template>
    <vue3-tree-vue :items="items"
                   :isCheckable="false"
                   :hideGuideLines="false"
                   v-model:selectedItem="selectedItem">
       <template v-slot:item-prepend-expander="treeViewItem" >
             <your-component-here-with-styling  />
       </template>
    </vue3-tree-vue>
</template>

Regards,

Yes, this is exactly what I need. These are not check-boxes, but expanders. I want to control their color, depending on some internal states. I will be quite good if this feature can be supported.

@SciManJD you can upgrade to 1.0.18 to customize your expander.


feat: add new hoverOverlay prop

Adds new hoverOverlay prop

This prop accepts optional contents to render over the player while the user is hovering over it

Resolves #54

:tada: This PR is included in version 9.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


download does not work

The code for uploading seems to be missing updates comparing to download code.
In main.py, function operation_download:
please add code like in operation_upload:
sources, destinations = destination_from_source(sources)
if len(destinations) == len(sources):
...
In main.py, function destination_from_source
newsources[i] = srcdst[0] might be better replaced by
newsources.append(srcdst[0])
glob.glob(srcdst[0]) returns an empty list, therefore the loop over listing does not work, just
filename = sources[i]
and omitting the loop seems to work

should be closed by 324ed74f31fe45314b558ef05d55c90f74d11759


Add derivatives support.

What does this PR do?
Add support for processor's result collection.
Motivation
Needed for integrating API security since schema extraction is returned in derivatives property.
Checklist

[x] Unit tests have been updated and pass

You forgot to update the typings in index.d.ts!

related: https://github.com/DataDog/dd-native-appsec-js/issues/43


alternate dofile() behavior

dofile("file.lua") should look for file.lua in the same directory as the .tic game file. Also, I don't like that TIC80 creates a new game folder for each version release. These two issues make it hard to use version control software, like GIT, to save progress.

Someone? Having the same problem??

it loads file.lua from the working directory
you could keep .tic and .lua files in the same directory and run TIC from it
as for new directory for each folder, going to stop do it when API and Cart layout will be more less stable

On OSX, dofile("filename.lua")  returns a "file not found" error. I have to enter the full directory name. This is why I've been keeping the .lua file in a separate folder. I'll check to make sure the behavior is different in windows.

yes, because 'filename.lua' not your working dir

I think the idea of including code externally is great 👍  In my limited (pico-8) experience, other in-engine editors are quite OK - but the exception is the code editor. My eyes start to bleed and my head hearts after a short while, and wihle I see that TIC-80 has a great addition of a function-outline - editing code with a real editor (like VSCode) helps so much when you're starting to have more than 200 lines.
So dofile is good - it also eliminates the danger of overwriting data when you're working in parallell with code and in-engine editors and accidentaly forget to save/reload properly each time you switch programs
It only needs to work a bit better with removing the need for tic-80 being launched from your working dir. I usually create a symlink to my dropbox dir in order to share carts on different computers. They become something like cartdir\dropbox. In Windows I have to:

Add tic-80 dir to PATH (once, unless this dir changes with versions)
Start a cmd-prompt
Go to the dropbox-dir
Start tic
Once TIC is started - I have to cd to my dropbox again
Finally load the cart

It's not a huge deal, but improving this would be more intuitive and support this "pattern" which I think people enjoy 😄

#715 duplicate


Update documentation of ribogrid data by position and length to riboviz_outputs.md

We need to check that the ribogrid data (read counts by position and length), as passed in to metagene_start_ribogrid_by_length.pdf, is described in docs/user/riboviz_outputs.md. I don't know what it's called though!
The current description is incorrect:

using data that is saved in the metagene_start_stop_read_counts.tsv file.

We should check and fix this as part of #417. Requesting @FlicAnderson to address as part of resolving merge conflicts etc in that?

The file gene_position_length_counts_5start.tsv  contains read counts by position and length, and is described in riboviz-outputs.md beside the ribogrid plots.

Here is an example of the gene_position_length_counts_5start.tsv file
gene_position_length_counts_5start.txt

I've created pull request #428 to address this, by correcting the reference.
BUT - wouldn't it be better to call that file metagene_position_length_counts_5start.tsv, because it is a metagene file? If you think so, let's fix that in #428 too.

That makes sense, I'll go through and make changes in the code and documentation today. The only bit I will struggle with is the .dot and .svg files as @mikej888 handled those last time.

@3mma-mack please ping me when done and I can update the .dot and .svg files again.

I have updated file names and variables from gene_position_length_counts_5start to metagene_position_length_counts_5start, and tested that everything still works, which it does.
@mikej888 would you be able to test and update the .dot and .svg, and the I think the issue is solved.

@3mma-mack I've updated .dot and .svg and checked your updates and all seems fine.


Feature Request: Configurable service name format in tracing data

This change resulted in span names being recorded as app-label:namespace.  The PR makes sense, however for users like me where we have a loosely 1-2-1 mapping between application and namespace it is just overly verbose as the two values are typically the same:

You can imagine how noisy this gets on deep nested spans.
I understand why the change was made, but it'd be much appreciated if it was a configurable thing.

/cc @douglas-reid

/assign @douglas-reid

I think this might be something we should consider as part of a refactoring of a Trace API, as well.  Similar to access log format, we could support traceOperationFormat or similar.
/cc @objectiser

@douglas-reid Agree would be good to support in Trace API - although this would be about the service name, not operation name. Suggestion: title should be something like "Configurable service name format in tracing data"?

@geeknoid @howardjohn not sure your bot is working :'(

Stop closing this :slightly_frowning_face:

@douglas-reid we still have to work around this by modifying our injector, is there anything in the newer tracing apis that allows us to handle this?
The config is:
        - --serviceCluster
        {{ if ne "" (index .ObjectMeta.Labels "app") -}}
        - "{{ index .ObjectMeta.Labels `app` }}.$(POD_NAMESPACE)"
        {{ else -}}

For us; the $(POD_NAMESPACE) part is just noise  so we're looking for a way to ensure serviceCluster is simply labels.app.

@Stono not at the moment. i'm not sure how to model that particular bit in the proposed APIs, as service cluster is set via bootstrap -- and the proposed APIs don't address bootstrap concerns.

Maybe it's not a observability concern then and more of an injector config option 🤷

@Stono that's my thought as well. Make this a meshconfig option.

@douglas-reid @howardjohn please can we reopen this, the workaround above seems to be broken in 1.12, we have pods with the serviceCluster being set:

But traces are once again coming through as namespace.name:

Decided to open a new issue (https://github.com/istio/istio/issues/36162) to track the bug as this issue was specifically about being able to control the serviceName programatically

Closing in favour of the discussion in https://github.com/istio/istio/issues/36162


add cloudconfig command

lxops has the functionality to apply cloud-config files to instances, but it does so through an lxops config file.
I propose adding a command to apply cloud-config files directly from the command line:
lxops apply -i <instance> -ostype alpine <cloud-config-file.cfg> ...

lxops apply -i <instance> -ostype alpine - <<END
#cloud-config
runcmd:
- hostname
END

There are currently separate projects cloudconfiglxd, cloudconfigincus that do this.
Incorporating this functionality in lxops will have the advantages:

avoid having to compile cloudconfiglxd and/or cloudconfigincus separately from lxops_lxd, lxops_incus.
use a unified and consistent command line interface, in lxops instead of duplicating it in cloudconfiglxd, cloudconfigincus.

Added instance cloudconfig command.
Also added ostypes command to list available os types.


Possibility to use same template for different components

I'd like to have:
class Page extends BlazeComponent
  @register 'pageTemplate'

  page: ->
    Pages.findOne FlowRouter.getParam("_id")

class AnyPage extends Page
  page: ->
    Pages.findOne()

template(name="pageTemplate")
   h1= page.title
   .content= page.content

And in some other template I'd like to do:
template(name="dashboard")
  div.current-page
    +Page

  div.any-page
    +AnyPage

In React such stuff is totally possible. As far as I understand in blaze-components I should register some additional template.
template(name="AnyPage")
  +pageTemplate page=page

AnyPage.register 'AnyPage'

No, you can use the same template for different components. The default template name is equal to the component name. But you can override this by returning a different name/template from template method. See here.
So something like:
class AnyPage extends Page
  template: ->
    'Page'

  page: ->
    Pages.findOne()

Cool. Thanks.


[FEATURE] Backup/restore Longhorn system itself

Rancher has introduced ETCD snapshot/restore, but we found that it doesn't work well with Longhorn. More importantly, there is currently no recommended way to backup the Longhorn system itself. It can be useful for the following scenarios:

Rollback in the case of an upgrade failure.
Recreate/replicate the cluster

In general, we can dump the related CRDs and rollback to them if something went wrong.
It requires some designs to offer the best experience to the end user.
REF: https://github.com/longhorn/longhorn/pull/1648

This is what we have now for backup/restore longhorn.

https://longhorn.io/docs/1.3.0/advanced-resources/cluster-restore/restore-to-a-cluster-contains-data-using-rancher-snapshot/
https://longhorn.io/docs/1.3.0/advanced-resources/cluster-restore/restore-to-a-new-cluster-using-velero/

Note: @FrankYang0529 is currently working on backing image backup and restore.

Add backup cmd to the backing image manager to upload to the backup targets.
Add fields lastBackingImage... to the related CR

For this issue, we can leverage the field lastBackingImage for restoring the rollout.
Relate issue https://github.com/harvester/harvester/issues/2237

Backing image backup/restore, ref https://github.com/longhorn/longhorn/pull/4385.

Testing in progress, will update the result here.


Refactor error handling

Jira: XXXXX
Changes:

...

@Matovidlo I haven't asked for a review yet :)
If someone would send me this for review, in its current state, I would reject it right away.


benchmark typos

Hi,
I evaluated my model with your benchmark and noticed that some benchmarks contain typos or strange parts in descriptions like propaganda_demonizace - >\n           in description, propaganda_emoce - /n./n as description, csfever_nli - následujúcích in description, etc.
I did not go through lm-eval thoroughly but I can see them used in results. I wonder if these are intentional to test models or not. I checked the datasets in HF but there is no specific info for the prompts used here.
I'll be glad for clarification.
Thank you very much!
Ondřej

Hello @olisicky. In general, we followed this philosophy. Contents of the prompts are human written and thus susceptible to errors. With 54 tasks, the errors are inevitable. We consider them a part of human language. The prompts really depend on the creativity of the person at the time of writing, and were not following a standardized process (the only requirement was that the human should be able to understand the prompt).
Regarding the prompt you mentioned, I hope you find the answers below:
I was writing/editing all the prompts mentioned in your post, so I think I can respond here.
propaganda_demonizace
is designed on purpose like it is, to test whether models are not susceptible to some kind of code writing style when prompted with >\n.
propaganda_emoce - I don't exactly remember what I aimed at here. I know I wanted to create some prompts for autoregressive language models (so not much of the description, just text + continuation). The description here seems to carry no more meaning than empty description. In reflection, maybe the empty description would be more suitable. However, I don't see why the current form should hurt or improve language model behavior, so it can stay like it is (and thus we can avoid reevaluating all models).
csfever_nli
This is an error of my Slovak/Czech code mixing. I tried to be careful, but you find the case where I was not successful. Given the lexical similarity between Czech and Slovak version of these two words, I wouldn't change the prompt. Also CSfever contains 7 prompts (as taken from previous work of Stefanik et al., and edited), which is more then 5 we required.
Also, technically, this would require reevaluating csfever across all models, with very low likelihood of scores changing (two different characters in 1 word, in 1 prompt, given max out of 7).
Thanks for the heads up.
Let me know if you find any more fishy things please.
Also be mindful that we pre-released v0.3 today with fixed subjectivity task.

Hi @MFajcik . Thanks for quick response. I was just curious if you aimed to some specific errors which would be interesting. I understand that some mistakes are inevitable. I can confirm that the changes are minimal for most of the propaganda benchmarks at least for my model. There was just one outlier for propaganda_nalepkovani see results below.
Old:
    "Propaganda Nálepkování": {
      "avg_mcauroc,none": 0.5093898809523809,
      "avg_mcauroc_stderr,none": "N/A",
      "macro_f1,none": 0.5095641686670223,
      "macro_f1_stderr,none": "N/A",
      "acc,none": 0.7724,
      "acc_stderr,none": 0.005901262992352133,
      "alias": "Propaganda Nálepkování"
    }

New:
    "propaganda_nalepkovani_4": {
      "alias": " - prompt-4",
      "acc,none": 0.815,
      "acc_stderr,none": 0.012285191326386708,
      "macro_f1,none": 0.5506556783763368,
      "macro_f1_stderr,none": "N/A",
      "avg_mcauroc,none": 0.5828794642857142,
      "avg_mcauroc_stderr,none": "N/A"
    }

I think that it is good decision not to re-evaluate all the models, this should be just a detail.
I have another question for you (sorry for bothering you with everything :D).
I pulled the repo after your last commits 4 days ago and now I have a problem that e.g. benczechmark_csfever_nli do not compute mcauroc and thus I cannot continue with evaluations. I noticed that you also recomputed leaderboard. Did you recomputed all the benchmarks or just some of them? I guess it will be problem on my side but would be nice to confirm with you that you did not get into similar problems.
Thank you!
Ondřej

Hi @MFajcik . Thanks for quick response. I was just curious if you aimed to some specific errors which would be interesting. I understand that some mistakes are inevitable. I can confirm that the changes are minimal for most of the propaganda benchmarks at least for my model.
I think that it is good decision not to re-evaluate all the models, this should be just a detail.
I have another question for you (sorry for bothering you with everything :D).
I pulled the repo after your last commits 4 days ago and now I have a problem that e.g. benczechmark_csfever_nli do not compute mcauroc and thus I cannot continue with evaluations. I noticed that you also recomputed leaderboard. Did you recomputed all the benchmarks or just some of them? I guess it will be problem on my side but would be nice to confirm with you that you did not get into similar problems.
Thank you!
Ondřej

Hello @olisicky, as this is related to this issue, maybe we can take it offline (or please do the new issue :)) (you can write me on<EMAIL_ADDRESS>so we exchange contacts). I ran the fever just now with newly cloned repo, and it went fine.


pull request for e2e test at 01/14/2017 04:45:09

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Update admin.middleware.js

Issue: #41
task  added

You are not supposed to delete anything. Write your code in the given empty functions.


requests.exceptions.HTTPError: 406 Client Error: Not Acceptable for client.get_plant_stats

Hi,
I just had installed a photovoltaic plant and I found your project in order to get the data for integrating into my own page.
The "stats = client.get_power_status()" example works fine to retrieve current data, however, when, according to the second example I try to retrieve the "daily" stats data, I get:
requests.exceptions.HTTPError: 406 Client Error: Not Acceptable for url: https://region01eu5.fusionsolar.huawei.com/rest/pvms/web/station/v1/overview/energy-balance?stationDn=NE%3D33586185&timeDim=2&queryTime=1662720704354&timeZone=2&timeZoneStr=Europe%2FVienna&_=1662720707250

The code correctly gets the one plant I have (through "plant_ids = client.get_plant_ids()", plant_ids[0] gets NE=33586185 as id). The call which fails is "plant_data = client.get_plant_stats(plant_ids[0])"
The FusionSolar app (on iOS ou in a browser) works fine and give me the day stats charts.
What can I do ?

Hi @MichelRabozee,
Thanks a lot for reporting this bug! Sorry about the delay! I'll try to fix it ASAP

Hi @MichelRabozee,
I did a quick check and the code still works for me.
As a first test: Can you check when you open the FusionSolar web app through a browser, that the URL starts with region01eu5.fusionsolar.huawei.com?
Depending on where you are located, the subdomain may be different.
Kind regards,
Johannes

Hi,
I just double checked my HomeAssistant log files: The same error occurred once 2 days ago but then never again. Maybe they had some kind of update running on their servers. Right now, I can't reproduce the error and for the last 2 days no additional errors were logged on HomeAssistant as well.

Hi,
my kiosk is at https://region01eu5.fusionsolar.huawei.com/pvmswebsite/nologin/assets/build/index.html#/kiosk?kk=xxxxx and my full web app is at https://region01eu5.fusionsolar.huawei.com which goes to https://eu5.fusionsolar.huawei.com/unisso/login.action?service=https%3A%2F%2Fregion01eu5.fusionsolar.huawei.com%2Funisess%2Fv1%2Fauth%3Fservice%3D%252Fnetecowebext%252Fhome%252Findex.html&decision=1#/LOGIN
HTH, and do not hesitate to ask more info (for instance, I can give you my kiosk code in PM so you may check),
kind regards,
Michel

Hi,
As a first step, can you maybe simply try whether the code works right now? I just ran all unit tests (which include the code shown in the README) and they worked fine.

Hi !
Sorry, still no go :-(
MacMini2:fusionsolar mrabozee$ ./get_today_fusionsolar_data.py
Found 1 plants
Found NE=33586185 plant
Traceback (most recent call last):
  File "/usr/local/lib/python3.9/site-packages/fusion_solar_py/client.py", line 46, in wrapper
    result = func(self, *args, **kwargs)
  File "/usr/local/lib/python3.9/site-packages/fusion_solar_py/client.py", line 298, in get_plant_stats
    r.raise_for_status()
  File "/usr/local/lib/python3.9/site-packages/requests/models.py", line 943, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 406 Client Error: Not Acceptable for url: https://region01eu5.fusionsolar.huawei.com/rest/pvms/web/station/v1/overview/energy-balance?stationDn=NE%3D33586185&timeDim=2&queryTime=1663450290719&timeZone=2&timeZoneStr=Europe%2FVienna&_=1663450293282

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/Users/mrabozee/sources/fusionsolar/./get_today_fusionsolar_data.py", line 16, in <module>
    plant_data = client.get_plant_stats(plant_ids[0])
  File "/usr/local/lib/python3.9/site-packages/fusion_solar_py/client.py", line 50, in wrapper
    result = func(self, *args, **kwargs)
  File "/usr/local/lib/python3.9/site-packages/fusion_solar_py/client.py", line 298, in get_plant_stats
    r.raise_for_status()
  File "/usr/local/lib/python3.9/site-packages/requests/models.py", line 943, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 406 Client Error: Not Acceptable for url: https://region01eu5.fusionsolar.huawei.com/rest/pvms/web/station/v1/overview/energy-balance?stationDn=NE%3D33586185&timeDim=2&queryTime=1663450290719&timeZone=2&timeZoneStr=Europe%2FVienna&_=1663450295503

The code I am using is:
<EMAIL_ADDRESS># -*- coding: UTF-8 -*-

from fusion_solar_py.client import FusionSolarClient

# log into the API - with proper credentials...
client = FusionSolarClient("Michel_Rabozée", "(Matignon14)", huawei_subdomain="region01eu5")

# get the plant ids
plant_ids = client.get_plant_ids()

print(f"Found {len(plant_ids)} plants")
print(f"Found {plant_ids[0]} plant")

# get the data for the first plant
plant_data = client.get_plant_stats(plant_ids[0])

# plant_data is a dict that contains the complete
# usage statistics of the current day. There is
# a helper function available to extract some
# most recent measurements
last_values = client.get_last_plant_data(plant_data)

print(f"Last production at {last_values['productPower']['time']}: {last_values['productPower']['value']}")

# In case you have a battery installed
print(f"Last battery charge at {last_values['chargePower']['time']}: {last_values['chargePower']['value']}")

# log out - just in case
client.log_out()

Hi @MichelRabozee ,
Just as a warning: You added your username and password in the code example.
I must also apologize: I used them to fix the bug. Your setup is a bit different as your dongle is registered as a separate device. That's why the ID fetching function returned the wrong id.
I've update the package and changed the way the station ids are found (dc8a4ea434d739681e3392e92ea2e579ac00d232). If you install the new version (0.0.7) it should work. Please let me know if it doesn't!

Thank you for having removed the insecure posts. I was in a hurry and completely forgot to obfuscate those :-( I will check the new version a bit later today, I have some issues to attend to.

Hello,
seems now to work perfectly ! Thank you !
Have a nice day.


[CI] - The process '/opt/hostedtoolcache/goreleaser-action/1.20.0/x64/goreleaser' failed with exit code 1

Maybe due to the latest release of GoReleaser (v1.20.0), the workflow "ci" is broken.

Are you part of the #hacktoberfest ? :)


[AutoPR compute/resource-manager] [Compute] Fix List VM Extension API name, description updates and other fixes.

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/3152

(message created by the CI based on PR content)

Codecov Report

Merging #2677 into restapi_auto_compute/resource-manager will decrease coverage by 0.73%.
The diff coverage is 20%.

@@                            Coverage Diff                            @@
##           restapi_auto_compute/resource-manager    #2677      +/-   ##
=========================================================================
- Coverage                                  54.95%   54.22%   -0.74%
=========================================================================
  Files                                       7845     5819    -2026
  Lines                                     174433   134333   -40100
=========================================================================
- Hits                                       95865    72845   -23020
+ Misses                                     78568    61488   -17080

Impacted Files
Coverage Δ

..._12_01/models/virtual_machine_scale_set_os_disk.py
100% <ø> (ø)
:arrow_up:

...ute/v2017_12_01/models/rollback_status_info_py3.py
55.55% <ø> (ø)
:arrow_up:

...e/azure/mgmt/compute/v2017_12_01/models/os_disk.py
100% <ø> (ø)
:arrow_up:

...compute/v2018_04_01/models/rollback_status_info.py
55.55% <ø> (ø)
:arrow_up:

...compute/v2017_12_01/models/rollback_status_info.py
55.55% <ø> (ø)
:arrow_up:

...e/mgmt/compute/v2017_12_01/models/data_disk_py3.py
100% <ø> (ø)
:arrow_up:

...ute/v2018_04_01/models/rollback_status_info_py3.py
55.55% <ø> (ø)
:arrow_up:

...18_04_01/operations/virtual_machines_operations.py
12.86% <ø> (+0.15%)
:arrow_up:

...01/models/virtual_machine_scale_set_os_disk_py3.py
100% <ø> (ø)
:arrow_up:

...ure/mgmt/compute/v2017_12_01/models/os_disk_py3.py
100% <ø> (ø)
:arrow_up:

... and 2762 more

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 89625c9...cd432dc. Read the comment docs.

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/2487


Nginx+docker-compose example

Hi,
I'm in the middle of a Sanic based project.
I wanted to use docker and Nginx but couldn't find a simple HOW-TO,
So I ended up creating a simple skeleton app for using sanic behind Nginx (using docker-compose).
Would love getting some feedback (as I'm new to Sanic), And also wanted to ask if it's OK adding some link for it in the docs - as I think nginx+Sanic would be a common combination.

Oops forgot to give a link - nginx+docker-compose example :)

Awesome! However, this might make more sense as part of our examples (in a directory within examples) then as a separate repo. There isn't really a place for this in the docs right now, since it's not really an extension.

Or, if you prefer to add it as an extension that's fine too...

Hi, @r0fls sorry for the late response, thanks for the feedback.
I would prefer to put it as an extension (at least for now), as I plan to rapidly improve the repo and add some more basic docs on starting a sanic project, would be easier as a separate project.

This is awesome 😄
@r0fls maybe it would be good to include something like these lines in the base configuration that automatically check if there's an environment variable for thing and override if they exist.


Cannot add mobs to creatures expect pigs [1.21 Fabric]

I have tried to put the human_reborn:human mob to the mob additions but it just refuses to spawn him, but when I change it to minecraft:pig, perfectly fine! Also tested with most hostile mobs like zombies, skeletons and withers and they dont seem to spawn either:
{
"chunkConstant": 17,
"rareSpawnTicksToWait": 400,
"monsterGroup": {
"capacity": 70,
"peaceful": false,
"rare": false,
"immediateDespawnRange": 128,
"despawnStartRange": 32
},
"creatureGroup": {
"capacity": 10,
"peaceful": true,
"rare": true,
"immediateDespawnRange": 128,
"despawnStartRange": 32
},
"ambientGroup": {
"capacity": 15,
"peaceful": true,
"rare": false,
"immediateDespawnRange": 128,
"despawnStartRange": 32
},
"axolotlGroup": {
"capacity": 5,
"peaceful": true,
"rare": false,
"immediateDespawnRange": 128,
"despawnStartRange": 32
},
"waterCreatureGroup": {
"capacity": 5,
"peaceful": true,
"rare": false,
"immediateDespawnRange": 128,
"despawnStartRange": 32
},
"waterAmbientGroup": {
"capacity": 20,
"peaceful": true,
"rare": false,
"immediateDespawnRange": 64,
"despawnStartRange": 32
},
"undergroundWaterGroup": {
"capacity": 5,
"peaceful": true,
"rare": false,
"immediateDespawnRange": 128,
"despawnStartRange": 32
},
"miscGroup": {
"capacity": -1,
"peaceful": true,
"rare": true,
"immediateDespawnRange": 128,
"despawnStartRange": 32
},
"passivePersistent": true,
"hostilePersistent": false,
"ambientPersistent": false,
"waterPersistent": false,
"mobSpawnAdditions": [
{
"biomeId": "minecraft:plains",
"biomeTag": "",
"mobId": "human_reborn:human",
"spawnGroup": "CREATURE",
"weight": 100,
"minCount": 1,
"maxCount": 4
}
],
"mobSpawnRemovals": [],
"mobSpawnReplacements": [],
"overrideSpawnerDefaultValues": false,
"minSpawnDelay": 200,
"maxSpawnDelay": 800,
"spawnCount": 4,
"maxNearbyEntities": 6,
"requiredPlayerRange": 16,
"spawnRange": 4
}

Never mind, theyre hostile mobs and probably cant spawn in any light level other than 0, would be good if you added like an exeption for this sort of stuff


Write a JavaScript program to find area of a circle

Description
Write a JavaScript program to find area of a circle
Area of circle = π × Radius^2
Radius : 2

Area : 12.566370614359172

How to contribute

Save the solution in FindAreaOfACircle.js file
Add FindAreaOfACircle.js file in find-area-of-a-circle folder

I'll grab this one)

!assign

The PR merged a while ago.


spin indexing #219

what do we think of something like this? #219
no docstrings or descriptions yet - I'll add them once everyones happy :)
created the order functions such that they take in the mode_idx and number of modes as arguments.
there is a lonely order_function (for up-then-down) under transforms/order_functions
here is a sample output:
from openfermion.transforms import get_fermion_operator
from openfermion.transforms import up_then_down
from openfermion.ops import normal_ordered

geometry = [("H", (0., 0., 0.)), ("H", (0., 0., 0.6))]
molecule = MolecularData(geometry, "sto-3g", 1,
                         description="0.6")
molecule.load()
molecular_hamiltonian = molecule.get_molecular_hamiltonian()
hamiltonian = normal_ordered(get_fermion_operator(molecular_hamiltonian))

print hamiltonian
print "\n----------------------\n"
print hamiltonian.reorder(order_function=up_then_down)
print "\n----------------------\n"
print hamiltonian.reorder(num_modes = 8, order_function=up_then_down)

0.17373064388 [3^ 0^ 2 1] +
-0.365770571815 [2^ 2] +
0.17373064388 [2^ 1^ 3 0] +
-0.515062453042 [3^ 1^ 3 1] +
-0.17373064388 [1^ 0^ 3 2] +
-0.365770571815 [3^ 3] +
-0.701337728655 [1^ 0^ 1 0] +
-0.688793096922 [2^ 1^ 2 1] +
-0.17373064388 [3^ 2^ 1 0] +
0.881962014317 [] +
-0.515062453042 [2^ 0^ 2 0] +
-0.724506023965 [3^ 2^ 3 2] +
-1.34221399305 [0^ 0] +
-1.34221399305 [1^ 1] +
-0.688793096922 [3^ 0^ 3 0]

----------------------

-0.515062453042 [1^ 0^ 1 0] +
-1.34221399305 [2^ 2] +
-0.17373064388 [2^ 0^ 3 1] +
0.17373064388 [3^ 0^ 1 2] +
-0.724506023965 [3^ 1^ 3 1] +
-0.365770571815 [3^ 3] +
-0.17373064388 [3^ 1^ 2 0] +
-0.688793096922 [1^ 2^ 1 2] +
0.881962014317 [] +
-0.701337728655 [2^ 0^ 2 0] +
-0.515062453042 [3^ 2^ 3 2] +
-1.34221399305 [0^ 0] +
-0.365770571815 [1^ 1] +
-0.688793096922 [3^ 0^ 3 0] +
0.17373064388 [1^ 2^ 3 0]

----------------------

0.17373064388 [5^ 0^ 1 4] +
-1.34221399305 [4^ 4] +
-0.688793096922 [1^ 4^ 1 4] +
-0.365770571815 [1^ 1] +
-0.17373064388 [5^ 1^ 4 0] +
-0.515062453042 [5^ 4^ 5 4] +
-0.17373064388 [4^ 0^ 5 1] +
-0.724506023965 [5^ 1^ 5 1] +
-0.701337728655 [4^ 0^ 4 0] +
0.17373064388 [1^ 4^ 5 0] +
-0.515062453042 [1^ 0^ 1 0] +
0.881962014317 [] +
-0.365770571815 [5^ 5] +
-1.34221399305 [0^ 0] +
-0.688793096922 [5^ 0^ 5 0]

this was quick prototyping haven't really tested it yet, please let me know if you spot bugs!

We also need to integrate this with the transformations. haven't done that yet!

A couple things 1) there seems to be some conflicts on this branch, perhaps your fork is a few PRs behind? 2) I think a big question is whether to implement the reordering as a method on FermionOperator or an external function. In general, we try very hard to keep these methods as external functions. One of the reasons is because we ultimately intend to implement the core data structures in C++ (one day) and it will be much easier to make changes like this if we keep the core classes lean. I am in favor of having this be an external function rather than a method. What do others think?
Also, should we perhaps have this reordering be general to SymbolicOperator rather than FermionOperator?

oh it didn't like the way I reverted molecular_data.
It can easily become an external function and generalized to SymbolicOperator.

from openfermion.transforms import get_fermion_operator
from openfermion.transforms import up_then_down
from openfermion.ops import normal_ordered
from openfermion.utils import reorder
from openfermion.transforms import bravyi_kitaev

geometry = [('H', (0., 0., 0.)), ('H', (0., 0., 0.6))]
molecule = MolecularData(geometry, 'sto-3g', 1,
                         description="0.6")
molecule.load()
molecular_hamiltonian = molecule.get_molecular_hamiltonian()
hamiltonian = normal_ordered(get_fermion_operator(molecular_hamiltonian))

print hamiltonian
print '\n----------------------\n'
print reorder(hamiltonian,order_function=up_then_down)
print '\n----------------------\n'
print reorder(hamiltonian,num_modes = 8, order_function=up_then_down)

bk_hamiltonian = bravyi_kitaev(hamiltonian)
print '\n----------------------\n'
print bk_hamiltonian
print '\n----------------------\n'
print reorder(bk_hamiltonian,order_function=up_then_down)

0.17373064388 [3^ 0^ 2 1] +
-0.365770571815 [2^ 2] +
0.17373064388 [2^ 1^ 3 0] +
-0.515062453042 [3^ 1^ 3 1] +
-0.17373064388 [1^ 0^ 3 2] +
-0.365770571815 [3^ 3] +
-0.701337728655 [1^ 0^ 1 0] +
-0.688793096922 [2^ 1^ 2 1] +
-0.17373064388 [3^ 2^ 1 0] +
0.881962014317 [] +
-0.515062453042 [2^ 0^ 2 0] +
-0.724506023965 [3^ 2^ 3 2] +
-1.34221399305 [0^ 0] +
-1.34221399305 [1^ 1] +
-0.688793096922 [3^ 0^ 3 0]

----------------------

-0.515062453042 [1^ 0^ 1 0] +
-1.34221399305 [2^ 2] +
-0.17373064388 [2^ 0^ 3 1] +
0.17373064388 [3^ 0^ 1 2] +
-0.724506023965 [3^ 1^ 3 1] +
-0.365770571815 [3^ 3] +
-0.17373064388 [3^ 1^ 2 0] +
-0.688793096922 [1^ 2^ 1 2] +
0.881962014317 [] +
-0.701337728655 [2^ 0^ 2 0] +
-0.515062453042 [3^ 2^ 3 2] +
-1.34221399305 [0^ 0] +
-0.365770571815 [1^ 1] +
-0.688793096922 [3^ 0^ 3 0] +
0.17373064388 [1^ 2^ 3 0]

----------------------

0.17373064388 [5^ 0^ 1 4] +
-1.34221399305 [4^ 4] +
-0.688793096922 [1^ 4^ 1 4] +
-0.365770571815 [1^ 1] +
-0.17373064388 [5^ 1^ 4 0] +
-0.515062453042 [5^ 4^ 5 4] +
-0.17373064388 [4^ 0^ 5 1] +
-0.724506023965 [5^ 1^ 5 1] +
-0.701337728655 [4^ 0^ 4 0] +
0.17373064388 [1^ 4^ 5 0] +
-0.515062453042 [1^ 0^ 1 0] +
0.881962014317 [] +
-0.365770571815 [5^ 5] +
-1.34221399305 [0^ 0] +
-0.688793096922 [5^ 0^ 5 0]

----------------------

(-0.299205107575+0j) [Z2] +
(0.0434326609699+0j) [Y0 Z1 Y2 Z3] +
(0.0434326609699+0j) [X0 Z1 X2 Z3] +
(0.194808676873+0j) [Z0 Z1] +
(0.194808676873+0j) [Z0] +
(0.181126505991+0j) [Z1 Z3] +
(0.17219827423+0j) [Z0 Z1 Z2 Z3] +
(0.175334432164+0j) [Z1] +
(0.0434326609699+0j) [X0 Z1 X2] +
(0.0434326609699+0j) [Y0 Z1 Y2] +
(0.12876561326+0j) [Z0 Z2] +
(-0.299205107575+0j) [Z1 Z2 Z3] +
(0.132366162585+0j) [] +
(0.12876561326+0j) [Z0 Z2 Z3] +
(0.17219827423+0j) [Z0 Z1 Z2]

----------------------

(0.0434326609699+0j) [X0 X1 Z2 Z3] +
(0.12876561326+0j) [Z0 Z1 Z3] +
(0.194808676873+0j) [Z0 Z2] +
(0.12876561326+0j) [Z0 Z1] +
(0.175334432164+0j) [Z2] +
(0.181126505991+0j) [Z2 Z3] +
(0.194808676873+0j) [Z0] +
(0.17219827423+0j) [Z0 Z1 Z2] +
(0.0434326609699+0j) [Y0 Y1 Z2] +
(-0.299205107575+0j) [Z1 Z2 Z3] +
(0.17219827423+0j) [Z0 Z1 Z2 Z3] +
(-0.299205107575+0j) [Z1] +
(0.0434326609699+0j) [Y0 Y1 Z2 Z3] +
(0.0434326609699+0j) [X0 X1 Z2] +
(0.132366162585+0j) []```

@conta877 let me know when you are ready for review.

I'm ready for input. (not review). Is this the way we want to proceed?
On Feb 27, 2018 17:04, "Ryan Babbush"<EMAIL_ADDRESS>wrote:

@conta877 https://github.com/conta877 let me know when you are ready
for review.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/quantumlib/OpenFermion/pull/224#issuecomment-369083903,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AV-mvIdinJMhSH-MbcgMEJ4SE04333jhks5tZKYvgaJpZM4SVmSG
.

I think so. It would be good hear from at least two of either:
@ncrubin @kevinsung @jarrodmcc to make sure they agree since all three seemed to have opinions...

I also think integration of this into transforms should be a separate pull
request so that can discuss.
On Feb 27, 2018 17:41, "Ryan Babbush"<EMAIL_ADDRESS>wrote:

I think so. It would be good hear from at least two of either:
@ncrubin https://github.com/ncrubin @kevinsung
https://github.com/kevinsung @jarrodmcc https://github.com/jarrodmcc
to make sure they agree since all three seemed to have opinions...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/quantumlib/OpenFermion/pull/224#issuecomment-369091101,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AV-mvK0oaSbwWPjVKtLWUTJrKcpUWUgSks5tZK7SgaJpZM4SVmSG
.

This looks fine to me.

I think this type of re-order will work well for most of the use cases I'm
thinking of.
I know you said you'd clean up the docstring and unused parameter stuff
after we agreed on the style, but I wanted to make sure I mentioned
something slightly more subtle, as it looks like when an order function is
passed a copy of the modified operator is returned, whereas when no order
function is passed, a reference to the original is returned.  I feel like
this could lead to slightly unexpected behavior in some cases.  In my
opinion it's okay to make the reorder function parameter not optional.  If
they don't want to reorder it, why risk unexpected behavior by letting them
call it.
On Tue, Feb 27, 2018 at 7:22 PM, Kevin J. Sung<EMAIL_ADDRESS>wrote:

This looks fine to me.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/quantumlib/OpenFermion/pull/224#issuecomment-369108669,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHXFPLrtYejXViHpiLyGJ5FWmT5MRcGFks5tZMZzgaJpZM4SVmSG
.

thats a really good point! I'll modify it to make the order function a
must.
On Feb 27, 2018 21:19, "Jarrod"<EMAIL_ADDRESS>wrote:
I think this type of re-order will work well for most of the use cases I'm
thinking of.
I know you said you'd clean up the docstring and unused parameter stuff
after we agreed on the style, but I wanted to make sure I mentioned
something slightly more subtle, as it looks like when an order function is
passed a copy of the modified operator is returned, whereas when no order
function is passed, a reference to the original is returned. I feel like
this could lead to slightly unexpected behavior in some cases. In my
opinion it's okay to make the reorder function parameter not optional. If
they don't want to reorder it, why risk unexpected behavior by letting them
call it.
On Tue, Feb 27, 2018 at 7:22 PM, Kevin J. Sung<EMAIL_ADDRESS>wrote:

This looks fine to me.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<https://github.com/quantumlib/OpenFermion/pull/224#issuecomment-369108669
,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/
AHXFPLrtYejXViHpiLyGJ5FWmT5MRcGFks5tZMZzgaJpZM4SVmSG>

.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/quantumlib/OpenFermion/pull/224#issuecomment-369124832,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AV-mvDWH2iyLTtu97fmx1_nSPJeSGOBzks5tZOHZgaJpZM4SVmSG
.

is it worth having a separate place for the order functions or should they be put in somewhere else? up_then_down look very lonely there right now.

I suggest that for now, we keep both the functions reorder and up_then_down in the same file, either utils/_operator_utils.py or transforms/conversion.py. I don't feel very strongly though.

I would agree that until we have more functions of this type (if we make
them) they should just go alongside the functions they are meant to be used
with for clarity instead of their own file.
On Wed, Feb 28, 2018 at 3:27 PM, Kevin J. Sung<EMAIL_ADDRESS>wrote:

I suggest that for now, we keep both the functions reorder and
up_then_down in the same file, either utils/_operator_utils.py or
transforms/conversion.py. I don't feel very strongly though.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/quantumlib/OpenFermion/pull/224#issuecomment-369419914,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHXFPIyorES8CmKNT4OIOmrEL_UmDE0wks5tZeDpgaJpZM4SVmSG
.

ready for review

Thank you @conta877! In addition to the inline comments I have two general things:

If you haven't done so, please run the PEP8 linter on your code and fix the style violations. You can install it with pip install pycodestyle.
So the reverse flag should have the property that if I reorder a FermionOperator twice in a row with the same parameters, except once with reverse=False and once with reverse=True, then I should get the same thing back? Can you add a test for this for the up_then_down_map?

@kevinsung thanks for the review! - I caught a bug putting in the reverse test.

Thanks @conta877 ! It turns out we did have that test (your pycharm is not crazy) but it passed due to namespace pollution from the * imports, which will be fixed in another PR. Anyway I think this is ready to be merged.


Packages should use dependencies instead of peerDependencies

Expected Behavior
We were hoping that peerDependencies could be used as a way to signal which dependencies should be shared with other packages. Even to the point where we inline the regular dependencies during the rollup build.
Current Behavior
Peer dependencies don't work well with yarn workspaces, transitive dependencies in the workspace don't propagate properly. For example, if a plugin has a peer dependency on react and the app depends on the plugin, yarn will complain about a missing peer dependency, even if the app depends on react.
Possible Solution
Skip using peerDependencies for now, make all packages just use dependencies instead. They pretty much behave the same anyway atm.
Context
Initially brought up in #576, and partially fixed in #581.

I can take this one on 👍

Fixed by #586


gl_PointSize issues on M1 Mac

First, point sizes seem to be somehow quantized, maybe they behave like integers. When using geometric zoom, the sizes jump abruptly, which looks quite annoying. The phenomenon is also visible when point size is mapped to a positional channel – annoying bands can be seen.
Second, the maximum size for points seems to be rather small. The ALIASED_POINT_SIZE_RANGE is from zero to 64. Points larger than that needs to be drawn rarely, but there's still a limit, anyway.
(Third, the max point size on WebGPU is always one.)
Not sure how to workaround these. The most comprehensive solution would be to use quads instead of points. It just wastes 6x memory and makes geometry building 6x slower. Instancing is one option, but its performance with such small geometries (just six vertices) is poor. Some GPGPU trick could also work – expand the geometries from a single point to six vertices using the GPU. Then there's also the option of vertex pulling, which could also be useful with other geometries, such as rects and text.

This issue seems to be fixed in Chrome Canary with the Metal ANGLE backend.


iOS 16 weird behavior

Long press is required to trigger an animation, tapping doesn’t work anymore. Does anyone know how to fix this? It was fine on iOS 15

same on iOS 17 :/


PhysicalFileResult should allow not rooted paths

We have a check in PhysicalFileResult that causes us to throw for non rooted paths. We should instead let the underlying filesystem \ working directory determine this without performing any checks.

I believe we did this because the concept of a relative path is completely different between *nix and Windows. Whereas on Windows it's perhaps a bit easier to know what's a relative path and what isn't, on *nix they can look identical. We decided to avoid any confusion and just require/assume that the path provided is rooted.

Could we change it to not throw a FileNotFoundException? Maybe a NotSupportedException or the ever popuar InvalidOperationException? It was confusing since I thought I had the path specified incorrectly in my sample

The question is, how would we know, in an OS-independent way? The idea for what we have currently is to say in the doc comment what you need to pass in, assume you got it right, and then let the underlying implementation from BCL talk to the OS and do whatever it wants.


Merge 5-29 updates to live

Docs Build status updates of commit 8116c9b:
:clock10: Pending: waiting for processors (5 builds ahead of you)
:warning: Docs Build is busy, currently there are 5 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit 8116c9b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Language/Concepts/Getting-Started/deftype-statements.md
:white_check_mark:Succeeded
View

api/Excel.Chart.ExportAsFixedFormat.md
:white_check_mark:Succeeded
View

api/Excel.Chart.Previous.md
:white_check_mark:Succeeded
View

api/Excel.PivotTable.PivotTableWizard.md
:white_check_mark:Succeeded
View

api/Excel.QueryTables.Add.md
:white_check_mark:Succeeded
View

api/Excel.Range.Previous.md
:white_check_mark:Succeeded
View

api/Excel.Worksheet.Activate(even).md
:white_check_mark:Succeeded
View

api/Excel.Worksheet.Activate(method).md
:white_check_mark:Succeeded
View

api/Excel.Worksheet.Application.md
:white_check_mark:Succeeded
View

api/Excel.Worksheet.AutoFilter.md
:white_check_mark:Succeeded
View

api/Excel.Worksheet.AutoFilterMode.md
:white_check_mark:Succeeded
View

api/Excel.Worksheet.BeforeDoubleClick.md
:white_check_mark:Succeeded
View

api/Excel.Worksheet.BeforeRightClick.md
:white_check_mark:Succeeded
View

api/Excel.Worksheet.Calculate(even).md
:white_check_mark:Succeeded
View

api/Excel.Worksheet.Calculate(method).md
:white_check_mark:Succeeded
View

api/Excel.Worksheet.Cells.md
:white_check_mark:Succeeded
View

api/Excel.Worksheet.Change.md
:white_check_mark:Succeeded
View

api/Excel.Worksheet.ChartObjects.md
:white_check_mark:Succeeded
View

api/Excel.Worksheet.CheckSpelling.md
:white_check_mark:Succeeded
View

api/Excel.Worksheet.CircleInvalid.md
:white_check_mark:Succeeded
View

api/Excel.Worksheet.CircularReference.md
:white_check_mark:Succeeded
View

api/Excel.Worksheet.ClearArrows.md
:white_check_mark:Succeeded
View

api/Excel.Worksheet.ClearCircles.md
:white_check_mark:Succeeded
View

api/Excel.Worksheet.CodeName.md
:white_check_mark:Succeeded
View

api/Excel.Worksheet.Columns.md
:white_check_mark:Succeeded
View

api/Excel.Worksheet.ConsolidationFunction.md
:white_check_mark:Succeeded
View

api/Excel.Worksheet.ConsolidationOptions.md
:white_check_mark:Succeeded
View

api/Excel.Worksheet.ConsolidationSources.md
:white_check_mark:Succeeded
View

api/Excel.Worksheet.Copy.md
:white_check_mark:Succeeded
View

api/Excel.Worksheet.Creator.md
:white_check_mark:Succeeded
View

api/Excel.Worksheet.CustomProperties.md
:white_check_mark:Succeeded
View

api/Excel.Worksheet.Deactivate.md
:white_check_mark:Succeeded
View

api/Excel.Worksheet.Delete.md
:white_check_mark:Succeeded
View

api/Excel.Worksheet.DisplayPageBreaks.md
:white_check_mark:Succeeded
View

api/Excel.Worksheet.DisplayRightToLeft.md
:white_check_mark:Succeeded
View

api/Excel.Worksheet.EnableAutoFilter.md
:white_check_mark:Succeeded
View

api/Excel.Worksheet.EnableCalculation.md
:white_check_mark:Succeeded
View

api/Excel.Worksheet.EnableFormatConditionsCalculation.md
:white_check_mark:Succeeded
View

api/Excel.Worksheet.EnableOutlining.md
:white_check_mark:Succeeded
View

api/Excel.Worksheet.EnablePivotTable.md
:white_check_mark:Succeeded
View

api/Excel.Worksheet.EnableSelection.md
:white_check_mark:Succeeded
View

api/Excel.Worksheet.Evaluate.md
:white_check_mark:Succeeded
View

api/Excel.Worksheet.ExportAsFixedFormat.md
:white_check_mark:Succeeded
View

api/Excel.Worksheet.FilterMode.md
:white_check_mark:Succeeded
View

api/Excel.Worksheet.FollowHyperlink.md
:white_check_mark:Succeeded
View

api/Excel.Worksheet.HPageBreaks.md
:white_check_mark:Succeeded
View

api/Excel.Worksheet.Hyperlinks.md
:white_check_mark:Succeeded
View

api/Excel.Worksheet.Index.md
:white_check_mark:Succeeded
View

api/Excel.Worksheet.ListObjects.md
:white_check_mark:Succeeded
View

api/Excel.Worksheet.MailEnvelope.md
:white_check_mark:Succeeded
View

This comment lists only the first 50 files in the pull request.
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Help with command to replace CtrlP in VIM

I have been trying to move to FZF, but cannot figure out the command.  For CtrlP, I have the following:
let g:ctrlp_user_command = 'ag --ignore={build,.git,.project,*.o,*.d,hw_1_5/*} %s -l --hidden -g ""'

How would I define a VIM command to replace the CtrlP?

See https://github.com/junegunn/fzf#environment-variables

I am looking for something that can be placed into project specific .vimrc.  Setting the environment variables will change the behavior for all VIM projects and terminal, which is not desirable.

Define it in your .vimrc, you can.

I tried this:
let g:FZF_DEFAULT_COMMAND = 'ag --ignore={build,.git,.project,*.o,*.d,hw_1_5/*} %s -l --hidden -g ""'
but it does not seem to work for me...

No more help kind sir?

See :help :let-environment

It would be nice if there was a wiki entry for this topic. I was just looking at fzf as a replacement for ctrlp.vim. With Ctrl-P I just had to do this:
Plug 'ctrlpvim/ctrlp.vim'

set wildignore+=**/node_modules/**,**/package-lock.json


WIP [#124720541] Add grafana.log_to_syslog property

When set we tee the logs to systlog using logger.

We will do a PR upstream to get feedback


GH-35205: [C++][Gandiva] Don't find system Zstandard when we use bundled one

Rationale for this change
If we use bundled Zstandared, we should not find system Zstandard for Gandiva.
Because it's always failed.
What changes are included in this PR?
Add a missing check whether we're using system Zstandard or bundled Zstandard.
Are these changes tested?
Yes.
Are there any user-facing changes?
Yes.

Closes: #35205

+1


[Feature] Toggle for getting back to user's default location.

Fixes #60
Explanation :
On clicking the button it takes the user back to its current location.
Screenshot of the UI:

@JyotimoyKashyap conflicts ^

@JyotimoyKashyap conflicts ^

Will verify and comment if its safe to merge this pull request once tested.

Good approach 👍

Final conflicts to resolve @JyotimoyKashyap

@yashika51 this PR is ready to be merged.


Reorganise main function

Introduces urfave/cli which is one of the most popular CLI packages
Extracts all of the business logic outside of the main package
Simplifies flags creation
Example usages:

adinov@AtanasMac: edge-image-builder-fork [cli-exp]× » ./eib                                                                                [21:32:41]
NAME:
   eib - Edge Image Builder

USAGE:
   eib [global options] command [command options]

COMMANDS:
   build    Build new image
   help, h  Shows a list of commands or help for one command

GLOBAL OPTIONS:
   --help, -h  show help

adinov@AtanasMac: edge-image-builder-fork [cli-exp]× » ./eib build -h                                                                                [21:32:44]
NAME:
   eib build - Build new image

USAGE:
   eib build [OPTIONS]

OPTIONS:
   --config-file value  Name of the image definition file
   --config-dir value   Full path to the image configuration directory
   --build-dir value    Full path to the directory to store build artifacts
   --validate           If specified, the image definition will be validated but not built (default: false)
   --help, -h           show help

I love a lot of this. It makes me infinitely more comfortable that we're correctly using -- before arguments with greater than 1 character. Moving a lot of this stuff under pkg is also a huge step.
We spoke about it in chat, but for completeness, the next step (not this PR) will be to refactor validate into its own subcommand. That will not only make more sense and look cleaner, but it further justifies the inclusion of a subcommand model.


Added support for corporate proxy variables.

Hi,
my company has a corporate proxy. rancher-cli runs behind that. Our rancher 2 server instance runs in the public internet.
We want the rancher-cli to connect to the rancher 2 server.
Our corporate proxy prevents rancher-cli from connecting to the internet. The http_proxy, ... variables are ignored.
I added support for environment variables to all calls of
   http.Transport(...)

Now it works.
Accept this PR, please.
Thanks a lot and best regards,
Josef

Is this still relevant? Even though it's old, seems like it could be.
I'm going to close due to age, but feel free to reopen if you think it's still relevant. Thanks!

@cjellick I'm now coming across this limitation myself. If I lodge an updated PR is it likely to get merged? (The code is now out of date).


Change order dropdown menu "Profiles"

move up "View Profiles" first and below that "Add Profiles"

I changed the order of the "Profiles" drop down menu so that "View Profiles" is before "Add Profiles"


Idea: Typing numbers more fluently using the sectors

Am I the only one hating it having to switch to the numbers pad to type numbers?
I thought about a solution quite a while. It might be cool to use the numbers pad switch sector as a starting point for typing numbers.
Possible movements could be...

Any thoughts about that?

I like the idea, let other users chime in.
On Sat, 19 Feb 2022, 16:11 lxne, @.***> wrote:

Am I the only one hating it having to switch to the numbers pad to type
numbers?
I thought about a solution quite a while. It might be cool to use the
numbers pad switch sector as a starting point for typing numbers.
Possible movements could be...
[image: Screenshot_20220219-111732~2]
https://user-images.githubusercontent.com/13134654/154797326-a4de6264-bcc1-4047-9867-636fb8abae1c.png
[image: Screenshot_20220219-111732~3]
https://user-images.githubusercontent.com/13134654/154797337-97a61687-007f-456e-a06a-32a410cbf9fd.png
[image: Screenshot_20220219-111732~4]
https://user-images.githubusercontent.com/13134654/154797342-ed1b2122-7e81-4146-b2f9-5b112098cf48.png
[image: Screenshot_20220219-111732~5]
https://user-images.githubusercontent.com/13134654/154797348-1f8dd62f-d669-48d6-8077-dc6f2088bceb.png
[image: Screenshot_20220219-111732~6]
https://user-images.githubusercontent.com/13134654/154797354-e736a038-49e4-4500-a4fe-e1a72fbe4973.png
[image: Screenshot_20220219-111732~7]
https://user-images.githubusercontent.com/13134654/154797357-44a99777-347f-412c-8c69-1acf55c8394f.png
[image: Screenshot_20220219-111732~8]
https://user-images.githubusercontent.com/13134654/154797361-25405fe1-f1f4-4e2d-ae48-8fbf3ac09d8e.png
[image: Screenshot_20220219-111732~9]
https://user-images.githubusercontent.com/13134654/154797363-a54e0aaa-f2b9-4423-ac7c-7d0cd9e6691b.png
[image: Screenshot_20220219-111732~10]
https://user-images.githubusercontent.com/13134654/154797366-16c0ce8d-fd47-499f-8ea1-f32304880344.png
[image: Screenshot_20220219-111732~11]
https://user-images.githubusercontent.com/13134654/154797369-41ae49c6-f4c9-48b1-873a-888a2af4ba65.png
Any thoughts about that?
—
Reply to this email directly, view it on GitHub
https://github.com/flide/8VIM/issues/253, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABX5WR43I6NZBO3QIGFNCTLU35XUBANCNFSM5O2KZEQA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

number 5 and 7 have the same gesture

number 5 and 7 have the same gesture: maybe 8 could end INSIDE_CIRCLE and 5 could be the 8 gesture? (specular to 2).

Thank you. I wanted to keep them quite short.
And obviously didn't realize 5 and 7 were the same.
But 7 would even be more simple just to the center and down.

i like this idea

I love this idea and would love to implement it. We need to agree on the gestures for each number.
Following points need to be kept in mind while looking for suitable gestures:

second movement area can not be 'inside_circle', as this is going to end up clashing with same existing or intended fast access action, for example suggested number 5 is left->circle->right->bottom, but since left->circle is mapped to paste functionality, this 5 is simply not gonna fly.
A gesture can not be a subset of another gesture (existing or new). For example the suggested 1 is a subset of 2 in terms of movement, so we would never be able to print 2.

Nothing else comes to mind right now, so we should be good if we can find easy to remember & easy to associate with the number kinda gestures for all 10 digits.

I like the idea too! And that's very true when you get used to movements it's annoying to use the number keyboard (and emoji, but that's another subject ;))
Although as the 2 points above seem quite hard to resolve with the proposed design (and as it's not a very 8vim approach to "exactly" reproduce human writing I believe?), here is another idea:
What about switching to a simple modified 8vim keyboard for numbers only?
For example we could activate it doing:

And use it like classic 8vim keyboard but with one more line on the top (and only one line to cross per number):

(As you can see I'm not a graphic designer at all though :smile:)
And then you can come back to classic keyboard with the same movement as the beginning.
That's just another suggestion but there might be better ways!
As a plus side effect it should be very quick to type binary code for those interested :rofl:

Following is what I have come up with, gestures start at the green dot.
Note: I am not very happy with '9', but this is the best I can come up with.
0

1

2

3

4

5

6

7

8

9

If I read correctly the source code, the gesture for 3 is already used for }
/8vim/src/main/res/raw/special_core_gestures.xml
<keyboardAction>
            <keyboardActionType>INPUT_TEXT</keyboardActionType>
            <movementSequence>NO_TOUCH;TOP;RIGHT;INSIDE_CIRCLE;RIGHT;BOTTOM;NO_TOUCH;</movementSequence>
            <inputString>}</inputString>
            <inputCapsLockString>></inputCapsLockString>
        </keyboardAction>

valid point, solution is required

I don't know if this is still an idea that might be done, the alternate keyboard design thing in issue #68 makes number input a lot easier. Is it planned to do something like that or is the idea scrapped or something?
Here's a link to the Figma prototype posted towards the end of the issue
https://www.figma.com/proto/1H4Sw8099Q556YtkyRP62x/Keyboard?node-id=3%3A1&scaling=contain

that's a whole different keyboard, 8vim can't morph into that.
On Mon, 7 Mar 2022, 20:56 uShldGetCeleste, @.***> wrote:

I don't know if this is still an idea that might be done, the alternate
keyboard design thing in issue #68
https://github.com/flide/8VIM/issues/68 makes number input a lot
easier. Is it planned to do something like that or is the idea scrapped or
something?
Here's a link to the Figma prototype posted towards the end of the issue
https://www.figma.com/proto/1H4Sw8099Q556YtkyRP62x/Keyboard?node-id=3%3A1&scaling=contain
—
Reply to this email directly, view it on GitHub
https://github.com/flide/8VIM/issues/253#issuecomment-1060809295, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABX5WRZJ6LWF4NXH3I5L4IDU6YN2VANCNFSM5O2KZEQA
.
You are receiving this because you commented.Message ID:
@.***>

I like the proposition "Numbers & Symbols" from https://github.com/flide/8VIM/issues/68#issuecomment-775749834 though, couldn't it work with current 8vim?

Personally, I agree and would be happy to see more of some of what @Jogrefer was saying. That approach which @Paperweb has just mentioned would be excellent. I am not opposed to the number gestures starting from the keypad, but I think I would prefer this approach that also integrates symbols. I don't like having to open a weird menu where I am not typing with gestures. That is not the reason I use this keyboard. I know this issue is about numbers, but want to voice how inspiring those designs are. All of the ones in jogrefer's long post in 68.

Earlier today I accidentally typed an ellipsis and I had no idea how, nor did I know where to find information on how it could be done. Am I missing something, or does this need to be addressed?

Same thing happened to me, I've just looked at the layout XMLs and I don't can't find what movement it is!
Although by doing that I noticed that some basic emojis can be typed without an external keyboard, such a relief!!
And also that few movements could find better use, for example the one pasting your email @flide 😆 (might be a good idea to let the user cuscomize this one for example?)

I recommend that whatever you do the keyboard remains mostly self-documenting

Actually it might be a good idea to add some kind of auto-generated documentation depending on the layout used, by simply parsing the XMLs and showing movements for each possible output. But I'm not sure that's very easy to implement, specially the UI/UX side...
Sorry we finda deviated a lot from the original subject, but I feel like this is still a constructive discution to make this app better :)

Awesome I am glad we are thinking in the same direction on that one. To finish clarifying some of my thoughts, which is related to both the new issue #267 and this one:
I don't think that continuing to add gestures for any characters is a good idea unless we are going to lean in the direction of gestures being customizable. I will share ideas for this in the other issue. But as for this issue, @flide I will use your drawing of number 7 as an example. If you are going to show people how to draw something, then you will want to demo by drawing through the center of each section that you draw. Otherwise you will be showing people a way to perform a gesture in a way that they are more likely to make a mistake (as I understand anyway). If you were to draw the 7 going through the center of each point, it would actually more closely resemble a question mark than a number 7, which is an entirely legitimate item to desire a gesture for, but there are only so many possible gesture movements. Which brings us back to what @bonswouar said about being able to customize gestures.
Final note: 2 very different approaches to typing numbers more fluently have been discussed here. One involves switching to alternate layouts, and one involves specific gestures.
These options are not mutually exclusive. But as far as I can tell, having a layout toggle for numbers and special characters that allows you to continue typing all of them is the MVP here, and it happens to accomplish a lot too.

These options are not mutually exclusive. But as far as I can tell, having a layout toggle for numbers and special characters that allows you to continue typing all of them is the MVP here, and it happens to accomplish a lot too.

I second that! Plus that would prevent any conflict with emojis or other special gestures.
It seems most of us like the number & special char layouts proposed in https://github.com/flide/8VIM/issues/68#issuecomment-775749834 , should we go with it?!

These options are not mutually exclusive. But as far as I can tell, having a layout toggle for numbers and special characters that allows you to continue typing all of them is the MVP here, and it happens to accomplish a lot too.

I second that! Plus that would prevent any conflict with emojis or other special gestures.
It seems most of us like the number & special char layouts proposed in #68 (comment) , should we go with it?!

I absolutely like the idea of having a layout toggle for numbers and special characters. What I would suggest is that you should be able to switch to that layout temporarily while writing to not break the flow. This is where our second thumb could join the effort.
So if you touch the toggle button while not writing (one finger gesture) it switches to the numbers & special char layout until you again touch the button. But if you are writing and you touch the button with your other thumb (or any other finger) the layout switches to numbers and special chars only for as long as you touch the button (two finger gestures).
I hope this makes sense :)


nx_packet_allocated return false for RISCV-64 bits arch

Hello,
I am encountering an issue when I used nx_packet_allocate because ULONG is considered to be 8 bytes for the RISCV-64 bits architecture.
It worked perfectly on my previous target CORTEX-R4F.
Any idea ?

In ThreadX, ULONG (and LONG) should be 32 bits. On other 64-bit processors ULONG is a defined to unsigned int, but not for RISCV64.
This is a bug in ThreadX, but I don't have access to a working RISCV-64 system to check that making this change doesn't cause any other issues.
Are you able to test this?
You need to change the definition of LONG and ULONG in tx_port.h for the riscv-64 port
https://github.com/eclipse-threadx/threadx/blob/485a02faec6edccef14812ddce6844af1d7d2eef/ports/risc-v64/gnu/inc/tx_port.h#L103
to match the correct implementation for a 64-bit processor:
https://github.com/eclipse-threadx/threadx/blob/485a02faec6edccef14812ddce6844af1d7d2eef/ports/cortex_a77/gnu/inc/tx_port.h#L86


Unable to use PyTzInfo

Question / Clarification
🌍 Environment

Your operating system and version: Ubuntu 21.04
Your python version:3.9.5
How did you install python (e.g. apt or pyenv)? Did you use a virtualenv?: system
Your Rust version (rustc --version): rustc 1.52.1 (9bc8c42bb 2021-05-09)
Your PyO3 version: 0.13.2
Have you tried using latest PyO3 main (replace version = "0.x.y" with git = "https://github.com/PyO3/pyo3")?: no

Details
As described at https://github.com/PyO3/pyo3/issues/884#issuecomment-622774188 and in tests here, I assumed should be able to "subclass" PyTzInfo and use it in PyDateTime::new, but I'm getting
error[E0308]: mismatched types
  --> src/lib.rs:84:22
   |
84 |                 Some(&tz_info)
   |                      ^^^^^^^^ expected struct `Py`, found struct `TzClass`
   |
   = note: expected reference `&Py<PyAny>`
              found reference `&TzClass`

What am I doing wrong?
partial code:
fn chrono_py_err(e: ParseError) -> PyErr {
    PyErr::new::<PyValueError, _>(PyValueError::new_err(e.to_string()))
}

#[pyclass(extends=PyTzInfo)]
struct TzClass {
    seconds: i32
}

#[pymethods]
impl TzClass {
    #[new]
    fn new(seconds: i32) -> Self {
        TzClass {seconds}
    }

    fn utcoffset<'p>(&self, py: Python<'p>, _dt: &PyDateTime) -> PyResult<&'p PyDelta> {
        PyDelta::new(py, 0, self.seconds, 0, true)
    }

    fn tzname(&self, _py: Python<'_>, _dt: &PyDateTime) -> String {
        String::from("+01:00")  // TODO
    }

    fn dst(&self, _py: Python<'_>, _dt: &PyDateTime) -> Option<&PyDelta> {
        None
    }
}

...
            let offset_seconds = dt.offset().local_minus_utc();
            let tz_info = TzClass::new(offset_seconds);
            let dt = PyDateTime::new(
                py,
                ...
                Some(&tz_info)
            )?;

Full example of my (very naive) attempt at solving this: https://github.com/samuelcolvin/rtoml/pull/25

Indeed; this is a similar problem to the changes being looked at in #1588.
In particular, there's two related pain points:

PyDateTime::new takes Option<&Py<PyAny>> rather than Option<&PyTzInfo>.
&PyCell<TzClass> implements Deref<Target = PyAny>; I think it makes sense for it to implement Deref<Target = PyTzInfo> (i.e. #[pyclass] should deref to the super type rather than directly to PyAny)

For now you can work around this by using Some(&tz_info.into_py(py)) instead of Some(&tz_info). Let's keep this issue open as I'm very tempted to improve the situation for either 0.14 or 0.15.

Humm, I've tried that but I'm getting
error[E0599]: the method `into_py` exists for struct `TzClass`, but its trait bounds were not satisfied
  --> src/lib.rs:83:31
   |
22 | struct TzClass {
   | --------------
   | |
   | method `into_py` not found for this
   | doesn't satisfy `TzClass: AsPyPointer`
...
83 |                 Some(&tz_info.into_py(py)),
   |                               ^^^^^^^ method cannot be called on `TzClass` due to unsatisfied trait bounds
   |
   = note: the following trait bounds were not satisfied:
           `TzClass: AsPyPointer`
           which is required by `&TzClass: pyo3::IntoPy<Py<PyAny>>`

I've updated the above PR with your suggestion in case seeing the error in CI helps, but I have no idea how to go about fixing this.

Unngh, sorry. How about Some(&Py::new(py, tz_info)?.into())?
The APIs on Rust <-> Python interface boundary need some love when even I'm getting them wrong 🤦♂

Nope, I'm getting
error[E0277]: the trait bound `TzClass: AsRef<PyAny>` is not satisfied
  --> src/lib.rs:96:45
   |
96 |                 Some(&Py::new(py, tz_info)?.into()),
   |                                             ^^^^ the trait `AsRef<PyAny>` is not implemented for `TzClass`
   |
   = note: required because of the requirements on the impl of `From<pyo3::Py<TzClass>>` for `pyo3::Py<PyAny>`
   = note: required because of the requirements on the impl of `Into<pyo3::Py<PyAny>>` for `pyo3::Py<TzClass>`

error: aborting due to previous error

What is working if it helps, is the following extremely ugly workaround:
let locals = PyDict::new(py);
locals.set_item("datetime", py.import("datetime")?)?;
locals.set_item("seconds", offset_seconds.to_object(py))?;
let code = "datetime.timezone(datetime.timedelta(seconds=seconds))";
let tz_info = py.eval(code, None, Some(&locals))?;

Then Some(&tz_info.into_py(py)) as you suggested before.
But obviously a solution without dropping back to python would be good (not yet sure how slow the above is).

Ok, last try: Some(&Py::new(py, tz_info)?.to_object(py))
... I think that at least one of the other methods not working is a design flaw. Sorry that you've had to suffer this.

Okay, that works, thanks so much.
(at least it wasn't me missing something obvious)

#1588 changed Datetime::new to take Option<&PyTzInfo>, so I'm going to close this issue as resolved.


ecal_sys_gui fails with Shared object "libsys_core.so" not found, required by "ecal_sys_gui"

Describe the bug
$ ecal_sys_gui
ld-elf.so.1: Shared object "libsys_core.so" not found, required by "ecal_sys_gui"

libsys_core.so is built but not installed:
$ find . -name "libsys_core*.so"
./work/.build/lib/libsys_core.so

CMAKE_ARGS=-DECAL_BUILD_VERSION=5.10.0  -DGIT_REVISION_MAJOR=5  -DGIT_REVISION_MINOR=10  -DGIT_REVISION_PATCH=0  -Dasio_INCLUDE_DIR=/usr/local/include  -Dtclap_INCLUDE_DIR=/usr/local/include -DCMAKE_C_COMPILER:STRING="cc"  -DCMAKE_CXX_COMPILER:STRING="c++"  -DCMAKE_C_FLAGS:STRING="-O2 -pipe -fno-omit-frame-pointer  -fstack-protector-strong -fno-strict-aliasing "  -DCMAKE_C_FLAGS_DEBUG:STRING="-O2 -pipe -fno-omit-frame-pointer  -fstack-protector-strong -fno-strict-aliasing "  -DCMAKE_C_FLAGS_RELEASE:STRING="-O2 -pipe -fno-omit-frame-pointer  -fstack-protector-strong -fno-strict-aliasing "  -DCMAKE_CXX_FLAGS:STRING="-pthread "  -DCMAKE_CXX_FLAGS_DEBUG:STRING="-pthread "  -DCMAKE_CXX_FLAGS_RELEASE:STRING="-pthread "  -DCMAKE_EXE_LINKER_FLAGS:STRING=" -fstack-protector-strong "  -DCMAKE_MODULE_LINKER_FLAGS:STRING=" -fstack-protector-strong "  -DCMAKE_SHARED_LINKER_FLAGS:STRING=" -fstack-protector-strong "  -DCMAKE_INSTALL_PREFIX:PATH="/usr/local"  -DCMAKE_BUILD_TYPE:STRING="Release"  -DTHREADS_HAVE_PTHREAD_ARG:BOOL=YES  -DCMAKE_INSTALL_RPATH_USE_LINK_PATH=YES  -DCMAKE_VERBOSE_MAKEFILE:BOOL=ON -DBUILD_SHARED_LIBS:BOOL=ON -DBUILD_SAMPLES:BOOL=OFF -DECAL_THIRDPARTY_BUILD_PROTOBUF:BOOL=OFF -DECAL_THIRDPARTY_BUILD_SPDLOG:BOOL=OFF -DECAL_THIRDPARTY_BUILD_TINYXML2:BOOL=OFF -DECAL_THIRDPARTY_BUILD_FINEFTP:BOOL=OFF -DECAL_THIRDPARTY_BUILD_TERMCOLOR:BOOL=OFF -DECAL_THIRDPARTY_BUILD_CURL:BOOL=OFF -DECAL_THIRDPARTY_BUILD_GTEST:BOOL=OFF -DECAL_THIRDPARTY_BUILD_HDF5:BOOL=OFF -DCPACK_PACK_WITH_INNOSETUP:BOOL=OFF -GNinja

Then - same problem with libQEcalParser.so:
$ ecal_sys_gui
ld-elf.so.1: Shared object "libQEcalParser.so" not found, required by "ecal_sys_gui"

Version: 5.10.0-nightly
OS: FreeBSD 13

Hello @yurivict, can you try to switch BUILD_SHARED_LIBS to OFF ? Otherwise, all libraries will be created as shared objects, although they are not intended to be installed in the deployment. By default and independent from what is specified in BUILD_SHARED_LIBS, the ecal_core gets always built as shared library.

Closing due to inactivity


Look into the API mentioned by Richard in his e-mail response

I looked into the API that Richard mentioned and I believe it's quite promising.
Here's a rough overview of how I think this can be used:

RandstructAction is a PluginASTAction
RandstructConsumer is an ASTConsumer
Randstruct is an ExternalASTSource that overrides the layoutRecordType() callback. It's within this callback that we must perform the entire construction (alignment, padding, everything) of the record layout while randomizing its fields.
I've been working on a prototype, but am currently having trouble finding where to wire the Randstruct ExternalASTSource into the plugin architecture. If I do it in the constructor of the RandstructConsumer, nothing happens (at least, my print statements don't happen). If I do it in HandleTopLevelDecl(), the compiler seg faults.


exploitdb: 2022-02-09 -> 2022-02-11

Motivation for this change
update to 2022-02-11.
Takes over #159263.
Things done

Built on platform(s)

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

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

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[X] Fits CONTRIBUTING.md.

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

1 package built:

exploitdb

[nix-shell:~/.cache/nixpkgs-review/pr-159273]$ ./results/exploitdb/bin/searchsploit Cain\ \&\ Abel 4.9.56
[i] Found (#1): /nix/store/2bs5n6kp9k3h554hxiyw9zv9h3n1jk1y-exploitdb-2022-02-11/share/exploitdb/files_exploits.csv
[i] To remove this message, please edit "/nix/store/2bs5n6kp9k3h554hxiyw9zv9h3n1jk1y-exploitdb-2022-02-11/share/exploitdb/.searchsploit_rc" for "files_exploits.csv" (package_array: exploitdb)

[i] Found (#1): /nix/store/2bs5n6kp9k3h554hxiyw9zv9h3n1jk1y-exploitdb-2022-02-11/share/exploitdb/files_shellcodes.csv
[i] To remove this message, please edit "/nix/store/2bs5n6kp9k3h554hxiyw9zv9h3n1jk1y-exploitdb-2022-02-11/share/exploitdb/.searchsploit_rc" for "files_shellcodes.csv" (package_array: exploitdb)

------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------ ---------------------------------
 Exploit Title                                                                                |  Path
------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------ ---------------------------------
Cain & Abel 4.9.56 - Unquoted Service Path                                                                                | windows/local/50728.txt
------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------ ---------------------------------
Shellcodes: No Results


fix(Marker): allow null and undefined as children

This is useful if you have code like the following:
<Marker {...props}>
  { showInfo &&
    <InfoWindow content="Some Info" />
  }
</Marker>

Right now, this will throw an Error about being unable to read props of undefined, so you have to use the following workaround (not ideal):
// workaround:
<Marker {...props}>
  { showInfo ?
    <InfoWindow content="Some Info" />
    : <noscript />
  }
</Marker>

With this change, the workaround is no longer needed and the first example works as expected :smile:

Thanks. Released v2.2.0


Add maturin workflow to generate wheels

@reyammer So the conclusion is that ort needs GLIBC >= 2.31 for pre-built binaries while the latest manylinux is 2_28. This means that we would need to compile ONNX Runtime from source and link it when building Magika, see https://github.com/pykeio/ort/issues/268 and https://ort.pyke.io/setup/platforms and https://ort.pyke.io/setup/linking. (That's assuming compiling ONNX Runtime from source with GLIBC 2.28 will work.)

Reopening this to test compiling ONNX Runtime from source.

Here are the wheels for examination.

Here are the wheels. I checked the linux one and the versions (in wheel name, Rust CLI, and Python version) are correct. Not sure what else to check. The workflow anyway checks even more.

About the wheels: yesterday I checked the linux one (but on a modern box) and the mac ones, and they worked! will add tests for the old ubuntu in a different PR. Other than this, this looks good!

About the wheels: yesterday I checked the linux one (but on a modern box) and the mac ones, and they worked! will add tests for the old ubuntu in a different PR. Other than this, this looks good!

The only missing test is the wheel test in the container (thus for Linux). We already test everything else (including the CLI in the container). If you believe the wheel and python code could behave differently in the container, then it's worth looking into, otherwise I believe this PR should be sufficient.

I checked the linux one and the versions (in wheel name, Rust CLI, and Python version) are correct.

I should also have checked sdist because it's a different flow. Sending #769 to fix the wheel version.


doc: drivers: deprecate driver init levels

Remove deprecated driver initialization levels in the docs.

Those levels are not deprecated.

Those are deprecated in this commit

Those are deprecated in this commit

initialization of devices in this level is deprecated, not the init level itself.

Those are deprecated in this commit

initialization of devices in this level is deprecated, not the init level itself.

This file is the documentation for the device driver model, and not the init system as a whole right?

This file is the documentation for the device driver model, and not the init system as a whole right?

afaik this is the only place init levels are documented in the docs.

This file is the documentation for the device driver model, and not the init system as a whole right?

afaik this is the only place init levels are documented in the docs.

The wording of the removed sentences and the page title make it clear that this content is meant for device drivers only. IMO, the lack of documentation for SYS_INIT() does not constitute the necessity of including deprecated APIs in driver inits.


ARM runner: so space left on device

building qt6-static fails at packaging:
-- Installing: C:/M/B/pkg/mingw-w64-clang-aarch64-qt6-static/clangarm64/qt6-static/bin/balsam.exe
-- Installing: C:/M/B/pkg/mingw-w64-clang-aarch64-qt6-static/clangarm64/qt6-static/bin/balsamui.exe
 CMake Error at BCLANGARM64/qtquick3d/tools/balsamui/cmake_install.cmake:42 (file):
   file INSTALL cannot copy file
   "C:/M/B/src/BCLANGARM64/qtbase/bin/balsamui.exe" to
   "C:/M/B/pkg/mingw-w64-clang-aarch64-qt6-static/clangarm64/qt6-static/bin/balsamui.exe":
   No space left on device.
 Call Stack (most recent call first):
   BCLANGARM64/qtquick3d/tools/cmake_install.cmake:47 (include)
   BCLANGARM64/qtquick3d/cmake_install.cmake:47 (include)
   BCLANGARM64/cmake_install.cmake:77 (include)

@jeremyd2019

I'm cleaning up some of the cruft on the runner (.cargo, .cache, go from userprofile), but I also plan to create a fresh runner today if it's idle.  I'd rather not make the drive bigger if I can help it.


Re-Opening of PR #1747

Credits for the commits goes to @ModernTTY
and I've made sure that ModernTTY is the Author of every commit he made, and the Date stays the same as well.
Only did a small contribution by removing a Trailing Tab in commit "Add autoruns"

Will close this PR, noticed something odd in the commits, will make a new PR for the same purpose.


form fiss 2023 QA workflow

Documenting how we QA the form fiss 2023 so others can understand the process. Hoping we can use this to update onboarding or https://github.com/NewGraphEnvironment/fish_passage_template_reporting at some point. This is based off my understanding while QA the data so please correct anything that is wrong.
While in the field, crews collected data on both the digital field forms and the fiss site cards, so it's okay if there are some data on the fiss site cards that aren't in the form, and vice versa.
The following changes were done to the skeena 2023 form fiss data and issues were not created for these changes because it happened very frequently. If an issue was created for other edits, then these changes were mentioned in the issue, see example here https://github.com/NewGraphEnvironment/fish_passage_skeena_2023_reporting/issues/78

Certain data are collected at only one ef site but need to be transferred to all other ef sites within the greater site. For example, temperature, ph, conductivity, and turbidity were only collected at 8525_ds_ef1 but this data will be used for all other 8525_ds and 8525_us sites. When QAing data in Q, we will copy temperature, ph, conductivity, and turbidity from 8525_ds_ef1 to all other 8525 sites. Sometimes crews will collect this data at one us site and one ds site, then we will copy this data to all us and ds sites respectively. The following data is often only collected at one site:

Temperature
pH
Conductivity
turbidity
stage

In the future we should script the above process using tidyr::fill() function. https://tidyr.tidyverse.org/reference/fill.html

Need to change dates once in R because we are having the same issue as here https://github.com/NewGraphEnvironment/fish_passage_skeena_2023_reporting/issues/49


Docs: Why is Debian Recommended for Docker?

https://frankenphp.dev/docs/docker/

FrankenPHP Docker images are based on official PHP images. Debian and Alpine Linux variants are provided for popular architectures. Debian variants are recommended.

The headline states that Debian is recommended. Documentation should provide clarification rather than just statements; therefore, an explanation or reference supporting this recommendation should be included.

My 2¢ and why I don't recommend musl/alpine in production:
Multiple reasons, mostly due to musl implementation details breaking something in Go/PHP. Also, fwiw, the 'official php' images are managed by the community and aren't maintained by php, nor does PHP have any unit tests running on alpine (https://github.com/php/php-src/actions/runs/10094146717 for example).
Secondly, if you search the PHP issues, musl bugs are fixed rather quickly, but only once they are reported. There isn't any proactive hunting for those bugs.

OH, and I almost forgot to mention the DNS shenanigans that happen on alpine php... if you haven't run into the "internet disappearing bug" yet, it's super confusing and was the straw that broke our back at work; when we moved to debian-only.
Personally, I wouldn't recommend it for production systems, just to shave a few dozen megabytes off an image size. If you absolutely need those megabytes, then go for it.

It's also worth noting that this probably won't always be the case. Rereading that it sounds like doom-and-gloom, but it isn't that bad. It works most of the time, but when it doesn't work, it can be very frustrating.

https://github.com/dunglas/frankenphp/issues/940
https://x.com/dunglas/status/1765738034782142699

Thanks @sneycampos for the link. For those who don't have access to X, I'll post the comments here:

Q: Is that FrankenPHP related, or do you recommend the Debian images for PHP even without FrankenPHP?
A: I recommend Debian with and without FrankenPHP! - @dunglas

Q: Do you have any performance numbers for using glibc?
A: No, but musl can prevent enabling the JIT: https://github.com/dunglas/symfony-docker/issues/555 - @dunglas

Interesting. I also had issues with iconv on the Alpine distribution.

The mentioned issue https://github.com/dunglas/symfony-docker/issues/555 mostly explains why the documentation recommends Debian. The performance issue discussed on Stack Overflow may also be relevant to some users.
@withinboredom's comment is also valuable.
Updating the docs or mentioning this issue could help others who encounter the same question why Debian is preferred. Since it's clear to me, I wouldn't mind closing this issue.

A docs PR would be very welcome!

just to shave a few dozen megabytes off an image size

160Mb for Alpine dunglas/frankenphp:php8-alpine
589MB for Debian dunglas/frankenphp:php8-bookworm

slightly more than "a few dozen" mega bytes, almost half a Gb! (429MB) difference - gulp

(429MB) difference - gulp

That's 3 extra seconds to download on a gbit connection, and you only have to download it once. On a 10gbit connection, that's less than half a second. Like I said, if you need that less space due to running on edge devices like a pi, then that is what you optimize for.

Not everyone has access to a 10gbit ...  sure data centers might, but developers on the end of dialup might not,  Developer experience counts too. I think sometimes we are so privileged with home/office internet speed in Europe/USA that we forget the rest of the world exists. And I say that as someone who lives on the island of Jersey, where we have the second fastest internet in the world, with fibre to the desk, but then when travelling (like today) Im on a hotel 2mbps connection...
And @sneycampos down thumbing me for stating the factual sizes of images from this project is just lame. I did not create the images, I just quoted their published sizes.
Anyway this is all off topic. The fact is using Debian is recommended for many good reasons, the trade-off is larger image sizes. Reducing Docker image sizes is crucial for streamlining development workflows, speeding up builds, and minimizing deployment times, all while saving valuable storage space. These are just the facts.

The main blocker for recommending Alpine images in production is https://github.com/php/php-src/issues/13648.
Laravel and many other tools on the hot path use this function.
Any help fixing this is very welcome.


Change Active Directory password using Kerberos Change-Password protocol and .NET Core 3

By reading Kerberos Change Password Protocol and Kerberos.NET Samples, I wrote the following code:
using Kerberos.NET;
using Kerberos.NET.Client;
using Kerberos.NET.Credentials;
using Kerberos.NET.Crypto;
using Kerberos.NET.Entities;
using System;
using System.Linq;
using System.Net.Sockets;
using System.Text;
using System.Threading.Tasks;

namespace KerberosDemo
{
    class Program
    {
        static async Task Main(string[] args)
        {
            string activeDirectoryServer = "<IP_ADDRESS>:88"; // kdc port
            string domain = "TEST.LOCAL";
            string username = "testuser@" + domain;
            string oldPassword = "123";
            string newPassword = "456";

            using var client = new KerberosClient(activeDirectoryServer);
            var kerbCred = new KerberosPasswordCredential(username, oldPassword, domain);
            await client.Authenticate(kerbCred);
            var serviceTicket = await client.GetServiceTicket("kadmin/changepw");

            var apReq = new KrbApReq
            {
                Ticket = serviceTicket.Ticket,
                Authenticator = KrbEncryptedData.Encrypt(Encoding.UTF8.GetBytes(newPassword).AsMemory(), kerbCred.CreateKey(), KeyUsage.ApReqAuthenticator),
                ProtocolVersionNumber = 1
            };

            var tcp = client.Transports.FirstOrDefault(t => t.Protocol == ProtocolType.Tcp);
            await tcp.SendMessage<KrbApReq, KrbAsRep>(domain, apReq);
        }
    }
}

I got the following exception at await tcp.SendMessage<KrbApReq, KrbAsRep>(domain, apReq);:

Unhandled exception.
System.Security.Cryptography.CryptographicException: ASN1 corrupted
data.    at
System.Security.Cryptography.Asn1.AsnReader.ReadTagAndLength(Nullable1& contentsLength, Int32& bytesRead) in d:\a\1\s\Kerberos.NET\Asn1\Experimental\AsnReader.cs:line 305    at Kerberos.NET.Entities.KrbError.CanDecode(ReadOnlyMemory1 encoded) in
d:\a\1\s\Kerberos.NET\Entities\Krb\KrbError.cs:line 23    at
Kerberos.NET.Transport.KerberosTransportBase.Decode[T](ReadOnlyMemory1 response) in d:\a\1\s\Kerberos.NET\Client\Transport\KerberosTransportBase.cs:line 41    at Kerberos.NET.Transport.TcpKerberosTransport.ReadResponse[T](NetworkStream stream, CancellationToken cancellation) in d:\a\1\s\Kerberos.NET\Client\Transport\TcpKerberosTransport.cs:line 69 at Kerberos.NET.Transport.TcpKerberosTransport.SendMessage[T](String domain, ReadOnlyMemory1 encoded, CancellationToken cancellation) in
d:\a\1\s\Kerberos.NET\Client\Transport\TcpKerberosTransport.cs:line 58
at KerberosDemo.Program.Main(String[] args)    at
KerberosDemo.Program.(String[] args)

Your help is appreciated. Thanks.
https://stackoverflow.com/questions/58670044/change-active-directory-password-using-kerberos-change-password-protocol-and-ne

There's a few things going on here. The message format doesn't appear to match what you're sending. You would need to build a new message that includes the length, ap-req, and krb-priv all in a single request.
The other thing is that it appears that you're sending the request to port 88 which is normal for all other messages, but changepw requires it go to 464.

From: Fouad Almalki<EMAIL_ADDRESS>Sent: Saturday, November 2, 2019 3:08:18 AM
To: SteveSyfuhs/Kerberos.NET<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [SteveSyfuhs/Kerberos.NET] Change Active Directory password using Kerberos Change-Password protocol and .NET Core 3 (#79)
By reading Kerberos Change Password Protocolhttps://tools.ietf.org/html/draft-ietf-cat-kerb-chg-password-02 and Kerberos.NET Sampleshttps://github.com/SteveSyfuhs/Kerberos.NET/tree/master/Samples, I wrote the following code:
using Kerberos.NET;
using Kerberos.NET.Client;
using Kerberos.NET.Credentials;
using Kerberos.NET.Crypto;
using Kerberos.NET.Entities;
using System;
using System.Linq;
using System.Net.Sockets;
using System.Text;
using System.Threading.Tasks;
namespace KerberosDemo
{
class Program
{
static async Task Main(string[] args)
{
string activeDirectoryServer = "<IP_ADDRESS>:88"; // kdc port
string domain = "TEST.LOCAL";
string username = "testuser@" + domain;
string oldPassword = "123";
string newPassword = "456";
        using var client = new KerberosClient(activeDirectoryServer);
        var kerbCred = new KerberosPasswordCredential(username, oldPassword, domain);
        await client.Authenticate(kerbCred);
        var serviceTicket = await client.GetServiceTicket("kadmin/changepw");

        var apReq = new KrbApReq
        {
            Ticket = serviceTicket.Ticket,
            Authenticator = KrbEncryptedData.Encrypt(Encoding.UTF8.GetBytes(newPassword).AsMemory(), kerbCred.CreateKey(), KeyUsage.ApReqAuthenticator),
            ProtocolVersionNumber = 1
        };

        var tcp = client.Transports.FirstOrDefault(t => t.Protocol == ProtocolType.Tcp);
        await tcp.SendMessage<KrbApReq, KrbAsRep>(domain, apReq);
    }
}

}
I got the following exception at await tcp.SendMessage<KrbApReq, KrbAsRep>(domain, apReq);:
Unhandled exception.
System.Security.Cryptography.CryptographicException: ASN1 corrupted
data. at
System.Security.Cryptography.Asn1.AsnReader.ReadTagAndLength(Nullable1& contentsLength, Int32& bytesRead) in d:\a\1\s\Kerberos.NET\Asn1\Experimental\AsnReader.cs:line 305 at Kerberos.NET.Entities.KrbError.CanDecode(ReadOnlyMemory1 encoded) in
d:\a\1\s\Kerberos.NET\Entities\Krb\KrbError.cs:line 23 at
Kerberos.NET.Transport.KerberosTransportBase.Decode[T](ReadOnlyMemory1 response) in d:\a\1\s\Kerberos.NET\Client\Transport\KerberosTransportBase.cs:line 41 at Kerberos.NET.Transport.TcpKerberosTransport.ReadResponse[T](NetworkStream stream, CancellationToken cancellation) in d:\a\1\s\Kerberos.NET\Client\Transport\TcpKerberosTransport.cs:line 69 at Kerberos.NET.Transport.TcpKerberosTransport.SendMessage[T](String domain, ReadOnlyMemory1 encoded, CancellationToken cancellation) in
d:\a\1\s\Kerberos.NET\Client\Transport\TcpKerberosTransport.cs:line 58
at KerberosDemo.Program.Main(String[] args) at
KerberosDemo.Program.
(String[] args)
Your help is appreciated. Thanks.
https://stackoverflow.com/questions/58670044/change-active-directory-password-using-kerberos-change-password-protocol-and-ne
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/SteveSyfuhs/Kerberos.NET/issues/79?email_source=notifications&email_token=AAJHTYIBR45PAMDUZ4ESJNLQRVGRFA5CNFSM4JIEPXY2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HWKBHMA, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAJHTYPXGTKAL4JYIGE3MGDQRVGRFANCNFSM4JIEPXYQ.


(PDB-3089) log stockpile enqueues at info level

Test PASSed

CLA signed by all contributors.

I'm not completely convinced that this is needed -- seems like it generates a fair amount of noise. If we want it though I think we should swap out the ids being logged in processing with the - string logged in enqueuing in order to make the connection clear.

Think we've handled this issue another way -- closing.


Pin av version to 11.0.0

hey @JadeCopet thanks for all your diligent work here, as always ❤ Any note on why you chose to pin to 11.0.0? Related to either of these?
https://github.com/PyAV-Org/PyAV/issues/1376
https://github.com/PyAV-Org/PyAV/issues/1378