error: 'Unrecognised access token'

I'm using the correct bot token in my config.yaml file, i even reset it and check it 3-4 times but the same error again and again
    'access-control-allow-headers': 'Origin, X-Requested-With, Content-Type, Accept, Authorization, Date'
    }
  },
  body: {
    errcode: 'M_UNKNOWN_TOKEN',
    error: 'Unrecognised access token',
    soft_logout: false
  },
  [Symbol(kCapture)]: false
}
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>start: `npm run-script build && node ./build/src/discordas.js -c config.yaml`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/krishnerkar/.npm/_logs/2021-01-21T17_41_24_000Z-debug.log

sounds like you didn't register your registration file with synapse & restart synapse?

sounds like you didn't register your registration file with synapse & restart synapse?

sounds like you didn't register your registration file with synapse & restart synapse?

well i did copy the registration file inside the synapse folder, how to restart synapse, im not sure

sounds like you didn't register your registration file with synapse & restart synapse?

well i did copy the registration file inside the synapse folder, how to restart synapse, im not sure

depends on your distro and installation method, typically systemctl restart synapse or systemctl restart matrix-synapse

depends on your distro and installation method, typically systemctl restart synapse or systemctl restart matrix-synapse

wait, i have a small doubt, in which folder do we have to run the following commands

wait, i have a small doubt, in which folder do we have to run the following commands

This repository is not for general help on system administration. Please find appropriate help channels for learning how to maintain a server instead.

This repository is not for general help on system administration. Please find appropriate help channels for learning how to maintain a server instead.

Hey, its still not working, the restart commands are not working, im on linux, ubuntu

Hey, its still not working, the restart commands are not working, im on linux, ubuntu


QSC initial implementation

This includes an initial implementation of the QSC algorithm. It does not support Byzantine behavior.

@nkcr I had to change the address iterator thing as we need the length of the set in the Call functions. Let me know what you think about the Take function.

@nkcr comments fixed.


Publishing existing package overrides existing package to 0 bytes file

Describe the bug
When trying to publish a package that already exists it says "npm ERR! publish fail Cannot publish over existing version."
But at the same time, something fails at the server and the existing file turns into a 0 bytes size file.
It doesn't happen all the time, but it happens.
To Reproduce
Steps to reproduce the behavior:

Try publishing a bunch of tgz packed packages with the command "npm publish xxxx.tgz" . (that already exist). I added a file at the bottom with some tgzs and script to execute it fast
get inside the server's storage place and execute command "find -iname *.tgz -type f -size -1c".
It will show you all tgz files under 1 byte.
Try installing the package you found and you'll get integrity error (0 bytes).

Expected behavior
Expected the package to not get overwritten.
Screenshots
Found files that are 0 bytes.
Let's look at the debug package.

Found in the logs a error that appers only on package that have been destroyed. It is marked

Docker || Kubernetes (please complete the following information):

Openshift
Docker - verdaccio:3.10.1

Configuration File (cat ~/.config/verdaccio/config.yaml)
# path to a directory with all packages
storage: ./storage

auth:
  htpasswd:
    file: ./htpasswd
    # Maximum amount of users allowed to register, defaults to "+inf".
    # You can set this to -1 to disable registration.
    #max_users: 1000

# a list of other known repositories we can talk to

packages:
  '@*/*':
    # scoped packages
    access: $all
    publish: $all

  '**':
    # allow all users (including non-authenticated users) to read and
    # publish all packages
    #
    # you can specify usernames/groupnames (depending on your auth plugin)
    # and three keywords: "$all", "$anonymous", "$authenticated"
    access: $all

    # allow all known users to publish packages
    # (anyone can register by default, remember?)
    publish: $all

    # if package is not available locally, proxy requests to 'npmjs' registry

# To use `npm audit` uncomment the following section
middlewares:
  audit:
    enabled: true

# log settings
logs:
  - {type: stdout, format: pretty, level: http}
  #- {type: file, path: verdaccio.log, level: info}

Debugging output
Logs at the pictures.
Additional context
adding a file with some packages for you to try.
packages-demo.zip

Upload them all with the script.bat inside
Run the script.bat again
Look inside the server for 0 bytes packages.

I've seen this issue on one of our Windows VMs as well (with our own packages, not using @pavels555's demo files. Sometimes it creates an empty file and sometimes it deletes the file.

