Stream-line of tasks with single Env

Motivation and Context
In domains like Continual/Lifelong Learning (or any mult-task context)  it would be convenient to be able to serve multiple tasks without the need to change Env, thus providing an easier interface. This change addresses this use-case building upon the current env-single task abstraction and extending it to env-multiple tasks.
How Has This Been Tested
Added pytest untittest for the proposed functionalities in test_cl_env.py and test_cl_config files. Added an interactive example habitat_cl_example.py.
Types of changes
New feature (non-breaking change which adds functionality).
Added support for task streaming through tasks property to CLEnv as well as task scheduling policy, essentially going from vanilla configuration (still compatible):
[ ... ]
TASK:
  TYPE: Nav-v0
  SUCCESS_DISTANCE: 0.2

  SENSORS: ['POINTGOAL_WITH_GPS_COMPASS_SENSOR']
  POINTGOAL_WITH_GPS_COMPASS_SENSOR:
    GOAL_FORMAT: "POLAR"
    DIMENSIONALITY: 2
  GOAL_SENSOR_UUID: pointgoal_with_gps_compass
[ ... ]

to:
TASKS:
  - TYPE: Nav-v0
    SUCCESS_DISTANCE: 0.169
    TASK_LABEL: MyTask
    SENSORS: ['POINTGOAL_WITH_GPS_COMPASS_SENSOR']
    SUCCESS:
      SUCCESS_DISTANCE: 0.2
    DATASET:  # you can specify your own dataset per task
      TYPE: PointNav-v1
      SPLIT: test
      DATA_PATH: data/datasets/pointnav/habitat-test-scenes/v1/{split}/{split}.json.gz
  - TYPE: Nav-v0
    SUCCESS_DISTANCE: 1.5
    # "global' dataset is used if unspecified
CHANGE_TASK_BEHAVIOUR:
  TYPE: FIXED # or RANDOM
  CHANGE_TASK_PROB: 0.1 # for RANDOM, how likely it is to change task at defined horizon
  AFTER_N_EPISODES: 10   # After how many episodes we want to change task
  AFTER_N_CUM_STEPS: 10 # After how many cumulative steps we want to change task
  LOOP: ORDER # RANDOM for sampling a different task with uniform probability

Checklist

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have completed my CLA (see CONTRIBUTING)
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

self = <multiprocessing.connection.Connection object at 0x7f1232449198>
size = 4, read = <built-in function read>

    def _recv(self, size, read=_read):
        buf = io.BytesIO()
        handle = self._handle
        remaining = size
        while remaining > 0:
            chunk = read(handle, remaining)
            n = len(chunk)
            if n == 0:
                if remaining == size:
>                   raise EOFError
E                   EOFError

Some of the sub processes are crashing when PyTest is run. Does the test suite pass locally with all the relevant datasets downloaded?

Specifically:
       1 failed
         - test/test_habitat_env.py:224 test_rl_vectorized_envs[True]

self = <multiprocessing.connection.Connection object at 0x7f1232449198>
size = 4, read = <built-in function read>

    def _recv(self, size, read=_read):
        buf = io.BytesIO()
        handle = self._handle
        remaining = size
        while remaining > 0:
            chunk = read(handle, remaining)
            n = len(chunk)
            if n == 0:
                if remaining == size:
>                   raise EOFError
E                   EOFError

Some of the sub processes are crashing when PyTest is run. Does the test suite pass locally with all the relevant datasets downloaded?

I wasn't able to reproduce the bug with the newest commit and I'm not quite sure how to affect subprocesses or the test you mentioned as I've essentially only added new lines and tests.
Is there any chance that can be related to some other issue/commit?

Hi @mathfac, thanks a lot for your thorough review. I understand there needs to a series of changes in order to add this contribution to core and I'm totally on board with it, in fact with this PR I mainly wanted to pitch in the idea of supporting multiple tasks (which I'd love to leverage for my thesis) and I'm happy to see you're open for supporting this feature.

Do you have working baselines or the benchmark you already tested it with?
If you're referring to a rl-baseline then no I still didn't train an agent using the multi-task setting but I'm planning to do that asap, I first wanted to put together a few tasks from the already implemented ones and design some meaningful experiment.

As I understand you consider only tasks with same observation and action space right?
I'd like to generalize as much as possible but I'm not sure atm what the best solution for supporting multiple obs/action spaces is, as I was currently more focused on tasks with same obs space. Please do let me know what you think the best approach here is.

Alternative way how we can think about Multi tasking is that mixed Dataset can have Episodes from different Tasks (class), where each episode has a reference to the initialized Task. Then all shuffling and cycling can be done on episodes level using EpisodeIterator capabilities.
I thought about this too when I started implementing the feature but I wasn't too sure about how explicit/visible the tasks would be at this level of abstraction; so I opted for inserting tasks at the highest level of abstraction leaving everything else as it was meant to be when designed. I didn't mind about having multiple EpisodeIterators handling different datasets with possibly different scheduling policies, but I don't have a strong opinion on that.

Regarding performance of the multitask env I thought some overhead was acceptable wrt the single-task environment, but I'll have to setup and run some benchmarks to put in some qualitative figures and maybe reason on top of that. I might have some trouble testing on large-scale distributed settings tho.
I'll make sure to implement the changes you requested as soon as possible.

Ping @erikwijmans


upload hangs

I copy pasted code from download page and ran it but it didn't work upload hangs without showing any progress

I was doing it wrong


GH Action for build and deploy website

Set up a GH Action for building and deploying the website without gh-pages gem. Further updated jekyll in gemfile.
Co-authored-by: @rettinghaus
Co-authored-by: @riedde

Thank you @doerners and co-authors for trying this workflow from https://jekyllrb.com/docs/continuous-integration/github-actions/
Before we go into details of the implementation, seems like the CSS has gone or is not recognized in the build output? See https://doerners.github.io/music-encoding.github.io/

That is true! However, when building the website locally styling get's rendered correctly. For the build on the fork the rendering is there if in the config.yml the style_baseurl is filled. Since some variables refer to that url somehow the site urls aren't build correctly. We couldn't figure out if it just applied to the build process for the fork.

As mentioned in #93, there are reasons for sticking with the Gemfile.lock: https://bundler.io/guides/rationale.html

As mentioned in #93, there are reasons for sticking with the Gemfile.lock: https://bundler.io/guides/rationale.html

Only reason I can imagine is that GitHub dependabot uses them Gemfile.lock to identify dependencies and check them for security issues.

@bwbohl Gemfile.lock is used to determine the exact version of the packages installed. It is needed to exactly reproduce a known working configuration since the Gemfile only gives ranges,

Thanks for adding the lock file back, @doerners

OK, Houston, ready to go :)

@musicEnfanthen Thanks for merging, but the page is not only build with the new action but also still with the old GitHub pages bot. We still have to switch the website to gh-pages branch.

Yes, I already did, thanks @rettinghaus

The latest build run takes the gh-pages branch as source: https://github.com/music-encoding/music-encoding.github.io/actions/runs/3221577452

@bwbohl Gemfile.lock is used to determine the exact version of the packages installed. It is needed to exactly reproduce a known working configuration since the Gemfile only gives ranges,
it also locks all the versions of dependencies too, while the Gemfile only lists the top level gems. It uses a hash too so that you can be fairly certain it would break if someone malicious started mucking about with version tags.

@ahankinson thanks for clarifying, and bringing to my attention that my comment was a bit short and unclear about what I was intending to say, being: I think it would be wise to stick to a Gemfile.lock for exactly those reasons. :-p

Just for clarity: This new process only replaces the build process of the website, meaning Jekyll site is now build via the customized GH action, giving more options to include Jekyll plugins (like e.g., Jekyll Scholar) that are blacklisted when building jekyll sites with Github pages. Nonetheless, we still rely on Github pages for the deployment process, since GH pages delivers the website now (without building it) from the gh-pages branch where it is pre-built with our action.

The build result is being pushed to the branch gh-pages which is the branch that is being deployed, apparently the buildproecss generates the .nojekyll file that you, @rettinghaus, were missing ;-)


Question/Feature: Is it possible to add the varying examples/n from the datasets?

The papers you cited (like Su+ 20, Wang+ 17, Dudik+ 14) all vary the number of examples and repeat the OPE experiments. I am referring to Figure (2a) from Wang+ 17:

Any pointers on how do I go about doing this efficiently in this repo are appreciated!

Hi Kishan! Thank you for interest.
I once implemented that kind of simulation with synthetic data. You can refer to this.
https://colab.research.google.com/drive/1_-kKolkqp5SgwOPKenxgssBMkFrfqEAQ?usp=sharing
Please don't hesitate to reach out, when you have questions! Some package updates will also come soon...


Send jenkins events async

Synchronous http event sending replaced by the async implementation, providing non-blocking way to communite between jenkins and 3rd-party endpoints.

[x] Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
[ ] Ensure that the pull request title represents the desired changelog entry
[x] Please describe what you did
[ ] Link to relevant issues in GitHub or Jira
[ ] Link to relevant pull requests, esp. upstream and downstream changes
[ ] Ensure you have provided tests - that demonstrates feature works or fixes the issue

Good day @JohnNiang !
Please tell about current forkflow. Should someone manually merge the approved branch, or it is an automated process, and all we can do is wait?
Task seems to be stuck at phase #2

https://github.com/jenkinsci/generic-event-plugin/pulls

When will this be released?

When will this be released?

Thank you for your reminder.
I will release a new version containing this  feature in the near future.

Hello, will there be a release?

Hello, will there be a release?

Sorry for the delay. I'm in progress on CD automatically recently.

Hello, will there be a release?

Hi @ant1dot , I'm trying to figure out automated plugin release at #5, please help us review it.

can not find node with label in jenkins: All nodes of label ‘maven-windows’ are offline
https://ci.jenkins.io/blue/organizations/jenkins/Plugins%2Fgeneric-event-plugin/detail/PR-5/1/pipeline

Hi @ant1dot , please check the latest release: https://github.com/jenkinsci/generic-event-plugin/releases/tag/38.v3c1b96fab48f

Hi @ant1dot , please check the latest release: https://github.com/jenkinsci/generic-event-plugin/releases/tag/38.v3c1b96fab48f

thank you very much, we will install and test it soon


Speed up the for  dynamical operation

Hi:
I have a big jpg source file with size of (32768x32768), and I have to extract from the source and generate a new image according to the client request.
Before I found sharp, I use Java2D interfaces like BufferedImage to do the operation, and it will take almost 20s to load the image at the first time, and then the following operation like resize crop will take about 500ms more or less. However it cost almost 1.2GB memory.
When I tried sharp, I found that the same operation like extract resize will take up only 120M memory, this is amazing, however each request will take almost 15s.
Is it possible to improve this?

Extracting multiple regions from the same input image sounds like you might be hand-rolling an image pyramid for some kind of "slippy map". If so, did you see tile?
Please can you provide a sample of the code you are using.

Yes, something like the slippy map, and I know I can  use the tile to pre-generates images, however I want to serve the images only when requested.
core codes like this:
var app= require("express")();
var sharp=require("sharp");
var src = sharp("D:/test.jpg");
app.get("5/:x/:y",function(req,res){
   var x=req.params.x, y=req.params.y;
   src.resize(32768/2,32768/2).extract({left:x*256,top:y*256,256,256}).then(....);
});
app.listener(...)

