InvertObj yields unexpected results when called with a deep nested object

var obj = {
    a: { foo: 'bar' },
    b: { baz: 'bam' }
}
var invertedObj = R.invertObj(obj); // Object {[object Object]: "b"}

Prop "a" is lost without trace (this also happens if both "a" and "b" have the same value .e.g Object {foo: "b"})
Object.keys(invertedObj).length !== Object.keys(obj)
The initial value is replaced by its toString() value making this operation irreversible.

This is very subtle bug to figure out, maybe an error/complete invalidation of result might save the day? Or a debate if the benefits of this function out weight the possible bugs that might occur?
The following is a naive JSON implementation but does it make sense?
  // Extracted from https://github.com/ramda/ramda/blob/master/src/invertObj.js

  // Add N space characters together
  var spacesN = R.compose(R.join(''), R.times(R.always(' ')));
  var history = [];
  var newKey = "";

  // Added conditional
  if (typeof obj[key] === 'object') {
    newKey = JSON.stringify(obj[key]);
  } else {
    newKey = obj[key]
  }
  // Makes sure duplicates don't overwrite each other
  if (history.indexOf(newKey) !== -1) {
     newKey += spacesN(idx);
  }
  history.push(newKey);
  out[newKey] = key;

Of course, the above code might introduce even more subtle bugs.

This function won't lose the duplicate: invert.
I'm afraid that neither makes much sense if your values are not Strings (or easily converted to Strings.)
Could you put your code together as a complete sample function, either here or in a gist, or in the repl?

I've put some code together here:
da7edbdebcefe605095e8b2031c9c2316203c4fa
If this is a good idea I can add further support for non-primitive types and get some tests going for those.

Ok, I see what you're doing.  Honestly, this seems like quite bizarre behavior to me.
My thought is that this is simply a case of "Doctor it hurts when I do this."  "Then don't do that!".
I simply don't see a good reason to use invert on objects that aren't simple, one-level deep data objects.  If you need more sophisticated processing, then you probably need a different tool.
What do others think?

I think this would be asking too much of invertObj.

I totally agree, the scope would just expand. An object manipulation library would be better suited for this.
Maybe adding a heads up to the invertObj docs instead?
b67ba27819815bd2284387c75db0e546b876b30a
Just for the reference, my use case was something on the lines:
var changeFilename = R.compose(R.inverObj, R.mapObj(R.add('path/to/file')), R.invertObj)

changeFilename({
  'filename': jsonData
  ....
})
//=> {
  'path/to/filename': jsonData
}

Check out mapKeys in the Cookbook: https://github.com/ramda/ramda/wiki/Cookbook#map-keys-of-an-object
var changeFilename = mapKeys(R.add('path/to/file'));

Amazing! Thank you for that, makes total sense!
Should I do a pull request for adding to the invertObj docs? https://github.com/ramda/ramda/commit/b67ba27819815bd2284387c75db0e546b876b30a

Yes, that would be worth including in the docs.  Thanks!

Great! I did a pull request just now https://github.com/ramda/ramda/pull/1407
I think its safe to close this issue thread. Thanks for all the help!


Fix form sizing classes for Bootstrap 4

For BS4, this removes the size class of the .form-group, and applies the appropriate sizes to the form control (input) and label (the latter only for horizontal layouts).
Fixes #494
cc @basz

confirmed to work! thanks

@basz great, thanks for trying it out in advance! Just have to get Travis happy...


Avoid keyboard

How can I avoid the keyboard with the modal? I tried KeyboardAvoidingView but that doesn't seem to do anything

I had the same issue when I started working on it and it is still open on react-native issues.
Did you find any workaround?

@mmazzarolo I just added a marginBottom and it then moves out of the way

Oh. That's nice 👍

It's still something to keep in mind, though. It's not a fix but a workaround :)


What qtile version is supported?

I'm getting following error both with 0.10.7 and recently released 0.11.0:
During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/nazar-pc/.local/lib/python3.6/site-packages/plasma/layout.py", line 64, in add
    self.root.restore(new)
  File "/home/nazar-pc/.local/lib/python3.6/site-packages/plasma/node.py", line 560, in restore
    raise NotRestorableError()
plasma.node.NotRestorableError

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/nazar-pc/.local/bin/qtile", line 11, in <module>
    load_entry_point('qtile', 'console_scripts', 'qtile')()
  File "/web/github/qtile/libqtile/scripts/qtile.py", line 125, in main
    q = make_qtile()
  File "/web/github/qtile/libqtile/scripts/qtile.py", line 119, in make_qtile
    state=options.state,
  File "/web/github/qtile/libqtile/manager.py", line 245, in __init__
    self.scan()
  File "/web/github/qtile/libqtile/manager.py", line 568, in scan
    self.manage(item)
  File "/web/github/qtile/libqtile/manager.py", line 627, in manage
    c = window.Window(w, self)
  File "/web/github/qtile/libqtile/window.py", line 770, in __init__
    group.add(self)
  File "/web/github/qtile/libqtile/group.py", line 270, in add
    i.add(win)
  File "/home/nazar-pc/.local/lib/python3.6/site-packages/plasma/layout.py", line 66, in add
    node.add_node(new, self.add_mode)
  File "/home/nazar-pc/.local/lib/python3.6/site-packages/plasma/node.py", line 539, in add_node
    self.parent.add_child_after(node, self)
  File "/home/nazar-pc/.local/lib/python3.6/site-packages/plasma/node.py", line 498, in add_child_after
    self.add_child(new, idx=old.index+1)
  File "/home/nazar-pc/.local/lib/python3.6/site-packages/plasma/node.py", line 495, in add_child
    Node.fit_into(node.siblings, total - (total / len(self)))
TypeError: unsupported operand type(s) for /: 'NoneType' and 'int'

Thanks for reporting!
I think I found a way to reproduce the bug. Will let you know when I got it fixed.

The bug is triggered when windows are added to the layout but the layout's configure() has not yet been called. In that case the screen dimensions are still unknown to the layout and it tries to do math with None values.
Do you mind checking if the patch fixes the problem for you?

Is this an issue on the qtile side? It seems like that shouldn't be
happening.
On Thu, Mar 1, 2018, 12:11 Arminius<EMAIL_ADDRESS>wrote:

The bug is triggered when windows are added to the layout but the layout's
configure() has not yet been called. In that case the screen dimensions
are still unknown to the layout and it tries to do math with None values.
Do you mind checking if the patch fixes the problem for you?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/numirias/qtile-plasma/issues/10#issuecomment-369662430,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AApkhUL02Dw_J8U8AApVVlbRYWx5diCOks5taCu9gaJpZM4SYEt2
.

#11 works for me


6.2.5 Operators

and_or = pipeline { ( '&&' | '||' ) {'\n'} pipeline } ;

Auto-merging src/ast/ast.c
CONFLICT (content): Merge conflict in src/ast/ast.c
Auto-merging src/ast/ast.h
CONFLICT (content): Merge conflict in src/ast/ast.h
Auto-merging src/ast/execution.c
CONFLICT (content): Merge conflict in src/ast/execution.c
Auto-merging src/lexer/lexer.c
CONFLICT (content): Merge conflict in src/lexer/lexer.c
Auto-merging src/lexer/lexer.h
CONFLICT (content): Merge conflict in src/lexer/lexer.h
Auto-merging src/parser/parser.c
CONFLICT (content): Merge conflict in src/parser/parser.c
Automatic merge failed; fix conflicts and then commit the result.


undefined method `read' for nil:NilClass when calling any endpoint

Checklist

[x] Upgrade Jets: Are you using the latest version of Jets? This allows Jets to fix issues fast. There's a jets upgrade command that makes this a simple task. There's also an Upgrading Guide: http://rubyonjets.com/docs/upgrading/
[ ] Reproducibility: Are you reporting a bug others will be able to reproduce and not asking a question. If you're unsure or want to ask a question, do so on https://community.boltops.com
[x] Code sample: Have you put together a code sample to reproduce the issue and make it available?  Code samples help speed up fixes dramatically. If it's an easily reproducible issue, then code samples are not needed. If you're unsure, please include a code sample.

My Environment

Software
Version

Operating System

Jets
5.0.13

Ruby
3.2.2

Expected Behaviour
Every call to the API should return an ok status even if lambdas

Current Behavior

Calling the api endpoint using api gateway tester:

Raises this error in the export controller lambda:
NoMethodError (undefined method `read' for nil:NilClass):

app/controllers/export_controller.rb:26:in `update_data'
app/controllers/export_controller.rb:3:in `create'
app/controllers/ExportController.rb:1:in `run'

I was having the same error when deploying but was fixed here
I have found some reference pointing to this method on jets, and looks invoke method for lambda client comes with payload nil or unavailable for some reason.
{
    "errorMessage": "undefined method `read' for nil:NilClass",
    "errorType": "Function<NoMethodError>",
    "stackTrace": [
        "/opt/ruby/gems/3.2.0/gems/jets-5.0.12/lib/jets/commands/call/caller.rb:78:in `remote_run'",
        "/opt/ruby/gems/3.2.0/gems/jets-5.0.12/lib/jets/commands/call/caller.rb:31:in `run'",
        "/opt/ruby/gems/3.2.0/gems/jets-5.0.12/lib/jets/job/base.rb:39:in `perform_later'",
        "/var/task/app/jobs/process_job.rb:30:in `block in aggregate_data'",
        "/opt/ruby/gems/3.2.0/gems/activerecord-<IP_ADDRESS>/lib/active_record/relation/delegation.rb:88:in `each'",
        "/opt/ruby/gems/3.2.0/gems/activerecord-<IP_ADDRESS>/lib/active_record/relation/delegation.rb:88:in `each'",
        "/var/task/app/jobs/process_job.rb:29:in `aggregate_data'",
        "/var/task/app/jobs/process_job.rb:36:in `story_aggregate_process'",
        "/var/task/app/jobs/process_job.rb:16:in `process_one_story'",
        "/opt/ruby/gems/3.2.0/gems/jets-5.0.12/lib/jets/job/base.rb:28:in `process'",
        "/opt/ruby/gems/3.2.0/gems/jets-5.0.12/lib/jets/exception_reporting.rb:37:in `block in process'",
        "/opt/ruby/gems/3.2.0/gems/jets-5.0.12/lib/jets/exception_reporting.rb:8:in `with_exception_reporting'",
        "/opt/ruby/gems/3.2.0/gems/jets-5.0.12/lib/jets/exception_reporting.rb:36:in `process'",
        "app/jobs/process_job.rb:1:in `run'",
        "/opt/ruby/gems/3.2.0/gems/jets-5.0.12/lib/jets/processors/main_processor.rb:32:in `instance_eval'",
        "/opt/ruby/gems/3.2.0/gems/jets-5.0.12/lib/jets/processors/main_processor.rb:32:in `run'",
        "/opt/ruby/gems/3.2.0/gems/jets-5.0.12/lib/jets/core.rb:171:in `process'",
        "/var/task/handlers/jobs/process_job.rb:9:in `process_one_story'"
    ]
}

Step-by-step reproduction instructions

