fix: renBTC doesn't exist on L2

@vbstreetz I'll let you merge this one if you think it resolves an issue. Otherwise you can close it.

@vbstreetz @clementbalestrat woops, forgot the context for this change - the fix would allow us to run staking with a local OVM setup, as I'm working on in https://github.com/Synthetixio/issues/issues/209.


Dead project ?

is it a dead project now ?

No. Just no people post issue here. If you use it, found issues, I'll fix it.
For the next version, I'm also in the planning, but it's just planning, now the main focus is still maintain 1.x


Coroutines finalizer

In case of the app crashing, we can lose coroutines which haven't been completed.
We might need to write more asynchronous code but we can't allow modules like tags storage to lose new data.
We should implement global exception handler ("last frontier"), which knows about spawned coroutines and ensures they complete before crashing. The same handler would send crash reports (#78).

Also this finalizer should be called when a user wants to kill the app, but we have some active coroutines.
In this case, the user should see progress spinner with label "Saving".


fix set reschedule policy docs

Swarm can't detect com.docker.swarm.reschedule-policy label, and it should be com.docker.swarm.reschedule-policies.
Thanks.
Signed-off-by: realityone<EMAIL_ADDRESS>

LGTM

Hi there, I have rebased codes from master.
Now all checks have passed yet.

LGTM

LGTM


Flex Upgrade

Checklist

[x] App update process has been followed
[x] Target branch is develop
[x] Application version has been bumped

Changes

Add tests for flex device
Use the latest useCase flows. remove depreciated flows.

Hello @ajinkyaraj-23
It seems like the workflows are not triggering but I don't understand why, workflow files seem valid both on LedgerHQ:develop and trilitech:main 🤔
I'll merge and trigger manually


Part3: StorIOSQLite mapping interop 1 to 3

Codecov Report

Merging #878 into master will increase coverage by 0.03%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master     #878      +/-   ##
============================================
+ Coverage     97.98%   98.02%   +0.03%
- Complexity      811      819       +8
============================================
  Files            98      100       +2
  Lines          2783     2835      +52
  Branches        308      312       +4
============================================
+ Hits           2727     2779      +52
  Misses           27       27
  Partials         29       29

Impacted Files
Coverage Δ
Complexity Δ

...h/storio3/sqlite/interop2to3/StorIOSQLite2To3.java
100% <100%> (ø)
4 <4> (?)

...h/storio3/sqlite/interop1to3/StorIOSQLite1To3.java
100% <100%> (ø)
4 <4> (?)

Continue to review full report at Codecov.

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


AutoMap option

Hello,
There is two glslang automap options that should be interesting in this tool:
shader->setAutoMapBindings(true);
shader->setAutoMapLocations(true);

This remove binding and location requirement in shader.

Thanks for the tip, I'll look into them and get back with you


Convert to TypeScript

Remove mocha/webpack/babel/browserify artifacts
Use jest with ts-jest for tests
Upgrade a package or two to get types
Switch Handlebars templates to just be TS files that export strings
Hope I got all the types correct

Don't want to deal with juggling branches, so I'm switching this repo to a develop branch and keeping master for released versions.
Merging this now so there can be TypeScript support in the default branch, but won't be released until I can do more clean-up and testing.


multiple validation return "too many errors"

On my use case at least.
With following code
var indicative = new(require('indicative'));

var stub      = {
    email       : ''
  };

var schema   = {
    email:  'required|email'
  };

//first call
 indicative
    .validateAll(schema, stub)
    .then(function(success){
      console.log('success', success);
    })
    .catch(function(errors){
      console.log('error', errors);
    });
//second call
 indicative
    .validateAll(schema, stub)
    .then(function(success){
      console.log('success', success);
    })
    .catch(function(errors){
      console.log('error', errors);
    });

I expect result
// for first validation
error [ { field: 'email',
    message: 'required validation failed on email',
    rule: 'required' } ]

// for second validation
error [ { field: 'email',
    message: 'required validation failed on email',
    rule: 'required' } ]

But I get this result
// for first validation
error [ { field: 'email',
    message: 'required validation failed on email',
    rule: 'required' },
  { field: 'email',
    message: 'required validation failed on email',
    rule: 'required' } ]

// for second validation
error [ { field: 'email',
    message: 'required validation failed on email',
    rule: 'required' },
  { field: 'email',
    message: 'required validation failed on email',
    rule: 'required' } ]

As you can see errors for both validation is returned on each result.

validateAll is an async function and re-running the function before completing the first one will not give parser the chance to clear off things.
Here you can do couple of things.

indicative
.validateAll(schema, stub)
.then(function () {
  return indicative.validateAll(schema,stub)
})
.then(function(success){
  console.log('success', success);
})
.catch(function(errors){
  console.log('error', errors);
});

Or have 2 instances of indicative
var indicative = new(require('indicative'));
var indicative1 = new(require('indicative'));

I think the problem is the single stack error.
You should have a new instance for each validateAll call and not a singleton.
As far as I understand, indicative is also a singleton (the same for Error class).
class Validator extends RULES

  'is': IS

  # singleton instance holder
  instance = null

  constructor: ->
    if !instance then instance = this
    super()
    return instance

Thus I have no chance to have multiple instance of Indicative.
In fact my use case is to validate rows
Promise.map(rows, function(row) {
 return myIndicative.validateAll(row, myRules)
  .then(function() {// my row is ok})
  .catch(function(err) { //report error on row })
})
.then(function() { // blabla })

Is it possible to make classes (Validator, Error, Messages) not singleton?

Even not being singleton will not help , coz you will be using same instance of indicative to have multiple validations running parallely not in sequence.

Ideal situation will be to make an array of rows and then validate them, having validation inside a loop will is not good choice

In node, you may have erratic problem by sharing the same state for multiple call.
I think my exemple is a simulation of simultaneous users calling validateAll at the same time.
To solve this, in validateField, you don't need Error class (maybe for formatting).
Just reject the error (without pushing it in the stack)
In validateAll, you could instanciate a new Error class.

Can you have a pull request for same , i am happy to merge it as long as it passes all the tests

oups... actually i never write coffescript.
I can try, let me check what I can do

What about writing in Javascript itself, i will manage the CoffeeScript part

I made a pull request here: https://github.com/Adonis-Js/indicative/pull/23

Merged as of v1.2.9 , thanks for your efforts :+1:

Added you to the contributors list

thanks ;)


Receiving payment_succeeded event on subscription cancelation

When I cancel subscription I receive customer.subscription.deleted event from Stripe which is OK.
After that I immediately receive invoice.payment_succeeded which even triggers error in code:
NoMethodError (undefined method `payment_succeeded' for nil:NilClass):
2016-09-13T16:30:27.066412+00:00 app[web.1]:   vendor/bundle/ruby/2.2.0/gems/koudoku-1.2.4/config/initializers/stripe_event.rb:13:in `block (2 levels) in <top (required)>'

Why am I even getting payment succeeded event on cancelation?

@knagode
Thanks for reporting this.
It looks like Stripe creates a final invoice to settle the remaining balance. That's what's causing this error.


How can I get container status

I refer to the usage of examples:
https://github.com/react-dnd/react-dnd/tree/main/packages/examples-hooks/src/04-sortable/simple
this is my code:
container:

Then I use useDragLayer it here：

it will give an error：

how can i solve it?
My goal is to get the state of the whole container, such as whether to drag or not，not item's status
thanks very mush

my fault...


add regression tests for delete_global_graphs

@rafsun42 I agree with @dehowef, very well done. Just check the Makefile indentation, as it doesn't look correct, as @dehowef mentioned.

@dehowef @jrgemignani Fixed the indentation issue.

Good job @rafsun42 !


docs: docs polish

I had to revert some things around webhooks and provider names b/c they're used for more than just documentation


Contructor dispatchers support

Dispatchers should also support constructors in some way, probably should be upstreamed to the compiler.

Dispatchers are not bound to the contracts themselves, since what binds them is the address and knowledge that we have a right interface.
This might be not doable because of that, without providing class_hash from the user. Is this something that is going to be an issue?

Should be doable via a plugin or upstream to compilator

Dispatchers could also generate a static method for deploying a instance of the contract from the provided class hash that would return Self. Something like
Dispatcher {
  // ...
  construct(class_hash: ClassHash, arg1: Type1, arg2: Type2, ...) -> Self {
    // perform deploy
    let addr = ...;
    Self { addr }
  }
}

Dispatchers could also generate a static method for deploying a instance of the contract from the provided class hash that would return Self. Something like...

That is actually not much different from the current deploy on the ContractClass, and would require also a custom plugin (or a modification/overriding of the cairo one) so that will be difficult to implement

@cptartur You can have different constructor signatures for a specific trait.

@cptartur You can have different constructor signatures for a specific trait.

Are you sure about that? I think that contract implementation can only have a single constructor.

@cptartur Contract implementation has one, but dispatchers are generated per trait not per implementation


[code-coverage] Add tests for HandleDecisionTaskFailed in service/history/decision

What changed?

Change tests name to address comments on #5901
Add unit tests for HandleDecisionTaskFailed

Why?
improve code coverage

How did you test it?
Unit tests

Potential risks
N/A

Release notes

Documentation Changes

Pull Request Test Coverage Report for Build 018eceb7-c380-4431-9dc8-04ab8226b030
Details

0 of 0   changed or added relevant lines in 0 files are covered.
15 unchanged lines in 7 files lost coverage.
Overall coverage increased (+0.02%) to 67.375%

Files with Coverage Reduction
New Missed Lines
%

common/task/weighted_round_robin_task_scheduler.go
1
89.05%

service/history/queue/timer_queue_processor_base.go
1
78.23%

common/peerprovider/ringpopprovider/config.go
2
81.58%

common/task/fifo_task_scheduler.go
2
87.63%

common/persistence/sql/sqlplugin/mysql/task.go
2
73.68%

common/persistence/sql/sqlplugin/mysql/db.go
2
79.49%

tools/cli/admin_db_decode_thrift.go
5
66.67%

Totals

Change from base Build 018eca11-8174-48a3-9058-79ba823db2e5:
0.02%

Covered Lines:
98217

Relevant Lines:
145776

💛  - Coveralls


cluster code coverage did not be included in the coverage report sometimes

Expected Behavior
Observed Behavior
I used cluster to test my server.
Koa server was run in a worker, and I tested the router of the server from master. This prevent the koa crash result in the whole process crash. But it seems that nyc can not calculated coverage report correctly sometimes.
the handle function of route existed in the controller file in controllers folder which it simply call some function in service file in services folder. Then the final coverage report sometimes did not includes controllers folder and services folder. It seems that the coverage in worker did not be included in the report. What curious me is that sometimes it will be included and I think it should be always included.
Bonus Points! Code (or Repository) that Reproduces Issue
This is my main test file. I run my application(koa server) in worker, while mocha in master.
// This is TypeScript code
import 'mocha'
import '../src/utils/config'
import { Test } from '../src/utils/Test'
import * as cluster from 'cluster'
import * as process from 'process'

if (cluster.isMaster) {
  const worker = cluster.fork()

  before('initial test data', async () => {
    await Test.Instance.connectDB()
    await Test.Instance.mockData()
    console.log('initial success')
  })

  after('destroy test data', async () => {
    await Test.Instance.destroy()
    console.log('destroy success')
  })

  worker.on('message', (msg: string) => msg === 'ok' && run())

} else if (cluster.isWorker) {
  Test.Instance.connectApp().then(() => process.send!('ok')).catch(e => console.log(e))
}

When the application in worker was connected, a signal was sent to master which call the run function mocha provide which it is used for starting test.
Forensic Information

Operating System: Arch Linux
Node version: v8.4.0
Npm version: v5.3.0
nyc version: 11.1.0
Nyc config:"nyc": {
  "cache": true,
  "reporter": [
    "lcov",
    "text"
  ],
  "extension": [
    ".js",
    ".ts"
  ],
  "exclude": [
    "test"
  ]
}

@ruiming ever make any progress on digging into this?
One thing comes to mind, are you using typescript-register to handle TypeScript parsing? If so, I would make sure that it's nyc that it's nyc itself that requires this module, by adding a require stanza to the nyc configuration.
If this does turn out to be the problem, I would love some help updating the README.md for nyc, since this order of operation problem often bites people.

No. I had stopped using cluster to test.
I had do some test to try to find the problem. Finally it proves that it is the problem of espower-typescript/guess(I am using power-assert)
To test below code:
// app.ts
import * as koa from 'koa'

export const server = () => {
  const app = new koa()

  app.use(async (ctx, next) => {
    ctx.status = 200
    await next()
  })

  app.listen(8001)
}

Test file:
import 'mocha'
import * as cluster from 'cluster'
import * as process from 'process'
import * as request from 'supertest'
import { server } from '../app'

if (cluster.isMaster) {
  const worker = cluster.fork()

  describe('', () => {
    it('', async () => {
      await request('http://localhost:8001')
        .get('/')
        .expect(200)
    })
  })

  worker.on('message', msg => run())
} else if (cluster.isWorker) {
  server()

  process.send!('app listen')
}

And I runnpm run test1 and npm run test2 with nyc disable cache.
"test1": "nyc mocha --compilers ts:espower-typescript/guess 'test/*.ts' --require source-map-support/register --delay --timeout=10000",
"test2": "nyc mocha --require ts-node/register 'test/*.ts' --require source-map-support/register --delay --timeout=10000"

The result is test1 with full coverage report and test2 with unknown report.
Then I just change the nyc cache from false to true, run test1 then test2 followed. test1 works still well but test2 with zero coverage report instead of unknown report.
To conclude, I think this problem is caused by this ts:espower-typescript/guess.

@ruiming great digging. Perhaps you can open an issue on espower-typescript/guess and reference this issue? It would be great to get this working for you.

I will, thanks.


OneShot broken on master

This is how the signal looks like, and apparently the ESC's don't like it:

The problem is the 2 callbacks in the pwm driver, the rate controller output and RC (the signal is fine when shutting off RC).
I have not checked when it broke exactly, just that it is not aa380f351fa01edfce71533c7eaefec4ff6446ee.

Two immediate ideas that come to mind.

Add a special case in the mixer_module to not register additional callbacks after actuator_controls_0/actuator_controls_1 if they're part of the mix.
Add a low level (minimum interval?) protection for oneshot. Are there well defined requirements?

I'll also add a simple minimum interval (maximum rate) mechanism to WorkItem.


Should make the media selector more configurable?

I'm using spider-rs to build a simple system which make a snapshot of my website, but it can't scrape the css/js/image.. and so on.
Will it be supported in future?

Hi @zishon, yes exposing the selector or gathering all assets that relate to the domain. Going to keep this issue open until the changes are made. Thank you

Released in 1.26.0 use the feature flag full_resources in order to enable.


Bindable isn't specific to configuration anymore

Bindable is used not only for listening configuration change but more general mechanism such as dependency injection.

Namespace change. Bindables are currently in osu.Framework.Configuration namespace.
Remove IHasDescription and IParseable from IBindable. It seems like IHasDescription is not used in both framework and osu. Possibly remove IHasDefaultValue and ICanBeDisabled too.

I was actually talking with @smoogipoo about bindables almost being worthwhile of splitting off into their own dedicated library. Would be interested in opinions on whether others see this as a useful thing to have for their non-osu-framework projects.
Can agree with moving to osu.Framework.Bindables or similar namespace.

I'd love for this to happen so I don't get all the libraries and other stuff when I basically just want the functionality around Bindables.

A minimum requirement for this would be making bindables threadsafe (or deciding that that isn't something we care about), which is not on our current agenda. There would need to be enough external interest in seeing this through for it to happen.

In the interest of keeping Bindables as high performance and lean as possible, they shouldn't be thread safe, imo.

The original scope of this ticket was to move bindables from the osu.Framework.Configuration namespace to the osu.Framework.Bindables namespace, which has since happened.
Unsure if this issue should still be kept open to track moving bindables to it's own separate library/repo.

Seems the repository is ready for that happen, closing the issue would makes this completely forgotten about. (although it's seemingly forgotten about currently)

Can be closed.


Bazel build is broken

Please answer these questions before submitting your issue.
Should this be an issue in the gRPC issue tracker?
Yes
What version of gRPC and what language are you using?
1.4pre1
What operating system (Linux, Windows, …) and version?
Linux
What runtime / compiler are you using (e.g. python version or version of gcc)
Bazel 0.5.1
What did you do?
If possible, provide a recipe for reproducing the error. Try being specific and include code snippets if helpful.
bazel fetch @grpc//...
............
ERROR: package contains errors: bazel.
ERROR: /home/ubuntu/.cache/bazel/_bazel_ubuntu/1cc89424d6356929076c22d262e240c4/external/grpc/bazel/BUILD:41:1: missing mandatory positional argument 'well_known_protos' while calling cc_grpc_library(name, srcs, deps, proto_only, well_known_protos, generate_mock, use_external = False, **kwargs).
What did you expect to see?
Obviously, a successful fetch
What did you see instead?
Error
Anything else we should know about your project / environment?
Please add proper test for bazel, and fix this problem

So, I'd need more information about what you're trying to achieve, and you're missing the point of the "Anything else we should know about your project / environment" question.
Right now if I run bazel build :all on my branch, it works fine. If I run bazel test test/... I am getting one broken test, but the rest seems fine too. So that doesn't really fit a "it's all broken" status.
Given you're trying to run bazel fetch, I am guessing you're trying to somehow grab remote modules. But we don't specify any in our WORKSPACE file at the moment, so while this may change in the future, running a fetch right now is kind of useless.
I'm tempted to file this as a case for Dr. Kromkheit, but I'm willing to hear more of your actual usage case that leads you to run a bazel fetch on gRPC even though it's not actually necessary to, aside from being future-proof for the potential case we start using remote bazel modules.


Calling save on an existing resource attempts to POST to and create new resource

Setup
Her::API.setup url: 'https://api-endpoint.com/api/v2' do |c|
  # Request
  c.use Auth0Authorization
  c.use FaradayMiddleware::EncodeJson

  # Response
  c.use Faraday::Response::Logger, ActiveSupport::Logger.new(STDOUT) if Rails.env.development?
  c.use Her::Middleware::DefaultParseJSON

  # Adapter
  c.use Faraday::Adapter::NetHttp
end

class User
  include Her::Model
end

Bug in action
Here is a pry session (with some sensitive info cleaned up)
[3] pry(main)> u = User.find('auth0|56e843ce9e61a7af469f8020')
=> #<User(users) nickname='something' >
[5] pry(main)> u.nickname = 'test'
=> "test"
[6] pry(main)> u.save
post https://api-endpoint.com/api/v2/users
User-Agent: "Faraday v0.10.0"
Authorization: "Bearer <BEARER_TOKEN>"
Content-Type: "application/json"
400
date: "Wed, 23 Nov 2016 19:40:41 GMT"
content-type: "application/json; charset=utf-8"
transfer-encoding: "chunked"
connection: "close"
vary: "origin,accept-encoding"
cache-control: "no-cache"
=> false

Discussion
As you can see, it attempts to create a new resource as opposed to creating a new one. I would expect this example to PATCH https://api-endpoint.com/api/v2/users/:user_id instead.
Env
Gem version: v0.8.2
Rails: <IP_ADDRESS>
Ruby: 2.3.1p112

Thanks for a super bug report, Ian. I haven't seen something like this before.
Does your user object have an id? Have you specified a different field to be the primary key? Does your user respond to new?() ?
Ed
--
Sent from my phone: please forgive the brevity.
Ed Jones

On 23 Nov 2016, at 19:47, Ian Ker-Seymer<EMAIL_ADDRESS>wrote:
Setup
Her::API.setup url: 'https://api-endpoint.com/api/v2' do |c|
Request
c.use Auth0Authorization
c.use FaradayMiddleware::EncodeJson
Response
c.use Faraday::Response::Logger, ActiveSupport::Logger.new(STDOUT) if Rails.env.development?
c.use Her::Middleware::DefaultParseJSON
Adapter
c.use Faraday::Adapter::NetHttp
end
class User
include Her::Model
end
Bug in action
Here is a pry session (with some sensitive info cleaned up)
[3] pry(main)> u = User.find('auth0|56e843ce9e61a7af469f8020')
=> #<User(users) nickname='something' >
[5] pry(main)> u.nickname = 'test'
=> "test"
[6] pry(main)> u.save
post https://api-endpoint.com/api/v2/users
User-Agent: "Faraday v0.10.0"
Authorization: "Bearer <BEARER_TOKEN>"
Content-Type: "application/json"
400
date: "Wed, 23 Nov 2016 19:40:41 GMT"
content-type: "application/json; charset=utf-8"
transfer-encoding: "chunked"
connection: "close"
vary: "origin,accept-encoding"
cache-control: "no-cache"
=> false
Discussion
As you can see, it attempts to create a new resource as opposed to creating a new one. I would expect this example to PATCH https://api-endpoint.com/api/v2/users/:user_id instead.
Env
Gem version: v0.8.2
Rails: <IP_ADDRESS>
Ruby: 2.3.1p112
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Does your user object have an id?

It does not. What is the best way to configure this?

that'll probably be it then :-) Something without an ID is considered new, I think. Can you return one in your result? If not, is there a primary key you can use as a substitute? You can configure your her-backed model to use that: https://github.com/remiprev/her#custom-primary-keys

Closing this since her depends on an Id or primary key to be defined.


[Bug] CRA preset does not support ES2020 dependencies

Describe the bug
Storybook + CRA preset fails due to ModuleParseError for projects containing dependencies in ES2020.
Steps to reproduce the behavior

Import a dependency with ES2020 syntax.
Start storybook.

See https://github.com/ph-fritsche/repro-storybook-es2020
Expected behavior
Accept imports with ES2020 like CRA does.
Environment

OS: Debian
Node.js version: 16.0.0
NPM version: 7.10.0

@ph-fritsche Were you able to work around this?

I was able to solve this with the following workaround:
  webpackFinal: async (config) => {
    // preset-create-react-app adds an include for the catch-all JS rule and
    // we need to remove it to allow ES2020 syntax in dependencies
    config.module.rules = config.module.rules.map((rule) => {
      if (!rule.oneOf) {
        return rule;
      }

      return {
        oneOf: rule.oneOf.map((oneOfRule) => {
          if (
            Array.isArray(oneOfRule.include) &&
            oneOfRule.include.length === 1 &&
            oneOfRule.include[0] === path.resolve(__dirname)
          ) {
            delete oneOfRule.include;
          }

          return oneOfRule;
        }),
      };
    });

    return config;
  },


MAINT: Generate more verbose reports (here, showing fieldmaps) if running in debug mode

This PR drafts a missing component for a faster assessment of problems with fieldmaps.

Codecov Report
Base: 44.71% // Head: 44.85% // Increases project coverage by +0.14% :tada:

Coverage data is based on head (e9a6f35) compared to base (786673b).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2872      +/-   ##
==========================================
+ Coverage   44.71%   44.85%   +0.14%
==========================================
  Files          43       43
  Lines        3509     3518       +9
==========================================
+ Hits         1569     1578       +9
  Misses       1940     1940

Impacted Files
Coverage Δ

fmriprep/workflows/bold/base.py
16.32% <0.00%> (-0.63%)
:arrow_down:

fmriprep/_version.py
42.09% <0.00%> (+2.54%)
:arrow_up:

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

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


Search for ICD JSON in display adapter and software component registry keys

This change extends the functionality of searching for ICD JSONs by adding
registry locations specific to given display adapter and software components
associated with this display adapter.
The exact locations in registry are queried using Windows public PnP
Configuration Manager API[1].
This change is required, as previous ICD locations (constant path
in "HKLM/Software") may be unreachable for drivers and their installers on
Windows RS3[2].
Similar change is being made for OpenCL[2]
[1]https://msdn.microsoft.com/en-us/library/windows/hardware/ff549713.aspx
[2]https://github.com/KhronosGroup/OpenCL-ICD-Loader/pull/21

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

These changes look good to me. I'm going to run the changes through our internal test farm to make sure they pass just fine, but they look good from reading over them.

I merged this pull request to master as commits 9beaa801945469eebaae6feb2cb3ac1f85cea292 and b69cb52f7c39c8f6faeed7e172f90c8c4f307043. I also added two commits. One of these prevents the same json manifest from being read twice, and the other changes some of your wanring messages to info messages. I changed the messages because our internal tests were failing on warnings, and because some of the messages looked like they really should have been info.
Just one note for future reference. We do have some coding conventions for the repo. I didn't try to enforce them in the code, because we've been notoriously lax about the conventions in the loader and I'm not going to hold you to standards we don't hold ourselves to, but I generally do try to format the commit messages according to those conventions. This is why I modified your commit messages. The general idea is that the first line should look like:
loader: Some message here

and be no longer than 50 characters. Any other lines after that should be a maximum of 70 characters.
But thanks for the pull request. This should be good to have it taken care of for the future.

Thanks Lenny for all the work here!

I don't understand the purpose of softwareComponentGUID. Why do you check for this?

@jjulianoatnv This is to look for ICDs in so-called SotwareComponent-s (https://docs.microsoft.com/en-us/windows-hardware/drivers/install/adding-software-components-with-an-inf-file)
If the loader fails to find ICDs in adapter tree, it tries to find ICDs in keys related to  child software components of this adapter.


Is there a way to open the date picker on input click ?

Is there a way to pop the date picker on the input click ?

Yes use on-focus free r demo see docs

thanks man

@pilouk How did you do this. Can you share it?

@praveenk-telsiz
Just add on-focus in the tag like this
  <sm-date-time-picker flex="50"
                         _on-focus_
                         flex-xs="100"
                         fname="field"
                         disable-year-selection="true"
                         label="Creation date"
                         ng-model="CreatedDateDisplay"
                         is-required="{{true}}"
                         format="lll"
                         mode="date-time"
                         week-start-day="Monday"
                         on-date-selected-call="dtpCreatedDateSelected(date)">
    </sm-date-time-picker>

Thanks !
On Wed, Nov 16, 2016 at 6:03 PM, pilouk<EMAIL_ADDRESS>wrote:

@praveenk-telsiz https://github.com/praveenk-telsiz
Just add on-focus in the tag like this

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mominsamir/smDateTimeRangePicker/issues/58#issuecomment-261132952,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQ5KF8QSTzfjrupRbVhjsktII2HKA0xwks5q-7XigaJpZM4KQgSb
.

@pilouk Do we have a proper documentation for this library?

@praveenk-telsiz
http://mominsamir.github.io/smDateTimeRangePicker/#/date-time-picker-api


Add a way to set custom css classes

Hi, since we ca set a custom css it could be useful having a way to set custom css classes to each speciment and use our CSS.
Something like
```html|class-MyClass

And
```html
class: "MyClass"

YES!!! Please support custom classes, this would be a huge help when trying to customize for our brand and styling.


[YouTube] Initial implementation of clips

Boilerplate: own code, improvement
## Please follow the guide below

You will be asked some questions, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your pull request (like that [x])
Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

[x] Searched the bugtracker for similar pull requests
[x] Read adding new extractor tutorial
[x] Read youtube-dl coding conventions and adjusted the code to meet them
[x] Covered the code with tests (note that PRs without tests will be REJECTED)
[x] Checked the code with flake8

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[ ] Bug fix
[x] Improvement
[ ] New extractor
[ ] New feature

Description of your pull request and other information
From issue #30843 (also https://github.com/yt-dlp/yt-dlp/issues/2543), YT is now supporting clips, which have a URL path like /clip/{opaque_encoding_string}. See also https://support.google.com/youtube/answer/10332730?hl=en.
In yt-dl such a URL is silently extracted as the whole video from which the clip was taken; in yt-dlp the clip is identified as unsupported.
This PR implements a simple-minded solution to extracting the clip by finding the start and end times, passing them as a hidden parameter, enforcing the FFmpegFD downloader when that parameter is present, and prepending the ffmpeg -ss ... and -to ... options with the passed values to the downloader options.
To do

[x] Confirm for separate video+audio streams
[ ] Should the ID, title or other metadata be modified from that of the original video to distinguish the clip from the original? (NB _url_transparent doesn't override the ID, but maybe pass to subclass YoutubeClipIE of YoutubeIE)
[ ] Similarly what about --download-archive?
[ ] Add formal tests
[ ] Should there be a framework for this to support other sites?
[ ] The clipping details should be recorded in the info_dict and unpacked from it so that --load-info-json ... works.

Resolves #30843.

or fix ffmpeg (apparently that happened but your ffmpeg looks to be newer -

Yes, that issue has been fixed by ffmpeg

In your output, format 303 is being selected in yt-dl but ffmpeg command is selecting 299+139 (avc1+aac) which aren't valid for webm. Is that why the error is being returned?

Right! That was indeed the issue
We should pass the correct stream using -map https://github.com/yt-dlp/yt-dlp/commit/6251555f1c670c66aa781bf9f29e9b5b33d90be1 https://github.com/yt-dlp/yt-dlp/commit/234416e4bf39d442e7abd036b7c59b8934a4086b. Note that the implementation will need to be slightly different for youtube-dl due to lack of https://github.com/yt-dlp/yt-dlp/commit/18e674b4f6bc738288b989d8b6a5f5698662a13b

Eagerly waiting for this.


Converting Char literal to String literal

What changes were proposed in this pull request?
In the PR, I propose to automatically convert a Literal with Char type to a Literal of String type.

Very logical: it unifies the logical programming model across Java/Scala & Python/R.


Calendar subscription and time zones

I have a subscribed calendar from Google. For events that I created myself the timezone in logseq will be correct. But for events that I've been invited to by someone in a different timezone, logseq displays that event in the timezone of its creator rather than me.
(Love the plugin, thanks!)

Do you mean that there are two time zone events in the same ics?


Fixed some bugs in projects、tasks and log.

What is the purpose of the change
(For example: Exchangis-Job defines the core ability of Exchangis, it provides the abilities of job management, job transform, and job launch.
Related issues: #407
Related issues: #406
Related issues: #405
Related issues: #404
Related issues: #403
Related issues: #402
Related issues: #401
Related issues: #397 . )
Brief change log
(for example:)

Set project name to uneditable on the front-end page when modifying the project;
Check whether any task exists when deleting an project. If yes, the project cannot be deleted;
The sub-task execution progress statistics are optimized to ensure the final consistency of the progress data;
When you edit a task, the front-end automatically displays the value of Execute User;
Fixed an issue where the name of a subtask is too long to run a task for a long time;
Fixed system exceptions when viewing non-current day logs;
Fixed an error message displayed on the log TAB page after the log query is abnormal.

Verifying this change
(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Configure a derivative task and verify it!

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes)
Anything that affects deployment: (no)
The Core framework, i.e., JobManager, Server.: (no)

Documentation

Does this pull request introduce a new feature? (yes)
If yes, how is the feature documented? (docs)

Please ensure it.

By the way, please resolve the github action build error.


Pow installer fails

When I'm try to install pow it fails:
	~ > () curl get.pow.cx | sh
	  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
	                                 Dload  Upload   Total   Spent    Left  Speed
	100  6813  100  6813    0     0   7745      0 --:--:-- --:--:-- --:--:-- 12687
	*** Installing Pow 0.3.1...
	*** Installing local configuration files...
	/Users/errorhandler/Library/LaunchAgents/cx.pow.powd.plist
	*** Installing system configuration files as root...
	/Library/LaunchDaemons/cx.pow.firewall.plist
	*** Starting the Pow server...
	*** Performing self-test...
	!!! Couldn't find a running Pow server on port 20559

	For troubleshooting instructions, please see the Pow wiki:
	https://github.com/37signals/pow/wiki/Troubleshooting

	To uninstall Pow, `curl get.pow.cx/uninstall.sh | sh`

I'm running Mac OS 10.6.7

Experienced this same issue using iTerm2 (aka iTerm).
Reran from Terminal, pow installer turned wifi off.
#epicfail.


Added 8ball command

Added an 8ball command

I will commit these changes locally. I thought this came under gurkbot#166
Sorry.

Am not sure if I should move replies into
https://github.com/gurkult/gurkbot/blob/main/bot/resources/bot_replies.yml

All done

Almost everything looks good. Minor change.

Done.

My first LGTM. This will forever be remembered by me. Thank you, looking forward for it to get merged.

sup

Can we wrap this up quick? If you have changes to request, please do, since I will be busy in the upcoming days.

Nice Nice thanks

Merge has been reverted due to a Git history problem. Please use branches when making pull requests.
I'd also like to see an issue before the next PR.


Fix some server crashes on incorrect functions usage. Add test.

Fixes some problems in #1563
added client --ignore-error  option

Ok, but the second part is implemented not enough carefully.

Please move Client.cpp changes to separate pull request.
The implementation is still incorrect, because "Parse error" message is inconsistent with the message from tryParseQuery function.


JavaScript interpreter

I want to add details about javascript interpreter.

i want to contribute here please assign me

Hi @SusheelThapa, Please assign this issue to me.

hi @SusheelThapa Please assign this issue to me

Hi @SusheelThapa assign me to this issue please.

I want to contribute please assign the issue to me


backend: Update dependencies

Updated

golang.org/x/oauth2 v0.10.0
k8s.io/* to v0.27.4
helm.sh/helm/v3 v3.12.2
github.com/stretchr/testify v1.8.4
github.com/rs/zerolog v1.29.1
github.com/knadh/koanf v1.5.0

Not updated (because issue)

github.com/go-playground/validator/v10 v10.14.1

There's an issue with update github.com/go-playground/validator/v10 v10.14.1
https://github.com/headlamp-k8s/headlamp/actions/runs/5668583349/job/15359589267

golang.org/x/oauth2 has several releases between golang.org/x/oauth2 v0.4.0 and v0.10.0

Tested general usage + port forwarding and LGTM.


Make sure ai import is a type, add zod to deps

I added zod to the deps explicitely.
This isn't broken in the published build, but without this, you cannot run ./bin/start from the srcbook/srcbook repo:


Tracker: FlatGFA Python bindings

Here's just a snapshot of some next steps for the Python bindings.
Project infrastructure:

[x] CI. #178 added some tests; they should run in GitHub actions. Maturin has a generate-ci command that should help with this. #180
[x] Build docs. Just like we have auto-published Sphinx docs for mygfa, let's do the same for the FlatGFA module. https://github.com/cucapra/pollen/pull/181

[ ] Expand/improve docs: at least divide into a few top-level sections.

[x] Publish on PyPI. I believe Maturin/Actions can help with this too.

API "surface area":

[ ] Mutation. Let's make it possible to build up a graph from scratch.
[ ] Expose the ops. Things that are currently CLI subcommands (in cmds.rs) should be put into a library of useful "ops" on the GFA data structure. This way, you can string together multiple operations in a Python invocation. In the limit, maybe we can reproduce odgi pipelines from our collaborators as Python scripts.

Python niceties:

[ ] The various objects should probably be made equatable/hashable.
[ ] Use str(obj) on paths, links, steps, etc. to produce GFA string fragments.
[ ] Names (of paths) should probably by str and not bytes? Seems fine to assume these are UTF-8. Nucleotide sequences can stay bytes.
[ ] Add typing hints. These would be especially helpful for the docs.

It's on PyPI (and therefore pip installable) now: https://pypi.org/project/flatgfa/


Added test for multiple send

I think with this final test, we could say that we have the send feature covered and if tests pass 100% we can close the send part of: https://github.com/polywrap/AutoTx/issues/166

/workflows/benchmarks agents/token/send 10


Fix Delicatessen preset

the preset to "delicatessen" is incorret (shop=delicatessen), because the correct is shop=deli.
http://wiki.openstreetmap.org/wiki/Tag:shop%3Ddeli

Hmm, looks like we already do have this preset as shop=deli, and it has been this way since it was added 2 years ago.  Is there a reason you opened this issue?


Web api dev

Q
A

Bug fix?
no

New feature?
no

New sample?
yes

Related issues?
n/a

What's in this Pull Request?
I've updated the code for angular-aad-webapi to GA.

thx David, will be processed in upcoming days.

thx David, appreciated.


Add Support for Revelator io44 Line In

Add support for the Line In channel on the io44
(everything else seems to work with io44)

What plugin was this again?
Loupedeck, StreamDeck or Touch Portal (or just in the API itself)?

Maybe the Revelator io44 is beyond the scope of the io24 project but here's what I can confirm in case you want to include it...
The Revelator io44 (as compared to io24) has an additional channel named "Line In".
ArmoredGenie's screenshot is what I see as well with my io44 in UC Surface.
The Touch Portal plugin does not list this channel in "Routing Table Change".
The Windows app does not list this either (see screenshot) in the GUI.

Thanks Oddbear!
Trucky

What plugin was this again? Loupedeck, StreamDeck or Touch Portal (or just in the API itself)?

I was referring to the StreamDeck Plugin. Thanks for all the work on this!

New beta added for the StreamDeck:
https://github.com/oddbear/Revelator.io24.Api/releases/tag/v0.0.2-streamdeck-beta
Does have support for io44 line in


Pip install not installing requirements

Shouldnt Pip Install the requirements in pip-requirements.txt?

DEPRECATION: Python 2.7 will reach the end of its life on January 1st, 2020. Please upgrade your Python as Python 2.7 won't be maintained after that date. A future version of pip will drop support for Python 2.7. More details about Python 2 support in pip, can be found at https://pip.pypa.io/en/latest/development/release-process/#python-2-support
Collecting ckanext-geoview
  Downloading https://files.pythonhosted.org/packages/e6/48/4e6fa5fe6a12d33440081a0873229bc6e8dbb38f959a6eb98513f166b99a/ckanext-geoview-0.0.16.tar.gz (988kB)
     |████████████████████████████████| 993kB 2.1MB/s
Building wheels for collected packages: ckanext-geoview
  Building wheel for ckanext-geoview (setup.py) ... done
  Created wheel for ckanext-geoview: filename=ckanext_geoview-0.0.16-cp27-none-any.whl size=1041138 sha256=2025eb8c27218ff4fcdef45cd73ea815f1671c0f2e9b42923fe11a70634b4a66
  Stored in directory: /home/user/.cache/pip/wheels/af/fc/00/0baa05a88dee758b515db082ee66e385715e235a8123f02356
Successfully built ckanext-geoview
Installing collected packages: ckanext-geoview
Successfully installed ckanext-geoview-0.0.16

(default) [user@server ckan]$ paster db upgrade -c /etc/ckan/default/production.ini

Traceback (most recent call last):
  File "/usr/lib/ckan/default/bin/paster", line 11, in <module>
    sys.exit(run())
  File "/usr/lib/ckan/default/lib/python2.7/site-packages/paste/script/command.py", line 102, in run
    invoke(command, command_name, options, args[1:])
  File "/usr/lib/ckan/default/lib/python2.7/site-packages/paste/script/command.py", line 141, in invoke
    exit_code = runner.run(args)
  File "/usr/lib/ckan/default/lib/python2.7/site-packages/paste/script/command.py", line 236, in run
    result = self.command()
  File "/usr/lib/ckan/default/src/ckan/ckan/lib/cli.py", line 357, in command
    self._load_config(cmd!='upgrade')
  File "/usr/lib/ckan/default/src/ckan/ckan/lib/cli.py", line 330, in _load_config
    self.site_user = load_config(self.options.config, load_site_user)
  File "/usr/lib/ckan/default/src/ckan/ckan/lib/cli.py", line 237, in load_config
    load_environment(conf.global_conf, conf.local_conf)
  File "/usr/lib/ckan/default/src/ckan/ckan/config/environment.py", line 112, in load_environment
    p.load_all()
  File "/usr/lib/ckan/default/src/ckan/ckan/plugins/core.py", line 140, in load_all
    load(*plugins)
  File "/usr/lib/ckan/default/src/ckan/ckan/plugins/core.py", line 154, in load
    service = _get_service(plugin)
  File "/usr/lib/ckan/default/src/ckan/ckan/plugins/core.py", line 256, in _get_service
    return plugin.load()(name=plugin_name)
  File "/usr/lib/ckan/default/lib/python2.7/site-packages/pkg_resources/__init__.py", line 2443, in load
    return self.resolve()
  File "/usr/lib/ckan/default/lib/python2.7/site-packages/pkg_resources/__init__.py", line 2449, in resolve
    module = __import__(self.module_name, fromlist=['__name__'], level=0)
  File "/usr/lib/ckan/default/lib/python2.7/site-packages/ckanext/geoview/plugin.py", line 7, in <module>
    import ckantoolkit as toolkit
ImportError: No module named ckantoolkit
txt?

This is a deep debate and there's no great answer. CKAN community has resolved to use requirements files.
I see we need to add this step to the README:
pip install -r pip-requirements.txt
like other extensions have.

@davidread This would mean changing the entire install approach, not just adding one line.
I am not a Python package expert at all, but isn't the idea when you do pip install ... that dependencies get installed automatically?

It's true for libraries, but less so for higher level 'projects', composed of lots of libraries.
There have been lots of cases where different CKAN extensions have conflicted in the versions required of their dependencies. If the authors specifies this in setup.py's install_requires, needed for automatic install, then you can't run the software, until you edit the setup.py. Whereas requirements files can be used or ignored. It might not be such a problem these days, with semver and us getting off pylons. It's just what the convention in the CKAN community now. Happy for you to start a discussion on Ckan Ideas to involve everyone.

Given I just had to remove two dependencies from an extension and then add two manually to get it to work (maybe not the best example) I can completely agree.
Then I will see if I can redraft the whole instructions to do the source code install and make a PR?

Yes, I guess step 2 would be better as something like:
pip install -e git+git@github.com:ckan/ckanext-xloader.git#egg=ckanext-xloader
pip install -r /usr/lib/ckan/default/src/ckanext-xloader/requirements.txt

What do you think?

This subject was recently discussed here . This module in particular has two dependencies that can be included in setup.py without much problem, and it will make setup much easier.


NCTL :: New commands to view on-chain account info

New commands to view on-chain account information plus some fixes due to regressions.

bors r+


I found a mistake in the app IG:dm

Okey.
I want to talk with my girlfriend, so i use this app for do that. Today in the afternoon, i talked with her and finally i closed the app. When i wanted to talk with her again i could not appear the conversations. so, what can i do for solve this problmes?.
Thanks

same as #235 . Pls read the previous issues before creating new ones.

Thanks. i'll see that. if i found something i will put the solution in other answer in this forum page


add one field  "example_id", but I can't see it in the "comput_loss" function

Hi,  I add one field example_id, but I can't see it in the comput_loss function, how can I do this? below is the information of inputs
*********************** inputs: {'attention_mask': tensor([[1, 1, 1,  ..., 0, 0, 0],
        [1, 1, 1,  ..., 0, 0, 0],
        [1, 1, 1,  ..., 0, 0, 0],
        ...,
        [1, 1, 1,  ..., 0, 0, 0],
        [1, 1, 1,  ..., 0, 0, 0],
        [1, 1, 1,  ..., 0, 0, 0]], device='cuda:0'), 'end_positions': tensor([ 25,  97,  93,  44,  25, 112, 109, 134], device='cuda:0'), 'input_ids': tensor([[ 101, 2054, 2390,  ...,    0,    0,    0],
        [ 101, 2054, 2515,  ...,    0,    0,    0],
        [ 101, 2054, 2106,  ...,    0,    0,    0],
        ...,
        [ 101, 2339, 2001,  ...,    0,    0,    0],
        [ 101, 2054, 2515,  ...,    0,    0,    0],
        [ 101, 2054, 2003,  ...,    0,    0,    0]], device='cuda:0'), 'start_positions': tensor([ 20,  90,  89,  41,  25,  96, 106, 132], device='cuda:0'), 'token_type_ids': tensor([[0, 0, 0,  ..., 0, 0, 0],
        [0, 0, 0,  ..., 0, 0, 0],
        [0, 0, 0,  ..., 0, 0, 0],
        ...,
        [0, 0, 0,  ..., 0, 0, 0],
        [0, 0, 0,  ..., 0, 0, 0],
        [0, 0, 0,  ..., 0, 0, 0]], device='cuda:0')}

# This function preprocesses a question answering dataset, tokenizing the question and context text
# and finding the right offsets for the answer spans in the tokenized context (to use as labels).
# Adapted from https://github.com/huggingface/transformers/blob/master/examples/pytorch/question-answering/run_qa.py
def prepare_train_dataset_qa(examples, tokenizer, max_seq_length=None):
    questions = [q.lstrip() for q in examples["question"]]
    max_seq_length = tokenizer.model_max_length
    # tokenize both questions and the corresponding context
    # if the context length is longer than max_length, we split it to several
    # chunks of max_length
    tokenized_examples = tokenizer(
        questions,
        examples["context"],
        truncation="only_second",
        max_length=max_seq_length,
        stride=min(max_seq_length // 2, 128),
        return_overflowing_tokens=True,
        return_offsets_mapping=True,
        padding="max_length"
    )

    # Since one example might give us several features if it has a long context,
    # we need a map from a feature to its corresponding example.
    sample_mapping = tokenized_examples.pop("overflow_to_sample_mapping")
    # The offset mappings will give us a map from token to character position
    # in the original context. This will help us compute the start_positions
    # and end_positions to get the final answer string.
    offset_mapping = tokenized_examples.pop("offset_mapping")

    tokenized_examples["start_positions"] = []
    tokenized_examples["end_positions"] = []

    tokenized_examples["example_id"] = []

    for i, offsets in enumerate(offset_mapping):
        input_ids = tokenized_examples["input_ids"][i]
        # We will label features not containing the answer the index of the CLS token.
        cls_index = input_ids.index(tokenizer.cls_token_id)
        sequence_ids = tokenized_examples.sequence_ids(i)
        # from the feature idx to sample idx
        sample_index = sample_mapping[i]
        # get the answer for a feature
        answers = examples["answers"][sample_index]

        tokenized_examples["example_id"].append(examples["id"][sample_index])

        if len(answers["answer_start"]) == 0:
            tokenized_examples["start_positions"].append(cls_index)
            tokenized_examples["end_positions"].append(cls_index)
        else:
            # Start/end character index of the answer in the text.
            start_char = answers["answer_start"][0]
            end_char = start_char + len(answers["text"][0])

            # Start token index of the current span in the text.
            token_start_index = 0
            while sequence_ids[token_start_index] != 1:
                token_start_index += 1

            # End token index of the current span in the text.
            token_end_index = len(input_ids) - 1
            while sequence_ids[token_end_index] != 1:
                token_end_index -= 1

            # Detect if the answer is out of the span (in which case this feature is labeled with the CLS index).
            if not (offsets[token_start_index][0] <= start_char and
                    offsets[token_end_index][1] >= end_char):
                tokenized_examples["start_positions"].append(cls_index)
                tokenized_examples["end_positions"].append(cls_index)
            else:
                # Otherwise move the token_start_index and token_end_index to the two ends of the answer.
                # Note: we could go after the last offset if the answer is the last word (edge case).
                while token_start_index < len(offsets) and \
                        offsets[token_start_index][0] <= start_char:
                    token_start_index += 1
                tokenized_examples["start_positions"].append(
                    token_start_index - 1)
                while offsets[token_end_index][1] >= end_char:
                    token_end_index -= 1
                tokenized_examples["end_positions"].append(token_end_index + 1)

    return tokenized_examples

Originally posted by @yanllearnn in https://github.com/huggingface/datasets/issues/3333#issuecomment-983457161

Hi ! Your function looks fine, I used to map squad locally and it indeed added the example_id field correctly.
However I think that in the compute_loss method only a subset of the fields are available: the model inputs. Since example_id is not a model input (it's not passed as a parameter to the model), the data loader doesn't need to return it by default.
However you can disable this behavior by setting remove_unused_columns to False to your training arguments. In this case in compute_loss you will get the full item with all the fields.
Note that since the model doesn't take example_id as input, you will have to remove it from the inputs when model(**inputs) is called

Hi, I have set args.remove_unused_columns=False and  training_args.remove_unused_columns=False, but the field doesn't been contained yet.
def main():
    argp = HfArgumentParser(TrainingArguments)
    # The HfArgumentParser object collects command-line arguments into an object (and provides default values for unspecified arguments).
    # In particular, TrainingArguments has several keys that you'll need/want to specify (when you call run.py from the command line):
    # --do_train
    #     When included, this argument tells the script to train a model.
    #     See docstrings for "--task" and "--dataset" for how the training dataset is selected.
    # --do_eval
    #     When included, this argument tells the script to evaluate the trained/loaded model on the validation split of the selected dataset.
    # --per_device_train_batch_size <int, default=8>
    #     This is the training batch size.
    #     If you're running on GPU, you should try to make this as large as you can without getting CUDA out-of-memory errors.
    #     For reference, with --max_length=128 and the default ELECTRA-small model, a batch size of 32 should fit in 4gb of GPU memory.
    # --num_train_epochs <float, default=3.0>
    #     How many passes to do through the training data.
    # --output_dir <path>
    #     Where to put the trained model checkpoint(s) and any eval predictions.
    #     *This argument is required*.

    argp.add_argument('--model', type=str,
                      default='google/electra-small-discriminator',
                      help="""This argument specifies the base model to fine-tune.
        This should either be a HuggingFace model ID (see https://huggingface.co/models)
        or a path to a saved model checkpoint (a folder containing config.json and pytorch_model.bin).""")
    argp.add_argument('--task', type=str, choices=['nli', 'qa'], required=True,
                      help="""This argument specifies which task to train/evaluate on.
        Pass "nli" for natural language inference or "qa" for question answering.
        By default, "nli" will use the SNLI dataset, and "qa" will use the SQuAD dataset.""")
    argp.add_argument('--dataset', type=str, default=None,
                      help="""This argument overrides the default dataset used for the specified task.""")
    argp.add_argument('--max_length', type=int, default=128,
                      help="""This argument limits the maximum sequence length used during training/evaluation.
        Shorter sequence lengths need less memory and computation time, but some examples may end up getting truncated.""")
    argp.add_argument('--max_train_samples', type=int, default=None,
                      help='Limit the number of examples to train on.')
    argp.add_argument('--max_eval_samples', type=int, default=None,
                      help='Limit the number of examples to evaluate on.')

    argp.remove_unused_columns = False
    training_args, args = argp.parse_args_into_dataclasses()
    args.remove_unused_columns=False
    training_args.remove_unused_columns=False

**************** train_dataset: Dataset({
    features: ['id', 'title', 'context', 'question', 'answers'],
    num_rows: 87599
})

**************** train_dataset_featurized: Dataset({
    features: ['attention_mask', 'end_positions', 'input_ids', 'start_positions', 'token_type_ids'],
    num_rows: 87714
})

Hi, I print the value, all are set to False, but don't work.
********************* training_args: TrainingArguments(
_n_gpu=1,
adafactor=False,
adam_beta1=0.9,
adam_beta2=0.999,
adam_epsilon=1e-08,
dataloader_drop_last=False,
dataloader_num_workers=0,
dataloader_pin_memory=True,
ddp_find_unused_parameters=None,
debug=[],
deepspeed=None,
disable_tqdm=False,
do_eval=False,
do_predict=False,
do_train=True,
eval_accumulation_steps=None,
eval_steps=None,
evaluation_strategy=IntervalStrategy.NO,
fp16=False,
fp16_backend=auto,
fp16_full_eval=False,
fp16_opt_level=O1,
gradient_accumulation_steps=1,
greater_is_better=None,
group_by_length=False,
ignore_data_skip=False,
label_names=None,
label_smoothing_factor=0.0,
learning_rate=5e-05,
length_column_name=length,
load_best_model_at_end=False,
local_rank=-1,
log_level=-1,
log_level_replica=-1,
log_on_each_node=True,
logging_dir=./re_trained_model/runs/Dec01_14-15-08_399b9290604c,
logging_first_step=False,
logging_steps=500,
logging_strategy=IntervalStrategy.STEPS,
lr_scheduler_type=SchedulerType.LINEAR,
max_grad_norm=1.0,
max_steps=-1,
metric_for_best_model=None,
mp_parameters=,
no_cuda=False,
num_train_epochs=3.0,
output_dir=./re_trained_model,
overwrite_output_dir=False,
past_index=-1,
per_device_eval_batch_size=8,
per_device_train_batch_size=8,
prediction_loss_only=False,
push_to_hub=False,
push_to_hub_model_id=re_trained_model,
push_to_hub_organization=None,
push_to_hub_token=None,
remove_unused_columns=False,
report_to=['tensorboard'],
resume_from_checkpoint=None,
run_name=./re_trained_model,
save_on_each_node=False,
save_steps=500,
save_strategy=IntervalStrategy.STEPS,
save_total_limit=None,
seed=42,
sharded_ddp=[],
skip_memory_metrics=True,
tpu_metrics_debug=False,
tpu_num_cores=None,
use_legacy_prediction_loop=False,
warmup_ratio=0.0,
warmup_steps=0,
weight_decay=0.0,
)

********************* args: Namespace(dataset='squad', max_eval_samples=None, max_length=128, max_train_samples=None, model='google/electra-small-discriminator', remove_unused_columns=False, task='qa')
2021-12-01 14:15:10,048 - WARNING - datasets.builder - Reusing dataset squad (/root/.cache/huggingface/datasets/squad/plain_text/1.0.0/d6ec3ceb99ca480ce37cdd35555d6cb2511d223b9150cce08a837ef62ffea453)
Some weights of the model checkpoint at google/electra-small-discriminator were not used when initializing ElectraForQuestionAnswering: ['discriminator_predictions.dense_prediction.weight', 'discriminator_predictions.dense_prediction.bias', 'discriminator_predictions.dense.weight', 'discriminator_predictions.dense.bias']
- This IS expected if you are initializing ElectraForQuestionAnswering from the checkpoint of a model trained on another task or with another architecture (e.g. initializing a BertForSequenceClassification model from a BertForPreTraining model).
- This IS NOT expected if you are initializing ElectraForQuestionAnswering from the checkpoint of a model that you expect to be exactly identical (initializing a BertForSequenceClassification model from a BertForSequenceClassification model).
Some weights of ElectraForQuestionAnswering were not initialized from the model checkpoint at google/electra-small-discriminator and are newly initialized: ['qa_outputs.bias', 'qa_outputs.weight']
You should probably TRAIN this model on a down-stream task to be able to use it for predictions and inference.
Preprocessing data... (this takes a little bit, should only happen once per dataset)

Hmmm, it might be because the default data collator removes all the fields with string type:
https://github.com/huggingface/transformers/blob/4c0dd199c8305903564c2edeae23d294edd4b321/src/transformers/data/data_collator.py#L107-L112
I guess you also need a custom data collator that doesn't remove them.

can you give a tutorial about how to do this?

I overwrite get_train_dataloader, and remove _remove_unused_columns, but it doesn't work.
    def get_train_dataloader(self) -> DataLoader:
        """
        Returns the training :class:`~torch.utils.data.DataLoader`.

        Will use no sampler if :obj:`self.train_dataset` does not implement :obj:`__len__`, a random sampler (adapted
        to distributed training if necessary) otherwise.

        Subclass and override this method if you want to inject some custom behavior.
        """
        if self.train_dataset is None:
            raise ValueError("Trainer: training requires a train_dataset.")

        train_dataset = self.train_dataset
        # if is_datasets_available() and isinstance(train_dataset, datasets.Dataset):
        #     train_dataset = self._remove_unused_columns(train_dataset, description="training")

        if isinstance(train_dataset, torch.utils.data.IterableDataset):
            if self.args.world_size > 1:
                train_dataset = IterableDatasetShard(
                    train_dataset,
                    batch_size=self.args.train_batch_size,
                    drop_last=self.args.dataloader_drop_last,
                    num_processes=self.args.world_size,
                    process_index=self.args.process_index,
                )

            return DataLoader(
                train_dataset,
                batch_size=self.args.train_batch_size,
                collate_fn=self.data_collator,
                num_workers=self.args.dataloader_num_workers,
                pin_memory=self.args.dataloader_pin_memory,
            )

        train_sampler = self._get_train_sampler()

        return DataLoader(
            train_dataset,
            batch_size=self.args.train_batch_size,
            sampler=train_sampler,
            collate_fn=self.data_collator,
            drop_last=self.args.dataloader_drop_last,
            num_workers=self.args.dataloader_num_workers,
            pin_memory=self.args.dataloader_pin_memory,
        )

Hi, it works now, thank you.

args.remove_unused_columns=False and training_args.remove_unused_columns=False
overwrite get_train_dataloader, and remove _remove_unused_columns
add new fields, and can be got in inputs.


Update SAM submodule, refs #0

Job VTB Documentation on 6b3ca0e wanted to post the following:
View the site here
This comment will be updated on new commits.

@travismui @hapfang the HTTF ring model is as you expected not working with the newer SAM. Did you figure out the issue on the SAM repo side?

@GiudGiud No fix on this yet, I will take another look this week and provide suggested fixes on this PR if we can resolve the issue.

@travismui any update on this?
I d like to get SAM to be up to date then I ll work on a general BlueCrab update


Synctoowner

Variables with the attribute:
[SyncVar(target = SyncTarget.Owner)]

will only be synchronized with the client with authority. This is useful for inventory and other data that doesn't need to be visible to other players.
Here is a full example:
public class SampleBehavior : NetworkBehaviour {

    [SyncVar]
    public int myVariable;

    [SyncVar(target = SyncTarget.Owner)]
    public string myData2;

    [SyncVar(target = SyncTarget.Owner)]
    public SyncListInt myOwnerList;
}

Codecov Report

Merging #8 into improvements will decrease coverage by 0.05%.
The diff coverage is 0%.

@@               Coverage Diff               @@
##           improvements      #8      +/-   ##
===============================================
- Coverage          3.77%   3.72%   -0.06%
===============================================
  Files                30      30
  Lines              7174    7278     +104
  Branches           1192    1210      +18
===============================================
  Hits                271     271
- Misses             6897    7001     +104
  Partials              6       6

Impacted Files
Coverage Δ

...Technologies-networking/Runtime/NetworkIdentity.cs
0% <0%> (ø)
:arrow_up:

...y-Technologies-networking/Runtime/NetworkServer.cs
0% <0%> (ø)
:arrow_up:

...echnologies-networking/Runtime/CustomAttributes.cs
0% <0%> (ø)
:arrow_up:

...echnologies-networking/Runtime/NetworkBehaviour.cs
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Fix potential fd leak

Fixes: #9760

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: zrss
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: nmittler
If they are not already assigned, you can assign the PR to them by writing /assign @nmittler in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pkg/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Hi @zrss. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/assign @ymesika

ping @nmittler for reviewing too, and it should be fixed ...

/ok-to-test
Replaces #9761

/lgtm

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: ymesika, zrss
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: nmittler
If they are not already assigned, you can assign the PR to them by writing /assign @nmittler in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pkg/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/assign @nmittler
/aasign @geeknoid

@zrss: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-dashboard.sh
5ce9190b9d13954c11217cd7a76f534dfcbd80d3
link
/test e2e-dashboard

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/approve

/test e2e-dashboard

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: linsun, ymesika, zrss
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pkg/OWNERS [linsun]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: linsun, nmittler, ymesika, zrss
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pkg/OWNERS [linsun,nmittler]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@zrss: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth.sh
5ce9190b9d13954c11217cd7a76f534dfcbd80d3
link
/test e2e-mixer-no_auth

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test e2e-mixer-no_auth

@zrss: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth.sh
5ce9190b9d13954c11217cd7a76f534dfcbd80d3
link
/test e2e-mixer-no_auth

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest
This bot automatically retries jobs that failed/flaked on approved PRs.
Review the full test history for this PR.
/lgtm cancel will stop retrying.

/retest
This bot automatically retries jobs that failed/flaked on approved PRs.
Review the full test history for this PR.
/lgtm cancel will stop retrying.

@zrss Do you mind rebasing to try to fix the failed required test?

:smile: i would do

/test e2e-mixer-no_auth

@zrss: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth.sh
5ce9190b9d13954c11217cd7a76f534dfcbd80d3
link
/test e2e-mixer-no_auth

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zrss can you please merge release-1.1 into your branch? There are changes required for the tests to pass.

@zrss can you reopen this under master branch and/or rebase your changes?

@howardjohn , done

see #13310


Typo in part1/cache_config.rst

Not sure if this is the preferred channel for bug reporting. Apologize in advance if it isn't.
There seems to be a typo in the __init__ function for L2Cache.
- super(L2Cache, self).__init__(options)
+ super(L2Cache, self).__init__()
The attached .py has the correct code.
Thanks for the clear and concise tutorial.

Where are you seeing the issue?
I see in part1/cache_config.rst that I have the options parameter that shouldn't be there. Do you see it somewhere else too?
Thanks for the bug report!

Sorry, forgot to escape my '+' and '-' so I may have been a little confusing. Think that's the only instance of the typo ;)


python311Packages.dvc: 3.26.2 -> 3.27.0

Diff: https://github.com/iterative/dvc/compare/refs/tags/3.26.2...3.27.0
Changelog: https://github.com/iterative/dvc/releases/tag/3.27.0
Description of changes

Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 261983 run on x86_64-linux 1

10 packages built:

dvc (python310Packages.dvc)
dvc-with-remotes
dvc-with-remotes.dist
dvc.dist (python310Packages.dvc.dist)
python310Packages.dvclive
python310Packages.dvclive.dist
python311Packages.dvc
python311Packages.dvc.dist
python311Packages.dvclive
python311Packages.dvclive.dist


stable/elastic-stack

Describe the bug
A clear and concise description of what the bug is.
Version of Helm and Kubernetes:
helm version:
v3.3.0-rc.1
Kubernetes version: 1.16
Which chart:
stable/elastic-stack
What happened:
I would like to deploy elastic-stack into Kubernetes cluster. I git clone the chart, use default settings. But, I got errors.

git clone https://github.com/helm/charts

Create namespace
kubectl create namespace elastic-stack

At $HOME/charts

$ helm install elastic-stack stable/elastic-stack --namespace=elastic-stack
Error: found in Chart.yaml, but missing in charts/ directory: elasticsearch, kibana, filebeat, logstash, fluentd, fluent-bit, fluentd-elasticsearch, nginx-ldapauth-proxy, elasticsearch-curator, elasticsearch-exporter
What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know:

You need to run the helm dependency commands to install the chart's dependencies. See https://helm.sh/docs/helm/helm_dependency/


Parameter type is still required even when anyOf is specified.

When manually defining an OpenAPI type with .openapi({...}), the type key is required, even when anyOf is specified in supplement of type. This might be an issue with OpenAPI 3.1 specifically.
Thank you for your work on this handy library.

Hey thanks for raising the issue! That's a good point that we should also allow specifying anyOf, oneOf and possibly allOf as overrides.
Any chance you can share what you're trying to represent to see if there's an alternative in the meantime?

I'd be happy to share. I have a recursive schema (let's say Order -> Customer -> Order[] -> etc). Because zod-openapi needs manual annotations (I think) for z.lazy(), I'm adding them like this:
const baseOrder = z.object({...})

const customerFieldSchema = () => z.string().or(Customer).nullable().openapi({ "x-expandable": true, description: "Expandable" });

export const Order: ZodCompositeType<
	typeof baseOrder,
	{ customer: ReturnType<typeof customerFieldSchema> }
> = baseOrder
	.extend({
		customer: z
			.lazy(customerFieldSchema)
			.openapi({
				anyOf: [
					{ type: "string" },
					{ $ref: `#/components/schemas/Customer` },
				],
				type: "placeholder" as "object", // then the `type` key is removed by a post-processing step
			}),
	})
	.strict()
	.openapi("Order");

The workaround in place now is just to include a type key and remove it later in middleware or post-processing. Maybe there's a better way to do this to absolve the need for the manual annotation?

Can I check if you're using zod-to-openapi or this library because the .openapi("Order") is from that library.
We use .openapi({ ref: "Order" }).
If you assign a ref to the Customer schema it should all work as we do support lazy 🙂

I see! I was using it through the Hono adapter and must have mixed up the libraries. I'll open an issue over at zod-to-openapi.
I'll see about using this library instead in the meantime. Thanks a bunch and sorry for the confusion.


Shorthand object notation for where

Usually a where condition specifies equality.
It would be much easier to just provide an object of a bunch of things I want to match against like this:
const person = await db
  .selectFrom('person')
  .selectAll()
  .where({ first_name: 'Jennifer' })
  .executeTakeFirst()

This just allows for more succinct code, Knex provides this option.

Nope


Helper: ID generating methods

We need an easy way to create URIs for objects like Manifests and Canvases

Config has default ID generation method
It makes ID optional if supplied it is override otherwise it creates a default.
Set base_uri

This would be a helper class in https://github.com/iiif-prezi/iiif-prezi3/tree/main/iiif_prezi3/helpers
Should:

Encode special characters (UTF-8)
Warn if it contains # as it could be a fragment selector

Something like: https://github.com/linked-art/crom/blob/master/cromulent/model.py#L288-L314

I don't know how to do this with pydantic and as a Helper that manipulates the class hierarchy.
Pydantic expects a minimally valid record to be passed as keywords into the init function of BaseModel.  We can inject some further code into base.Base.init ... but it needs to manipulate the call to BaseModel's constructor, rather than running after the instance has been built.

Update - I stopped trying to monkeypatch the class in, and instead just set a property on the skeleton generated classes. Doesn't follow the same pattern as the other helpers, but it doesn't need to, as it doesn't expose functionality on instances.
Comments welcome (here) on the approach taken:

https://github.com/iiif-prezi/iiif-prezi3/blob/id_generator/iiif_prezi3/base.py#L6-L13
https://github.com/iiif-prezi/iiif-prezi3/blob/id_generator/iiif_prezi3/helpers/auto_fields.py

Thanks! (@glenrobson @mbennett-uoe @zimeon)

Discussed on the call (25th Feb) and are happy with the approach. We think you could do something like:

class NewDefault(object):
    def generate_defaults(self, what, **kw):
        if not 'id' in kw:
            return {'id': "http://glen.com/" + reposerviceGetNewID()}

Manifest.defaults.append(NewDefault)

m = Manifest(label="My Manifest")

to set your own default id generator.

Assuming that reposerviceGetNewID() is visible to the function, then yes :)


Better import declaration names for added fields

This PR adds better import declaration names for added fields. We'll use the Field suffix  when we see the field originates from the base realm (where all our catalog entries are for fields). Otherwise we won't try to guess card or field (which can't be done from a purely syntax perspective), but we will make sure that the declaration doesn't collide with a JS built-in, like Map, or Array.
(also prettier wasn't run on one of the test modules so i had to do that as well)

It seems like we must be able to know for sure because if we are going to wrap it with contains or containsMany it should have a Field suffix and if we are going to wrap it with linksTo or linksToMany it should have a Card suffix.

Well the catalog entry tells us this. But we don’t. Have a catalog entry in all situations (like the inherit case). But we can make a better guess when we do have a catalog entry.

It seems like we must be able to know for sure because if we are going to wrap it with contains or containsMany it should have a Field suffix and if we are going to wrap it with linksTo or linksToMany it should have a Card suffix.

Well the catalog entry tells us this. But we don’t have a catalog entry in all situations (like the inherit case). But we can make a better guess when we do have a catalog entry.

Or sorry, I misspoke. I’m getting this work mixed up with the create definition file work. We definitely always have a catalog entry for each new field added. Yes I should be able to consistently rely on that to know if I have a card or field.


FEAT: display test count (total and why not by file)

x tests passed
y failed tests
z test skipped

I see in the code :

tests failed
known failure
And i don't understand the difference.
Are test failed only the one that fail for the first time ?
Are known failure thoses who have never pass before ?
Or is it something disconnected from past run ?

test.todo() and test.serial.todo() count up "tests todo"
test.failing() count up "known failure"
and test with failing assertions count up "tests failed"
if You run plain ava
ava tests/fixtures/tests/first-test.mjs

You get
  5 tests failed
  1 known failure
  2 tests todo

nearly the same for browser-ava
 ./src/browser-ava-cli.mjs --chromium tests/fixtures/tests/first-test.mjs

6 tests failed
1 known failure
2 tests todo

6 instead of 5 failures from the none working assertion skip (line 106 of first-test.mjs)


You cannot pass arguments to an inline script

Task name: AzurePowershell@5
Description: Remove inaccurate text
Documentation changes required: Y.  Done see https://github.com/MicrosoftDocs/azure-devops-docs/pull/10895
Added unit tests: N
Attached related issue: Y.
https://github.com/MicrosoftDocs/azure-devops-docs/issues/9742
https://github.com/MicrosoftDocs/azure-devops-docs/issues/9578
Checklist:

[ ] Task version was bumped - please check instruction how to do it
[ ] Checked that applied changes work as expected

Not stale.


Multisend Issue with Not Signing Second TX - v0.9.23

Some users are experiencing an issue where a Multisend fails on the second Signing of Transaction in v0.9.23
These sends do not include subassets (if that matters)
Some solutions I have seen that may help is:

Changing the BTC fee (something custom but near the wanted fee)
Sending less assets in the multisend (10 or less)
Refreshing Freewallet (to the latest BTC block if left open for some time) and trying again
Restarting Freewallet and trying again

Some other issues that may be related to this one if I was to start theorizing ...

Users have reported an error trying to use multi send with Trezor hardware wallets and in one circumstance had an issue using the multisend function while having a separate hardware address imported into Freewallet. May not be related but thought I would include the data.

Another thing to note here is less users have reported multisend issues with the prior version (0.9.22) regarding the "Not Signing Second TX" issue
So it may also help to downgrade to 0.9.22 for a very large single multisend

@oskgie on Telegram mentioned: have unadded the trezor addresses like you suggested... and everything works now. just sent >100 cards from my normal address via multisend successfully.
Wow is the trezor hardware addresses import related here??

I can confirm multisend does not work with Trezor.  The second part of the p2sh is not signed correctly.  As the Trezor device does not speak Counterparty it is not aware of the signatures required on the encoded inputs

Yesterday (1/24/24) using Version 0.9.25 AND 0.9.26 I got an error when attempting to do a multi-send. FW would broadcast the first transaction but then would send an error message during transaction 2. BTC got sent to a random address each time which I presume is lost forever. I attempted the multi-send 4 times. Twice on a laptop with no hardware wallet addresses and twice on a Windows desktop that DOES have hardware addresses imported.
I did not capture all the errors, but here are two:
Error validating transaction: Error running script for input 0 referencing bd58ff9e65feb25f57033b4ef5ddaf58e5189c033996edb591cac234fec55af7 at 0: scriptsig not push only for P2SH.
Error validating transaction: Rejected script for input 0 referencing dd66b8bce0ddc86a9a4cce7603ab8d78a57ce48c1bfa2dcf5d8767903fd62632 at 0..

I can confirm multisend does not work with Trezor.

@B0BSmiths Correct, I will be adding MPMA support for trezor in the future... just need to update the code a bit so that we handle signing the second tx... Javier and I will dig into this when we have some time.

Yesterday (1/24/24) using Version 0.9.25 AND 0.9.26 I got an error when attempting to do a multi-send.

@subterranean1 FYI, the outputs in MPMA tx1 are still signable/spendable... not random... so, even if you generate part 1 of a 2 part MPMA tx, then part 2 fails... you are still able to gather/collect all the dust from those part1 txs.
Currently no tool exists to find these failed half-way-done MPMA sends and gather the dust up, but its technically possible, and i'm sure tools will be built in the future to provide the ability to gather dust with a few clicks.
TLDR, your not losing any BTC to "random" addresses... any addresses generated in an MPMA tx1 are spendable / signable.

Thank you for the quick response. To be clear, I was not trying to use a Trezor for multisend. I was using my laptop and then desktop.I was directed to a post that mentioned removing hardware wallet addresses from freewallet might fix the problem.I’ll make note of the addresses where the failed tx fees reside in hopes of retrieving them at a later date On Jan 28, 2024, at 11:13 AM, J-Dog @.***> wrote:
I can confirm multisend does not work with Trezor.
@B0BSmiths Correct, I will be adding MPMA support for trezor in the future... just need to update the code a bit so that we handle signing the second tx... Javier and I will dig into this when we have some time.
Yesterday (1/24/24) using Version 0.9.25 AND 0.9.26 I got an error when attempting to do a multi-send.
@subterranean1 FYI, the outputs in MPMA tx1 are still signable/spendable... not random... so, even if you generate part 1 of a 2 part MPMA tx, then part 2 fails... you are still able to gather/collect all the dust from those part1 txs.
Currently no tool exists to find these failed half-way-done MPMA sends and gather the dust up, but its technically possible, and i'm sure tools will be built in the future to provide the ability to gather dust with a few clicks.
TLDR, your not losing any BTC to "random" addresses... any addresses generated in an MPMA tx1 are spendable / signable.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>


XTransformer Bug: cuda failed

Description
There exist some bugs when I run the XTransformer model. I run the training command as instrcucted in https://github.com/amzn/pecos/blob/mainline/pecos/xmc/xtransformer/README.md . However, I meet the bug: "RuntimeError: Expected object of device type cuda but got device type cpu for argument #3 'index' in call to _th_index_select". I use the disable-gpu command and the code can be run. So I wonder if there exist some bugs in the gpu utils code of XTransformer. Thanks!
How to Reproduce?
Steps to reproduce
python3 -m pecos.xmc.xtransformer.train --trn-text-path ${X_txt_path} \
                                            --trn-feat-path ${X_path}  \
                                            --trn-label-path ${Y_path} \
                                            --model-dir ${model_dir}

(Paste the commands you ran that produced the error.)

What have you tried to solve it?

Error message or code output
 File "/opt/conda/envs/python3.6/lib/python3.6/site-packages/pecos/xmc/xtransformer/model.py", line 375, in train
    return_dict=True,
  File "/opt/conda/envs/python3.6/lib/python3.6/site-packages/pecos/xmc/xtransformer/matcher.py", line 1333, in train
    matcher.fine_tune_encoder(prob, val_prob=val_prob, val_csr_codes=val_csr_codes)
  File "/opt/conda/envs/python3.6/lib/python3.6/site-packages/pecos/xmc/xtransformer/matcher.py", line 1079, in fine_tune_encoder
    label_embedding=(text_model_W_seq, text_model_b_seq),
  File "/opt/conda/envs/python3.6/lib/python3.6/site-packages/torch/nn/modules/module.py", line 550, in __call__
    result = self.forward(*input, **kwargs)
  File "/opt/conda/envs/python3.6/lib/python3.6/site-packages/pecos/xmc/xtransformer/network.py", line 234, in forward
    inputs_embeds=inputs_embeds,
  File "/opt/conda/envs/python3.6/lib/python3.6/site-packages/torch/nn/modules/module.py", line 550, in __call__
    result = self.forward(*input, **kwargs)
  File "/opt/conda/envs/python3.6/lib/python3.6/site-packages/transformers/models/bert/modeling_bert.py", line 989, in forward
    past_key_values_length=past_key_values_length,
  File "/opt/conda/envs/python3.6/lib/python3.6/site-packages/torch/nn/modules/module.py", line 550, in __call__
    result = self.forward(*input, **kwargs)
  File "/opt/conda/envs/python3.6/lib/python3.6/site-packages/transformers/models/bert/modeling_bert.py", line 215, in forward
    inputs_embeds = self.word_embeddings(input_ids)
  File "/opt/conda/envs/python3.6/lib/python3.6/site-packages/torch/nn/modules/module.py", line 550, in __call__
    result = self.forward(*input, **kwargs)
  File "/opt/conda/envs/python3.6/lib/python3.6/site-packages/torch/nn/modules/sparse.py", line 114, in forward
    self.norm_type, self.scale_grad_by_freq, self.sparse)
  File "/opt/conda/envs/python3.6/lib/python3.6/site-packages/torch/nn/functional.py", line 1724, in embedding
    return torch.embedding(weight, input, padding_idx, scale_grad_by_freq, sparse)
RuntimeError: Expected object of device type cuda but got device type cpu for argument #3 'index' in call to _th_index_select

Environment

Operating system: ubuntu
Python version: 3.6
Pytorch version: 1.5.1

Hi @wwangwitsel , the bug exists in the libpecos-0.1.0 version for single GPU training and it's fixed in this PR. The fix will be included in the next release. Meanwhile, you can checkout the latest code and use Installation from Source to avoid that.

Closing this issue after 5 days. You can reopen it if there are further questions. Thanks.


security/acme-client: please add support for active24 DNS API

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[x] I have read the contributing guide lines at https://github.com/opnsense/plugins/blob/master/CONTRIBUTING.md
[x] I have searched the existing issues, open and closed, and I'm convinced that mine is new.
[x] When the request is meant for an existing plugin, I've added its name to the title.

Is your feature request related to a problem? Please describe.
no
Describe the solution you'd like
acme.sh already contains necessary provider so please just add it to GUI
https://github.com/acmesh-official/acme.sh/blob/master/dnsapi/dns_active24.sh
Describe alternatives you've considered

we considered to change registrar but active24 is the only registrar with API support in our country
we tested opnsense BIND API but it is not working (https://github.com/acmesh-official/acme.sh/issues/3735)

This will be available in the upcoming release 3.12 of os-acme-client.


Please add "Convert using the Azure portal" session as Windows

there are "Convert using the Azure portal" session in WIndows. Please add the same one here.
https://docs.microsoft.com/en-us/azure/virtual-machines/windows/convert-unmanaged-to-managed-disks#convert-using-the-azure-portal

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

ID: 4a056e00-8b21-8439-27c4-f506bbf76dbe
Version Independent ID: 54752162-9e09-4e7c-09fa-135a6a799ee4
Content: Convert a Linux virtual machine in Azure from unmanaged disks to managed disks - Azure Managed Disks
Content Source: articles/virtual-machines/linux/convert-unmanaged-to-managed-disks.md
Service: virtual-machines-linux
GitHub Login: @roygara
Microsoft Alias: rogarana

@nmikuni Thanks for the feedback.
@roygara I think the steps to convert the disks in portal from unmanaged to managed are same for Windows and Linux.
Can i duplicate that block("Convert using the Azure portal") from Virtual machines Windows to Linux?
Please suggest

@nmikuni I have assigned the issue to the content author to investigate further and update the document as appropriate.

@nmikuni Thank you for reaching out, sorry for the delayed response.
Yes, these steps are the same between Windows and Linux, so I will add them to the documentation for Linux as well.

The changes have been merged and should be live some time tomorrow.
Again, thank you for taking the time to reach out and provide suggestions for improving our docs, we really do appreciate it!
#please-close


Rats cicd

Please fix the compile error in Basic Compilation Check(alinux2)

Please add LVI binutils tools in compile test dockerfile and update the images, otherwise the compiler check will error.

Please add LVI binutils tools in compile test dockerfile and update the images, otherwise the compiler check will error.

It has been updated yesterday

Will merge this pr after passing the rats-tls test in the private repo


MGDOBR-794: Add azure event hubs action

MGDOBR-794
PR owner checklist
Please make sure that your PR meets the following requirements:

[ ] Your code is properly formatted according to this configuration.
Run mvn clean verify -DskipTests so that imports are correctly set.
[ ] Your commit messages are clear and reference the JIRA issue e.g: "[MGDOBR-1] - $clear_explanation_of_what_you_did"
[ ] The layers in the kustomize folder have been updated accordingly.
[ ] All new functionality is tested
[ ] Pull Request title is properly formatted: MGDOBR-XYZ Subject
[ ] Pull Request contains link to the JIRA issue
[ ] Pull Request contains link to any dependent or related Pull Request
[ ] Pull Request contains description of the issue
[ ] Pull Request does not include fixes for issues other than the main ticket

PR reviewer(s) checklist

[ ] Check if code and Github action workflows have been modified and if the modification is safe
[ ] If the modification is safe, add the safe to test label

How to trigger pipelines and use the bots:

Run the end to end pipeline
Annotate the pull request with the label: safe to test. If you want to run the pipeline again, remove and add it again.

Rebase the pull request
Comment with: /rebase.

Deploy BOT
Comment with `/deploy <dev,stable> when the PR has been merged to deploy to a target environment.

Is the title of this PR wrong?
It seems to duplicate (in part?) https://github.com/5733d9e2be6485d52ffa08870cabdee0/sandbox/pull/986

The PR looks good to me. Checked now.

In theory MC fixed the connector, shall we give it another try @sutaakar ? Just ping me when you have time to test this so that I can allocate the resources if you need

@Aakanksha0311 will check that again, last month it was working properly.

@Aakanksha0311 will check that again, last month it was working properly.

Yes, checked and LGTM.


Main - Octopus and Mech

There is a section of the main scene, in the water area where I can see

an octopus working on a mech


Adding a Search box demo

**Is your feature request related to a problem? Please describe. **
I saw there is no demo for a search box. It's great to have one for easy intermigration.
Describe the solution you'd like
The search field should be able to search from multiple results. It should be able to dynamically change its results on the basis of current text in the search field.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context

I think for a search box, we could try adding the Angular Autocomplete component (https://material.angular.io/components/autocomplete/overview). I haven't tested if this component would be easy to add to Mesop or not.
So I think if we were to add search box demo, we'd try to make it look similar to the Angular Autocomplete component.
I do agree that this is a good use case to cover. Thanks for the suggestion.

Sure, I will try to make a pr on it.

Going to close this issue since we'll likely be adding the Angular Autocomplete component.


Can't get it to work with components

I can get this working using your example but when I try with with my own components, although everything renders, the media query is ignored.
This works..
  render() {
    return (
        <MediaQuery minDeviceWidth={800}>

test

        </MediaQuery>
      )
}

This renders and throws no error but nothing happens resizing to below 800px...
  render() {
      return (
        <MediaQuery minDeviceWidth={800}>
          <InstagramFeed instagramData={this.state.instagramData}/>
        </MediaQuery>
      )
    }

InstagramFeed is just a basic array of list items. I'm using es6 class syntax, is there anything I need to do in regards to binding this?

@nevace So the InstagramFeed component returns an actual array of elements?

Yeah, here's what I'm returning in the component's render method and a screen grab of the DOM.
  const photos = this.props.instagramData.map((data, i) => {
      return (
        <li style={{backgroundImage: `url(${data.images.standard_resolution.url})`}} key={`instImg${i}`}>

      )
    })

 return (

          {photos}

      );

Can you try updating to the latest version?

I'm on version 1.01. I'll be working on this the next couple of days so I'll have a play about.

The problem is that you are using minDeviceWidth which queries the device's width and not the window with, therefore resizing the window will not trigger this media query unless the physical device reports itself as being < 800px wide. Use minWidth instead.

 I'll try this later but assume it's the issue. It's strange how the above code was working on the div though. I'll close as soon as I've checked. Cheers.


Pretty permalinks

When generating pages put them in a folder with an index.html to make pretty permalinks. For example
_content/about.md

would become
about/index.html

This would also allow us to remove the .htaccess.

Fixed in https://github.com/gilbitron/Handle/commit/96f85fc91ecd7ad95082f920c97f78dbdfdaf22d