A compromise between memory usage and processing speed might be to hold smaller compressed versions as singletons in memory, something like:
const test16384 = await sharp("D:/test.jpg")
  .resize(32768 / 2, 32768 / 2)
  .toBuffer();

app.get("5/:x/:y", function (req,res) {
  const { x, y } = req.params;
  sharp(test16384)
    .extract({ left: x * 256, top: y * 256, ... )
    .toBuffer()
    .then( ... );
});

Thanks I got that.


Canvas is loading empty

Not getting the item on the left-side panel to drop on the canvas

Create item using right mouse button

@SamkitSanghvi this is just an example of using the library, not a ready-made editor. what made you think that you can create groups here. Please read the description carefully

Ohh
I thought I can playaround with the demo
same as the call-center DEMO

Yes, you can add nodes to canvas, customize its and etc


Extend project support until 2021

Hi,
I think that cutting off support quickly/completely is not a good idea, as the product can live on long after it has been "replaced" with something new and better. For example, the university where I want to study at still uses NXTs alongside with EV3s. Why should they throw them away if they work and EV3s do have considerable cost? For mapping, AI/ML & LIDAR they might not be the best, but for (basic?) control engineering & acting as physical frontends for more complex stuff they're OK.
It'd also be nice to have some enumeration of what kinds of support this affects:

solving of user problems
accepting new features
accepting any commits
etc.

Best regards,
Jakub

Thanks for the comment, I will review it.

It seems that after all, the successor product has arrived already. The PyBricks implementation is apparently going to run on non-EV3 platforms and it is going to have an unified API over these platforms. https://pybricks.com/downloads/

LEGO MINDSTORMS EV3

ev3dev version is available now
there was even an experimental port to EV3RT (source)

Powered Up / BOOST / Control+ is apparently planned

At this point I understand why LEGO is invested in MicroPython. I don't know about a popular microcontroller-oriented implementation of a JVM that would be actively developed (this excludes NXJ/TinyVM). Also, Java requires compilation to an intermediate bytecode and the final binary. On the other hand, MicroPython:

is actively developed
supports many platforms
comes with a beginner-friendly language
does not require special compilers, just a text editor

Using ev3dev for the OS seems like an independent and good choice - it probably made the development easier for them (rewritten drivers, up-to-date Linux environment) and it also establishes a relationship with the community.

On the other hand, I don't think that EV3 is going to be discontinued/replaced soon - I think that Powered Up is a parallel product branch aiming at a lower price point & easier setup, while being more limited than the EV3 line. However I guess that the next-gen full programmable brick will use the Powered Up style connectors (perhaps with a conversion cable to the old RJ11-style ones).

Interesting (https://github.com/pybricks/pybricks-api):

Each Pybricks firmware comes with the Pybricks package. All modules, classes, methods, and functions in that package have optimized implementations in C for high performance and reduced memory footprint.

They will be able to avoid some sort of "classloading" & warmup for the library code this way.

Sorry @JakubVanek, I don´t understand this comment.
This is a Java project, not a C project.

Sorry @JakubVanek, I don´t understand this comment.
This is a Java project, not a C project.

Yes, however it depends on what your goal is - whether you want to write everything in Java (also a worthwhile goal) or if you just want to provide a Java API.

Sorry @JakubVanek, I don´t understand this comment.
This is a Java project, not a C project.

Yes, however it depends on what your goal is - whether you want to write everything in Java (also a worthwhile goal) or if you just want to provide a Java API.

The idea behind my comment is that by conceding on having everything implemented in a high-level language like Python (and thus giving up easy editing and perhaps introspection/reflection and maybe extensibility; => single language goal), they were likely able to provide a faster load times and higher runtime performance (=> API goal).

Hi @JakubVanek,
I am not sure @JakubVanek.
But this set of comments are not related with issue.

https://www.lego.com/en-dk/product/lego-mindstorms-ev3-31313?fbclid=IwAR1NGikpPazhWfvRh_KGU6mEFWadh7ss_yeNrPsng6mSnUhh_WbVPli-AV0
https://www.stonewars.de/geruechte/lego-mindstorms-51515-snowflake/?fbclid=IwAR1x83lpVqFps98K1jJpITTFTo_zXTjroeQWRau-tZtN8A_3rcoBhzj5FwU

https://www.lego.com/en-us/aboutus/news/2020/june/lego-mindstorms-robot-inventor?fbclid=IwAR3g8wv7MeQaIH61iS7O05kyvh6T8oj829gWfkqSqpXGXJLSFuc0ur3U4Lk


Jinja2 fails to render template with more than 98 if/elif statements

Expected Behavior
No errors on rendering a template.
Actual Behavior
Having the following jinja template (this.jinja2 ) with 99 if/elif statements lines; when the template is rendered it shows the following error:
IndentationError: too many levels of indentation

On the other hand, if one of the elif statements line is removed (keeping 98 elif) the template is rendered correctly without any error.
It seems that there is a hard limit on the number of if/elif statements.
Template Code
this.jinja2
{% if name == 'server0' %}
{% elif name == 'server1' %}
{% elif name == 'server2' %}
{% elif name == 'server3' %}
....
{% elif name == 'server96' %}
{% elif name == 'server97' %}
{% elif name == 'server98' %}
{% endif %}

this.py
import os
import jinja2
def render(tpl_path, context):
    path, filename = os.path.split(tpl_path)
    return jinja2.Environment(
        loader=jinja2.FileSystemLoader(path or './')
    ).get_template(filename).render(context)

print(render('./this.jinja2', {'name': 'test'}))

Full Traceback
Traceback (most recent call last):
  File "this.py", line 9, in <module>
    print(render('./this.jinja2', {'name': 'test'}))
  File "this.py", line 7, in render
    ).get_template(filename).render(context)
  File "/usr/lib/python2.7/dist-packages/jinja2/environment.py", line 791, in get_template
    return self._load_template(name, self.make_globals(globals))
  File "/usr/lib/python2.7/dist-packages/jinja2/environment.py", line 765, in _load_template
    template = self.loader.load(self, name, globals)
  File "/usr/lib/python2.7/dist-packages/jinja2/loaders.py", line 125, in load
    code = environment.compile(source, name, filename)
  File "/usr/lib/python2.7/dist-packages/jinja2/environment.py", line 551, in compile
    return self._compile(source, filename)
  File "/usr/lib/python2.7/dist-packages/jinja2/environment.py", line 512, in _compile
    return compile(source, filename, 'exec')
  File "./this.jinja2", line 499
    if 0: yield None
                   ^
IndentationError: too many levels of indentation

Your Environment

Python version: 2.7.6
Jinja version: 2.7.2-2

This happens because elif actually compiles to else: if ...:. Not sure what's the reason for that (probably there is one though).
Either way, I would recommend you to refactor your code to not have that many elifs - that sounds rather awful to have in the first place (but at least it seems to be generated code, so maybe not that bad)

I might have a fix for it - see the references PR. However, your Jinja version is ancient. This fix would go into 2.9.5

The thing is that I've found this bug using salt configuration management (that uses jinja templating for the states), not writing a python program directly. But of course, a state can be splitted in several files.
Thanks @ThiefMaster


Always allow ˋref_model=Noneˋ

Feature request
For optimisation with reference model, in most cases the reference model is the same as the trained model. We should allow the user to specify the ref model only when they don't want to use the trained model.
Currently this is possible, but only when using PEFT, which is very counter-intuitive. And even using this situation, if you want to provide a ref model that is different from the trained model, you have to define force_use_model. Even more counter-intuitive.
Currently

model = ref_model and no peft

DPOTrainer(model=model, ref_model= ref_model)  # where ref_model should be another instance

model = ref_model and peft

DPOTrainer(model=model)

model != ref_model and no peft

DPOTrainer(model=model, ref_model=ref_model)

model != ref_model and peft

args = DPOConfig(force_use_ref_model=True)
DPOTrainer(model=model, ref_model=ref_model, args=args)

Proposed

model = ref_model

DPOTrainer(model=model)

model != ref_model

DPOTrainer(model=model, ref_model=ref_model)

Motivation
Make the lib use more intuitive.
Your contribution
For sure ;)

Quoting from the other issue:

However, handling ref_model/model is pretty tricky currently, maybe wait until https://github.com/huggingface/trl/issues/2047 is solved?

Is there an explanation for why ref_model and model are tricky? Is there a subtlety here?

I believe that this may be due to the implementation being carried out in multiple stages: first the initial version, followed by PEFT support, then integration with DeepSpeed... It's probably a good time to re-think it as a whole.
However, we must be careful not to introduce any regressions or breaking changes : we must test all the parameter combinations.

In that case, I think it makes sense to just fix the other issue first because the fix for that issue is an equality check, right?

Perhaps you should give it a try. It's difficult to assess the changes involved.

Implemented for Online DPO in #2041. It can probably be taken as reference


Push down collection and partition key range resolution for circuit breaker.

PR Summary

Push down collection resolution and partition key range calls within the retry loop for Document calls in point operation cases - this way if an error code bubbles up as a part of metadata resolution calls - it can be retried as a part of Document calls.
Fix circuit-breaker threshold to mark a partition as Unavailable after exception count equals consecutiveExceptionCountToleratedForReads or consecutiveExceptionCountToleratedForWrites and not when an additional exception is thrown beyond the configured thresholds.
Normalize collection link to not include trailing or leading slashes.
Bubble up 404 Read Session Not Available for PartitionKeyRange resolution - this way these calls can be retried (prior behavior was to throw a NullPointerException or IllegalStateException when the CollectionRoutingMap is not populated).

/azp run java - cosmos - tests

/azp run java - cosmos - tests

/azp run java - cosmos - tests

/azp run java - cosmos - tests

/azp run java - cosmos - tests

/azp run java - cosmos - tests

/azp run java - cosmos - tests

/azp run java - cosmos - tests

/check-enforcer override

Closes #41429


RideScopeType AllTrips missing

The documentation mentions the AllTrips scope type, but it's not defined in RidesScope.swift.

Good eye @EPage-Ed,
Thanks for reporting the incorrect documentation. The AllTrips scope is not currently supported in the SDK, but we are planning to add it soon. Stay tuned!

Fixed in 0.5.0


only export a provider id if the value/namesystem is provided

Pull requests into cqm-parsers require the following. Submitter and reviewer should :white_check_mark: when done. For items that are not-applicable, note it's not-applicable ("N/A") and :white_check_mark:.
Submitter:

[ ] This pull request describes why these changes were made.
[ ] Internal ticket for this PR:
[ ] Internal ticket links to this PR
[ ] Code diff has been done and been reviewed
[ ] Tests are included and test edge cases
[ ] Tests have been run locally and pass

Reviewer 1:
Name:

[ ] Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
[ ] The tests appropriately test the new code, including edge cases
[ ] You have tried to break the code

Reviewer 2:
Name:

[ ] Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
[ ] The tests appropriately test the new code, including edge cases
[ ] You have tried to break the code

Codecov Report

Merging #47 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master     #47      +/-   ##
=========================================
+ Coverage   89.18%   89.2%   +0.01%
=========================================
  Files          68      68
  Lines        1683    1686       +3