Code Sample
export_controller.rb:
class ExportController < ApplicationController
  def create
    export_params[:update_data].present? ? update_data : export_csv

    render json: { message: "Export and Updating Data in progress" }
  end

  private

  def export_params
    params.permit(:story_id, :locale, :email, :update_data, :update_chart_data)
  end

  def export_csv
    ExportJob.perform_later(:export, { story_id: export_params[:story_id],
                                       locale: export_params[:locale],
                                       email: export_params[:email] })
  end

  def update_data
    ProcessJob.perform_later(:process_one_story, { story_id: export_params[:story_id],
                                                   locale: export_params[:locale],
                                                   email: export_params[:email] })
  end
end

process_job.rb
# frozen_string_literal: true

class ProcessJob < ApplicationJob
  class_managed_iam_policy(
    'service-role/AWSLambdaBasicExecutionRole'
  )
  def process_data
    Story.aggregated_viewings.find_each do |story|
      story_aggregate_process(story)
    end
  end

  def process_one_story
    story_id = event["story_id"]
    story = Story.includes(:viewings).find_by(id: story_id)
    story_aggregate_process(story)
    { message: "Data processing story: #{story_id} - done" }
  end

  private

  def remove_data(story)
    DataAggregation.where(story_id: story).delete_all
    story.viewings.where(started: true).update_all(aggregated: false)
    story.update_attribute(:update_csv, false)
  end

  def aggregate_data(story)
    viewings = story.viewings
    viewings.where(aggregated: false, started: true).each do |viewing|
      AggregateJob.perform_later(:aggregate_data, { story: story.id, viewing: viewing.id })
    end
  end

  def story_aggregate_process(story)
    remove_data(story) if story.update_csv
    aggregate_data(story)
  end
end

aggregate_job.rb
# frozen_string_literal: true

class AggregateJob < ApplicationJob
  class_reserved_concurrent_executions 10

  def aggregate_data
    perform_data(event)
  end

  private

  def perform_data(event)
    story = Story.find_by(id: event["story"])
    viewing = Viewing.find_by(id: event["viewing"])

    aggregate_data_for_language(story, viewing)
    viewing.update_attribute(:aggregated, true)
  end

  def aggregate_data_for_language(story, viewing)
    story.transcription_languages.each do |language|
      csv_row, single_row, multi_row = Export::Answers.new(story: story, language: language, viewing: viewing ).generate_csv_line
      data_aggregation = build_update_attributes(viewing, language, csv_row, single_row, multi_row, story)
      DataAggregation.update_or_create_by({ viewing_id: viewing.id, locale: language }, data_aggregation)
    end
  end
  def build_update_attributes(viewing, language, csv_row, single_row, multi_row, story)
    {
      story_id: story.id,
      viewing_id: viewing.id,
      data_order: viewing.start_time,
      multi_choice_data: multi_row.to_csv,
      single_choice_data: single_row.to_csv,
      data: csv_row.to_csv,
      locale: language
    }
  end
end

config/routes.rb
Jets.application.routes.draw do
  root "jets/welcome#index"

  get 'aggregate', to: 'aggregate#process'
  resources :export, only: [:create]

  # Reveal health status on /up that returns 200 if the app boots with no exceptions, otherwise 500.
  # Can be used by load balancers and uptime monitors to verify that the app is live.
  get "up" => "jets/health#show", as: :jets_health_check

  # The jets/public#show controller can serve static utf8 content out of the public folder.
  # Note, as part of the deploy process Jets uploads files in the public folder to s3
  # and serves them out of s3 directly. S3 is well suited to serve static assets.
  # More info here: https://rubyonjets.com/docs/extras/assets-serving/
  any "*catchall", to: "jets/public#show"
end

config/application.rb
module ExportData
  class Application < Jets::Application
    config.load_defaults 5.0

    config.project_name = "export-data"
    config.mode = "api"

    config.prewarm.enable = true

    config.controllers.default_protect_from_forgery = false

    if Jets.env == 'production'
      config.function.vpc_config = {
        security_group_ids: ENV['SECURITY_GROUP_IDS'].split(','),
        subnet_ids: ENV['SUBNET_IDS'].split(',')
      }
    end
    # Docs:
    # https://rubyonjets.com/docs/config/
    # https://rubyonjets.com/docs/config/reference/
  end
end

Solution Suggestion
Since it looks it's for logging purpose, we could validate response would be validate payload it's not nil before trying to access it.

Don't think this applies anymore with the release of Jets 6

https://blog.boltops.com/2024/05/23/ruby-on-jets-6-0-rails-support/


Standardize project in getting-started-dbt-core.md

Description & motivation

The repo name is dbt-tutorial, and screenshots show the path ~/dbt-tutorial/..., but one path references dat-projects.
Prerelease docs
If this change is related to functionality in a prerelease version of dbt (delete if not applicable):

[ ] I've added versioning components, as described in "Versioning Docs"
[ ] I've added a note to the prerelease version's Migration Guide

Checklist
If you added new pages (delete if not applicable):

[ ] The page has been added to website/sidebars.js
[ ] The new page has a unique filename

If you removed existing pages (delete if not applicable):

[ ] The page has been removed from website/sidebars.js
[ ] An entry has been added to _redirects
[ ] Run link testing to update the links that point to the deleted page

