Accessing belongs_to association without a key creates a useless query

Environment

Ruby 3.2.2
Rails 7.1.1
ActiveHash 3.2.1

Problem
When accessing a belongs_to association, if the related id is missing then a query for id IS NULL is made. This is different from the standard ActiveRecord behaviour and I think in 99% of cases a redundant query. Seems to be caused by the internal use of find_by_id with a blank argument: https://github.com/active-hash/active_hash/blob/master/lib/associations/associations.rb#L177
You can reproduce and observe with this script:
require "bundler/inline"

gemfile(true) do
  source "https://rubygems.org"

  git_source(:github) { |repo| "https://github.com/#{repo}.git" }

  gem "rails", "7.1.1"
  gem "sqlite3"
  gem "active_hash", "3.2.1"
end

require "active_hash"
require "active_record"
require "logger"

ActiveRecord::Base.establish_connection(adapter: "sqlite3", database: "db/development.sqlite3")
ActiveRecord::Schema.define do
  create_table :authors, force: true
  create_table :magazines, force: true do |t|
    t.belongs_to :author, null: true
  end
end

class Author < ActiveRecord::Base
end

class Book < ActiveHash::Base
  include ActiveHash::Associations

  fields :author_id

  belongs_to :author
end

class Magazine < ActiveRecord::Base
  belongs_to :author, optional: true
end

book = Book.new(author_id: nil)
magazine = Magazine.create!(author_id: nil)

ActiveRecord::Base.logger = Logger.new(STDOUT)

puts "-" * 50
puts "Accessing ActiveHash association with NULL key"
book.author
puts "-" * 50
puts "Accessing ActiveRecord association with NULL key"
magazine.author
puts "-" * 50

Potential Solution
Don't bother querying if the key is NULL, if there is some weird edge case where you would want this then perhaps the behaviour could be enabled with a config option or optional argument when defining the association.

So the request is to throw an error if the model does not have a fields :author_id or it does not respond_to?(:author_id)?
Wonder if this will cause an issue if a model defines associations before fields.
(I almost think it would be a reasonable "breaking" change)

@kbrock

So the request is to throw an error if the model does not have a fields :author_id or it does not respond_to?(:author_id)?

No, it is that when the association key is nil then just do a noop instead of calling find_by_X and making a useless db query. ActiveRecord associations will not create a database query in this event however active hash associations will: https://github.com/rails/rails/blob/main/activerecord/lib/active_record/associations/association.rb#L290-L304
Hope that makes sense 🙏

Ugh. Your test script is pretty clear here. Sorry. (see the last query below)
I do like how rails avoids this query. I am a little concerned that people will want to query for a nil id. Yes, Rails only supports models with a unique present id, but since when have people followed strict Rails?
irb(main):004:0> Vm.find_by(:id => nil)
  Vm Load (2.2ms)  SELECT "vms".* FROM "vms" WHERE "vms"."id" IS NULL LIMIT $1  [["LIMIT", 1]]