One thing I wanted to note is the server where we don't see this issue is running Windows Server 2008 R2. The one with the issue uses Windows Server 2016.
Poking around the code, it looks like Verdaccio moves the existing tarball out of the way, then tries to put it back on error (I'm not 100% sure, I'm having a hard time following the code, and can't find a way to attach a debugger running through IIS).
My best guess is that this code is what corrupts the file:
Package: @verdaccio\local-storage\lib\local-fs.js
Function: renameTmp
// windows can't remove opened file,   // but it seem to be able to rename it   const tmp = tempFile(dst);   _fs2.default.rename(dst, tmp, function (err) {     _fs2.default.rename(src, dst, cb);     if (!err) {       _fs2.default.unlink(tmp, () => {});     }   });
My error file is identical to @pavels555 - including the Write After End message.

thanks @pavels555 for diving into it, effectively the issue is the following. If there is an error on fs.exists. If the tarball exist for some reason we are trying to get a temp file, but in that very moment the error emit close the uploadStream and having in account the uploadStream.pipe(file); is piped with file, we open the file and we cannot open ERR_STREAM_WRITE_AFTER_END and stream that has ended. So, my solution (or what I have on mind) is don't create tmp file and stop the execution there if the file does not exist.

I found a solution, it seems at #892 we forgot something essential, the issue is at the local storage plugin. If the resource exists we should not open a file create the stream, it just does not make sense.
I'll publish here a patched version, I'd like you guys to try it in your side, it works fine, I'll merge the patch.

npm install -g verdaccio@fix-1191 --registry https://registry.verdaccio.org

I installed the patch and it looks like the bug is fixed - thank you!!!

Perfect ! I will release it ! Thanks for the quick test

<EMAIL_ADDRESS>


Added Cpp interpreter

Added a generic C++ contract interpreter.
Implement the generic interpreter and integrate it in the PDO enclave lib
Implement a client side (python) scripts for testing the interpreter
Integrate interpreter into PDO in simulated SGX mode
Updated PDO build process to allow conditional compilation of the specified interpreter

Closing this to get it out of the queue - the branch will continue to be available.
For others that find this and are wondering why - this is intermediate work that will enable us to eventually add support for other contract interpreters besides Gipsy Scheme. However, this intermediate step does not provide an interpreter itself - just a framework for one. To avoid confusion, we are not going to merge this into the mainline code until we have something that can actually be used.


Error with BaseGameUtils.resolveConnectionFailure

Anyone knows something about this?

Your passing a String variable where it now requires an int.  Instead of hard coding your error message.  Put the error message in the String resource file and use the ID if the String instead.
Example from Google Developer:
    if (!BaseGameUtils.resolveConnectionFailure(this,
            mGoogleApiClient, connectionResult,
            RC_SIGN_IN, R.string.signin_other_error)) {
        mResolvingConnectionFailure = false;
    }


embassy-rp: UART flush isn't flushy enough

The blocking uart's blocking+async flush only waits until the buffered data is enqueued, NOT until the data is all out on the wire:
https://github.com/embassy-rs/embassy/blob/871d82de7165781ca6164de72664c9a6a90b3e66/embassy-rp/src/uart/buffered.rs#L419-L427
Instead, you also need to busy loop on the uart.busy() flag that reports whether the transmitter is active. I do that here:
https://github.com/jamesmunns/erdnuss-pub/blob/3d98a55aa7a90141a525e4443c927400f8516dd2/source/rp2040/src/lib.rs#L74-L77
There is no interrupt for this state. AFAICT, you have to busy poll from the time of interrupt (when DMA has completed and all bytes enqueued), maybe past the interrupt when the FIFO falls low again, and then busy loop until the transmitter is idle.
This is not compliant with embedded-io's flush guarantees. For blocking, this is NBD, we can just busy loop. It's annoying for async tho.

This is the same kind of deficiency being fixed by #2416

The busy loop is exactly what I’m doing for controlling the RS485 transceiver.
However, it would be nice to at least have an async flush for the fifo, then you only have to busy-loop on the last byte.
I guess one could start a (hardware) timer in the uart interrupt for TX fifo empty, that will expire as the last bit is expected to leave the shift register.

I re-checked the datasheet and I remembered why in another project I disabled the FIFO for uart - there is no interrupt for an empty FIFO. The best you can get is an interrupt when 4 bytes are remaining to be sent (with watermark level set to 7/8.
That is still better than nothing I suppose - we can have async for everything, but last 5 bytes (4 in the fifo + 1 in the shift register).
We could get better by disabling the FIFO, but then we get a much higher risk of losing bytes on reception, since there are no separate controls for RX and TX FIFO. In the end, this is using the same interrupts and registers, so it could be a config option with no difference in code for different modes.
//offtopic: I remember scratching my head about how could they release a microcontroller with no hardware TE control and no TX complete style interrupt. I’ve only seen this on some PIC16 micros, never in a 32-bit chip. The best you can do is utilize PIO to control TE for you, but that is hard to make bit-accurate (i.e. make the TE rise right after stop bit).

@enbyted yeah, there's also a detail I got bit with on my RS485 project: there is a "Line Idle" interrupt, but it only fires if the FIFO is not empty, which means if you use DMA then the FIFO is almost always empty.
That's why I ended up using a line break in my wire protocol instead of just a line idle time.

That’s also why I’m going with line break, though realistically for my needs 9-bit frames (i.e. what Atmel I think calls the uart with addressing mode or something like that) would be more efficient due to usually sending very short frames. Once I get some other parts of my system running I might work on some support for that, actually.
Anyhow, for this issue fixing the flush to also busy loop until transmission is fully complete sounds like the right approach.
Would you be interested in getting a PIO-based async flush and/or TE controller into embassy? Since PIO can send interrupts it could wake up an async task for flush implementation, plus it’s nearly the same operation as controlling the TE line, I think that would be doable in a relatively-generic way. I’d need to look closer at the APIs, but I might be interested in working on that in near-ish future (a few weeks time).

Just for the record:
the only solutions I can think of are:

use a timer and "guess/approximate" how long it'll take to finish flushing

CON: This is the most complex and makes embassy-time a hard dependency

do awhile uart.busy() { yield_now().await } semi busyloop
do awhile uart.busy() {} busyloop


Gulpfile changes

Uses gulpfile.babel.js
Bumps gulp
Starts to use ES6

Finally I also started moving all the contents of tasks into functions. There's more refactoring work to do here, but you'll notice that there's a bunch of extra tasks that people create when writing gulp when you can actually just use normal functions. I started the work of moving things over like this:
function clean(dir, cb) {
  del([dir], cb);
}

gulp.task('clean', done => clean(destinationFolder, done));
gulp.task('clean-tmp', done => clean('tmp', done));

Again, more refactoring can be done here, but for now everything lives within separate functions than their tasks.

Hmm, for some reason istanbul is getting all upset

Opened an issue in Babel

Sry, updated the Gulpfile this morning @thejameskyle :speak_no_evil:

@thejameskyle want to split this out into smaller, easier-to-merge PRs? This is good stuff.

ping @thejameskyle

Opened two issues for the changes here: #211 and #212


Prometheus chart healthcheck does not honor web.external-url

Bug Report:
When defining a custom ingress rule based on paths[1], you'll need to specify web.external-url[2] as additional args. When this parameter is set, the readiness probe fails because :9090/status responds with a 404[3]. In this case, the readiness probe should honor the path defined byweb.external-url or web.route-prefix so that Prometheus stays running.
Kubernetes 1.7.2
Helm 2.5.1
Helm command run to reproduce:
helm install stable/prometheus  --set alertmanager.image.tag=v0.8.0 --set kubeStateMetrics.image.tag=v1.0.0-rc.1  --set nodeExporter.image.tag=v0.14.0 --set server.image.tag=v1.7.1 -f vars.yaml

[1]
apiVersion: extensions/v1beta1
kind: Ingress
metadata:
  name: prometheus
  namespace: default
  annotations:
spec:
  rules:
  - host: mydomain.com
    http:
      paths:
      - path: /prometheus
        backend:
          serviceName: sappy-chinchilla-prometheus-server
          servicePort: 80

[2]
vars.yaml
server:
  extraArgs:
    web.external-url: http://mydomain.com/prometheus

[3]
ubuntu@control-01:~$ curl <IP_ADDRESS>:9090/status
404 page not found
ubuntu@control-01:~$ curl <IP_ADDRESS>:9090
Found.

ubuntu@control-01:~$ curl <IP_ADDRESS>:9090/prometheus
Moved Permanently.

ubuntu@control-01:~$ curl <IP_ADDRESS>:9090/prometheus/
Found.

ubuntu@control-01:~$ curl <IP_ADDRESS>:9090/prometheus/status
<!DOCTYPE html>

...

@bradbeam i figured this out after digging around the prometheus source code.
You also need to add this cli option
server:
  extraArgs:
    web.route-prefix: "/"

@bradbeam i figured this out after digging around the prometheus source code.
You also need to add this cli option
server:
  extraArgs:
    web.route-prefix: "/"

This should be documented somewhere. Saved me so much time.

I know this ticket is already closed, but I'd like to add that the current stable/prometheus helm chart has the following value available that you can set to define the external-url:
By default:
baseURL: ""
Change it to:
baseURL: "http://<insert-your-url-here/prometheus"
When you launch the helm chart with baseURL defined, it will create the pod/container with the extraArg --external-url=baseURL
You still need to set the web.route-prefix: "/" extraArg for the readiness/liveness probes to work (which is need for the container to come up):
extraArgs:
  web.route-prefix: "/"


as_node2d syntax is awkward

Instead of as_node2d godot uses:
Node2D *node_2d = cast_to<Node2D>(node);

I've pushed a new container with cast_to<Object, Node, Node2D, Node3D>

API implementation here: https://github.com/libriscv/godot-sandbox/pull/70/commits/a00721598bc1c98210ca0895b766ffe452d119ae


feat(toolkit): do not override google drive files on new selection

AI Description

This pull request updates the CreateAgent and UpdateAgent components in the src/interfaces/coral_web/src/components/Agents directory.
Summary

The PR introduces changes to the way artifacts are handled when the tool name is TOOL_GOOGLE_DRIVE_ID.
It ensures that duplicate artifacts are removed by using the uniqBy function from Lodash.
The tools_metadata array is updated to include only unique artifacts for TOOL_GOOGLE_DRIVE_ID, while keeping other tool artifacts unchanged.

Code Changes

The uniqBy function from Lodash is imported in both CreateAgent.tsx and UpdateAgent.tsx.
In CreateAgent.tsx, when handling data from Google Drive, the code now constructs an updatedArtifacts array, which includes existing artifacts for TOOL_GOOGLE_DRIVE_ID and new artifacts from the data.docs array.
The tools_metadata array is then updated with a new object containing TOOL_GOOGLE_DRIVE_ID and the unique updatedArtifacts.
Similar changes are made in UpdateAgent.tsx, where the updatedArtifacts array is constructed and used to update the artifacts property of the updateGoogleDriveTool object.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 81.38%. Comparing base (86eefe2) to head (b5317ad).
Report is 6 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #464      +/-   ##
==========================================
- Coverage   81.63%   81.38%   -0.25%
==========================================
  Files         198      199       +1
  Lines        8303     8355      +52
==========================================
+ Hits         6778     6800      +22
- Misses       1525     1555      +30

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


direct url 404

Direct access to pages via link will result in 404. Can only access starting from home page.

Done. Refer to https://stackoverflow.com/questions/63462828/404-error-on-refresh-with-spa-react-router-app-in-github-pages .

Add 404.html. Change pathSegmentsToKeep=1. Change the title.
Add js script to index.html, in head and body.
BrowserRouter basename, change path from 'process.env.PUBLIC_URL'+"/?" to "/?".


Fix client build

Client cannot be built with cargo build -p iroha_client again due to recent changes in #3929

what is done
`iroha git:(iroha2-dev) cargo build -p iroha_client
Compiling iroha_wasm_builder v2.0.0-pre-rc.20 (/Users/michaeltimofeev/Documents/projects/iroha/wasm_builder)
Compiling iroha_client v2.0.0-pre-rc.20 (/Users/michaeltimofeev/Documents/projects/iroha/client)
error: failed to run custom build command for iroha_client v2.0.0-pre-rc.20 (/Users/michaeltimofeev/Documents/projects/iroha/client)
Caused by:
process didn't exit successfully: /Users/michaeltimofeev/Documents/projects/iroha/target/debug/build/iroha_client-3d7b98df93d4a098/build-script-build (exit status: 1)
--- stdout
cargo:rerun-if-changed=tests/integration/smartcontracts
--- stderr
Error: Failed to check the smartcontract at path: /Users/michaeltimofeev/Documents/projects/iroha/client/tests/integration/smartcontracts
Caused by:
cargo check returned non zero exit code (exit status: 101). Stderr:
warning: /Users/michaeltimofeev/Documents/projects/iroha/schema/Cargo.toml: unused manifest key lints (may be supported in a future version)
  consider passing `-Zlints` to enable this feature.
  warning: /Users/michaeltimofeev/Documents/projects/iroha/smart_contract/trigger/Cargo.toml: unused manifest key `lints` (may be supported in a future version)

  consider passing `-Zlints` to enable this feature.
  warning: /Users/michaeltimofeev/Documents/projects/iroha/smart_contract/executor/Cargo.toml: unused manifest key `lints` (may be supported in a future version)

`
`
iroha git:(iroha2-dev) cargo +nightly-2022-12-22 build -p iroha_client_cli
Compiling proc-macro-utils v0.8.0
Compiling rustversion v1.0.14
Compiling percent-encoding v2.3.0
Compiling form_urlencoded v1.2.0
Compiling manyhow-macros v0.8.1
error[E0658]: use of unstable library feature 'is_some_and'
--> /Users/michaeltimofeev/.cargo/registry/src/github.com-1ecc6299db9ec823/manyhow-macros-0.8.1/src/lib.rs:283:33
|
283 |             path.pop().as_ref().is_some_and(TokenTreePunct::is_semi),
|                                 ^^^^^^^^^^^
|
= note: see issue #93050 https://github.com/rust-lang/rust/issues/93050 for more information
= help: add #![feature(is_some_and)] to the crate attributes to enable
For more information about this error, try rustc --explain E0658.
error: could not compile manyhow-macros due to previous error
warning: build failed, waiting for other jobs to finish...
`


Add webviz related package dependencies for deterministic build

Summary
Problem: webviz-core  depends on the internal packages such as regl-worldview and react-key-listerner, and currently we run the CI build against the latest package src code, not on the actual npm packages. Ideally, we'd like to update any package, publish, and update webviz-core in separate PRs.
Fix: add the packages as dependencies and modify webpack.config to not look for node modules in /packages directory, unless USE_LOCAL_PACKAGES env variable is set.
Test plan
CI should pass.
Versioning impact
No impact.

Closing this as it's not worth the effort to change.


Update youtrack extension

Description
Added Assignee to Issue Details Page
Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

@tiulpin do you want to take a look before merge?

LGTM


April 26 - Tester

This is a paid opportunity through the BCDevExchange.To apply, visit the opportunity page and submit a proposal before Friday, April 27, 2018

This is a test.


:construction_worker: Enable ccache for GitHub Action runners

Using hendrikmuhs/ccache-action, build times of the CI jobs are significantly sped up via caching.

Using MSVC, CMake currently does not support the CMAKE_CXX_COMPILER_LAUNCHER flag, thus, ccache (or sccache for that matter) is not properly called. This could only be fixed at the moment by moving away from MSVC as the generator in favor of Unix Makefiles or Ninja. In that case, however, we would lose many build configurations. I do not want to make this trade. Instead, we will accept the sub-par build performance on Windows CI for the moment hoping that a future update to CMake or ccache fixes this issue.


Clean up 3rd party license references and contribution guidelines

Signed-off-by: John Mertic<EMAIL_ADDRESS>

@cary-ilm Can you review and have ILM legal review the license file changes to make sure they are comfortable aligning with the standard BSD-3-Clause verbiage?


virtio, krun: Name all the worker threads

Helps debugging with gdb, otherwise they inherit another thread name and it gets confusing.

@slp PTAL


Fixed the button group's radius to be as a wrapper for the whole group #8447

#8447

@abdullahsalem Since this does change the default behavior, it would be better to target to branch v6.3. Can you point it there and we'll merge in for 6.3?

Sure, @rafibomb.


Intance model properties are undefined

Versions

sequelize: 5.21.7
sequelize-typescript:  1.1.0
typescript: 3.8.3

I'm submitting a ...

[ x ] bug report
[ ] feature request
Actual behavior:
I execute a findOne function to get a model instance, but the properties are undefined
Expected behavior:
I would like to get the properties values
Steps to reproduce:
const city = await CityModel.findOne({ where: { name: 'example' } })
console.log(city.name);

Related code:
If I execute the city.getDataValue('name') I see the right value, but simply running city.name I get undefined.
If I execute console.log(city) all works well:
CityModel {
    dataValues: {
        id: 'fb73e40a-9055-42b8-9a21-d8368ef13222',
        name: 'Example',
        createdAt: 2020-04-27,
        updatedAt: 2020-04-27
    },
    _previousDataValues: {
        id: 'fb73e40a-9055-42b8-9a21-d8368ef13222',
        name: 'Example',
        createdAt: 2020-04-27,
        updatedAt: 2020-04-27
    },
    ...
},

My model:
import {
  Model,
  Table,
  DataType,
  Column,
  PrimaryKey,
  Default,
  HasMany,
} from 'sequelize-typescript';
import UserModel from './user.model';

@Table({ tableName: 'Cities', timestamps: true })
class CityModel extends Model<CityModel> {
  @PrimaryKey
  @Default(DataType.UUIDV4)
  @Column(DataType.UUIDV4)
  id!: string;

  @Column(DataType.STRING)
  name!: string;

  @Column(DataType.STRING)
  initials!: string;

  @Column(DataType.STRING)
  status!: string;

  @HasMany(() => UserModel, 'cityId')
  users!: UserModel[];
}

export default CityModel;

Same problem. It seems like I need to call getDataValue.

Same problem. It seems like I need to call getDataValue.

I created a quick repo here showing the issue: https://github.com/jrm2k6/laughing-octo-enigma
When running the command to execute a findAll this is what I get back:
Executing (default): SELECT "uuid", "name", "createdAt", "updatedAt", "deletedAt" FROM "users" AS "users" WHERE ("users"."deletedAt" IS NULL);
[
  users {
    dataValues: {
      uuid: '36497139-77d2-4edb-ba49-481a589f2f8c',
      name: 'John Smith',
      createdAt: 2021-02-22T01:15:35.993Z,
      updatedAt: null,
      deletedAt: null
    },
    _previousDataValues: {
      uuid: '36497139-77d2-4edb-ba49-481a589f2f8c',
      name: 'John Smith',
      createdAt: 2021-02-22T01:15:35.993Z,
      updatedAt: null,
      deletedAt: null
    },
    _changed: {},
    _modelOptions: {
      timestamps: true,
      validate: {},
      freezeTableName: false,
      underscored: false,
      paranoid: true,
      rejectOnEmpty: false,
      whereCollection: null,
      schema: null,
      schemaDelimiter: '',
      defaultScope: {},
      scopes: {},
      indexes: [],
      name: [Object],
      omitNull: false,
      sequelize: [Sequelize],
      hooks: {}
    },
    _options: {
      isNewRecord: false,
      _schema: null,
      _schemaDelimiter: '',
      raw: true,
  1 node_modules/
      attributes: [Array]
    },
    isNewRecord: false,
    uuid: undefined,
    name: undefined
  }
]

I also use sequelize-typescript with nestjs and I am not having the same issue but haven't found out if it does something different.
Hope that helps so that we can figure out if we are doing something wrong or not.
Thanks!

@jrm2k6 The problem is I can not get the value just using user.name, but if I use user.getDataValue('name') works fine.
I tested it with your github project and the result was the same.

@gideaoms Yes, this is why I added a way for anybody to get the same results as we have and see if it is something wrong with our setup or not. Makes it easier for a maintainer to understand the issue and be able to debug if needed.

@RobinBuschmann Sorry to tag directly in here but any idea if this is something related to the setup or anything else?

Ok I did some more investigating and I think in my example, it boils down to using babel-node vs ts-node.
In here: https://github.com/jrm2k6/laughing-octo-enigma/commit/e745244c1b995f174adf2d48263b30ac52c49e3c
I changed my tsconfig to use commonjs instead of esnext, and use ts-node instead of babel-node.
When I run node -r ts-node/register index.ts it lets me access the model properties directly. I am not 100% sure yet but I think with the way my babel-node config is set up, it will generate model properties automatically initialized with undefined, overriding what sequelize-typescript might be doing when it comes to accessing model properties. Still a guess though.

Yep, it makes sense.

Encountered this issue when upgrading typescript from 3 to 4. Data was only in dataValues but not in the class fields. Setting useDefineForClassFields to true in my tsconfig seems to have fixed it.

That's weird. For me doing the exact opposite (setting useDefineForClassFields to false) fixed it. I'm targeting ESNext on Node 14.

I had actually set it to false but just wrote the wrong value in the comment. Thanks.

For me, it had nothing to do with babel (which I wasn't using to begin with).
The solution was entirely to add "useDefineForClassFields": "false" to my tsconfig.json.
Thanks, @triesmon!

This appears to be the change in babel that breaks this behavior:
https://github.com/babel/babel/pull/11747

Take a look at the updated sequelize documentation regarding the issue with class fields: https://sequelize.org/v6/manual/model-basics.html#caveat-with-public-class-fields

The new documentation is a big improvement over where we were at a week ago, unfortunately the declare keyword trick only works with tsc tooling and not with babel due to the above linked PR.

The fact that babel still emits public class fields tagged with declare when using decorators is problematic. Makes https://github.com/sequelize/meetings/issues/16 all the more pressing.

I have worked around this in my project with a model class decorator – it's very ugly, but could provide the basis for another way to fix the issue more formally in sequelize by applying the getter/setters after the model constructor.
https://github.com/outline/outline/blob/main/server/models/decorators/Fix.ts

First of all great project! Strange that README.md not updated with 'declare' workaround, as well as with ReturnType<() => Model>; for associated fields.
To prepare PR with "declare" column attributes and "ReturnType" associated attributes?

I landed here with a similar issue. The issue arose when I updated my tsconfig to the recommended settings for node 20
It seems that setting "lib": ["ES2023"] causes this issue. Upon some investigation, it actually seems like setting lib to this value may switch on useDefineForClassFields. After I forced useDefineForClassFields off, the issue went away. I would not consider this a resolution though, because it seems like the wrong decision long term.
TSConfig:
{
  "compilerOptions": {
    "allowJs": false,
    "baseUrl": "./",
    "declaration": true,
    "useDefineForClassFields": false,
    "emitDecoratorMetadata": true,
    // "esModuleInterop": true,
    "experimentalDecorators": true,
    "forceConsistentCasingInFileNames": true,
    "incremental": true,
    "module": "CommonJS",
    "moduleResolution": "Node",
    "noImplicitReturns": true,
    "noUnusedLocals": true,
    "noUnusedParameters": true,
    "outDir": "./dist",
    "removeComments": true,
    "skipLibCheck": true,
    "strict": true,
    "target": "ES2017",
    "lib": [
      "ES2023"
    ],
    "typeRoots": [
      "node_modules/@types",
      "src/types"
    ],
    "resolveJsonModule": true,
    "sourceMap": true,
    "inlineSources": true,
    "sourceRoot": "/",
    "allowSyntheticDefaultImports": true
  },
  "exclude": [
    "node_modules",
    "dist",
    "knex/templates",
  ]
}

Node: 20.11.0
TypeScript: 5.4.5 (although issue appeared on older version too)
Sequelize: 6.37.3
Sequelize-TypeScript: 2.1.6

For me, it had nothing to do with babel (which I wasn't using to begin with).
The solution was entirely to add "useDefineForClassFields": "false" to my tsconfig.json.
Thanks, @triesmon!

Thanks so much! Spent a lot of time trying to find out a problem, this solved my issue!


New extension:

Description
Create Wheels Joints for a two wheeled vehicle or any transport
1-cars 2-bikes 3-trucks with 2 axes
That's it.
How to use the extension
How to use the extension
Create 3 sprite objects.
One for car/vehicle and two others for front and back wheels.
Add Physcis behavior for these objects.
At Events, add a Beginning of the Scene condition and this extension only action.
Follow the fields and fill them with the respective values.
Checklist

[X] I've followed all of the best practices.
[X] I confirm that this extension can be integrated to this GitHub repository, distributed and MIT licensed.
[X] I am aware that the extension may be updated by anyone, and do not need my explicit consent to do so.

What tier of review do you aim for your extension?
Community (Unreviewed)
Example file
VEHICLE JOINT CREATOR.zip
Extension file
WheelJointCreator.zip

Duplicate of #966.  Please update that issue to continue this extension.


Send null instead of 0 for null data

For me, temperature and humidity are always null:
(MQTT) Received sensor data. Temperature: null. Humidity: null

In the Home app, this shows as 32° (in the US) and 0% humidity. Is there a way properly send HomeKit null instead of 0? Is there a way to get this value from the fan?

Actually, I'm seeing values now! Yay. It would still be great if we could send null instead of 0 though. Updating the issue title.

If I recall correctly the fan has to be on for these values to be returned properly from the fan. (Especially the humidity value, since I think air passing through the sensor allows for an accurate reading.)
I'm not sure what returning a null response to HomeKit would do, but it sounds like a possible solution if you want to give it a go and submit a PR.

From reading the specification, it looks like it only supports float values from 0-100 :(
We should say there is a "fault" if the value comes back 0, but Home.app (at least on iOS 11) doesn't support displaying this. There is also a "tampered" status, but that isn't supported either.
Looks like there isn't anything we can do about this :(


Fix typo on openscapes jsonnet

This didn't have any effect on the deployed cluster. I
verified this with:

jsonnet openscapes.jsonnet -y > openscapes.kops.yaml
Remove ... at end of the file
kops replace -f openscapes.kops.yaml
kops rolling-update cluster. This has no effects, so it was already
the correct setup.

This has no effects, so it was already the correct setup.

Great, approving and merging now.


Remove and de-duplicate tags with leading/trailing whitespace

Description

We have some records in our data where there are duplicate tags, only the duplicate tag has leading or trailing whitespace. Here's an example: https://api.openverse.engineering/v1/images/2d454032-0cc1-48a5-8f40-e9235f1a4f12/
"tags": [
        {
            "name": "abigfave",
            "accuracy": null
        },
        {
            "name": " abigfave",
            "accuracy": null
        },
        {
            "name": "artisanal",
            "accuracy": null
        },
        {
            "name": " artisanal",
            "accuracy": null
        },
        {
            "name": "color",
            "accuracy": null
        },
        {
            "name": " color",
            "accuracy": null
        },
        ...
    ],

This might need to be tackled in two steps, or a least an operation which covers both cases:

Remove leading/trailing whitespace from existing tags
Deduplicate tags which might previously have been separate ones

We will also want to check, similar to #1566, that any new tags added always have extra whitespace stripped.
Additional context

Related to #430

@WordPress/openverse-catalog I'd like to take a shot at this issue. Am I correct to assume this should use the batched update DAG?

If possible, it'd be best to combine both of those steps into a single batched update, that way we don't have to do two passes on the data! Might make for a tricky query, but then we only have to run it once 😄

I guess select distinct trimming_function(tag.name), or something along those lines would work?
Thanks for the input, y'all.

Reopening for the pending execution of the trim_and_deduplicate_tags DAG.

Solved in #4557.


🚧 Add simpler Unfold overload

This PR adds a simpler overload of Unfold that requires only a single function instead of 3. Example usage:
var xs = MoreEnumerable.Unfold(1, s => s <= 100 ? (true, s * 2, s) : default);
Console.WriteLine(string.Join(", ", xs));

Prints:
1, 2, 4, 8, 16, 32, 64

Pending work:

[ ] Add unit tests

How is this different than MoreEnumerable.Generate(1, s => s *2).TakeWhile(s => s <= 100)?

How is this different than MoreEnumerable.Generate(1, s => s *2).TakeWhile(s => s <= 100)?

That specific example is not, but Unfold anyway is Generate + TakeWhile + Select rolled into one. If TState and TResult are identical then Select can be dropped. Historically, Generate was there first, back in 2009. Unfold was added later in 2017. Either can be written in terms of the other.
For comparison, below is a slightly more involved example that generates the Fibonacci sequence where all numbers are less than 100. Using Generate + TakeWhile + Select:
var fibonacciNumbersLowerThan100 =
    MoreEnumerable.Generate((Curr: 0, Next: 1), s => (s.Next, s.Curr + s.Next))
                  .TakeWhile(s => s.Curr < 100)
                  .Select(s => s.Curr);

Using Unfold:
var fibonacciNumbersLowerThan100 =
    MoreEnumerable.Unfold((Curr: 0, Next: 1),
                          s => (s.Curr, Next: (Curr: s.Next, Next: s.Curr + s.Next)),
                          s => s.Curr < 100,
                          s => s.Next,
                          s => s.Curr);

Using the overload proposed here:
var fibonacciNumbersLowerThan100 =
    MoreEnumerable.Unfold((Curr: 0, Next: 1),
                          s => s.Curr < 100 ? (true, (s.Next, s.Curr + s.Next), s.Curr) : default);

nit: for this example, this would be cleaner as it does not rely on a ternary or default
var fibonacciNumbersLowerThan100 =
    MoreEnumerable.Unfold((Curr: 0, Next: 1),
                          s => (s.Curr < 100, (s.Next, s.Curr + s.Next), s.Curr));

That said, I'm curious to whom you see this as a potential benefit.

I don't see this version of Unfold (or even the original one) as simpler than Generate().TakeWhile().Select(); on the contrary, I'd argue that Generate() is more advantageous because it relies on composability which is both more flexible and more consistent in reading.
I'm not sure that I see this version of Unfold as using less lambdas, because the implementation requires three lambdas, so it is still a four-lambda solution in the end.
I might be able to buy a performance argument, but I'd have to see a benchmark to confirm the validity.

nit: for this example, this would be cleaner as it does not rely on a ternary or default

Yes, but with the ternary (for illustration) and more generally, you avoid wasteful computation (like even s.Curr + s.Next) if not needed because the break condition's been met.

That said, I'm curious to whom you see this as a potential benefit.

People used to unfold from other languages and/or runtimes/libraries (such as someone used to Seq.unfold in F#) will find it more aligned and natural.

I don't see this version of Unfold (or even the original one) as simpler than Generate().TakeWhile().Select(); on the contrary, I'd argue that Generate() is more advantageous because it relies on composability which is both more flexible and more consistent in reading.

It's not trying to be simpler than Generate + TakeWhile + Select. It's overloading the existing Unfold with a simpler version.

I'm not sure that I see this version of Unfold as using less lambdas, because the implementation requires three lambdas, so it is still a four-lambda solution in the end.

The implementation is immaterial as it can change so the point I was makign is how many lambdas the developer has to author.

I might be able to buy a performance argument, but I'd have to see a benchmark to confirm the validity.

There's no performance argument being made for now.


Max # of policies that can be created?

Team,
Is it possible to document whether or not there are limitations on the maximum number of policies that can be created by Teams admins (whether that be an overall max limit, or max limits per specific types of policies)?
Thank you
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 905e2c09-ef3e-07cc-0809-6ff171157c47
Version Independent ID: c0d33162-183d-fff8-32e9-a999f391ea51
Content: Limits and specifications for Microsoft Teams - Microsoft Teams
Content Source: Teams/limits-specifications-teams.md
Service: msteams
GitHub Login: @MicrosoftHeidi
Microsoft Alias: heidip

Hello @davidjfriedman David, I'm trying to get this information, any special use case where you hit a limit?
Thanks

Nothing unique--just a question that has come up so figured worth checking to see if it could be documented.  Thank you

"New-CsTenantDialPlan:  Maximum[1000] tenant Dial plans number limitation is reached for this tenant. Remove any unused tenant Dial Plans and try again..."

"New-CsTenantDialPlan: Maximum[1000] tenant Dial plans number limitation is reached for this tenant. Remove any unused tenant Dial Plans and try again..."

That one is documented here https://learn.microsoft.com/en-us/microsoftteams/what-are-dial-plans

@davidjfriedman I was given the information that there is no max number of policies


Update dependencies

Hi,
I have remove the Node 6 Test. And adding Node 10 (current LTS) and 12 (next version).
This seem to be a good choice because Node 6 is outdated now.
Thanks.

@joecohens

Thank you @Brouilles !


Updating reporting status view

Hi @brockfanning
Would love to update our reporting status page as the UK has done, showing only reported and exploring (Green and Red status).  Let me know how to incorporate this change.
thanks!
Dawn

See this line:
https://github.com/dawncomer/open-sdg-data-starter/blob/7a549cd3d451e17ae257e7e6b96c60d6d02261db/_prose.yml#L155
It has 4 items right now. You could delete 2 of them.

yes! this worked, thanks @brockfanning


Remove more test requirements

Description:
This continues my quest of minimizing which requirements are installed during our test runs. Went through all the tests that require us to install dependencies and looked if I quickly could alter them so that we no longer need the dependency. Either by mocking out the dependency or methods that use that dependency.
Checklist:
If the code does not interact with devices:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] Tests have been added to verify that the new code works.

Current dev:

After this cleanup:

Result: non conclusive.
Oh well, still an improvement.


Local Server points to localhost when trying to download artifacts

I have a running local instance in my PC with Docker. wandb local and everything works fine: people can connect to it from other machines by pointing to my IP, accessing to the web service and using Python to upload data to a run.
The only thing that is not working is when trying to download an artifact (with several files), the execution either gets blocked or it just downloads a 1-byte file. Same when trying to download any file of an artifact from the web dashboard: it tries to download the file by pointing to localhost:8080/ so it cannot resolve it.
From my local PC, everything works: Both web and Python scripts.
Thanks in advance.

  You're also risking data loss by not licensing and connecting external MySQL and S3 storage.  All that being said, you likely want to update the Host setting at /system-admin to point to your external IP instead of localhost.


Error when using expr predicate in has clause

The following semantically identical change to the in-hospital mortality example leads to an error.
# Task: 24-hour In-hospital Mortality Prediction
predicates:
  admission:
    code: event_type//ADMISSION
  discharge:
    code: event_type//DISCHARGE
  death:
    code: event_type//DEATH
  discharge_or_death:
    expr: or(discharge, death)

trigger: admission

windows:
  input:
    start: NULL
    end: trigger + 24h
    start_inclusive: True
    end_inclusive: True
    has:
      discharge_or_death: (None, 0)
      _ANY_EVENT: (5, None)
    index_timestamp: end
  gap:
    start: trigger
    end: start + 48h
    start_inclusive: False
    end_inclusive: True
    has:
      admission: (None, 0)
      # discharge: (None, 0)        # <- remove
      # death: (None, 0)
      discharge_or_death: (None, 0) # <- add
  target:
    start: gap.end
    end: start -> discharge_or_death
    start_inclusive: False
    end_inclusive: True
    label: death

I came across this while trying to create the MEDS-DEV in-ICU mortality task for AUMCdb, which uses the shortcut above.
This is not high priority but good to fix because it took me a while to figure out where in the config the error originated from.

This is tracked in #105 and I'm fixing it currently. Closing as a duplicate.


OCPBUGS-13532: Reconcile when networkData Secret changes

Update the baremetal-operator dependency to pick up https://github.com/openshift/baremetal-operator/pull/296 for the PreprovisioningImage controller.

/retest-required

/lgtm


Python bindings for replay only

Python bindings without Qt
I'm looking for a way to replay a single frame from a python script and dump the performance counters for each draw call into a csv file. Essentially automating the file generation.
I've tried to build the python bindings of renderdoc for a few hours on Linux and ended up giving up.
The 2 dependencies : Shiboken2 & PySide2 are actually available on Ubuntu but a couple of issues with CMake not finding those dependencies (actually I fixed that locally) and the Shiboken package not shipping a .pc file for the python3 bindings make this essentially unusable, unless you're will to build those 2 dependencies yourself (there was another issue where it was mentioned that the pip packages are not an option either).
I'm no expert in python binding but it seems pyside2 at least is used for the UI binding.
Would making bindings just for non UI bits of renderdoc be possible without those 2 dependencies?

I think there's some kind of misunderstanding here, I don't know if it's on my side or your side.
PySide2 (and shiboken) are used for exposing the Qt widgets in the UI bindings to python. This allows you to do e.g. pyrenderdoc.GetMainWindow().Widget().setWindowTitle("I am making a Qt call to set the title") and then you set the Qt main window title to that string. All of Qt5 is exposed via those bindings when available.
If PySide2 isn't available at compile time, then the return of Widget() there is an opaque handle. So it's only useful for e.g. manipulating the docking window state etc with other qrenderdoc module functions that accept the opaque handle and unbox it on C++ side. It's not possible to have the Qt widgets fully accessible on scripting without PySide2 or some equivalent python Qt bindings library.
If you don't have PySide2, cmake will print a message like "PySide2 not found - Qt will not be accessible in python scripting." to indicate that you won't be able to access Qt in the python scripting, but the python bindings will be built as normal otherwise. On linux the standalone python module will be placed in the lib/ output folder, and if you open qrenderdoc the python shell will work as normal aside from the limitations on accessing Widget() as above.
For your purposes can you not just use the replay bindings without requiring access to the Qt widgets? Either directly with the python module and command line python, or else with the UI scripting access just running normal replay commands. I guess I'm not understanding whether you want Qt scripting access or not - if you do then PySide2 is required, and if you don't then I don't understand what the problem is exactly.

Yeah I'm not interested in manipulating any Qt widget. Maybe I got confused...
I was expected a module directory to be created and that I should point the python path to that.
Since it's not there I thought I was missing a build dependency.

The python module is built by default, and is only disabled if ENABLE_PYRENDERDOC is turned off in cmake. There's no module directory, library is output as lib/renderdoc.so. I'm not sure where module would come from.

Thanks, sorry for the noise.


[Feature] Creating Placeholders for Text

Feature Description
I want to add placeholders for a variety of different text sizes and types.
Reason for Change
This change is required to show the user how each font looks in the different text sizes and types so that they may have a more informed opinion regarding deciding a font over another in their own project.
Where are the changes needed?
I will have to create new files for the components required to achieve this.
Code Snippets
No response
Anything Else?
No response

I haven’t been in Figma for a few days. Have we got a template design yet? Regardless, seems sensible; I’m happy for you to gets started.
I suppose we can just default to system-ui, sans-serif or some sensible font stack like that—until Jace has drag-and-drop-a-font feature in place.


Create LICENSE

@FlyingBird95 @vandriko  @kloostert  - do you people agree with using MIT? Or you have a strong reason for going with a viral license?

Fine with me...
On Feb 7, 2018 20:49, "Mircea Filip Lungu"<EMAIL_ADDRESS>wrote:

@FlyingBird95 https://github.com/flyingbird95 @vandriko
https://github.com/vandriko @kloostert https://github.com/kloostert -
do you people agree with using MIT? Or you have a strong reason for going
with a viral license?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/flask-dashboard/Flask-Monitoring-Dashboard/pull/47#issuecomment-363888911,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJkLZEGJ8WLJEyDInq_lk-C6VclXydoaks5tSf4_gaJpZM4RpbEd
.

MIT is also fine with me

Yes, same here.


How to get CUB-DG dataset?

Hi, I really like this job! I am so interested in the CUB-DG dataset. How could I get it? Thanks!

Thanks for your attention! We plan to provide a better release version (along with CUB-DG dataset) in Mid March.


chore: Add new domain abstraction

This pulls out the domain changes of #85

This PR pulls out the domain from kzg since it is also needed for kzg_multi and removes kzg specific methods that were on domain and just made them functions instead. We also added a CosetFFT abstraction since the erasure_code package uses it and kzg verification needs it


Quasar: [Vue warn]: Failed to resolve component: [Quasar component name]

Laravel Vite Plugin Version: 0.7.3
Laravel Version: 9.48
Node Version: 18.12.1
PNPM Version: 7.26.1
Host operating system: Windows 10
Web Browser & Version: Chromium 109
Running in Sail / Docker: No

Description:
I just migrated from the community plugin and I am getting errors in the console from Vue, in the form of:
[Vue warn]: Failed to resolve component: QLayout
If this is a native custom element, make sure to exclude it from component resolution via compilerOptions.isCustomElement.
[stack trace goes here]

Note that after running pnpm build, it works (although the CSS is messed up).
It used to work before, but it's now broken. Does anyone know what's up?
Steps To Reproduce:
vite.config.ts
import inertia from './resources/client/plugins/inertia-layout'
import laravel from 'laravel-vite-plugin'
import vue from '@vitejs/plugin-vue'
import { defineConfig } from 'vitest/config'
import { quasar, transformAssetUrls } from '@quasar/vite-plugin'

export default defineConfig({
  test: {
    include: ['resources/client/**/*.test.ts'],
  },
  plugins: [
    inertia(),
    quasar({
      autoImportComponentCase: 'pascal',
    }),
    vue({
      template: { transformAssetUrls },
    }),
    laravel({ input: 'resources/client/main.ts', refresh: true }),
  ],
  resolve: {
    alias: {
      '@': '/resources/client',
    },
  },
})

Simple Vue component:
<script setup lang="ts">
import type { QBtnProps } from 'quasar'

interface TooltipButtonProps extends QBtnProps {
  text: string
}

defineProps<TooltipButtonProps>()
</script>

<template>
  <QBtn color="deep-orange" push round>
    <QTooltip>
      {{ text }}
    </QTooltip>
  </QBtn>
</template>

Hi there,
Thanks for reporting but it looks like this is a question which can be asked on a support channel. Please only use this issue tracker for reporting bugs with the library itself. If you have a question on how to use functionality provided by this repo you can try one of the following channels:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

However, this issue will not be locked and everyone is still free to discuss solutions to your problem!
Thanks.


Add force orientation support

I am not sure that whether all users need this feature. But, in my situation, I surely need this. So, I took the liberty of writing this PR.

Back to Phaser 2, we have a feature call force orientation, which is described at here. I think Phaser 3 should have this feature, too.
Why we need force orientation?
When a game is running in general browser (for example, not in Cordava), and it should be displayed in Landscape Direction. But, when the user locks his / her phone in Portrait Direction, the game will be displayed like this following current policy of ScaleManager:

Above condition isn't what we want, apparently.
If user want to play this game in a better layout, he / she should disable orientation lock of phone, and rotate the phone is physical world, then the game should be displayed like this:

That's what I want to say: In order to play a game which is in Landscape Direction and display it in better layout, user have to disable orientation lock in Phone's OS and rotate Phone by themselves.
What is the policy should be?
When Phone's orientation lock is enabled
For a game should be displayed in Landscape Direction - online demo:

For a game should be displayed in Partrait Direction - online demo:

Why not use ScaleManager#lockOrientation?
This API is implemented via Screen.lockOrientation(). But, as MDN said:

This feature is no longer recommended. Though some browsers might still support it, it may have already been removed from the relevant web standards, may be in the process of being dropped, or may only be kept for compatibility purposes. Avoid using it, and update existing code if possible; see the compatibility table at the bottom of this page to guide your decision. Be aware that this feature may cease to work at any time.

In most of browser, it just call a depracated API without doing any other useful thing.
Current thought about how to implement force orientation
Step 1: add a toggle
Add a toggle for the policy of force oriencation - GameConfig#forceOrientation.
Step 2: implement the policy
Get expected layout of game according game.config.width / game.config.height:

when game.config.width >= game.config.height, ScaleManager#isGameLandscape is true.
when game.config.width < game.config.height, ScaleManager#isGamePortrait is true.

Get current layout of screen via ScaleManager#isScreenLandscape / ScaleManager#isScreenLandscape.

Currently, ScaleManager#isScreenLandscape / ScaleManager#isScreenLandscape is simple and crude, i will improve it later.

Compare ScaleManager#isScreenLandscape / ScaleManager#isScreenLandscape with ScaleManager#isGameLandscape / ScaleManager#isGamePortrait to determine that if Phaser should:

rotate the canvas.
remap input onto a rotated canvas.

Current progress
Complete.

When input works, clicking green point in demo will print 'Oh!' in browser's console.
And the input is broken when the canvas is rotated.

@photonstorm After reading the code of InputManager, I still have no idea where is the best place to remap input. Maybe you can give me some tips. ;)

Update: Found the way to remap input onto a rotated canvas, working on it.

remap input onto a rotated canvas is done.

Lots of people email me about this PR, so I wrote following comment.

After I wrote the PR, I realized the PR is too big, so I closed it and try another method to extend Phaser.
Patched Phaser
At that time, Phaser's version is 3.16.2. I patch it with external SceneManager support(which controls orientation in Phaser). The code is in custom branch with these two commit:

feat: add support for using external ScaleManager
fix: deactive Pointer when its identifier is not matched

Patched Phaser will use custom SceneManager when scaleManagerClass is set in the configuration.
You can modify current version of Phaser by referencing these two commits.
Custom SceneManager
After patching Phaser,I created a custom SceneManager provides the orientation lock at here.
You can create your own SceneManager by referencing above path.
Last
At that time, Phaser's WebGL performance isn't well on some Android device. Under the pressure of project schedule，I fallback to Pixi.js, and the idea of extending Phaser is put aside.
Maybe, the performance problem is solved now. You can give it a try.
And if you like, give it back to the community. Good Luck!


Typehint for loader/film.py

Checklist
GitHub

[x] I've given this PR a concise, self-descriptive, and meaningful title
[x] I've linked relevant issues in the PR body
[x] I've applied the relevant labels to this PR
[x] I've assigned a reviewer

PR contents

[x] I've consulted ivadomed's internal developer documentation to ensure my contribution is in line with any relevant design decisions
[x] I've added relevant tests for my contribution
[x] I've updated the relevant documentation for my changes, including argparse descriptions, docstrings, and ReadTheDocs tutorial pages

Description

add type hinting for loader/film.py
Linked issues

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

12 of 15 (80.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.01%) to 68.839%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

ivadomed/loader/film.py
12
15
80.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.01%

Covered Lines:
4098

Relevant Lines:
5953

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

6 of 13 (46.15%)  changed or added relevant lines in 1 file are covered.
47 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.02%) to 71.89%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

ivadomed/loader/film.py
6
13
46.15%

Files with Coverage Reduction
New Missed Lines
%

ivadomed/loader/film.py
47
21.48%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
4097

Relevant Lines:
5699

💛  - Coveralls


Weird scaling issue on 2.0.0 - maybe 4k related

I just installed the new version and everything's scaling randomly when moused over, screenshot : http://i.imgur.com/VeGbLMJ.jpg
maximizing or restoring the window resets everything for a second but it soon rescales weirdly.
I uninstalled the 2012 VC++ redist I installed to use v1.9.2, uninstalled the 2013 redist included with 2.0, uninstalled and reinstalled the software, nothing helps.
I'm using GPMD on a 4k display, but it worked correctly with the previous version.
also, thanks a lot for this - using it every day :)

Same issue on a 1920x1080 display - maybe it has to do with windows display scaling? I have mine set at 125%.

Same issue here on a surface pro 2 (1920x1080)

I'll try look into this later today, but I'll ping @amaitland who might know if this is a CEF bug?

@joelpereira Do you have display scaling turned on?

 Disabling windows high DPI scaling on the .exe properties solves it, but the previous versions scaled just fine on higher res displays.

The temporary fix for you guys is this

on a 15.6" 4k display it's way too small to be usable, so I reverted to 1.9.2 =)

Clicking the "Disable display scaling on high DPI settings" fixed the weird display issues, but now things are smaller for my fat fingers. =)
Thanks!

@RemyFV @joelpereira If it is now too small for you to use you can use the shortcut "Ctrl +" to zoom in.  Not quite the same as DPI scaling but I think it will have the same affect till I can fix this.
Your zoom level is saved on exit aswell so its not going to be annoying :+1:

4K display w/ 150% scaling
x64
symptom:
scrolling with wheel, display seem correct scale.
but as soon as mouse is moved, scale enlarges,

@erolfox Have you tried enabling that compatabilty mode option?

@MarshallOfSound where?
did try 100% scaling on Win10, same scaling issue

https://github.com/MarshallOfSound/Google-Play-Music-Desktop-Player-UNOFFICIAL-/issues/125#issuecomment-169097043

ah, windows compatibility
that takes care of the flashing and scaling dynamically. now just static too large (see pic)

Now try the zooming trick in this comment
https://github.com/MarshallOfSound/Google-Play-Music-Desktop-Player-UNOFFICIAL-/issues/125#issuecomment-169118970
You can use "Ctrl +" and "Ctrl -" to zoom in and out

@MarshallOfSound You may need to add a call to Cef.EnableHighDPISupport(), it's a new method introduced in 45.0.0
https://github.com/cefsharp/CefSharp/blob/cefsharp/45/CefSharp.Core/Cef.h#L458
Should probably be called in Program.cs, somewhere early on, like https://github.com/MarshallOfSound/Google-Play-Music-Desktop-Player-UNOFFICIAL-/blob/master/Google Play Music/Program.cs#L24

Thanks @amaitland I'll get a build running now so these guys can test it and I can push out a fix ASAP

@RemyFV @eddielement @joelpereira @jwfitt @FuzzAU @eturk1
Can you guys all please try this beta build
64 Bit: https://mega.nz/#!LFdCgQwD!aRcAtkrVgXnbw2Klu6tu2hfWZ2VMoWP_r33EdJJZC1o
32 Bit: https://mega.nz/#!2YkT0DCa!VHDZG3W2LDruQR95-6AWBkkc8mn1JpisWu1_Lrt4NCw

ctrl +/- scales the base graphics, but it graphics quickly redraw.
move the mouse over app and that causes redraw to double scale
vid of error: https://vid.me/rX2C
pressed ctrl- 5 times then scrolled

@eturk1 Woah, thats actually awful.  Sorry for pushing this out.  I really need a high DPI screen to test though :cry:
Have you tried the beta builds I just posted

When I run the beta build it tells me I need to update, I take it I skip that for now?

Just tried the beta build, looks like the weird flickering stuff I was getting before that etruk illustrated is gone, BUT the UI is not scaling like it used to in 1.9.6

@FuzzAU Yeah, ignore the update.  It's just a bit confused
@jwfitt Is your DPI scaling turned off in the compatibility settings still?  Might want to double check that :+1:

no worries @marshall...
I wrote code for 14 years. (last century LOL). I know what it's like. maybe we have a beta test crew to help out?
fresh beta killded the mad continual redraws. well done!
now we just have the scaling issue. mouseover, scroll, interaction doesn't cause any redraws.
now just the scale :)