Looks like it's failing for unrelated reasons (can't find architecture for build). :(


Fix service dependency in multi-cloud

What type of PR is this?
Issue Fix PR

Uncomment only one  /kind <> line, hit enter to put that in a new line, and remove leading whitespace from that line:
/kind new feature

kind bug fix

/kind cleanup
/kind revert change
/kind design
/kind documentation
/kind enhancement

What this PR does / why we need it:
Fixes the order of service start up so that services do not fail.
Which issue(s) this PR fixes:

Fixes #1237
Test Report Added?:

Uncomment only one  /kind <> line, hit enter to put that in a new line, and remove leading whitespace from that line:
/kind TESTED
/kind NOT-TESTED

Test Report:

Special notes for your reviewer:

@AmitRoushan , @asifdxtreme, @PravinRanjan10  Please review .

IMO, The PR need still missed out some services (datamover and dataflow) for datastore dependencies.

IMHO, #1237 issue can only be closed after handling inter dependencies of all services.

Services has dependency on tidb too. Should we handle that too?

IMO, The PR need still missed out some services (datamover and dataflow) for datastore dependencies.

Handled all the services


display traffic data on gauge in a readable way

The spent traffic is currently displayed in arbitrary units (percentage?), which imho is not what a user wants to know.
I would propose simply displaying the current credit, preferably on a fixed scale up to the maximum credit to provide a quick way to check it.
Users in need of more detailed information (e.g. because they are very low on credit) can still look it up in the usersuite.

@toothstone That is false, since the limits “0-100” are the default values which are automatically set by the gauge plugin. Setting the max value to 0 causes exactly this.
For the non-standard behaviour (with data) see the picture:

Ok, that looks way better, but since a lot of people will visit the page from external IPs we might want to make that view look better as well.
￼
However, I still dont think the normal user is interested in the spent traffic, people just want to see at a glance how soon they will run out.

With “how soon they will run out”, do you imply a “time left” approach, or a “traffic left”? Because how much traffic is left is just what the gauge displays. When running out of traffic, the bar almost completely covers the chart and is automatically gets more and more red (see (this)[http://justgage.com/] ⇒ Demos ⇒ Show color by sector)

Time left is obviously pretty much impossible, as I stated in my first I would just prefer to have credit displayed instead of spent volume. A fixed scale would be faster to read and probably easier to understand for newbies. But lets just wait for feedback from real users, if ppl complain we can easily change it.

I too suggest traffic volume left instead of traffic volume spent as this conveys the information the user is searching for in the best manner. Gauges typically resemble fuel and the like, which is good analogy for the traffic they have.
The maximum of the gauge should be static and not dynamic (I don't how it's done currently). Set it always to the maximum amount of traffic one could save, i.e. currently 63 GiB.


Initially failed to switch to PR mode when checking out PR

In my multiroot workspace, I found PR 212501 in "Assigned to Me", right clicked, "Checkout Pull Request". It seems like it checked out the PR but didn't show the UI for the current PR. I see a bunch of errors in the log. Not sure why I got the 403s. I reloaded the window, and then it worked as expected.
The log shows checking out the PR and then reloading the window.
ghpr2.log

This looks like GitHub had a temporary issue. I think you probably already have the default log level set to debug for GitHub Pull Request now, but if you don't, would you mind setting it? I've added more debug logging to confirm that we're asking GitHub for the correct PR.


bazel build libtensorflowlite has issue during compiling

tizen5.0_cross_toolchain_vd_kant_target_armv7l_host_x86-64/bin/armv7l-tizen-linux-gnueabi-gcc -MD -MF bazel-out/armv7-opt/bin/external/mkl_dnn/_objs/mkldnn_single_threaded/simple_concat.d '-frandom-seed=bazel-out/armv7-opt/bin/external/mkl_dnn/_objs/mkldnn_single_threaded/simple_concat.o' -iquote external/mkl_dnn -iquote bazel-out/armv7-opt/bin/external/mkl_dnn -isystem external/mkl_dnn/include -isystem bazel-out/armv7-opt/bin/external/mkl_dnn/include -isystem external/mkl_dnn/src -isystem bazel-out/armv7-opt/bin/external/mkl_dnn/src -isystem external/mkl_dnn/src/common -isystem bazel-out/armv7-opt/bin/external/mkl_dnn/src/common -isystem external/mkl_dnn/src/cpu -isystem bazel-out/armv7-opt/bin/external/mkl_dnn/src/cpu -isystem external/mkl_dnn/src/cpu/gemm -isystem bazel-out/armv7-opt/bin/external/mkl_dnn/src/cpu/gemm -isystem external/mkl_dnn/src/cpu/xbyak -isystem bazel-out/armv7-opt/bin/external/mkl_dnn/src/cpu/xbyak -w -DAUTOLOAD_DYNAMIC_KERNELS -fPIC '-std=c++14' -fexceptions '-DMKLDNN_THR=MKLDNN_THR_SEQ' -c external/mkl_dnn/src/cpu/simple_concat.cpp -o bazel-out/armv7-opt/bin/external/mkl_dnn/_objs/mkldnn_single_threaded/simple_concat.o)
Execution platform: @local_execution_config_platform//:platform
In file included from external/mkl_dnn/src/common/mkldnn_thread.hpp:20:0,
from external/mkl_dnn/src/cpu/simple_concat.cpp:17:
external/mkl_dnn/src/common/utils.hpp:45:1: error: static assertion failed: Intel(R) MKL-DNN supports 64 bit only
static_assert(sizeof(void*) == 8, "Intel(R) MKL-DNN supports 64 bit only");
^~~~~~~~~~~~~
Target //tensorflow/lite:libtensorflowlite.so failed to build

@anglerbug123
Request you to fill issue template.
Provide the exact sequence of commands / steps that you executed before running into the problem.Thanks!


VRPTW - how to set different Time Window for different vehicle

Hi all,
I have a problem where the same task has different Time Window based on the vehicle is visiting it.
For example, if the task A is visited by the vehicle 1, it has a time window [a,b]; if the same task is visited by the vehicle 2, it has a time window [c,d].
Is this possible?
Thanks!

duplicate the nodes, add the copies in a disjunction. Fix the vehicle var
of each node, and assign different time windows to each node.
Laurent Perron | Operations Research |<EMAIL_ADDRESS>| (33) 1 42 68 53
00
Le mer. 16 oct. 2019 à 11:48, AsiaLootus<EMAIL_ADDRESS>a
écrit :

Hi all,
I have a problem where the same task has different Time Window based on
the vehicle is visiting it.
For example, if the task A is visited by the vehicle 1, it has a time
window [a,b]; if the same task is visited by the vehicle 2, it has a time
window [c,d].
Is this possible?
Thanks!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/1648?email_source=notifications&email_token=ACUPL3NV3QXAB5HSWDZIOG3QO3PPVA5CNFSM4JBJAJ2KYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HSDQCIQ,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUPL3OFJNPBFIZ3KNJIH6DQO3PPVANCNFSM4JBJAJ2A
.


  "message": "Expected [object Object] to be a GraphQL schema."

hi,
I followed the video here https://www.youtube.com/watch?v=DNPVqK_woRQ it is really great. thank you for that.
I only get the following error in the screenshot below:

please help!

This is best posted to stackoverflow since it doesn't directly relate to ReactQL. That tutorial vid is nearly 2.5 years old; it's very likely that the Sequelize API has changed since then.


Web Safari - VBA - User cannot manually input dates in calendar on Mac Safari

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Access staging.new.expensify.com
Sign into a valid account
Tap Profile > Workspace > Connect to bank
Proceed with VBA flow until you reach the "Company information" step
Scroll down to "Incorporation Date" and click on each date input field

Expected Result:
User expects to be able to manually input a date, similar to the behaviour in Google Chrome
Actual Result:
It is impossible for the user to manually input a date, every time the user manually selects an input field they are pushed back to the "Calendar". This forces the user to scroll through years and years of calendars to find the correct date and is extremely time consuming.
Workaround:
Unknown
Platform:

Where is this issue occurring?

Web - Safari

Version Number: v1.2.3-0
Reproducible in staging?: Y
Reproducible in production?: Y
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:
https://user-images.githubusercontent.com/43996225/191365174-07dba8c7-4488-423d-b750-780c3af32002.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause internal team
Slack conversation:
View all open jobs on GitHub

Hey @kavimuru!
The behaviour in Safari and Chrome really is very different! Reproduced the issue on staging, here is a video comparison on how it works on Chrome (left) and Safari (right):
https://user-images.githubusercontent.com/6564265/191507964-45a0ecbe-9e0e-49f5-a1c0-fcb6796136c1.mov
But this is just the default behaviour of the <input type="date"> on Safari! So I don't think it should be viewed as a problem despite the fact the user has to scroll because it starts at the current date so that should minimize scrolling as much as possible.
The user can still type in the date on Safari pressing tab like I do on the video (tbh I think is unlikely to happen) or choose the date from the Datepicker (which will prob happen but I think that's ok).
Since this doesn't block the user and the date can still be selected and typed as I mentioned, and also since this is the default behaviour of each browser for the date field, I'm closing this one!
But please feel free to re-open if you disagree!


Cosmetic problem

Hi,
Here a small cosmetic issue when add a new hub:

the translation files have been updated. You may have to press F5 / Browser refresh to make sure the latest translation files are loaded


Improve documentation on Configuration page

Feature Request / Improvement
Hi team! We're currently adopting Iceberg format but struggling to configure everything. For example, we'd like to change the parquet compression code from its default to snappy.
I can see that this is an option in the configuration:

But how to actually use the configuration options and apply them seems to be missing from the page. Is there a pyiceberg.configure method, are these options that should be passed into .append, should this be set as an environment variable with some form of translation from write.parquet.compression-codec to PYICEBERG__WRITE_PARQUET__COMPRESSION_CODEC? Later on in the page in the Catalogs section, there are some ways detailed as to how to configure catalogs, but I'm not sure if this applies to writer configuration as well (why would it not be at the top if it applied to everything).
I'll open a PR on this under the assumption that the Catalog configuration section is generally applicable, and to also fix the incorrect description in the write.parquet.page-row-limit.

Later on in the page in the Catalogs section, there are some ways detailed as to how to configure catalogs, but I'm not sure if this applies to writer configuration as well (why would it not be at the top if it applied to everything).

Yes, this is indeed the case. Sorry for the confusion there, and thanks for raising a PR. These PRs are very helpful to the community, since the folks that already use PyIceberg know what to look for, and skip parts of the documentation.

@Fokko I've updated the lint in the PR. Also, what are your thoughts on enabling Github discussions? I have a few questions about best practises and similar but I don't want to clutter Issues with things that aren't actual issues.

Enabling Github Discussions has been brought up once or twice, but it hasn't been yet decided. Mainly because there are several places to discuss things, for example, Slack, GitHub Issues or the ASF mailing list. Feel free to raise Github issues, or start a thread on either the mailinglist or Slack (we have a #python channel).

Thanks @Fokko, I've joined the community slack and made a post to the compaction channel about a different topic, looking forward to getting more engaged with the community. Cheers mate.


Short solution needed: "Get curl version" (curl)

Please help us write most modern and shortest code solution for this issue:
Get curl version (technology: curl)
Fast way
Just write the code solution in the comments.
Prefered way

Create pull request with a new code file inside inbox folder.
Don't forget to use comments to make solution explained.
Link to this issue in comments of pull request.

Get curl version


Add shortcuts to select different tabs

VSCode Version: 1.3.0
OS Version: OS X 10.11.5

This is a feature request to default CMD + 1 to go to the first tab, CMD + 2 to go to the second tab and so on...
The current behaviour seems unusual as it splits the editor into 2 or 3 sections. Also, similar shortcuts from CMD + 4 to CMD + 9 are not utilised.

We decided to not change this (popular) keybinding. You can modify your keybindings to get this behaviour though, e.g.
{ "key": "cmd+1",                "command": "workbench.action.openEditorAtIndex1" }


Minor changes due to SE-0130: Replace String.init(repeating: Character, count: Int) by String.init(repeating: Sttring, count: Int)

Related to apple/swift#3758

This one needs to merged together with apple/swift#3758, because it uses new APIs introduced there. I'm testing on my Linux machine currently.


[1.x] update news.md and readme.md for 1.8.0 release

Description
update news.md and readme.md for 1.8.0 release (#19975 on master)

@samskalicky thanks


fix(#139): do not check for host endianness in IP conversion

Description
Function determineEndian() always returns the same value since the host byte order is always little-endian, so it can be skipped.
What's more, the byte order of IP address values passed by eBPF programs doesn't need to be changed, because Golang's net.IP type expects big-endian byte arrays.
Details:
We always enter this if-branch binary.LittleEndian.PutUint32(ip, nn) which looks like it performs conversion TO little-endian, while it actually expects to receive input already IN little-endian.
Note, the input number's bytes are kept in the same order - this function merely converts a uint32 to a byte array. The result is we convert a big-endian number to a big-endian byte array.
func (littleEndian) PutUint32(b []byte, v uint32) {
	_ = b[3] // early bounds check to guarantee safety of writes below
	b[0] = byte(v)
	b[1] = byte(v >> 8)
	b[2] = byte(v >> 16)
	b[3] = byte(v >> 24)
}

Crucially, net.IP expects big-endian byte arrays. For example, IP <IP_ADDRESS> (0x0F0E0000 in host order, and 0x00000E0F in network order) should be passed in as [0F, 0E, 00, 00] (most significant byte at the lowest memory address) and so doesn't need inverting. Example here: https://play.golang.com/p/ykiP6mLB_gG
In summary, we currently do not perform byte order conversion and shouldn't start to, either in GO or eBPF code.
Related Issue
Fixes #139
This PR looks almost identical to https://github.com/microsoft/retina/pull/161 which was rejected because it didn't include byte order conversion in eBPF, but turns out that's not needed. It also uses the correct uint-to-array conversion functions.
Checklist

[x] I have read the contributing documentation.
[x] I signed and signed-off the commits (git commit -S -s ...). See this documentation on signing commits.
[x] I have correctly attributed the author(s) of the code.
[x] I have tested the changes locally.
[x] I have followed the project's style guidelines.
[x] I have updated the documentation, if necessary.
[x] I have added tests, if applicable.

Relates to #95


Center in a ViewController

I'm using it to load data in a UISplitViewController, is there a way to center it in the detail view controller instead of the entire window (iPad) ?
Thank you !

Not yet! but that's a great idea! I will definitely add that in a future release :)

I will be updating the control in the coming weeks. This feature would be included!

Thank you :+1:


[Suggestion] Replace dependency on installed Microsoft Excel

With the NPOI library you can read/write Office documents easily without any dependency on an installed Office-Product. That might be a good replacement for the current dependency on an installed Microsoft Excel on the machine. Already using it in one of our projects successfully, the implementation is really straight-forward.

Thank you for this suggestion. That dependency had been bugging us as well (for multiple reasons). I will be working on transitioning to NPOI as soon as I can.

I'm currently working on the implementation and will have a pull-request ready in the next few weeks. :-)


Clean up error logs

At the moment, error logs look as follows:
dune-release: [ERROR] The following command exited with code 128:
                      git --git-dir .git tag -a -m Distribution v0.2.0+test
                      v0.2.0+test HEAD

We should

get rid of "dune-release:" at the beginning: when a user runs a dune-release-command which fails, starting the error log with "dune-release" doesn't contribute any useful information;
not indent any line.

That way, the command that's failed might not get split into two lines.

I've split up #225 into three parts. This issue is one of them.


Snippet from README prints nothing

temp.py -> https://raw.githubusercontent.com/pygments/pygments.rb/master/vendor/pygments-main/pygments/lexers/_mapping.py
$ irb -rpython3-parser
irb(main):001:0> class MyFuncVisitor < Python3Parser::Visitor
  def visit_funcdef(ctx)
    puts ctx.NAME.text
    visit_children(ctx)
  end
end
=> :visit_funcdef
irb(main):007:0> parser = Python3Parser::Parser.parse(File.read("temp.py"))
=> #<Python3Parser::Parser:0x00007fa2c081e588>
irb(main):008:0> parser.visit(MyFuncVisitor.new)
=> nil
irb(main):009:0>

(ruby 2.4.9, macOS)

Hmm I don't see any function definitions in that pygments file, that's why the code doesn't print anything.

Oh, is it by design?

Not sure what you mean? The visitor from the README prints out the names of any method definitions (i.e. defs) it finds. The file you provided doesn't contain any method definitions, so nothing gets printed.

There are many, many other visit_* methods you could implement for all the other syntax features in the Python language, like identifiers, function calls, etc. Take a look at the visitor code for a complete list of the methods you can define.

I see. Can you tell which visitor I should use to obtain the LEXERS from that file? I've already parsed them our using regex but using a proper parser seems to be a nicer solution. Specs use the visit_testlist_comp but that's probably not what I should use, it's for a test.

Ok, so it turns out to be pretty non-trivial to extract the value of the LEXERS variable. Here's what I came up with:
module Utils
  def self.unquote(str)
    str[1..-2]
  end
end

# lexer object (these names might be wrong, not sure what's in these tuples)
Lexer = Struct.new(:mod, :name, :abbrevs, :globs, :mimes)

# root of the visitor hierarchy
class LexersVisitor < Python3Parser::Visitor
  attr_reader :lexers

  def visit_expr_stmt(ctx)
    # we only care about what's assigned to the LEXERS local var
    lhs, op, rhs = ctx.children
    return unless lhs && op && rhs
    return unless lhs.text == 'LEXERS' && op.text == '='

    @lexers = LexerVisitor.find_lexers(rhs)
  end
end

# extract each entry from the dict assigned to LEXERS
class LexerVisitor < Python3Parser::Visitor
  def self.find_lexers(ctx)
    visitor = new
    visitor.visit(ctx)
    visitor.lexers
  end

  attr_reader :lexers

  def initialize
    super
    @lexers = {}
  end

  # this is the visit method for dicts
  def visit_dictorsetmaker(ctx)
    each_lexer_pair(ctx) do |lang, metadata_test|
      mod, name, abbrevs, globs, mimes = LexerMetaVisitor.find_metadata(metadata_test)
      @lexers[lang] = Lexer.new(
        mod, name, abbrevs, globs, mimes
      )
    end
  end

  private

  # iterates over each key/value pair in the LEXERS dict
  def each_lexer_pair(dict_ctx)
    each_test(dict_ctx).each_slice(2) do |key_test, value_test|
      yield [Utils.unquote(key_test.text), value_test]
    end
  end

  # Iterates over each of the "test" entries in the LEXERS dict and
  # yields them as key/value pairs. Unfortunately antlr-gemerator
  # creates a new array every time you call #test (which is also
  # horribly slow), so instead of calling .test.each, we take
  # advantage of #test_at(idx). Unfortunately there's also no way
  # to get the number of tests, so instead this endless range is
  # necessary. Ugh.
  #
  # Oh, and I have no idea why the python3 grammar calls this
  # particular thing a "test." Doesn't make sense to me.
  def each_test(ctx)
    return to_enum(__method__, ctx) unless block_given?

    (0..).each do |i|
      if test = ctx.test_at(i)
        yield test
      else
        break
      end
    end
  end
end

# interprets all the info in the metadata tuples
class LexerMetaVisitor < Python3Parser::Visitor
  def self.find_metadata(ctx)
    visitor = new
    visitor.visit(ctx)
    visitor.stack.pop[0]
  end

  attr_reader :stack

  def initialize
    super
    @stack = [[]]
  end

  def visit_atom(ctx)
    ctx.children.each do |child|
      # the tuples contain strings, which are expressed in the ANTLR world
      # as "terminal" nodes
      if child.is_a?(Python3Parser::TerminalNodeImpl)
        handle_terminal(child)
      else
        super
      end
    end
  end

  private

  # handles nested tuple structures, probably overkill for what we need here,
  # but kinda fun to write so what the hell
  def handle_terminal(ctx)
    case ctx.text
      when '('
        [].tap do |arr|
          @stack.last << arr
          @stack.push(arr)
        end
      when ')'
        @stack.pop
      else
        @stack.last << Utils.unquote(ctx.text)
    end
  end
end

Here's how you would use it:
parser = Python3Parser::Parser.parse(File.read('_mapping.py'))
visitor = LexersVisitor.new
parser.visit(visitor)
visitor.lexers  # => returns a hash containing Lexer objects keyed by language

Unfortunately the grammar I used to generate this parser is pretty annoying, so you have to jump through hoops to get it to do what you want. Hopefully this helps and can serve as a bit of documentation for anyone else looking for a good example.


variable.yaml enhancements

This PR tracks a list of improvements to the variables.yaml file.

[ ] Copy variable.yaml to duqtools workspace
[x] Log location of variables.yaml used -> this is already in runs.yaml
[ ] Add option to specify location of variables.yaml in duqtools.yaml, i.e. variables_file: Optional[Path]
[ ] Add option to specify extra variables in duqtools.yaml, because it is somewhat cumbersome to have to copy the variables.yaml, modify it and then use it.
[ ] Get list of sensible defaults from @aaronkho
[ ] Expose list of default variable names on the documentation
[ ] duqtools list -> New cli command to list variables

Originally posted by @v1kko in https://github.com/CarbonCollective/fusion-dUQtools/issues/320#issuecomment-1263226840:
    Looks good, this should reduce config file bloat a lot

I like the modularity, we can always add an option to load the default config file in another way later:
variables_file: Optional[Path] as an extra option for example

I have one question/suggestion:

What happens if I want to only add 1 extra variable? It looks like I'd have to copy the default, add my stuff, and store it somewhere where duqtools will pick it up, and in that way the defaults would be replaced forever, even if they are updated in a later version.

Maybe it is good to still have  way to define 'extra' variables, which do not replace the default files, this could be done with an extra extra_variables: Optional[Path] in the config file, which is then appended/overwritten to the defaults

Here is a list of variables that would probably be useful to look expose to the user at all times, as they are very useful to any type of validation. The commented variables cause duqtools to error out, typically from a KeyError though. Has the fix discussed last week (provide a null value for missing fields) been implemented?
variables.txt

Hi @aaronkho , thanks for the really nice list! I think this will make a big step in the usability of the code. I will add it in #346.
The fix for actual missing fields is something we still need to think about how to implement. I will create a new issue to make sure we don't forget.


Update customize-branding.md

Adding an important note: "Hyperlinks added with Sign-in page text are rendered as a text in the native environments e.g. desktop and mobile applications."

@AlexanderMars : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


plz help i have this error when i put the game on my computer.What i need to do??

java.lang.RuntimeException: Can not initialize graphics device.
at org.terasology.engine.subsystem.lwjgl.LwjglGraphics.initDisplay(LwjglGraphics.java:182)
at org.terasology.engine.subsystem.lwjgl.LwjglGraphics.postInitialise(LwjglGraphics.java:100)
at org.terasology.engine.TerasologyEngine.postInitSubsystems(TerasologyEngine.java:255)
at org.terasology.engine.TerasologyEngine.(TerasologyEngine.java:160)
at org.terasology.engine.Terasology.main(Terasology.java:100)
Caused by: org.lwjgl.LWJGLException: Pixel format not accelerated
at org.lwjgl.opengl.WindowsPeerInfo.nChoosePixelFormat(Native Method)
at org.lwjgl.opengl.WindowsPeerInfo.choosePixelFormat(WindowsPeerInfo.java:52)
at org.lwjgl.opengl.WindowsDisplay.createWindow(WindowsDisplay.java:252)
at org.lwjgl.opengl.Display.createWindow(Display.java:306)
at org.lwjgl.opengl.Display.create(Display.java:848)
at org.lwjgl.opengl.Display.create(Display.java:757)
at org.terasology.engine.subsystem.lwjgl.LwjglGraphics.initDisplay(LwjglGraphics.java:177)
... 4 more

Hi there - unfortunately it sounds like your video card is not fully working or new enough to support Terasology :-(
What computer do you have and do you know if you have the latest graphics drivers for your system?

Cervator, can we be sure that what we write here will reach back to the user who reported the issue?
Good to know there is this other failure mode. In this case it's much harder to get information about the hardware/software configuration, but maybe there is a way.

@terasologyfan should be getting email notifications from GitHub for updates to this issue. Whether he or she gets and notices the email I can't say, and we don't have any other options so I hope so :-)
I was wondering exactly what build this is from (the game log file would be great) as it is hard to know if your newer logging is actually in place. Unless you can tell from that snippet anyway.

No, unfortunately this failure is earlier than the new logging: by my understanding in this case the opengl context failed to be created. Even in the latest build the glGetString() commands wouldn't have been able to execute.
By the way. At least TerasologyEngine has a logEnvironment() method that provides some limited information that could be useful in these circumstances. But it'd be good to have something like oshi or similar, to obtain as much hardware/software specific information as possible.

Also, how come we don't get the full log?

Most likely because the CrashReporter caught it and by default only displays the final stacktrace, often users just copy paste that here rather than realize there's a "full log" tab available too :-)
I put in an issue to change the default some time ago: https://github.com/MovingBlocks/CrashReporter/issues/12 - it is probably fairly straight forward to change sometime

Do they have to create a github account to create an issue? Does the account by default forward this conversations to them via email or is this something the user needs to configure? I'm asking because paulanne failed to respond in issue #1548 and I wonder if he saw our conversation at all.

Yep, have to create an account to create an issue, so they would've had to put in an email. I'm not sure if you have to confirm said email though, so if somebody made an account with a gibberish email we'd be relying on them to remember visiting the issue again later.
Unfortunately some users find the game, run it, crash, report something, then never seem to return. Many more probably stall even earlier in the process.

I think we have to have faith that if a user goes to the trouble or reporting an issue then they will check and follow up on it. There's not much else we can do.
Although I would prefer people took this sort of issue to the forums in the first instance. Github issues are intended for bugs to be fixed, not tech support style issues (at least until it becomes apparent there is something deeper to be fixed).

I agree with that. I'm going to add an option to CrashReporter that directs users to the support forum and make it the default.
Maybe it should be more like a wizard though:

Upload crash log
Describe it in a forum post (register first)
Check back later

This would also help with the issue that people just c&p their stacktrace instead of the full log.

I have a Samsung computer and my graphic cards are up to date if you can
fix this error i will be greatfull minecraft and teraology are my life plz
if you van fix this send me an email
Em 28/01/2015 20:22, "Rasmus Praestholm"<EMAIL_ADDRESS>escreveu:

Hi there - unfortunately it sounds like your video card is not fully
working or new enough to support Terasology :-(
What computer do you have and do you know if you have the latest graphics
drivers for your system?
—
Reply to this email directly or view it on GitHub
https://github.com/MovingBlocks/Terasology/issues/1563#issuecomment-71908456
.

Hi again @terasologyfan - we need more than "Samsung" like a model name or number :-)
Also would help with the game log, probably at something like C:\Users[username]\Saved Games\Terasology\log if you're on Windows
@immortius / @msteiger - agreed on forum etc. Would be good to highlight easier defaults, https://github.com/MovingBlocks/CrashReporter/issues/12 has been floating around a while for that :-)

Hi again too im AM just a boy so i dont know what i need to do my computer
is from Portugal its from the samsung enterprize what do i need to do????
Em 29/01/2015 15:05, "Rasmus Praestholm"<EMAIL_ADDRESS>escreveu:

Hi again @terasologyfan https://github.com/terasologyfan - we need more
than "Samsung" like a model name or number :-)
Also would help with the game log, probably at something like
C:\Users[username]\Saved Games\Terasology\log if you're on Windows
@immortius https://github.com/immortius / @msteiger
https://github.com/msteiger - agreed on forum etc. Would be good to
highlight easier defaults, MovingBlocks/CrashReporter#12
https://github.com/MovingBlocks/CrashReporter/issues/12 has been
floating around a while for that :-)
—
Reply to this email directly or view it on GitHub
https://github.com/MovingBlocks/Terasology/issues/1563#issuecomment-72039289
.

Look at the case of the computer, do you see a name or some numbers anywhere? Is it a laptop or a desktop PC? Is it old?

Is it the model code.or just the model???
Em 29/01/2015 15:13, "Rasmus Praestholm"<EMAIL_ADDRESS>escreveu:

Look at the case of the computer, do you see a name or some numbers
anywhere? Is it a laptop or a desktop PC? Is it old?
—
Reply to this email directly or view it on GitHub
https://github.com/MovingBlocks/Terasology/issues/1563#issuecomment-72040627
.

As many names and numbers as you can find please :-)
Is it a laptop? How long have you had it?

I had this laptop like one year ago the model is:NP-N145 and the modelo
code is:NP-N145-JP03PT
Em 29/01/2015 16:21, "Rasmus Praestholm"<EMAIL_ADDRESS>escreveu:

As many names and numbers as you can find please :-)
Is it a laptop? How long have you had it?
—
Reply to this email directly or view it on GitHub
https://github.com/MovingBlocks/Terasology/issues/1563#issuecomment-72053730
.

I will put new graphic Cards on my laptop ok
Em 29/01/2015 16:21, "Rasmus Praestholm"<EMAIL_ADDRESS>escreveu:

As many names and numbers as you can find please :-)
Is it a laptop? How long have you had it?
—
Reply to this email directly or view it on GitHub
https://github.com/MovingBlocks/Terasology/issues/1563#issuecomment-72053730
.