=========================================
+ Hits         1501    1504       +3
  Misses        182     182

Impacted Files
Coverage Δ

lib/qrda-export/helper/cat1_view_helper.rb
78.26% <100%> (+1.51%)
:arrow_up:

Continue to review full report at Codecov.

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


PROBLEMS tab still shows problems.

Type: Bug
The "PROBLEMS" tab still shows problems after closing out the file I was working on. I would think that when the file was closed the tab would refresh and there would be no problems showing because there is no file open.
VS Code version: Code 1.77.3 (704ed70d4fd1c6bd6342c436f1ede30d1cff4710, 2023-04-12T09:16:02.548Z)
OS version: Windows_NT x64 10.0.22621
Modes:
Sandboxed: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-8600 CPU @ 3.10GHz (6 x 3096)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
undefined

Memory (System)
15.83GB (7.38GB free)

Process Argv
--crash-reporter-id c464a395-b17a-4805-a83e-77e80f2d2338

Screen Reader
no

VM
0%

Extensions (7)

Extension
Author (truncated)
Version

regionfolder
map
1.0.22

data-workspace-vscode
ms-
0.4.0

mssql
ms-
1.18.0

sql-bindings-vscode
ms-
0.3.0

sql-database-projects-vscode
ms-
0.21.0

notepadplusplus-keybindings
ms-
1.0.7

powershell
ms-
2023.3.3

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscoreces:30445986
pythondataviewer:30285071
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593cf:30376535
pythonvs932:30410667
cppdebug:30492333
vsclangdf:30486550
c4g48928:30535728
dsvsc012:30540252
pynewext54:30695312
azure-dev_surveyone:30548225
nodejswelcome1cf:30587006
3biah626:30602489
pyind779:30671433
89544117:30613380
pythonsymbol12:30671437
6233i204:30672705
azdwalk:30721579
pythonms35:30701012
19089226:30706075
pythonfmttextcf:30716743

Extension which created these problems is supported to handle that. Please file the issue against the extension creating the problems.


Ignore SSL Errors when obtaining configuration

My dev environment, the auth server has an invalid SSL cert. When my client tried to get the OpenId config from https://auth-server/.well-known/openid-configuration I get the following error:
WinHttpException: A security error occurred
System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()

HttpRequestException: An error occurred while sending the request.
System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()

IOException: IDX10804: Unable to retrieve document from: 'https://auth-server/.well-known/openid-configuration'.
Microsoft.IdentityModel.Protocols.HttpDocumentRetriever+<GetDocumentAsync>d__8.MoveNext()

InvalidOperationException: IDX10803: Unable to obtain configuration from: 'https://auth-server/.well-known/openid-configuration'.

Here is how I configure IdentityServer4:
services.AddAuthentication(IdentityServerAuthenticationDefaults.AuthenticationScheme)
    .AddIdentityServerAuthentication(options =>
    {
        options.Authority = authority;
        options.SupportedTokens = SupportedTokens.Jwt;
        options.ApiSecret = secret;
        //change this to true for SSL
        options.RequireHttpsMetadata = false;
    });

Can IdentityServer4 be configured to ignore cert errors in this scenario?

Can IdentityServer4 be configured to ignore cert errors in this scenario?

The fix you're asking for doesn't match the problem you're encountering, as you said yourself here:

When my client tried to get the OpenId config from https://auth-server/.well-known/openid-configuration I get the following error:

Any update on the issue? closing for now - feel free to re-open if it needs further discussion.


Error loading the icon_caption_blip2 model.