Yeah, scaling was enabled in the exe properties, also killed the user data folder the preferences get saved in to be sure.

@eturk1 can you explain some steps that cause your scaling to fail?
I can't reproduce. I've got a 3440x1440

@eturk1  Thats good to hear
I'm assuming that now the zoom does work as a temporary fix for the scaling issues though correct?
@jwfitt Can we try a complete uninstall and reinstall? Sorry for putting you through this but you are basically my guinea pig at the moment :cactus:

Sorry for not mentioning it, but I did a full windows uninstall and cleared my appdata/roaming prefs directory for the program when it didn't appear to be working and then reinstalled. Same results.
Great program btw! Thanks for the efforts!

Wait a sec guys
@jwfitt and @eturk1
If you guys messed around with the zooming shortcuts.  It is very possible that that setting was preserved through an uninstall.  Can you try reseting the zoom with "Ctrl 0" (I think)

win10 x64
4K 3840x2160 @ 150% scaling
@MarshallOfSound , yes, pressing Ctrl- 21 times seems to get it close. ;)
w/ some mad tiny text

Never messed with the ctrl +/- scaling
It's the core UI as you can see in the main control bar at the top of the program:

did an uninstall with iobit. should remove any reg entries.
reinstalled and everything is tiny as @jwfitt
so, scale of UI and draw surface are similar.