Thanks for the model info, that helped - looks like this? http://www.tecnikk.co.uk/index.php/laptops/samsung/np-n145-jp03-black.html
But yeah, unfortunately that is likely too weak to play Terasology :-(
Can you play on another computer? You typically cannot change the video card in a laptop, especially not on netbooks.
Sorry about that. It might be that we'll make Terasology compatible with weaker/older systems one day, but I can't say when, if ever.

Thx for all im sorry for you because i cant play Terasology.
Em 29/01/2015 16:37, "Rasmus Praestholm"<EMAIL_ADDRESS>escreveu:

Thanks for the model info, that helped - looks like this?
http://www.tecnikk.co.uk/index.php/laptops/samsung/np-n145-jp03-black.html
But yeah, unfortunately that is likely too weak to play Terasology :-(
Can you play on another computer? You typically cannot change the video
card in a laptop, especially not on netbooks.
Sorry about that. It might be that we'll make Terasology compatible with
weaker/older systems one day, but I can't say when, if ever.
—
Reply to this email directly or view it on GitHub
https://github.com/MovingBlocks/Terasology/issues/1563#issuecomment-72056968
.

One day in the future you'll be able to! Hopefully we'll see you back then. Good luck and have fun :-)

I found the specs for his GPU:
http://www.notebookcheck.net/Intel-Graphics-Media-Accelerator-3150.23264.0.html
Interestingly, it supports OpenGL 1.5. So, now I'm wondering if the context
creation process attempts to generate a 2.0 context and why.

LOL =D

OK i will ser thx
Em 29/01/2015 19:43, "manu3d"<EMAIL_ADDRESS>escreveu:

I found the specs for his GPU:
http://www.notebookcheck.net/Intel-Graphics-Media-Accelerator-3150.23264.0.html
Interestingly, it supports OpenGL 1.5. So, now I'm wondering if the context
creation process attempts to generate a 2.0 context and why.
—
Reply to this email directly or view it on GitHub
https://github.com/MovingBlocks/Terasology/issues/1563#issuecomment-72090534
.

I will see if it works i mail you again.
Em 29/01/2015 19:43, "manu3d"<EMAIL_ADDRESS>escreveu:

I found the specs for his GPU:
http://www.notebookcheck.net/Intel-Graphics-Media-Accelerator-3150.23264.0.html
Interestingly, it supports OpenGL 1.5. So, now I'm wondering if the context
creation process attempts to generate a 2.0 context and why.
—
Reply to this email directly or view it on GitHub
https://github.com/MovingBlocks/Terasology/issues/1563#issuecomment-72090534
.


Fixes for missing Burp GraphQL response data

This fixes some issues I was having where the GraphQL query returned incomplete data causing the following errors:
[pid: 28|app: -|req: -/-] <IP_ADDRESS> (-) {44 vars in 733 bytes} [Mon Jun  6 00:23:12 2022] POST /api/v2/import-scan/ => generated 59 bytes in 75 msecs (HTTP/1.1 500) 7 headers in 212 bytes (1 switches on core 1)
[06/Jun/2022 00:23:12] ERROR [dojo.api_v2.exception_handler:32] 'NoneType' object is not iterable
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/rest_framework/views.py", line 506, in dispatch
    response = handler(request, *args, **kwargs)
  File "/usr/local/lib/python3.10/site-packages/rest_framework/mixins.py", line 19, in create
    self.perform_create(serializer)
  File "/app/./dojo/api_v2/views.py", line 1971, in perform_create
    serializer.save(push_to_jira=push_to_jira)
  File "/app/./dojo/api_v2/serializers.py", line 1452, in save
    test, finding_count, closed_finding_count, test_import = importer.import_scan(scan, scan_type, engagement, lead, environment,
  File "/app/./dojo/importers/importer/importer.py", line 297, in import_scan
    parsed_findings = parser.get_findings(scan, test)
  File "/app/./dojo/tools/burp_graphql/parser.py", line 30, in get_findings
    return self.create_findings(data.get('Issues'), test)
  File "/app/./dojo/tools/burp_graphql/parser.py", line 34, in create_findings
    finding_data = self.parse_findings(scan_data)
  File "/app/./dojo/tools/burp_graphql/parser.py", line 74, in parse_findings
    finding = self.create_finding(issue)
  File "/app/./dojo/tools/burp_graphql/parser.py", line 131, in create_finding
    finding['Evidence'] = self.parse_evidence(issue.get('evidence'))
  File "/app/./dojo/tools/burp_graphql/parser.py", line 159, in parse_evidence
    for data in request_dict.get('request_segments'):
TypeError: 'NoneType' object is not iterable
[06/Jun/2022 00:23:12] ERROR [django.request:224] Internal Server Error: /api/v2/import-scan/

And
django-defectdojo-uwsgi-1         | [06/Jun/2022 13:34:55] ERROR [dojo.engagement.views:637] 'NoneType' object has no attribute 'replace'
django-defectdojo-uwsgi-1         | Traceback (most recent call last):
django-defectdojo-uwsgi-1         |   File "/app/./dojo/engagement/views.py", line 622, in import_scan_results
django-defectdojo-uwsgi-1         |     test, finding_count, closed_finding_count, _ = importer.import_scan(scan, scan_type, engagement, user, environment, active=active, verified=verified, tags=tags,
django-defectdojo-uwsgi-1         |   File "/app/./dojo/importers/importer/importer.py", line 297, in import_scan
django-defectdojo-uwsgi-1         |     parsed_findings = parser.get_findings(scan, test)
django-defectdojo-uwsgi-1         |   File "/app/./dojo/tools/burp_graphql/parser.py", line 30, in get_findings
django-defectdojo-uwsgi-1         |     return self.create_findings(data.get('Issues'), test)
django-defectdojo-uwsgi-1         |   File "/app/./dojo/tools/burp_graphql/parser.py", line 34, in create_findings
django-defectdojo-uwsgi-1         |     finding_data = self.parse_findings(scan_data)
django-defectdojo-uwsgi-1         |   File "/app/./dojo/tools/burp_graphql/parser.py", line 72, in parse_findings
django-defectdojo-uwsgi-1         |     self.combine_findings(issue_dict.get(issue_name), issue)
django-defectdojo-uwsgi-1         |   File "/app/./dojo/tools/burp_graphql/parser.py", line 82, in combine_findings
django-defectdojo-uwsgi-1         |     description = html2text.html2text(issue.get('description_html'))
django-defectdojo-uwsgi-1         |   File "/usr/local/lib/python3.8/site-packages/html2text/__init__.py", line 947, in html2text
django-defectdojo-uwsgi-1         |     return h.handle(html)
django-defectdojo-uwsgi-1         |   File "/usr/local/lib/python3.8/site-packages/html2text/__init__.py", line 142, in handle
django-defectdojo-uwsgi-1         |     self.feed(data)
django-defectdojo-uwsgi-1         |   File "/usr/local/lib/python3.8/site-packages/html2text/__init__.py", line 138, in feed
django-defectdojo-uwsgi-1         |     data = data.replace("</' + 'script>", "</ignore>")
django-defectdojo-uwsgi-1         | AttributeError: 'NoneType' object has no attribute 'replace'

Can you add also some unittest? Mainly add the template where mentioned fields are missing.

I've never actually written a unit test before, I can give it a go, here is a sample of an response that would break the old parser:
{
  "Issues":[
    {
      "issue_type": {
        "name": "TLS certificate",
        "description_html": null,
        "remediation_html": null,
        "vulnerability_classifications_html": "
\n
CWE-295: Improper Certificate Validation
\n
CWE-326: Inadequate Encryption Strength
\n
CWE-327: Use of a Broken or Risky Cryptographic Algorithm
\n
",
        "references_html": "

SSL/TLS Configuration Guide

"
      },
      "description_html": null,
      "remediation_html": null,
      "severity": "medium",
      "path": "/",
      "origin": "https://verysecretcompany.domain",
      "serial_number": "0000000000000000000000",
      "evidence": [
        {
          "request_index": 0,
          "request_segments": null
        }
      ]
    }
  ]
}


Command-line arguments with argv

In the sys module there is a list called argv which gives access to command-line arguments. Actually sys.argv[0] gives you the name of the script. The command-line arguments start from sys.argv[1]. If you want to know the number of elements in argv you can use.
import sys
len(sys.argv)

If you want to know the number of command-line arguments especially you rather use:
len(sys.argv[1:])

This is called slicing, sys.argv[1:] will start from index 1. You can use this to iterate through the arguments with
for a in sys.argv[1:]:

Write a program, called args.py that prints the name of the script, the number of command-line arguments the program received and list the arguments as well. Here is an example of the output you should get:
$ args.py a b c "hello there" 1 2 3
The script name is ./args.py
Got 7 command-line arguments
Here they are:
  1. a
  2. b
  3. c
  4. "hello there"
  5. 1
  6. 2
  7. 3

having difficulty with this issue. Kindly help out please.

have you written any code for this? if so, so a commit but without adding closes #16. Instead type ref #16
I'm not sure where you are struggling. Maybe if I see some code I'll know better.


NTR: Testing new term request

Here is a place for new definition.
The term should go under.....

Great! you succeeded creating an issue for term creation request  in AgrO


Updated README.md

Step by step guide added for beginners out there.

Fixes Issue

Changes proposed

Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] My change requires changes to the documentation.
[x] I have updated the documentation accordingly.
[ ] All new and existing tests passed.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

Hey, I have updated the README.md file . That will help the beginners out there. it is a step by step guide.

Okay I will raise a new PR for this, Thanks for the advice.


⚠ chatgpt.chenshaowen.com has degraded performance

In a69ea08, chatgpt.chenshaowen.com (https://chatgpt.chenshaowen.com) experienced degraded performance:

HTTP code: 200
Response time: 13757 ms

Resolved: chatgpt.chenshaowen.com performance has improved in 2d24430 after 12 minutes.


Change or disable plain link renderer

When writing a link inline without markdown formatting, such as http://www.google.com, it will automatically become a link. Can I change this behavior (I only want it rendered when prepended by ^ or \s), or alternatively, disable this feature?
I've tried overriding autolink in the renderer but it doesn't seem to change anything.

@chjj I would be willing to help with this issue, if you give me a pointer on how to accomplish it.

#984


Add mutate existing docs for 1.7.0

Signed-off-by: ShutingZhao<EMAIL_ADDRESS>Related issue
Closes https://github.com/kyverno/website/issues/520.

Proposed Changes

Checklist

[x] I have read the contributing guidelines.
[x] I have inspected the website preview for accuracy.
[x] I have signed off my issue.

Thanks for the review @chipzoller ! I have addressed your comments.

I have a few more comments to make. Sorry, will have to come back to this later today.

I have a few more comments to make. Sorry, will have to come back to this later today.

Sure!

Thanks very much for helping with the docs! See in-line comments.

Thanks, addressed all comments!

One comment appears unresolved.

Changed in the PR, forgot to resolve it :(


fix create gateway

internal gateways require an internal domain
🐰🐰🐰

Would it be possible to make internal_domain match the camelCase spec of the other properties, then i can merge this :)

not really a fan of how I did this, open to suggestions 🐰

this is great, thanks!


Update README.md

As a Developer
I need display the detailed information about Promo service
So that everyone will know how to use the Promo service
Details and Assumptions
README.md should include the following information:

The dev link and prod link
list of  query route
The setup instruction

Acceptance Criteria
Given [some context]
When [action taken]
Then [outcome observed]

[Uploading... Screenshot 2023-05-03 at 5.02.22 PM.png]


No command specified error on startup

Working through the first app 1, 2, 3 tutorial and pod is failing to launch.  Not sure if this is a known issue or just user error on my part ( probably the latter :P ).
krusty                                                            0/1     CreateContainerError   0          14m
Event stream shows.
Error: failed to generate container "<omitted>" spec: no command specified, (via Pod krusty)

I just tried walking through the tutorial and could not reproduce this issue with the latest canary release. I'm going to close this for now assuming that it's been resolved in more recent releases. Let us know if you run into this again. Thanks for the report!


Having trouble with authorization

Hi i have the basic licence and i am trying to get access from cortex in python with these parameters:
{     "jsonrpc": "2.0",     "method": "authorize",     "params": {         "clientId": "N0QrgbkfPrO7CpwqC74dBpRyVqqSZNd3bPOvrV0v",         "clientSecret": "**********",         "license": "",         "debit": 10     },     "id": 4 }
However i got a message:
{'code': -32002, 'message': 'Invalid License Key.'}
I CHECKED EEG ACCESS during setup of app.
My emotivID is: andres.nevarez and my clientID is: N0QrgbkfPrO7CpwqC74dBpRyVqqSZNd3bPOvrV0v
Can you help me please? thank you

Had the wrong license

I already have PRO-LITE however i can't access to eeg data or facial on API. Thanks

Hi @andresnevarez ,
With PRO-Lite license you can not access EEG data via API . For developer, you can reference license plan here https://www.emotiv.com/developer/
Because your free license can not access to EEG data so when you CHECKED EEG ACCESS during setup of app -> got the above error. However with free license , you can access to BCI API but you need to create new one without "CHECKED EEG ACCESS"
Please try and let us know the result. Thank you


How can I serve different http uri with one service when using generic call in http mode ?

I'm developing a gateway service, and my backend service is based on kitex. Generic call seems quite easy for this. But I have some confusions. Here's some details :
The gateway have two different uris

/gateway/api/resource/v1
/gateway/api/resource/v2

And both of them will call the same service, like

fooResp FooService(1: fooRequest req)

But I can't find any guidance for this purpose, seems that generic call can only support one api address for one service.
   BizResponse BizMethod2(1: BizRequest req)(api.post = '/life/client/:action/:biz', api.baseurl = 'ib.snssdk.com', api.param = 'true', api.serializer = 'form')
How can I serve different http uri with one service when using generic call in http mode ?

Hi @Liqifan, sorry for missing the message. This is IDL Definition Specification for Mapping between Thrift and HTTP


Failed to compile when Installing go1.4

when I tried to install go1.4 using gvm install go1.4
terminal returned failed to compile and failed to use installed version
and this is the error
# cmd/pprof
/home/ryuunotaro/.gvm/gos/go1.4/pkg/linux_amd64/runtime/cgo.a(_all.o): unknown relocation type 42; compiled without -fpic?
/home/ryuunotaro/.gvm/gos/go1.4/pkg/linux_amd64/runtime/cgo.a(_all.o): unknown relocation type 42; compiled without -fpic?
runtime/cgo(.text): unexpected relocation type 298
runtime/cgo(.text): unexpected relocation type 298
# cmd/go
/home/ryuunotaro/.gvm/gos/go1.4/pkg/linux_amd64/runtime/cgo.a(_all.o): unknown relocation type 42; compiled without -fpic?
/home/ryuunotaro/.gvm/gos/go1.4/pkg/linux_amd64/runtime/cgo.a(_all.o): unknown relocation type 42; compiled without -fpic?
runtime/cgo(.text): unexpected relocation type 298
runtime/cgo(.text): unexpected relocation type 298

same question #210


Users have to create a fake request object to use piwikapi outside of Django

This is something that will be fixed as soon as I have some time to do it (or somebody sends a pull request). The docs and source have an example of a fake request class, it's quite trivial to build, but should still be avoided.

The new refactor branch removes the Django dependency. Still need to update tests and docs.


ff2mpv-go: init at 1.0.1

Description of changes
I had this packaged in an overlay for much too long, finally adding it upstream.
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[X] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2902

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/3056


The dockerhub_docs script isn't producing the correct output after this PR (it still only lists amd64

The dockerhub_docs script isn't producing the correct output after this PR (it still only lists amd64
Originally posted by @karianna in https://github.com/adoptium/containers/issues/47#issuecomment-912073014

This may not actually be a bug, but saving this here so I can investigate with more coffee later

That's because amd64 is the only available JRE on JDK11 right?

probably :-)


add David to example users

I don't think we should be adding more users to the "examples" here ... if anything we should be removing them! ... happy to merge this for now, but these will be removed eventually, since this should not contain real users


Fill the holes in the proof of the relaying–broadcasting equivalence theorem

This resolves #55.

Could you please also merge the latest master commits into this branch? In particular, I’m interested in having 86c90ce in the branch, so that the build process doesn’t regenerate data under the session name Utilities (I’ve deleted all such data on my computer and would like my ~/.isabelle directory to stay clean 😉).

Could you please also merge the latest master commits into this branch? In particular, I’m interested in having 86c90ce in the branch, so that the build process doesn’t regenerate data under the session name Utilities (I’ve deleted all such data on my computer and would like my ~/.isabelle directory to stay clean ).

Done.

Could you also please adapt the code to use the features brought by #158?

I discovered that we actually don’t need the inner forward/backward bridge redundancy lemmas. Our use of them apparently stems from a time when we didn’t have inner_bidirectional_bridge_redundancy yet. Now we have it, and the code is more high-level when using it (no unfolding of bidirectional bridges).
I think we should retire the inner forward/backward redundancy lemmas, as they aren’t needed anymore and they are rather specialized. What do you think, @javierdiaz72?

I discovered that we actually don’t need the inner forward/backward bridge redundancy lemmas. Our use of them apparently stems from a time when we didn’t have inner_bidirectional_bridge_redundancy yet. Now we have it, and the code is more high-level when using it (no unfolding of bidirectional bridges).
I think we should retire the inner forward/backward redundancy lemmas, as they aren’t needed anymore and they are rather specialized. What do you think, @javierdiaz72?

I agree that they are rather specialized, so we should retire them.

I think we should retire the inner forward/backward redundancy lemmas, as they aren’t needed anymore and they are rather specialized. What do you think, @javierdiaz72?

I agree that they are rather specialized, so we should retire them.

I’ve filed an issue for that (#163). Could you please work on that as well? Thanks. 🙂

Could you also please adapt the code to use the features brought by #158?

Done.

Hi, @javierdiaz72! I’ve performed an overhaul of the Relaying_Broadcasting_Equivalence theory. Could you please look carefully through the new version, checking whether there are any problems with it? Thanks a lot.

Hi, @javierdiaz72! I’ve performed an overhaul of the Relaying_Broadcasting_Equivalence theory. Could you please look carefully through the new version, checking whether there are any problems with it? Thanks a lot.

Beautiful! Approved. ;)


MarkdownView doesn't work

When I run in my network callback,it seems doesn't work.The code like this:
 @Override public void respondSuccess(final Trees data) { // my netwrokCallback
         Thread thread = new Thread(new Runnable() {
     @Override public void run() {
       try {
         // get markdown txt from bootstrap
         Blob blob = mGitDateService.getBlob("twbs", "bootstrap",
             "e0e7c32bdd7d6d91efc36ad9da1a39549c9b0e13").execute().body();
         final String txtMarkdown = new String(Base64.decode(blob.content, Base64.DEFAULT));
         Log.d(TAG, "run: " + txtMarkdown); // log->success
         getActivity().runOnUiThread(new Runnable() {
           @Override public void run() {
             mTxtDescription.setText(txtMarkdown); // I test the code in a TextView, show normal
             mMarkdownView.setMarkDownText(txtMarkdown); // show abnormal,the markdown is blank
             Log.d(TAG, "run: " + "already set markdownTxt"); // log->success
           }
         });
       } catch (IOException e) {
         e.printStackTrace();
       }
     }
   });
   thread.start();
 }

@tajchert was already solved this issue!  see this pull requests #5

@LittleHans8 Fixed and released in version 1.0.2 @tajchert thanks


Make all-through schools eligible

This is following a discussion with the policy team.

 Make "All-through" schools eligibile


Remove user personal details filters data

Closes: #1192
API Link : https://api.realdevsquad.com/users/search?state=IDLE&state=ACTIVE&state=OOO
Description:
The current above API returns personal details like Phone , E-mail and GitHub Access tokens. In this PR we are removing user personal details from the response.
Test Coverage

Screenshot
Before

After

We should create a data access layer, which will always return the user's filtered data

We should create a data access layer, which will always return the user's filtered data/
So that if passing it through that code, will remove sensitive data

Okay so can I create the issue ticket for this ?


.deb install

I installed go-ssb-room_2.0.6_Linux_x86_64.deb
I had a look in  sudo nano /etc/systemd/system/go-ssb-room.service
Which showed. localhost:8899
I had a look at https://github.com/ssb-ngi-pointer/go-ssb-room/blob/master/docs/files/debian-postinstall.sh
So I did
adduser --system --home /var/lib/go-ssb-room go-ssb-room
chown go-ssb-room /var/lib/go-ssb-room
sudo systemctl daemon-reload
sudo systemctl start go-ssb-room
http://<IP_ADDRESS>:8899/ displays 'bad host'
I don't really understand what this line means: "you need to at least change the https domain in the systemd service."
Is it configured to work out of the box on localhost?
cat /etc/os-release
PRETTY_NAME="Debian GNU/Linux 11 (bullseye)" 5.10.0-13-amd64

:wave: @samuk, it is referring to the option provided by the go-ssb-room binary:
➜  server (master) ✔ ./server -h
Usage of ./server:
  ...
  -https-domain string
    	which domain to use for TLS and AllowedHosts checks
  ...

You can see an example of this in the example systemd file:
https://github.com/ssb-ngi-pointer/go-ssb-room/blob/6f5edbaa438fe473cc865e283bb3fd32de231c44/docs/files/example-systemd.service#L15
Where you need to change -https-domain my-example-room.somewhere to your domain.
I think this could be clearer in the output, do you have a suggestion? We could submit a docs PR then.


VM provisioning: add the --vbox-only flag

Description
This PR merges each CrownLabs playbook with the corresponding vanilla one, introducing a --vbox-only flag to the setup-crownlabs-vm script to opt out from the installation of the CrownLabs components during the VM configuration phase (the documentation is update accordingly).
Additionally, it addresses the issues raised by the shellcheck tool.

@giorio94 This looks very very good to me.
Just asking: can we move the requirement checking to the actual action requested to the script? This would allow the script to run on Windows in order to create the OVA. It seems to me that the only requirement that is not easy to satisfy on Windows is to have Docker running; if we move the check of Docker when it is actually needed, we can work most of the time on Windows as well. Thanks!


Internal server error when authorizing user who's not in the dfunkt database

These lines:
https://github.com/datasektionen/pico/blob/master/middlewares.ts#L55-L59
Retrvieves the user's current mandates from dfunkt. If the user does not exist in dfunkt's database (i.e. if the user has neither had a mandate in the chapter nor logged in on the website) dfunkt returns a 404. This is not handled and we just return a 500 without any logs.
dfunkt should probably return an empty array of mandates for unknown users, but pico should arguably not rely on dfunkt to work to allow users to create links.

I think the pico relying on dfunkt is in order for the "mandate-owned" links feature to work, but yeah, it should obviously be able to handle an error-response


refactorBE:Initialize pointer variables in the class to nullptr by default

…r by default
Proposed changes

In the source code of the Doris be module, the default initialization of pointer member variables in the class is not uniform, and there is no default initialization, {nullptr},=nullptr. So in this large-scale unification, use=nullptr to set it.

Use = nullptr

T *
const T *
T **
const T **
shared_ptr
unique_ptr

Don't set default initialization

T * const

Set {nullptr} to = nullptr
Unchanged parts

extern
null/NULL
Customized pointers through using, typedef, and other methods

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...

Unchanged parts

extern
null/NULL
for traditional usages like NULL, could you change them to nullptr by the way?

Unchanged parts

extern
null/NULL

for traditional usages like NULL, could you change them to nullptr by the way?

of course, It's my pleasure。


Adds shorturl python module.

Also adds the file .gitignore in order to ignore python cache directory (pycache)

Thank you very much!
Your Python version is now included: https://github.com/delight-im/ShortURL/commit/8eec6e7596e8a5bd8159753be2aeaaffb53f613b
I have left out the .gitignore, though, hope that's okay.

No problem :)


change failure to http 409

Codecov Report

Merging #174 (f0be4e2) into main (901decf) will increase coverage by 0.36%.
The diff coverage is 93.33%.

@@            Coverage Diff             @@
##             main     #174      +/-   ##
==========================================
+ Coverage   96.95%   97.31%   +0.36%
==========================================
  Files          44       45       +1
  Lines        2167     2309     +142
==========================================
+ Hits         2101     2247     +146
- Misses         36       37       +1
+ Partials       30       25       -5

Impacted Files
Coverage Δ

dtmgrpc/dtmgimp/types.go
100.00% <ø> (ø)

dtmsvr/trans_type_msg.go
86.66% <80.00%> (-6.67%)
:arrow_down:

dtmsvr/trans_status.go
95.32% <82.35%> (+1.80%)
:arrow_up:

dtmutil/utils.go
95.23% <87.09%> (-0.36%)
:arrow_down:

dtmsvr/api.go
96.00% <92.85%> (+0.08%)
:arrow_up:

dtmcli/dtmimp/trans_base.go
100.00% <100.00%> (ø)

dtmcli/dtmimp/utils.go
98.07% <100.00%> (+4.97%)
:arrow_up:

dtmcli/types.go
100.00% <100.00%> (ø)

dtmcli/xa.go
95.45% <100.00%> (ø)

dtmgrpc/type.go
100.00% <100.00%> (ø)

... and 12 more

Continue to review full report at Codecov.

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


Update to Play JSON 2.5.2, add “updateAs” method

This includes two commits from @MountVacation to update the version of Play JSON.

Just realized that my last commit on master also went into this PR, so now it’s not mergeable. Let me know if you want to merge, then I can remove it. But in the meantime I’ve published my fork to my own Bintray repo.

Github tells me I can merge it...
What is the problem?

The problem is that it now has a commit where I changed the publishing setup :)