I found that the icon_caption_blip2_model on the huggingface cannot be correctly loaded. The code "model = Blip2ForConditionalGeneration.from_pretrained(model_name_or_path, device_map=None, torch_dtype=torch.float16)" get the following errors:
Error details:
      Traceback (most recent call last):
      File "/home/llm/miniconda3/envs/seeclick/lib/python3.8/site-packages/transformers/modeling_utils.py", line 533, in load_state_dict
        return torch.load(
      File "/home/llm/miniconda3/envs/seeclick/lib/python3.8/site-packages/torch/serialization.py", line 1005, in load
        with _open_zipfile_reader(opened_file) as opened_zipfile:
      File "/home/llm/miniconda3/envs/seeclick/lib/python3.8/site-packages/torch/serialization.py", line 457, in __init__
        super().__init__(torch._C.PyTorchFileReader(name_or_buffer))
    RuntimeError: PytorchStreamReader failed reading zip archive: failed finding central directory

    During handling of the above exception, another exception occurred:

    Traceback (most recent call last):
      File "/home/llm/miniconda3/envs/seeclick/lib/python3.8/site-packages/transformers/modeling_utils.py", line 542, in load_state_dict
        if f.read(7) == "version":
      File "/home/llm/miniconda3/envs/seeclick/lib/python3.8/codecs.py", line 322, in decode
        (result, consumed) = self._buffer_decode(data, self.errors, final)
    UnicodeDecodeError: 'utf-8' codec can't decode byte 0x80 in position 128: invalid start byte

    During handling of the above exception, another exception occurred:

    Traceback (most recent call last):
      File "main.py", line 12, in <module>
        caption_model_processor = get_caption_model_processor(model_name_or_path="./OmniParser/icon_caption_blip2/", device=device)
      File "./OmniParser/utils.py", line 46, in get_caption_model_processor
        model = Blip2ForConditionalGeneration.from_pretrained(
      File "/home/llm/miniconda3/envs/seeclick/lib/python3.8/site-packages/transformers/modeling_utils.py", line 3677, in from_pretrained
        ) = cls._load_pretrained_model(
      File "/home/llm/miniconda3/envs/seeclick/lib/python3.8/site-packages/transformers/modeling_utils.py", line 4084, in _load_pretrained_model
        state_dict = load_state_dict(shard_file, is_quantized=is_quantized)
      File "/home/llm/miniconda3/envs/seeclick/lib/python3.8/site-packages/transformers/modeling_utils.py", line 554, in load_state_dict
        raise OSError(
    OSError: Unable to load weights from pytorch checkpoint file for './OmniParser/icon_caption_blip2/pytorch_model-00001-of-00002.bin' at './OmniParser/icon_caption_blip2/pytorch_model-00001-of-00002.bin'. If you tried to load a PyTorch model from a TF 2.0 checkpoint, please set from_tf=True.

I tried different transformers versions and created a new conda env, but the error still occured. I guess it's casued by a corrupted file in the uploaded huggingface model. Would you please take a look at the files?

It seems to be an issue with my local git-lfs. I download the file manually and it works well.


groupby.diff with datetime column

Hi,
I'm trying to convert my Pandas script to Koalas script, and found this error when trying to do GroupBy.diff with datetime column.
Here's the code to reproduce the case:
import databricks.koalas as ks
import datetime as dt
df = ks.DataFrame({'a': [1, 2, 3, 4, 5, 6], 'b': [1, 1, 2, 3, 5, 8], 'c': [dt.date(2019, 1, 1), dt.date(2019, 1, 3), dt.date(2019, 1, 10), dt.date(2019, 1, 11), dt.date(2019, 1, 20), dt.date(2019, 1, 21)]}, columns=['a', 'b', 'c'])
df.groupby(['b'])['c'].diff()

Referring to https://koalas.readthedocs.io/en/latest/reference/api/databricks.koalas.groupby.GroupBy.diff.html
I assume that right now it doesn't support datetime column when diff?
PS. here is the Pandas script which is working properly.
import pandas as pd
import datetime as dt
df = pd.DataFrame({'a': [1, 2, 3, 4, 5, 6], 'b': [1, 1, 2, 3, 5, 8], 'c': [dt.date(2019, 1, 1), dt.date(2019, 1, 3), dt.date(2019, 1, 10), dt.date(2019, 1, 11), dt.date(2019, 1, 20), dt.date(2019, 1, 21)]}, columns=['a', 'b', 'c'])
df.groupby(['b'])['c'].diff()

I think this issue basically due to the lack of calender support in PySpark (SPARK-24695). We might have to work around with long type for now.

@HyukjinKwon , Seems like SPARK-24695 is resolved, can we support calendar in Koalas now ??


feat: MVP Gadget Shell

Adds the basic gadget shell binary, with the following features:

Loads keys from a keystore file (created by tangle or subxt)
Uses subxt to connect to tangle over RPC
Uses the RPC Client to query the chain's runtime API.

Closes #94

CI does not have to be green, since it needs more work to be done by @tbraun96 too.


Add Basic UI

Add Basic UI
In this pull request I did the following:
📌 Create the App class that implements a basic command line interface that let users create person, create books, list persons, list books, create rentals, and show rentals of a person.

Thank you for your review and approval! 🙌🏼


TS2322: Type '~lib/@graphprotocol/graph-ts/common/collections/Bytes' is not assignable to type '~lib/string/String'.

Hi, when I enter the command graph build, I get the following error in my src/token.ts file:

✔ Apply migrations
✔ Load subgraph from subgraph.yaml
  Compile data source: Token => build/Token/Token.wasm
⠋ Compile subgraphERROR TS2322: Type '~lib/@graphprotocol/graph-ts/common/collections/Bytes' is not assignable to type '~lib/string/String'.

   entity.indexedTokenIPFSPath = event.params.indexedTokenIPFSPath
                                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 in src/token.ts(70,33)

ERROR TS2322: Type '~lib/@graphprotocol/graph-ts/common/collections/Bytes' is not assignable to type '~lib/string/String'.

   entity.indexedTokenIPFSPath = event.params.indexedTokenIPFSPath
                                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 in src/token.ts(174,33)

✖ Failed to compile subgraph: Failed to compile data source mapping: 2 compile error(s)
Error: Failed to compile data source mapping: 2 compile error(s)
    at Compiler._compileDataSourceMapping (/usr/local/lib/node_modules/@graphprotocol/graph-cli/src/compiler/index.js:307:13)
    at /usr/local/lib/node_modules/@graphprotocol/graph-cli/src/compiler/index.js:216:20
    at updateInDeepMap (/usr/local/lib/node_modules/@graphprotocol/graph-cli/node_modules/immutable/dist/immutable.js:1971:22)
    at updateInDeepMap (/usr/local/lib/node_modules/@graphprotocol/graph-cli/node_modules/immutable/dist/immutable.js:1980:23)
    at updateInDeepMap (/usr/local/lib/node_modules/@graphprotocol/graph-cli/node_modules/immutable/dist/immutable.js:1980:23)
    at Map.updateIn (/usr/local/lib/node_modules/@graphprotocol/graph-cli/node_modules/immutable/dist/immutable.js:1278:26)
    at /usr/local/lib/node_modules/@graphprotocol/graph-cli/src/compiler/index.js:215:24
    at /usr/local/lib/node_modules/@graphprotocol/graph-cli/node_modules/immutable/dist/immutable.js:3016:46
    at List.__iterate (/usr/local/lib/node_modules/@graphprotocol/graph-cli/node_modules/immutable/dist/immutable.js:2206:13)
    at mappedSequence.__iterateUncached (/usr/local/lib/node_modules/@graphprotocol/graph-cli/node_modules/immutable/dist/immutable.js:3015:23)
UNCAUGHT EXCEPTION: Error: The AssemblyScript compiler crashed when compiling this file: 'src/token.ts'
Suggestion: try to comment the whole file and uncomment it little by little while re-running the graph-cli until you isolate the line where the problem happens.
Also, please contact us so we can make the CLI better by handling errors like this. You can reach out in any of these links:
- Discord channel: https://discord.gg/eM8CA6WA9r
- Github issues: https://github.com/graphprotocol/graph-cli/issues

Any ideas? Thanks!

Hi!  That suggests that the type of the event parameter is different to the type of the field on the entity. You should make them the same, or transform the event parameter accordingly. If you are still having issues, Discord is the best place to get help (in #subgraph-development)
Discord channel: https://discord.gg/eM8CA6WA9r


/admin/default-hooks/gitea/new 500

Description

verversion: 1.18.0+rc1
Gitea Version
1.18.0+rc1
Can you reproduce the bug on the Gitea demo site?
Yes
Log Gist
No response
Screenshots
No response
Git Version
No response
Operating System
No response
How are you running Gitea?
docker run -it \
--name=gitea \
-p443:3000 \
-p<IP_ADDRESS>:2222:22 \
-e USER_UID=1001 \
-e USER_GID=1000 \
-v `pwd`/public:/data/gitea/public \
-v `pwd`/templates:/data/gitea/templates \
-v `pwd`/data:/data/gitea \
-v `pwd`/log:/data/gitea/log \
-v `pwd`/custom/conf/app.ini:/data/gitea/conf/app.ini \
-v /home/git/.ssh/:/data/git/.ssh \
gitea/gitea:latest

Database
SQLite

It would be helpful if the provided logs were as text rather than a screenshot, as the screenshot has slightly cut off the salient part of the error message.
From the bit I can tell there is a template error with some property HeaderAuthorization being missing on the some struct provided to the template (likely a struct named hook given the context)

https://github.com/go-gitea/gitea/blob/998fe26051514ee74bb9c2f0e7de56014933133b/templates/repo/settings/webhook/settings.tmpl#L254

There is no HeaderAuthorization in 1.18 - you're using a template from 1.19/main in 1.18 and that is why you have this problem.
Yes, I got it , you are right。thinks

by the way , gitea/gitea:latest ，  Is it 1.18 actually ?

The docker tag gitea/gitea:latest is the latest release or release-candidate. gitea/gitea:dev represents the last successful build from the main branch.
In the templates, you're correct that {{AppUrl}} will be the set-up application url.
BTW you can use the gitea embedded command to extract any built-in templates rather than risk downloading the incorrect version for your current version of gitea.


Feature/railties 7

⚠ DO NOT RELY ON THIS BRANCH FOR DEPLOYMENT ⚠

Hi there @tagliala !
We just wanted to notify you that we're using your PR to use active admin with rails 7.
If you make force pushes, it'll mess up on our side, but I guess this is okay since this is some sorte of edge development. We're watching this PR on our side to get notifications and update the bundle when needed.
Do you have any plans/schedule to work on the failing tests to hopefully get this merged into active admin main repo ?

If you make force pushes, it'll mess up on our side

Sorry to hear that, but please make a fork and do not rely on this PR for deployed applications, it is highly experimental and I will push force when needed

Do you have any plans/schedule to work on the failing tests to hopefully get this merged into active admin main repo ?

Please take a look at activeadmin/activeadmin#7235, there are some blocking dependencies upstream and a known issue with autoload that I do not know how to fix

@tagliala I am getting this issue with this branch, how to fix it?

just run bundle update activeadmin again. I'm force-pushing to this branch, so the old reference is not available anymore
Please also do not rely on this branch for deployments

just run bundle update activeadmin again. I'm force-pushing to this branch, so the old reference is not available anymore
Please also do not rely on this branch for deployments

@tagliala then what is the solution to use activeadmin with rails 7? Please suggest.

@sachin-amzur you can fork this repo and point to a branch you have control on


Autocompletion got "bad option: -t" in zsh

Environmentz
macOS 10.15.7 (19H2)
zsh 5.8 (x86_64-apple-darwin19.6.0)
docker-ls version: 0.5.0 (git:0eb4c9a) (go1.15.5 darwin amd64)
Error message
docker-ls
source <(docker-ls autocomplete bash)

stdout:
/dev/fd/14:type:795: bad option: -t

docker-rm
source <(docker-rm autocomplete bash)

stdout
/dev/fd/14:type:569: bad option: -t

I'm confused.  Do you use this with zsh or bash?  Since you take the output of autocomplete bash (instead of zsh) and use it with zsh (at least both the version number as well as issue subject tell so) !?

@stesie I'm so sorry...
It's my mistake!!!
Long story short.
There's a bug in my rc script for both bash & zsh.
And it leads me to take the wrong args as the current shell.
So, case closed! Thanks!


Allow arbitrary host/port/encryption

Ticket

Purpose

Changes

Side effects

QA Notes

Deployment Notes

Yep that's true, its related to s3-compat.
Replacing the calls with client library is because the signature stuff in boto2 is delusional. the signature stuff in boto3 works as intended.
It is going to be synchronous as written now, but there's two options -
1. Just wrap it in a await loop.run_in_executor()
2. There's an asyncio wrapper around boto3


Allow encoding spaces as %20 instead of +

I ran into an issue with a parser that doesn't replace + with spaces, it might be a good idea to have a to_string function that accepts additional arguments regarding url encoding (like a config struct).
Thanks!

serde_urlencoded doesn't do the encoding itself, it just delegates to url::form_urlencoded.
You can use url::form_urlencoded::parse_with_encoding and then serde_urlencoded::Deserializer::new.


fix(ApplicationCommandOptions): ensure toJSON return is valid

Please describe the changes this PR makes and why it should be merged:
Small mistake in number options made max and minValues to not get properly passed to api snake_case properties. This PR fixes that and also adds tests t
Status and versioning classification:

Code changes have been tested, or there are no code changes
I know how to update typings and have done so, or typings don't need updating
This PR includes breaking changes (methods removed or renamed, parameters moved or removed) ⭐ (technically this was private properties that are now also public...so it's not breaking per say)

Codecov Report

Merging #69 (695a88d) into main (481e1f8) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main       #69   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           21        21
  Lines          306       306
  Branches        42        42
=========================================
  Hits           306       306

Impacted Files
Coverage Δ

...actions/slashCommands/mixins/NameAndDescription.ts
100.00% <ø> (ø)

src/interactions/slashCommands/options/integer.ts
100.00% <100.00%> (ø)

src/interactions/slashCommands/options/number.ts
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Replace BackAndroid with BackHandler

<EMAIL_ADDRESS>deprecated the BackAndroid component in favor of BackHandler which has the same api.
See deprecation notice here
This pr replaces usage of BackAndroid with BackHandler, as it shows warnings for projects running 0.44

Will also need to add a notice in the README for those using older versions of react-native, since BackHandler is only available 0.44^

@iRoachie Hey man, thanks for that. It means it won't work with older RN versions?

@xotahal Yea. As I said you'll have to put in the readme a note, telling them which version to download based on their react-native version.

@xotahal Would you like me to make that note in the README? It'd have to be another PR though, i mistaked and deleted my fork :D

Hey man :) Actually I don't really like the idea we're not going to support older RN just because of this BackAndroid.
We could use this for now: https://github.com/react-community/react-navigation/pull/1520/files
What do you think?

Oooooo that is nice lol I definitely prefer that
—
Kyle Roach
On 22 May 2017, 8:25 AM -0400, Jiří Otáhal<EMAIL_ADDRESS>wrote:

Hey man :) Actually I don't really like the idea we're not going to support older RN just because of this BackAndroid.
We could use this for now: https://github.com/react-community/react-navigation/pull/1520/files
What do you think?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub (https://github.com/xotahal/react-native-material-ui/pull/139#issuecomment-303085289), or mute the thread (https://github.com/notifications/unsubscribe-auth/AFr89k02kbCssxrL8iyEPSkjBPU-EF7yks5r8X7HgaJpZM4NaRdz).


STONEINTG-661: Fix golangci reported issues

Check commits for details
Maintainers will complete the following section

[ ] Commit messages are descriptive enough (hints)
[ ] Code coverage from testing does not decrease and new code is covered (check the PR coverage on codecov)
[ ] Controllers diagrams are updated when PR changes controllers code  (if applicable)

/test integration-service-e2e

/test integration-service-e2e

/test integration-service-e2e

/test integration-service-e2e

/test integration-service-e2e

/test integration-service-e2e

/test integration-service-e2e

rebased

/test integration-service-e2e


CLI: Show grid initial nodes health

With the Kontena IPAM based on etcd, you will not be able to deploy services to a grid with fewer initial nodes created/connected than required for an etcd quorum.
This PR adds grid nodes configuration health checks to the CLI.

Show the configured initial_size in kontena grid current  / kontena grid show

Improve kontena node ls to show the initial node states

Add kontena node health to show the connected state for a given node

Add kontena grid health to show disconnected and missing initial nodes

See  #1470 for server/agent health checks
Example
Creating a new  --initial-size=3 grid one node at a time.
Starting from an empty grid, the kontena node ls is empty, and kontena grid health is red:

Adding the first node, the kontena node ls and kontena grid health are red:

Adding the second node, the kontena node ls and kontena grid health are yellow:

Adding the third node, the kontena node ls and kontena grid health are green:

Adding the fourth node, everything is green:

If there are fewer initial nodes connected than configured for the grid, but still a sufficient majority for etcd quorum, show a warning health status as etcd operations will still succeed:

If there are fewer initial nodes connected than required for etcd quorum, then show an error health status as etcd operations will fail:

This should probably be included in 1.0.x, because it's a CLI-only fix, and it's really easy to forget what the initial-nodes configuration/state for the grid actually is.
The agent-side stuff in #1470 could then be for 1.1.0

Improved the kontena grid health messages; CBA to update the screenshots; from the specs

Grid does not have any initial nodes, and requires at least 1 of 1 nodes for operation
Grid only has 1 initial nodes, and is not high-availability
Grid only has 1 initial nodes, and is not high-availability
Grid has all 1 of 1 initial nodes connected
Grid only has 1 of 2 initial nodes required for operation
Grid only has 2 initial nodes, and is not high-availability
Grid has all 2 of 2 initial nodes connected
Grid only has 1 of 2 initial nodes required for operation
Grid only has 2 of 3 initial nodes required for high-availability
Grid only has 2 of 3 initial nodes required for high-availability
Grid only has 2 of 3 initial nodes required for high-availability
Grid has all 3 of 3 initial nodes connected

Squashed onto git 1.1.dev master.

Ruby <2.3 rspecs failing on <<~END syntax.

Added some specs for the kontena node ls show functions.

PTAL @jakolehm @jnummelin

Where did that version column come? :)

Where did that version column come? :)

#996


Readme mention $(ipython locate)

Did you ment jupyter --config-dir?

I guess I did! I've been referring to mostly @minrk's IPython extensions and @parente's tutorial which references using ipython locate.
I'm bad with the new command line args which is no good but I'll update this.

Sure, but most extension are old, and i it's good habit to (try to) give good example when creating new extensions. Otherwise you will crawl ender support requests :-P

Indeed — thanks for pointing out the issue. Let me know if anything else looks old.

Indeed — thanks for pointing out the issue. Let me know if anything else looks old.

the manual copying looks odd. This would be a good case to try https://github.com/jupyter/notebook/pull/879 and make installation activation easier.

Yeah, I was hoping to do that after the 4.2 release.


add komodo logo

:star: PR built and available in a preview environment chrishankh-abc-pr-4 here

:star: PR built and available in a preview environment chrishankh-abc-pr-4 here


change hostpath to configmap

Description
The summary of the proposed changes as long as the relevant motivation and context.
Issues
https://github.com/opea-project/GenAIInfra/issues/39
Type of change
List the type of change like below. Please delete options that are not relevant.

[ x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would break existing design and interface)

Dependencies
N/A
Tests
UT passed

glad to have a config map!


Gateway: Combining HTTPS listener with TLS-termination and TLS listener with TLS-passthrough

Describe the bug:
I'm trying to create a Gateway where I use both a HTTPS listener with a certificate provided by Cert-manager, and a TLS listener with TLS-passthrough.
apiVersion: gateway.networking.k8s.io/v1
kind: Gateway
metadata:
  name: test
  namespace: gateway
  annotations:
    cert-manager.io/issuer: cloudflare-issuer
spec:
  gatewayClassName: cilium
  listeners:
    - protocol: HTTPS
      port: 443
      name: https-gateway
      hostname: "*.example.com"
      tls:
        certificateRefs:
          - kind: Secret
            name: test-cert
      allowedRoutes:
        namespaces:
          from: All
    - protocol: TLS
      port: 443
      name: proxmox-tls-passthrough
      hostname: "proxmox.example.com"
      tls:
        mode: Passthrough
      allowedRoutes:
        namespaces:
          from: All

When I add the TLS listener the Gateway becomes unresponsive for all HTTPRoutes and TLSRoutes connected to it.
The event log for the the Gateway states:
Skipped a listener block: [spec.listeners[1].tls.certificateRef: Required value: listener has no certificateRefs, spec.listeners[1].tls.mode: Unsupported value: "Passthrough": supported values: "Terminate"]

Expected behaviour:
I expect the Gateway to work with both listeners.
Cert-manager should allow/ignore the TLS listener running in Passthrough mode.
Steps to reproduce the bug:
Create the above Gateway.
Anything else we need to know?:
Cilium 1.15.1 provides the GatewayClass.
I initially believed this to be a Cilium-issue, but with further investigation it looks to be an issue with  Cert-manager.
A workaround is to create two Gateways, each with their own listener. Alternatively route the HTTPS listener Gateway Service through to the TLS listener Gateway to only expose one LoadBalancer IP.
Environment details::

Kubernetes version: 1.29.3
Cloud-provider/provisioner: Bare metal
cert-manager version: 1.14.4
Install method: Kustomize + Helm

# kustomization.yaml
helmCharts:
  - name: cert-manager
    repo: https://charts.jetstack.io
    version: 1.14.4
    includeCRDs: true
    releaseName: cert-manager
    namespace: cert-manager
    valuesFile: values.yaml

# values.yaml
installCRDs: true

config:
  apiVersion: controller.config.cert-manager.io/v1alpha1
  kind: ControllerConfiguration
  featureGates:
    ExperimentalGatewayAPISupport: true

/kind bug

Hey @vehagn thanks for raising. Where does this log come from?

Skipped a listener block: [spec.listeners[1].tls.certificateRef: Required value: listener has no certificateRefs, spec.listeners[1].tls.mode: Unsupported value: "Passthrough": supported values: "Terminate"]

Does the TLS listener prevent the Certificate being created for the HTTPS listener in the single file example?

Thanks for picking up the issue @hawksight,
The log comes from the events when you run kubectl describe on the Gateway resource.
I’m unsure of what you mean by “single file example.” I think the certificate is successfully created for the HTTPS-listener, it could be from before adding the TLS-listener (I can double check later).
My main issue is that HTTPRoutes connected to the Gateway becomes unresponsive, meaning that I can’t access the Services behind them.
I tried to solve the issue in the linked PR, but haven’t built an image of the branch and tested it fully.

"single file example" I meant this YAML:
apiVersion: gateway.networking.k8s.io/v1
kind: Gateway
metadata:
  name: test
  namespace: gateway
  annotations:
    cert-manager.io/issuer: cloudflare-issuer
spec:
  gatewayClassName: cilium
  listeners:
    - protocol: HTTPS
      port: 443
      name: https-gateway
      hostname: "*.example.com"
      tls:
        certificateRefs:
          - kind: Secret
            name: test-cert
      allowedRoutes:
        namespaces:
          from: All
    - protocol: TLS
      port: 443
      name: proxmox-tls-passthrough
      hostname: "proxmox.example.com"
      tls:
        mode: Passthrough
      allowedRoutes:
        namespaces:
          from: All

I was trying to understand if cert-manager was actually preventing the Gateway from working, or if that is a Gateway concern. I'll look at the PR more closely to understand the change.
Can you share how you have your Gateway installed? Is it via the standard YAML & CRDs or via a particular project that implements the GatewayAPI?

Thanks for the clarification @hawksight.
I've done some more testing which I try to explain in detail below.
The testing leads me to believe the connectivity issues might be linked to Cilium Issue #32371. Though I still think the BadConfig Warning message I've attempted to fix in PR #6986 for TLSRoutes in Passthrough mode is an improvement.

I'm fetching the Gateway CRDs from https://github.com/kubernetes-sigs/gateway-api/releases/download/v1.0.0/experimental-install.yaml. I'm using the experimental install since I want to use the Gateway spec.infrastructure.annotations field to explicitly set the Gateway Service IP. I've omitted this field in the test Gateway.
The full configuration can be found at https://gitlab.com/vehagn/mini-homelab
Doing a new test I'm first creating a Gateway without the Cert-manager annotation
apiVersion: gateway.networking.k8s.io/v1
kind: Gateway
metadata:
  name: test
  namespace: gateway
#  annotations:
#    cert-manager.io/issuer: cloudflare-issuer
spec:
  gatewayClassName: cilium
  listeners:
    - protocol: HTTPS
      port: 443
      name: https-gateway
      hostname: "*.stonegarden.dev"
      tls:
        certificateRefs:
          - kind: Secret
            name: test-cert
      allowedRoutes:
        namespaces:
          from: All
    - protocol: TLS
      port: 443
      name: proxmox-tls-passthrough
      hostname: "proxmox-test.euclid.stonegarden.dev"
      tls:
        mode: Passthrough
      allowedRoutes:
        namespaces:
          from: All

and a TLSRoute
apiVersion: gateway.networking.k8s.io/v1alpha2
kind: TLSRoute
metadata:
  name: test
  namespace: proxmox
spec:
  parentRefs:
    - name: test
      namespace: gateway
  hostnames:
    - "proxmox-test.euclid.stonegarden.dev"
  rules:
    - backendRefs:
        - name: proxmox-euclid
          port: 443

I'm now able to reach proxmox-test.euclid.stonegarden.dev through the Gateway.

Interestingly both listeners report one attached route.
The I comment out the Cert-manager annotation again
#  annotations:
#    cert-manager.io/issuer: cloudflare-issuer

and create a HTTPRoute for the Gateway.
apiVersion: gateway.networking.k8s.io/v1
kind: HTTPRoute
metadata:
  name: http-route
  namespace: whoami
spec:
  parentRefs:
    - name: test
      namespace: gateway
  hostnames:
    - "https-test.stonegarden.dev"
  rules:
    - matches:
        - path:
            type: PathPrefix
            value: /
      backendRefs:
        - name: whoami
          port: 80

I now get ERR_CONNECTION_RESET when trying to access https-test.stonegarden.dev. The TLSRoute endpoint proxmox-test.euclid.stonegarden.dev still works.
The HTTPRoute status indicates that it should work.
status:
  parents:
    - conditions:
        - lastTransitionTime: '2024-05-09T22:28:45Z'
          message: Accepted HTTPRoute
          observedGeneration: 2
          reason: Accepted
          status: 'True'
          type: Accepted
        - lastTransitionTime: '2024-05-09T22:28:45Z'
          message: Service reference is valid
          observedGeneration: 2
          reason: ResolvedRefs
          status: 'True'
          type: ResolvedRefs
      controllerName: io.cilium/gateway-controller
      parentRef:
        group: gateway.networking.k8s.io
        kind: Gateway
        name: test
        namespace: gateway

and the Gateway report two routes attached to the HTTPS-listener
status:
  addresses:
    - type: IPAddress
      value: <IP_ADDRESS>
  conditions:
    - lastTransitionTime: '2024-05-09T22:28:36Z'
      message: Gateway successfully scheduled
      observedGeneration: 11
      reason: Accepted
      status: 'True'
      type: Accepted
    - lastTransitionTime: '2024-05-09T22:28:36Z'
      message: Gateway successfully reconciled
      observedGeneration: 11
      reason: Programmed
      status: 'True'
      type: Programmed
  listeners:
    - attachedRoutes: 2
      conditions:
        - lastTransitionTime: '2024-05-09T22:28:36Z'
          message: Listener Programmed
          observedGeneration: 11
          reason: Programmed
          status: 'True'
          type: Programmed
        - lastTransitionTime: '2024-05-09T22:28:36Z'
          message: Listener Accepted
          observedGeneration: 11
          reason: Accepted
          status: 'True'
          type: Accepted
        - lastTransitionTime: '2024-05-09T22:28:36Z'
          message: Resolved Refs
          reason: ResolvedRefs
          status: 'True'
          type: ResolvedRefs
      name: https-gateway
      supportedKinds:
        - group: gateway.networking.k8s.io
          kind: HTTPRoute
    - attachedRoutes: 1
      conditions:
        - lastTransitionTime: '2024-05-09T22:28:36Z'
          message: Listener Programmed
          observedGeneration: 11
          reason: Programmed
          status: 'True'
          type: Programmed
        - lastTransitionTime: '2024-05-09T22:28:36Z'
          message: Listener Accepted
          observedGeneration: 11
          reason: Accepted
          status: 'True'
          type: Accepted
        - lastTransitionTime: '2024-05-09T22:28:36Z'
          message: Resolved Refs
          reason: ResolvedRefs
          status: 'True'
          type: ResolvedRefs
      name: proxmox-tls-passthrough
      supportedKinds:
        - group: gateway.networking.k8s.io
          kind: TLSRoute

Then I uncomment the Cert-manager annotation again
  annotations:
    cert-manager.io/issuer: cloudflare-issuer

And I can still connect to the TLSRoute endpoint, but not the HTTPRoute endpoint.
Deleting the Gateway and waiting for Argo CD to recreate it the TLSRoute endpoint also stops working.
Deleting the Gateway and waiting for Argo CD to recreate again it the TLSRoute endpoint now works again.
Deleting and recreating the gateway appears to continue this flip-flop pattern.
Cert-manager diligently reattaches the certificate it created earlier each time.

/reopen

/remove-lifecycle rotten


Shulker shell with affix

really not sure how this happened

It happened naturally, I'm not really sure if it generated in a box or dropped from a mob though

This happens because convert rules are active and a mod (supplementaries i think?) makes shulker shells into wearable helmets.
You can blacklist either the shulker loot table or remove the equipment type from shells via config.  I may look into making that a default config option.

Shulker shells are now set to type NONE by default.


'npm link' not working ?

as npm link not working, every time I am publishing the package to check a minimal changes.
am I the only person facing this problem ?
please help me to fix this issue . .

So I ran into this on my development machine running Windows 10/npm 6.4.1.  The root of the evil appears to be if your library (lets call it 'new-library') depends on another library via 'npm link some-library'.  When you run 'npm link' for 'new-library', the existing link to 'some-library' gets destroyed, and the library does not compile.
I hacked around this by:

Removing the 'prepare' script from the package.json
Making 'some-library' a peer dependancy of 'new-library' (not sure if this was actually needed)
After this, npm link would run without destroying the link to 'some-library'.  Just make sure that 'new-library' is built, and it should work.

TSDX doesn't touch any internals like npm link, so whatever the issue is here (no repro), it's unrelated to TSDX.
I've also personally used npm link with libraries written in TSDX with no issues and as have other users.

I hacked around this by:

I haven't had this problem so I can't really comment, don't quite know what the problem is there. Just for reference, I don't normally use a prepare script and peer deps are the way Lerna works as well for monorepos.


Use webauthn plugin from a frontend

Frontend demo: https://github.com/JohnGuilding/webauthn-plugin-demo


facebook dropped an enhanced coco_wholebody json with face - 308 keypoints vs 133 in standard

https://github.com/facebookresearch/sapiens/issues/76
https://github.com/facebookresearch/sapiens/blob/main/assets/keypoints308.gif
https://github.com/facebookresearch/sapiens/tree/main
any plans to upgrade the pose to use this?
if you could share any pre-processing code that could help - may give it a crack when have free time / gpu bandwidth.

I am not going to be upgrading this to the Sapiens work but contributions are welcome!
Regarding pose pre-processing, they are just transformed to this T2PSample dataclass defined here: https://github.com/clement-bonnet/text-to-pose/blob/main/t2p/t2p_sample.py#L27


In v0.7.6, the gotls module works exceptionally in pie mode on x64 platform.

Describe the bug
In v0.7.6, the gotls module works exceptionally in pie mode on x64 platform.
In v0.7.5, gotls works fine with this example.
Screenshots
v0.7.6:

v0.7.5:

Linux Server/Android (please complete the following information):

Env:

---------------------------------------
eCapture Makefile Environment:
---------------------------------------
PARALLEL                 16
----------------[ from args ]---------------
CROSS_ARCH
ANDROID                  0
DEBUG                    0
---------------------------------------
UNAME_M                  x86_64
UNAME_R                  6.5.0-26-generic
CLANG_VERSION            12
GO_VERSION               1.21
---------------------------------------
CMD_CLANG                clang
CMD_GIT                  git
CMD_GO                   go
CMD_INSTALL              install
CMD_LLC                  llc
CMD_MD5                  md5sum
CMD_PKGCONFIG            pkg-config
CMD_STRIP                llvm-strip
---------------------------------------
VERSION                  0.7.6-20240330-3486f5b
LAST_GIT_TAG             0.7.6-20240330-3486f5b
BPF_NOCORE_TAG           6_5_0-26-generic.0_7_6-20240330-3486f5b
CROSS_COMPILE
KERN_RELEASE             6.5.0-26-generic
KERN_BUILD_PATH          /lib/modules/6.5.0-26-generic/build
KERN_SRC_PATH            /lib/modules/6.5.0-26-generic/build
TARGET_ARCH              x86_64
GOARCH                   amd64
LINUX_ARCH               x86
LIBPCAP_ARCH             x86_64-unknown-linux-gnu
AUTOGENCMD               test -f kern/bpf/x86/vmlinux.h || bpftool btf dump file /sys/kernel/btf/vmlinux format c > kern/bpf/x86/vmlinux.h
---------------------------------------
rpmdev-setuptree         rpmdev-setuptree
tar                      tar
rpmbuild                 rpmbuild
---------------------------------------

OS: Ubuntu 22.04
Arch: x86_64
Kernel Version: 6.5.0
Version: 0.7.6

Additional context
Add any other context about the problem here.

fixed by #516

I did a simple test of v0.7.6 and PR#516 in an x64 Ubuntu 22.04 environment.
The test was to execute the command sudo . /bin/ecapture gotls --elfpath=/usr/bin/dockerd --hex while executing docker login in another terminal.
The result shows that eCapture was able to correctly find the crypto/tls.(*Conn).Read symbol entry, but was unable to locate the RET command:

At the same time, docker login returns a connect error:

I think the connect error issue probably has nothing to do with the hook position. The hook here is looking for the RET instruction to hook, so it shouldn't affect network communication. There must be another reason, maybe it's just a sporadic network failure.

It looks like eCpture is indeed affecting docker's network communication.
Normally docker login should return Error response from daemon: Get "https://registry-1.docker.io/v2/": unauthorized: incorrect username or password, however eCapture intervenes and returns an EOF error, while docker pull also returns an EOF error.
The test is relatively simple. I'm not sure if eCapture hooking other golang projects (buildmode=pie) would cause the same thing.

fixed at #516 and #517


Remove ConfigMap sections

For https://issues.redhat.com/browse/DPTP-4113
To be merged after https://github.com/openshift/ci-tools/pull/4391
/hold

/lgtm

/unhold


Enable running of check test files through iree-run-mlir/iree-run-module

Now that everything is in the check test framework it has become impossible to run the files through normal iree-run-mlir:
unable to resolve VM function 'check.expect_eq'

This should work, either by registering the check module or by turning the check ops into no-ops. Debugging is now much more difficult as two tools need to be run instead of one.
The easiest fix is to add a pass/or translation flag to strip check dialect ops entirely.

We also should allow the return of arbitrary values and print them out. Trying to adapt one of these tests right now to run under iree-run-mlir causes it then to not run under iree-check-test. Printing out things is useful in tests (like logging in a gunit test).

I'd like to understand a bit more about the use case here. What's better about iree-run-mlir foo vs iree-translate foo | iree-check-module - or just running the test through the build system? I've never really liked iree-run-mlir: it feels like a frankentool that's trying to do too many different things (and ends up containing a weird stitching together of absl and llvm as a consequence). Or is the complaint that for model examples you use iree-run-mlir (or iree-run-module) but now for unit tests you need a different tool?
Regarding returning from functions, I deliberately made all the test functions void. I think that's standard for test frameworks. At least the ones I've used (across languages) never have the test function returning anything and I'm worried about allowing it because it's not meaningful for the test itself. It seems like what you really want is the ability to print out values, right? Returning them from a function just causes iree-run-mlir to print them. What about just adding an op for it? I think an iree.print op wouldn't be hard to add as part of a custom module. I was expecting that that would be coming out of the strings module by this point, but I think maybe it's taking a more principled approach of actually having string types whereas we could easily take a shortcut and just hardcode how to print buffer views. That said, if you're running it through iree-check-module you should either know what the value is (because there's an expect) or it should be printing out the value (because the expect fails).

Renamed - this isn't about iree-run-mlir specifically but also iree-run-module/iree-benchmark-module. Today the checked in models are unfortunately check tests, meaning that someone can't simply compile and run iree/test/e2e/models/ tests through iree-benchmark-module, for example.

(as part of packaging and deployment we want to be able to provide models that users can use for running, benchmarking, and correctness checking to isolate system issues from their models)

Sounds like https://github.com/google/iree/pull/8916 fixed this by turning check ops into no-ops (via optional imports). We could still replace the check module with built-in support for comparisons and asserts though (that's tracked in https://github.com/google/iree/issues/5854).


Update mdoc, sbt-mdoc to 2.2.20

Updates

org.scalameta:mdoc
org.scalameta:sbt-mdoc

from 2.2.14 to 2.2.20.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta" } ]

labels: library-update, semver-patch

Superseded by #171.


Fix outdated comments for unknown type

Just fixing (make them up-to-date) comment lines for #30637 which changes default generic type parameter constraint from {} to unknown.

Thanks!


Error when setting monolog.level

Can't find why this falls with error ( Fatal error: Class 'Logger' not found in 'blabla' ):
$app->register(new MonologExtension(), array(
    'monolog.class_path'    => __DIR__.'/vendor/monolog/src',
    'monolog.logfile'       => __DIR__.'/development.log',
    'monolog.name'          => 'cars',
    'monolog.level'         => Logger::INFO
));

Without monolog.level it works pretty good.

This is the sort of issue that should be reflected in the docs.

Composer is the long term solution to this, see http://nelm.io/blog/2011/12/composer-part-1-what-why/


[Snyk] Security upgrade nginx from 1.19-alpine to 1.26.0-alpine

This PR was automatically created by Snyk using the credentials of a real user.![snyk-top-banner](https://github.com/andygongea/OWASP-Benchmark/assets/818805/c518c423-16fe-447e-b67f-ad5a49b5d123)
Snyk has created this PR to fix 5 vulnerabilities in the dockerfile dependencies of this project.
Keeping your Docker base image up-to-date means you’ll benefit from security fixes in the latest version of your chosen image.
Snyk changed the following file(s):

packages/playground/Dockerfile

We recommend upgrading to nginx:1.26.0-alpine, as this image has only 8 known vulnerabilities. To do this, merge this pull request, then verify your application still works as expected.
Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Out-of-bounds Read SNYK-ALPINE313-APKTOOLS-1533754
  714

Double Free SNYK-ALPINE313-CURL-1585246
  714

Incorrect Default Permissions SNYK-ALPINE313-CURL-2938009
  714

Buffer Overflow SNYK-ALPINE313-OPENSSL-1569448
  714

Out-of-bounds Write SNYK-ALPINE313-ZLIB-2976175
  714

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Double Free
🦉 Incorrect Default Permissions

Closed as duplicated of https://github.com/threefoldtech/tfgrid-sdk-ts/pull/3166/files#diff-6331ba2480002168e9f557759316e8ddfe4a8aeaa30a7a0d2190760ed4753eb6R24


JWT token not valid, refreshing (forever)

Getting this constantly in logs. Known good credentials for Envoy. Also tried with generated token to no avail (same error). What am I doing wrong?

Wrong credentials data in config?

Still the issue ?

Yep - confirmed still with known good credentials.

But it’s working after refresh, get me the log please.

Please post the log.

Hello @grzegorz914,
Thank you for creating this plugin. I am having the same issue.
Here is my system info:
IQ Gateway SKU: X-IQ-AM1-240-3
Firmware: D7.3.621
I have homebridge set up inside of a docker container.
I have my envoy system set up wirelessly. envoy.local works properly, and my credentials give me proper access to my system. I have a very basic setup of the plug-in, with only my email, password, and S/N provided to the plug-in - as well as support for  Envoy with firmware v7.x.x and newer enabled.
Here are my logs for v9.5.2-beta.9. Please let me know if you need anything else to troubleshoot this issue.
P.S. The plugin-in UI asks for the system password twice - this may be a visual bug.
[11/27/2024, 9:47:23 PM] [homebridge-enphase-envoy] Restarting child bridge...
[11/27/2024, 9:47:23 PM] Got SIGTERM, shutting down child bridge process...
[11/27/2024, 9:47:28 PM] [homebridge-enphase-envoy] Child bridge process ended
[11/27/2024, 9:47:28 PM] [homebridge-enphase-envoy] Process Ended.

Please put up address instead host please and let me know

Where ask for password twice, please post the screenshot please.

displays duplicate credentials fixed in 9.5.2, try and let me know

Thank you for the quick response. The visual bug is fixed.

Here is my attempt with a defined IP address, and login credentials provided:
[11/28/2024, 10:43:04 AM] [homebridge-enphase-envoy] Restarting child bridge...
[11/28/2024, 10:43:04 AM] Got SIGTERM, shutting down child bridge process...
[11/28/2024, 10:43:09 AM] [homebridge-enphase-envoy] Child bridge process ended
[11/28/2024, 10:43:09 AM] [homebridge-enphase-envoy] Process Ended. Code: 143, Signal: null
[11/28/2024, 10:43:16 AM] [homebridge-enphase-envoy] Restarting Process...
[11/28/2024, 10:43:17 AM] [homebridge-enphase-envoy] Launched child bridge with PID 17449
[11/28/2024, 10:43:18 AM] [homebridge-enphase-envoy] Loaded homebridge-enphase-envoy v9.5.2 child bridge successfully
[11/28/2024, 10:43:18 AM] [homebridge-enphase-envoy] Device: <IP_ADDRESS> Home Enphase, JWT Token expired, refreshing.
[11/28/2024, 10:43:50 AM] [homebridge-enphase-envoy] Device: <IP_ADDRESS> Home Enphase, Error: Start error: Error: Check JWT token error: Get JWT token error: Refresh token error: Error: Login to Enlighten error: Request failed with status code 401, trying again.
[11/28/2024, 10:44:03 AM] [homebridge-enphase-envoy] Device: <IP_ADDRESS> Home Enphase, JWT Token expired, refreshing.
[11/28/2024, 10:44:33 AM] [homebridge-enphase-envoy] Device: <IP_ADDRESS> Home Enphase, Error: Start error: Error: Check JWT token error: Get JWT token error: Refresh token error: Error: Login to Enlighten error: Request failed with status code 401, trying again.
[11/28/2024, 10:44:48 AM] [homebridge-enphase-envoy] Device: <IP_ADDRESS> Home Enphase, JWT Token expired, refreshing.
[11/28/2024, 10:45:18 AM] [homebridge-enphase-envoy] Device: <IP_ADDRESS> Home Enphase, Error: Start error: Error: Check JWT token error: Get JWT token error: Refresh token error: Error: Login to Enlighten error: Request failed with status code 401, trying again.
[11/28/2024, 10:45:33 AM] [homebridge-enphase-envoy] Device: <IP_ADDRESS> Home Enphase, JWT Token expired, refreshing.
[11/28/2024, 10:46:03 AM] [homebridge-enphase-envoy] Device: <IP_ADDRESS> Home Enphase, Error: Start error: Error: Check JWT token error: Get JWT token error: Refresh token error: Error: Login to Enlighten error: Request failed with status code 401, trying again.

Here is my attempt with a defined IP address, and valid token provided:
[11/28/2024, 10:58:41 AM] [homebridge-enphase-envoy] Restarting child bridge...
[11/28/2024, 10:58:41 AM] Got SIGTERM, shutting down child bridge process...
[11/28/2024, 10:58:46 AM] [homebridge-enphase-envoy] Child bridge process ended
[11/28/2024, 10:58:46 AM] [homebridge-enphase-envoy] Process Ended. Code: 143, Signal: null
[11/28/2024, 10:58:53 AM] [homebridge-enphase-envoy] Restarting Process...
[11/28/2024, 10:58:54 AM] [homebridge-enphase-envoy] Launched child bridge with PID 18197
[11/28/2024, 10:58:55 AM] [homebridge-enphase-envoy] Loaded homebridge-enphase-envoy v9.5.2 child bridge successfully
[11/28/2024, 10:58:55 AM] [homebridge-enphase-envoy] Device: <IP_ADDRESS> Home Enphase, Cookie refresh success.
[11/28/2024, 10:59:11 AM] [homebridge-enphase-envoy] Device: <IP_ADDRESS> Home Enphase, Error: Start error: Error: Update power production state error: Request failed with status code 401, trying again.
[11/28/2024, 10:59:40 AM] [homebridge-enphase-envoy] Device: <IP_ADDRESS> Home Enphase, Cookie refresh success.
[11/28/2024, 10:59:56 AM] [homebridge-enphase-envoy] Device: <IP_ADDRESS> Home Enphase, Error: Start error: Error: Update power production state error: Request failed with status code 401, trying again.
[11/28/2024, 11:00:25 AM] [homebridge-enphase-envoy] Device: <IP_ADDRESS> Home Enphase, Cookie refresh success.
[11/28/2024, 11:00:38 AM] [homebridge-enphase-envoy] Device: <IP_ADDRESS> Home Enphase, Error: Start error: Error: Update power production state error: Request failed with status code 401, trying again.

Also to note, when trying to switch from credentials to token, homebridge reported: "Error failed to save config". This issue only arises when switching directly from Generated Token to Enlighten Credentials, and vice versa. When None is selected first, saved, and then switched to the new desired Token Generation Mode, this issue does not happen.
My steps in reproducing are:
reset to the default json file configuration for the plugin (no credentials data provided)
Setup the plugin using the UI for either token or credentials
Save the config, and allow the plugin to start
Edit the plugin config, go to credentials data, and change the Token generation mode to the other mode( do not select none)
Enter the new Token generation credentials(or token).
Save - > gives error

Check your credentials data first, error 401 is wrong user or password.

When I try to sign in to the website https://enlighten.enphaseenergy.com/login/login, I get a json response "message: Too many active sessions..". Could it be that running the plugin overnight left too many sessions running, without closing them? I am using the same credentials as yesterday.

I am unsure of how I could close my active sessions so that I can troubleshoot again.

The issue with config save cannot be reproduced on my side, all is saved correct.
Looks like you have temporary account locked, wait little and try again.

Im pushed 9.5.3 wit some small changes, try and let me know

9.5.3 Worked to an extent. I am still unable to log in to my account due to the active session issue, but the plugin was able to read some information with the provided token. Attached is my log
homebridge-enphase-envoy.log.txt


Error in log.py while updating modules to new info messages structure

Got this error while adding new info to modules
Traceback (most recent call last):
  File "/usr/lib/python2.7/multiprocessing/process.py", line 267, in _bootstrap
    self.run()
  File "/usr/lib/python2.7/multiprocessing/process.py", line 114, in run
    self._target(*self._args, **self._kwargs)
  File "/root/OWASP-Nettacker/core/attack.py", line 95, in start_attack
    verbose_level, socks_proxy, retries, methods_args, scan_id, scan_cmd)
  File "/root/OWASP-Nettacker/lib/vuln/xdebug_rce/engine.py", line 160, in start
    'CATEGORY': "scan", 'SCAN_ID': scan_id, 'SCAN_CMD': scan_cmd}, log_in_file, language)
  File "/root/OWASP-Nettacker/core/alert.py", line 95, in info
    __log_into_file(thread_tmp_filename, "w", "0", language)
  File "/root/OWASP-Nettacker/core/log.py", line 219, in __log_into_file
    submit_logs_to_db(language, data)
  File "/root/OWASP-Nettacker/database/db.py", line 163, in submit_logs_to_db
    host=log["HOST"], date=log["TIME"], port=log["PORT"], type=log["TYPE"], category=log["CATEGORY"],