I think I know how to fix the UI scaling.  But not sure about the scaling inside Google Play Music.  I'm actually at work still so I'll look into this in a few hours when I get home.  Everything I've done so far is through SSH :+1:

@RemyFV @eddielement @joelpereira @jwfitt @FuzzAU @eturk1
Got another beta build coming through
You will have to uninstall the old version before installing this one
I didn't increment the version
Can you guys please try this
64 Bit: https://mega.nz/#!jNFyXZBK!A-q0HxNPwbnbp_S2Yd7HQH1zMG0TGrfM4tIHsBZebWM
32 Bit: https://mega.nz/#!TENWQKTD!CeSkvf3muv4A8kMR-s-HhcvAnHA4Pm66g1pc-dcm5bo

latest build unchanged. still tiny interface.

@eturk1 Have you reset the zoom?

Yeah, still mini for me - uninstalled and manually wiped all folders like before. Same tiny UI :/

@eturk1 @jwfitt
Another build, this one might look a bit funky
64 bit: https://mega.nz/#!KE9yRQBa!nOKdudFUn7FpZZCd621JovNZ1QNz0CsaGL3oKopi1cA

Afraid not, same results on clean install

@jwfitt Can you double check compat settings and everything, Cos it definitely looks different on all 3 computers I have access to