I am happy to remove that commit if you are willing to merge and publish yourself :)

tell me what you prefer me to do, I'm not using the zipper anymore, so you're the lead now ;)

I suppose I can maintain it in my fork since we are using it at work :)


Calculate the overall time in GC [Parallel GC] for uploaded file

Unfortunately, I don't have a Serial GC log file that is the simplest one I suppose. However, I have parallel GC one.  I attached to the issue.
parallel-gc-debug.txt
Let try calculate the overall time of that application was in GC phase. You need to find and parse lines that similar to the following one:
[0.220s][info ][gc,cpu               ] GC(0) User=0.11s Sys=0.00s Real=0.00s

http://openjdk.java.net/jeps/158


Tap Feature

I do not know if this is feasible or not but it will nice to see the same colors when your tap to the entity
See  my picture, colorful but when tap it displays the default

Sorry, it is not possible. This is a generic more-info popup and I don't have control over its UI


[Meta] Train simple nwp model

Detailed Description

[ ] Train simple nwp model that uses CNN and one channel.
[x] save model to s3

other repos:

s3 bucket: https://github.com/openclimatefix/nowcasting_infrastructure/issues/42
update forecast: https://github.com/openclimatefix/nowcasting_forecast/issues/41

Context
This will be good to be use as v1 of the nowcasting platform - https://docs.google.com/presentation/d/1qbi3Gpmfyc3PN1yZeUmlMbtV9Jl77HQ1sZ-L48gNFeY/edit#slide=id.g112471ef353_0_6

