Hey Kevin

A little refactoring

Thanks so much, Sergei!! This is a great improvement. I only had 1 hour yesterday to publish this gem, and wasted most of that time trying to get the embedded rails app working in the specs. This weekend I go to a spiritual seminar, but I hope next week I will have more time to polish this gem. Anyways, you seem to have fixed all those things reasonably well. Super-Awesome! :)
Here is an idea: include Straides into ApplicationController by default, and add the ability to disable that behavior in an initializer file (config/initializers/straides.rb). I think that makes more sense than forcing people to manually include Straides into every controller of theirs. What do you think about that?
Cheers,
Kevin

include Straides into ApplicationController by default, and add the ability to disable that behavior in an initializer file (config/initializers/straides.rb)

From the first look, sounds like "use 'unless not' instead of 'if'. Lemme rethink this...
I have an app:

./controllers

application_controller.rb
./ admin

base_controller.rb
posts_controller.rb

./ api

base_controller.rb
posts_controller.rb

./ dev

base_controller.rb
posts_controller.rb

./ mobile

base_controller.rb
posts_controller.rb

./ web <-- Scope, not a namespace

base_controller.rb
posts_controller.rb

Each controller folded into namespace or scope (in terms of routes).
I do not want to use Straides in admin, dev and api namespaces (i.e. api responds with custom error message in json format).
But I wanna use it in my web scope and mobile namespace:
app/controllers/web/base_controller.rb:```
class Web::BaseController < ApplicationController
include Straides
end

_app/controllers/mobile/base_controller.rb:_```
class Mobile::BaseController < ApplicationController
  include Straides
end

And everything I need is included into "base controllers" and inherited in "posts controllers". I do not need "to manually include Straides into every controller".
My initial thought here: do not split your app into initializers and controllers if you deal only with controllers (leads to unexpected behaviour of your app).
In case you want to exclude Straides from some of your controllers under the same namespace - you should rethink your confessions in this life :) Do you agree?
Have a good spiritual weekend, Kevin. I need to do the same :)

From another hand, it is not a problem for people at all, since Straides uses its own error class.

Hey Sergei, the weekend was perfect! I achieved an empty mind in the spiritual workshop, released the Straides gem as Open Source, and dated a girl from Japan!
I don't understand your last messages. If you don't want to use Straides in some controllers, then simply don't use it, i.e. simply don't call "error" in them. Including it by default in ApplicationController doesn't harm anything, since Straides doesn't change any of the default behavior of Rails.
I think in 99% of the cases people want to just include the Straides gem and then use the "error" method, without having to do any more setup. Therefore it is a reasonable default behavior to activate Straides by default, i.e. include it in ApplicationController. For bigger projects there might be a need to configure Straides, and for those cases there is an initializer file where people can override the default settings with their own custom settings. Examples are to disable the automatic inclusion in ApplicationController, provide a different name for the "error" method etc.
I think that's the Rails way! Have an opinionated framework that makes reasonable default choices, which can be configured if one needs custom behavior.

I don't understand your last messages. If you don't want to use Straides in some controllers, then simply don't use it, i.e. simply don't call "error" in them. Including it by default in ApplicationController doesn't harm anything, since Straides doesn't change any of the default behavior of Rails.

Sure. My last comment was about it :)

Therefore it is a reasonable default behavior to activate Straides by default, i.e. include it in ApplicationController.

Why do we talk about ApplicationController? You mean ActionController::Base?

Examples are to disable the automatic inclusion in ApplicationController, provide a different name for the "error" method etc. I think that's the Rails way! Have an opinionated framework that makes reasonable default choices, which can be configured if one needs custom behavior.

That makes sence, thanks Kevin
Lets continue our work! :)

Why do we talk about ApplicationController? You mean ActionController::Base?

Sergei, thank you so much for your guidance. This has put me in the right direction to figure out how to implement configuration for this gem!
I'm sending you a separate review request.
Cheers,
Kevin


Add JohnnyData

SpongeAPI | SpongeCommon
Also makes it less focused on vindicators in-case other mobs add johnny functionality later on or another implementation does.

Johnny is ready to join bleeding if no one has any feedback on the API or implementation.


Chore:Upgrade to maven wrapper