TypeError: 'int' object has no attribute '__getitem__'

Hi,
thanks for reporting, here is a little more details, this problem is coming from __log_into_file() when the condition lead the code to submit_logs_to_db(language, data) and the data is not JSON structure (for instance: __log_into_file(thread_tmp_filename, 'w', '1', language) and thread_tmp_filename is not True in this condition if _builder(_paths(), default_paths())["tmp_path"] in filename:).
def __log_into_file(filename, mode, data, language, final=False):
    """
    write a content into a file (support unicode) and submit logs in database. if final=False its writing log in
    the database.

    Args:
        filename: the filename
        mode: writing mode (a, ab, w, wb, etc.)
        data: content
        language: language
        final: True if it's final report otherwise False (default False)

    Returns:
        True if success otherwise None
    """
    if version() is 2:

        if _builder(_paths(), default_paths())["tmp_path"] in filename:
            if not final:
                flock = lockfile.FileLock(filename)
                flock.acquire()
            with open(filename, mode) as save:
                save.write(data + '\n')
            if not final:
                flock.release()
        else:
            if final:
                with open(filename, mode) as save:
                    save.write(data + '\n')
            else:
                submit_logs_to_db(language, data)

    else:

        if _builder(_paths(), default_paths())["tmp_path"] in filename:
            if not final:
                flock = lockfile.FileLock(filename)
                flock.acquire()
            with open(filename, mode, encoding='utf-8') as save:
                save.write(data + '\n')
            if not final:
                flock.release()
        else:
            if final:
                with open(filename, mode, encoding='utf-8') as save:
                    save.write(data + '\n')
            else:
                submit_logs_to_db(language, data)
    return True