Double-checked:

Can I see a screenshot of the player

Main player controls seem ok, I was primarily noticing the core menu bar being microscopic:

Did the previous bets look like that as well?

Yeah, they have all looked essentially the same as far as I could tell. 1.9.6 was the last version where all the UI scaling being handled by windows seemed to work normally with the program.

OK, so I'm removing the breaking bug tag as it appears to be working properly at the moment apart from the top bar being a bit small.  Pushing out 2.0.1 to handle the buggy DPI thing

okay I just woke up, so I did not try anything before now. I downloaded and installed the last link you posted here @MarshallOfSound and the scaling seems to work correctly : http://i.imgur.com/qqjt3gS.jpg
the app started up sized for 1/4th the screen (correct for 1080p), but resizing the window didn't break anything and zoom in/out/reset works perfectly like it would in chrome.
the only thing that doesn't resize is the 3 buttons block in the top right windows title bar, but it's really not important.
in that state, for me, the bug is mostly solved and the software perfectly usable - win 10, 4k screen resolution, no compatibility options, windows base scaling at 200%.
I can give you remote access to my laptop on a clean testing session if you want as well, Ammyy or Teamviewer...

@RemyFV thanks for the detailed report, that clarified what I suspected.  I'm about to push a pre-release and will link it here when I do.  Could you test that in a few minutes and then I'll make the release public.
The remote thing is a nice offer but maybe another time, I'm heading off for the night soon :+1:

still one last funny bug in that scaling thing : http://i.imgur.com/aymtWmR.jpg
this is the maximum size of the mini player now, it feels all alone and lost in a cruel, cruel desktop x)
anytime for the remote access, thanks for your work!

Lol, I suspected that would be the case with the mini player because I use hard coded pixel values for it.  I'll have to interpret the DPI manually with that.

@RemyFV Just published the pre release, could you take a look :+1:

downloaded, installed, works exactly the same for me (no scaling issue that is, but functionally not different from the megaupload file)  - I even get the "high DPI breaking update" warning at start. should I uninstall everything before reinstalling or is it normal ?

The DPI warning will be there until I make this release final :)

and one additional thing : until the mini player can be resized there is no other way to close it and go back to the large UI than killing the entire app since the controls are over the restore button :(

@RemyFV I just updated the pre-release with a fix that I think will fix the mini player issue.  Could you test them ASAP so I can push out this release.
Cheers

nope, exactly the same behavior - I can shrink it to almost nothing but the max size is 1/4th what it should be on a 4k display (if you keep the same ratio), controls extend outside the box and over the restore button.

@RemyFV  did you uninstall the old one first?  You need to completely remove the old ones for these debug builds

no I didn't, so I did it now - everything works :)
zoom level reset at start and everything was very small but ctrl+ works to resize to whatever you like, and scaling works on the mini player as well. is the maximum size limit really needed though ?
maybe you should put those zoom keys in the startup warning once so people know and are not stuck with a very small UI.

Will do, releasing now

@RemyFV Can you do one final test of a version for me.  I just put out a pre-release of 2.0.2

uninstalled, downloaded, reinstalled - first start with a HUGE scale (~200% of your new base scale apparently), ctrl-0 reset to a usable size which seems to be the same scale as on a 1080p resolution.
mini player controls are huge as well, and resizing the box does not affect them.

So to clarify @RemyFV
Initial Load everything was far too big (This was due to your previous zoom value)
After resetting the zoom everything was correct
Mini player controls are correct or incorrect??

Initial Load everything was far too big (This was due to your previous zoom value) After resetting the zoom everything was correct

yes.

Mini player controls are correct or incorrect??

incorrect, attaching screenshot soon.

and actually key combinations work on the mini, ctrl-0 reset to standard size - window might not have been active when I tried the first time... sorry about that, wasting your time :(

Can we ensure that resizing the mini player doesn't screw with the sizing?

just tried it, reopening the mini player and resizing the window goes back to the huge scaling even if I reset the zoom in between every action.
the main window does not display this behavior and keeps zoom level consistent between app launchs, though.

@RemyFV Fix incoming on the 2.0.2 pre-release

everything works fine with the last 2.0.2 release :)
any address I can use to paypal you a beer or two ?

There is a paypal button on the README if you want to :+1:
:beer:

@RemyFV @eddielement @eturk1 @joelpereira @FuzzAU @yopspanjers @FuzzAU
Hey guys,
I think I have completely nailed this fix now.  UI Scaling on all components seems to be working.
Check out the builds below:
x86 Build
x64 Build
Hopefully these builds work perfectly :+1:


Enable strict null check for ./vs/base/test/common/history.test.ts

#65233

[ ] "./vs/base/test/common/history.test.ts"

Not sure how to fix this. yarn strict-null-check
https://github.com/Microsoft/vscode/blob/f903e6d226f953e158deb397de871fd34dce1979/src/vs/base/test/common/history.test.ts#L116-L126
result.push(historyNavigator.current());

result is defined as a type string[]
historyNavigator.current() is defined as type <T> | null, in this case string | null.
I'm thinking of changing the types for HistoryNavigator.<method> back to just return T and not T | null
Thoughts ?

Decided to change
string[] to Array<string | null>


Fixed bug in splitting headers

First, thanks for all your work on Purescript and its libraries.  I benefit greatly from your work.
I found and fixed a nasty bug this morning...
xhr.getAllResponseHeaders() returns the headers separated by CRLF, not just LF.
See here and here.
Presently, purescript-affjax splits only on \n.  See [here] (https://github.com/slamdata/purescript-affjax/blob/master/src/Network/HTTP/Affjax.js#L64).
Ordinarily, it wouldn't be a problem.  But when your workflow involves extracting an 'X-Auth-Token' from the header of an AJAX response, and then sending that token value as a header value of a new AJAX request, the new request will error out if the header value contains CR (or LF).

Thanks!


update content

Hi. First, great package!
I have a small problem:
If I create a new folder on the physical server (Windows 2012 Server) in the root or in another subfolder and populated with files, I can see the folder and files (in browser) only if I remove the cookies for the site and refresh again!
How can I configure your project to update at every access the list of folders or files?

This behaviour is only on Firefox. I will use Google Chrome instead.

The same problem reappears. Not updated content even in Chrome.

This sounds like browser caching, what browser are you using? Sometimes when I don't see changes on the page immediately I'll "force reload" the page using Ctrl+ Shift + R (in Firefox).
Various ways of force reloading from other browsers can be found here.

@PHLAK
I hoped to find with Laravel code this kind of udapte. Thanks, anyway!


Inconsistent downloading from Twitter

Checklist

[x] I'm reporting a broken site support
[x] I've verified that I'm running youtube-dl version 2019.11.05
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar issues including closed ones

Verbose log

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'-a', u'/home/Joseph/Documents/plainText/ytdltw.txt', u'--no-part', u'--restrict-filenames']
[debug] Batch file urls: [u'https://twitter.com/i/status/1180524380985679872']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2019.11.05
[debug] Python version 2.7.17 (CPython) - Linux-5.3.8-200.fc30.x86_64-x86_64-with-fedora-30-Thirty
[debug] exe versions: ffmpeg 4.1.4, ffprobe 4.1.4
[debug] Proxy map: {}
[twitter]<PHONE_NUMBER>985679872: Downloading webpage
[twitter:card]<PHONE_NUMBER>985679872: Downloading webpage
[twitter:card]<PHONE_NUMBER>985679872: Downloading guest token
[twitter:card]<PHONE_NUMBER>985679872: Downloading JSON metadata
[twitter:card]<PHONE_NUMBER>985679872: Downloading m3u8 information
[debug] Default format spec: bestvideo+bestaudio/best
[debug] Invoking downloader on u'https://video.twimg.com/ext_tw_video/1180524198667653125/pu/pl/672x848/oFbQ_db8z8Eep9kH.m3u8'
[hlsnative] Downloading m3u8 manifest
[hlsnative] Total fragments: 47
[download] Destination<EMAIL_ADDRESS>ERROR: unable to download video
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 2018, in download
    url, force_generic_extractor=self.params.get('force_generic_extractor', False))
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 807, in extract_info
    return self.process_ie_result(ie_result, download, extra_info)
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 900, in process_ie_result
    new_result, download=download, extra_info=extra_info)
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 862, in process_ie_result
    return self.process_video_result(ie_result, download=download)
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 1643, in process_video_result
    self.process_info(new_info)
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 1930, in process_info
    raise UnavailableVideoError(err)