=> nil
irb(main):005:0> VmOrTemplate.find(nil)
/Users/kbrock/.gem/ruby/3.1.4/gems/activerecord-<IP_ADDRESS>/lib/active_record/relation/finder_methods.rb:456:
in `find_with_ids': Couldn't find Vm without an ID (ActiveRecord::RecordNotFound)
irb(main):014:0> vm = VmOrTemplate.first ; nil
  VmOrTemplate Load (0.5ms)  SELECT "vms".* FROM "vms" ORDER BY "vms"."id" ASC LIMIT $1  [["LIMIT", 1]]
=> nil
irb(main):015:0> vm.ext_management_system ; nil
  ExtManagementSystem Load (1.1ms)  SELECT "ext_management_systems".*
  FROM "ext_management_systems" WHERE "ext_management_systems"."id" = $1 LIMIT $2 [["id", 3], ["LIMIT", 1]]
  ExtManagementSystem Inst Including Associations (0.2ms - 1rows)
=> nil
irb(main):016:0> vm = Vm.where(:ext_management_system => nil).first ; nil
  Vm Load (0.8ms)  SELECT "vms".*  I am a little concerned that people will want to query for a nil id. Yes, Rails only supports models with a unique present id, but since when have people followed strict Rails?

Sadly I agree 😭 My next thought was an opt-in option on the association method to enable this behaviour. Seems clunky and it would be hard naming it but then at least it doesn't break unexpectedly.


Lack of memory

How did you exactly run the Tree-base jupyter notebook on the original CIDIDS2017 dataset, it's insane!!! 16 GB of memory is far from being enough to support the whole exps.

The original CICIDS2017 dataset is about 1GB. I'm also working on a machine with 16GB of memory and I believe it's enough to run it. Please try it on another machine.


support, redirects: fix failure to acquire HTTPS certs for redirected domains

Issue #2655 describes the problem, and the resolution was to not redirect unconditionally, but to all requests not related to ACME challenges.

Closes #2655

Thank you @yuvipanda!!!


docker container status

FROM openjdk:8
LABEL author="neelarao"
RUN wget https://referenceappkhaja.s3-us-west-2.amazonaws.com/spring-petclinic-2.2.0.BUILD-SNAPSHOT.jar
EXPOSE 8080
CMD ["java", "-jar", "spring-petclinic-2.2.0.BUILD-SNAPSHOT.jar"]
when running a container by using above dockerfile,
contianer status showing as "up"

but while running container from below dockerfile,
FROM openjdk:8
LABEL author="neelarao"
RUN wget https://github.com/spring-projects/spring-petclinic.git
EXPOSE 8080
CMD ["java", "-jar", "spring-petclinic.git"]

container status showing as "exited"
is there anything wrong in CMD instruction?

my self, i'm resolved.

my self, i'm resolved.

How did you resolve?

in the second dockerfile, the cloned link was  https://github.com/spring-projects/spring-petclinic.git,  it is just developed app, in further we need to build it then send it to S3 bucket or Artifactory, and then we have to clone it from their (S3 or artifactory).

in the second dockerfile, the cloned link was  https://github.com/spring-projects/spring-petclinic.git,  it is just developed app, in further we need to build it then send it to S3 bucket or Artifactory, and then we have to clone it from their (S3 or artifactory).

Thank you


[chainer.exporters.caffe] Export eps param of BatchNormalization

I met a problem for exporting caffe model.
exporters.caffe don't export eps param of BatchNormalization.
Chainer sets eps param to 2e-5 by default. however, Caffe sets eps param to 1e-5 by default.
The output value becomes very different in the regression task.

Thanks for the feedback! We'll fix this in #4884.

Sweet!


Add missing metadata index.

We're running metaflow-service v2.4.4, and noticed the metaflow-backend was missing some metadata_v3 indices. This fixes the issue by adding the expected index.

@trhodeos can you help me with where is the index expected?

I only have a slow query that I was trying to fix... I'm not 100% sure where it gets generated:
SELECT * FROM (
SELECT
tasks_v3.flow_id AS flow_id,tasks_v3.run_number AS run_number,tasks_v3.run_id AS run_id,tasks_v3.step_name AS step_name,tasks_v3.task_id AS task_id,tasks_v3.task_name AS task_name,tasks_v3.user_name AS user_name,tasks_v3.ts_epoch AS ts_epoch,tasks_v3.last_heartbeat_ts AS last_heartbeat_ts,tasks_v3.tags AS tags,tasks_v3.system_tags AS system_tags,COALESCE(attempt.attempt_id, 0) as attempt_id,attempt.started_at as started_at,
(CASE
WHEN COALESCE(GREATEST(attempt.attempt_finished_at, attempt.task_ok_finished_at), next_attempt_start.ts_epoch) IS NULL
AND tasks_v3.last_heartbeat_ts IS NOT NULL
AND @(extract(epoch from now())-tasks_v3.last_heartbeat_ts)>120
THEN tasks_v3.last_heartbeat_ts*1000
ELSE COALESCE(GREATEST(attempt.attempt_finished_at, attempt.task_ok_finished_at), next_attempt_start.ts_epoch)
END) as finished_at
,attempt.attempt_ok as attempt_ok,
(CASE
WHEN attempt.attempt_ok IS NOT NULL
THEN NULL
ELSE attempt.task_ok_location
END) as task_ok
,
(CASE
WHEN attempt.attempt_ok IS TRUE
THEN 'completed'
WHEN attempt.attempt_ok IS FALSE
THEN 'failed'
WHEN COALESCE(attempt.attempt_finished_at, attempt.task_ok_finished_at) IS NOT NULL
AND attempt_ok IS NULL
THEN 'unknown'
WHEN COALESCE(attempt.attempt_finished_at, attempt.task_ok_finished_at) IS NOT NULL
THEN 'completed'
WHEN next_attempt_start.ts_epoch IS NOT NULL
THEN 'failed'
WHEN tasks_v3.last_heartbeat_ts IS NOT NULL
AND @(extract(epoch from now())-tasks_v3.last_heartbeat_ts)>120
AND COALESCE(attempt.attempt_finished_at, attempt.task_ok_finished_at) IS NULL
THEN 'failed'
WHEN tasks_v3.last_heartbeat_ts IS NULL
AND @(extract(epoch from now())*1000 - COALESCE(attempt.started_at, tasks_v3.ts_epoch))>86400000
AND COALESCE(attempt.attempt_finished_at, attempt.task_ok_finished_at) IS NULL
THEN 'failed'
WHEN tasks_v3.last_heartbeat_ts IS NULL
AND attempt IS NULL
THEN 'pending'
ELSE 'running'
END) AS status
,
(CASE
WHEN tasks_v3.last_heartbeat_ts IS NULL
AND @(extract(epoch from now())*1000 - COALESCE(attempt.started_at, tasks_v3.ts_epoch))>86400000
AND COALESCE(attempt.attempt_finished_at, attempt.task_ok_finished_at) IS NULL
THEN NULL
WHEN tasks_v3.last_heartbeat_ts IS NULL
AND attempt IS NULL
THEN NULL
ELSE
COALESCE(
GREATEST(attempt.attempt_finished_at, attempt.task_ok_finished_at),
next_attempt_start.ts_epoch,
tasks_v3.last_heartbeat_ts*1000,
@(extract(epoch from now())::bigint*1000)
) - COALESCE(attempt.started_at, tasks_v3.ts_epoch)
END) as duration
FROM tasks_v3
LEFT JOIN LATERAL (
SELECT
max(started_at) as started_at,
max(attempt_finished_at) as attempt_finished_at,
max(task_ok_finished_at) as task_ok_finished_at,
max(task_ok_location) as task_ok_location,
attempt_id :: int as attempt_id,
max(attempt_ok) :: boolean as attempt_ok,
task_id
FROM (
SELECT
task_id,
ts_epoch as started_at,
NULL::bigint as attempt_finished_at,
NULL::bigint as task_ok_finished_at,
NULL::text as task_ok_location,
NULL::text as attempt_ok,
(CASE
WHEN pg_typeof(value)='jsonb'::regtype
THEN value::jsonb->>0
ELSE value::text
END)::int as attempt_id
FROM metadata_v3 as meta
WHERE
tasks_v3.flow_id = meta.flow_id AND
tasks_v3.run_number = meta.run_number AND
tasks_v3.step_name = meta.step_name AND
tasks_v3.task_id = meta.task_id AND
meta.field_name = 'attempt'
UNION
SELECT
task_id,
NULL as started_at,
ts_epoch as attempt_finished_at,
NULL as task_ok_finished_at,
NULL as task_ok_location,
NULL as attempt_ok,
(CASE
WHEN pg_typeof(value)='jsonb'::regtype
THEN value::json->>0
ELSE value::text
END)::int as attempt_id
FROM metadata_v3 as meta
WHERE
tasks_v3.flow_id = meta.flow_id AND
tasks_v3.run_number = meta.run_number AND
tasks_v3.step_name = meta.step_name AND
tasks_v3.task_id = meta.task_id AND
meta.field_name = 'attempt-done'
UNION
SELECT
task_id,
NULL as started_at,
ts_epoch as attempt_finished_at,
NULL as task_ok_finished_at,
NULL as task_ok_location,
(CASE
WHEN pg_typeof(value)='jsonb'::regtype
THEN value::jsonb->>0
ELSE value::text
END) as attempt_ok,
(regexp_matches(tags::text, 'attempt_id:(\d+)'))[1]::int as attempt_id
FROM metadata_v3 as meta
WHERE
tasks_v3.flow_id = meta.flow_id AND
tasks_v3.run_number = meta.run_number AND
tasks_v3.step_name = meta.step_name AND
tasks_v3.task_id = meta.task_id AND
meta.field_name = 'attempt_ok'
UNION
SELECT
task_id,
NULL as started_at,
NULL as attempt_finished_at,
ts_epoch as task_ok_finished_at,
location as task_ok_location,
NULL as attempt_ok,
attempt_id as attempt_id
FROM artifact_v3 as task_ok
WHERE
tasks_v3.flow_id = task_ok.flow_id AND
tasks_v3.run_number = task_ok.run_number AND
tasks_v3.step_name = task_ok.step_name AND
tasks_v3.task_id = task_ok.task_id AND
task_ok.name = '_task_ok'
) a
WHERE a.attempt_id IS NOT NULL
GROUP BY a.task_id, a.attempt_id
) as attempt ON true
LEFT JOIN LATERAL (
SELECT ts_epoch
FROM metadata_v3 as next_attempt_start
WHERE
tasks_v3.flow_id = next_attempt_start.flow_id AND
tasks_v3.run_number = next_attempt_start.run_number AND
tasks_v3.step_name = next_attempt_start.step_name AND
tasks_v3.task_id = next_attempt_start.task_id AND
next_attempt_start.field_name = 'attempt' AND
(attempt.attempt_id + 1) = (next_attempt_start.value::jsonb->>0)::int
LIMIT 1
) as next_attempt_start ON true
) T
WHERE flow_id = 'MyFlow' AND run_number = '78624' AND step_name = 'start' AND task_id = '681747'
ORDER BY attempt_id DESC

The diff is that this query uses run_number and flow_id, where most of the metadata indexes have run_id and flow_id... The PKey usees run_number and flow_id, but it also includes id, which this query doesn't include.

the index already exists, no?

Ah looks like it does, but doesn't on our db 🤦 Not sure why it doesn't exist for us. Let me manually apply that migrations (and double check that all of them were actually applied).


Fix Castables issue #1312

Summary
Fix for issue with Castables described in https://github.com/barryvdh/laravel-ide-helper/issues/1312
If type of a Castable can not be determined from return type of get method from castUsing()->CastsAttributes then the original Castable class should be used.
Type of change

[x] Bug fix (non-breaking change which fixes an issue)

Checklist

[x] Existing tests have been adapted and/or new tests have been added
[x] Code style has been fixed via composer fix-style
[x] Add a CHANGELOG.md entry

LGTM, but please don't ignore the PR template and add a changelog entry 🙏🏼

Hi Markus, thanks for your Approval. This is my first ever Open Source project/lib PR so I'm not 100% sure how to do it right. Could you please elaborate more on what I could do better re. PR template?

LGTM, but please don't ignore the PR template and add a changelog entry 🙏🏼

Thanks and sorry, that's my first ever PR to an Open Source repo so I'm not always sure about the protocol. I added the CHANGELOG.md entry.

Thanks!


Modern UI design - Just like m3.material.io

What problem does this feature solve?
This will help to create web pages with a modern UI feel component, I like your component but it seems to be outdated UI (No offence)
What does the proposed API look like?
You can get inspiration from https://m3.material.io/components

Hi @bharatbamaniya
Ng-zorro is the Angular implementation of a design system called Ant. We are currently at Ant version 4, and will move to version 5 in the future.
You can check the design section of Ant document to become more familiar with it.
As an implementation we are to follow the Ant design system rules; meaning the rules of material will never be applied in our library, since material in another design system and has an Angular UI implementation for its own called Angular material.

什么时候能更新到ant design 5规范，有具有的时间表吗？


tried to write some web application using uwebsocket http but realised it's not easy

@uNetworkingAB this is really a great piece of software but i was wondering if there are more examples whereby this can be used as a full fledged web application framework like accepting a form input and parameters processing.
i think you could have reached a speed limit with io_uring for now with uwebsocket so was wondering if maybe you can consider helping the community expand this into something more "enterprise" skewed for web application framework development?
maybe not now but do hope this can be one of the directions. thx.

Pretty sure we already support both of those features. Queries and multipart.

@uNetworkingAB it's not obvious in the examples given
https://github.com/uNetworking/uWebSockets/tree/master/examples
do u know where i can get better docs with other's examples to work with? a basic "cms" / todo example.

That's not the average feedback I've received so far. That said, there is always going to be a certain cut-off base competence needed for any project.


Test navigation get test site flow

Implement first version of the get test site flow.

Response will be displayed in a carousel.
For now assume the user will provide its postal code, support for GPS coordinates sent as metadata will be added later.
For now assume no image in the response. Map image will be added later once #276 is done.
The Call button should be a simple tel: url.
The Destination button should be a link to maps with either GPS coordinates or the address of the testing site (depending on the information we get from Clinia API). The use of apple maps URL seems to do a better job than Google Maps on most devices, redirecting the user to Google Maps when appropriate, we have to test it on additional devices/browser though. Map Links Scheme
The more information button should send an intent to which the bot will reply with the address and opening hours of the site. It's probably best to implement this part in another PR and to leave out this button for now.

Here is a sample of the message format expected by the widget:
dispatcher.utter_message(
    attachment={
        "type": "template",
        "payload": {
            "template_type": "generic",
            "elements": [
                {
                    "title": "Test site name",
                    "subtitle": "Test site description",
                    "image_url": "http://....",
                    "buttons": [
                        {
                            "title": "Call<PHONE_NUMBER>",
                            "type": "web_url",
                            "url": "tel:514-555-1234",
                        },
                        {
                            "title": "Destination",
                            "type": "web_url",
                            "url": "http://maps.apple.com/?q=51.507269,-0.127695",
                        },
                        {
                            "title": "More information",
                            "type": "postback",
                            "payload": "/an.intent",
                        },
                    ],
                },
                ...
            ],
        },
    }
)

Blocked by #265, #268


Month/Week Change

How can I be notified that week/Month has been changed
I am fetching 2 month data from backend services
now I want to be notified when user had switched to third month
Using JZLongPressWeekView
Installed Pods

There is a delegate method from JZBaseViewDelegate called initDateDidChange.
You can simply refer to the code DefaultViewController line 62.

I only used JZLongPressWeekView
Assigning this protocol "JZBaseViewDelegate"  didn't Called the init Date method

Actually I need Scrolling Direction so, based on which comparing dates I can request my backend server for required slots to show

Thanks Got To Know From Your Demo Code

As you mentioned to use initDateChange Function, I did same
but resulted in a strange issue
on Some devices like iPhone XS Max, iPhone X InitDate Change function is skipped to call
Please find attached Video - https://drive.google.com/open?id=14upTzeJ6qX8SPrRUbph0YQye2WFZBz-z
I showed toast as soon as InitDataChage function is called
As you could see sometime toast don't show up bcz that function is failed to call.
Can you please suggest Solution what I can do to get it solved?
Thanks

Hey @zjfjack Any Response. Please

It seems that there are some issues with the pagination, I will look into it soon. If I cannot reproduce it, I will let you know ASAP.

Thanks this will really help me out. It get stuck on some deices as I mentioned like iPhone XS Max, iPhone X whereas it works perfectly on iPhone 6s. Have a look into it.

I might find the issue which is a quite annoying issue related to some other issues #46
I will try to fix it, but you have to wait until next weekend.

Hey, mate.
I finally resolve this issue with 0.7.0 version released. It is actually a quite major and difficult problem for the pagination.
You can check the release comment here.
If you have any other issue, let me know. I will close this issue.
Thanks


corrected the square root bug

I have tried to fix the issue. It is working in the mathematical expression also.
Fixes #67

Moreover, it is accepting expression in the format of √+9 and √-9 also.

Is there any problem with this PR?

You have done some unnecessary changes like deleting a tag and again adding it @lavish-github

I am extremely for my mistake, but I am not able to understand where I have made those changes😅. Regarding index.html, some of the changes have been made automatically by the linter extension. If something else is there, then I request you to kindly let me know, so that I can prevent such mistakes in future. Thanks

No worries, but please keep in mind that making changes without a specific purpose can create a negative impression. Overall, it's fine, but next time, change something only if it has a meaningful purpose; otherwise, leave it as it was before. 😄

Sure


Architecture: Building Block View

Section Building Block View (design/building-block-view.qmd). This is the one of the most important sections and the biggest. So we can all work on different sections if needed.

[x] Subsection: Whitebox overall system, Level 1

[x] Insert C4 Container diagram in section "Building Block View (Level 1)".
[x] Table: Black box descriptions (brief) of the contained blocks
[x] List: Black box descriptions (more detailed) of the contained blocks

[ ] Subsection: Level 2

[ ] Insert C4 Component diagram in section "Building Block View (Level 2)".
[ ] Table: Black box descriptions (brief) of the contained blocks

[ ] (Optional) Subsection: Level 3

[ ] Insert C4 Code diagram in section "Building Block View (Level 3)".

Many of these things have been fixed, are no longer relevant, or are addressed in other issues. Closing


python3 manage.py migrate  .Send error please help me

[root@localhost FasterRunner-master]# python3 manage.py migrate
System check identified some issues:
WARNINGS:
?: (mysql.W002) MySQL Strict Mode is not set for database connection 'default'
HINT: MySQL's Strict Mode fixes many data integrity problems in MySQL, such as data truncation upon insertion, by escalating warnings into errors. It is strongly recommended you activate it. See: https://docs.djangoproject.com/en/2.1/ref/databases/#mysql-sql-mode
Operations to perform:
Apply all migrations: admin, auth, contenttypes, djcelery, fastrunner, fastuser, sessions
Running migrations:
_mysql_exceptions.ProgrammingError: (1064, "You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '(6) NOT NULL)' at line 1")
The above exception was the direct cause of the following exception:
django.db.utils.ProgrammingError: (1064, "You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '(6) NOT NULL)' at line 1")
During handling of the above exception, another exception occurred:
django.db.migrations.exceptions.MigrationSchemaMissing: Unable to create the django_migrations table ((1064, "You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '(6) NOT NULL)' at line 1"))
`