@Ali-Razmjoo, should I write a try catch in the database code to give erroneous message if this is done?

I think we need a better solution, maybe we can check if the data type is JSON/event or not?


Small improvement for standard usecase

Just used trait in read command instead of concrete class. It is more common use case.

Bug bankruptcy


Fix problem with int size during KGX merge step

Trying to fix this error during merge in Jenkins (see this stackoverflow ticket)
16:48:31  Traceback (most recent call last):
16:48:31    File "run.py", line 166, in <module>
16:48:31      cli()
16:48:31    File "/var/lib/jenkins/workspace/dge-graph-hub_kg-covid-19_master/gitrepo/venv/lib/python3.7/site-packages/click/core.py", line 1137, in __call__
16:48:31      return self.main(*args, **kwargs)
16:48:31    File "/var/lib/jenkins/workspace/dge-graph-hub_kg-covid-19_master/gitrepo/venv/lib/python3.7/site-packages/click/core.py", line 1062, in main
16:48:31      rv = self.invoke(ctx)
16:48:31    File "/var/lib/jenkins/workspace/dge-graph-hub_kg-covid-19_master/gitrepo/venv/lib/python3.7/site-packages/click/core.py", line 1668, in invoke
16:48:31      return _process_result(sub_ctx.command.invoke(sub_ctx))
16:48:31    File "/var/lib/jenkins/workspace/dge-graph-hub_kg-covid-19_master/gitrepo/venv/lib/python3.7/site-packages/click/core.py", line 1404, in invoke
16:48:31      return ctx.invoke(self.callback, **ctx.params)
16:48:31    File "/var/lib/jenkins/workspace/dge-graph-hub_kg-covid-19_master/gitrepo/venv/lib/python3.7/site-packages/click/core.py", line 763, in invoke
16:48:31      return __callback(*args, **kwargs)
16:48:31    File "run.py", line 84, in merge
16:48:31      load_and_merge(yaml, processes)
16:48:31    File "/var/lib/jenkins/workspace/dge-graph-hub_kg-covid-19_master/gitrepo/kg_covid_19/merge_utils/merge_kg.py", line 33, in load_and_merge
16:48:31      merged_graph = merge(yaml_file, processes=processes)
16:48:31    File "/var/lib/jenkins/workspace/dge-graph-hub_kg-covid-19_master/gitrepo/venv/lib/python3.7/site-packages/kgx/cli/cli_utils.py", line 486, in merge
16:48:31      stores = [r.get() for r in results]
16:48:31    File "/var/lib/jenkins/workspace/dge-graph-hub_kg-covid-19_master/gitrepo/venv/lib/python3.7/site-packages/kgx/cli/cli_utils.py", line 486, in <listcomp>
16:48:31      stores = [r.get() for r in results]
16:48:31    File "/usr/lib/python3.7/multiprocessing/pool.py", line 657, in get
16:48:31      raise self._value
16:48:31  multiprocessing.pool.MaybeEncodingError: Error sending result: '<kgx.source.graph_source.GraphSource object at 0x7f2acfe59c50>'. Reason: 'error("'i' format requires -2147483648 <= number <=<PHONE_NUMBER>")'