UnavailableVideoError: [Errno 36] File name too long<EMAIL_ADDRESS>

Description

Downloading from Twitter works intermittently. Seems that youtube-dl doesn't like file names that are "too long".

Duplicate of which Issue?

For what it's worth, I discovered that forcing a smaller file name for the destination file works OK.

For what it's worth, I discovered that forcing a smaller file name for the destination file works OK.

How?

Downloading from Twitter can cause a too-lengthy file name. When that happens, try again with using the --id switch, which suppresses the video title from appearing in the destination file name.


Offline-First Considerations: index.html caching

If using CloudFront you should not cache index.html.
Otherwise an old index.html might get populated in the cache
for a new service-worker.js. That does not get recovered
until a new service-worker.js is deployed.
Step-by-step:

create-react-app sw-cache-problems ; cd sw-cache-problems
npm run build
mv build cloud-front-simulation
perl -pi -e 's/To get started/BANANA/g' src/App.js
npm run build
serve -s cloud-front-simulation/  # And keep it running while continuing below
Go to http://localhost:5000 in e.g. Google Chrome and observe 'To get started' appearing on the page
Close the browser
mv build/service-worker.js cloud-front-simulation/service-worker.js   # We wait with index.html, pretending it is still cached in CloudFront
cp -r build/static cloud-front-simulation/
Go to http://localhost:5000 again
You will notice in Console "New content available; please refresh" so do that. However, you still get the old index.html of course
Close the browser
mv build/index.html cloud-front-simulation/index.html    # To simulate CloudFront cache experied and it fetched the new version
Go to http://localhost:5000 again
If you press F5 you get the old site (with 'To get started'). If you press Shift+F5 you get the new site (with 'BANANA') since you bypass SW . That's because the sw cache was populated while index.html was still old. And now we're stuck like this until a new service-worker.js is deployed

Nevermind, it's tricky to succinctly but accurately describe the problem, let's just hope users know what they are doing ;)


docs(config) : Add enabledWasmLoadingTypes and wasmLoading for output

fix #3964