Manage to train a model  - results are all ok. As good as needed
https://app.neptune.ai/o/OpenClimateFix/org/predict-pv-yield/e/PRED-951/all?path=validation%2Fplot%2F&attribute=epoch_1_4

model is saved here - https://s3.console.aws.amazon.com/s3/buckets/nowcasting-ml-models-development?region=eu-west-1&tab=objects

This has been deployed


Demonstrate successful compilation on arduino-esp32 master

Thanks for this PR, i will test it and merge.

@xtrinch Can you create PR to platformio branch please?

Sure, can do, but that branch does not exist so I can't make a PR to that?
Also, I've updated it to include the files I've previously deleted and everything works fine except I wasn't able to figure out which of the new midi functions in master are equivalent to the old ones, so I've commented them out.

I just created branch

Cool, thx, changed the base branch


Technical accuracy of what Event ID 1201 is about

Hello,
Just want clarify the technical accuracy of Event ID 1201 as AD FS does not seem to log it when there is an account lockout as stated in this document. Event ID 1201 is logged for Application Token Failures. https://docs.microsoft.com/en-us/windows-server/identity/ad-fs/troubleshooting/ad-fs-tshoot-logging also states that Event ID 1201 is about Application Token Failures, not lockouts, which is actually true when we look at the Event Logs.
On the other hand, Event ID 1210 is logged to indicate a lockout. So, it seems like there is perhaps a typo and 1201 should be 1210. Please validate.
Thanks,
Nimantha

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