Maven Wrapper is an excellent choice for projects that need a specific version of Maven (or for contributors that don't want to install Maven at all). Its tech improvement so raising it as chore.

I don't see the immediate need for this. Looking at https://github.com/googleapis/google-cloud-java, they don't seem to be using it either. Developers can use most Maven 3.x distributions to build and test this repo. If that changes, we can reconsider this.


On Android calendars are return in a map <key, value>

Hello,
On Android (device or simulator) in my application when I retrieve the calendar list I always have a map (key-value) and not a list of calendars. Work's fine on Ios.
When I'm using the sample app, I have not this problem.

Maybe a setting android of my projet, but which one ? It's not a problem of permission.
Thx for help
Using device_calendar: 3.1.0, I've also tried 2.0.0

How are you retrieving calendars? Are you using retrieveCalendars in device_calendar.dart? That output doesn't seem like a calendar at all.

Yes that's what i did.
The example projet is working fine, so i decided to take all src files inside my project.
And it doesn't work, same problem
I tried also to find an android setting that could be different and the root cause but i still dont find.
I've also tried tu used the same version of gradle...Same problem.

I found the problem, but i can't explain it.
The plugin doesn't work when we have 2 buildTypes in the build.gradle in the project
buildTypes {
release {
signingConfig signingConfigs.release
...
}
/debug {
....
}/
}


Restructure internal entity organization

Add an extra layer for markets to the data structure holding blocks and entities such that blocks with same library and name but different markets can exists together in the development zone/download path and cache.

Solution is merged from major-refactor.


Empty documents fields of same image with data on studio

Package:

azure-ai-formrecognizer==3.3.0

OS:

Manjaro Linux
Ubuntu 22

Python 3.8.19
Description:
I used a document in Document Intelligence Studio and correctly got the data back.
Using the same document via the API or the Python SDK, the fields are not identified.
To Reproduce

Take a PNG image;
Use via Studio and successfully obtain the fields recognized with key/value;
Use the same via Python SDK and have an empty result for the fields.

Expected behavior
Same result between Studio and SDK.
Screenshots
Both results were using the same image
Expected behavior successfully executed by Studio

Unexpected behavior successfully executed by Studio. Missing "Fields"

Used Snippet
# import libraries
import os
from azure.core.credentials import AzureKeyCredential
from azure.ai.formrecognizer import DocumentAnalysisClient
import base64

with open("IMAGE_PATH", "rb") as image_file:
  readed_image = image_file.read()

endpoint = os.getenv('AZURE_ENDPOINT')
key = os.getenv('SUBSCRIPTION_KEY')

def analyze_invoice():
    document_analysis_client = DocumentAnalysisClient(
      endpoint=endpoint,
      credential=AzureKeyCredential(key)
    )

    poller = document_analysis_client.begin_analyze_document(
        model_id="prebuilt-healthInsuranceCard.us",
        document= readed_image
    )

    hci_us = poller.result()

    if(hci_us == None):
      return
    if hci_us.documents:

        for idx, az_document in enumerate(hci_us.documents):
          # Here the problem for any number of documents

          az_document.fields # EMPTY
          az_document.fields.get("Copays") # EMPTY, any key

if __name__ == "__main__":
    analyze_invoice()

Hi @ArthurEgide - thanks for opening an issue! We'll take a look asap!

Hello @swathipil, just adding information...
Even when I use the demonstrative image, the values are empty.
poller = document_analysis_client.begin_analyze_document_from_url(
  model_id="prebuilt-healthInsuranceCard.us",
  document_url='https://raw.githubusercontent.com/Azure-Samples/cognitive-services-REST-api-samples/master/curl/form-recognizer/rest-api/insurance-card.png'
)

Hi @ArthurEgide ,

Can you provide the API version and endpoint you're using in Studio? This would help me to investigate the issue in SDK.
I notice you're using azure-ai-formrecognizer, we have rebranded it to azure-di-documentintelligence. New APIs and features support will only be available in the new package, it's recommended to switch to the new one. You can follow this doc for migration: https://github.com/Azure/azure-sdk-for-python/blob/main/sdk/documentintelligence/azure-ai-documentintelligence/MIGRATION_GUIDE.md

Thanks


[main] Upgrade to latest dependencies

Cron -knative-prow-robot
/cc knative-sandbox/eventing-writers
/assign knative-sandbox/eventing-writers
Produced by: knative-sandbox/knobots/actions/update-deps
Details:
bumping knative.dev/eventing d7a8a95...1096fc2:
  > 1096fc2 Update actions (# 6397)
  > c3e1396 Propagate log level to env var in source ConfigWatcher (# 6391)
  > 9449169 Update community files (# 6390)
bumping knative.dev/hack f4972b4...12d3e2a:
  > 12d3e2a switch to UTC (# 176)
  > b4d4d73 Update community files (# 178)
bumping knative.dev/pkg 2d8305b...19adf79:
  > 19adf79 upgrade to latest dependencies (# 2524)
  > b0ff5ad Update actions (# 2523)
  > f907b31 Update community files (# 2520)

/lgtm
/approve


On Linux, shortcut ctrl+shift+alt+u is used to enter a Unicode character

I have recently installed your extension, but since then I could not use Ctrl+Shift+Alt+u shortcut to enter Unicode letters in Sublime Text. That shortcut (alongside with Ctrl+Shift+u, which does not work in ST, at least in ST3+) is used to enter Unicode characters on Linux.
Now I am able to disable a shortcut using (for example) a noop command, but I am not able to restore the original behaviour (i.e. prior to asigning your command to that shortcut) without editing Default.sublime-keymap (L68-78) in your repo.
Could your fix this issues please? :wink:

Hello,
The best thing you can do to remediate the issue with the binding is to unbind as you did with the noop, but you have to rebind the action to another key combination.
Example
[
  {
    "keys": [
      "ctrl+shift+alt+7"
    ],
    "command": "srtbyli",
    "args": {
      "sort": "decimal",
      "reversed": true
    }
  },
  {
    "keys": [
      "ctrl+shift+alt+u"
    ],
    "command": "noop"
  }
]

You can add this JSON in the user key binds, if you want.

Go to "Preference"
Choose "Key bindings"; this will open a window with two panels.
In the right window, there should be an empty JSON array (by default only)
Copy the previous JSON

This will work, and gives you the option to change the key binds to your preferences.
The only thing I can do, i to remap the option to another combination only on Linux, but this cause more confusion than good, in my opinion; since the key binds are a mess, can be used by other plugins, ect.
I will add a new menu & add more documentation on how to remap the combinations.

The addition is now pushed
ST3 & ST4 -> V1.3.0
ST2 -> V1.2.0
This will simplify the overriding of the key bindings and the best that I can do for this issue.

@Doi9t, I think I wasn’t clear enough. Those shortcuts are global/system-wide shortcuts on Linux to insert Unicode charactors. By setting them to noop command, on3 cannot restore the behaviour, but merely disable them (i.e. nothing will be done when they are pressed. Therefore this isn’t a solution, no even a workaround.
I see only one option: change the default shortcut(s). Or don’t set any and let the user configure them.

As an alternative to not setting any keyboard shortcuts, is is possible in ST to have a package setting using which now could disable all the default packages shortcuts, while keeping the Default.sublime-keymap file?


baichuan2-7b trt fp16/int8/int4 generation results consistence is far different from origin hf model

By utilizing examples/baichuan/build.py to convert baichuan2 model to be baichuan2-fp16/int8/int4, then run run.py, the results consistence is bad, comparing with hf fp16 model. strings comparsions(word by word comparison) is just 50%, manual evaluating is 75%, which means we judge generation results by our subjectivities, the generating-output's meaning is similar, we think it is correct.
building cmd:
trt-f16
python build.py --model_version v2_7b
--model_dir /baichuan2/
--dtype float16
--use_gemm_plugin float16
--use_gpt_attention_plugin float16
--output_dir ./tmp/baichuan_v2_7b/trt_engines/fp16/1-gpu/
trt-int8
python build.py --model_version v2_7b
--model_dir /baichuan2/
--dtype float16
--use_gemm_plugin float16
--use_gpt_attention_plugin float16
--use_weight_only
--weight_only_precision int8
--output_dir ./tmp/baichuan_v2_7b/trt_engines/int8_weight_only/1-gpu/

To verify the correctness, it is recommended to use summarize.py because it is hard to get fully same results on generative model. We also plan adding more metrics to help evaluating the model ability in the future.

@byshiue Thanks a lot, I'll try summarize.py

To verify the correctness, it is recommended to use summarize.py because it is hard to get fully same results on generative model. We also plan adding more metrics to help evaluating the model ability in the future.

Can’t wait for these new features to be released！ It is very important for quantitatively evaluating model performance.

Close this bug because the issue is inactivated. Feel free to ask here if you still have question/issue, we will reopen the issue.

Recently I have also found there's a pretty much difference between huggingface outputs and trt_llm outputs of llama2 13B, both in fp16 precision. It's pretty hard to locate where the difference originates from. So many factors, including the sampling algorithm, transformer architecture optimization, paged attention, etc. However, I observe there's some degradation in the quality of outputs in many cases in the greedy search decoding strategy.


Mentorship program announcement

Hello,
The text below will be submitted as an issue to the nodejs/node project to announce the mentorship program and open the door for enrollment. Please read it and provide feedback before having it submitted. CC @nodejs/community-committee @nodejs/tsc
follow the link to see the issue content in a gist: https://gist.github.com/Bamieh/b87fb3f279665966dec86f35a741c8d9
OR Click here to see a copy of the gist

Hello dears! 🦌
I am glad to announce the start of the enrollment process of the Node.js Mentorship program.
General overview
The goal is simple. Getting more people to contribute to Node.js and to help evolve the Node.js ecosystem by providing general help for all Node.js users.

To bring more contributors to Node.js projects by mentoring people about the Node.js environment and ecosystem, helping them contribute to Node.js, championing their PRs through code review, and giving general help.

The mentorship program duration is 6 months. The matched mentor/mentee pair will agree on specific goals to reach during this duration. The program will be super beneficial and rewarding to both the mentors and mentees.
Please have a quick look at the mentorship repo for the full details:
https://github.com/nodejs/mentorship
How can I help?
It would be awesome to have you enrolled in the mentorship program, either as a mentor or a mentee.
Joining this program will help in driving the Node.js open-source project forward by bringing more contributors to the project, increasing confidence in using Node.js, and making more Node.js experts!
For Mentors
Mentors play a huge role in defining the mentee's experience with Node.js. You must have some kind of contribution to any project under the Node.js umbrella to become a mentor.
Mentors provide general guidance and resources for their mentees to answer their questions. You do not need to know everything to be a mentor. Simply guiding your mentee to the right resources is enough.
Fill in this form to enroll as a mentor: https://goo.gl/forms/O6VX4ZglvCfX94vu2
For Mentees
You can enroll as a mentee whether you are new to Node.js or you want to get tailored guidance for a specific goal you have in mind.
If you are interested in contributing to Node.js, learning more about the core Node.js project, or you're interested in need general guidance around Node.js. The mentorship program is a great starting point!
Fill in this form to enroll as a mentee: https://goo.gl/forms/TUEAIYVqoFXA4WpF2
When did this happen?
The mentorship program started as an initiative under the Node.js Community Committee. You can get more in context by starting here: https://github.com/nodejs/community-committee/issues/172
You are awesome, and we'd love to have you!
Cheers!

Great to see us getting to the point were we can start bringing people into the mentorship program.

I've reached out to @gtewallace to see if we can get those Google Forms under a Node.js Foundation account 🤗

@bnb awesome thanks!

I signed up as a mentee :smile:

This is fantastic. Good job!!

I noticed the code of conduct link points to the mentorship repo, which in
turn directs people to the admin repo. Would it perhaps make sense to
simply link to the admin repo?
On Fri, Feb 23, 2018, 7:26 AM Matteo Collina<EMAIL_ADDRESS>wrote:

This is fantastic. Good job!!
—
You are receiving this because you are on a team that was mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nodejs/mentorship/issues/10#issuecomment-367996032,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAecV9__Z0q4DHoFKwpmjGpzViOGe0cXks5tXq6DgaJpZM4SN_zt
.

@MylesBorins it might make more sense I'm not actually sure. But it has an additional text about confidentiality of mentors and mentees.

@Bamieh Free to sign up as mentee now?

Hey @flickz! The application forms are opened now, please follow the links in the main repo README to apply and get all the details about the program! Hope see you soon!

Hi, kudos to this awesome program! Read all excellent details and signed up as a mentee. Congrats to the program and hope to be considered as a mentee. 😄

@BeniCheni Thanks for the feedback! We'd love to have you.

This whole thing is incredible. The kind of thing you hope would happen but doesn't often come along.  Hoping we all unwrap a golden ticket!

How many hours is the mentee expected to commit towards the program daily ?

This is great! Happy to be here! Thanks for keeping the program focused. Kudos to the node team!

@ankibalyan Thank you!

@PrajwalM2212 this should be coordinated with your match based on the goals you've set, however the program expects a minimum amount of meetings between the pair, and commitment to update the "mentorship bio" to report meetings and milestones. In order to graduate successfully from the program you need to have reached your mentorship goals, or parts of it at least.

@Bamieh  I have applied as a mentee clearly addressing my goals and experiences. Thanks to the organisation for such a great and generous program.

@Bamieh  I have a few questions
Has the program already started ?
How long should I wait before getting accepted ?

@PrajwalM2212 Thanks for your interest and support! we're currently in the first round which will end in September, after that we'll announce a new round.
The mentor/mentee ratio is super skewed so getting everyone applied to join might take a while. Our plan however is to increase the mentoring capacity gradually with time

@Bamieh I applied to the mentorship program. How long should i wait before getting accepted?
Thanks for your time.

Hi everyone, I have applied for the Node mentorship program (as a mentee) . Really looking forward to be part of the foundation. :D

Hello Everybody, Do we have a date on when the mentorship program is supposed to start this year?

@Hamza-324 The matching process is estimated so start on June 10th. The application form is open for mentees to apply. You can find it on the first line of the README in this repo.

Closing this issue as per @amiller-gh's recommendation to avoid confusion.


252 fix navbar component position inconsistency

By creating this pull request you agree to the terms in CONTRIBUTING.md.
https://github.com/Infineon/.github/blob/master/CONTRIBUTING.md
--- DO NOT DELETE ANYTHING ABOVE THIS LINE ---

added fonts
added 'fixed' Prop that makes the navbar fixed or not fixed.
added the navbar to example apps

📦 Published PR as canary version: 18.2.17--canary.256.c01ac1a6cf227d3cc369bc78108716e3eb5a2bc6.0

:sparkles: Test out this PR locally via:
npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>

I saw the navbar has a white background, just noticed that in case of Angular, the default background is something greyish and that looks weird with a white navbar. Will take that to the next backlog refinement meeting


Crosshair

When dragging (last right point), the crosshair flickers back and forth between that point and the x-value of the last non-null data point of the other series. In addition, the tooltip flickers and makes it hard to look at.
Example: http://jsfiddle.net/8j883n0g/

Related with #4752


Getting Powerapps cookie error after navigating to the app

Getting the below cookie error after navigating to the app while running the test
Tried allowing third party cookies in browser settings but still the error occurs for every new session created while running the tests. Is any option need to be changed in the code to get rid of this error?

I think you may need to update the settings as suggested in the following link.
https://docs.microsoft.com/en-gb/powerapps/troubleshooting-startup-issues#enable-storage-of-third-party-cookies-and-local-data-in-your-browser-or-app

I think you may need to update the settings as suggested in the following link. https://docs.microsoft.com/en-gb/powerapps/troubleshooting-startup-issues#enable-storage-of-third-party-cookies-and-local-data-in-your-browser-or-app

Already tried and this is not working for the new session

@AddalaDeviPriya Try set CookieСontrolsMode to 0
Example bellow.
BrowserOptions SharedOptions = new BrowserOptions
{
	CookieСontrolsMode = 0
};
var _webClient = new WebClient(SharedOptions);

@Shakevg  Tried setting CookieСontrolsMode = 0 and Third Party cookies are enabled. But when this value is set to zero, App is asking for MFA. How to bypass this phone OTP authentication?

@AddalaDeviPriya How does it work if you enable cookies manually (screen below) when you start the browser manually? Is it required to sign in too?
I just handled this case by implementing a click to the sign-in button if dialog appeared.

@Shakevg  I manually changed my browser settings to allow all cookies and tried login to the app. It still asks for Sign in with OTP.
I am not getting this sign in OTP if CookieСontrolsMode = 1.It only pops up powerapps error. MFA  appears when the value is set to 0. Looks like MFA is dependent on this key. Is there a way to handle this?

@AddalaDeviPriya, there is a bug in the Microsoft.Dynamics365.UIAutomation.Sample\TestSettings.cs,
within public static BrowserOptions Options => new BrowserOptions {
Add the following code, else the CookieСontrolsMode will default to 1.
CookieСontrolsMode = SharedOptions.CookieСontrolsMode;

@Shakevg I manually changed my browser settings to allow all cookies and tried login to the app. It still asks for Sign in with OTP. I am not getting this sign in OTP if CookieСontrolsMode = 1.It only pops up powerapps error. MFA appears when the value is set to 0. Looks like MFA is dependent on this key. Is there a way to handle this?

@AddalaDeviPriya I suppose you should work with dev team to undestand how end user will work with it, becuase looks like it is not related with framework


update nwjs v0.12.3

from v0.12.1 to v0.12.3

Thank you for the contribution. It needed some fixes, so they were made in c0bb4ebb92d3684bb58815dadc7341a06c4068bf. Your contribution is still included (and still credited to you), with the appropriate modifications. Please feel free to ask about any of the changes.


can't interact with editor

I wanted to try this out, so I created a new Django v3.2.13 project and created a single model:
class Story(models.Model):
    title    = models.CharField(max_length=255)
    contents = TipTapTextField()

When I try to add a story in Django Admin, I see a resizable white rectangle that looks like a text box - however, I can't interact with it at all.  Clicking does nothing, not even placing a cursor.

I tried both Firefox and Chromium on Ubuntu 20.04.

I would need more info. What does the network tab of the devtools say about request to skypack CDN. I suspect that the files are taking too much time in loading it...

I opened the network tab and clicked Add.  The list was too long to capture in a single screenshot, so there's two slightly-overlapping ones:

I also chose "Save All As HAR File", but GitHub won't let me attach it.
However, I think a better clue is the contents of the Console:

have you provided it with the tiptapOutputFormat parameter? take a look here https://github.com/django-tiptap/django_tiptap/blob/ba895a973bcde48506adb4b53e830ff85f38fd08/django_tiptap/templates/forms/tiptap_textarea.html#L1350-L1354. if you don't provide anything, it would not be able to set that variable and so it'll throw an error.
valid values are "html" or "json"

have you provided it with the tiptapOutputFormat parameter?

I didn't know that was required.  Based on the README,md, all I did was:

add 'django_tiptap' to INSTALLED_APPS
add a field of type django_tiptap.fields.TipTapTextField

Where/how am I supposed to set the value?  Is it in the config?  Because the config section of README.md indicates that DJANGO_TIPTAP_CONFIG["tiptapOutputFormat"] already defaults to "html".

Locally it works for me, maybe @solen0id might be able to help out on this one

Django 5, letest package, same error
Uncaught ReferenceError: initialEditorContent is not defined
and editor doesn't work

I had this problem (also using Django 5) then noticed that the css hadn't been included anywhere. Needed to add the following to my form template -
<link rel="stylesheet" href="{% static 'django_tiptap/css/styles.css' %}">

Once I had that, the form renders (although the toolbar only starts to render once the widget gets focus). The formatting is all over the place for the toolbar, though I think this may just be that the svg's aren't particularly consistent in size and centering.
There's a problem with the tippy dropdown for table. It renders off-screen (and hidden) if the button is at the right. Seems some text/fields missing from it as well:

The form template also has a syntax error - line 3-4, the include directive is split over 2 lines so is not interpreted by django, should be:
{% include "django/forms/widgets/attrs.html" %}


POWERn up to n=25

I have a wemos with mcp23017 portexpamder and now I have really more than 8 relais.
It still did it locally but could you please add power up to n=25?
POWERn:        {type: 'boolean', role: 'switch',                   read: true, write: true,  storeMap: true}
Thanks a lot!

Hi,
how are your settings at "Auto creation of states"?
For me its not a smart solution to just add POWER states up to 25 in datapoints.js
Next week somebody needs 40 :)
So your POWERn line would be nice.
I will try to fix this, but i need time...

Hi,
how are your settings at "Auto creation of states"?
For me its not a smart solution to just add POWER states up to 25 in datapoints.js
Next week somebody needs 40 :)
So your POWERn line would be nice.
I will try to fix this, but i need time...

Hi,
The setting are true at my installation.
OK, no problem if there is a smarter solution :-)

Hi,
The setting are true at my installation.
OK, no problem if there is a smarter solution :-)

If you need a quick solution, you could do the following:
Open /opt/iobroker/node_modules/iobroker.sonoff/lib/datapoints.js in you prefered editor and copy/past all the missing POWER9-25 in. Then on admin page goto Adapter and upload the Sonoff adapter:

Click on the arrow up icon.
Then restart the Adapter.
I will figure out a better solution, but it will take sometime. There are still 5 pull request open, i don't know how long it will take...

If you need a quick solution, you could do the following:
Open /opt/iobroker/node_modules/iobroker.sonoff/lib/datapoints.js in you prefered editor and copy/past all the missing POWER9-25 in. Then on admin page goto Adapter and upload the Sonoff adapter:

Click on the arrow up icon.
Then restart the Adapter.
I will figure out a better solution, but it will take sometime. There are still 5 pull request open, i don't know how long it will take...

Ok, i found a smarter solution :)
Will see, how long it will take to accept this pull request :)

Ok, i found a smarter solution :)
Will see, how long it will take to accept this pull request :)

Ok, i found a smarter solution :)
Will see, how long it will take to accept this pull request :)

Thank you. If I should test it please let me know

Ok, i found a smarter solution :)
Will see, how long it will take to accept this pull request :)