[x] Read and sign the [CLA][1]. PRs that haven't signed it won't be accepted.
[x] Make sure your PR complies with the [writer's guide][2].
[x] Review the diff carefully as sometimes this can reveal issues.
[x] Do not abandon your Pull Request: [Stale Pull Requests][3].

Thanks!


Cucumber tests not running Build Error

Step1:  Run the automated tests Cucumber with this command "mvn clean install -DbrowserProfile=chrome -Denv=default -Dcucumber.options="--tags @demo" -P acceptance-test@
Expected Result: Tests are starting in Browser
Actual Result:  error

Have you followed the getting started steps from the wiki? https://github.com/NTTDATA-UK/cinnamon/wiki/getting-started

are you kidding me bro, let me know if you can run them

Looks like you are running an old version (0.1.2), please try again using the latest version from master and confirm if you are still having the issue


Exception when I try to download a local file.

I get the following exception whenever I try to download data or metadata from a local datasource. I don't have any cloud datasources configured. Only a local folder.
INFO:     <IP_ADDRESS>:40004 - "GET /api/datasources/local/local/bpsk_rect_20sps.sigmf-data HTTP/1.1" 500 Internal Server Error
ERROR:    Exception in ASGI application
Traceback (most recent call last):
File "/home/nix/.local/lib/python3.12/site-packages/uvicorn/protocols/http/h11_impl.py", line 406, in run_asgi
result = await app(  # type: ignore[func-returns-value]
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/nix/.local/lib/python3.12/site-packages/uvicorn/middleware/proxy_headers.py", line 70, in call
return await self.app(scope, receive, send)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/nix/.local/lib/python3.12/site-packages/fastapi/applications.py", line 1054, in call
await super().call(scope, receive, send)
File "/home/nix/.local/lib/python3.12/site-packages/starlette/applications.py", line 123, in call
await self.middleware_stack(scope, receive, send)
File "/home/nix/.local/lib/python3.12/site-packages/starlette/middleware/errors.py", line 186, in call
raise exc
File "/home/nix/.local/lib/python3.12/site-packages/starlette/middleware/errors.py", line 164, in call
await self.app(scope, receive, _send)
File "/home/nix/.local/lib/python3.12/site-packages/starlette/middleware/exceptions.py", line 65, in call
await wrap_app_handling_exceptions(self.app, conn)(scope, receive, send)
File "/home/nix/.local/lib/python3.12/site-packages/starlette/_exception_handler.py", line 64, in wrapped_app
raise exc
File "/home/nix/.local/lib/python3.12/site-packages/starlette/_exception_handler.py", line 53, in wrapped_app
await app(scope, receive, sender)
File "/home/nix/.local/lib/python3.12/site-packages/starlette/routing.py", line 756, in call
await self.middleware_stack(scope, receive, send)
File "/home/nix/.local/lib/python3.12/site-packages/starlette/routing.py", line 776, in app
await route.handle(scope, receive, send)
File "/home/nix/.local/lib/python3.12/site-packages/starlette/routing.py", line 297, in handle
await self.app(scope, receive, send)
File "/home/nix/.local/lib/python3.12/site-packages/starlette/routing.py", line 77, in app
await wrap_app_handling_exceptions(app, request)(scope, receive, send)
File "/home/nix/.local/lib/python3.12/site-packages/starlette/_exception_handler.py", line 64, in wrapped_app
raise exc
File "/home/nix/.local/lib/python3.12/site-packages/starlette/_exception_handler.py", line 53, in wrapped_app
await app(scope, receive, sender)
File "/home/nix/.local/lib/python3.12/site-packages/starlette/routing.py", line 72, in app
response = await func(request)
^^^^^^^^^^^^^^^^^^^
File "/home/nix/.local/lib/python3.12/site-packages/fastapi/routing.py", line 297, in app
raw_response = await run_endpoint_function(
^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/nix/.local/lib/python3.12/site-packages/fastapi/routing.py", line 210, in run_endpoint_function
return await dependant.call(**values)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/nix/source/orgininal_iqengine/IQEngine/api/app/iq_router.py", line 158, in get_iqfile
azure_client.set_sas_token(decrypt(datasource.sasToken.get_secret_value()))
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'NoneType' object has no attribute 'get_secret_value'
I'm starting IQEngine by running "make run-api", "make dev", and "make run-plugins" from separate terminals.
I'm running Fedora 40, with python 3.12.7.

Ah nice catch, will fix now


adding flow config to new project

@brentvatne looks like there is no flow config file generated for project created using xde whereas there is one created using create-react-native command line.
Thoughts ?

Keeping the flowconfig up to date takes more work than might be expected since RN changes a lot and we can't commit to it right now. Adding a flowconfig to a project is easy if someone has one that's up to date and maintained, so if someone wanted to make a repo with a maintained flowconfig then people who want Flow could easily get the config. It's just something that the community & people interested in Flow are best set up to provide.


Mongodb: fix C warnings and avoid crash caused by GC

Everything is in the title.
For more about the crash see issue #1629.

+1 for the workaround

@Morriar, do you plan to update with the proposal from @xymus ?


Why the training prec@1/5 is always 0.00000?

When I run the train.py using the MS1M as training dataset, the Training Prec@1 and Training Prec@5 are always 0.000000. Could anyone tell me why?

your training loss is too large, continue training

did u have solved ?? @sstzal

I have the same problem with you even though my loss around 17 after epoch 20th @@

did u have solved ?? @sstzal

Yes, i have solved the problem. Just wait, and the training loss will fall.

in order to your pre # 0. how many for epoch and loss ?? @sstzal

After about two epoches, the Training Prec@1 and Training Prec@5 will grow slowly. However, I have noticed that you have the problem even after epoch 20th, I think you maybe have some mistakes in your code.

in order to your pre # 0. how many for epoch and loss ?? @sstzal

After about two epoches, the Training Prec@1 and Training Prec@5 will grow slowly. However, I have noticed that you have the problem even after epoch 20th, I think you maybe have some mistakes in your code.

yes, maybe i have to check my code again.
thank u for replying me

although my Prec@1,5 is always 0.0000 but Evaluation is 0.7xxxx @@...
and
Is 6k imgs (100MB) data train not enough ??. hence, my prec@1,5 is always 0.000xx although in 40th epoch @@

@NHDat2 if you train with arcface, you should change the hyperparameter m and s.

I also have this question, and I have already trained 21 epochs. However, the performance on validation set become better and better ...  I use the same dataset which the readme.md mentioned.

arcface loss？ 892379162<EMAIL_ADDRESS>签名由 网易邮箱大师 定制 On 04/10/2020 10:15, zhangxilun wrote: I also have this question, and I have already trained 21 epochs. However, the performance on validation set become better and better ... I use the same dataset which the readme.md mentioned. — You are receiving this because you commented. Reply to this email directly, view it on GitHub, or unsubscribe.

Yes, I use arcface loss. Another key point I need to mention is that I change the embedding size from recommended 512 to 256, because I'd like to use the algorithm in my own mobile device. Maybe the 256-d tensor doesn't have enough feature to represent the differences between different persons?

the hyperparameters m and s should be changed. you can set m=45 s=0.3，train one epoch. if it's ok,the top acc will not always zero. ------------------ 原始邮件 ------------------
发件人<EMAIL_ADDRESS>发送时间: 2020年4月13日(星期一) 上午10:56
收件人<EMAIL_ADDRESS>抄送: "reverseSystem"892379162<EMAIL_ADDRESS>主题: Re: [ZhaoJ9014/face.evoLVe.PyTorch] Why the training prec@1/5 isalways 0.00000? (#93)
Yes, I use arcface loss. Another key point I need to mention is that I change the embedding size from recommended 512 to 256, because I'd like to use the algorithm in my own mobile device. Maybe the 256-d tensor doesn't have enough feature to represent the differences between different persons?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

I had the same issue.
I have researched the issue and found that with cross entropy loss the precision do improved.
Changing of focal.py
from:
class FocalLoss(nn.Module):
    def __init__(self, gamma = 2, eps = 1e-7):
        super(FocalLoss, self).__init__()
        self.gamma = gamma
        self.eps = eps
        self.ce = nn.CrossEntropyLoss()

    def forward(self, input, target):
        logp = self.ce(input, target)
        p = torch.exp(-logp)
        loss = (1 - p) ** self.gamma * logp
        return loss.mean()

to:
class FocalLoss(nn.Module):
    def __init__(self, gamma = 2, eps = 1e-7):
        super(FocalLoss, self).__init__()
        self.gamma = gamma
        self.eps = eps
        self.ce = nn.CrossEntropyLoss(reduction='none')

    def forward(self, input, target):
        logp = self.ce(input, target)
        p = torch.exp(-logp)
        loss = (1 - p) ** self.gamma * logp
        return loss.mean()

have resolved (at least, for me) the issue, and now the precision is improved also with FocallLoss

@ReverseSystem001 Can you show me where the hyperparameters m and s?

I have the same issue, but I don't have enough GPU. I trained on google colab and had to reduce the number of images from the dataset (train on  casia-maxpy-clean dataset with 401 image folder).
Top1 and Top5 doesn't change after 29 epoches.
has anyone a solution for this problem?

Evaluation is 0.7xxxx

I have changed the m and s hyper parameters. Changed self.ce = nn.CrossEntropyLoss(reduction='none'). Yet precision not increasing.


Proxy persist proxy ports

Persist proxy ports in /var/run/cilium/state/proxy_ports_state.json. This file is written each time we would upsert a TPROXY iptables rule and restored on restart. If the file does not exist proxy ports are restored from iptables rules instead. This is needed for upgrade compatibility.
Earlier commits in this series fix related issues in proxy port handling that are mainly meaningful for restoration with daemonset Envoy proxy:

Clear proxy port on failure so that we'll try another random port next time
Rename AllocateProxyPort as AllocateCRDProxyPort, simplify prototype
Remove "localOnly", it is already always true

The last two commits introduce the main changes:

Restore all proxy ports from iptables rules (rather than only DNS proxy port)
Persist proxy ports in /var/run/cilium/state/proxy_ports_state.json

Since this mainly enhances use of daemonset Envoy proxy by reducing Listener churn on Cilium agent restarts and daemonset Envoy is enabled by default only from Cilium 1.16, there is no need to backport these changes.
Note: builds on https://github.com/cilium/cilium/pull/32824 (first 4 commits), have it merged first!

/test

/test

/test

/test


Request: Implement access controls allowing a user to request certs for any subdomain of ${DOMAIN}

Enhancement Proposal
Hi,
I've run into some (at least to me) unexpected behaviour with this charm that I think we can improve for a better overall experience.
It's been established that if a user is allowed to request certs for example.com, this implicitly grants access to request a wildcard cert for *.example.com as the challenge domain is the same for either (see Canonical internal chat link).
What this doesn't do is allow that user to submit requests for any subdomain-specific certs, e.g. test.example.com.
Could we please look into either:

Leaning into this implicit access, authorising the user with access to example.com to request a cert for any valid domain matching ^.*\.example\.com$.
As we've established that granting access to *.example.com alongside example.com currently does nothing, change the behaviour so that granting access to *.example.com is a method of explicitly allowing the user to request a specific cert for any subdomain of example.com.
Adding some other method of explicitly allowing the user to request a certificate for any subdomain of example.com, such as juju run httprequest-lego-provider/0 allow-domains --string-args username=example-user domains='example.com' subdomains='example.com' or allow-subdomains --string-args username=example-user parent-domains='example.com'.

Keen to hear your thoughts.
Thanks!

After thinking on this some more I'm actually inclined to suggest that the implicit behavior of allowed-domains=example.com allowing a user to request the wildcard cert for *.example.com should be considered a bug in violation of the principle of least privilege, and should be replaced with a mechanism to explicitly grant access for requesting wildcard certs only where needed and justified.

@arturo-seijas Can you have a look, please?

After thinking on this some more I'm actually inclined to suggest that the implicit behavior of allowed-domains=example.com allowing a user to request the wildcard cert for *.example.com should be considered a bug in violation of the principle of least privilege, and should be replaced with a mechanism to explicitly grant access for requesting wildcard certs only where needed and justified.

Hi @jsimpso, what is that you have in mind and how would it look like from a user's perspective?

@jsimpso friendly ping

Apologies for the delayed response.

After thinking on this some more I'm actually inclined to suggest that the implicit behavior of allowed-domains=example.com allowing a user to request the wildcard cert for *.example.com should be considered a bug in violation of the principle of least privilege, and should be replaced with a mechanism to explicitly grant access for requesting wildcard certs only where needed and justified.

Hi @jsimpso, what is that you have in mind and how would it look like from a user's perspective?

On the topic of wildcard certificates, I think that if we grant access to example-user to request certificates for example.com:
juju run httprequest-lego-provider/0 allow-domains --string-args username=example-user domains='example.com'

The permission granted should be limited what is explicitly stated: example-user should be allowed to request a certificate for "example.com".
If we want to allow that user to request requests for wildcard certificates, that should be an explicit action. From the perspective of the administrator making that access request, I could see this working one of two ways:

Requiring the string "*.example.com" to be allowed in order to generate a wildcard certificate for *.example.com

juju run httprequest-lego-provider/0 allow-domains --string-args username=example-user domains='*.example.com'

Requiring a separate argument altogether to control access to generate wildcard certificates:

juju run httprequest-lego-provider/0 allow-domains --string-args username=example-user wildcard_domains='example.com'

On the topic of subdomains, I think it would be useful to have an access control option allowing users to request any specific subdomain of a given domain rather than using a wildcard certificate. For example:

Allowing example-user to request certificates for any specific subdomain of example.com, but not a wildcard certificate, and not a certificate for *.example.com itself.

juju run httprequest-lego-provider/0 allow-domains --string-args username=example-user subdomains='example.com'

Allowing example-user to request certificates for example.com, any specific subdomain of example.com, but not a wildcard certificate for *.example.com:

juju run httprequest-lego-provider/0 allow-domains --string-args username=example-user domains='example.com' subdomains='example.com'

Hopefully that makes sense, please let me know if not! Thanks :)

@gregory-schiano , what do you think?


Selecting Repository->Git LFS->Initialize immediately crashes

I have tested this on both LInux and Windows.
Repro:
No repository open
Select Repository -> Git LFS -> Initialize
App immediately exits

Hi @MarcelInTO which version did you use?

I am using v1.3.0
No, if I have a simple repo, the exit does not happen.  It only seems to happen if I have no repository open.
The reason I did that command without a repo open was that I was about to clone a repo that I knew used LFS but I was on a new machine and did not think that I had done a "git lfs install" yet to setup the smudge filters in my .gitconfig file. I figured that this command in Gittyup would do that. That might have been incorrect thinking - though even if it is not meant for what I attempted to use it for, it probably should not exit.