Close so I can rename a broken Jenkins pipeline - problem with Jenkin not being able to clean workspace


Support categories= argument for pyarrow

[x] Tests added / passed
[x] Passes flake8 dask
[x] Fully documented, including docs/source/changelog.rst for all changes
and one of the docs/source/*-api.rst files for new API

Depends on https://github.com/apache/arrow/pull/1620 (should be part of the upcoming 0.9 release)
fastparquet -> pyarrow roundtrip fails due to https://github.com/dask/fastparquet/issues/305

Any way this can be made to be backwards compatible with older versions of pyarrow? I'm fine with either:

Erroring nicely if categories are provided (preferably at graph build time) if an older version of pyarrow is used.
Backporting this functionality to work even with pyarrow 0.8.0.

The first one is probably easier, not sure. Up to you.

@jcrist I went the long way and implemented three paths. Each one being the fastest for the given PyArrow version. While the one for 0.8 is quite lengthy, it should be much faster (mostly due to less memory copies) than the one that falls back to Pandas.

Travis failure is due to a job exceeding the maximal execution time.

Thanks @xhochy!


Strip menu bug electric boogaloo: Strip menu opening when trying to wake sleeping players

About the PR

Not sure if #14272 brought this with it or if it was there before. But it now should not jumpscare you with the strip menu if you try to wake someone up when clicking on them.
Not sure if this this should have a changelog but I'm adding one. Tell me to remove it if you don't think it needs it
Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl: Vasilis

fix: Strip menu won't open when trying to wake sleeping players.

wait shit i forgot to merge my old fix


Modalities CT/MR and more structures ?

Two questions :

What should be done to adapt the TS to MRI scans in stead of CT ?
Could you add subcortical structures (thalamus, caudate, putamen, pallidum, hippocampus, amygdala, and nucleus accumbens) please ?

Thanks again for this tool !

Two questions :

What should be done to adapt the TS to MRI scans in stead of CT ?
Could you add subcortical structures (thalamus, caudate, putamen, pallidum, hippocampus, amygdala, and nucleus accumbens) please ?

Thanks again for this tool !

I would also like to use the tool for MRI axial slices, at minimum to segment the vertebrae.  I am trying out preprocessing of the MRI images to where the bones would be easier for the TS model to "see".
I can work with you and any others on this.
The TS tool is great! So far it has segmented the vertebrae on CT axial slices very well for our use case. @wasserth we appreciate your contribution to the community!

We are currently working on adding support for MRI images and for subcortical structures. But it will take a bit more time.


ava.lang.NoSuchMethodError when building the SQL files in hapi-fhir-jpaserver-base maven install

I'm suddenly getting this error when building a new version of the hapi-fhir-jpaserver-base jar:
[ERROR] Failed to execute goal de.juplo:hibernate4-maven-plugin:1.0.5:export (o10g) on project hapi-fhir-jpaserver-base: Execution o10g of goal de.juplo:hibernate4-maven-plugin:1.0.5:export failed: An API incompatibility was encountered while executing de.juplo:hibernate4-maven-plugin:1.0.5:export: java.lang.NoSuchMethodError: org.hibernate.cfg.beanvalidation.TypeSafeActivatorAccessor.applyRelationalConstraints(Ljavax/validation/ValidatorFactory;Ljava/util/Collection;Ljava/util/Properties;Lorg/hibernate/dialect/Dialect;)V
I've changed no code.

Hi @pogo61 ,
Weird! That type of error has showed up before when we've changed Hibernate versions and not kept the hibernate4-maven-plugin up to date with the changes but I wouldn't expect it could just happen randomly.
Are you using the latest master codebase from Git, or a branch?
Attach a debug log from the build of that module to this bug?
e.g.:
cd hapi-fhir-jpaserver-base
mvn -X install > build.log

Hi James,
I'm using the latest from the Git trunk.
I managed to remove the error by:
Creating a new project in Eclipse and deleting and re-creating the local maven repository.
You can close this, but I think it was an eclipse/maven issue.
PP
From: James Agnew <notifications<EMAIL_ADDRESS>Reply-To: jamesagnew/hapi-fhir <reply<EMAIL_ADDRESS>Date: Monday, June 1, 2015 at 11:25 PM
To: jamesagnew/hapi-fhir <hapi-fhir<EMAIL_ADDRESS>Cc: Paul Pogonoski <paulpog<EMAIL_ADDRESS>Subject: Re: [hapi-fhir] ava.lang.NoSuchMethodError when building the SQL files in hapi-fhir-jpaserver-base maven install (#177)
Hi @pogo61https://github.com/pogo61 ,
Weird! That type of error has showed up before when we've changed Hibernate versions and not kept the hibernate4-maven-plugin up to date with the changes but I wouldn't expect it could just happen randomly.
Are you using the latest master codebase from Git, or a branch?
Attach a debug log from the build of that module to this bug?
e.g.:
cd hapi-fhir-jpaserver-base
mvn -X install > build.log

Reply to this email directly or view it on GitHubhttps://github.com/jamesagnew/hapi-fhir/issues/177#issuecomment-107461772.

Thanks for letting me know, glad it worked out.
-James


Masking in sequential model

Hi there,
I am working on a seq 2 seq regression (every embedding in the input sequence goes to a real value). Since my input length varies I am using masking. I was wondering if masking needs to be done only at the input layer or do I need to have masking at the output as well? Here is my model specification:
    # Model definition
    self.model = Sequential()
    # to remove items from the shorter sequences than the max length
    self.model.add(Masking(5.000000, input_shape=(max_length, self.embedding_mat.shape[1])))
    self.model.add(LSTM(hidden_size, return_sequences=True))
    self.model.add(TimeDistributed(Dense(1,activation="relu")))
    self.model.compile(loss='mse', optimizer='adadelta',metrics=["mse"])

Thank you for your help.

#957


Resolves cannot read property 'accessToken' of undefined error while connecting to a cloud DB

This PR fixes #13936
Solution context:
The "Cannot read property 'accessToken' of undefined" error was occurring because the refreshToken method can return undefined when the handleInteractionRequired method determines that there isn't a need for additional user interactions to authenticate a login.
In the event that a firewall rule is missing or the database is in a paused state, the following error message will be seen and the database will need to be checked in the Azure portal.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.002%) to 42.226%

Files with Coverage Reduction
New Missed Lines
%

extensions/machine-learning/src/common/processService.ts
2
81.48%

Totals

Change from base Build<PHONE_NUMBER>:
-0.002%

Covered Lines:
27785

Relevant Lines:
61562

💛  - Coveralls


Remove compiled python files

This commit removes all the .pyc files, since there's no point in
versioning binary files. It also adds an entry to .gitignore to avoid
pyc files being compiled in the future. See
https://docs.python.org/release/1.5.1p1/tut/node43.html

My two cents
I hope to see a HackerCafe in Brazil soon! :)


Fix switch keys being ignored when a Switch contains a flag

See #95
Added 3 tests - one for existing behavior, and 2 new tests for the bug I refer to.

Thanks for the pull request!
✅ Reported a bug
✅ Wrote tests
✅ Opened a PR
This is the best kind of contribution! 😍


Fix component removal

This won't delete absolutely every resource created for each component. As we learned in the first go around we have some lookups with registers, that end up not happening resulting if failed tasks for undefined variables, and other situations that could make a full cleanup for each component quite tedious.
What this PR will do is delete routes, services, deployments, daemonsets, replicasets, and pods, which I believe covers the most important aspect: getting rid of pods for an unwanted component to free up resources.

This should fix https://github.com/fusor/mig-operator/issues/24


sc.exe requires space after the = sign - Server 2008 R2 Enterprise

sc.exe requires spaces after the = sign
Correct Syntax
sc.exe create WindowsAzureGuestAgent binpath= "C:\WindowsAzure\Packages_[date]_[time[\GuestAgent
\WindowsAzureGuestAgent.exe" start= auto DisplayName= "Windows Azure Guest Agent"
Current Syntax
sc.exe create WindowsAzureGuestAgent binpath="C:\WindowsAzure\Packages_[date]_[time[\GuestAgent
\WindowsAzureGuestAgent.exe" start=auto DisplayName="Windows Azure Guest Agent"
Work around is to manually run the command either with the correct path, or perform a repair after the service is registered. Only then the agent will update itself and report.

We encountered this same problem, confirmed by inspecting the C:\WindowsAzure\Logs\TransparentInstaller.log file. Seems to be present in version 2.7.41491.1018.
Our approach was to uninstall the old agent version, and then install 2.7.41491.1044, which completed successfully without errors. Agent service is now present, and VM Agent reports as ready.
Some of the affected Server 2008 R2 VMs had a working Windows Azure VM Guest agent prior to this problem. Pure speculation at this point: we're questioning if the auto-update feature grabbed the version with the invalid SC.exe syntax and put those machines into this state.
Either way, uninstalling the version ending in 1018 and directly installing the latest version resolved the problem in all the cases we ran into.


fix(e2e): assign nonce manually for parallel transactions (#539)

Closes #539
Checklist:

[x] I have performed a self-review of my own code
[ ] I have documented my code in accordance with the documentation guidelines
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[ ] I have run the unit tests
[x] I only have one commit (if not, squash them into one commit).
[x] I have a descriptive commit message that adheres to the commit message guidelines

Please review the CONTRIBUTING.md file for detailed contributing guidelines.

Closing this PR because it forks from a different PR still in review.


Add a section on ternary operators

Does this seem sane?
Do we need more sections on other boolean expressions of any sort?

Perhaps make explicit never nest them i.e. x == 1 ? a : x == 2 ? b : c

This is already our style; not sure why we didn't write it down sooner. Thanks for adding!

This good to merge now? Any other thoughts?


rename themes modules with underscore ?

That should allow purely declarative imports, as things are string IPython should be able to auto import them correctly.
It might be possible to have a shim that does the right things. I can give it a try for the shim.

The base16 builder generates dashed names by spec. I decided to go low
tech, but you prefer to write some kind of import hook, don't you? Can that
be done in a totally transparent way or should the hook be installed first
(which would be not much better than importing a string)?


Spec should make a list of all things that a particular target should precisely specify

I'm going to mark this as stalled. But feel free to re-open if you feel it should be actively discussed.