Thank you. If I should test it please let me know

Sorry, it wasn' that easy, I found a bug. Will check on the weekend...

Sorry, it wasn' that easy, I found a bug. Will check on the weekend...


Remaining tasks to ready for Ruby 3.1 or later

[x] https://github.com/fluent/fluentd/pull/3585
[x] https://github.com/fluent/fluent-plugin-s3/pull/388
[x] CI Failure with Ruby 3.2 on Windows: https://github.com/fluent/fluentd/issues/3869
[ ] (Optional) Support async-2.0 or later: https://github.com/fluent/fluentd/pull/3842

Importing Ruby 3.2 has been landed at #425


Issue/#84

Fix #84
Fix #78

@rultor merge

@rultor merge

@kerelape OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@kerelape Done! FYI, the full log is here (took me 2min)


Added docblocks to all public methods, and configured projects to require docblocks.

Codecov Report

Merging #31 (b4d1723) into main (512437b) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##             main      #31   +/-   ##
=======================================
  Coverage   36.28%   36.28%
=======================================
  Files          24       24
  Lines         226      226
=======================================
  Hits           82       82
  Misses        144      144

Impacted Files
Coverage Δ

...sh.DataProduct.Configuration/DataProductOptions.cs
0.00% <ø> (ø)

...aMesh.DataProduct.Configuration/FeaturesOptions.cs
0.00% <ø> (ø)

...DataMesh.DataProduct.Configuration/OwnerOptions.cs
0.00% <ø> (ø)

...ataMesh.DataProduct.Configuration/SchemaOptions.cs
100.00% <ø> (ø)

...DataProduct.Configuration/SchemaRegistryOptions.cs
66.66% <ø> (ø)

....DataProduct.Generator/Generators/GeneratorBase.cs
96.00% <ø> (ø)

...ataProduct.Generator/Generators/ModelsGenerator.cs
91.83% <ø> (ø)

...ataMesh.DataProduct.Generator/NamespaceResolver.cs
0.00% <ø> (ø)

...DataMesh.DataProduct/Apis/ApisStartupExtensions.cs
0.00% <ø> (ø)

...taProduct/Apis/GraphQL/GraphQLStartupExtensions.cs
0.00% <ø> (ø)

... and 13 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


[SPARK-19147][CORE] netty throw NPE

What changes were proposed in this pull request?

TransportClientFactory.createClient() is called by task and TransportClientFactory.close() is called by executor.
When stop the executor, close() will set workerGroup = null, NPE will occur in createClient which generate many exception in log.
For exception occurs after close(), treated it as an expected Exception
and transform it to InterruptedException which can be processed by Executor.
Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

New test added in TransportClientFactorySuite

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Wait, why would we be creating a new client after the executor is shut down?

Here is the scene, when the executor is killed before shuffle process, but the task was created and ready to fetch blocks. Then, the NPE will occur if task try to create a new client.
Such NPE will confuse user to find why executor is killed.

@srowen thanks for the comments, I'll update the pr later.

@colinmjj, also please fill other items in PR description.

Can one of the admins verify this patch?

Test build #4871 has started for PR 25759 at commit b2510fb.

Test build #4871 has finished for PR 25759 at commit b2510fb.

This patch fails Spark unit tests.
This patch does not merge cleanly.
This patch adds no public classes.

@srowen The patch is updated, for the exception from task after executor.stop, add exception process to deal with it.

Test build #4876 has started for PR 25759 at commit 8fa404c.

Test build #4876 has finished for PR 25759 at commit 8fa404c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged to master

Hi, All.
Can we have this in branch-2.4 since SPARK-19147 is a long standing issue?

Although it's 'just' a cosmetic issue with how errors are logged in a sort of corner case, it's also a minor change. I'm fine if you want to back-port to 2.4.

Thanks, @srowen . I'll backport this to branch-2.4.


Camera movement

We should add some simple 3d camera control using wasd and arrow keys. Might be changed or removed later, but will be nice for debugging and looking at particles from different perspectives.
Depends on:
#3 - Event handling

Fixed with bbd47bc


hello

hi how can I find you? I am trying to pack my own wine can you help me ?

To pack your own WINE, you need to use the appimagetool (and install WINE and deps on one directory).
You can see how it's done on:

Arch Linux:
https://github.com/ferion11/Wine_Appimage/blob/master/deployscript/archlinux-wine-deploy.sh
or
https://github.com/Hackerl/Wine_Appimage/blob/master/deployscript/arch-winedeploy.sh

Debian:
https://github.com/Hackerl/Wine_Appimage/blob/master/deployscript/debian-winedeploy.sh

Open Suse:

https://github.com/Hackerl/Wine_Appimage/blob/master/deployscript/opensuse-winedeploy.sh

The idea is the same in all options: 1- Install WINE e deps on one directory, 2- Create one script AppRun that will run the installation, 3- And use the appimagetool to pack it.


Webpack config: adding oft files to loader test

Extending loader test to support OFT fonts

Coverage remained the same at 95.0% when pulling ae9ddc08e2e33374087e609c664ed9f17f1d25e4 on davidglbr:master into 32fd8c834dfb0a0bb0435b4d8608006e66e15ebc on coryhouse:master.

Coverage remained the same at 95.0% when pulling ae9ddc08e2e33374087e609c664ed9f17f1d25e4 on davidglbr:master into 32fd8c834dfb0a0bb0435b4d8608006e66e15ebc on coryhouse:master.

While I'm fine with the move to support OFT, I'd rather not change the rest of the loader config. Can the rest of the loader config stay the same?

Hey Cory,
Thank you for this AWESOME repo!
To your question, of course. Only changed the text regex string to be
either TTF or OFT, all the rest is intact.
Cheers,
David
ᐧ
On 23 November 2016 at 13:00, Cory House<EMAIL_ADDRESS>wrote:

While I'm fine with the move to support OFT, I'd rather not change the
rest of the loader config. Can the rest of the loader config stay the same?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/coryhouse/react-slingshot/pull/333#issuecomment-262588386,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJ3eFrpZ3WKb6ww8LrXwUK8YR4d_Z0jOks5rBH9QgaJpZM4K60gP
.

My bad,
Yes. Let's please eliminate the change on the right.
Should I update the pull request?
Cheers,
David
ᐧ
On 23 November 2016 at 15:41, Cory House<EMAIL_ADDRESS>wrote:

@coryhouse commented on this pull request.
In webpack.config.prod.js
https://github.com/coryhouse/react-slingshot/pull/333#pullrequestreview-9941669
:

@@ -71,7 +71,7 @@ export default {
{test: /.jsx?$/, exclude: /node_modules/, loader: 'babel'},
{test: /.eot(?v=\d+.\d+.\d+)?$/, loader: 'url?name=[name].[ext]'},
{test: /.woff(2)?(?v=[0-9].[0-9].[0-9])?$/, loader: 'url?limit=10000&mimetype=application/font-woff&name=[name].[ext]'},

 {test: /\.ttf(\?v=\d+.\d+.\d+)?$/, loader: 'url?limit=10000&mimetype=application/octet-stream&name=[name].[ext]'},

 {test: /\.[ot]tf(\?v=\d+.\d+.\d+)?$/, loader: 'file-loader?limit=10000&mimetype=application/octet-stream'},

Can we eliminate the changes on the right?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/coryhouse/react-slingshot/pull/333#pullrequestreview-9941669,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJ3eFuaiYulSfP1JjF7ranajJxFQyB_Tks5rBKTzgaJpZM4K60gP
.

That'd be great, thanks!
On Nov 23, 2016 4:05 PM, "David Glauber"<EMAIL_ADDRESS>wrote:

My bad,
Yes. Let's please eliminate the change on the right.
Should I update the pull request?
Cheers,
David
ᐧ
On 23 November 2016 at 15:41, Cory House<EMAIL_ADDRESS>wrote:

@coryhouse commented on this pull request.
In webpack.config.prod.js
<https://github.com/coryhouse/react-slingshot/pull/333#
pullrequestreview-9941669>
:

@@ -71,7 +71,7 @@ export default {
{test: /.jsx?$/, exclude: /node_modules/, loader: 'babel'},
{test: /.eot(?v=\d+.\d+.\d+)?$/, loader: 'url?name=[name].[ext]'},
{test: /.woff(2)?(?v=[0-9].[0-9].[0-9])?$/, loader:
'url?limit=10000&mimetype=application/font-woff&name=[name].[ext]'},

{test: /.ttf(?v=\d+.\d+.\d+)?$/, loader: 'url?limit=10000&mimetype=
application/octet-stream&name=[name].[ext]'},

{test: /.[ot]tf(?v=\d+.\d+.\d+)?$/, loader: 'file-loader?limit=10000&
mimetype=application/octet-stream'},

Can we eliminate the changes on the right?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<https://github.com/coryhouse/react-slingshot/pull/333#
pullrequestreview-9941669>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/
AJ3eFuaiYulSfP1JjF7ranajJxFQyB_Tks5rBKTzgaJpZM4K60gP>
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/coryhouse/react-slingshot/pull/333#issuecomment-262641332,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABnFpSAW5cLFHijn_D07ZTO_9yWjNlHiks5rBLiMgaJpZM4K60gP
.

@coryhouse  done

Coverage remained the same at 95.0% when pulling 2ee043d100c648ac3b80e05c2a7c149dbeb59339 on davidglbr:master into 32fd8c834dfb0a0bb0435b4d8608006e66e15ebc on coryhouse:master.

Coverage remained the same at 95.0% when pulling 2ee043d100c648ac3b80e05c2a7c149dbeb59339 on davidglbr:master into 32fd8c834dfb0a0bb0435b4d8608006e66e15ebc on coryhouse:master.

Thanks! 👍


Add support for Visual Studio 2019, msvc toolset 14.2+

Thanks.  I've moved this over to CMake MR 3221.  For future contributions please see CONTRIBUTING.rst and submit a merge request to our Gitlab instance.


Pytorch PPO Implementation, dimension difference

Hello, apologies if I do this wrong I don't contribute to open source often. I was attempting to run the Pytorch PPO implementation and kept getting several errors regarding the dimension of the observation 'o'. I believe this is because the return values from the environment are being mishandled. See my proposed changes below:
Original:
o, ep_ret, ep_len = env.reset(), 0, 0

# Main loop: collect experience in env and update/log each epoch
for epoch in range(epochs):
    for t in range(local_steps_per_epoch):
        a, v, logp = ac.step(torch.as_tensor(o, dtype=torch.float32))

        next_o, r, d, _ = env.step(a)
        ep_ret += r
        ep_len += 1

        # save and log
        buf.store(o, a, r, v, logp)
        logger.store(VVals=v)

        # Update obs (critical!)
        o = next_o

        timeout = ep_len == max_ep_len
        terminal = d or timeout
        epoch_ended = t==local_steps_per_epoch-1

        if terminal or epoch_ended:
            if epoch_ended and not(terminal):
                print('Warning: trajectory cut off by epoch at %d steps.'%ep_len, flush=True)
            # if trajectory didn't reach terminal state, bootstrap value target
            if timeout or epoch_ended:
                _, v, _ = ac.step(torch.as_tensor(o, dtype=torch.float32))
            else:
                v = 0
            buf.finish_path(v)
            if terminal:
                # only save EpRet / EpLen if trajectory finished
                logger.store(EpRet=ep_ret, EpLen=ep_len)
            o, ep_ret, ep_len = env.reset(), 0, 0

To fix this I just get rid of the 'info' return value from env.reset, but I imagine this might not work for all environments (if they have extra returns for the env.step() for example).
Adjusted:
(o,_), ep_ret, ep_len = env.reset(), 0, 0

# Main loop: collect experience in env and update/log each epoch
for epoch in range(epochs):
    for t in range(local_steps_per_epoch):
        a, v, logp = ac.step(torch.as_tensor(o, dtype=torch.float32))

        next_o, r, d, _ = env.step(a)
        ep_ret += r
        ep_len += 1

        # save and log
        buf.store(o, a, r, v, logp)
        logger.store(VVals=v)

        # Update obs (critical!)
        o = next_o

        timeout = ep_len == max_ep_len
        terminal = d or timeout
        epoch_ended = t==local_steps_per_epoch-1

        if terminal or epoch_ended:
            if epoch_ended and not(terminal):
                print('Warning: trajectory cut off by epoch at %d steps.'%ep_len, flush=True)
            # if trajectory didn't reach terminal state, bootstrap value target
            if timeout or epoch_ended:
                _, v, _ = ac.step(torch.as_tensor(o, dtype=torch.float32))
            else:
                v = 0
            buf.finish_path(v)
            if terminal:
                # only save EpRet / EpLen if trajectory finished
                logger.store(EpRet=ep_ret, EpLen=ep_len)
            (o,_), ep_ret, ep_len = env.reset(), 0, 0

Let me know if anyone else has run into this problem or if this fix works.

Hi, Kevin. I'm regularly using that same algorithm, pytorch PPO, and never
had that problem. What error are you getting exactly? And on which
environment(s)?
Best.
Alberto
El mié, 22 feb 2023, 21:25, kevin @.***> escribió:

Hello, apologies if I do this wrong I don't contribute to open source
often. I was attempting to run the Pytorch PPO implementation and kept
getting several errors regarding the dimension of the observation 'o'. I
believe this is because the return values from the environment are being
mishandled. See my proposed changes below:
Original:
o, ep_ret, ep_len = env.reset(), 0, 0
Main loop: collect experience in env and update/log each epoch
for epoch in range(epochs):
for t in range(local_steps_per_epoch):
a, v, logp = ac.step(torch.as_tensor(o, dtype=torch.float32))
    next_o, r, d, _ = env.step(a)
    ep_ret += r
    ep_len += 1

    # save and log
    buf.store(o, a, r, v, logp)
    logger.store(VVals=v)

    # Update obs (critical!)
    o = next_o

    timeout = ep_len == max_ep_len
    terminal = d or timeout
    epoch_ended = t==local_steps_per_epoch-1

    if terminal or epoch_ended:
        if epoch_ended and not(terminal):
            print('Warning: trajectory cut off by epoch at %d steps.'%ep_len, flush=True)
        # if trajectory didn't reach terminal state, bootstrap value target
        if timeout or epoch_ended:
            _, v, _ = ac.step(torch.as_tensor(o, dtype=torch.float32))
        else:
            v = 0
        buf.finish_path(v)
        if terminal:
            # only save EpRet / EpLen if trajectory finished
            logger.store(EpRet=ep_ret, EpLen=ep_len)
        o, ep_ret, ep_len = env.reset(), 0, 0

To fix this I just get rid of the 'info' return value from env.reset, but
I imagine this might not work for all environments (if they have extra
returns for the env.step() for example).
Adjusted:
(o,_), ep_ret, ep_len = env.reset(), 0, 0
Main loop: collect experience in env and update/log each epoch
for epoch in range(epochs):
for t in range(local_steps_per_epoch):
a, v, logp = ac.step(torch.as_tensor(o, dtype=torch.float32))
    next_o, r, d, _ = env.step(a)
    print(f"next_o: {next_o}")
    print(f"r: {r}")
    print(f"d: {d}")
    print(f"_: {_}")
    ep_ret += r
    ep_len += 1

    # save and log
    buf.store(o, a, r, v, logp)
    logger.store(VVals=v)

    # Update obs (critical!)
    o = next_o

    timeout = ep_len == max_ep_len
    terminal = d or timeout
    epoch_ended = t==local_steps_per_epoch-1

    if terminal or epoch_ended:
        if epoch_ended and not(terminal):
            print('Warning: trajectory cut off by epoch at %d steps.'%ep_len, flush=True)
        # if trajectory didn't reach terminal state, bootstrap value target
        if timeout or epoch_ended:
            _, v, _ = ac.step(torch.as_tensor(o, dtype=torch.float32))
        else:
            v = 0
        buf.finish_path(v)
        if terminal:
            # only save EpRet / EpLen if trajectory finished
            logger.store(EpRet=ep_ret, EpLen=ep_len)
        (o,_), ep_ret, ep_len = env.reset(), 0, 0

Let me know if anyone else has run into this problem or if this fix works.
—
Reply to this email directly, view it on GitHub
https://github.com/openai/spinningup/issues/384, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFRLIPR3LFAGJCMUDAKQDJDWYZY4VANCNFSM6AAAAAAVEZHQVM
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Hello,
I'm using a custom environment following the OpenAI Gym api.
This would be the error (although this time for td3)

Traceback (most recent call last):
File "src/laser_gym.py", line 43, in
train_test()
File "src/laser_gym.py", line 25, in train_test
td3(env_fn=env_fn, ac_kwargs=ac_kwargs, steps_per_epoch=1000, epochs=260, logger_kwargs=logger_kwargs)
File "/home/kevin/work/spinningup/spinup/algos/pytorch/td3/td3.py", line 334, in td3
test_agent()
File "/home/kevin/work/spinningup/spinup/algos/pytorch/td3/td3.py", line 276, in test_agent
o, r, d, _ = test_env.step(get_action(o, 0))
File "/home/kevin/work/spinningup/spinup/algos/pytorch/td3/td3.py", line 267, in get_action
a = ac.act(torch.as_tensor(o, dtype=torch.float32))
ValueError: expected sequence of length 2 at dim 1 (got 1)

My reset and step functions are defined as follows:
def reset(self, options=None):
    self._agent_location = np.random.random_integers(int(self.LOW/2), int(self.HIGH/2), size=2).astype(self.DTYPE)
    obs = self._get_obs()
    info = self._get_info()
    return obs, info

def step(self, action):
    _new_agent_location = self._agent_location + np.matmul(self.response_matrix,action)
    rew = self.reward.get_reward(self._agent_location, action, _new_agent_location)
    self._agent_location = _new_agent_location

    if (np.abs(self._agent_location) > self.HIGH).any() or (np.abs(self._agent_location) < 0.05).all():
        done = True #hit the edge
    else:
        done = False

    obs = self._get_obs()
    info = self._get_info()

    if self.render_mode == 'human':
        self.render()

    return obs, rew, done, info

I am a little confused how this could work for other people when it seems like all the environment's should return the same tuple (https://www.gymlibrary.dev/api/core/#gym.Env.reset)


converter: do not append batch size if non is provided

It actually creates an incompatibility between nydusify and nydus-image. Let's keep the default (0) working at least.

Maybe we need to change the codecov policy to verify overall coverage, not patch coverage. cc @adamqqqplay

OK, I will fix this.

@bergwolf Please rebase upstream/main, to skip the codecov test.

A better solution is using pkg/converter/tool/feature.go feature detection. cc @hangvane

It actually creates an incompatibility between nydusify and nydus-image. Let's keep the default (0) working at least.

A better solution is using pkg/converter/tool/feature.go feature detection. cc @hangvane

@bergwolf @imeoer Thanks for your suggestion! I will implement the feature detection to address the incompatibility issue with batch size.


feature Implement hdfs accessor

Proposed changes
Implement hdfs accessor
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run compile

run buildall


Custom update prefix

Instead of hard coded message (Updated at ...) make the this message configurable in the YAML file

Closed by pull request #2


sysfs-gpio: support ATSAM CPUs unconventional gpio naming

What kind of new feature are you looking for?

Hardware: ATSAMA5Dxx

Do you plan to:

contribute an initial driver in experimental: Yes
do all the work needed to get the driver promoted to _stable_¹: hopefully some day

Hi, I'm trying to get periph.io working on the ATSAMA5D27. The gpio naming for this cpu is somewhat non-standard in that in /sys/class/gpio it uses names like "PA3", "PD25" instead of "gpio121". This causes the host/sysfs/gpio driver not to find any of the GPIO.
So, I forked the host/sysfs/gpio driver to host/atsam/gpio and got it to work. Currently the code is very rough (but at least WaitForEdge is working).
Does anyone have any thoughts how support for this chip should be implemented? Could sysfs/gpio be made more general, or is simply forking the sysfs/gpio code the right path?
My experimental code is here:
https://github.com/cbrake/periph/tree/master/host/atsam
(needs a lot of cleanup)
Thanks!

It's been four years, so the code should simply switch to /sys/bus/gpiochipN.
Tracked as https://github.com/periph/host/issues/39


Add a simple prettier plugin for harp themes.

This change introduces a plugin for formatting harp themes using
prettier.
Signed-off-by: Roberto Raggi<EMAIL_ADDRESS>

Codecov Report

Merging #1444 into master will not change coverage by %.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1444   +/-   ##
=======================================
  Coverage   55.93%   55.93%
=======================================
  Files         271      271
  Lines       24315    24315
  Branches     5437     5437
=======================================
  Hits        13601    13601
  Misses      10714    10714

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 f0a87e7...19f9c2b. Read the comment docs.


proposed search hint w github.com asset

Just a little PR for adding a hint in the search nav, like this! The hint disappears on focus of the input. Could use its own nicely colored asset :)

This PR also addresses #1278 and conveniently covers #1277

Something about this change is causing the search input to jump horizontally when focus/blurred. Also there are a few conflicts. Otherwise, looks good.


add video playlist url to opengraph headers

Requires an indigo update to at least https://github.com/bluesky-social/indigo/commit/96112be8cf46c28c44f82b9cda3b446d8a262b00 (Aug 13)
I'm not actually sure if M3U8 works as an embed, but it probably should since it's usually browser-to-browser?

I guess video support was on a branch? 🤔

Tried the same thing here #5320 more or less assuming that m3u8 isn't supported


Authenticate and authorizer the user

Asana ticket: Use Cognito + ADFS for auth
Ensure that they are logged in and a member of the document-viewer
Cognito group.
Add ueberauth, ueberauth_cognito, and guardian as dependencies.

@paulswartz Thanks! Ready for another look please when you have a chance.


Inconsistent performance

I have tested the performance in other images. The results seem wrong.

And I also found that the code uses language-sam to detect objects. So does it need to claim the name of the objects first?

Thanks for your interest in the work! The model may not work well with all viewpoints/objects. It could also depend on the object label/crop. We are working on improving this.
In terms of the usage of language-sam: we use it for simplicity. Any method to provide object crop will work.

Thanks for the reply. I have tested several images. The results seem not very good. Is there any wrong or bug?
It would be strong if it could work on the whole image without cropping.
Another example:


Resolved issue with 'unmasked' where precision is set

I'm utilizing jQuery.data() to bind the settings/parameters to the element. I then reuse the settings/parameters within the 'unmasked' method to determine the decimal value.

@NemSavic this pull request perfectly solve the problem but creates so many others... we need to solve this problems to have the pull request accepted. If someone have an idea how to correct it please help us.

I created a pull request #214 with your implementation @NemSavic, and solving all the side effects too.
I hope I've helped.


APC UPS Daemon (No USB) where and how can I define the UPS IP address

Hi
I have an APC SMT750I UPS with UPS Network Management Card 2 (AP9631)
I can reach the UPS from browser / telnet / SNMP. IP address is: 192.168.252
I wanted to use this addon but I don't know where and how can I define the card's IP address in the addon.
I tried to setup in the config:
name: APC UPS
cable: ether
type: net
device: 192.168.252
extra: []

I tried too:
device: '192.168.252:user:password'
and
device: '192.168.252:*user*:password'
But I got the same error every time:
Feb 12 22:26:06 a722577e-apcupsd-net syslog.info syslogd started: BusyBox v1.30.1
Feb 12 22:26:06 a722577e-apcupsd-net daemon.notice apcupsd[6]: apcupsd 3.14.14 (31 May 2016) unknown startup succeeded
Feb 12 22:26:06 a722577e-apcupsd-net daemon.info apcupsd[6]: NIS server startup succeeded
Feb 12 22:26:21 a722577e-apcupsd-net daemon.warn apcupsd[6]: Communications with UPS lost.
send-mail: account default not found: no configuration file available

The USB version is works fine anyway.
Any idea?
Thanks
T

I've never dealt myself with an APC network UPS.
I'm not sure if it's a typo, but 192.168.252 is not an IP address. You should have 4 numbers, not 3.
Beyond that, I recommend you read through this thread which seems to have a lot of different troubleshooting advice.
A few tidbits:

net is used for connecting to another apcupsd instance. snmp is for connecting to SNMP UPSes (which I think yours is)
The username/password is not necessarily what the web username/password is

Try checking those things, and read through that thread, and if you still can't get it working, come back here.

There haven't been any updates to this issue so I'm closing it.


Refresh the bookshelf

Steps to reproduce
1.Delete or change the classification of comics on the shelf page.
2.Operation does not take effect.
Expected behavior
Same as the original
Actual behavior
Same as description, you must switch pages (for example, switch to the settings page) to refresh.
Crash logs
No response
Tachiyomi version
1.8.5
Android version
Android 12
Device
redmi k50 miui13
Other details
I think this problem is a variation of #648 , or it may be a bug introduced when fixing it.
I didn't encounter the same problem in Tachiyomi 0.14.2-5188
Acknowledgements

[X] This is a TachiyomiSY specific issue that does not happen in Tachiyomi Preview.
[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with an extension, I should be opening an issue in the extensions repository.
[X] I have tried the troubleshooting guide.
[X] I have updated the app to version 1.8.5.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

like this.The information in the dialog box is correct.

Closing in favor of https://github.com/jobobby04/TachiyomiSY/issues/734


Enable node highlighting in Neotree

When a user switches buffers (or they refresh or whatever), we should really be using the Neotree's renderer.focus_node function to highlight the selected node.

This has been completed in v1.1 already. Closing out


Adding code style/quality tools

I propose we add style and quality tools to the Travis build, which would both communicate the project's code requirements to developers automatically, and alleviate the need for reviewers to comment on style violations. My recommendations:

http://phpmd.org/
https://github.com/FriendsOfPHP/PHP-CS-Fixer

I'll be happy to set those up

There's also codesniffer; at my last PHP shop we used all three of these. I'm not sure what PHP_CS adds over phpmd

PHP_CodeSniffer detects violations of specified code style. PHP-CS-Fixer simply builds on top of that to auto-fix those violations for you. Not sure if it would ever make sense to run both since PHPCSF covers that entire use-case (identification (through --dry-run) of violations as well as fixing). PHPMD isn't as much concerned with code-style as it is an attempt at static analysis of code. It's going to look for duplication of code. It's going to look at method cyclomatic complexity. Etc. It will also probably light up like Las Vegas when pointed at OpenCFP so it might feel odd for someone submitting a simple fix to get notes about completely unrelated code "being inefficient". Food for thought. Maybe we just want to introduce code-style checks for now?
In addition, we also have (a not well publicized) Codeclimate registration at https://codeclimate.com/github/opencfp/opencfp that I think is really a better solution for the job. Rather, it's already covering the case of "inefficient code" etc etc. Through configuration, I believe we can get it to report status changes on PRs as a check as well. We mentioned something similar to them at php[tek] this year, but I lost track of where that went.
Questions for consideration:

Will we have PHP CS Fixer doing dry-runs detecting code style violations or will we be somehow configuring it to actually make changes + commits to PRs. FWIW I think I'd prefer the former; having it simply communicate the violation.
Will we have PR merges require that these code-style tests pass or do we make it less strict than that.
Rather than hitting every PR with phpmd what if we better publicized the Codeclimate registration and pointed folks there as a "low hanging fruit" or opportunity to contribute. We could also better track that over time.

Take nothing above as assertive / conclusive. I'm in a rush and was just getting some ideas out! Definitely open for discussion.

I'm not a fan of autocorrect in the background; I was trying out the tools on a different codebase, and it did break some things.
My suggestion would be, set up CS-Fixer for dry run to break Travis, assuming there's not major blockers as far as fixing existing issues in the code as it is today.
I'll try a branch and see what that looks like

@yitznewton
:+1: for php-cs-fixer! Also comes with possibility to cache between builds, see https://github.com/FriendsOfPHP/PHP-CS-Fixer/tree/master!

@yitznewton
Also cool: provide a Makefile with a cs target, so you can run the fixer locally, too!

Heh @localheinz, where have I seen that before :wink:

@yitznewton
Maybe here: https://github.com/zendframework/modules.zendframework.com/blob/master/.travis.yml?
Hehe.

Mwahaha https://github.com/easybiblabs/oauth2-client-php/blob/master/Makefile#L14-L16

@yitznewton
Till added it for GetCourse! Great idea back then.

@yitznewton
Turns out, though, that I was the one originally adding phpcs and phpmd for getcourse/api - code was in a mess back then.

Looks like it's resolved


Add bigIntegerValue impl for LongIntElementImpl

Fixes #48.
Implements AnyElement.bigIntegerValue for LongIntElementImpl.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #49 (f8418f1) into master (7d926bc) will increase coverage by 0.03%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master      #49      +/-   ##
============================================
+ Coverage     81.05%   81.09%   +0.03%
- Complexity      501      502       +1
============================================
  Files            29       29
  Lines           586      587       +1
  Branches         85       85
============================================
+ Hits            475      476       +1
  Misses           62       62
  Partials         49       49

Impacted Files
Coverage Δ
Complexity Δ

...c/com/amazon/ionelement/impl/LongIntElementImpl.kt
84.21% <100.00%> (+0.87%)
14.00 <1.00> (+1.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 7d926bc...f8418f1. Read the comment docs.


Suggestions for new releases

Hi there!
Flexinizr rocks alot, i would like to include it in sierra library but i have some suggestions. What do you think about...

Remove non flex related classes (they are useful, but in my poinion shouldn't be part of this repo )
.fixed, .container, .hide, .truncate ...
Define variable values in a separated file.
$basis-percents, $max-flex-order, $responsive-images, $container-width

The rest is awesome. Like it alot.
Good job!

Hey, sorry for the late response! This project has sort of been abandoned due to me changing jobs and out of web development. I do want to get back to it soon, so when I do I'll take your suggestions to heart! Thanks for the comment, and feel free to fork.


chore: modify error case

Description

closes: #XXXX
ref. https://github.com/line/lbm-sdk/pull/876#issuecomment-1418530769
Motivation and context

How has this been tested?

Screenshots (if appropriate):
Checklist:

[x] I followed the contributing guidelines and code of conduct.
[ ] I have added a relevant changelog to CHANGELOG.md
[x] I have added tests to cover my changes.
[ ] I have updated the documentation accordingly.
[ ] I have updated API documentation client/docs/swagger-ui/swagger.yaml

Could you add more description?

Okay. I added associated link


nicer console ui using spinner

In non-verbose mode there are sometimes longer pauses (e.g. installing a bigger action on the container) where it's unclear to the user if wskdebug is ready or still working. A spinner style output would be much nicer.
Candidates:

https://github.com/sindresorhus/ora
https://github.com/sindresorhus/cli-spinners

Moved to https://github.com/apache/openwhisk-wskdebug/issues/8.


Codecov is not working ok

We are not retrieving the files list

https://codecov.freshdesk.com/support/solutions/articles/43000593943-files-not-showing-in-codecov-fixing-paths- ?

it was related to v3/dev being used instead of v3%2fdev


IPAM extension proposal

Proposal for #387

Limit API scope to what is currently needed.
Proposal for providing a way to register ipam provider with libnetwork
on the same line as network driver registration works

Signed-off-by: Alessandro Boch<EMAIL_ADDRESS>

Is this the proposed API for the upcoming pluggable model ?
If not, I think we should do it together rather than changing the API multiple times.

Thanks @aboch can you please squash the commits ?

@mavenugo  squashed
@mavenugo @mrjana Please take a look at the design. Will add test code after your feedback.

Closing in favor of #525


Cyber Visualizer does not display Channels data

Hi My System information is below:

OS: Ubuntu 18.04
Apollo installed from binary. I mean I am using Docker. Installed using this tutorial to clone the repo, etc: https://www.svlsimulator.com/docs/archive/2020.06/apollo-master-instructions/
Apollo version: Apollo.Auto 6.0

Steps to reproduce the issue:
All the steps and results are reproduced and can be checked on this video: https://youtu.be/HTi8CC0TaP8
However I have also described all of them below:
1 -cd Apollo

Start the docker image:
./docker/scripts/dev_start.sh
2-  Enter the the image
./docker/scripts/dev_into.sh
3 -Build the modules
./apollo.sh build_opt_gpu

2- Replay a recorded file of all channels:
2.1 cyber_recorder play -f<PHONE_NUMBER>003.record
2- or play the simulation set on LGSVL simulator
2.1 - Set the Dreamview
 ./scripts/bootstrap_lgsvl.sh
2.2 - Set Cyber Bridge
 ./scripts/bridge.sh
2.3 - /trigger the modules on control Tab (Dreamview): Routing, planning, perception, localization, TF Transform, Control
3 - Open other terminal to launch the cyber monitor:
source cyber/setup.bash
cyber_monitor

4- Open another terminal to launch the cyber_visualizer:
source cyber/monitor
cyber_visualizer

5- Access the active channels on cyber_monitor:

6 -Check inside a channel (localization for example) that there is flow of data:

7 -Afterwards Try to see this Data visualization ( Plot or image) on Cyber Visualizer:
Click over a channel and then press play or f8:

Unfortunately NO DATA
In a summarize In any channel is possible to see the data flow (in an image format or in programmatically format: Lists, csv, or other) just a black Screen.
With exception for Camera and Lidar Data Channels. For these channels the Cyber Visualizer works.
However I need to get data or plot for the other channels!
Currently Cyber Visualizer just support Camera and Lidar? What about the other channels data visualization. I mean Plot Graphically ? How is it possible?
I have already tried other methods, without success as you can check here: https://github.com/ApolloAuto/apollo/issues/13948
Thanks in advance

I have solved this issue and provided the tutorial here on my branch in apollo forked repo,  if someone be interested:
https://github.com/marcusvinicius178/apollo/tree/extract_chan_data/cyber/python/cyber_py3/channels_data_extraction


getInitialMessage() is not working for flutter

We are aware of this and have escalated to engineering. We'll update here when resolved.

Closing as per @fbenevides comment above.


Workflow: Migrate QEMU playbook check to github workflow

[ ] commit message has one of the standard prefixes
[ ] faq.md updated if appropriate
[ ] other documentation is changed or added (if applicable)
[ ] playbook changes run through VPC or QPC (if you have access)
[ ] VPC/QPC not applicable for this PR
[ ] for inventory.yml changes, bastillion/nagios/jenkins updated accordingly

Migrated QEMU playbook check functionality. Relevant docs here https://github.com/uraimo/run-on-arch-action
The gh action does not support riscv, so that will still need to remain in QPC.
This yaml file is alot easier to maintain than the entire QPC jenkins job, and its alot easier to add new arch distro combos

Not sure this is the right approach to force a github action to do this on every PR - will this not end up in a situation where all changes will result in a significant (potentially 1 day+) turnaround for PR approvals ... And repeated for all subsequent pushes/review fixes? I'd personaly prefer to stick with the existing QPC implementation where it can be run by individuals as required, and also on a scheduled regular basis to check the state of the playbooks.

Not sure this is the right approach to force a github action to do this on every PR - will this not end up in a situation where all changes will result in a significant (potentially 1 day+) turnaround for PR approvals

@sxa why not trigger it by adding a label? you could do something like this:
on:
  pull_request:
    types: [ labeled ]

jobs:
  build:
    if: ${{ github.event.label.name == 'run-playbook-checker' }}

It's an option but I'm not convinced there's a compelling advantage to running it that way unless we're fixated on running these things in GitHub

Its alot easier to add platforms; instead of creating a new disk, updating the QEMU bash script and the jenkins job, I can add 2 lines to the yaml file

Thats annoying. When one fails the rest get cancelled mid execution

Strange that it failed here
  TASK [Debug : Display Debug Information] ***************************************
  fatal: [localhost]: FAILED! => {"msg": "The task includes an option with an undefined variable. The error was: 'ansible_default_ipv4' is undefined\n\nThe error appears to have been in '/home/runner/work/infrastructure/infrastructure/ansible/playbooks/AdoptOpenJDK_Unix_Playbook/roles/Debug/tasks/main.yml': line 5, column 3, but may\nbe elsewhere in the file depending on the exact syntax problem.\n\nThe offending line appears to be:\n\n#####################\n- name: Display Debug Information\n  ^ here\n"}

Because all of the variables in debug default to undefined if they are undefined

I'm of the opinion that the Swap and Crontab roles should not run on non adoptium machines. They make changes to the machine that only we at adoptium need. Anyone else feel this way?

The gh action does not support riscv, so that will still need to remain in QPC.

@Haroon-Khel I've just pushed risc-v support to the action so you should be able to add that

QEMU workflows got triggered when I added the label to the pr. Works as expected 👍🏻

Looks like you have to remove and re add the label for subsequent pushes to the same pr

@sxa for review. This pr is at a stage where it can be merged

@Haroon-Khel Doesn't look like RISC-V has been added after the earlier comments.


47 wallets for airdrop hunter

Related Safe Addresses
0x8a7e2F5522a2B88c304B6E55012E4DE998B9E13E
0x0ffc126b095525a36a69d3db9b91426257386b88
0x807ef5e50a6a68fc485488c907099daf6a2d2a21
0x4c772b7025696ae8a8ea9b6cbd28b17ba408f10d
0x30415a59f058659870307d0BD1D86073f82dcAbE
0xd0208523563a28ef547cfa0d7baae41c2b406ed8
0x3a615f5732b1dbeb93dfe6017ae10c7ea20eaa90
0xf9138b750d72208baf94478ddd1a20900c64143f
0x83f11092798324e76a5d5feaf3e6ee87cac2fa5f
0xd214bc9f411d6f92ece8eca1af56a2c0bb913afb
0x92006804baa3ae202c3b377234e6177c2b465eed
0xc093eb788a0377b0edbc4b89d3d559c574e49a82
0xfebe2b6af497b9ac2d17eb008314dd1cf1c6a7aa
0xf143ed827c31c7f519445bcfa09281ec36b87c0c
0xa494159efed07ffd1067049c2f610418fe61a332
0x65269102997dce692fef1e2c6919363292db8ab9
0x2DcfA65f1fa4370ABF709E467Bd47A924C95F23F
0xb81617EfD1ef1D4f7B9986F35A8B557C6Ff323dA
0xf5390A093B960Af510C1565e0Ca846ddBE9434D6
0x54D0Aafd908983a96156d98906AFbe66b3fb901b
0xEd9A0b641ffB13f4E55F04dc1E533033D15Fcfe3
0x765e27419750D8c915f5112658602503CA1d57fD
0x36af31222f71E120840651B28Bc63D32508E96E4
0x3b5F1A9E08DFEb03002e07fbC0727525559138D2
0x116a2b47DE982BFD773B14a3E8C8C76e41370f96
0xb8CBC29faf29c32625f25A391993b993A33B132B
0x68A90b88C092d6601894D8D9beBe6547119d54e9
0x35803FaFd46790e847D3695051cB1218d049B633
0x80E766Ac23FF449a77551E7b1c57f1640b15874f
0xDeA4540CF784EEDD4ABd9EF0Cf6aC8e0Fd040D57
0x8fffAeC3b4C3eb69B144d541EC59677572A1957C
0x4bB8Ecd67a169745De327e7E8D6DBa35a7aB406c
0xA3CE148C6260816CF90619f76B446C174BA47b3C
0x322851CD0718619b5d1C11400AB4d7D82faBFB54
0xE91B29E483789CECA58C76BEDaD515f18A568EB0
0x1C95139a844259F2553f9B2b637bC40cf75326df
0xBdB1759712A7940f5509a8898e17BC7fefc575FB
0x46bf5165a1b1597d85703e615a0f406525fD4530
0x175F064566E8994733feb3A87A7076D6999F1014
0xCd201948494F4cC10f4A0a226Ff89292cb70CB6A
0x3067414F0967df2e2468882Af986b0b564C3a6d9
0x8d1a17B25b5d867AE5B230419130b7b50296702F
0x97b14970Dd2bbfE9C35CD98B4b194A8dcA1dB1AB
0x89d86e8461BfE282342761652F3D13Cdc58Ec9aF
0x75f175c2Af6887805F43C010d2867d20B57a93d1
0xe6B44A6481FcAa6C2035D4BFA7Ac68269aFD4cb5
0xD8F493cB313139D1BC727DA3Ada325131d3333FB
#Reasoning
initial account:  0x94cd68f311901d7cc151c48c62fa8cbdca6d2c04   this account creat  47 safe wallets in a short time. Most of thses wallets just have 2 or 3 same transcation from same addresses in same time.  it is clear that this guys is airdrop hunter.  All safe addresses are just for airdrop and no more transcation. i have checked all these wallets one by one.
Methodology
just discovered by excel and etherscan , it is so clearly for this airdrop hunter.
Safe Address
my safe wallet is   0x4c4682F81A95F6D76444e3ea20753AD5B7d643b4

already hunted in #24

initial account: 0x94cd68f311901d7cc151c48c62fa8cbdca6d2c04 this account creat 47 safe wallets in a short time. Most of thses wallets just have 2 or 3 same transcation from same addresses in same time. it is clear that this guys is airdrop hunter. All safe addresses are just for airdrop and no more transcation. i have checked all these wallets one by one.

I just checked the first address. There are more than 2-3 txs. This needs more info. Please elaborate on why this one is not legitimate usage.
https://gnosis-safe.io/app/eth:0x8a7e2F5522a2B88c304B6E55012E4DE998B9E13E/transactions/history

initial account: 0x94cd68f311901d7cc151c48c62fa8cbdca6d2c04 this account creat 47 safe wallets in a short time. Most of thses wallets just have 2 or 3 same transcation from same addresses in same time. it is clear that this guys is airdrop hunter. All safe addresses are just for airdrop and no more transcation. i have checked all these wallets one by one.

I just checked the first address. There are more than 2-3 txs. This needs more info. Please elaborate on why this one is not legitimate usage.
https://gnosis-safe.io/app/eth:0x8a7e2F5522a2B88c304B6E55012E4DE998B9E13E/transactions/history

I have already investigated it deeply, please see #24 if you have time~~~ I have tagged every addresses with a reason.

Closing this since #24 is closed as well


feat(design): introduce some usefull components

Description
Starting working on notebook
Validation
it's should be cool and cover all functionality
Related Issues
no related issues
Check List

[X] I have formatted using npx turbo format
[X] I have checked the types using npx turbo type-check
[X] I have tested the build using npx turbo build

next step is to do what you see what you get. and we'll have done the bulk of the components.


Update script for typescript-language-server

References
Looking into using typescript-language-server and following the docs to install it, it looks like jupyter_lsp was not able to find the language server:

Updating the script seems to be fixing it:

Code changes
Update the script for typescript-language-server.

User-facing changes
Users can install and use typescript-language-server with JupyterLab LSP.

Backwards-incompatible changes
This supports the latest version of typescript-language-server, but might break previous versions when the file was still cli.js?

Chores

[ ] linted
[ ] tested
[ ] documented
[ ] changelog entry

The test failures are relevant. It looks the location of the binary entry point has changed last year and we are pinning to an old version. I think we should bump the version, but also consider using the bin field from package.json.

I think we should bump the version

Done in https://github.com/jupyter-lsp/jupyterlab-lsp/pull/1064/commits/f014c9e9aaf58cc37e7c3b5abfd455f31d0e12a8 :+1:

but also consider using the bin field from package.json.

That makes sense, although there might be several scripts available, and the name might not match the name of the module itself?  I guess this would apply to other specs derived from NodeModuleSpec, so maybe something to track in a separate issue?

maybe something to track in a separate issue?

Opened https://github.com/jupyter-lsp/jupyterlab-lsp/issues/1066

@jtpio this is available in jupyter-lsp 2.2.4 - thanks again!


How to use the sync version?

I see that there is a sync version. How do I use it? I looked for but do not see any documentation.
I tried: require('csv-stringify/sync') without success.

require 'csv-stringify/lib/sync' and please take a look at the tests


Bug in README

There's a bug in the readme, stating that you should configure this gem with jekyll-rushed-analytics property in the _config.yml, but actually it should be jekyll_rushed_analytics, according to https://github.com/ap-automator/jekyll-rushed-analytics/blob/master/lib/jekyll-rushed-analytics.rb
Will provide a PR in a few

Here goes https://github.com/ap-automator/jekyll-rushed-analytics/pull/22

Actually, are documentation mistakes called bugs?

Thanks, I think it poorly documented.
But it create bug basically, have a nice day :)


[IOTDB-2564]Not allowed to activate template on a path not set template

Coverage decreased (-0.03%) to 67.716% when pulling 6022c58cd063f2e06cb9daf5fdedf240d3604197 on bigreybear:i2564 into f4ecf3a5b711a9985076946f3e10202404546c5a on apache:master.


Call to undefined method [package]

Hello,
when I include the service provider it throws me a Call to undefined method [package] exception,
I use Laravel 5 and have required "pingpong/modules": "2.0.*@dev" in the json.
What am I doing wrong?
Thank you very much!

Update: Sadly this package is no longer compatible with Laravel 5 as the package() method has been removed very recently. I really hope that you will update it.
Thanks!


Data format

Hello,
How should the data be kept in the ~/.torch/data/cifar10 directory to train SENet on CIFAR-10 data?
Thanks!

https://github.com/moskomule/senet.pytorch/blob/23839e07525f9f5d39982140fccc8b925fe4dee9/cifar.py#L11
Add download=True will download the dataset.


Automatically add a comment when a bug is moved to Core::Performance

Hey,
When a bug is moved to the Core::Performance component, our process involves adding a comment to request additional information from the reporter.
Would it be possible to automatically add this comment with a needinfo to the reporter whenever a bug is moved to Core::Performance?
Here is the text:
This bug was moved into the Performance component. Reporter, could you make sure the following information is on this bug?

 - For slowness or high CPU usage, capture a profile with http://profiler.firefox.com/ , upload it and share the link here.
 - For memory usage issues, capture a memory dump from about:memory and attach it to this bug.
 - Troubleshooting information: Go to about:support, click "Copy raw data to clipboard", paste it into a file, save it, and attach the file here.

Thank you.

We were thinking of some edge cases that should be taken care of:

if the bug is moved by mistake => the bot could wait for ~1 day before adding a comment, to allow for a user to move the bug again.
avoid adding the comment twice (if the bug is moved back and forth)

Thanks in advance!
cc @davehunt

Yes, it is totally possible.

I limited this automation to defect bugs. You could find a dry run in https://github.com/mozilla/relman-auto-nag/pull/1916#issuecomment-1454818117
@julienw should we ignore meta bugs?

Hey Suhaib, thanks for looking at this!
I believe it makes sense to limit to defect bugs. Ignoring meta bugs sounds like a good idea too! (even if they probably shouldn't be defect in the first place?).

Ignoring meta bugs sounds like a good idea too! (even if they probably shouldn't be defect in the first place?).

Thank you! Here is an example of a meta bug that its type is defect: https://bugzilla.mozilla.org/show_bug.cgi?id=1260981. Such cases will be excluded.


Missing web support

I can't see that this package support web on pub.dev

We are building a mobile/web app at the moment. We'll add web support soon.

This will be an ongoing process so I'll close this issue.

any update on this ?


Export to ONNX

Bug description
Hi, I'm trying to convert my model Detr to onnx ,  I followed the guide but I get this error:
f"torch>=2.0 requires onnx to be installed to use {type(self).__name__}.to_onnx()"
I am working on Google Colab.
Can you help me?
What version are you seeing the problem on?
master
How to reproduce the bug
checkpoint_path = '/content/lightning_logs/version_0/checkpoints/max.ckpt'
checkpoint = torch.load(checkpoint_path)

# Crea una nuova istanza del modello e assegna i pesi del checkpoint
model = Detr(lr=1e-4, lr_backbone=1e-5, weight_decay=1e-4)
model.load_state_dict(checkpoint['state_dict'])
model.to_onnx("test.onnx",  export_params=True)

Error messages and logs
# Error messages and logs here please

ModuleNotFoundError: torch>=2.0 requires onnx to be installed to use Detr.to_onnx()
1362             )
1361                 f"torch>=2.0 requires onnx to be installed to use {type(self).__name__}.to_onnx()"
-> 1360             raise ModuleNotFoundError(
1359         if _TORCH_GREATER_EQUAL_2_0 and not _ONNX_AVAILABLE:
1358         """
/usr/local/lib/python3.10/dist-packages/pytorch_lightning/core/module.py in to_onnx(self, file_path, input_sample, **kwargs)
1 frames
----> 1 model.to_onnx("test.onnx",  export_params=True)
 in <cell line: 1>()
ModuleNotFoundError                       Traceback (most recent call last)
Environment

Current environment
#- Lightning Component (e.g. Trainer, LightningModule, LightningApp, LightningWork, LightningFlow):
#- PyTorch Lightning Version (e.g., 1.5.0):
#- Lightning App Version (e.g., 0.5.2):
#- PyTorch Version (e.g., 2.0):
#- Python version (e.g., 3.9):
#- OS (e.g., Linux):
#- CUDA/cuDNN version:
#- GPU models and configuration:
#- How you installed Lightning(`conda`, `pip`, source):
#- Running environment of LightningApp (e.g. local, cloud):

More info
No response

@MarcoPrassel You need to install the onnx package as the error tells you:
pip install onnx

@awaelchli

Thank you @awaelchli ,
I had installed onnx but the error was always present, now everything seems to be OK, although I get this error:
Could not export to ONNX since neither input_samplenormodel.example_input_array attribute is set.
I am not an expert,  this is the first time I use pytroch lightning, my goal is to export the model trained with Detr to the onnx format, can you explain how I can do this?

@MarcoPrassel Just pass in the input sample argument as suggested by the error message. Here are the docs with examples: https://lightning.ai/docs/pytorch/stable/deploy/production_advanced.html#compile-your-model-to-onnx
You can find these by just typing "onnx" in to the search bar.

Let me know if you have any more issues :)

@awaelchli
I have an erro when using  to_onnx() method.
The model's forward like this:
def forward(self, windows_batch):
        insample_y = windows_batch["insample_y"]
        insample_mask = windows_batch["insample_mask"]
        futr_exog = windows_batch["futr_exog"]
        hist_exog = windows_batch["hist_exog"]
        stat_exog = windows_batch["stat_exog"]
                        ...
                        ...

I using the to_onnx() method like this:
trained_md.eval()
data_=trained_md.dataset
in_=iter(data_)
in_=next(in_)
input_names=[]
input_dict = {}
for itm in in_:
    if in_[itm] is not None:
        input_dict[itm] = in_[itm][-1:]
        input_names.append(itm)
in_=input_dict
trained_md.to_onnx(file_path='Nbx.onnx',
                   input_sample=in_,
                   input_names=input_names,

Then give me an erro:
TypeError: NBEATSx.forward() missing 1 required positional argument: 'windows_batch'

How can I fix this issue?


Deprecate UI1.0 related properties

This PR deprecates remnants of UI1.0, this mostly includes properties of already deprecated types.

@GerardasB @NancyMcCallB @raplemie
did we want this in 3.5?
release branch has been created so we need to bp this if so
cc @skirby1996

@mdastous-bentley @MarcBedard8 plz review. just a deprecation warning

@GerardasB @NancyMcCallB @raplemie did we want this in 3.5?
release branch has been created so we need to bp this if so
cc @skirby1996

Yes, I think we need to cherry-pick this back if it gets merged in time.

@GerardasB @NancyMcCallB @raplemie did we want this in 3.5?
release branch has been created so we need to bp this if so
cc @skirby1996

Not necessarily, I think it's fine if we get this only in to 3.6.


Support for webpack4

Fixes chunk cleanup for stats and uses new hook format for integrating with webpack@4.

Hi @bookwyrm! Thanks for the contribution!
Do you think you'd be able to leave the webpack3 path in place too, so this is compatible with webpack 3 and 4?
An example of how this has been done before is here! https://github.com/slackhq/csp-html-webpack-plugin/blob/master/plugin.js#L138-L154
Again, really appreciate the contribution :)

Is there any news on this PR at all?
Was hoping to use this in a new Webpack 4 build that I'm working on 😃

@brettsmason I just haven't had a chance to get back to this to make the suggested updates so that it works with webpack@3. If you want, you can install from my fork to get the behavior with webpack@4 in your project.
npm install --save-dev git+https://github.com/bookwyrm/webpack-extraneous-file-cleanup-plugin.git#webpack4

@AnujRNair Just pushed an update to keep webpack3 based on the provided example. I have confirmed that it still works with webpack4 but haven't tested webpack3.

Thank you @bookwyrm! I just published a new version to npm with your changes - much appreciated!


Allow to set fields to Default when constructing struct from query

This commit adds a #[diesel(default)] field attribute to
#[derive(Queryable)], that allows a user to set a field to a default
value instead of loading it from the database. This is usefull if the
same struct is used to serve some API that requires additional
information to be added.

:-1: from me on this idea. It's too unclear what this does, and I don't think there's an easy way to make it obvious. I don't think it's obvious whether this ignores a database column, populates it on null, or changes the expected SQL type.

Do you see a problem with this feature in general, or with the naming?
In the second case we could do something similar to serde and use #[diesel(skip)].

@sgrif To point to a practical use of this, see this example from my wundergraph crate.
If there is any other way to do something similar that does not involve manually implementing Queryable I'm fine with using it.

With the feature in general. I think this affects more things in non-obvious ways than serde-skip does.

So is there any other way to solve this without manually implementing Queryable on each struct needing this?


Upgrade our stack to trtllm 0.13

https://github.com/NVIDIA/TensorRT-LLM/releases/tag/v0.13.0

Marking closed


How to pass error messages from backend.

Hi
We are having a small issue, where all of the error messages come from the proxy and are not passed from the backend application.
Inside the docker container
List of files that are involved.
root@apache-modsecurity-proxy:/usr/local/apache2/conf/extra/httpd-multilang-errordoc.conf
root@apache-modsecurity-proxy:/usr/local/apache2/error/*
root@apache-modsecurity-proxy:/usr/local/apache2/conf/httpd.conf

Now the problem is that inside the container the apache configuration httpd.conf does NOT include the httpd-multilang-errordoc.conf by default. It is commented out.
I can't seem to find any other place where the custom error messages are loaded in.

Hi!
which error are you trying to show?

I will expand on this a little.
We have a keycloak installation and an application that use it for authentication.
Keycloak uses the modsecurity-crs:apache and the application uses the modsecurity-crs:nginx
Keycloak should return an 401 unauthorized to the user, but it returns this.

The problem is that apache is doing ProxyErrorOverride on.

Quick option for testing this: get this file, save it locally. Change ProxyErrorOverride to off. Use docker-compose or other to volume mount your local file over the original. Test and see if that solves your problem.

Yes, that worked, thank you for the help!

We can add that as an env variable.


pbmm2 illegal instruction

Operating system
Which operating system and version are you using?
CentOS
Package name
pbmm2 1.0.0 (commit 1.0.0)
Which package / tool is causing the problem? Which version are you using, use tool --version. Have you updated to the latest version conda update package? Have you updated the complete env by running conda update --all? Have you ensured that your channel priorities are set up according to the bioconda recommendations at https://bioconda.github.io/#set-up-channels?
Conda environment
What is the result of conda list? (Try to paste that between triple backticks.)
Describe the bug
Running the executable with no options (or any options) results in "Illegal instruction"
Error message
illegal instruction
To Reproduce
pbmm2
(just running)
This architecture crashes:
cpu family      : 16
model           : 9
model name      : AMD Opteron(tm) Processor 6176
stepping        : 1
microcode       : 0x10000d9
cpu MHz         : 800.000
cache size      : 512 KB
physical id     : 0
siblings        : 12
core id         : 0
cpu cores       : 12
apicid          : 16
initial apicid  : 0
fpu             : yes
fpu_exception   : yes
cpuid level     : 5
wp              : yes
flags           : fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ht syscall nx mmxext fxsr_opt pdpe1gb rdtscp lm 3dnowext 3dnow constant_tsc art rep_good nopl nonstop_tsc extd_apicid amd_dcm pni monitor cx16 popcnt lahf_lm cmp_legacy svm extapic cr8_legacy abm sse4a misalignsse 3dnowprefetch osvw ibs skinit wdt nodeid_msr hw_pstate retpoline_amd ibp_disable vmmcall npt lbrv svm_lock nrip_save pausefilter
bogomips        : 4600.03
TLB size        : 1024 4K pages
clflush size    : 64
cache_alignment : 64
address sizes   : 48 bits physical, 48 bits virtual
The following architecture works fine:
processor       : 0
vendor_id       : GenuineIntel
cpu family      : 6
model           : 26
model name      : Intel(R) Xeon(R) CPU           E5540  @ 2.53GHz
stepping        : 5
microcode       : 0x1d
cpu MHz         : 1600.000
cache size      : 8192 KB
physical id     : 0
siblings        : 4
core id         : 0
cpu cores       : 4
apicid          : 0
initial apicid  : 0
fpu             : yes
fpu_exception   : yes
cpuid level     : 11
wp              : yes
flags           : fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush dts acpi mmx fxsr sse sse2 ss ht tm pbe syscall nx rdtscp lm constant_tsc arch_perfmon pebs bts rep_good nopl xtopology nonstop_tsc aperfmperf eagerfpu pni dtes64 monitor ds_cpl vmx est tm2 ssse3 cx16 xtpr pdcm dca sse4_1 sse4_2 popcnt lahf_lm ssbd ibrs ibpb stibp tpr_shadow vnmi flexpriority ept vpid dtherm ida spec_ctrl intel_stibp flush_l1d
bogomips        : 5053.99
clflush size    : 64
cache_alignment : 64
address sizes   : 40 bits physical, 48 bits virtual
power management:
The difference between flags is:
< 3dnow
< 3dnowext
< 3dnowprefetch
< abm
< amd_dcm

acpi
aperfmperf
7c5,6
< art

arch_perfmon
bts
10d8
< cmp_legacy
12d9
< cr8_legacy
14a12
dca
16,17c14,22
< extapic
< extd_apicid

ds_cpl
dtes64
dtherm
dts
eagerfpu
ept
est
flexpriority
flush_l1d
20d24
< fxsr_opt
22,24c26,29
< hw_pstate
< ibp_disable
< ibs

ibpb
ibrs
ida
intel_stibp
26d30
< lbrv
30d33
< misalignsse
32d34
< mmxext
36d37
< nodeid_msr
39,40d39
< npt
< nrip_save
42d40
< osvw
45,46c43,45
< pausefilter
< pdpe1gb

pbe
pdcm
pebs
54d52
< retpoline_amd
56c54,56
< skinit

spec_ctrl
ss
ssbd
59,61c59,62
< sse4a
< svm
< svm_lock

sse4_1
sse4_2
ssse3
stibp
62a64,66
tm
tm2
tpr_shadow
65,66c69,73
< vmmcall
< wdt

vmx
vnmi
vpid
xtopology
xtpr

Finally, minimap2 runs fine on both cpus

You are right. I forgot to add that to the FAQ. We explicitly compile with sse4_1 to have the speed boost. We currently don't support CPUs without that instruction set. Have you compared the speed differences between those two servers for minimap2? I'm curious.

No, but I can. I can also just exclude these nodes from cluster
submission.. they make up ~30% of our cores and 70% is more than what I'm
allowed to use. I thought it would be a bigger issue but jobs were landing
on the opteron nodes with priority.
On Fri, Apr 26, 2019 at 10:42 AM Armin Töpfer<EMAIL_ADDRESS>wrote:

You are right. I forgot to add that to the FAQ. We explicitly compile with
sse4_1 to have the speed boost. We currently don't support CPUs without
that instruction set. Have you compared the speed differences between those
two servers for minimap2? I'm curious.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/PacificBiosciences/pbbioconda/issues/136#issuecomment-487141445,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAKPGPUH2SE6WFKFTR36W6DPSM5H3ANCNFSM4HIW33IA
.

Please use those nodes that support sse4_1 and I'll try to add a proper runtime error.


fix: Update checkbox colors to the correct values

SUMMARY:
Fixing colors on checkbox based on Yair's comments here
GITHUB ISSUE (Open Source Contributors)
JIRA TASK (Eightfold Employees Only):
https://eightfoldai.atlassian.net/browse/ENG-23851
CHANGE TYPE:

[x] Bugfix Pull Request
[ ] Feature Pull Request

TEST COVERAGE:

[ ] Tests for this change already exist
[ ] I have added unittests for this change

TEST PLAN:

Codecov Report

:exclamation: No coverage uploaded for pull request base (main-next@89d7d37). Click here to learn what that means.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             main-next     #230   +/-   ##
============================================
  Coverage             ?   87.02%
============================================
  Files                ?      208
  Lines                ?     6861
  Branches             ?     2283
============================================
  Hits                 ?     5971
  Misses               ?      875
  Partials             ?       15

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 89d7d37...33a80de. Read the comment docs.


(fix): allow setting X as sparse when X is dense view

[x] Closes #768
[x] Tests added
[x] Release note added (or unnecessary)

@flying-sheep I guess I just wasn't even sure this was the desired result but your approval says that we are on the right track.  I will add a warning.


HTML Element Handling

Which nodes matter? For example, here are some we're currently not processing:
<html lang="en-US">
	<head>
		<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
	</head>
	<body data-absolute-enabled="true" style="font-family:Calibri;font-size:11pt">

Partially addressed by commits leading up to 3d7eedc4b34. We handle:

title
creation timestamp
links (anchors)
images


Spring Security SDK

Summary

I have noticed when developing several Java/Spring/Spring Security based apps that use Yoti Auth (My Venue, Tags, Connections, Reed Demo) there is often a common need to write similar code that essentially is:

A Controller or Filter to get the Yoti token.
A service to call the Yoti client  with the token value.
If using spring security, you'll also need an AuthenticationToken and AuthenticationProvider to populate the SecurityContext with a Principal.

Now I have added a new module that makes integration with Spring Security more straight forward (e.g. this module allows the client application developer to use familiar Spring Security concepts like @Secured, AuthenticationTokens, AuthenticationProviders, Principals and so on.

Spring Boot Sample

Updated the sample to use this new lib to demonstrate how it might be used. Now it's only necessary to implement a YotiUserService, a Principal and configure the application to use the provided Filter and the app is good to go. No configuring a Yoti Client or implementing an endpoint (like /login to get the token and decode it). This is all done for you and you just provide a YotiUserService that will be called when someone logs in with Yoti.

Spring Boot Auto Configuration

I have noticed when developing several Java/Spring based apps that use Yoti Auth (My Venue, Tags, Connections, Reed Demo) there is often a common need to configure the dashboard application ID (as well as the SDK ID) so that you can direct users to the connect login page. Consequently, I have added a common auto config properties object for this to reduce duplication. It is entirely optional and clients can decide not to use it.

General Clean Up & Documentation

Applied some IDE auto formatting and bug fixes to all modules.
Added extra documentation for new modules and updated versions.
Added known issue for com.yoti.api.client.HumanProfile#getFullName method.

Deprecated broken API method

See https://github.com/getyoti/java/issues/14 for full description.
Deprecated the com.yoti.api.client.HumanProfile#getFullName method, provided a new convenience method and prepared libs for new release.

Artefact Publication
The following module versions have been created/updated and need to be deployed to Maven Central if this PR is merged/approved:

java-sdk-spring-boot-auto-config:1.2
java-sdk-impl:1.2
java-sdk-api:1.2
yoti-java-sdk-spring-security:1.0-SNAPSHOT

@qzagarese agreed, good points.
I'll revert changes to this module and resubmit PR.

@qzagarese I have reverted the example module back to the way it worked before the security module sample. I've made the security module 1.0-SNAPSHOT version. Hopefully other changes have also been adequately addressed via the comments above as requested.
Let me know if you believe further changes are required before being acceptable to merge.

@qzagarese thank you Mr Zagarese.

@davidgoate releasing now. Thank you for this nice contribution :-)

@davidgoate release complete --> https://oss.sonatype.org/#nexus-search;quick~yoti