ID: f0238d6f-4073-172e-de62-e2d973d993b7
Version Independent ID: 3789bcce-bebd-8a9d-c814-45ce6c1bfe47
Content: Configure AD FS Extranet Smart Lockout Protection
Content Source: WindowsServerDocs/identity/ad-fs/operations/Configure-AD-FS-Extranet-Smart-Lockout-Protection.md
Product: windows-server
Technology: ad-fs
GitHub Login: @billmath
Microsoft Alias: billmath

#assign:robinharwood
Thanks for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. The requested updates have not been made, although there have been further updates with subsequent releases which supersede the updates in your issue. We are closing this issue for now, but if you feel that it's still a concern, please respond and let us know. If you determine another possible update to our documentation, please don't hesitate to reach out again. #please-close

This is still indeed an issue because the issue I raised was not rectified. The explanation of Event ID 1201 described here is not actually true when it comes to actual logs on Windows Server. It is still technically inaccurate.

From: Robin Harwood @.>
Sent: 13 April 2022 5:35 PM
To: MicrosoftDocs/windowsserverdocs @.>
Cc: aut0pil0t @.>; Author @.>
Subject: Re: [MicrosoftDocs/windowsserverdocs] Technical accuracy of what Event ID 1201 is about (#5852)
#assign:robinharwood
Thanks for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. The requested updates have not been made, although there have been further updates with subsequent releases which supersede the updates in your issue. We are closing this issue for now, but if you feel that it's still a concern, please respond and let us know. If you determine another possible update to our documentation, please don't hesitate to reach out again. #please-close
—
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/windowsserverdocs/issues/5852#issuecomment-1097894026, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADX4LRZNRSERTLHBRI5ZLHDVE2POPANCNFSM5DOSYVQQ.
You are receiving this because you authored the thread.Message ID: @.***>

Only Microsoft employees can close issues or pull requests via commenting in this repo.

Only Microsoft employees can manage issues via commenting in this repo.

Hi Nimantha,
Apologies, I see the error in my closure. I was looking doc details, content source section but that doesn't appear to be the article you are reference. Just to confirm the issue is with the AD FS Troubleshooting - Events and Logging article, not the Configure AD FS Extranet Lockout Protection article in the content source?
Thanks,
Robin
#please-open

Hi Robin,
No, that's incorrect. I'm referring to the same article that I have stated originally, specifically the section https://docs.microsoft.com/en-us/windows-server/identity/ad-fs/operations/configure-ad-fs-extranet-smart-lockout-protection#ad-fs-extranet-smart-lockout-events

Thanks for clarifying Nimantha, leave it with me and I will look into this.

#assign:xelu86

Hey @aut0pil0t,
Thanks again for contacting Microsoft. We value your feedback.
A documentation change has been made correcting this Event ID and will be live after the next business day. We will now be closing this thread.
Thank you and have a most pleasant day. #please-close


Remove cutlass integration tests

All equivalent checks that were done in the cutlass tests have been moved into the Rust integration tests which use the composite buildpacks to execute.

TODO: remove cutlass tests from required validations


create a nice color hover effect to the search box on the home page

we can add some color hover effect to the search button box on the home page
and the color will be according to the submit button
please assign with level 2
thank you @vansh-codes

Please assign me this issue under gssoc extended label.


Fix: Disable refresh token for inactive user.

Issue Description:
A refresh token generated for a previously active user was allowed to issue new access token although the user is inactive.
Fix:
Added a check to verify if the user claim in refresh token corresponds to an active user.
Related issues

https://github.com/jazzband/djangorestframework-simplejwt/issues/812

@wissam-al-kahwaji
Thank you for your suggestion.
Signals are quite powerful, but I would hesitate using them here.
As a signal will be associated with the save() method in User models, but  a user can be disabled without a call to save().

@ajay09
Do you mean to disable the user from the database directly?

@wissam-al-kahwaji
Yes when the db is directly updated.
But also when we use bulk_update() as that doesn't call the save() method of the model.
https://docs.djangoproject.com/en/5.0/ref/models/querysets/#django.db.models.query.QuerySet.bulk_update

@ajay09
Actually, all you need to do is add a field containing the user identifier in the blacklist to search for update codes or user_id, along with adding another field as a boolean value to tell us if it's active or not. Thus, you're retaining only one query here, and the reason for adding the boolean field is to express that the user account in general has been suspended, not just the unique identifier. Please note that in this solution, you'll need to store the user without the unique identifier, thus relying on determining whether the user or the unique identifier is listed in the blacklist.

@ajay09
Actually, all you need to do is add a boolean field to inform us if it's active or not. Thus, you're retaining only one query here, and the reason for adding the boolean field is to express the suspension of the user account in general, not just the refresh token. Please note that in this solution, you'll be storing the user without the refresh token, thus relying on determining whether the user or the refresh token is listed in the blacklist.

I don't think we should modify the blacklist table to store whether use is active/inactive, preserving Separation of concerns.

It will be responsible for clarifying whether the user is completely blacklisted or whether it is just the token

Hmm. I don't think this is the correct fix. Instead the refresh serializers should be calling api_settings.USER_AUTHENTICATION_RULE in their validate methods, like TokenObtainSerializer already does.
The implementation you have in this PR requires that blacklisting (an optional feature) is enabled.
Also JWTAuthentication should be using USER_AUTHENTICATION_RULE instead of only checking User.is_active (to gain access to any custom rules).
Also please add a test to verify the fix.

@vainu-arto
That's a very nice suggestion.
I will move the logic to TokenRefreshSerializer.
Thanks.

@wissam-al-kahwaji
I have updated the code. Could you please provide a re-review?
Thanks.

@wissam-al-kahwaji Excuse me, but can we move ahead with this?

Can we get this merged?

@timmyomahony
It's saying "At least 1 approving review is required by reviewers with write access"

Uhmm.. can i take a look at this?

Yes please!

let's push this out there :)

I don't believe it, folks.

Please, make a PyPI release ASAP.


[HTTP] Add charset to the Content-Type header

Make it the default behavior and let used override this behavior.

Closing this as this has been implemented.

With the content-type header value user can mention the charset


Please add Multi-CPU architecture for Docker image

Hello, I would kindly request adding support for Multi-CPU architecture on Docker Image with building and pushing images for different architectures during CI/CD workload.

I think the only way to realistically do this is to build the container images from GitHub actions.

I agree with jelmer, we should set up GitHub actions so that on release, the artifacts and docker images are set up automatically.
@daniel-san05 is that something you could help us with?

Not OP but I'd be happy to help with that; I have some experience with build automation, an d GH actions in particular and would love to contribute.

@janw That'd be great!

I agree with jelmer, we should set up GitHub actions so that on release, the artifacts and docker images are set up automatically. @daniel-san05 is that something you could help us with?

Completely forgot answering the question :-(
But if @janw is contributing already it will go its way ;-).

I compiled one for personal use in the meantime https://hub.docker.com/r/kendofriendo/isso

Tested and it is working fine. 😊
And of course: Happy new year!


random serial

I am still experiencing the same issue with #311. The problem can be reproduced by:

start the proxy server
open firefox and view a hijacked page
restart the proxy server(which means all cached certificates at the server-side are cleared)
view the same page without closing the browser

I think the problem is cause by a misuse of hash for certificate serial number. As each time signHost is called, a different certificate is generated but the serial is gonna be the same, which would trigger an alert in Firefox.
I think it would be better to simply generate a random serial number.

How would you keep it consistent between runs?

We do not need to keep it consistent between runs. What we needed to prevent is generating certificates with the same serial number, even between runs. I cannot see a side-effect as long as the serial numbers are kept different from each other.

@elazarl Hello! Also interested in some solution... Thanks!

Cleaning my own stale PRs, looks like solved in #355


Can you provide a example of nsdperftool.py syntax use? I am getting an error when trying to execute the program

Hello, Koet.py is working good but I want to only run perftool, however i am not sure on how to make it work, so far I am receiving the following error:
[root@gabriel-node-mgr SpectrumScale_NETWORK_READINESS]# ./nsdperfTool.py --client gabriel-node-1 --server gabriel-node-mgr
2023-07-13_06:18:53: MainThread: CMD: ip addr show
Traceback (most recent call last):
File "./nsdperfTool.py", line 526, in
localNode = getLocalNode(allNodes)
File "./nsdperfTool.py", line 238, in getLocalNode
rc, ipaddr_output, ec = runcmd("ip addr show")
File "./nsdperfTool.py", line 427, in runcmd
[out, err] = p.communicate(timeout = int(conf["ttime"]) + timerWindow)
ValueError: invalid literal for int() with base 10:
Any help would be appreciated

Hi
It never been the idea of this to use only a underlying subset of this. It was designed modularly to have multiple tests (latency and throughput now) in a modular way centralized for automation and human consumption from the koet (a Finnish word loosely translated as tests or experiences) script.
Having said that if you really want only to run throughput, just comment the latency calls at koet .. bonus if you add a paremeter and a pull request for doing exactly that; and maybe another one for doing only latency as it is a massive killer of performance. We simulated latencies on same environments when we came up with the KPI for ECE and 1.0 to 2.0 ms gives you near half the performance
I am going to close this one as it is not being used as designed nor a problem of the koet path


Font sizes

Hi,
Ive looked at:
https://github.com/littlevgl/lvgl/issues/609
https://github.com/littlevgl/lvgl/issues/354
and
https://github.com/littlevgl/lvgl/issues/247
but I'm still not sure what to do here.
I've enabled #define LV_FONT_ROBOTO_28    1 in lv_config.h and added the following into my code:
`    static lv_style_t st;
lv_style_copy( &st, &lv_style_plain );
st.text.color = LV_COLOR_WHITE;
st.text.font = &lv_font_roboto_28;
lv_label_set_text(label, "22:30");
lv_obj_set_style( label, &st );

`
but my font size is still tiny. See attached photos.

Is there any way to scale these up so they are 5x to 10x bigger?
Thanks in advance
Alex

We use GitHub issues for development related discussions.
Please use the forum to ask questions.
It's possible to sign in to the forum with your GitHub account... you don't need to make a separate account.


#31711 breaks assignability

// @strict: true
function g(headerNames: any) {
    let t = [{ hasLineBreak: false, cells: [] }];
    const table = [{cells: headerNames }].concat(t);
}

Expected behavior:
No error.
Actual behavior:
Error on concat(t): { hasLineBreak: boolean, cells: never[] } is not assignable to { cells: any }. hasLineBreak is missing.
Playground Link:
Related Issues:

Found in the user tests, in prettier.

Looks like it got the assignability check backwards, that’s odd.

From in-person discussion, I think we are going to take this as a breaking change for 3.6.

Wait, what's the accepted breaking change?  The error mentioned in the OP?  Because if so the error message is nonsensical:

{ hasLineBreak: boolean, cells: never[] } is not assignable to { cells: any }. hasLineBreak is missing.

With that wording, it looks like a bug--the apparent bug being that a structural assignability check was done with the source and target reversed.  Even if the breakage is acceptable, the error message IMO isn't as it's exactly backwards a.c.t. typical structural typing semantics.

The issue here is when to widen the object expression in property access (obj.x) and element access (obj["x"]) expressions. We have been quite inconsistent here. It used to the that we'd always widen obj in a property access and never widen obj in an element access. That's obviously not right as the two are supposed to be synonymous. In #31711 we switched to widening obj only when it is an empty array literal in a property access expression (e.g. [].concat(...)), but that isn't right either.
I've given it some thought, and I think the real solution is to widen obj when the property or element access is (a) the target of an assignment, or (b) a method access in a call expression (i.e. the this of the call). These are the situations in which we are mutating (or possibly mutating) the object and therefore should widen its type.

To demonstrate the inconsistency, this is our behavior in 3.5.1 and earlier:
// @strict: true
function g1(headerNames: any) {
    let t = [{ hasLineBreak: false, cells: [] }];
    const table = [{cells: headerNames }].concat(t);  // No error
}

function g2(headerNames: any) {
    let t = [{ hasLineBreak: false, cells: [] }];
    const table = [{cells: headerNames }]["concat"](t);  // Error
}

@ahejlsberg I’m not actually doubting the type error itself - I’m just confused as to how TypeScript arrives at the conclusion that “(derived) is not assignable to (base)” (per usual structural rules) - even in cases of contravariance it normally gets this right:
// @strictFunctionTypes: true
let f: (x: Derived) => void;
let g: (x: Base) => void;
g = f;
// error because (ultimately) Base is not assignable to Derived


NetCore SDK 2.2 missing in the new Ubuntu Virtual environment version 20200621.1

Describe the bug
We are currently using 16.04 LTS in Azure build and this breaks our build as we are using netcore 2.2 sdk for our application which isn't readily available on any new Ubuntu virtual environment installed softwares starting from version released on 20200621.1
Area for Triage:
https://github.com/actions/virtual-environments/blob/master/images/linux/Ubuntu1604-README.md
Question, Bug, or Feature?:
Will it be possible to have the NETCore SDK 2.2 added again as a pre-installed software in Ubuntu virtual environments 16.04, 18.04, 20.04 ?
Virtual environments affected

Ubuntu 16.04 LTS
Ubuntu 18.04 LTS
Ubuntu 20.04 LTS

Expected behavior
We expected to have NETCore SDK 2.2 like the previous releases of Ubuntu virtual environments.
Actual behavior

Open a Azure YAML build script and use vmImage: 'ubuntu-16.04'
Add build and test task.
Run the pipeline.

Hi @wizz2020
There was an announcement https://github.com/actions/virtual-environments/issues/975
If you still need to use .NET Core 2.2.x, you will be able to install it in runtime using the following tasks:
UseDotNet (Azure DevOps)
setup-dotnet (GitHub Actions)
I'm going to close the issue, but feel free to contact us if you have any concerns.
Thank you!

Thank you very much. UseDotNet (Azure DevOps) works for me.


⚠ Bofrost.de has degraded performance

In 9beca29, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1814 ms

Resolved: Bofrost.de performance has improved in e9d241b after 6 minutes.


Release v0.6.1 Linux artifact is not functional.

What happened:
Release artifact eksctl-anywhere-v0.6.1-linux-amd64.tar.gz is not functional.
What you expected to happen:
Any command issued to eksctl anywhere using release v0.6.1 returns nothing. For example:
$ eksctl anywhere version
$ eksctl anywhere -h

When checking the file signature, it looks weird (comparing artifact from v0.6.1 with v0.6.0:
$ file /usr/local/bin/eksctl-anywhere
/usr/local/bin/eksctl-anywhere: ELF 64-bit LSB executable, x86-64, version 1 (SYSV), dynamically linked, interpreter /lib64/ld-linux-x86-64.so.2, too large section header offset 28686592

$ file /usr/local/bin/eksctl-anywhere.old
/usr/local/bin/eksctl-anywhere: ELF 64-bit LSB executable, x86-64, version 1 (SYSV), dynamically linked, interpreter /lib64/ld-linux-x86-64.so.2, Go BuildID=gUlv0ssAJFK5V8STD6-2/e83vRMgUyxYvubndrH8Q/g4XdUxfBmzRnpcus2vqd/TTBeqNaNjCjk9MxcXgGY, stripped

I've double checked the binary signature and the download is valid.
How to reproduce it (as minimally and precisely as possible):
Just download the plugin release for Linux and test it.
Environment:

EKS Anywhere Release: 0.6.1
EKS Distro Release: 0.76.0

Never mind. I messed things up here... 😵💫