I am facing same issue. How did you resolved this?

Same issue here, any updates ?

Try this at the top of django settings.py
from django.db.backends.mysql.base import DatabaseWrapper
DatabaseWrapper.data_types['DateTimeField'] = 'datetime' # fix for MySQL 5.5

Same issue here, any updates ?

from django.db.backends.mysql.base import DatabaseWrapper
DatabaseWrapper.data_types['DateTimeField'] = 'datetime' # fix for MySQL 5.5

@rishabh7683 Essas linhas salvou minha vida aqui obrigado pra ficou assim:

Versão do Python 3.8.3   64 bits

mysqlclient==2.0.3

settings:

'default': {
'ENGINE': 'django.db.backends.mysql',
'NAME': 'Nome_Banco',
'USER': 'root',
'PASSWORD': 'senha',
'HOST': 'endereço_ip',
'PORT': '3306',
}

- Colocar no inicio do Settings:

	```
from django.db.backends.mysql.base import DatabaseWrapper
	DatabaseWrapper.data_types['DateTimeField'] = 'datetime' # fix for MySQL 5.5


JDL generation no longer accepting istioRoute after 6.0.1 upgrade

Overview of the issue
The JDL file worked fine in jhipster 5.8.  This JDL was taken from a jhipster kubernetes demo.
Since upgrading to jhipster 6.0.1, it now gives the following error. Taking out the following two JDL directives will successfully generate the project.
istio autoInjection
istionRoute true
But these two are needed.  Again this worked perfectly in jhipster 5.8
❯ jhipster import-jdl jhipster-app.jdl --debug
INFO! Using JHipster version installed globally
INFO! Debug logging is on
DEBUG!  Executing CLI only script
DEBUG!  cmd: import-jdl from ./import-jdl
DEBUG!  args: jhipster-app.jdl, commands: Object, options: Object, _execs: Object, _allowUnknownOption: true, _args: Object, _name: import-jdl, _noHelp: false, parent: Object, _description: Create entities from the JDL file passed in argument.
By default everything is run in parallel. If you like to interact with the console use '--interactive' flag., _argsDescription: undefined, _events: Object, _eventsCount: 1
INFO! Executing import-jdl jhipster-app.jdl
INFO! Options: debug: true, from-cli: true
DEBUG!  JDLProcessor started with jdlFiles: jhipster-app.jdl and options: debug: true, from-cli: true
INFO! The JDL is being parsed.
error: Error message:
MismatchedTokenException: Expecting --> '}' <-- but found --> 'istionRoute' <--
at line: 225, column: 3
DEBUG!  Error:
SyntaxError: Error: MismatchedTokenException: Expecting --> '}' <-- but found --> 'istionRoute' <--
at line: 225, column: 3
ERROR! Error while parsing applications and entities from the JDL SyntaxError: Error: MismatchedTokenException: Expecting --> '}' <-- but found --> 'istionRoute' <--
at line: 225, column: 3
SyntaxError: Error: MismatchedTokenException: Expecting --> '}' <-- but found --> 'istionRoute' <--
at line: 225, column: 3
at callApiMethod (/usr/local/Cellar/jhipster/6.0.1/libexec/lib/node_modules/generator-jhipster/node_modules/jhipster-core/lib/reader/jdl_reader.js:88:11)
at parse (/usr/local/Cellar/jhipster/6.0.1/libexec/lib/node_modules/generator-jhipster/node_modules/jhipster-core/lib/reader/jdl_reader.js:72:10)
at Object.parseFromFiles (/usr/local/Cellar/jhipster/6.0.1/libexec/lib/node_modules/generator-jhipster/node_modules/jhipster-core/lib/reader/jdl_reader.js:41:10)
at parseFiles (/usr/local/Cellar/jhipster/6.0.1/libexec/lib/node_modules/generator-jhipster/node_modules/jhipster-core/lib/jdl/jdl_importer.js:82:20)
at JDLImporter.import (/usr/local/Cellar/jhipster/6.0.1/libexec/lib/node_modules/generator-jhipster/node_modules/jhipster-core/lib/jdl/jdl_importer.js:64:30)
at JDLProcessor.importJDL (/usr/local/Cellar/jhipster/6.0.1/libexec/lib/node_modules/generator-jhipster/cli/import-jdl.js:76:41)
at JDLProcessor.importJDL (/usr/local/Cellar/jhipster/6.0.1/libexec/lib/node_modules/generator-jhipster/cli/import-jdl.js:292:38)
at module.exports (/usr/local/Cellar/jhipster/6.0.1/libexec/lib/node_modules/generator-jhipster/cli/import-jdl.js:446:21)
at Command.command.allowUnknownOption.description.action.args (/usr/local/Cellar/jhipster/6.0.1/libexec/lib/node_modules/generator-jhipster/cli/cli.js:72:36)
at Command.listener (/usr/local/Cellar/jhipster/6.0.1/libexec/lib/node_modules/generator-jhipster/node_modules/commander/index.js:315:8)
Motivation for or Use Case

Reproduce the error

Related issues

Suggest a Fix

JHipster Version(s)
jhipster 6.0.1
JHipster configuration
Below is the jhipster info output.  There is no .yo-rc.json as error is seen in the jhipster import-jdl command.
❯ jhipster info
INFO! Using JHipster version installed globally
INFO! Executing jhipster:info
INFO! Options: from-cli: true
Welcome to the JHipster Information Sub-Generator
JHipster Version(s)
/Users/bdealey/git/sts/kubernetes/k8s03-jh601-ms-ist-np-mci
└── (empty)

cat: no such file or directory: .yo-rc.json
JHipster configuration, a .yo-rc.json file generated in the root folder

.yo-rc.json file

JDL for the Entity configuration(s) entityName.json files generated in the .jhipster directory

JDL entity definitions

Environment and Tools
java version "1.8.0_192"
Java(TM) SE Runtime Environment (build 1.8.0_192-b12)
Java HotSpot(TM) 64-Bit Server VM (build 25.192-b12, mixed mode)
git version 2.21.0
node: v10.14.1
npm: 6.9.0
yeoman: 2.0.5
Docker version 18.09.2, build 6247962
docker-compose version 1.23.2, build 1110ad01
INFO! Congratulations, JHipster execution is complete!
Entity configuration(s) entityName.json files generated in the .jhipster directory
JDL file attached.  Added .txt extension to it to allow it to be attached for this form.
Browsers and Operating System
OS: MacOS
There is no browser as this error is generated from the command ine

[X
jhipster-app.jdl.txt

] Checking this box is mandatory (this is just to show you read everything)

Forgot to mention that the JDL directive of "istioRoute" is still documented on the JDL web page: https://www.jhipster.tech/jdl/

I think its removed in new version. May be docs are not updated yet.
IstioRoutes are generated by default now
On Wed, 22 May 2019, 6:38 pm bdealey<EMAIL_ADDRESS>wrote:

Forgot to mention that the JDL directive of "istioRoute" is still
documented on the JDL web page: https://www.jhipster.tech/jdl/
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jhipster/jhipster-core/issues/330?email_source=notifications&email_token=AAIOKF4VBDXXVGX4RN46ACDPWVZJJA5CNFSM4HOWC222YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODV7UONY#issuecomment-494880567,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAIOKF2N5JOJ4D6HPIS4I3DPWVZJJANCNFSM4HOWC22Q
.

This is taken care of. Thanks for reporting this


Several Issues Encountered During Model Training

Thank you for publicly releasing your code! However, I encountered several problems while training the model:

At https://github.com/czh-98/STAR/blob/master/lib/dlmesh.py#L909, it appears that mask and dense face are not on the same device. I resolved this by moving the mask to the GPU.
At https://github.com/czh-98/STAR/blob/master/lib/trainer.py#L691, modifying the retarget_pose attribute in the trainer class does not seem to alter its value. This causes a bug at https://github.com/czh-98/STAR/blob/master/lib/dlmesh.py#L874 because retarget_pose remains None. I'm unsure of the underlying reason, but I fixed this by encapsulating the function that sets retarget_pose within the dlmesh class.
I couldn't locate data/FLAME_masks/FLAME.obj after downloading the FLAME Vertex Masks and FLAME Mediapipe Landmark files as described in the README. Could you provide specific instructions on how to obtain this file?

Thank you!

Hi, thanks for your attention.

For 1, I checked the device, and it does in different devices. Although it still works on my own server, I have fixed it.
For 2, I run the code on my server. The training script does not raise errors and will update the pose, I do not know the reasons...😂
For 1 and 2, I think it might be because you have a multi-GPU server, while I only tested on a single GPU server, so I would suggest running the script with CUDA_VISIBLE_DEVICES=0 python xxx.
For 3, I just uploaded the FLAME.obj file for convenience.

Let me know if you have any other questions :)

@AlvinYH For question 2, I think you may use the torch 2.0+. I have debugged the code, and find out the solution. That's because on the Line 111-113 in train.py, the self. model is compiled by torch and transferred from DLMesh to an Optimized model and cannot read initialized retarget_pose sequence. I delete these three lines and it works for me. : )

@czh-98  Thanks for your reply! As @Jackiemin233 mentioned, I did use torch 2.0+, and deleting lines 111-113 fixed the bug. Thank you both!
However, when using torch 2.0+, I encountered an inplace operation error:
RuntimeError: one of the variables needed for gradient computation has been modified by an in-place operation: [torch.cuda.FloatTensor [25193]], which is output 0 of LinalgVectorNormBackward0, is at version 1; expected version 0 instead.
This error traces back to line 161 in lib/guidance/shape_reg.py during the computation of the Laplacian smoothness loss. I have since downgraded to torch 1.12 and resumed training. But I'm curious about the cause of this bug and if there is a solution other than downgrading the torch version.

@czh-98 Thanks for your reply! As @Jackiemin233 mentioned, I did use torch 2.0+, and deleting lines 111-113 fixed the bug. Thank you both! However, when using torch 2.0+, I encountered an inplace operation error: RuntimeError: one of the variables needed for gradient computation has been modified by an in-place operation: [torch.cuda.FloatTensor [25193]], which is output 0 of LinalgVectorNormBackward0, is at version 1; expected version 0 instead.  This error traces back to line 161 in lib/guidance/shape_reg.py during the computation of the Laplacian smoothness loss. I have since downgraded to torch 1.12 and resumed training. But I'm curious about the cause of this bug and if there is a solution other than downgrading the torch version.

I tried torch 2.0+ and noticed this issue is due to the inplace operation loss[get_flame_vertex_idx()] *= 5. I modified it to avoid such operations as a+=b to a=a+b. Then it should also work for torch 2.0+.


How to invoke a publisher direct method from another module with chunking

This is not a bug, it is just a kindly request to help us to invoke one of the publisher module direct methods with chunking, to overcome the direct methods payload size limitations by using the same Furly libraries you used in the publisher module and other components.
Unfortunately there seems to be no public documentation about the used Furly packages.
I was able to figure out, how to call the publisher module direct methods from cloud (analogous to how this can be done without chunking when using Microsoft.Azure.Devices.ServiceClient library).
If you are interested on that, I could contribute with the sample code for that, just let me know, what is the best way to provide my code.
However I'm struggling to figure out how to invoke a publisher direct method from another module, running either on the same EdgeDevice like publisher or on some child EdgeDevice. We used to use Microsoft.Azure.Devices.Client.ModuleClient method _moduleClient.InvokeMethodAsync(deviceId, moduleId, methodRequest); for doing that as chunking was not required.
Since I was not able to find any code in the Industrial-IoT solution doing the same with the Furly libraries,  maybe you can give me some hint which of the Furly libraries can be used and how to achieve that from another module?
I would also really appreciate if you could point to some class/method in the Industrial-IoT solution where I could figure out, how the implementation of a publisher direct method (i.e. GetConfiguredNodesOnEndpoint) looks like and how the Furly is used to create and return the big response payload in chunks. I expected to find somewhere code like _moduleClient.SetMethodHandlerAsync(methodName, methodHandler, userContext, cancellationToken);, used to register a direct method handler by using Microsoft.Azure.Devices.Client.ModuleClient library, but there is no code like that.
I suppose this is somehow encapsulated in one of the Furly libraries, but I have no clue what and where to look for.

I recommend using the http rest API from another module to work around size limits.

Hi @marcschier. Thanks for supporting us.
What is about security of the http rest API when being invoked from other Edge Devices? Since the endpoint in this case wouldn't be isolated in docker engine network only anymore, we would need to secure it by some kind of authentication and with TLS.
This was actually the main reason for us to keep using Direct Methods, since all these security aspects are already solved there.
Do you have any recommendations about securing the publisher http rest API?
Please note, our questions are not related to the publisher module only. In our solution we also have our own custom module providing i.e. some plant configuration data via Direct Methods to other Edge Devices in the plant. We are also looking for some solution to overcome the Direct Methods size limitations for already existing Direct Methods of our own modules. We need to consider security aspects, . The idea was to solve it in the similar way, like you solved it in Industrial IoT.

There is an API key that must be provided per call.  It and the server certificate can be retrieved from the module twin.  But that will not be possible for module to module so I would propose to add a device method in 2.9.3 to retrieve both and the use those when interacting with the https endpoint.  A feature that I can add in November or early December if that is ok.  Better than chunking which is unreliable and bound to change as it comes through said dependency which I would like to remove eventually.

This sounds like a reasonable solution to me.
Just one more question related to the server certificate: is this going to be a kind of a self signed certificate issued by the publisher module itself?
And regarding the dependency you would like to remove eventually: does it mean, the chunking mechanism used when invoking a publisher direct methods with big payloads from the server (cloud) side is also going to be removed or changed?

The latest preview is available at mcr.microsoft.com/iotedge/opc-publisher:2.9.3-preview2.  There are 2 new API: GetApiKey and GetServerCertificate which can only be invoked over IoT Hub direct methods.  Samples and documentation will follow.


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-2.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 486

Questions ?
Please refer to the Backport tool documentation


ENH Faster method for creating injected instances

~17% faster method for instantiating things via injector
I validated the performance using the following code in my projects PageController
<?php

use SilverStripe\Admin\ModalController;
use SilverStripe\CMS\Controllers\ContentController;

class PageController extends ContentController
{
    protected function init()
    {
        parent::init();

        $time_pre = microtime(true);
        for($i=0;$i<100000;$i++) {
            //$this->injectOld(); // This took 0.2139 seconds
            $this->injectNew(); // This took 0.1776 seconds
        }
        $time_post = microtime(true);
        $exec_time = $time_post - $time_pre;
        echo sprintf("This took %.4f seconds", $exec_time);

    }

    private function injectOld()
    {
        $class = ModalController::class;
        $params = ['MyController', 'MyName'];
        $reflector = new ReflectionClass($class);
        return $reflector->newInstanceArgs($params);
    }

    private function injectNew()
    {
        $class = ModalController::class;
        $params = ['MyController', 'MyName'];
        return new $class(...array_values($params));
    }
}

@kinglozzer are you happy to re-approve/merge post small change (it's still functionally the same)


Update Helm release metallb to v0.1.30

This PR contains the following updates:

Package
Update
Change

metallb
patch
0.1.23 -> 0.1.30

Configuration
📅 Schedule: At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, click this checkbox.

This PR has been generated by Renovate Bot.

Close in favor of #17

Renovate Ignore Notification
As this PR has been closed unmerged, Renovate will now ignore this update (0.1.30). You will still receive a PR once a newer version is released, so if you wish to permanently ignore this dependency, please add it to the ignoreDeps array of your renovate config.
If this PR was closed by mistake or you changed your mind, you can simply rename this PR and you will soon get a fresh replacement PR opened.


Account

it says my account Has been checkpoint

yep, now you need to confirm your identity, if you are lucky enough its just that, perhaps this guy just stool your credentials.

Username And Passsword??


Spoken language change support

Hi,
I like to use your extension. But, it is annoying to seeing the repositories that I am not able to understand. So, here is my little help to you for adding spoke language query to requests (with tests). Hope to see this in market soon.

wow, this is awesome, thanks for the contribution! there are some minor UX issues I need to think about: It was "All Languages" and now it is "Programing Languages", I will think and get back to it soon!

@huchenme
Actually, I decided to make this change in my last commit when I implement the popular spoken languages.
Firstly, the new select for spoken language default value 'No Preference' like GitHub. But, it hasn't any clue about the aim for this select. So, it feels like there is UX problem.
One option might be the adding labels for both selects (Spoken Language & Programming Language).
Instead of, I changed new select default value ('ALL') label to 'Spoken Language'.
Now, consider there is two select, one's default label 'Spoken Language' and other one 'All Language'. It is also looks like UX problem.
So, this was my opinion.

Yeah I thought about all options and actually wanted to address it in a setting modal in V2, but since you have already implemented, I will consider again to ship this feature first
On 3 May 2020, 10:12 PM +0800, Berkan Sivri<EMAIL_ADDRESS>wrote:

@huchenme
Actually, I decided to make this change in my last commit when I implement the popular spoken languages.
Firstly, the new select for spoken language default value 'No Preference' like GitHub. But, it hasn't any clue about the aim for this select. So, it feels like there is UX problem.
One option might be the adding labels for both selects (Spoken Language & Programming Language).
Instead of, I changed new select default value ('ALL') label to 'Spoken Language'.
Now, consider there is two select, one's default label 'Spoken Language' and other one 'All Language'. It is also looks like UX problem.
So, this was my opinion.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

@berkansivri Thank you very much for the PR, 1.9.0 should be available in a couple of hours


toolbox(cosmos): registerThorname => registerMayaname

There is currently a registerThorname() function.
Can you also create a registerMayaname() function?

I could take this in the next days 😉  we are also building an UI for it.

I could take this in the next days 😉 we are also building an UI for it.

Did you manage to look into this yet? 🙂

Didnt have time yet 🙈

I'm also short with time, I could in a couple of weeks perhaps.

Starting now 😉


[BUG] Deeplinking on mobile wallets fails for signing transactions using wagmi

Describe the bug
After connecting a wallet with connectkit on a mobile device, if you attempt to sign either a transaction or message using the wagmi hooks, then the wallet app does not open and you stay in the browser. You also don't get notifications to do anything so it looks like your app just hangs, but it's waiting for the signing to happen.
To reproduce
https://codesandbox.io/s/condescending-euler-sluziu
This also happens when using a custom wagmi client config instead of getDefaultClient.
Edit: An example in production can be found at https://drops.samking.studio, and the repo can be found here.
Expected behavior
When signing either a transaction or message using the wagmi hooks, the connected wallet app should open e.g. MetaMask mobile, or Rainbow etc. with the prompt to sign the transaction/message.
Screenshots
N/A
Environment details
iOS v16.1 and Safari
<EMAIL_ADDRESS><EMAIL_ADDRESS>Additional context
If I just replace connectkit with @rainbow-me/rainbowkit then the wagmi hooks work perfectly on mobile.

Loving the new SIWE update, but this issue is still present on mobile wallets, even using the demo on the docs site. I’ve done a couple of implementations even on the latest wagmi and the issue still persists. Switching over to rainbow avoids the problem entirely, so I’m not sure if it’s something to do with the connectors setup in wagmi. I’d really prefer to use connectkit since I can get it to fit within my apps visual style a lot better. Any updates on what might be causing the issue?

Hey @samkingco, this is something we're looking into. I was hoping to get this fix ready for the latest update but couldn't quite figure out exactly why this issue is occuring.
So far I've discovered that if you create your own wagmi client instead of using our getDefaultClient helper it seems to resolve itself – but from what we can tell the returned value is perfectly fine.
We will still be investigating this further and will let you know when we find a fix.

Hi, @lochie I too am having trouble with this issue.
Can you please let me know your current situation?

Just an update, we've spent quite some time looking into this and still have yet to debug this one. We'll continue investigating as this is an important feature we need to get working.

Any update yet? @lochie

Is it possible to move createMessage out of the signIn function here?
Lots of async activity before the actual signMessageAsync function can cause iOS deeplink to fail.
https://wagmi.sh/react/prepare-hooks#ios-app-link-constraints
If you can change it to pass a message to the useSignMessage hook here then that would reduce time from the user clicking the sign button to getting a sign prompt.

confirmed, deeplinking issue occurs on ios safari, even on initial walletconnect redirect. obviously hunting down this bug has occupied probably hundreds of collective developer-hours, but this behavior makes using connectkit for an app impossible at the moment.
based on what i have learned, links like this should be a true <a> tag with target="_blank" so they don't redirect from the page itself, and in the worst case just open a new tab that can be closed. doing a redirect in a div's click handler is what i believe is causing the behavior here.
separately, as @bcxbb mentioned, when proposing a tx or signature via walletconnect, no other async actions should be done in the button click handler, or a similar behavior will occur, in my experience
https://user-images.githubusercontent.com/1535001/222880024-7081a5e4-e516-4e90-bd4c-40e115848e54.mp4

@shrugs i think what you're seeing here is different issue than this thread is talking about. The rnbwappdotcom.app.link page you're seeing is a fallback design of Rainbow when deeplinking doesn't work within their iOS redirect page for walletconnect, whilst what some people here are seeing is no deeplinking on signing requests after connecting to a wallet on iOS.
Your suggestion to switch from a window.location.href redirect to an anchor with _blank might help in some cases, but will require the user to close that tab once deeplinking to the respective native app has occurred.
We've been looking the txn deeplinking issue on and off and haven't found the root of this issue yet, we're not sure if this is an issue on wagmi's end or connectkit's end, and since replicating seems to only be possible on a per-project level debugging is a bit complicated.

hey y'all, any update here? we are seeing the original issue -- deeplink occurs during initial connect but does not happen on sign in. happy to help if needed to get this complete. cc @lochie

We're also seeing this issue on ConnectKit 1.4 and wagmi 1.0. Works with w3Modal.

I'm unable to reproduce this now  so I believe this has been resolved with the WalletConnect 2.0 migration. If anyone can confirm this as resolved on their end with the latest version of ConnectKit I'd be keen to close this issue.

Closing this issue as I believe it's resolved.


Logging: Log in pre-armed mode at very low rate (e.g. 5 Hz)

The goal is to have access to logs for user support. We might want to opt to be smart about auto-deleting these logs and always just keeping the latest one around.

Is this related to #4083? If so maybe just keeping the latest one would not do the job. Maybe it should be the last that did not take off.

In general a simple logrotate would be nice.

@bkueng @priseborough I think we should figure out how to do this.

Hey, this issue has been closed because the label status/STALE is set and there were no updates for 30 days. Feel free to reopen this issue if you deem it appropriate.
(This is an automated comment from GitMate.io.)


Merge main into qa-beta

Description

Checklist (Definition of Done)

[ ] Unit and integration tests added
[ ] Added test description under Test manual
[ ] Documentation added if necessary
[ ] CI and all relevant tests are passing
[ ] Add the ticket number to the PR title if available, i.e. ROX-12345: ...
[ ] Discussed security and business related topics privately. Will move any security and business related topics that arise to private communication channel.

Test manual
TODO: Add manual testing efforts
# To run tests locally run:
npm run test

Codecov Report
Patch and project coverage have no change.

Comparison is base (9753129) 8.90% compared to head (33ebab0) 8.90%.

:exclamation: Current head 33ebab0 differs from pull request most recent head d671e2b. Consider uploading reports for the commit d671e2b to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##           qa-beta    #113   +/-   ##
=======================================
  Coverage     8.90%   8.90%
=======================================
  Files           37      37
  Lines          550     550
  Branches       112     112
=======================================
  Hits            49      49
  Misses         465     465
  Partials        36      36

Impacted Files
Coverage Δ

src/components/InstanceDetailsList.js
0.00% <0.00%> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Fixed bug originated due to FallbackPolicy max_history being None

Proposed changes:

Fixes #594

Status (please check what you already did):

[x] made PR ready for code review
[ ] added some tests for the functionality
[ ] updated the documentation
[ ] updated the changelog

Could you please try this PR: https://github.com/RasaHQ/rasa_core/pull/591, and check if Fallback policy works as expected?

#591 fixes the problem in agent.py but one error still remains due to the following line in ensemble.py (line 101) class PolicyEnsemble method _max_histories:
if isinstance(p.featurizer, MaxHistoryTrackerFeaturizer):
Since the FallbackPolicy is missing the featurizer.

Thank you for noticing this!

Closing this since this issue is being addressed in #591


Support deployment to Kubernetes

Please support deployment to Kubernetes.

Good morning @thanhtoan1196 and thanks for this idea!
I understand each word of this sentence but have no clue what it means.
do you want deployer to build docker images and communicate with kubernetes cluster?

There is a plenty of ways it can be achieved. What in the scope?

do you want deployer to build docker images and communicate with kubernetes cluster?

@Schrank That's right, for example it builds image from Laravel app and deploys to k8s

@antonmedv I have ever used https://devspace.sh/cli/docs/quickstart to build image.


mito: allow use of Basic Authentication and OAuth2 in command

This allows use of the configuration options available in the Filebeat input to improve parity for development playground purposes. The implementation is simpler here and does not make use of the go-ucfg package, so there are some differences in config behaviour. Only limited validation of the config is performed.
Configuration is in the YAML exposed via the cfg option. The options available are similarly interpreted as the config in Filebeat with the options corresponding to the following structs:
type config struct {
	Globals map[string]interface{} `yaml:"globals"`
	Regexps map[string]string      `yaml:"regexp"`
	Auth    *authConfig            `yaml:"auth"`
}

type authConfig struct {
	Basic  *lib.BasicAuth `yaml:"basic"`
	OAuth2 *oAuth2        `yaml:"oauth2"`
}

type oAuth2 struct {
	Provider string `yaml:"provider"`

	ClientID       string     `yaml:"client.id"`
	ClientSecret   *string    `yaml:"client.secret"`
	EndpointParams url.Values `yaml:"endpoint_params"`
	Password       string     `yaml:"password"`
	Scopes         []string   `yaml:"scopes"`
	TokenURL       string     `yaml:"token_url"`
	User           string     `yaml:"user"`

	GoogleCredentialsFile  string `yaml:"google.credentials_file"`
	GoogleCredentialsJSON  string `yaml:"google.credentials_json"`
	GoogleJWTFile          string `yaml:"google.jwt_file"`
	GoogleJWTJSON          string `yaml:"google.jwt_json"`
	GoogleDelegatedAccount string `yaml:"google.delegated_account"`

	AzureTenantID string `yaml:"azure.tenant_id"`
	AzureResource string `yaml:"azure.resource"`
}

Please take a look.

That was a little tricky, but done. The OAuth2 case could also be tested, but will require significant additional complexity to be added to the test supports.


Running Tailscale inside K3s Docker Container (curl not available)

I was able to get the instructions at https://docs.k3s.io/installation/network-options?_highlight=tailscale#integration-with-the-tailscale-vpn-provider-experimental to work on a regular node. I'm now trying to achieve something similar with the rancher/k3s Docker image, with no luck so far.
The main issue is that curl is not available, and I don't see a way of installing it. I'm not even able to get a shell in the container it seems. Would appreciate any help! I'm using rancher/k3s:v1.27.7-k3s2.
For example:
$ k3s % docker run -it --rm rancher/k3s:v1.27.7-k3s2 curl --version

Unable to find image 'rancher/k3s:v1.27.7-k3s2' locally
v1.27.7-k3s2: Pulling from rancher/k3s
a1cd439dfa97: Already exists
03962ff28f66: Already exists
Digest: sha256:56c2a6885db2c69e002142aea3b80cc54b40459e4685500525aa729e16693fe7
Status: Downloaded newer image for rancher/k3s:v1.27.7-k3s2
No help topic for 'curl'

Same thing happens with sh.

The k3s image doesn't contain curl, and the default entry point runs /bin/k3s so you're essentially running /bin/k3s curl which is not a valid command. Why are you trying to run k3s as if it were curl?

Yeah that's what I figured was happening. The reason I wanted to use curl to install Tailscale into the docker image was because the provider I'm using (vast.ai) only supports running docker images, it doesn't give you traditional "bare metal" access. But I suppose I could use a different base image to install K3s and the Tailscale on the host, and proceed from there.
Thanks for the quick reply!


Added Pragmatic Ethics slides. Note that this is the current release …

…of this deck, and may not be exactly what was presented; sorry about that.

No worries. Thanks!
Regards,
Phil
<EMAIL_ADDRESS>Philip Japikse | Microsoft MVP, ASPInsider, MCSD, MCDBA, PSM II, PSD, CSM
Consultant, Coach, Author, Trainer | Founder, Cincy Deliverhttp://www.cincydeliver.org/ Conference
japikse<EMAIL_ADDRESS>| c:<PHONE_NUMBER> | twitter: @skimedichttp://www.twitter.com/skimedic | skype: phil.japikse
www.skimedic.com/bloghttp://www.skimedic.com/blog | Microsoft MVPhttp://bit.ly/japikse-mvp | Session Abstractshttp://tinyurl.com/skimedic-abstracts | Lynda Courseshttps://www.lynda.com/Phil-Japikse/7908546-1.html
From: Bill Horvath II<EMAIL_ADDRESS>Sent: Wednesday, August 28, 2019 04:08 PM
To: skimedic/cincydeliverpresentations<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [skimedic/cincydeliverpresentations] Added Pragmatic Ethics slides. Note that this is the current release … (#27)
…of this deck, and may not be exactly what was presented; sorry about that.

You can view, comment on, or merge this pull request online at:
https://github.com/skimedic/cincydeliverpresentations/pull/27
Commit Summary

Added Pragmatic Ethics slides. Note that this is the current release of this deck, and may not be exactly what was presented; sorry about that.

File Changes

A 2019/Pragmatic Ethics for Software Professionals.pdfhttps://github.com/skimedic/cincydeliverpresentations/pull/27/files#diff-0 (0)

Patch Links:

https://github.com/skimedic/cincydeliverpresentations/pull/27.patch
https://github.com/skimedic/cincydeliverpresentations/pull/27.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/skimedic/cincydeliverpresentations/pull/27?email_source=notifications&email_token=AAB54HMHNWH4CDCG7FRLS5LQG3LKHA5CNFSM4IRR54CKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HIAVWWA, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAB54HKPUBNNQUFYOCUREJ3QG3LKHANCNFSM4IRR54CA.


Add support for HP-UX IA64N

These commits make it flawlessly compile on HP-UX for me. All tests pass.

Do no yet merge, I am examing the visibility behavior.

It turns out that the jni_md.h does not properly handle JNIEXPORT also __attribute__ ((visibility("default"|"protected"|"hidden"))) is available and works. I will inquire with HPE support. The bundled jni_md.h will do for now.

OK, for those who will stumble upon this. The issue has been confirmed by the HPE HP-UX JVM team and is slated to be fixed (jni_md.h) in either March for June update of the JDK for HP-UX.

The current PR works perfectly on HP-UX and I am able to add 2 000 000 rows in less than 90 s into the database.

@xerial Would you be so kind and have a look and merge?

ok. Merged


Full ReScan if not complete then hide

If this is not completed, we need to hide this. Also the text is wrong. Check the text I have suggested in the issue.

Closing this issue as it is present in v1.6.5(281)


Add a new heading for general knowledge and design practices

We want to move some things out of Javascript/Node and into a more general Programming or Design headers. A big example of this is Big O complexity which is a general computer science concept that doesn't belong to JS and nesting it under JS only makes it confusing.
We should however note in those questions as we move them what language the examples will be in and any caveats thereof.
Otherwise we're going to get dinged pretty hard for 'Why is this Javascript when it applies to X langauge too" or "Why do you have duplicate questions for different languages"(should we support them in an ever changing landscape)
I'd suggest a general Programming and Design categories to host such questions

For now, I just want to keep this project JavaScript / HTML / CSS and something tightly connected to those I mentioned. However, I do agree with you. We definitely want to separate those that are not so much JS related. We will add them to general category, but not yet. Once we come up with more questions and answers for that category we will add it. I don't want a website that has 1 question in each new category 😕

Fair enough, can't argue with that. When enough are in the general category that can be branched off then I guess we can do it then

Well, I am closing this until we collect way more questions so we can branch them. I will hopefully reopen this in a month and then we can work on it.


Refactor GatewayCleanup function

The "GatewayCleanup" function restores the state of the GW node on the cluster if required.
The test should call the "SaveGatewayNodeForTest" function in order to save the state of the GW node into the Framework.
At the end of the test, the "GatewayCleanup" function will restore the state of the GW node if defined within the Framework state.
Signed-off-by: Maxim Babushkin<EMAIL_ADDRESS>

Hi @skitt @sridhargaddam
I'll appreciate if you could take a look at this PR.
It already has 1 approval and merging it will unblock me for the next patches.
Thank you!

For future reference, your commit and PR titles should give the purpose of the change. In this case, the first line of your commit message would make a good title; something like “Restore gateway node state after tests”.


[entropy_src, top] Add option to stub out entropy_src

This PR introduces a method to stub out entropy_src.
Unlike otbn stubbing, this does not render the entire entropy function unusable.
The entropy_src_core function is optimized away, and an lfsr is used to replace the outputs.
I've tried to hook-up the sideband signals such that dif_entropy_smoketest works.  This allows us to "fake" the activity.
Overall, this is not a super clean solution.. but we can probably make it a bit better by encapsulating the stub logic into its own module. Let me know what you guys think.
Just as a reminder, this was done to give us some more breathing room on the FPGA.

let me know if others have any thoughts. If not we can work on merging this in.
@vogelpi


Improvements (#1705)

Hey @0xRomain, there are currently problems with your Provider, with this PR but even before (so what we currently have on https://factory.sismo.io/). Only the first request works.
And are you sure to only fetch the first 1000 data (an not iterate using the skip parameters?)
Maybe I can close this PR and let you can create a new one to fix this? (let me know if you are okay with this)

Hi @MartinGbz
Sorry I don't know why PR was created, we are still working on that on our fork, we will do a PR when everything worked

Hi @MartinGbz Sorry I don't know why PR was created, we are still working on that on our fork, we will do a PR when everything worked

Okay no problems, I'm closing this one ;)


Add property grid widget via extension

Update ui-test-extension to provide a basic property widget.
Update map-layers so when map-layers is used as an extension default props are passed to the widget.

Just adding myself, to keep track of the ui-test app changes. thx again for adding that sample Bill!

Just adding myself, to keep track of the ui-test app changes. thx again for adding that sample Bill!


Create a puppet module

As the title says, we need a puppet module.

Some initial work has been done here: https://github.com/tobru/puppet-webhookd/

The Puppet module exists, it does its job, so that's it for this task.


Move async into experimental and merge it

Task
We need all async functionality to be in the experimental.* namespace before merge
Motivation
Not sure if this belongs in an "issue". But since the feature is spread out over so many PRs and those PRs seem to come and go, I think we need some permanent home that documents and tracks the state of things. This issue will be that home, unless there is a better place.
GitHub Project
@pwinston's Napari Rendering GitHub Project has links to all the PRs.
Merged
These PRs were already merged:

async-1 - Stub ChunkLoader
async-2 - ChunkLoader thread pool
async-2.5 - Vispy changes

Next Steps
The next PR async-3 adds the ChunkCache, but we want to pause now and get everything into experimental namespaces, including the vendored dependencies. This might result in changes to async-3 through async-12 or maybe some new PRs as well.
The goal is that everything async specific is in experimental namespaces so that it's clear it is opt-in non-final code. Anything not in experimental must be final and reviewed like any other code.

This was closed by #1659 async-2.6 which moved the async code to experimental


Autoplay button?

So I'm using the developer version and I'm confused. How do I get the "Up Next" button switch to always be on? the documentation isn't very clear about how I should make it always enabled.

Its can be On or Off just as normal as you don't have YTC. However, YTC broke the UI for it: so it always looks like "off". Check #1677

@fireattack Okay, thanks for clarifying this issue. No point keeping this open, I'll wait for an update.


ceres-solver: fix darwin build

The Basel BUILD file conflicts with the cmake build directory on
case-insensitive filesystems, eg. darwin.
/cc ZHF #45961
Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

Success on x86_64-darwin (full log)
Attempted: ceres-solver
Partial log (click to expand)
/nix/store/d0n1wq2m7ccaca6zism9w7asp5mdvi1y-ceres-solver-1.14.0

Failure on aarch64-linux (full log)
Attempted: ceres-solver
Partial log (click to expand)
-- Found Eigen version 3.2.10: /nix/store/ark1wnkz68pllpxgmxrkmr7azzwj0947-eigen-3.2.10/include/eigen3
CMake Error at CMakeLists.txt:225 (message):
  -- Ceres requires Eigen version >= 3.3.4 on aarch64.  Detected version of
  Eigen is: 3.2.10.

-- Configuring incomplete, errors occurred!
See also "/build/ceres-solver-1.14.0/build/CMakeFiles/CMakeOutput.log".
builder for '/nix/store/99i998dz0hpsjpzxbrb9k7w3k07qyaxb-ceres-solver-1.14.0.drv' failed with exit code 1
error: build of '/nix/store/99i998dz0hpsjpzxbrb9k7w3k07qyaxb-ceres-solver-1.14.0.drv' failed

Success on x86_64-linux (full log)
Attempted: ceres-solver
Partial log (click to expand)
-- Installing: /nix/store/9j7bv5r05037lvwxxhx0xgr4fhm16lwy-ceres-solver-1.14.0/lib64/cmake/Ceres/FindGflags.cmake
-- Installing: /nix/store/9j7bv5r05037lvwxxhx0xgr4fhm16lwy-ceres-solver-1.14.0/lib64/libceres.a
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/9j7bv5r05037lvwxxhx0xgr4fhm16lwy-ceres-solver-1.14.0
strip is /nix/store/h0lbngpv6ln56hjj59i6l77vxq25flbz-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/9j7bv5r05037lvwxxhx0xgr4fhm16lwy-ceres-solver-1.14.0/lib64
patching script interpreter paths in /nix/store/9j7bv5r05037lvwxxhx0xgr4fhm16lwy-ceres-solver-1.14.0
checking for references to /build in /nix/store/9j7bv5r05037lvwxxhx0xgr4fhm16lwy-ceres-solver-1.14.0...
moving /nix/store/9j7bv5r05037lvwxxhx0xgr4fhm16lwy-ceres-solver-1.14.0/lib64/* to /nix/store/9j7bv5r05037lvwxxhx0xgr4fhm16lwy-ceres-solver-1.14.0/lib
/nix/store/9j7bv5r05037lvwxxhx0xgr4fhm16lwy-ceres-solver-1.14.0

aarch64 failure is unrelated

backported: 0b3e0cdab2f


Pre emulator launch script

Using this PR only for testing. Correct PR is here https://github.com/ReactiveCircus/android-emulator-runner/pull/247


How to deal with ATK & AT-SPI merger?

Several accessibility-related projects have been merged into a single repository.

https://gitlab.gnome.org/GNOME/at-spi2-core/-/merge_requests/78
https://viruta.org/accessibility-repos-are-now-merged.html
https://discourse.gnome.org/t/atk-and-at-spi2-atk-merged-into-at-spi2-core/10032
https://gitlab.gnome.org/GNOME/gnome-build-meta/-/merge_requests/1617

We need to decide how to deal with that before GNOME 43 is released in autumn:

Create a single package for all the projects and make the old ones aliases.
Work with upstream to make the projects individually buildable and installable.
Patch stuff locally.

cc @NixOS/gnome @NixOS/freedesktop

In https://github.com/NixOS/nixpkgs/pull/182618, I have decided to go with the first option, keeping the aliases in all-packages to allow for potentially splitting the package in the future.

Several accessibility-related projects have been merged into a single repository.

This is really unfortunate and seems rushed.
Their goal is admirable:

Remove all the hand-written DBus code in at-spi2-core; replace with gdbus as generated from the XML files.

... but until they finish doing that, they're forcing everything downstream of gtk3 (and even gtk2!) to use that scary code that needs to be removed.  ATK is a hard, non-disableable dependency of gtk2 and later, and now so is at-spi2-core.
That really seems like a step backwards.  Personally I'll be pinning ATK to 2.44.1 until they release a version that takes the compensating two steps forward.

Apparently, ATK can now be built separately so we might want to split it out again: https://github.com/NixOS/nixpkgs/pull/236817


Use as part of linting

I'd like to use this as part of npm run lint command, (which does the usual css, js, hbs)
For it to be useful, ember-intl-analyzer would need to return an error rather than a warning.
Perhaps a flag would help for this?

My bad, seems to already do that


In we still use the 'REGISTRY_URL' variable

In https://github.com/QCDIS/NaaVRE/blob/main/docker/repo_utils/conf_vl_repos.py#L85 we use REGISTRY_URL

In https://github.com/QCDIS/NaaVRE/blob/main/.github/workflows/make.yml#L39 we also use 'REGISTRY_URL'.
The variable is not present in Action variables

https://github.com/search?q=repo%3AQCDIS%2FNaaVRE REGISTRY_URL&type=code

REGISTRY_URL is still supported, and should be set when the registry does not match the git repository.

In conf_vl_repos.py, we handle both cases, where REGISTRY_URL is set and not set.
The CI tests also work but with a caveat: when REGISTRY_URL is not set in the repo, make.yml calls test-build wheel.yml with REGISTRY_URL="". This means that test_evn_vars.py passes, because it uses assertIsNotNone. However, conf_vl_repos.py goes into the right if branch because it uses if REGISTRY_URL. The REGISTRY_URL test should be removed.

The following files can be updated to provide defaults without REGISTRY_URL:

https://github.com/QCDIS/NaaVRE/blob/8de4a48829cf0807fdcd1c0f89230afab358cc67/docker/export_VARS#L10
https://github.com/QCDIS/NaaVRE/blob/8de4a48829cf0807fdcd1c0f89230afab358cc67/docker/docker_VARS#L10


Unix C library compatibility

Changes to enable wrapping of the system C library on mac OS and Linux.
See commit logs for detail.
Currently I am wrapping: <errno.h>, <string.h>, <limits.h>, <stdlib.h>, <pthread.h>, <sys/ioctl.h>, <termios.h>, <fcntl.h>, <poll.h>, <sys/epoll.h> <unistd.h>, <sys/stat.h>, <net/if.h>, <sys/socket.h>, <signal.h>, <sys/wait.h>, <sys/syscall.h> and <spawn.h>.
... and i'm using these options:
   Dict{String,Any}(
            "general" => Dict{String,Any}(
                "is_local_header_only" => false,
                "auto_mutability" => true,
                "auto_mutability_includelist" => ["termios"],
                "library_name" => ""
            ),
            "codegen" => Dict{String,Any}(
                "use_ccall_macro" => true,
                "wrap_variadic_function" => false
            ),
            "codegen.macro" => Dict{String,Any}(
                "macro_mode" => "aggressive",
                "ignore_pure_definition" => true
            )
        )

See https://github.com/notinaboat/UnixIO.jl/blob/c03f1695999666ab95177b5e1713e2fe1cd1eae0/packages/UnixIOHeaders/src/UnixIOHeaders.jl#L162-L188

Is there any reason why you choose not to use Ygg system headers?

Is there any reason why you choose not to use Ygg system headers?

Those headers appear to be out of date.
For example, I use the POSIX spawn interface here: https://github.com/notinaboat/UnixIO.jl/blob/julia18/src/UnixIO.jl#L1282-L1284
But, the POSIX_SPAWN_SETSID symbol is not defined in the Ygg headers...
julia> using Clang

julia> dirs =Clang.JLLEnvs.get_system_dirs("x86_64-apple-darwin14")
5-element Vector{String}:
 "/Users/samoconnor/.julia/artifa" ⋯ 57 bytes ⋯ "64-apple-darwin14/4.8.5/include"
 "/Users/samoconnor/.julia/artifa" ⋯ 63 bytes ⋯ "le-darwin14/4.8.5/include-fixed"
 "/Users/samoconnor/.julia/artifa" ⋯ 43 bytes ⋯ "9/x86_64-apple-darwin14/include"
 "/Users/samoconnor/.julia/artifa" ⋯ 56 bytes ⋯ "e-darwin14/sys-root/usr/include"
 "/Users/samoconnor/.julia/artifa" ⋯ 70 bytes ⋯ "-root/System/Library/Frameworks"

julia> dirs[1]
"/Users/samoconnor/.julia/artifacts/303e65f6896f295459c81aa311b5eb993f702d89/lib/gcc/x86_64-apple-darwin14/4.8.5/include"

e.g. the file spawn.h is found...
% find /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/ -name spawn.h
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk//usr/include/spawn.h
...

But the symbol POSIX_SPAWN_SETSID is not found...
% find /Users/samoconnor/.julia/artifacts/303e65f6896f295459c81aa311b5eb993f702d89 -name spawn.h | xargs grep POSIX_SPAWN_SETSID
%

Compare to searching under Applications/Xcode.app ...
% find /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/ -name spawn.h | xargs grep POSIX_SPAWN_SETSID
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk//usr/include/sys/spawn.h:#define POSIX_SPAWN_SETSID              0x0400
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk//System/Library/Frameworks/Kernel.framework/Versions/A/Headers/sys/spawn.h:#define POSIX_SPAWN_SETSID              0x0400

... also, I'm a little uneasy about using platform headers from https://github.com/phracker/MacOSX-SDKs I would rather use official headers form the platform vendor.

Those headers appear to be out of date.
For example, I use the POSIX spawn interface here: https://github.com/notinaboat/UnixIO.jl/blob/julia18/src/UnixIO.jl#L1282-L1284
But, the POSIX_SPAWN_SETSID symbol is not defined in the Ygg headers...

The default gcc version used in Clang.jl is v<IP_ADDRESS>, which can be updated to a newer one. But in your use case, I think it's better to just use headers on your local system.

Background FYI:
My motivation for creating UnixIO.jl came from working on an embedded system (R'Pi-linux) and having trouble with unreliable serial port IO. I could write a tiny C program to open /dev/ttyS0 and read/write with 100% reliability, but from Julia it was unreliable. I started trying to find the root cause, but that was complicated by all the intermediate layers in Base.IO and libuv. It seems that the libuv project simply does really intend to support serial ports: libuv/libuv#1970
So, the intention of UnixIO.jl is:

I just want to talk directly to the native OS C library on my machine.
I want to avoid any risk that might be introduced by 3rd party intermediate layers.

Make sense to me now. Thanks for the PR. :)


python312Packages.mypy: 1.10.1 -> 1.11.2

Description of changes
Updates mypy. Targeting changing due to ~6.5k rebuilds.

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/)
24.11 Release Notes (or backporting 23.11 and 24.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.
Pinging @vcunat for review.

Add a :+1: reaction to pull requests you find important.

@dotlambda Done, and updated to the recently released 1.11.2

@dotlambda Sorry about the extra ping but could you merge this?
Thanks!

Done!


Display current server time in /modlog

http://i.imgur.com/X0KehCO.png
This gives a point of reference, to know how the modlog's timestamps
translate to your own timezone.

May not be a good idea to pullreq two things that different in the same pullreq. Can you split these out?
Also have you done the license in the past?

I'll rebase this pr to only contain the modlog change, and open another one for /whoare.
Here's my license post (this was really hard to dig up, neither github nor google could find it even when I remembered the exact phrasing): https://github.com/Zarel/Pokemon-Showdown/pull/2396#issuecomment-175923508
(I'm Orda-Y, btw.)

You haven't removed the /whoare change from this PR yet.

Looks like I had synched again by accident. Should be good, now.

Looks weird. What do you think about displaying the current time on the top-right corner instead?

Here's what Slayer95's proposal would look like: http://i.imgur.com/adDj2Fu.png

Doesn't look bad I guess...

maybe add brackets and omit "Current server time:" - maybe just "now:" inline

feelscool

Going to merge this, if anyone has a problem with it they can pullreq/commit a redesign


Fallout V&D - New Burst Buttons

Submission Checklist
Required

[ ] The pull request title clearly contains the name of the sheet I am editing.
[ ] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[ ] The pull request makes changes to files in only one sub-folder.
[ ] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

New Sheet Details

The name of this game is: <   >  (i.e. Dungeons & Dragons 5th Edition, The Dresden Files RPG)

The publisher of this game is: <   > (i.e. Wizards of the Coast, Evil Hat)

The name of this game system/family is: <   > (i.e. Dungeons & Dragons, FATE)

[ ] I have followed the Character Sheets Standards when building this sheet.

[ ] I have authorization from the game's publisher to make this an official sheet on Roll20 with their name attached.
[ ] This game is not a traditionally published game, but a copy of the game rules can be purchased/downloaded/found at: <   >
[ ] This sheet is for an unofficial fan game, modification to an existing game, or a homebrew system.

Changes / Description
New Burst Buttons have been implemented. Damage has been split between Dice and Flat.

Character Sheet Info Roll20 Internal Use only.

Hi @Skyeris!
Unfortunately, we'll have to reject this PR, as you've made some changes to the attributes that will be destructive to the end user's data. To fix this, you'll need to write sheet workers to transition the data gracefully from the old attribute to the new attributes. Other than that, this PR is fine, so feel free to make a new one once you've done that work.

Hello @NBrooks-Roll20
The only change that was made that would remove user data is the weapon_damage-1 and weapon_damage-2 to divide in to dice and flat damage. Our community is fine with this change as we've told them ahead of time before implementing this.
Is their another attr that was missed or is that the only one?

Okay, in that case, we'll merge this, but for future reference if you're making a destructive change like this but have a good reason for it, a link to that notification or telling us in advance would speed up this process. Thanks! :)

@NBrooks-Roll20 I will do that for the future. Thank you. We have a problem and I don't know who else to turn to about it. The sheets appearance was altered in a strange way. I posted it in issues. Nothing in the appearance was altered for this update and I have no idea what to do.


v-for key with jQuery-managed elements

I think in some cases, we cannot reuse a jQuery-managed element, but unless we use the right v-for key, Vue may try to do so.
As part of this, check the approach for the field key list.

I made changes to FieldKeyList as part of #386. The addition of Popover should also make it easier to manage related jQuery wrappers. I don't see anywhere else where we use a jQuery wrapper within a v-for, so I'll go ahead and close this issue.


Implement a manual override of the %(release) variable

Will be useful for cbeta/EA users to allow to set to the last known chum
releale.
Fixes #40

@piggz: I would have been bit more verbose with the help message regarding the option. In addition, it would be helpful to display status in more specific manner. Please see 26ca6e13c31053ec7e4e5a3201d718a56745db19 for my suggestions. Feel free to drop them, copy-and-paste or cherry-pick them.
After making your decision, it is fine for merging as all is working.


support for FreeBSD

Describe the feature
Reth currently does not compile on FreeBSD (13.2). The rust toolchain I used was 1.70.0. Cause of compilation error (at least the first one that pops up when cargo build) is that the metrics-process library does not implement metric collector for FreeBSD. I was wondering if the metrics subsystem can be disabled at compilation time (and the dependency disabled), so that at least the core functionalities of Reth can run on FreeBSD.
[updates below]
It seems that adding support for FreeBSD to metrics-process won't be too hard, and I'm more than happy to take a shot, so that reth can bump the version of metrics-process and hopefully support FreeBSD. But is there any other dependency that currently relies on Linux/Windows/macOS?
(submitting the ticket because I do not find any other discussion re. freebsd in the issues list; please let me know if it is not the correct place to discuss this issue)
Additional context
No response

Okay I was able to compile Reth on FreeBSD with the following tweaks. I'm not sure if any of them is unnecessary, and will do a bisection to figure out. But for now here's what I did:

Silent the compilation error of metrics-process with the following patch. It's not really a fix in the sense that the library won't provide metrics on FreeBSD (it's just an empty implementation), but at least it compiles. It does not seem hard to add real support for FreeBSD either.

diff --git a/src/collector.rs b/src/collector.rs
index 3b47870..3fd11d2 100644
--- a/src/collector.rs
+++ b/src/collector.rs
@@ -8,6 +8,9 @@ mod linux;
 #[path = "collector/windows.rs"]
 mod win;

+#[cfg(target_os = "freebsd")]
+mod freebsd;
+
 /// Process metrics
 /// https://prometheus.io/docs/instrumenting/writing_clientlibs/#process-metrics
 #[derive(Debug, Default, PartialEq)]
@@ -41,6 +44,9 @@ pub use linux::collect;
 #[cfg(target_os = "windows")]
 pub use win::collect;

+#[cfg(target_os = "freebsd")]
+pub use freebsd::collect;
+
 #[cfg(test)]
 mod tests {
     use super::*;

After the previous step, all dependencies should compile. But when compiling the reth binary itself, linker will complain that it cannot find __cpu_model and another symbol. The two missing symbols come from libgcc, so I needed to install it (pkg install gcc12-devel).
Somehow rust won't try to link with libgcc, so I needed to add println!("cargo:rustc-link-lib=static=gcc"); to bin/reth/build.rs.
Rust will now complain that it cannot find libgcc, so I added linker = "gcc12" to .cargo/config, and set RUSTFLAGS="-L/usr/lib" when calling cargo.

I'm happy to prepare a PR for these changes if the Reth team deems support for FreeBSD valuable.


Help:! unable to load the pretrained model from local/colab directory

Hi i downloaded the  saved to a directory in colab.
when i try to load the model im getting "We assumed '/content/drive/My Drive/bert_training/uncased_L-12_H-768_A-12/config.json"
this is how i loaded the model:
from transformers import BertForMaskedLM
BertNSP=BertForMaskedLM.from_pretrained('/content/drive/My Drive/bert_training/uncased_L-12_H-768_A-12/')
is this the correct way of loading model from the directory when i have downloaded the pretrained model ?
Im getting error " '/content/drive/My Drive/bert_training/uncased_L-12_H-768_A-12/config.json' "
the downloaded model had these naming conventions where file name start with bert_  but the BertForMaskedLM class is expecting the file name to be config.json .
bert_config.json
bert_model.ckpt.data-00000-of-00001
bert_model.ckpt.index  vocab.txt
bert_model.ckpt.meta
FULL ERROR:
Model name '/content/drive/My Drive/bert_training/uncased_L-12_H-768_A-12/' was not found in model name list (bert-base-uncased, bert-large-uncased, bert-base-cased, bert-large-cased, bert-base-multilingual-uncased, bert-base-multilingual-cased, bert-base-chinese, bert-base-german-cased, bert-large-uncased-whole-word-masking, bert-large-cased-whole-word-masking, bert-large-uncased-whole-word-masking-finetuned-squad, bert-large-cased-whole-word-masking-finetuned-squad, bert-base-cased-finetuned-mrpc, bert-base-german-dbmdz-cased, bert-base-german-dbmdz-uncased). We assumed '/content/drive/My Drive/bert_training/uncased_L-12_H-768_A-12/config.json' was a path or url to a configuration file named config.json or a directory containing such a file but couldn't find any such file at this path or url.
when i renamed the above 4 files by removing bert from all 4 file names , i get this error even though the "model.ckpt.index" files exist  "OSError: Error no file named ['pytorch_model.bin', 'tf_model.h5', 'model.ckpt.index'] found in directory /content/drive/My Drive/bert_training/uncased_L-12_H-768_A-12/ or from_tf set to False"

i got the same problem


mkvtoolnix: 21.0.0 -> 22.0.0

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools.
This update was made based on information from https://repology.org/metapackage/mkvtoolnix/versions.
These checks were done:

built on NixOS
ran /nix/store/qzarbwgjblj2vw9wc017gich3a5rv9f0-mkvtoolnix-22.0.0/bin/mkvmerge -h got 0 exit code
ran /nix/store/qzarbwgjblj2vw9wc017gich3a5rv9f0-mkvtoolnix-22.0.0/bin/mkvmerge --help got 0 exit code
ran /nix/store/qzarbwgjblj2vw9wc017gich3a5rv9f0-mkvtoolnix-22.0.0/bin/mkvmerge -V and found version 22.0.0
ran /nix/store/qzarbwgjblj2vw9wc017gich3a5rv9f0-mkvtoolnix-22.0.0/bin/mkvmerge --version and found version 22.0.0
ran /nix/store/qzarbwgjblj2vw9wc017gich3a5rv9f0-mkvtoolnix-22.0.0/bin/mkvinfo -h got 0 exit code
ran /nix/store/qzarbwgjblj2vw9wc017gich3a5rv9f0-mkvtoolnix-22.0.0/bin/mkvinfo --help got 0 exit code
ran /nix/store/qzarbwgjblj2vw9wc017gich3a5rv9f0-mkvtoolnix-22.0.0/bin/mkvinfo -V and found version 22.0.0
ran /nix/store/qzarbwgjblj2vw9wc017gich3a5rv9f0-mkvtoolnix-22.0.0/bin/mkvinfo --version and found version 22.0.0
ran /nix/store/qzarbwgjblj2vw9wc017gich3a5rv9f0-mkvtoolnix-22.0.0/bin/mkvextract -h got 0 exit code
ran /nix/store/qzarbwgjblj2vw9wc017gich3a5rv9f0-mkvtoolnix-22.0.0/bin/mkvextract --help got 0 exit code
ran /nix/store/qzarbwgjblj2vw9wc017gich3a5rv9f0-mkvtoolnix-22.0.0/bin/mkvextract -V and found version 22.0.0
ran /nix/store/qzarbwgjblj2vw9wc017gich3a5rv9f0-mkvtoolnix-22.0.0/bin/mkvextract --version and found version 22.0.0
ran /nix/store/qzarbwgjblj2vw9wc017gich3a5rv9f0-mkvtoolnix-22.0.0/bin/mkvpropedit -h got 0 exit code
ran /nix/store/qzarbwgjblj2vw9wc017gich3a5rv9f0-mkvtoolnix-22.0.0/bin/mkvpropedit --help got 0 exit code
ran /nix/store/qzarbwgjblj2vw9wc017gich3a5rv9f0-mkvtoolnix-22.0.0/bin/mkvpropedit -V and found version 22.0.0
ran /nix/store/qzarbwgjblj2vw9wc017gich3a5rv9f0-mkvtoolnix-22.0.0/bin/mkvpropedit --version and found version 22.0.0
found 22.0.0 with grep in /nix/store/qzarbwgjblj2vw9wc017gich3a5rv9f0-mkvtoolnix-22.0.0
directory tree listing: https://gist.github.com/7834479431ce4c1f041e9b5d9da3129d

cc @codyopel @fuuzetsu @rnhmjoj for review

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: mkvtoolnix
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowBroken = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowBroken = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: mkvtoolnix
Partial log (click to expand)
shrinking /nix/store/qzarbwgjblj2vw9wc017gich3a5rv9f0-mkvtoolnix-22.0.0/bin/mkvinfo
shrinking /nix/store/qzarbwgjblj2vw9wc017gich3a5rv9f0-mkvtoolnix-22.0.0/bin/mkvtoolnix-gui
shrinking /nix/store/qzarbwgjblj2vw9wc017gich3a5rv9f0-mkvtoolnix-22.0.0/bin/mkvextract
gzipping man pages under /nix/store/qzarbwgjblj2vw9wc017gich3a5rv9f0-mkvtoolnix-22.0.0/share/man/
strip is /nix/store/fzcs0fn6bb04m82frhlb78nc03ny3w55-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/qzarbwgjblj2vw9wc017gich3a5rv9f0-mkvtoolnix-22.0.0/bin
patching script interpreter paths in /nix/store/qzarbwgjblj2vw9wc017gich3a5rv9f0-mkvtoolnix-22.0.0
checking for references to /tmp/nix-build-mkvtoolnix-22.0.0.drv-0 in /nix/store/qzarbwgjblj2vw9wc017gich3a5rv9f0-mkvtoolnix-22.0.0...
postPatchMkspecs
/nix/store/qzarbwgjblj2vw9wc017gich3a5rv9f0-mkvtoolnix-22.0.0

Success on aarch64-linux (full log)
Attempted: mkvtoolnix
Partial log (click to expand)
shrinking /nix/store/sgcrrddfr33chaqk70392qgj9p79q7k9-mkvtoolnix-22.0.0/bin/mkvextract
shrinking /nix/store/sgcrrddfr33chaqk70392qgj9p79q7k9-mkvtoolnix-22.0.0/bin/mkvinfo
shrinking /nix/store/sgcrrddfr33chaqk70392qgj9p79q7k9-mkvtoolnix-22.0.0/bin/mkvmerge
gzipping man pages under /nix/store/sgcrrddfr33chaqk70392qgj9p79q7k9-mkvtoolnix-22.0.0/share/man/
strip is /nix/store/3zq400fri5dv7d30lpxlqm2v9y1iis6j-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/sgcrrddfr33chaqk70392qgj9p79q7k9-mkvtoolnix-22.0.0/bin
patching script interpreter paths in /nix/store/sgcrrddfr33chaqk70392qgj9p79q7k9-mkvtoolnix-22.0.0
checking for references to /build in /nix/store/sgcrrddfr33chaqk70392qgj9p79q7k9-mkvtoolnix-22.0.0...
postPatchMkspecs
/nix/store/sgcrrddfr33chaqk70392qgj9p79q7k9-mkvtoolnix-22.0.0


SPARKTA-415 Fixed bug with Java 7 in waitFor method

Fixed bug with Java 7.
Method waitFor(timeout, timeformat) is not available in Java 7 (only for Java 8).
I've used the method waitFor() without timeout (Documentation here)
Ticket in Jira
Definition of Done

Using Java 7 Spark should be launched properly in cluster mode
Using Java 8 Spark should be launched properly in cluster mode

+1

+1

:shipit:

Comment accepted. Will treat it ASAP

Squashing and checks in progress.


[ci:component:github.com/gardener/machine-controller-manager:v0.35.2->v0.36.0]

How to categorize this PR?

/area TODO
/kind TODO
/priority normal
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

Please add the release notes before merging this PR. Somehow the release notes didn't get transported.
All machine classes do now support an optional `.{spec.}credentialsSecretRef` field in addition to today's `.{spec.}secretRef` field. If `.{spec.}credentialsSecretRef` is non-nil then the provider credentials will be read out of this secret. The user-data for the machine bring-up is still required to be part of the secret referenced by `.{spec.}secretRef`.

Some machine class secrets are now supporting alternative data keys:
* The machine class secret for Alicloud machines does now also accept the data keys `accessKeyID` and `accessKeySecret` as alternatives for today's keys.
* The machine class secret for AWS machines does now also accept the data keys `accessKeyID` and `secretAccessKey` as alternatives for today's keys.
* The machine class secret for Azure machines does now also accept the data keys `clientID`, `clientSecret`, `subscriptionID` and `tenantID` as alternatives for today's keys.
* The machine class secret for GCP machines does now also accept the data key `serviceaccount.json` as alternatives for today's key.

Bumped AWS SDK version to v1.23.13

An issue causing panic when the encoded machine template hash length is less than expect limit is now fixed.

MCM will delete Azure machines even if the underlying resource group is already deleted.

Set Machine Phase to Terminating before draining.

Machine force deletion computation is based on deletionTimestamp instead of LastUpdatedTimestamp.

Update docker images to use gcr copy

Update docker image versions to `golang:1.15.5` & `alpine:3.12.1`