Bugfix: prevent fieldsets being nested

This fixes an HTML issue, where the fieldsets were inadvertently (I think) nested. Nested fieldsets can be tricky to navigate for screen reader users.

Thanks! Can you include a screenshot or URL so I can see the change?

@slorek if you visit https://psd-pr-691.london.cloudapps.digital/cases/2006-0049/activity/new and add a "meeting", fill out the form and press Continue, you’ll get to the second page. Currently the fieldset with the "Related attachments" legend is nested within the fieldset with the "Details" legend. This PR separates them so they are no longer nested.
The visual appearance is the same.
(Spotted as part of work on #687)


Add opcode counter Inspector impl

ref https://github.com/paradigmxyz/reth/issues/6421
add a simple tracer that counts opcodes, very similar to the existing AccessList Inspector
@DoTheBestToGetTheBest do you want to take this?

yep taking this

#24


Decrease the size of the logo in 'Detailed project view'

(NO NEED TO HAVE THE PURPLE BACKGROUND, JUST DECREASE THE SIZE)

vs.

@adelablue Mock-up
https://www.dropbox.com/sh/ekoyuqi2tkggg6b/AADJJp4PCWFPy7DcMhR828sqa/Preview?dl=0


[TASK] Payouts and Nominations table data got cached when switching Chain but Nominations still works fine

Description
After switching chain, the Payouts table still displays data from the previous chain. This is because the data for the table is still stored in Local Storage. The data is cached for both Payouts and Nominations table but the Nominations table still has the correct behavior somehow.
Recording
https://github.com/webb-tools/webb-dapp/assets/69841784/25f8164b-9a50-404e-bb31-e83b7b967637

Closed since this will be covered in #2152


OKCJUG food for June 14

https://github.com/techlahoma/okcjug-planning/issues/12

<3
On Tue, Jun 14, 2016, 10:27 AM Amanda Harlin<EMAIL_ADDRESS>wrote:

Closed #105 https://github.com/techlahoma/techlahoma/issues/105.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/techlahoma/techlahoma/issues/105#event-692002527, or mute
the thread
https://github.com/notifications/unsubscribe/AACuD9i7GO7yPqxMhOn2yMMjBTYwAHbrks5qLshIgaJpZM4IwA7f
.


Your first pull request

But not your last!
This is how you'd review someone on your team changing the code on your project.
Let's review it when you get back here Friday!

OK!


KQL validation tests- support ASIM\lib\functions

Required items, please complete
Change(s):

See guidance below

Reason for Change(s):

See guidance below

Version Updated:

Required only for Detections/Analytic Rule templates
See guidance below

Testing Completed:

See guidance below

Checked that the validations are passing and have addressed any issues that are present:

See guidance below

Guidance <- remove section before submitting

Before submitting this PR please ensure that you have read the following sections and filled out the changes, reason for change and testing complete sections:
Thank you for your contribution to the Microsoft Sentinel Github repo.

Details of the code changes in your submitted PR.  Providing descriptions for pull requests ensures there is context to changes being made and greatly enhances the code review process.  Providing associated Issues that this resolves also easily connects the reason.

Change(s):

Updated syntax for XYZ.yaml

Reason for Change(s):

New schema used for XYZ.yaml
Resolves ISSUE #1234

Version updated:

Yes
Detections/Analytic Rule templates are required to have the version updated

The code should have been tested in a Microsoft Sentinel environment that does not have any custom parsers, functions or tables, so that you validate no incorrect syntax and execution functions properly.  If your submission requires a custom parser or function, it must be submitted with the PR.

Testing Completed:

Yes/No/Need Help

Note: If updating a detection, you must update the version field.

Before the submission has been made, please look at running the KQL and Yaml Validation Checks locally.
https://github.com/Azure/Azure-Sentinel#run-kql-validation-locally

Checked that the validations are passing and have addressed any issues that are present:

Yes/No/Need Help

Note: Let us know if you have tried fixing the validation error and need help.

References:

Guidance for Detection checks
General contribution guidance
PR validation troubleshooting

Hi @sivanguetta :

Do you want me to review the code itself?
The change in YAML structure would break my generation script, I think. Do you want me to update it?

Hi @sivanguetta :

Do you want me to review the code itself?
The change in YAML structure would break my generation script, I think. Do you want me to update it?

It would be great if you could review, I have asked Ori to review as well.
Let's discuss about the new structure in our meeting


Fix comment syntax inside jsx

What's the use case for this? Has this been requested?

@gabelevi isn't it more consistent for comment syntax to work everywhere?
This is workaround for providing generic type to component
function Component<T>(props: {|type: T|}) {}
const el = <Component /*::type={1}*/ />

@gabelevi I updated main post

I can't test it now, but I'm not sure if this would be allowed too?
</*::Test*/ />

@gabelevi I'm going to close it because I'm not really want to introduce some unexpected behaviour here with jsx, also I think this is not worth it


Change type name for cyfunctions and use it in repr

Shorten the type names:

cython_function_or_method -> cyfunction
fused_cython_function -> fused_cyfunction

And use these type names in the __repr__.

Ok, this looks nicer, and 3.0 is probably the time to change something like this, but … isn't it just deliberately breaking code that tests for the function "type" by name? (I wouldn't swear that there's no is such code out there.) I don't want people to start resorting to regular expressions for such a use case. (Actually, I'd rather not see such a use case in real life, but …)

Not "deliberately" but I see your point: this has some chance of breaking backwards compatibility.
Actually, the main thing in this PR is the change to __repr__: I wanted to see the difference between ordinary cyfunctions and fused functions. But if you're fine with accepting the change to __repr__ without changing the type names, that's fine for me too.


[ImageUploader] Write test to remove exceptions

Need to write unit test for this feature to bring code coverage up above thresholds.
This issue should raise coverage for ImageCard, ImageUploader, and DashboardEditor

Fixed upstream


Body tracking crashes due to driver error or dll not found

Describe
When I don't install the driver or lost some dll and to use the body tracking, it crash, nothing error info show me.
Feature
First start up and checking driver and dll or provide a function like k4abt_set_error_callback.

@yijiew can you comment here on the types of missing dependencies which emit log messages in 0.9.1 and which result in crashes?
@venscn this is an issue we are working on. Some issues (such as an out of date driver) do not have a good way for us to produce an error message since the crash is internal to the GPU libraries.

When CUDA dlls or Visual C++ Redistribution dlls are missing, the tracker creation will fail and error messages will be logged. But when the driver version is too low to support CUDA 10.0, there isn't a good way to log the error message as @cdedmonds mentioned. We will see what we can do to check the driver version.

Fixed in v0.9.5.


ModuleNotFoundError: No module named 'options.base_options'

Hi, Thanks for sharing great work!
I want to train your code.
However, I think there is some missing file.
Ex. options/base_options.py
Thanks!

Hi, Thanks for sharing great work!
I want to train your code. However, I think there is some missing file. Ex. options/base_options.py
Thanks!

Hi,
Thanks raising the concern. As author team, we've also noticed the missing file days ago. This might be due to gitignore mechanism when we upload the code. We are working on fixing it, please expect the code to be fully reproduceable soon.
Best.

Thank you for your quick response!
I understood the current situation.
Sincerely.

Hi,
We have updated the codes, and added the missing base_options.py (now it's in the root dir, not in 'options'.
Hope that solves your concern, and thanks for making our repo better!
Best.


Java Client Library for ROS 2

Hi Niels,
It is really good to see that you are working on the OpenTCS NeNa project and have updated your work for Foxy. I see that you have switched to IHMC Java communication for the ROS 2 connection, instead of RCL Java. Can you explain why you did this and what the consequences are?
Ps. Maybe you should consider renaming the project as it is becoming a general OpenTCS ROS 2 bridge.
Wilco

Hello Wilco,
Thank you for reaching out. Surely, I can elaborate on my decision to switch from RCLJava to IHMC:

RCLJava is not yet updated for ROS 2 Foxy. Experiments for using the latest RCLJava version (targeting ROS 2 Dashing) for Foxy have been done, but it did work due to small differences in several message types.
The IHMC is completely stand-alone and therefore not dependent on a local ROS 2 installation on your system. This makes it much easier to install the OpenTCS-NeNa software and eliminates the need to build the OpenTCS-NeNa software on your machine.
This also means that it is no longer needed to hard-code your ROS 2 workspace in the OpenTCS-NeNa build script.
Due to IHMC its independence, this software should now also work on other operating systems, such as Windows. However, this has not been tested.
The previous OpenTCS-NeNa release (1.1) used OpenTCS 4.17 which uses Java 8. The IHMC library used to be incompatible with Java 8.
This OpenTCS-NeNa release (2.0) uses OpenTCS 5.0 which uses Java 13, making it now compatible with the IHMC library.`
The biggest advantage of the IHMC library, its independence, is also its largest drawback.
Applications such as ROS2 RQT, will no longer recognize the OpenTCS node because it is now
seen as an external entity.

I will consider renaming this project on a next release. I agree with you that the software is in fact a general OpenTCS ROS 2 bridge.
Niels


Can't upload secondary place

In mantle.yml I have 2 places:
   places:
      start:
        file: GameLobby.rbxlx
        configuration:
          name: Monsters Nearby
          description: A game!
          maxPlayerCount: 20

      waiting:
        file: GameWaitingLobby.rbxlx
        configuration:
          name: Monsters Nearby Waiting Lobby

When running mantle deploy --environment development
I get error
Failed: Roblox error (404 Not Found): 404 | Page Not found
I think it is because Roblox deprecated http://data.roblox.com/Data/Upload.ashx
Used in

Blog post: https://devforum.roblox.com/t/official-list-of-deprecated-web-endpoints/62889/62
NB! This is an awesome project!

Yup I agree with your assessment. The latest release of mantle includes support for open cloud APIs for publishing places. Check out the updated authentication docs here: https://mantledeploy.vercel.app/docs/authentication#roblox-open-cloud-api-key
Closing this issue but if this doesn't resolve it feel free to reopen.

Thanks for responding so fast!
I have configured MANTLE_OPEN_CLOUD_API_KEY and everything works fine for project with 1 place.
When using more than one place, it gives error:

My mantle version is blake-mealey__mantle-0.11.15-deploy.
Perhaps I am missing some part of the configuration for a project with more than 1 place. Any help is much appreciated!

@blake-mealey This doesn't seem to be due to http://data.roblox.com/Data/Upload.ashx deprecation.
Roblox has entirely removed the https://www.roblox.com/ide/places/createV2 endpoint used in place creation API (https://github.com/blake-mealey/mantle/blob/861c56f2db3fc563e7187a68f255b3327582bf0c/mantle/rbx_api/src/places/mod.rs#L156C20-L156C62) .
In the creator hub they seem to use
https://apis.roblox.com/universes/v1/universes/(UNIVERSE ID)/places/(PLACE ID)/add-place (no post data; bool in response)  to link already-existing (from another universe) place to some universe.
There does not seem to be a better alternative which would create a new blank place.

Thanks for those details @Etheroit. I inspected the Roblox Studio traffic with wireshark and weirdly I am unable to capture the request that they are using to create the blank place in the Game Settings modal. I wonder if they switched to a non-http protocol? Seems pretty weird.
I think a workaround for this would be to:

create a new experience
remove the default start place from the experience
archive the experience
add the place to the desired experience

This would be pretty hacky and would leave an archived experience in the creator's account.

I started a thread in the dev forum since this removal was not documented in the official web endpoint deprecation thread and I believe it should have been: https://devforum.roblox.com/t/api-for-creating-a-place-for-an-experience-directly-has-been-removed-without-a-replacement/3062085

@blake-mealey  Are you sure it's possible to remove the start place from the experience? I believe it should not be?
You've also mentioned POST https://www.roblox.com/universes/removeplace API endpoint in your post which will for sure also be deprecated eventually as it's under www.roblox.com domain.
I've also tried looking for the API using fiddler but it doesnt seem to initiate any http request.

@blake-mealey Welp. That was fast. You've got a response already.
https://devforum.roblox.com/t/unable-to-create-places-within-an-experience-through-apis/3011837/3 seems to be the answer.
I would try to implement a fix but honestly I'm not sure how API responses are parsed in your code - sorry. :(

Welp. I've given it a shot. Please let me know if anything is wrong but looking at other endpoints the PR should be good.

You're right, my workaround would not have worked since you can't remove the root place from an experience. Good thing there is a replacement API for it! Thanks for the PR, I'll take a look at it.

This is fixed in the latest release:  https://github.com/blake-mealey/mantle/releases/tag/v0.11.16

Thanks guys! You are fast!
It created the places now. But it is still failing at one step for one file. It might be too large. Game.rbxlm is 126MB.
Error itself is:
Failed: http 500: {"type":"https://tools.ietf.org/html/rfc7231#section-6.6.1","title":"An error occurred while processing your request.","status":500,"traceId":"00-1bf8a17cfeef6af3d10acb4e989e2a69-742916ef57c1deef-00"}

Will give full output:
Deploying resources:
╷
│  + Creating: place_waiting
│    ╷
│    │  Dependencies:
│    │      - experience:
│    │          assetId:<PHONE_NUMBER>
│    │          startPlaceId:<PHONE_NUMBER>7
│    │  Inputs:
│    │    + place:
│    │    +   isStart: false
│    │

│    ╰─ Succeeded with outputs:
│         + place:
│         +   assetId:<PHONE_NUMBER>6
│

│  + Creating: placeFile_waiting
│    ╷
│    │  Dependencies:
│    │      - place:
│    │          assetId:<PHONE_NUMBER>6
│    │      - experience:
│    │          assetId:<PHONE_NUMBER>
│    │          startPlaceId:<PHONE_NUMBER>7
│    │  Inputs:
│    │    + placeFile:
│    │    +   filePath: GameWaitingLobby.rbxlx
│    │    +   fileHash: 5442c6c6cc0ee72607d9b420643cddef14057c0bf329fa43b3f90ece544a085f
│    │

│    ╰─ Succeeded with outputs:
│         + placeFile:
│         +   version: 2
│

│  + Creating: placeConfiguration_waiting
│    ╷
│    │  Dependencies:
│    │      - place:
│    │          assetId:<PHONE_NUMBER>6
│    │  Inputs:
│    │    + placeConfiguration:
│    │    +   name: "[DEVELOPMENT] Monsters Nearby Waiting Lobby"
│    │    +   description: Created with Mantle
│    │    +   maxPlayerCount: 50
│    │    +   allowCopying: false
│    │    +   socialSlotType: Automatic
│    │    +   customSocialSlotsCount: ~
│    │

│    ╰─ Succeeded with outputs:
│         + placeConfiguration
│

│  + Creating: place_game
│    ╷
│    │  Dependencies:
│    │      - experience:
│    │          assetId:<PHONE_NUMBER>
│    │          startPlaceId:<PHONE_NUMBER>7
│    │  Inputs:
│    │    + place:
│    │    +   isStart: false
│    │

│    ╰─ Succeeded with outputs:
│         + place:
│         +   assetId:<PHONE_NUMBER>4
│

│  + Creating: placeFile_game
│    ╷
│    │  Dependencies:
│    │      - place:
│    │          assetId:<PHONE_NUMBER>4
│    │      - experience:
│    │          assetId:<PHONE_NUMBER>
│    │          startPlaceId:<PHONE_NUMBER>7
│    │  Inputs:
│    │    + placeFile:
│    │    +   filePath: Game.rbxlx
│    │    +   fileHash: 615ff55037c81f61bc925adb31850d8445bb63c7bd5085f8c326ea9c8e3082e8
│    │

│    ╰─ Failed: http 500: {"type":"[[https://tools.ietf.org/html/rfc7231#section-6.6.1","title":"An](https://tools.ietf.org/html/rfc7231#section-6.6.1%22,%22title%22:%22An)](https://tools.ietf.org/html/rfc7231#section-6.6.1%22,%22title%22:%22An) error occurred while processing your request.","status":500,"traceId":"00-1bf8a17cfeef6af3d10acb4e989e2a69-742916ef57c1deef-00"}
│

│  + Creating: placeConfiguration_game
│    ╷
│    │  Dependencies:
│    │      - place:
│    │          assetId:<PHONE_NUMBER>4
│    │  Inputs:
│    │    + placeConfiguration:
│    │    +   name: "[DEVELOPMENT] Monsters Nearby Waiting Game"
│    │    +   description: Created with Mantle
│    │    +   maxPlayerCount: 50
│    │    +   allowCopying: false
│    │    +   socialSlotType: Automatic
│    │    +   customSocialSlotsCount: ~
│    │

│    ╰─ Succeeded with outputs:
│         + placeConfiguration
│

│

╰─ Failed 1 changes(s) while evaluating the resource graph. See above for more details.

Perhaps I should make another ticket as this problem is fixed.
And this problem does not seem like a problem with mantle, but Roblox API itself.
What are your recommendations? Should I create a bug report to Roblox devforum.
Or should I use a different file format that is not so large. rbxlx file was created with rojo, not sure how to make it to rbxl (that should be 50% smaller)

@MayGo You're right - file size appears to be the issue. 100mb seems to be the most recent file limit.
https://devforum.roblox.com/t/open-cloud-publishing-your-places-with-api-keys-is-now-live/1485135/122?u=etheroit
As for generating .rbxl file instead of .rbxlx, just modify the extension in rojo's build cmd.
rojo build -o build.rbxl instead of build.rbxlx.
Mantle seems to support .rbxl according to docs. Please let me know if this fix works for you!

Underestimated the file size shrinkage, actual was 126MB=>3.5MB, so now everything succeeds.
Thanks for pointing to that devforum post.
Super thankful for fixing this @Etheroit @blake-mealey
My whole motivation for working on Roblox projects again was riding on automating everything in Github Actions. 😊


Launcher[1] disconnected unexpectedly

VSCode Version:
Version: 1.40.1
Commit: 8795a9889db74563ddd43eb0a897a2384129a619
Date: 2019-11-13T16:47:44.719Z
Electron: 6.1.2
Chrome: 76.0.3809.146
Node.js: 12.4.0
V8: 7.6.303.31-electron.0
OS: Darwin x64 19.0.0
OS Version(Local):
ProductName:	Mac OS X
ProductVersion:	10.15
BuildVersion:	19A602
OS Version(remote):
Linux version 3.10.0-693.el7.x86_64
Steps to Reproduce:

press F5 to debug, will show below error.

python version
2.7.5
launch.json

{
    // Use IntelliSense to learn about possible attributes.
    // Hover to view descriptions of existing attributes.
    // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
    "version": "0.2.0",
    "configurations": [
        {
            "name": "Python: Current File",
            "type": "python",
            "request": "launch",
            "program": "${file}",
            "cwd": "${workspaceFolder}/demo/guide-python",
            "console": "integratedTerminal"
        }
    ]
}

error

(.env<EMAIL_ADDRESS>xgboost_new]# env PTVSD_SESSION_ID=1 PTVSD_LAUNCHER_PORT=39878 /search/odin/pcf/tools/.env/bin/python /root/.vscode-server/extensions/ms-python.python-2019.11.49689/pythonFiles/lib/python/new_ptvsd/no_wheels/ptvsd/launcher /search/odin/pcf/tools/xgboost_new/demo/guide-python/cross_validation.py
E+00000.010: /handling #1 request "launch" from Adapter-1/
             Handler 'launch_request' (file u'/root/.vscode-server/extensions/ms-python.python-2019.11.49689/pythonFiles/lib/python/new_ptvsd/no_wheels/ptvsd/launcher/../../ptvsd/launcher/adapter.py', line 35)
             couldn't handle #1 request "launch" from Adapter-1:

             Traceback (most recent call last):
               File "/root/.vscode-server/extensions/ms-python.python-2019.11.49689/pythonFiles/lib/python/new_ptvsd/no_wheels/ptvsd/launcher/../../ptvsd/common/messaging.py", line 763, in _handle
                 result = handler(self)
               File "/root/.vscode-server/extensions/ms-python.python-2019.11.49689/pythonFiles/lib/python/new_ptvsd/no_wheels/ptvsd/launcher/../../ptvsd/launcher/adapter.py", line 147, in launch_request
                 debuggee.spawn(process_name, cmdline, cwd, env, redirect_output)
               File "/root/.vscode-server/extensions/ms-python.python-2019.11.49689/pythonFiles/lib/python/new_ptvsd/no_wheels/ptvsd/launcher/../../ptvsd/launcher/debuggee.py", line 76, in spawn
                 "pointerSize": struct.calcsize("P") * 8,
             TypeError: Struct() argument 1 must be string, not unicode

             Stack where logged:
               File "/usr/lib64/python2.7/threading.py", line 785, in __bootstrap
                 self.__bootstrap_inner()
               File "/usr/lib64/python2.7/threading.py", line 812, in __bootstrap_inner
                 self.run()
               File "/usr/lib64/python2.7/threading.py", line 765, in run
                 self.__target(*self.__args, **self.__kwargs)
               File "/root/.vscode-server/extensions/ms-python.python-2019.11.49689/pythonFiles/lib/python/new_ptvsd/no_wheels/ptvsd/launcher/../../ptvsd/common/messaging.py", line 1520, in _run_handlers
                 handler()
               File "/root/.vscode-server/extensions/ms-python.python-2019.11.49689/pythonFiles/lib/python/new_ptvsd/no_wheels/ptvsd/launcher/../../ptvsd/common/messaging.py", line 809, in _handle

Does this issue occur when all extensions are disabled?: Yes/No

Please file the issue against the python extension.

@egamma Is this problem solved? What should I do?


ENG-2099 - feat: add python quickstart

Description

Testing required outside of automated testing?

[ ] Not Applicable

Screenshots (if appropriate):

[ ] Not Applicable

Rollback / Rollforward Procedure

[ ] Roll Forward
[ ] Roll Back

Reviewer Checklist

[ ] Description of Change
[ ] Description of outside testing if applicable.
[ ] Description of Roll Forward / Backward Procedure
[ ] Documentation updated for Change

:tada: This PR is included in version 1.5.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Discuss start symbols and reusable dynsem rules

https://github.com/spoofax-shell/doc/pull/65#discussion_r67364282

Done.


Changing GDP and GDP/capita data in Mimi GIVE

I was wondering what the easiest way to modify the GDP data from RFF SSPs is. I found the source RFF SSPs but couldn’t figure out how and where they were getting imported into GIVE.
I was looking at /packages/MimiGIVE/src/components/PerCapitaGDP.jl, packages/MimiRFFSPs/src/components/SPs.jl, packages/MimiGIVE/src/components/netconsumption.jl as places where GDP and per capita GDP shows up but couldn’t figure out what the best way to uniformly change the values is.
I would still like the new GDP data to be linked to the original RFF SSPs (for e.g. make the GDP value calculated for each Monte Carlo simulation 80% of the RFF SSP original value).
Would you mind just sharing the basic structure of how to accomplish this?
copied from: https://forum.mimiframework.org/t/changing-gdp-and-gdp-capita-data-in-mimi-give/271/2

Hi there! Great question, my advice would be to modify the MimiGIVE model using the modification API and adding a component to modulate the socioeconomic values, without having to edit the inputs at all, since that should be easiest and safest for you. Some instructions are here for that kind of modification -- in particular you'll be using add_comp! and then connect_param! as I indicate below.

Taking a look at main_model.jl, basically what happens is that we add a Socioeconomic component that drives gdp and population trajectories, this happens here
Then we connect those to other components anywhere you see something like
connect_param!(m, :PerCapitaGDP => :gdp, :Socioeconomic => :gdp)

So if you were, for example, to use a multiplier to on gdp you would do something like as follows.

using Mimi
using MimiGIVE

# create the new component you want to use
@defcomp gdp_modifier begin
    gdp_input = Parameter(index=[time, country])
    gdp_multiplier = Parameter()
    gdp_output = Variable(index=[time,country])

    function run_timestep(p,v,d,t)
        for c in d.country
            v.gdp_output[t,c] = p.gdp_input[t,c] * p.gdp_multiplier
        end
    end
end

# obtain original MimiGIVE model
m = MimiGIVE.get_model()

# add your new component right after the original Socioeconomics component
add_comp!(m, gdp_modifier, after = :Socioeconomic, first = 2020);

# set the multiplier
update_param!(m, :gdp_modifier, :gdp_multiplier, 0.8)

# connect the Socioeconomic input GDP to your component
connect_param!(m, :gdp_modifier => :gdp_input, :Socioeconomic => :gdp)

# now connect your component to the gdp per capita component, replacing the original
# connections
connect_param!(m, :PerCapitaGDP => :gdp, :gdp_modifier => :gdp_output)
connect_param!(m, :DamageAggregator => :gdp, :gdp_modifier => :gdp_output)
connect_param!(m, :global_netconsumption => :gdp, :gdp_modifier => :gdp_output)
connect_param!(m, :Agriculture_aggregator_gdp => :input, :gdp_modifier => :gdp_output)
connect_param!(m, :energy_damages => :gdp, :gdp_modifier => :gdp_output)
connect_param!(m, :dice2016R2_damage => :gdp, :gdp_modifier => :gdp_output)
connect_param!(m, :hs_damage => :gdp,  :gdp_modifier => :gdp_output)

# run your new model
run(m)

# take a look at your gdp_multiplier component in the explorer and make sure
# things look right, you can play around with components to check things look
# as expected here, or output data to do so manually
explore(m)

Just a quick note here, you are quite welcome to modify as you wish this is an open source model! I will quickly note though, that the RFFSP pathways are built such that the CO2 pathways are from expert elicitation conditioned on gdp per capita, so if you modify gdp, or gdp per capita, but not the emissions, you are breaking that link.

Hi Lisa,
Thank you so much, that was super helpful.
Yes makes total sense on the link breakage.
I am actually trying to transform the GDP values into a utility adjusted measure and was starting with a crude derate (say 80%). I tried to implement what I am trying to do using your very helpful example code but ran into an error.
Here's what I am trying to do:
Get input GDP data from RFF SSPs (which you showed me how to);
For each time period, transform the GDP data using this formula:
GDPoutput = [(1/number of countries) * sum(countryGDP^(1-parameter))] ^ (1/(1-parameter));
And then overwrite the GDP data feeding into MimiGIVE with this transformed GDP values.
Building off of your example, here is what I did:
using Mimi
using MimiGIVE
create the new component you want to use
@defcomp gdp_modifier begin
gdp_input = Parameter(index=[time, country])
util_curvature = Parameter() #added instead of GDP multiplier
gdp_input_transform = Variable(index=[time,country])
mean_gdp_input_transform = Variable(index=[time,country])
gdp_output = Variable(index=[time,country])
function run_timestep(p,v,d,t)
    for c in d.country
        v.gdp_input_transform[t,c] = (p.gdp_input[t,c]).^(1-util_curvature) #transform each element of the array
        v.mean_gdp_input_transform[t] = sum(v.gdp_input_transform[t,:])./length(d.country) #average of transformed country GDP for a given time period
        v.gdp_output[t,c] = (v.mean_gdp_input_transform[t]).^(1/(1-util_curvature)) #convert back to GDP
    end
end

end
obtain original MimiGIVE model
m = MimiGIVE.get_model()
add your new component right after the original Socioeconomics component
add_comp!(m, gdp_modifier, after = :Socioeconomic, first = 2020);
set the parameter value for curvature
update_param!(m, :gdp_modifier, :util_curvature, 1.5) #changed so that the parameter here is util curvature
connect the Socioeconomic input GDP to your component
connect_param!(m, :gdp_modifier => :gdp_input, :Socioeconomic => :gdp)
now connect your component to the gdp per capita component, replacing the original
connections
connect_param!(m, :PerCapitaGDP => :gdp, :gdp_modifier => :gdp_output)
connect_param!(m, :DamageAggregator => :gdp, :gdp_modifier => :gdp_output)
connect_param!(m, :global_netconsumption => :gdp, :gdp_modifier => :gdp_output)
connect_param!(m, :Agriculture_aggregator_gdp => :input, :gdp_modifier => :gdp_output)
connect_param!(m, :energy_damages => :gdp, :gdp_modifier => :gdp_output)
connect_param!(m, :dice2016R2_damage => :gdp, :gdp_modifier => :gdp_output)
connect_param!(m, :hs_damage => :gdp,  :gdp_modifier => :gdp_output)
run your new model
run(m)
But I am running into the following error:
UndefVarError: util_curvature not defined
Stacktrace:
[1] macro expansion
@ ./In[3]:16 [inlined]
[2] run_timestep_Main_gdp_modifier(p::Mimi.ComponentInstanceParameters{NamedTuple{(:gdp_input, :util_curvature), Tuple{Mimi.TimestepArray{Mimi.FixedTimestep{1750, 1, 2300}, Union{Missing, Float64}, 2, 1, SubArray{Union{Missing, Float64}, 2, Matrix{Union{Missing, Float64}}, Tuple{UnitRange{Int64}, Base.Slice{Base.OneTo{Int64}}}, false}}, Mimi.ScalarModelParameter{Float64}}}}, v::Mimi.ComponentInstanceVariables{NamedTuple{(:gdp_input_transform, :mean_gdp_input_transform, :gdp_output), Tuple{Mimi.TimestepArray{Mimi.FixedTimestep{1750, 1, 2300}, Union{Missing, Float64}, 2, 1, SubArray{Union{Missing, Float64}, 2, Matrix{Union{Missing, Float64}}, Tuple{UnitRange{Int64}, Base.Slice{Base.OneTo{Int64}}}, false}}, Mimi.TimestepArray{Mimi.FixedTimestep{1750, 1, 2300}, Union{Missing, Float64}, 2, 1, SubArray{Union{Missing, Float64}, 2, Matrix{Union{Missing, Float64}}, Tuple{UnitRange{Int64}, Base.Slice{Base.OneTo{Int64}}}, false}}, Mimi.TimestepArray{Mimi.FixedTimestep{1750, 1, 2300}, Union{Missing, Float64}, 2, 1, SubArray{Union{Missing, Float64}, 2, Matrix{Union{Missing, Float64}}, Tuple{UnitRange{Int64}, Base.Slice{Base.OneTo{Int64}}}, false}}}}}, d::NamedTuple{(:time, :other_ghg, :ozone_depleting_substances, Symbol("4"), Symbol("2"), :country, :fund_regions, :segments, :ag_mapping_input_regions, :ag_mapping_output_regions, :energy_countries, Symbol("3")), Tuple{Vector{Mimi.AbstractTimestep}, Vector{Int64}, Vector{Int64}, Vector{Int64}, Vector{Int64}, Vector{Int64}, Vector{Int64}, Vector{Int64}, Vector{Int64}, Vector{Int64}, Vector{Int64}, Vector{Int64}}}, t::Mimi.FixedTimestep{2020, 1, 2300})
@ Main ~/.julia/packages/Mimi/L0MHf/src/core/defcomp.jl:15
[3] run_timestep(ci::Mimi.LeafComponentInstance{Mimi.ComponentInstanceVariables{NamedTuple{(:gdp_input_transform, :mean_gdp_input_transform, :gdp_output), Tuple{Mimi.TimestepArray{Mimi.FixedTimestep{1750, 1, 2300}, Union{Missing, Float64}, 2, 1, SubArray{Union{Missing, Float64}, 2, Matrix{Union{Missing, Float64}}, Tuple{UnitRange{Int64}, Base.Slice{Base.OneTo{Int64}}}, false}}, Mimi.TimestepArray{Mimi.FixedTimestep{1750, 1, 2300}, Union{Missing, Float64}, 2, 1, SubArray{Union{Missing, Float64}, 2, Matrix{Union{Missing, Float64}}, Tuple{UnitRange{Int64}, Base.Slice{Base.OneTo{Int64}}}, false}}, Mimi.TimestepArray{Mimi.FixedTimestep{1750, 1, 2300}, Union{Missing, Float64}, 2, 1, SubArray{Union{Missing, Float64}, 2, Matrix{Union{Missing, Float64}}, Tuple{UnitRange{Int64}, Base.Slice{Base.OneTo{Int64}}}, false}}}}}, Mimi.ComponentInstanceParameters{NamedTuple{(:gdp_input, :util_curvature), Tuple{Mimi.TimestepArray{Mimi.FixedTimestep{1750, 1, 2300}, Union{Missing, Float64}, 2, 1, SubArray{Union{Missing, Float64}, 2, Matrix{Union{Missing, Float64}}, Tuple{UnitRange{Int64}, Base.Slice{Base.OneTo{Int64}}}, false}}, Mimi.ScalarModelParameter{Float64}}}}}, clock::Mimi.Clock{Mimi.FixedTimestep}, dims::NamedTuple{(:time, :other_ghg, :ozone_depleting_substances, Symbol("4"), Symbol("2"), :country, :fund_regions, :segments, :ag_mapping_input_regions, :ag_mapping_output_regions, :energy_countries, Symbol("3")), Tuple{Vector{Mimi.AbstractTimestep}, Vector{Int64}, Vector{Int64}, Vector{Int64}, Vector{Int64}, Vector{Int64}, Vector{Int64}, Vector{Int64}, Vector{Int64}, Vector{Int64}, Vector{Int64}, Vector{Int64}}})
@ Mimi ~/.julia/packages/Mimi/L0MHf/src/core/instances.jl:317
[4] run_timestep(cci::Mimi.ModelInstance, clock::Mimi.Clock{Mimi.FixedTimestep}, dims::NamedTuple{(:time, :other_ghg, :ozone_depleting_substances, Symbol("4"), Symbol("2"), :country, :fund_regions, :segments, :ag_mapping_input_regions, :ag_mapping_output_regions, :energy_countries, Symbol("3")), Tuple{Vector{Mimi.AbstractTimestep}, Vector{Int64}, Vector{Int64}, Vector{Int64}, Vector{Int64}, Vector{Int64}, Vector{Int64}, Vector{Int64}, Vector{Int64}, Vector{Int64}, Vector{Int64}, Vector{Int64}}})
@ Mimi ~/.julia/packages/Mimi/L0MHf/src/core/instances.jl:326
[5] run(mi::Mimi.ModelInstance, ntimesteps::Int64, dimkeys::Nothing)
@ Mimi ~/.julia/packages/Mimi/L0MHf/src/core/instances.jl:366
[6] run(m::Model; ntimesteps::Int64, rebuild::Bool, dim_keys::Nothing)
@ Mimi ~/.julia/packages/Mimi/L0MHf/src/core/model.jl:539
[7] run(m::Model)
@ Mimi ~/.julia/packages/Mimi/L0MHf/src/core/model.jl:529
[8] top-level scope
@ In[3]:46
Would you mind sharing what you think I am doing wrong here?
Thanks for your patience!

Hi there! Yes, so this is an easy thing to forget I do it all the time :)
The error
UndefVarError: util_curvature not defined

Is occurring because within the run_timestep function you refer to util_curvature instead of p.util_curvature so the macro is looking for a separate variable instead of looking up util_curvature in the parameters dictionary p. So you'll want:
@Defcomp gdp_modifier begin
  gdp_input = Parameter(index=[time, country])
  util_curvature = Parameter() #added instead of GDP multiplier
  gdp_input_transform = Variable(index=[time,country])
  mean_gdp_input_transform = Variable(index=[time,country])
  gdp_output = Variable(index=[time,country])

  function run_timestep(p,v,d,t)
      for c in d.country
          v.gdp_input_transform[t,c] = (p.gdp_input[t,c]).^(1-p.util_curvature) #transform each element of the array
          v.mean_gdp_input_transform[t] = sum(v.gdp_input_transform[t,:])./length(d.country) #average of transformed country GDP for a given time period
          v.gdp_output[t,c] = (v.mean_gdp_input_transform[t]).^(1/(1-p.util_curvature)) #convert back to GDP
      end
  end
end

You're going to run into a different issue here next though, that has to do with order of calculations. I can help with that, so to clarify, I see that v.mean_gdp_input_transform has no country index ... but v.gdp_output does have a country index. The way it is written now, v.gdp_output is going to be the same for every country.
In this case, you need to compute all the countries before you calculate v.mean_gdp_input_transform otherwise sum(v.gdp_input_transform[t,:]) will give you a missing because the countries have not all been computed yet within the loop.

So to clarify, what you would do to make the above work is this, but I think you likely want to modify to make sure output gdp is different by country.
@defcomp gdp_modifier begin
    gdp_input                   = Parameter(index=[time, country])
    util_curvature              = Parameter() #added instead of GDP multiplier
    gdp_input_transform         = Variable(index=[time, country])
    mean_gdp_input_transform    = Variable(index=[time])
    gdp_output                  = Variable(index=[time,country])

    function run_timestep(p,v,d,t)
        for c in d.country
            v.gdp_input_transform[t,c] = (p.gdp_input[t,c]).^(1-p.util_curvature) #transform each element of the array
        end

        v.mean_gdp_input_transform[t] = sum(v.gdp_input_transform[t,:])./length(d.country) #average of transformed country GDP for a given time period

        for c in d.country
            v.gdp_output[t,c] = (v.mean_gdp_input_transform[t]).^(1/(1-p.util_curvature)) #convert back to GDP
        end
    end
end


Using ng-zorro-antd with mgechev/angular-seed possible?

Explanation
Since my chinese is very limited i didn't understand if ng-zorro-antd is limited to angular-cli or works with other popular seed-projects, too?!
I'm submitting a...

[ ] Bug report
[ ] Feature request
[x] Documentation issue or request
[ ] Regression (a behavior that used to work and stopped working in a new release)
[x] Support request

Current behavior

I tried to add ng-zorro-antd to my angular-seed project. I extended project.config.ts with
let additionalPackages: ExtendPackages[] = [{
    name: 'ng-zorro-antd',
    path: 'node_modules/ng-zorro-antd/bundles/ng-zorro-antd.umd.min.js'
}];

this.addPackagesBundles(additionalPackages);

and added
import { NgZorroAntdModule } from 'ng-zorro-antd';

as well as
NgZorroAntdModule.forRoot()

to my app.module.ts but it crashes when running npm start

Environment

Angular version: 4.4.4

ng-zorro-antd version: 0.5.3

Browser:
- [X] Chrome (desktop) version 61.0.3163.100
- [ ] Firefox version XX
- [ ] Safari (desktop) version XX
- [ ] IE version XX

For Tooling issues:
- Node version: v6.9.4
- Platform:  Mac

please use angular-cli or other config that passed our integration test.
https://github.com/NG-ZORRO/ng-zorro-antd/tree/master/integration


[Feature] Syntax highlight text should have option to applying semantic tokens

The following issue: https://github.com/Microsoft/monaco-editor/issues/1171
describes how to get syntax highlighting for some text, but even if semantic token highlighting is enabled, it doesn't seem possible to apply it to the code in the monaco.editor.colorize API, so, this is a request to be able to do that.

Unfortunately, we don't have the resources to address this in the forseeable future.
I doubt this can be implemented easily, but if I'm wrong I'm happy to help getting a PR by the community merged!

Ok... I don't know if I'll be able to allocate time for it on my side, but if that becomes something required for me, I may end up implementing it (in which case it's possible I create a PR -- if that's the case I'll get in touch to get directions after studying a bit of the related code myself).

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.
If you wonder what we are up to, please see our roadmap and issue reporting guidelines.
Thanks for your understanding, and happy coding!


Checking OAuth validity...

I execute your script on a remote machine, so I used --noauth_local_webserver flag.
I got the same message in this case too:

Checking OAuth validity...
Your browser has been opened to visit:
If your browser is on a different machine then exit and re-run this
application with the command-line parameter
--noauth_local_webserver

Did you fix this issue?

I had the same issue, and resolved by changing adding outh_nonlocal=True to this line
gc = pygsheets.authorize(outh_file="credentials.json",outh_nonlocal=True)
After that it worked fine.

Thanks for the info @NorthernMan54


chore: repository governance

Repository governance update
This PR was automatically created by the AVM Team hive-mind using the grept governance tool.
We have detected that some files need updating to meet the AVM governance standards.
Please review and merge with alacrity.
Grept config source: git::https://github.com/Azure/Azure-Verified-Modules-Grept.git//terraform
Thanks! The AVM team :heart:

@Azure/avm-core-team-technical-terraform can I get an approval for this grept PR?

Supersceeded by #49


Split off Ridgepole::DSLParser::* to each files

This changes give benefits below:

easy to read boundary of classes what is without dependencies on indents
easy to read similar implemantations about @__definition between Context and TableDefinition

Changes Unknown when pulling f22e6772ef65a16ce88e9b4809e3c7ba455f5873 on aycabta:split-off-dsl-parser into ** on winebarrel:0.7**.

Thank you!

v0.6.6.beta2 has been released.


Missing Documentation for Scheduled Trigger

My Azure DevOps builds no longer support setting scheduled triggers in the pipeline.  I am seeing this message on 6/19/2019 and 6/20/2019  'You can no longer define scheduled triggers in the editor. Instead, you must add schedules to your YAML file. For more information, see https://go.microsoft.com/fwlink/?linkid=2080841'
That URL sent me here.  There is no documentation here for Scheduled Triggers.  Please add it so I can change the time which my build runs.

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

ID: 7458d5b2-1bb7-7d87-7ce6-2b0c430b0155
Version Independent ID: edf5bbd3-a3f0-5426-e985-31290f8afd05
Content: YAML schema - Azure Pipelines
Content Source: docs/pipelines/yaml-schema.md
Product: devops
Technology: devops-cicd
GitHub Login: @vtbassmatt
Microsoft Alias: macoope

This is also contradicting the following documentation page: Build pipeline triggers - Scheduled Triggers

Scheduled builds are not yet supported in YAML syntax. After you create your YAML build pipeline, you can use pipeline settings to specify a scheduled trigger.

@RichardMeadowsTC The doc just went live, here is a link to it:
https://docs.microsoft.com/en-us/azure/devops/pipelines/yaml-schema?view=azure-devops&tabs=schema


remove add/remove for groups

These methods fall outa convention for this library. Groups should accept arrays in args. Those arrays should be mutable. For example:
var handlers = [io, mem];
var bus = mesh.sequence(handlers);
handlers.push(mesh.wrap(function() {
});
handlers.unshift(mesh.wrap(function() {
});

sort(handlers); // mutate actual array here

This is also more flexible and carries fewer assumptions.  Another example:
var group = createGroup(io, mem);
var bus = mesh.sequence(group.source);
group.add(anotherHandler, -Infinity);
group.remove(anotherHandler);

groups now accept arrays.


SteamAuth with Slim Framework

Hi
i use this library for custom profile for my players
if i not include this on my framework (slim-framework) its working
But if include on framework i got error with login he redirect on 404 not found
Any soluce ?

Change $steamauth['loginpage'] to the page you want to login to.

@SmItH197
Ty for reply
i cant see steam login page
demo here
https://www.homelife-roleplay.fr/test/index.php/profil

Where have you uploaded the steamauth folder?

https://www.homelife-roleplay.fr/test/lib/steamauth/

Try moving it to /test/steamauth?

i work on my localhost's server
i upload then for you see my bug
not working after moving steamauth in / i got same error


OS and Uptime improvement

The issues
greetings! here's some issues with the code i am not capable of fixing myself :pleading_face:
OS
https://github.com/KikyTokamuro/fetch.scm/blob/0ff8769a67a479adec569018a869acc898a019a6/fetch.scm#L42
/etc/os-release is not a cross-distro file
Uptime
https://github.com/KikyTokamuro/fetch.scm/blob/0ff8769a67a479adec569018a869acc898a019a6/fetch.scm#L28
/proc/uptime doesn't exist in every OS too
My suggestion
i am sorry, i am not a Schemer, so, unfortunately, i can't do a thing myself. but you may have a look at disfetch. uptime layer may seem a little tricky and frustrating, but i think you can handle it better in Scheme than me in Bash. OS though is pretty simple. paç fat! :smile:

Hey @q60! Thanks for the issue ;)
65cfc8e - Fixed getting the name of the distro:

/etc/os-release
/usr/lib/os-release
lsb_release -sd
uname -o

229331c - Fixed getting the uptime:

/proc/uptime
uptime -p

In the future, I will add an option that does not fall under any of the above.
At the moment, these changes solve the shortcomings you mentioned?

sure they do. but /etc/os-release is a symbolic link to /usr/lib/os-release, so there's no need in checking both of them

Okay, thanks for the advice, I'll change.
Could you provide a screenshot of the work so that I can be sure that everything works fine?
The initial launch will compile the script and there will be an extra exhaust of unnecessary information (this, unfortunately, cannot be disabled in the guile), but it will not be present on next launches.


Save a photo to Gallery [Android]

It is possible save the photo on android gallery after takePicture function?

i just do this.
didn't know if there is a new features in the plugin, but i did it manually...i did it using another cordova plugin, cordova-base64-to-gallery. essentially i draw in a canvas the image returned and then i'll give the base64 encoding of the image to the plugin for save it to FS.
Here's a sample code:
        //on picture
		cordova.plugins.camerapreview.setOnPictureTakenHandler(function(result){
            //var img = document.getElementById('originalPicture');
			var img = new Image();

            //var canvas = document.getElementById("myCanvas");

            img.onload = function () {
                var canvas = document.createElement('canvas');
                canvas.height = img.height;
                canvas.width = img.width;
                var ctx = canvas.getContext("2d");
                ctx.drawImage(this, 0, 0);
                var base64Img = canvas.toDataURL();
                cordova.base64ToGallery(
                    base64Img,
                    'originalImg_',
                    function(msg) {
                        alert("Base64 success: " + msg);
                    },
                    function(err) {
                        alert("Base64 error: " + err);
                    }
                );
            }

            img.src = result[0];//originalPicturePath;

			//document.getElementById('previewPicture').src = result[1];//previewPicturePath;
		});

Thanks a lot, man.
Works perfectly!

+1
Thanks @andrea-86 for the work around


Typescript infer queryKey to unknown

Code at example https://github.com/jotaijs/jotai-tanstack-query/blob/main/examples/01_typescript/src/App.tsx#L10

correct expect:
id infer to number

Maybe it's a bug, or a typescript limitation.
A naive workaround: https://tsplay.dev/WPBrqN

same as this? #25
explicitly annotating get as Getter kinda works. If I remember correctly it would say string | number rather than unknown.

Okay, let's close this as duplicate of #25.


WIP: Ensure we rebuild the base Dapper image if needed

This checks the build timestamp of the base Dapper image against files
we know we share, and rebuilds it if necessary.
Signed-off-by: Stephen Kitt<EMAIL_ADDRESS>

Is this still relevant?

Is this still relevant?

I’m working on an update of this which initialises the tracker files based on the images’ build timestamp.


Support for Sessions to be usable via Headers, so they can be used from the CLI

Additional Conext
This is one I've had the idea of for a while. At the moment sessions in Pode are only supported in a browser via cookies, which means you can't use them in functions like Invoke-RestMethod. Normally, anything via a REST API is always sessionless, but at times there is the need for session support.
Describe the Change
The idea is to extend sessions so that you can tell Pode to send back a SessionId using headers, rather than via cookies. The session could be sent back in a header called X-Pode-SessionId.
This would require an update to the Enable-PodeSessionMiddleware function with a new -Header switch parameter (and filter out cookie parameters using Parameter Sets). Headers could also use the existing -Secure switch, as one plan I had is to not only encrypt a session using the supplied secret, but to also encrypt using the UserAgent and RemoteIPAddress - to help prevent session sharing.
In the main session middleware, the session will instead be extracted from the X-Pode-SessionId header and verified. The header itself could be created by some POST /login route, and the response will contain the session header for subsequent requests.
Sessions will still support expiry timestamps, and be destroyable via Remove-PodeSession - should you need a DELETE /logout route.
The format of the X-Pode-SessionId could change, but my initial thinking is something as follows:
X-Pode-SessionId: <session>;<expiry-timestamp>

The session in the header will just be:
X-Pode-SessionId: <sessionId>

Like cookies, the sessionId will be signed using the supplied secret.
There's a new -Strict switch on the Enable-PodeSessionMiddleware function. If this is set, then the supplied secret will be extended using the client's UserAgent and RemoteIPAddress from the request.

Decided to change the Header name, it now works like normal session cookies instead of being hard-coded. The default is pode.sid, and you can change the name with -Name on Enable-PodeSessionMiddleware.


Private inputs

When the private inputs are large, we suggest using binary files as private inputs. The format is also binary, so we don't need to encode it to string and then decode it.

Better to have XXX:file just like 32:i64, where XXX is the filename. So that user can specify private inputs by 32:i64 data.json:file ...


Include req in panic message for mock failures

Fixes https://github.com/loyd/clickhouse.rs/issues/89

Actualized in https://github.com/loyd/clickhouse.rs/commit/65829d3de76910d0eb4627e2fdf7467131f9a251


Size limit

Is there a limitation to the size of the ply file?  And do you have any methods for reading in .pcd files instead of ply?  Below is the output of teaser_cpp_ply when I attempted with my own ply data.
Read 194507 total vertices
Read 164016 total vertices
terminate called after throwing an instance of 'std::bad_alloc'
what():  std::bad_alloc
Aborted (core dumped)

TEASER++ takes in correspondences, not raw point cloud. So you need to use PCL or some other libraries to first generate features and correspondences. Then feed those correspondences to TEASER++.

So, taking  teaser_cpp_ply as an example, you are saying the the ply that is input into the example is a correspondence in the form of a .ply file?

The example read a .ply file that contains a 3D model, applies random 3D transformation with outliers, and estimated the transformation.

Right, so if I input two of my own 3D model represented as a .ply's to the example and I get the error that I listed in my post then I am still curious- is there an upper limit on the size of the ply to this example program?
I have attempted both to simply swap the example model with my own model and I have loaded two files and took out the random transformation and in both cases I have gotten seg faults and allocation errors.

In your case, because the input and output are of different sizes, Eigen failed to do the proper memory allocation.

This is the case given the output I posted, but it seg faults if I put in the .ply with 164016 total vertices to the given example and apply the random transformation. In this case there is only one input as is consistent with the example but it still seg faults due to size. (Insofar as I can tell.)

I see. So I don't have a hard memory limit, but 164016 vertices will definitely consume a lot of memory, and seg faulting due to memory will definitely happen if your RAM runs out. You can confirm it by maybe checking with your system monitor.

I suspect this is what is happening.  Thank you for your prompt replies!

@rmslick No problem! Open any new issues if you have new questions.


Loginbox II

I tried the #249 and played a little bit around and found something I personally would more like - so I thought I just show it and we discuss if it make sense and we want to use it/some of it :)

(Also rebased on multi-monitor)

@SebTM Yes the text looks better centered over-under... But, you're only showing it with one modifier on. If you have both on, the text will push out of the box (as discussed on #229). This is why I chose to left align them. Also the spacing between the greeter and layout is too big, IMO. Time, greeter, and layout should be more evenly spaced.

Okay, I understand - the spacing between greeter and keyboard was intended, the indicator down make sense, so is it ok when I rebase #249, center verify/failed and merge it?


Error when deleting a "MultiplayerSpawner" tracked node after a client has left.

Tested versions

Reproducible in v4.3.dev6.official [89850d553], 4.2.1 stable and master.

System information
Godot v4.3.dev6 - Windows 10.0.22631 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 3060 Laptop GPU (NVIDIA; <IP_ADDRESS>58) - AMD Ryzen 9 5900HS with Radeon Graphics (16 Threads)
Issue description
When a client leaves, if you delete a node that was spawned by a Multiplayer Spawner, the following error pops up:
E 0:00:08:0910   _remove_node_cache: Condition "!pinfo" is true. Continuing.
  <C++ Source>   modules/multiplayer/scene_cache_interface.cpp:60 @ _remove_node_cache()

This happens at this point in the code (Taken from current master):
void SceneCacheInterface::_remove_node_cache(ObjectID p_oid) {
	NodeCache *nc = nodes_cache.getptr(p_oid);
	if (!nc) {
		return;
	}
	if (nc->cache_id) {
		assigned_ids.erase(nc->cache_id);
	}
	for (KeyValue<int, int> &E : nc->recv_ids) {
		PeerInfo *pinfo = peers_info.getptr(E.key);
		ERR_CONTINUE(!pinfo);            ////////////   <<--- Error happens here.
		pinfo->recv_nodes.erase(E.value);
	}
	for (KeyValue<int, bool> &E : nc->confirmed_peers) {
		PeerInfo *pinfo = peers_info.getptr(E.key);
		ERR_CONTINUE(!pinfo);
		pinfo->sent_nodes.erase(p_oid);
	}
	nodes_cache.erase(p_oid);
}

Steps to reproduce

Run two instances of the minimum repro project.
In one instance click Host
In the other instance click Join
Close the instance that clicked Join.
See the error pop up on the Host.

Minimal reproduction project (MRP)
network_issue_minimum_repro.zip

It was a simple fix, did a pull request for it.
When a client disconnects on on_peer_change it looks like the intention of the code was to remove the peer ID from the list of received ids of that node with nc->recv_ids.erase(p_id); but what is actually in the code today is nc->recv_ids.erase(E.key); which in this case is trying to erase the remote cache id of the node from the list of received ids of the node, which doesn't really make sense since that is a list of peer ids that received that node.
The error happens because since the id wasn't being removed from the list of recv_ids when the node was eventually freed it tried to get the peer_info for that id and failed since the peer at this point no longer exists, showing the error.

Oops. Didn't mean to close with the previous comment.


Egg photo showed improperly in S3E17 meeting notes

Egg photo showed improperly in S3E17 meeting notes, we can check here


If there's a conflict, the action won't create a PR

Hey there, I have seen some actions not creating PRs although the workflow run is successful. My suspicioun is that PRs that have merge conflicts are not created although the action completes succefully. Can we "fail" on merge conflict (or create the PR anyway)?
Thanks!

Scratch that, the issue was that I was checked out with a different PAT than the one I stated in the action. When this is the case, the action does nothing but shows succesful run.

sorry for the late response, thanks for mentioning your issue and how to solve them. I want to work out some kind of troubleshooting and common issues and consider to add this there.


Camera feed fails to update live when launching app from widget

Hello again,
Very minor bug, but wanted to point it out just in case.
When launching the app via the widget, the camera feed stays frozen with the frame that was seen in the widget and doesn't actually update back to live until switching tabs (thus refreshing the view).

Same problem after last update here


allow-endpoints should accept wildcards or better handle Github artifact hosts

As per the GitHub runner requirements, a GitHub runner that needs to handle caches and artefacts should be able to connect to *.blob.core.windows.net hosts.
E.g. my build workflow connected to kv4gacprodeus2file3.blob.core.windows.net:443, to download the yarn cache, and then optionally would upload it again in a post step (via action/setup-node action, which uses actions/cache).
The GH cache action uses an indirect API; an API call is made to artifactcache.actions.githubusercontent.com to get the blob location, which in my case was kv4gacprodeus2file3.blob.core.windows.net. However, there is no guarantee that a next run will use that same hostname!
My next run made a connection to gheus21ubt20eus26diag.blob.core.windows.net during the post-complete step, as the provisioner tried to upload some artefact. The harden-runner blocked this as the host was not whitelisted.
I fully realise that just whitelisting *.blob.core.windows.net is not a great idea either; there could be any number of malicious payloads lurking in Azure blob stores, so ideally there should be a way to whitelist just the known GitHub artefact and cache hosts. This may require petitioning GitHub to use dedicated hostnames for their cache / artefact hosts and / or the use of SSL certificates on those hosts that can be verified.

Thanks @mjpieters for creating the issue! I agree with you. In fact I have created an issue at actions/toolkit to make the artifact endpoint location available during a run, so harden-runner does not need to ask developer for it. This cache location issue definitely needs to be addressed in a better way.
Do want to clarify that the gheus21ubt20eus26diag.blob.core.windows.net is not used for cache. In your run, the call to cache was not blocked. If you look at the lint job in the run, the call is made to cache.
gheus21ubt20eus26diag.blob.core.windows.net is used for some diagnostics after the job is finished. I had reached out to GitHub about it, and will follow up on it. But that does not affect the workflow at all.

Yes, I had seen that gheus21ubt20eus26diag is only accessed after run completion. But, since the logs I can access do not cover that stage, I can't verify what is being uploaded.

Hi @sethmoo, I observed that you used harden-runner in https://github.com/google/identity-credential/blob/master/.github/workflows/unit-test.yml and refer to this issue in the workflow file.
I want to clarify that fixing this issue is not a blocker to using the block mode. What happened in your case was that the cache endpoint mp1yacprodeus1file3.blob.core.windows.net was from your fork of the repo. Whereas the one that got blocked si05acprodeus1file1.blob.core.windows.net is the cache endpoint for the parent repo https://github.com/google/identity-credential. So, if you just add the endpoint si05acprodeus1file1.blob.core.windows.net:443 to the allowed-list, and change to block mode, it should run fine.
The cache endpoint is different for a fork relative to the parent repo. If you had run the audit mode in a branch of the same repo, it should have worked fine.
There are multiple workflows that are using block mode with the cache endpoint specified. Here is an example:
https://github.com/ben-manes/caffeine/blob/27bc594262ada030f76d668e7fc8e72e91defa4a/.github/workflows/build.yml#L9
So, while I would love to fix this issue, and hope to do that once the cache endpoint is known, it is not a blocker to using the block mode. I will add this info to the README file.
Also, if you have feedback on the README or developer experience of using the harden-runner Action, please do share. Thanks!

Thanks!
Our workflow doesn't handle any sensitive info, and is never used to publish anything (it's only for continuous integration/tests). I'd rather not have to worry about playing whack-a-mole in case the CDN decides to change a hostname on us. Thus, I think I'll just stick with audit for now. The best fix, if we really wanted an allow list, would probably be to host the binaries we need at a well-known location. Adding wildcards to the hostnames is probably not the best solution.
In terms of feedback, harden-runner has been really nice to use. It came recommended to me by a coworker and I was delighted with how quickly I could integrate it into our workflow.

This issue is fixed in the latest release. Harden runner now auto detects the cache endpoint.
More details in this blog post: https://blog.stepsecurity.io/harden-runner-github-action-now-auto-detects-cache-endpoints-1ad77fe2da0a
I will go ahead and close this issue. Please reopen if you think it is not working as expected. Thanks!


2-Take the frontend structure from  lara5-vue2-api

https://github.com/Janis-Rullis-IT/lara5-vue2-api
Branch

1-setup-env

Keep
Only the vue2 related part. Remove project-specific files.
When done

Merge to main branch and keep 1-setup-env as a snapshot.

Proof that it runs


Upgrade to OpenAPI Specification

Is your feature request related to a problem? Please describe.
Upgrade to OpenAPI Specification
Describe the solution you'd like
Update to new API standards

Duplicated #548


Is this repo still under maintenance

this package has compatibility issues since Laravel 6.x, and now Laravel 7.x either. If this package is no longer maintained I would remove it forever from my project.

Install version v1.0.4


Extended opInf

Thank you. The only modification I did was to keep the name AutoencoderKoopman, since that is really a Koopman operator.


fix: pin major version of open-next to 1

Currently, cdk-nextjs is not compatible with the latest v2+ version of open-next. Work is being done to update the project to be compatible, but for now we'll make sure new users coming to cdk-nextjs for the first time are to deploy a working Next.js site.
see: https://github.com/jetbridge/cdk-nextjs/issues/126
Fixes #

@kevin-mitchell, I'm not sure of the pros/cons of this, but an alternative is to make open-next@1 a peerDependency. Thoughts? Then it's more explicit that this project depends on it.

thanks!


Implement Generation 2 Hyper-V VM support

Feature request https://github.com/LIS/LISAv2/issues/211

Let's review and discuss the changes on Monday and not merge until then.

Regression Tested on Hyper-V and Azure using the flag VMgeneration = 1, test category functional/ core area
[hyperv] VHD under test F:\Hyper-V\VHDs\lisav2\ubuntu_18.04.vhd
[hyperv] ARM Image under test  :  :  :
[hyperv] Total Executed TestCases          17 (         15 Pass,          0 Fail,          2 Abort)
[hyperv] Total Execution Time(dd:hh:mm) 0:1:17
[hyperv] XML file: TestConfiguration
[hyperv]
[hyperv] #Sr. Test Name : Test Result : Test Duration (in minutes)
[hyperv] ----------------------------------------------------
[hyperv] 1. CORE-VERIFY-HEART-BEAT : PASS : 2.45
[hyperv] 2. CORE-VERIFY-SHUTDOWN : PASS : 1.35
[hyperv] 3. CORE-VERIFY-TIME-SYNC : PASS : 1.98
[hyperv] 4. CORE-INITRD-MODULES-CHECK : PASS : 0.91
[hyperv] 5. CORE-CPU-VERIFY-ONLINE : PASS : 0.76
[hyperv] 6. CORE-VERIFY-LIS-MODULES-VERSION : PASS : 0.77
[hyperv] 7. MAX-VCPU : PASS : 5.1
[hyperv] 8. PAUSED-CRITICAL-HEARTBEAT : PASS : 4.05
[hyperv] 9. CHECK-NUMA : PASS : 2.05
[hyperv] 10. CHECK-NUMA-MAXIMUM : PASS : 2.19
[hyperv] 11. REBOOT-DIFFERENT-MEMORY-SETTINGS : PASS : 4.02
[hyperv] 12. TIMESYNC-BASIC : ABORTED : 1.95
[hyperv] 13. TIMESYNC-SAVEDVM : PASS : 16.22
[hyperv] 14. TIMESYNC-PAUSEDVM : PASS : 16.06
[hyperv] 15. TIMESYNC-SAVEDVM-CHRONYOFF : PASS : 8
[hyperv] 16. TIMESYNC-PAUSEDVM-CHRONYOFF : PASS : 7.96
[hyperv] 17. TIMESYNC-VDSO : Aborted : 1.87
[hyperv]
[azure] #Sr. Test Name : Test Result : Test Duration (in minutes)
[azure] ----------------------------------------------------
[azure] 1. CORE-INITRD-MODULES-CHECK : PASS : 3.59
[azure] 2. CORE-CPU-VERIFY-ONLINE : PASS : 2.71
[azure] 3. CORE-VERIFY-LIS-MODULES-VERSION : PASS : 2.98
[azure]
[azure]

@juhlee-microsoft @simonxiaoss @iamshital  please check this.
We have to further discuss the related changes that might be required to test cases, but support for Gen2 VM is covered in this PR and it has been tested successfully with both VM generations deployments.

@lizzha  please approve the review as all requests have been addressed.
Thank you!


magically hex TDT ID if given in decimal

TDT IDs are given in decimal in many block explorers: https://ropsten.etherscan.io/token/0x20ea5b307ee28eefc15a3e6d945c5e8da5a8add9
If a decimal TDT ID is provided, automatically convert from decimal to a hex address.

Ooh good idea, thanks @brandoncurtis!


Make run open new thread instead of infinite while loop

The run function just opens a loop and sleeps. That's pretty bad - there's no way to stop the running without stopping the program.
We should replace it with an async thread that re-runs every X seconds (until another function tells it to stop).

I get where youre coming from, infinite loops are bad.
Currently compiling php with thread safety and such, hope to get it working

Yeah, im giving up on threading right now
The thread extension is not working well on php 7.1, and i dont feel like forcing composer to work right with it.
Maybe in the future

see start.sh, it threads with POSIX


Stuck while deploying contract

If you are reporting a problem with Token Wizard, please include the following information:
Which network did you use? (Mainnet, Kovan, Rinkeby, etc.)
Mainnet
If you were able to create it, what is the URL of your crowdsale?
No, just https://wizard.oracles.org/4
Do you have screenshots showing the problem?

Do you see errors in the dev console? If yes, please include a screenshot
I see one error now. After refresh several times this page, still getting on
If you see errors, please right click on them and "Save as..". Zip saved file and attach it to the Issue.
Your answer

@Agritoken could you

if you see errors, please right click on them and "Save as..". Zip saved file and attach it to the Issue.

I´m not sure if is what you need, but is what I got. Let me know. Txs
wizard.oracles.org-1523034601336.zip

I re-run a new crowdsale process and I think issue was in relation to MinCap field, which was blank. Once I filled then works fine. The issue has been stated by other user also.
If you consider so, this can be closed. Thanks


Virtualizarr + Coiled Serverless Example Notebook

Inspired by @thodson-usgs's Lithops example, I've created a Virtualizarr example using coiled serverless functions.

1TB virtual dataset from 924 NetCDF files
9 minutes and ~$0.24 of cloud cost on coiled

Would love some feedback if anyone has thoughts.

[ ] Changes are documented in docs/releases.rst

Awesome!!
I'm a bit unclear where the in-memory datasets live at each point in the computation when using coiled functions. You do the reference generation on a bunch of separate instances, but in order to to combine_by_coords they all have to be on the same instance. At what point does that transfer occur?

I'm a bit unclear where the in-memory datasets live at each point in the computation when using coiled functions. You do the reference generation on a bunch of separate instances, but in order to to combine_by_coords they all have to be on the same instance. At what point does that transfer occur?

I'm pretty sure the .map returns a generator of all the virtual datasets to my local laptop.
Coiled allows you to run notebooks in the cloud. I successfully tried running all the reference generation serverless functions from a coiled cloud notebook and that also worked great. It might be a good option for a larger dataset, but I think since the manifest arrays are so memory efficient, I didn't see any issues doing the reduce on a laptop.
I also tried starting the reference generation severless functions from a larger serverless function (that was meant to be the reduce machine) and it was kinda wacky.


This is a pull request to merge our dev branch into main. Do this when we're ready for some sort of release.

Added a wall example

Since the EA and ABM have been integrated I am merging this pull request.


Do_removepandas

Skip first row of dataset by providing spark schema. We remove pandas
as a hard dependency (although hail will install it anyway)

I branched this from another branch :(

Codecov Report

Merging #37 (2ed49cc) into main (8c77cd1) will increase coverage by 0.10%.
The diff coverage is 75.00%.

@@            Coverage Diff             @@
##             main      #37      +/-   ##
==========================================
+ Coverage   49.36%   49.47%   +0.10%
==========================================
  Files          24       24
  Lines         470      473       +3
==========================================
+ Hits          232      234       +2
- Misses        238      239       +1

Impacted Files
Coverage Δ

src/etl/intervals/andersson2014.py
56.25% <75.00%> (+1.07%)
:arrow_up:

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


Update azure-cli-arm-commands.md

Moved HDInsight content alphabetically between Group and Insights.

Hi @matt1883, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (mahi). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

#sign-off

@dlepow - Dan, this is your article, can you review and let me know if this is OK to merge?

@tysonn - This change looks fine to merge. Thanks!


catalog addAnnotations

This PR is to add functionality to be able to add annotations at the catalog level. I refactored createCatalog in import.js to always call addACLs because the logic to return if none were available is already written into the catalog functionality.

It seems like you're sending annotations in multiple requests using the catalog/1/annotation/<annotKey> api. I'm not familiar with the catalog level annotation, but in schema or table level you can send the whole document to catalog/1/schema/<schemaName>/annotation and ermrest already supports it. Doesn't ermrest have the same api for catalog annotation? to create all the annotations at once using catalog/N/annotation api?


test(wandb): add test

This PR adds a test for WandbCallback

Thanks for the test! Will try to fix the problem when I have time.


README updates and better dsim printing

Hello @mdw362! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file druzhba_run.py:

Line 178:80: E501 line too long (197 > 79 characters)


Update to embedded-hal 1.0.0

Tested locally, not with the stm32 though.
shared-bus still loops in 0.2.x of embedded-hal. I suspect it is not the right solution anymore given embedded-hal-bus, but that might be best as a separate change.

Closing in favor of this PR because the upgrade allowed me to clean up a lot more than I expected. Thank you for getting it started!


refactor

@davies I tried to generate the doc but it doesn't "compile":
/Users/meng/src/spark/python/docs/pyspark.ml.rst:10: WARNING: autodoc: failed to import module u'pyspark.ml'; the following exception was raised:
Traceback (most recent call last):
  File "/Users/meng/anaconda/lib/python2.7/site-packages/sphinx/ext/autodoc.py", line 335, in import_object
    __import__(self.modname)
  File "/Users/meng/src/spark/python/pyspark/ml/__init__.py", line 18, in <module>
    from pyspark.ml.param import *
  File "/Users/meng/src/spark/python/pyspark/ml/param/__init__.py", line 20, in <module>
    from pyspark.ml.util import Identifiable
  File "/Users/meng/src/spark/python/pyspark/ml/util.py", line 23, in <module>
    from pyspark.ml.param import Params
ImportError: cannot import name Params

Hold on, i'm still work on it.

@mengxr The docs and tests are ready, please review!


How to calculate the avg score of reward bench?

So that we can match the rank of reward bench.

For Chat & Chat-hard & Reasoning & Safety, I believe that it should be average with equal weights. But you may also need to compute the prior set by the code of reward bench...
My suggestion is that if you find your model is good with respect to the previous 4 metrics, you can crate a PR at reward bench to ask the authors' help.

Thanks for your help!


Add support for Concurrent Tasks (Laravel Octane)

Describe you feature request
It would be very nice to support the Octane Concurrent Task feature in the FrankenPHP driver. Currently, only Swoole supports this feature and that's why some people (including me) prefer to keep using Swoole while the FrankenPHP has shown a better benchmark result.
Describe the solution you'd like
I want to set my Octane's driver to FrankenPHP and still be able to write concurrent tasks in a Laravel application:
[$users, $servers] = Octane::concurrently([
    fn () => User::all(),
    fn () => Server::all(),
]);

And I think such behavior can be accomplished using Go's Goroutines and WaitGroup features.
Describe alternatives you've considered
Currently, I'm using Swoole driver in my Laravel application.

In my opinion, this would be better to delegate this to the parallel extension or to Swoole (the extension) itself.
Actually, this should already work as long as they extension are built with FrankenPHP.
This may be possible to implement a similar feature in  Go using Goroutines, but I think that it isn't worth it as the current solutions work with FrankenPHP (and they already have large ecosystems), and we'll likely have no performance gain compared to them.
WDYT?


Add config flow

add config flow to the goodwe integration as a step towards an official HomeAssistant integration.
~This is still a work in progress, DO NOT MERGE YET!~
To do:

~add DataUpdateCoordinator instead of refresh job~
~clean up code (black, isort, pylint, flake8, pydocstyle) and fix imports etc.~
test the code

The new new inverter_class field is used in the config flow on line 84 to extract the inverter family/class that is discovered during the first connection (in which the family/class is not provided and auto discovery is used).
That information is then saved in the config data and on all normal starts of the integration (restart HomeAssistant etc.) this data is used to supply the family/class on line 36 of the init file, in that way on normal starts no time is wasted on trying to determine the correct family/class.

Aaah I see.
But thinking, couldn't we use type(inverter).__name__  instead of additional instance attribute ?

@mletenay you are probably right, I will change it

@mletenay I think the code of this PR is ready, the config flow is implemented, a DataUpdateCoordinator is implemented and everything is formatted and the imports are checked.
Could you give it a try and see if it actually works?
(I can't test it myself since I don't own a Goodwe inverter yet)

Great !
The code looks fine, I'll test it today evening and merge it if everything works as expected.

I did not manage to find enough time for this tonight, did just brief testing, but everything seems to be fine so far !
Will review and merge it on Wednesday ...

Thanks, I have merged to PR (and than fixed some minor string typos).
It works nice, but we lost during this change the code created here https://github.com/mletenay/home-assistant-goodwe-inverter/commit/239e54f7a100a23a98c00dbb6d520d9a7d28cea5 around line 186.
It was added there just recently to address  unreliability of the UDP protocol where requests are rather frequently lost in some environments.
How should we implement this behavior after we switched to DataUpateCoordinator ?

@mletenay I think the packet loss schould be dealt with in the upstream library where we schould just retry the update command after a resonable timeout for up to 3/4 times.
However if you would really want to, you could still implement a simular code to make a couple of retries inside the async_update_data() function here: https://github.com/mletenay/home-assistant-goodwe-inverter/blob/9d630b9e195dc1f71e156e321c8d4c2c1f55aadf/custom_components/goodwe/init.py#L46
Or if you don't want to retry emidatly and wait for the next scan_interval, you could overwrite the available property of the sensor classes and make up your own available logic by adding a "failed_attempts" key to the data in async_update_data() that can count how many times the requests have failed, and based on that mark the device as available/unavailable.

Why did you remove the device_info from the sensor entities?
Schouldn't all those entities be linked to the single device entry for the inverter?

Very glad to hear that the code is actually working on a real Goodwe inverter.
Actually was not expecting it to work on the first try ;)

It worked like a charm on first run, well done !
I have removed the device info from the individual sensors an left it only on the main entity, since the sensor device info was actually being overwritten over and over and there was at the end only device info of the last sensor.
I was probably consequence of the "identifiers": {(DOMAIN, self._config_entry.unique_id)}, be all the same for all sensors ?
Or maybe just the "name": self.name, should be fixed/static for all the sensors ?

Regarding the UDP retries, the library performs (3) retries already. The code in sensors was additional on top of that.
If from some reason connection to inverter got lost and all the requests fail, we want to set sensor values to None to actually see/record the problem. But there are people out there with occasional/random dropouts (larger than the standard library retry mechanism). And for those we intentionally keep (stale) sensor values for couple of scan_intervals till its set to None.

yea indeed the name schould be constant and schould be the name of the device aka the inverter, not from the entities.

You could also use self._config_entry.title instead of the constant default name.
But honestly it does not really matter since users can overwrite the name anyway though the UI.

I'll try the "available" property approach, it's actually what I tried to achieve with that clumsy retry counting.
My knowledge of HA internal is pretty shallow, so I just did the simplest way I've imaged ;-)

Regarding the UDP retries, the library performs (3) retries already. The code in sensors was additional on top of that.
If from some reason connection to inverter got lost and all the requests fail, we want to set sensor values to None to actually see/record the problem. But there are people out there with occasional/random dropouts (larger than the standard library retry mechanism). And for those we intentionally keep (stale) sensor values for couple of scan_intervals till its set to None.

I think the best approach would then be to overwrite the available property and keep track of the amount of request tries

I am not sure if we have acces to the previous "data" dict when running the next async_update_data(), which is needed to increment the retry counter.
Therfore it might be nessesary to implement the retry counter in the upstream lib as part of the inverter class.

Actually a good idea, I will pass the number of consecutive retry failures inside the MaxRetriesException raised by the library and handle that one in async_update_data()


Spike and slab feature sampling priors (feature weighted sampling)

Hi @guolinke ,
being thinking about the feature_baggingand crossing myself information with permutation importances and other tools related to feature selection on decision tree ensemble models, I have reached an idea about a possible feature on LightGBM.
Threre could set the posibility of performing the feature bagging according to assigned probabilities of inclusion for each feature on each boosting iteration or each randomforest tree.
It would work as some kind of a proxy to spike-and-slab technique. Do not know if it would improve the full random option, as it would also produce trees with different number of features used in each iteration, but it would include more the a priori best variables.
The main idea is to run LightGBM, do permutation importances, then set spike-and-slab probability of inclusions based on this permutation importances, then run again LightGBM with those feature sampling prior probabilities.
Maybe I am asking for an already existing feature as feature_weight, but I have not managed to find it, excuse me in this case.
Regards.

Just having by-feature probability of sampling for any given tree learner, would be awesome.

Closed in favor of being in #2302. We decided to keep all feature requests in one place.
Welcome to contribute this feature! Please re-open this issue (or post a comment if you are not a topic starter) if you are actively working on implementing this feature.


Bug: player head moves to bottom right corner of screen for no reason

Every time I try Controllable this happens in Minecraft.

Minecraft 1.14.4
Forge 28.1.70
OptiFine HD U F4
Controllable-1.14.4-0.6.1

I have a Bluetooth Xbox One S (1708) controller.

I don't know what you're talking about. Have you got a video? Maybe you need to adjust the dead zone in the settings

I tested it again and I now suspect that it might be from my own controller. I'll update you if I get my hands on another controller and test it again.

Kinda have the same thing, Player looks down slowly, in menus its fine
In Minecraft 1.12.2 with Optifine HD U F5 using a program called Rainway to play on my phone, the "controller is showned as
"ShanWan ps3/PC wired Gamepad" and it doesnt have any problems with other games.

I believe I'm having a similar issue with my older xbox 360 wireless pad. I have to set a deadzone of at least 0.2 to compensate for the joystick drift (probably could be lower, but the menu cursor has some weird minimum speed that's quite high so a mild drift is amplified tenfold when navigating menus), when I lightly press my joystick in one of the four cardinal directions, the view will move diagonally rather than in that in the direction I'm pressing, however it'll straighten out when I press the joystick further.
The issue is amplified the higher I set the deadzone, if set to 0.9 than the view will go as far as moving left and right when pressing up and down on the joystick, If I set the deadzone to the minimum value than the issue goes away but of course the game becomes basically unplayable with the used controller.

I'm not sure if this is related but I fixed an issue recently with camera moving unpredictable. If this is still an issue on new versions, please open a new issue.

I'm not sure if this is related but I fixed an issue recently with camera moving unpredictable. If this is still an issue on new versions, please open a new issue.


Feature/parse none values

Hey @recog-arch, thanks for your feedback. Could you give an example of what input triggers this? Would this PR also fix your issue? https://github.com/ShreyaR/guardrails/pull/225

Archiving, as #225 may have resolved, please re-open if needed


Issue with k8s.io/docs/editdocs/

This is a...

[ ] Feature Request
[ ] Bug Report

Problem:
Proposed Solution:
Page to Update:
https://kubernetes.io/...

/close


Feature request: Add a condition to a pipeline run to detect completion

Feature request
I'd like to see a new condition on pipelineruns that indicates if they are completed. I'd like this to be boolean. Right now to see if a pipelinerun is complete, i need to check if the Succeeded condition is either True or False. kubectl wait doesn't handle multiple possible values for a condition, it defaults to true or optionally accepts a desired value.
Use case
I'd like to use kubectl wait --for=condition to wait for a pipelinerun to complete. The only condition exposed by tekton is Succeeded. I could use kubectl wait --for=condition=Succeeded --timeout=10m, but if it fails in the first 5 seconds, my code will wait 10 minutes for that wait to timeout (the value in my example), and then i need to see if it failed or if its still running.
For my use case, i had to write my own shell functions to check for completion, but using kubectl wait... is desired. Example shell functions:
tkn_pr_status() {
  namespace=$1
  name=$2
  kubectl -n $namespace get pipelinerun $name -o<EMAIL_ADDRESS>== "Succeeded")].status}'
}

tkn_pr_done () {
  status=$(tkn_pr_status "$@")
  [ "$status" == "True" ] || [ "$status" == "False" ]
}

function tkn_pr_wait {
  # usage: wait_for_pr <timeout_secs> <namespace> <name>
  timeout=$(($1 + $(date +%s)))
  shift

  while :; do
    if [ $(date +%s) -gt $timeout ]; then
        echo "Timeout exceeded waiting for pipeline run to complete"
        return 1
    fi

    if tkn_pr_done "$@"; then
        echo "Pipelinerun has finished"
        return 0
    fi

    echo "Waiting..."
    sleep 30
  done
}

I think this could also be implemented on tkn.
cc @tektoncd/core-maintainers @tektoncd/cli-maintainers

I think this could also be implemented on tkn.
cc @tektoncd/core-maintainers @tektoncd/cli-maintainers

/remove-lifecycle stale
This is a pretty standard way to track completion of jobs/tasks/etc in kubernetes. As an application that runs on kuberenetes, it would be reasonable to expect tekton to work with the native kubernetes tools.
https://kubernetes.io/docs/reference/generated/kubectl/kubectl-commands#wait


The Location Tracker Tutorial: Part 1 document

The document is not completed. User is hard to follow it. The commands are in the document that do not work.
The information should be added as below:

Include Windows OS set up in the document.
Include the tools that need to be installed before creating this application. Have the detail links that to download those tools.
3)The prerequisites should be clear and have a detail. Avoid to send a link to IBM developerWorks for end user.

It looks like this pull request is to merge changes from the master branch into the markerpositionbranch. Instead, I believe you intended to issue a pull request to merge changes from the markerposition branch into master.


AttributeError: 'module' object has no attribute 'encodebytes'

While receiving the mws report data we are getting his error
File "/usr/local/lib/python2.7/dist-packages/mws/mws.py", line 344, in get_report
return self.make_request(data)
File "/usr/local/lib/python2.7/dist-packages/mws/mws.py", line 213, in make_request
parsed_response = DataWrapper(data, response.headers)
File "/usr/local/lib/python2.7/dist-packages/mws/mws.py", line 114, in init
hash_ = calc_md5(self.original)
File "/usr/local/lib/python2.7/dist-packages/mws/mws.py", line 68, in calc_md5
return base64.encodebytes(md.digest()).strip('\n')
AttributeError: 'module' object has no attribute 'encodebytes'
Please help

This looks like you are using a python 3 port of the module but your python interpreter is 2.7.
https://github.com/czpython/python-amazon-mws/blob/master/mws/mws.py#L68

Hi
Yes i am using python 2.7 ,
Regarding : you are using a python 3 port of the module
how to switch to  python 2.7 port from python 3 port
i have just typed pip install mws
How can is resolved this issue .

The repository for your issue is this one https://github.com/jameshiew/mws
You can fix your problem by installing Python 3
or pip install https://github.com/czpython/python-amazon-mws/archive/master.zip which would get you the one from this repository here.

Hi Thanks for your prompt response.
i can't switch to python 3, now and i future  also i have to use python2.7 only due to some framework restriction .
Is the library not  logger work for python 2.7, if yes please tell me any alternative
Is there is a way to run the same in python 2.7

I wrote you how to use the python 2.7 version, that is the one where you opened this issue. When you run pip install mws you'll install a python3 fork of this (https://github.com/jameshiew/mws). An alternative would be boto which has a mws module too: https://github.com/boto/boto and is actively developed.
I might get back into development of this library here too. But I don't have access to an MWS account at the moment.

should i use pip install python-amazon-mws  for python 2.7

No. Use: pip install https://github.com/czpython/python-amazon-mws/archive/master.zip the one on pypi is from 2014.

Thanks


Revert "feat: Drops requiring funds to enable auto-teleport"

Reverts

kodadot/nft-gallery#8547

Context

[x] Opens #8482
[x] Opens #8526

yes it works here

https://deploy-preview-8620--koda-canary.netlify.app/ahp/drops/chroma


[SWS-67] - Service details Endpoint

Here the Service Details endpoint (SWS-67).

It is built under path: "/api/namespaces/{namespace}/services/{service}"
It shows for each service the data specified in SWS-68. Here you have an example:

Tests are going to be implemented in an upcoming PR. This way, I unlock de @aljesusg work with the UI side.

LGFM, Can we merge this and make the changes and tests in other PR? I need this for SWS-69

Are we going to have test in this PR?

@mwringe sure, I am working on it already. But since this is a blocker, we agreed on move on.

@xeviknal not having the test is ok as long as we are tracking that we still need to do the tests. Is it being tracked as a task that needs to be done?

@mwringe just added it. https://issues.jboss.org/browse/SWS-168

If we can get the commits squashed down have it start with the JIRA name, then this LGTM and should be merged :)

With productpage of bookinfo dependencies are /unknown in openshift. Is it normal?
{"name":"productpage","namespace":{"name":"istio-system"},"labels":{"app":"productpage"},"type":"ClusterIP","ip":"<IP_ADDRESS>","ports":[{"name":"http","protocol":"TCP","port":9080}],"endpoints":[{"addresses":[{"kind":"Pod","name":"productpage-v1-1037638640-hv5c2","ip":"<IP_ADDRESS>"}],"ports":[{"name":"http","protocol":"TCP","port":9080}]}],"pods":[{"Name":"productpage-v1-1037638640-hv5c2","labels":{"app":"productpage","pod-template-hash":"1037638640","version":"v1"}}],"route_rules":null,"dependencies":{"v1":["/unknown","/unknown"]}}

I believe 'unknown' is when something is something unknown to istio. So if something from your network is accessing your product page, it would be displayed as 'unknown' since we don't know what exactly is it.

Mmm ok I thought that dependencies of bookinfo sample of istio was completed.

With productpage of bookinfo dependencies are /unknown in openshift. Is it normal?
Yes, indeed if you look into the servicegraph app you would see that the traffic coming from the main ingress router is tagged as "unknown", at least we are using same wording (it comes directly from prometheus).

Perhaps this special case can be labeled different in a next iteration, but that value is expected.

Squash done and added JIRA number on the commit :)

I am merging as @aljesusg approved earlier and this is a squash of the previous work.

@aljesusg , @lucasponce I think in the future would could label "unknown" as "External", or something similar.

I see this is already merged.
I have some questions regarding the example posted in the first PR comment (it might not be relevant anymore?)

It looks like in endpoints section, there is a port and it is also similar to the ports section above it. What is the goal of the port in endpoints section and is it data duplication?
In route rules section the same rule appears twice - is this a bug? or just for the sake of example?

@abonas

tbh, I just followed research conclusions SWS-68. @lucasponce might tell you better.
No, it isn't a bug. I probably got too crazy adding route rules in that service.

Justa note, in the service graph we replace the "unknown" service (sourceService = "") with "internet".  We should come up with a consistent name in the graph and details in Sprint 2.

Justa note, in the service graph we replace the "unknown" service (sourceService = "") with "internet". We should come up with a consistent name in the graph and details in Sprint 2.

@jshaughn it is better to consult someone about this (like the Istio team for instance) and be careful about such replacements.
because it is not necessarily the internet :)
and it might confuse someone who consumes this data in another way (another UI/api/whatever) and would want to compare either manually or automatically ("what is this internet and where is it coming from")

I like 'unknown' here because we don't know what is calling it. I could be something accessing it from the external network, it could be something accessing it locally on the same box, it could even be a pod accessing it that just doesn't hasn't been 'istio-fied' yet.

I'm fine with anything, as I said earlier, istio sets the source service to "", so that's not super helpful.  Vizceral used the "internet" convention, so that is what we have in sprint 1.  Don't really need too much commentary against an initial choice, just need the name you want.


Tell me to install the latest .NET in the application instead of just giving me an error midway through running it without a clue what to do.

Well it says that in the readme and we don't have direct control over the .net errors

You do. You can just try catch it or load dynamically and realize its wrong and then tell me to install latest .NET. :upside_down_face:

Don't recall using the term feature, but alright. Pull requests are always welcome if you find this such a dire bug.

@TedsterTech @Joshua-Ashton

As you said

This isn't even my project.

And as you so politely said:

So go do that.

If that is not your project , why did you reply like that in the first place?
"we don't have direct control over the .net errors"

I'm a contributor and I'm aware of the time limitations of both myself and the ED creator. At this time, solving this issue (that occurs seldomly and is explained in the readme) is not within our current effort's scope. The way that the installer is currently written we don't control the loading process of the .NET dependency (or try catch every usage), that would take implementation changes.
I'm not denying this is an error or calling it a feature as was facetiously suggested, I'm saying that it's a rare error we are not focusing our efforts on. The application itself (ED) is functioning fine and the fix for the installer is explained in the readme for the few encountering the error.

You can easily do 1 big try catch around it or another method I described. This wouldn't take you a lot of time. You objectively do have complete control over these and can use every method I suggested. 🤠

In fact, for the try catch you'd only need to do it upon the first use of a InteropServices (try catch the first function that uses it) and then just show me an error dialog saying what I have to do there.

The code does not currently do that, I think I mentioned that so while it is possible, it doesn't currently control the error flow. I think that was a communication error, as I'm referring to the current code and you seem to be directing it to me:

we don't have direct control over the .net errors

This was meant as we don't have anything in the code handling the errors, as in no controls in place. That's why I followed up by saying that isn't the current loading schema. The only thing we currently have is the note on the readme for the few people that have the error.
Unfortunately, it would take more time than you might think to complete the entire process as the installer would need to be recompiled by the repo owner, after a PR, in order to upload it to the website. The version on the website is actually out of date compared to what is on the repo because he is currently quite busy with other things. Also as I said before, PRs are always welcome. Feel free to use any of the methods you pointed out above, I personally would go with number 4 for better UX than number 1.
Have a nice night!


checkov: 2.0.1209 -> 2.0.1210

Description of changes
https://github.com/bridgecrewio/checkov/releases/tag/2.0.1210

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (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 177495 run on x86_64-linux 1

1 package built:

checkov