(Proposal) Fix is/isnot expr of vim9script

This issue is Reported by @thinca
@kat0h made this patch
issue
is/isnot expr of Vim9 script are looks broken.
This is reproduction code.
vim9script
echo 'foo' is 'foo'    # => false
echo 'foo' is 'bar'    # => false
echo 'foo' isnot 'foo' # => true
echo 'foo' isnot 'bar' # => true

var a = 'foo'
echo a is a            # => false
echo a isnot a         # => true

cause
Vim script uses typval_compare_string() function in src/typval.c to compare strings.
in Vim9 script context, it checks if the pointers of the two values passed as arguments equal.
But, LOAD command of assembly pushes value to stack after copy it. so values are placed different address.
I think, It seems that string value is not an object. So, I think vim can't compare that the string instances are identical.
vim9script
var a = 'foo'
var b = a
a = 'bar'
echo b #=> 'foo'

solution
I deleted code which checks that string instances are same.
This patch includes breaking change. Now, null_string is '' returns true, and null_string isnot '' returns false.
regards.

This issue is Reported by @thinca
@kat0h made this patch
issue
is/isnot expr of Vim9 script are looks broken.
This is reproduction code.
vim9script
echo 'foo' is 'foo'    # => false
echo 'foo' is 'bar'    # => false
echo 'foo' isnot 'foo' # => true
echo 'foo' isnot 'bar' # => true

var a = 'foo'
echo a is a            # => false
echo a isnot a         # => true

cause
Vim script uses typval_compare_string() function in src/typval.c to compare strings.
in Vim9 script context, it checks if the pointers of the two values passed as arguments equal.
But, LOAD command of assembly pushes value to stack after copy it. so values are placed different address.
I think, It seems that string value is not an object. So, I think vim can't compare that the string instances are identical.
vim9script
var a = 'foo'
var b = a
a = 'bar'
echo b #=> 'foo'

solution
I deleted code which checks that string instances are same.
This patch includes breaking change. Now, null_string is '' returns
true, and null_string isnot '' returns false.

This choice was made quite a while ago.  Nearly everywhere a string is
copied before it is passed on, thus "is" would nearly always return
false.  Using that as a reason to fall back to comparing the string value
doesn't make sense either, now two completely unrelated strings may have
"is" return true, just because they happen to have the same value.
One can argue for one or the other.  In a case like this I rather keep
it as it is.  Changing it now is likely to cause trouble in a few
places, and I don't see it helping much.
Another way to look at it: like you mention, strings are not objects,
thus "is" always returns false.
Yet another way would be to give an error when trying to use "is" on
strings.  But that likely causes even more trouble.
A very complicated way to solve this would be to use reference counting
for strings.  Thus intead of making a copy the strings would indeed be
identical.  The reference counting is needed to make sure the string is
deleted if, and only if, the last reference to it is dropped.  It might
be a bit more efficient to avoid making copies, but then keeping the
reference counts correct adds overhead (which should not be
underestimated).
--
Facepalm reply #9: "Speed up, you can drive 80 here" "Why, the cars behind us
are also driving 60"
/// Bram Moolenaar -- @.*** -- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

Thank you very much for checking my PR!

@brammool
This behavior is different from legacy scripts, but this is not documented.
I believe this should be explained in the documentation.
:help vim9-differences
:help convert_legacy_function_to_vim9
:help expr-is

I thought it was mentioned somewhere, but I can't find it now.
I'll add a note.
--
Why doesn't Tarzan have a beard?
/// Bram Moolenaar -- @.*** -- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///


[Icon Request] Support for Bats

[x] I'm sure this issue is not a duplicate.
[x] I'm submitting an icon request:

Type: extension
Icon Name: bats
Sample original Icon: :bat:
Extensions: .bats
Shebang: #!/usr/bin/env bats

Language Extensions: jetmartin.bats
More info: https://github.com/bats-core/bats-core

I am Batman! 🦇


export OS_* defines (fixes #715)

Moved OS_* defines to a separate public header and added a GLOG_ prefix.

@sergiud can we release a new version to fix this?

Please see the corresponding milestone. The date can change though.


case parsing

Hi,
I run into problems parsing this kind of case/when
case "a:b"
  when "a:b"
    p a:b
  default
    p default

The error is expected "indent", but got "filter", but really it should handle "a:b" as a string literal - right?
This work around is a bit of a kludge, but at least it works:
-var test = "a:b";
case "a:b"
  when test
    p a:b
  default
    p default

Someone needs to check if this can be reproduced on the latest lexer/parser and add a test case to github.com/pugjs/pug-lexer

lexer reports this error:
Error: Pug:2:12
    1| case "a:b"
  > 2|   when "a:b"
------------------^
    3|     p a:b
    4|   default
    5|     p default

unexpected text ""
   "
    at makeError (/home/timothy_gu/pug/pug-error/index.js:32:13)
    at Lexer.error (/home/timothy_gu/pug/pug-lexer/index.js:52:15)
    at Lexer.fail (/home/timothy_gu/pug/pug-lexer/index.js:1254:10)
    at Lexer.advance (/home/timothy_gu/pug/pug-lexer/index.js:1314:15)
    at Lexer.callLexerFunction (/home/timothy_gu/pug/pug-lexer/index.js:1269:23)
    at Lexer.getTokens (/home/timothy_gu/pug/pug-lexer/index.js:1325:12)
    at lex (/home/timothy_gu/pug/pug-lexer/index.js:12:42)
    at repl:1:13
    at REPLServer.defaultEval (repl.js:252:27)
    at bound (domain.js:287:14)

Closing. See pugjs/pug-lexer#57 instead.


Relative path config.py

Describe the bug
Index.py fails when app starts from the home path. If I manually change the absolute path in the config.py file it works fine.
Debug logs
Traceback (most recent call last):   File "/home/minipc/syncarr/index.py", line 11, in <module>     from config import (   File "/home/minipc/syncarr/config.py", line 152, in <module>     sync_bidirectionally = int(sync_bidirectionally) ValueError: invalid literal for int() with base 10: ''

I'm not sure what you mean by 'home path'. Can you give an example of what commands you are calling and what you are changing to fix it?

fixed in v1.9.1

fixed in v1.9.1


define $DNS_JSON in the same way as $PROVISIONER_JSON

23794e55b started referring to DNS_JSON without ever defining it, so apparently this ability to override DNS settings during install via /{etc,root}/crowbar/dns.json has been broken for 3 years and 3 days ...
Another example of why #shellsucks - undefined variables are silently used without any error or even warning.

+1

@aspiers you can make shell bail out on undefined variables by setting "-u".

@aspiers I'll close, feel free to reopen if you disagree

The bug is there, but not for the reason I describe in this PR.  If you look at the commit this PR refers to, you'll see that I wrote the code you linked to, so it's somewhat embarrassing / amusing that I had forgotten about that and misunderstood the source of the bug.  The problem is that DNS_JSON gets defined too late, just a few lines after it is used.
So we still need to fix this, but it will be cleaner to submit a new PR.  That said, I've reopened this one as a reminder that we need to submit a new PR.


chore: enable qlik eslint config updates

:tada: This PR is included in version 2.2.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Not working: {{#autoform ..  template="bootstrap3-horizontal" label-class="col-sm-3" input-col-class="col-sm-9"

In one of the examples you have this, but this doesnt actually work :
{{> quickForm id="demo" schema=schemaFromJSON type="method" meteormethod="demoSubmission" template="bootstrap3-horizontal" label-class="col-sm-3" input-col-class="col-sm-9"}}
I am using, and it does not work, it will show the labels above the textfields, and ignore the extra rulls after the type="update" :
{{#autoform collection="Keys" doc=this type="update" template="bootstrap3-horizontal" label-class="col-sm-3" input-col-class="col-sm-9"}}

It works for quickform, but you're using an autoform. They don't have theme templates so the template="bootstrap3-horizontal" doesn't mean anything in your case. Instead you should put the template="bootstrap3-horizontal" label-class="col-sm-3" input-col-class="col-sm-9" on each afFormGroup within your autoform.

I am a little confused as to why there is 2 versions, a {{#autoform}} and a {{#quickform}}

The autoForm is the basic form wrapper in which you can design your own form. The quickForm renders the form for you based on the fields in the schema, using a theme template.


Fix argocd-redis-ha service account name

What is this PR About?
As per https://github.com/redhat-cop/helm-charts/pull/191/files#r704790946
This PR reverts the service account name in the clusterrolebinding to use {{ .Values.name }} instead of .namespace
cc: @redhat-cop/day-in-the-life

@jameseck ; why do you want to revert? am not sure I understand what this changes gives/doesnt. it just changes the name of a resource.

The name of the serviceaccount that is created from the helm chart (which is the subject of the clusterrolebinding) is {{ .Values.name }}-argocd-redis-ha and not {{ .Values.namespace }}-argocd-redis-ha.
The comment by @ckavili confirms that the service account subject reference should be using {{ .Values.name }} as it did before PR 191.


[EventRequest] SalesPost.Codeunit.al (codeunit 80 "Sales-Post")

Hi MS,
Please help us in migrating our verticalized solution by adding below Event in SalesPost.Codeunit.al Codeunit.
Standard Code

Modified Code

Event Signature

Thanks
SUD

If you use AL code copied from VSCode instead of screenshots it will speed up the process of implementation.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


[Bug] allow the last comment in a file follow previous indent

What happened?
Robocop reports following as [W] 1007 Line is over-indented (uneven-indent), but I think it should be allowed. The issue happens only when comments are placed on the last line of the file.
*** Test Cases ***
Test
    [Documentation]  doc
    Keyword

*** Keywords ***
Keyword
    [Documentation]  this is doc
    No Operation  # todo: do something
    # g

However, also this should be ok, like it is currently:
*** Test Cases ***
Test
    [Documentation]  doc
    Keyword

*** Keywords ***
Keyword
    [Documentation]  this is doc
    No Operation  # todo: do something
    # g

# EOF

Operating System
Linux
Robocop version
2.3.0

You're right. over/under indented rules are bit complicated right now - I've actually started some improvements (for #504 - to allow for strict checks). I will take a look if it's something I can also fix/improve.

I have actually fixed it without touching the logic - the problem lied in small (stupid) mistake with wrong range :) There will be the change to the uneven-indent (strict checks instead of "fuzzy" one right now) but it's separate topic for this fix - which will be included in the next release.


_auth: finish porting sha256_password and caching_sha2_password

Unlike the other authentication methods, sha256_password_auth and
caching_sha2_password_auth are effectful, and call conn.write_packet() and
conn._read_packet().
To do this most straightforwardly, these functions need to become async
functions, so they can await conn.write_packet and conn.read_packet.

Looks straightforward enough... feel free to merge once CI passes, as far as I'm concerned.
Do you want to add a test?

https://github.com/PyMySQL/PyMySQL/pull/682/files#diff-a99307c61c3c55875b5bb65e0b045ccd074e277afe3cca9665e7dbd33023daf9 should provide coverage for this; I haven't looked at the CI configuration on this repository--I suppose those tests aren't being run?

I'm just going to merge this due to how simple the change is; the (numerous) unrelated CI issues should be fixed as part of #8 . Coverage is also already included.


wrc104 Handle S3 403

Adds handling of Access Denied errors from s3 backend and passes them back in a response.
Needed by https://github.com/fjelltopp/who-romania-ckan/issues/104
Testing
Tested with who-romania-ckan deployment, it is build as image v0.5.3c
Checklist
Put an x in the boxes that apply to this pull request (you can also fill these out after opening the pull request).
You may not need to check all boxes.

[ ] The Jira ticket for this issue has been updated to "Ready to Review" or equivalent.
[ ] I have developed these changes in discussion with the appropriate project manager.
[ ] My code follows the general Fjelltopp documentation (see Confluence).
[ ] I have made corresponding changes to the Fjelltopp documentation (see Confluence).
[ ] I have rebased this branch with master.
[ ] New dependency changes have been committed.
[ ] I have added automated tests that prove my fix is effective or that my feature works.
[ ] New and existing tests pass locally with my changes.
[ ] My changes generate no new warnings.
[ ] I have performed a self-review of my own code.
[ ] I have assigned at least one reviewer.
[ ] I have assigned at least one label to this PR: "patch", "minor", "major".

@mixmixmix - are we dropping py3.7 support? What have they done upstream with that?

They moved to 3.12.... The docker image in this repo uses 3.7, but our image in fjelltopp-base-images uses 3.8.

they changed the way how project is built among other things, but there are not changes in documentation - so it is unfinished & out of data in the upstream repo.

They moved to 3.12.... The docker image in this repo uses 3.7, but our image in fjelltopp-base-images uses 3.8.

Let's update the image in this repo to 3.8 and remove the 3.7 tests then? I think that should really be part of this PR so it doesn't just hang forever

Why part of this PR? It was broken before and not sure there is any point in trying to tidy it up now.

done!


Automate the process of adding new users to the organization

Edit:
@AllanRegush found a github action ActionsDesk/invite-user that invites new users

I read the discussion here https://github.com/EddieJaoudeCommunity/EddieJaoudeCommunity.github.io/issues/35#issuecomment-684839794
And i found we can use github octokit to invite new members to the organization.
There is two functions, i'm not sure which one we should use 😁

Create an organization invitation
Set organization membership for a user

We can create a GitHub actions that triggers on new issues and add the issue author if the issue has a specific title
for example: add me to the organization

Thanks @alrifay 👍

This github action might not fit our needs. Consider forking the project and making the action query the api for missing information. i.e. email

We still need this, I remove the no-issue-activity label

This is still relevant stale bot

This is still relevant stale bot

This has been done in #718 - still needs testing

This has been done in #718 - still needs testing

This works 👍 I am going to close

This works 👍 I am going to close


Window Title

I there a way to change the window title from the running script?

No, the window title is hardcoded to be the name of your application. You can change the title by changing the name of the application.


[C++] Make ChunkResolver::ResolveMany output a list of ChunkLocations

Describe the enhancement requested
Instead of two indexes buffers. This helps with cache locality and performance of kernels using it (so far, only the non-merged "array_take" implementation for chunked arrays).
Component(s)
C++

Issue resolved by pull request 43928
https://github.com/apache/arrow/pull/43928


Cannot connect to the DB when it is on another host.

Checklist
Yes I am using the latest version.
Describe the bug
The database connection is refused when the containers are separate.
To Reproduce
I need to run the app on a docker macvlan so I have adjusted the example to be like so:
version: "3"
services:
  app:
    image: jc21/nginx-proxy-manager:latest
    restart: always
    networks:
      br0:
        ipv4_address: <IP_ADDRESS>
    ports:
      - 80:80
      - 81:81
      - 443:443
    volumes:
      - ./config.json:/app/config/production.json
      - ./data:/data
      - ./letsencrypt:/etc/letsencrypt
    depends_on:
      - db
    #environment:
    # if you want pretty colors in your docker logs:
    #- FORCE_COLOR=1
  db:
    #image: mariadb:latest
    build: .
    restart: always
    networks:
      br0:
        ipv4_address: <IP_ADDRESS>
    environment:
      MYSQL_ROOT_PASSWORD: "root"
      MYSQL_DATABASE: "npm"
      MYSQL_USER: "npm"
      MYSQL_PASSWORD: "password"
    volumes:
      - ./data/mysql:/var/lib/mysql

networks:  br0:
    external: true

This is the Dockerfile I am using for the DB:
FROM mariadb:latest

LABEL<EMAIL_ADDRESS>
RUN apt-get update && apt-get upgrade -y

# UPDATE BIND ADDRESS TO ALLOW REMOTE ACCESS (USE A FIREWALL TO MANAGE TRAFFIC)
RUN sed -Ei 's/#bind-address\s+=\s+<IP_ADDRESS>/bind-address=<IP_ADDRESS>/g' /etc/mysql/my.cnf

EXPOSE 3306

And here is my config.json:
{
  "database": {
    "engine": "mysql",
    "host": "<IP_ADDRESS>", // db when using networks (below)
    "name": "npm",
    "user": "npm",
    "password": "password",
    "port": 3306
  }
}

I also tried putting the both containers on a local bridge network and then connecting to the DB like usual (without adjusting the bind-address) same issue.
Expected behavior
I am confident that at least one of these methods should have allowed a remote connection, I based this directly on the MariaDB docs https://mariadb.com/kb/en/library/configuring-mariadb-for-remote-client-access/
Operating System

Unraid 6.6.7

Additional context
I can't think of anything else, but let me know if you can think of anything.
docker-compose version 1.23.2, build 1110ad01

Okay, as it turns out I wasn't waiting long enough for the database server to start, on a positive note if anyone needs to access MariaDB remotely like me the attached Dockerfile works :).

Okay, as it turns out I wasn't waiting long enough for the database server to start, on a positive note if anyone needs to access MariaDB remotely like me the attached Dockerfile works :).

Which attached Docker file? The Docker file in your comment has a lot of invalid syntaxes.

@LeehamElectronics If you want to access the MariaDB have a look at https://github.com/jc21/nginx-proxy-manager/discussions/1529#connect-to-mariadb-dbeaver, that will probably help you.

@LeehamElectronics If you want to access the MariaDB have a look at https://github.com/jc21/nginx-proxy-manager/discussions/1529#connect-to-mariadb-dbeaver, that will probably help you.

Thanks foe the clarification, how were you able to access the NPM mariaDB remotely? I have tried exposing ports to it in my docker compose file but the port doesn't show as open. Here is my docker compose file:
services:
  app:
    image: 'jc21/nginx-proxy-manager:latest'
    ports:
      - '80:80'
      - '81:81'
      - '443:443'
    environment:
      DB_MYSQL_HOST: "db"
      DB_MYSQL_PORT: 3306
      DB_MYSQL_USER: "npm"
      DB_MYSQL_PASSWORD: "passss"
      DB_MYSQL_NAME: "npm"
    volumes:
      - ./data:/data
      - ./letsencrypt:/etc/letsencrypt
    restart: always
  db:
    image: 'jc21/mariadb-aria:latest'
    ports:
      - '400:3306'
    environment:
      MYSQL_ROOT_PASSWORD: 'passss'
      MYSQL_DATABASE: 'npm'
      MYSQL_USER: 'npm'
      MYSQL_PASSWORD: 'passss'
    volumes:
      - ./data/mysql:/var/lib/mysql
    restart: always

@LeehamElectronics That's exactly how it should work. What database tool are you using? And you can't connect using port 400?


Add charles-rest here

Add project charles-rest to this repository and wrap charles-github-ejb inside it.

@charlesmike hi there!!!

Hi, @amihaiemil! I can help you index your Github-hosted website, provided that the repository name respects the format author.github.io and the command is given by the repo's author.

done


Fixed the priority values for syslog levels

The syslog levels (debug, info, notice, warning, error, crit, alert) defined by winston.config.syslog.levels appear to be reversed. For example, when the syslog level is set to notice, then notice, warning, error, crit, and alert messages should be displayed. However, the reverse is happening: debug, info, and notice are displayed and the others are not.
The following test code illustrates this error:
var winston = require('winston');

// Uncomment the next line to fix the bug
// winston.config.syslog.levels = { debug: 0, info: 1, notice: 2, warning: 3, error: 4, crit: 5, alert: 6, emerg: 7 };

winston.setLevels(winston.config.syslog.levels);
winston.level = 'notice';

winston.debug('This should NOT be logged');
winston.info('This should NOT be logged');
winston.notice('This should be logged');
winston.warning('This should be logged');

This pull request fixes the logging levels for winston.config.syslog.levels so it behaves correctly.
Note: The default logging levels for Winston (silly, debug, verbose, info, warn, error) work perfectly. It's only the syslog levels that are defined backwards.

Note: The build failure on Travis CI is not related to this pull request. This same failure occurs on the current tip of master (commit 2064bfe3b0f3da77c798d248a1609dbc24ff077e) from which I forked. However, this pull request is still valid and works. FYI

This is a known issue that should be fixed very shortly. It is resolved in<EMAIL_ADDRESS>(#732) via https://github.com/winstonjs/winston/commit/fb9eec08f97dc84b66501a377cfdeb9c75e293f7


Zipkin-specific observer support

Naive approach to adding support for zipkin-specific observers.
Merging two interfaces does not seem to be viable, either type-safety- or performance-wise.
closes #156

Fixed newly-introduced golangci-lint complaining regarding repeating string constants in test files in last force push.
There is still two complaints from upstream remaining, regarding parseTagsAsZipkinOptions cyclomatic complexity and benchmarkWithOps's numItems parameter always being 0.

Cleaned up implementation a bit.

Added more test-cases.

Removed redundant FinisherWithDuration interface, as zipkin.Span already provides that method.
This in turns makes sure that FinishWithOptions() cannot return without actually finishing the span.
Added SetRemoteEndpoint in SetTag() as well as OnSetRemoteEndpoint() for ZipkinSpanObserver.


Field added through designer not accessible through extension

AL Extension Version: 0.16.19969
Version: 11.0.23019

A field was added through the Object designer (including custom and ISV)

Downloaded symbols (published/installed other extensions)
The added field is not accessible through AL. Other extension fields are accessible.

Is there an option that allows for "custom" field access?

Please check if EnableSymbolLoadingAtServerStartup is enabled in your Server Settings. Read more here:
https://docs.microsoft.com/en-us/dynamics-nav/developer/devenv-running-cside-and-al-side-by-side

That solved it! Thank you!


tiltfile: custom host in port forwards

Fixes #2013.
If this gets merged, I can create a PR to update the docs as well.

The way to specify host is with a string like '<IP_ADDRESS>:3000:3000', which I believe should be standard.

ok, sgtm. Two minor points:

re: the hostname regexp, you'd be surprised how hard this is to get right! See: https://github.com/golang/go/issues/17659

re: ya, I don't think the relative pros vs cons of *string vs string are that big, it's more important that the codebase is internally consistent. going to to change it to a string for consistency.

thanks for the addition!

@nicks

Interesting, thanks for the link!
That's fair.

Thank you for your prompt review and merge!


Stop overloading backend_name in statevector simluators

Summary
This commit fixes an issue when using the statevector simulators where
accessing the backend_name attribute of a results object would return a
bound method instead of the actual string for the backend name. This was
caused by the statevector simulators were overloading the backend_name
property of its parent classes with a pointer to the name() method. If
this was necessary it should have overloaded the value and called name()
instead of setting the method as backend_name. However this isn't even
necessary because backend_name is assigned automatically already. So
this commit just removes the unnecessary overloads.
Details and comments
Fixes #1116

I've removed the testing because things are failing for unrelated reasons in the clifford simulator: https://travis-ci.org/Qiskit/qiskit-terra/jobs/442859915#L1694
I'll work on adding the tests as a separate commit after this merges.


Cornerstone3D and Vite

I have been attempting to utilize Cornerstone3D in a Svelte web app which uses Vite. Unfortunately, when I try to import @cornerstonejs/core into the app, the build fails.

Note: If I import things from @cornerstonejs/core/dist/umd/index everything compiles and works fine, but I cannot utilize the TS types.
Reproduction steps:

Generate new Svelte app using Vite (npm create vite@latest)
npm install
npm install @cornerstonejs/core (etc...)
npm run dev
Above error is thrown

I've created a repo with nothing but a fresh Svelte app and Cornerstone3d to demonstrate the problem:
https://github.com/ty-ler/cornerstone3d-svelte-vite-repro

Thanks for the reproduction repo!
I was actually also wondering what we should be using for the main/module etc fields in package.json for Cornerstone. We currently output three things: UMD, CommonJS, and ESM.
My understanding is that we need UMD for browser-based script-tag usage, so I put that as the default main.
I was under the impression that ESM was not widely supported yet, but was the intended future format, so I put that as module and also pointed the types to the index.d.ts in the ESM folder.
I left CommonJS in there in case someone needed it for something.
We could potentially switch main to the ESM version and then add the browser property to point to UMD instead (https://docs.npmjs.com/cli/v8/configuring-npm/package-json#browser). You could try doing this, rebuilding, and then using yarn link to see if this resolves your issue. If it does, and there's no other downsides I'd be happy to merge it.
My only hesitation with switching to ESM as main is that we are currently using the ESM output generated by TypeScript, and it does not generate valid ESM modules, since they are missing the .js in their imports (see this thread https://github.com/microsoft/TypeScript/issues/40878). It doesn't look like the TypeScript team plans to fix this issue, so we would probably need to find a workaround for that. One super-ugly workaround that people describe is to use .js while importing from a .ts file - the .js won't get transpiled out and so the result will be valid. I hate the idea of doing that though but haven't had a chance to look for a better alternative.

Hey, thanks for the response!
I tried out several things including changing browser to point to the UMD entry point and changing main to point to ESM, but nothing seemed to solve my issue.
Something I don't quite understand is why the module field uses the .ts extension vs .js. The output files in dist/esm don't actually include a .ts file anywhere (only .d.ts). I assume that some bundlers are able to work out the file name discrepancy using the associated source map file. Though, maybe there's something that I'm just not fully understanding.
I noticed that when I tried switching the module extension to .js, I received a different error:
✘ [ERROR] Could not resolve "lodash.clonedeep"

    node_modules/@cornerstonejs/core/dist/esm/volumeLoader.js:4:22:
      4 │ import cloneDeep from 'lodash.clonedeep';
        ╵                       ~~~~~~~~~~~~~~~~~~

I'm wondering if the lodash.clonedeep imports are changed to a different style of import - would we be able to use the .js extension for module, and not have to change the main or browser fields at all? Lodash offers a few different ways to import the library. When I am using lodash in my own projects, I usually import methods using the lodash/cloneDeep style.
I will do some testing on my own now and update if I make any progress. Just wanted to document my thoughts so far.

So after some testing just now, I believe I have figured out what was causing my issue. It is unrelated to the part about lodash above. The error with lodash showed up after manually editing the package.json for @cornerstonejs/core from inside the node_modules of my Vite project.
What wasn't fully obvious to me was the effect that changing .ts to .js had at build time vs. just changing the extension in the (already built) installed module package.json.
When I changed the extension to .js, rebuilt @cornerstonejs/core, linked, then finally installed the linked package in my Vite project, everything seemed to work great.
I'll create a PR for this change in a few minutes.

FYI I tried to get your repo working and ran into some other issues:
https://github.com/ty-ler/cornerstone3d-svelte-vite-repro/pull/1#issue-1191254076
We'll try to iron these out ASAP. I'm not sure what is wrong with the export of RenderingEngine...

Just to follow up, we tested a webpack setup and didn't have the same issue (RenderingEngine showing up as undefined) that I see in the vite example, so that leads me to conclude it's something wrong with vite or Svelte, and not an issue with the ESM build itself.

In my case it was complaining that it was missing dependency<EMAIL_ADDRESS>So I just did npm -i @kitware/vtk.js and got it to work.


Error when adding LBO interface from controller WebUI.

I'm trying to test LBO capability of NTS when deployed onprems (experience-kit-20.03). However, when adding a breakout interface from the controller WebUI, as suggested in the Preparing set-up for Local Breakout Point (LBP), I get the error in the screenshot and nts stops.

From the nts container running in the node, I can see these error messages:
NES: [ERR] Missing: section PORT2, entry lbp-mac, in config file.
NES: [ERR] Failed to add dataplane rules.
NES: [ERR] Could not initialize nes ACL.
NES: [ERR] Could not initialize ctrl.

Any ideas of what is wrong?
While I try to get the config file, here some references in the code and a more complete log of nts:
https://github.com/open-ness/edgenode/blob/6b6a486b7258078965103cd332285b2e1f8e5b35/internal/nts/daemon/nts/nts_acl.c#L135-L138
https://github.com/open-ness/edgenode/blob/6b6a486b7258078965103cd332285b2e1f8e5b35/internal/nts/daemon/nts/nts_acl.c#L444-L448
$ sudo docker logs --follow nts

Starting to listen on socket /var/lib/appliance/nts/control-socket

NES: [INFO] No "DNS" in config file, DNS agent won't start
NES: [WARNING] DNS agent not working
NES: [INFO] NTS_IO started
NES: [INFO] NES_IO started
NES: [INFO] KNI initialized
VHOST_CONFIG: vhost-user server: socket created, fd: 1061
VHOST_CONFIG: bind to /var/lib/appliance/nts/qemu/usvhost-1
NES: [INFO] Accepted connection on descriptor 1066 (/var/lib/appliance/nts/control-socket)
NES: [INFO] Client disconnected, sock_fd: 1066
NES: [INFO] Accepted connection on descriptor 1066 (/var/lib/appliance/nts/control-socket)
NES: [INFO] Allocating KNI port 0
KNI: pci: 00:00:00 	 00:00
NES: [INFO] New KNI device registered, waiting to be authorized.
NES: [INFO] KNI inteface vEth0 has been created for 2f6396e2f54f7016d2919e32de16524b7b5a86173fcaa236fc5c427c48c5dd6e
KNI DAEMON: [ERROR] kni_docker_daemon(187): Failed to remove the KNI inteface for 2f6396e2f54f7016d2919e32de16524b7b5a86173fcaa236fc5c427c48c5dd6e
NES: [INFO] Client disconnected, sock_fd: 1066
KNI DAEMON: [INFO] kni_docker_daemon(232): vEth0 attached to 2f6396e2f54f7016d2919e32de16524b7b5a86173fcaa236fc5c427c48c5dd6e, namespace[/var/run/docker/netns/1fa92e00f5e3]
NES: [INFO] Accepted connection on descriptor 1066 (/var/lib/appliance/nts/control-socket)
NES: [INFO] Adding authorization entry for MAC: 4a:24:65:d1:f6:21
NES: [WARNING] Overlapping rule: prio:5,enb_ip:<IP_ADDRESS>/0,srv_ip:<IP_ADDRESS>/32
NES: [INFO] Client disconnected, sock_fd: 1066
NES: [INFO] Accepted connection on descriptor 1066 (/var/lib/appliance/nts/control-socket)
NES: [INFO] Removing KNI interface for 2f6396e2f54f7016d2919e32de16524b7b5a86173fcaa236fc5c427c48c5dd6e
NES: [INFO] Removing KNI device 0
NES: [INFO] KNI device 0 removed
KNI DAEMON: [INFO] kni_docker_daemon(240): vEth0 removed from 2f6396e2f54f7016d2919e32de16524b7b5a86173fcaa236fc5c427c48c5dd6e, namespace[/var/run/docker/netns/1fa92e00f5e3]
NES: [INFO] Client disconnected, sock_fd: 1066
NES: [INFO] Accepted connection on descriptor 1066 (/var/lib/appliance/nts/control-socket)
NES: [INFO] No more entries in NTS_ACL_LOOKUP_0
NES: [INFO] Client disconnected, sock_fd: 1066
NES: [INFO] Signal received, exiting now...
EAL: Detected 6 lcore(s)
EAL: Detected 1 NUMA nodes
EAL: Multi-process socket /var/run/dpdk/vhost-1/mp_socket
EAL: No free hugepages reported in hugepages-1048576kB
EAL: Probing VFIO support...
EAL: VFIO support initialized
OVS DAEMON: [INFO] ovs_docker_daemon(277): OVS disabled - shutting down
KNI DAEMON: [INFO] kni_docker_daemon(305): [Started]
KNI DAEMON: [INFO] kni_docker_daemon(306): Waiting for containers events
EAL: PCI device 0000:03:00.0 on NUMA socket -1
EAL:   Invalid NUMA socket, default to 0
EAL:   probe driver: 15ad:7b0 net_vmxnet3
EAL: PCI device 0000:0b:00.0 on NUMA socket -1
EAL:   Invalid NUMA socket, default to 0
EAL:   probe driver: 15ad:7b0 net_vmxnet3
EAL: PCI device 0000:13:00.0 on NUMA socket -1
EAL:   Invalid NUMA socket, default to 0
EAL:   probe driver: 15ad:7b0 net_vmxnet3
EAL: PCI device 0000:1b:00.0 on NUMA socket -1
EAL:   Invalid NUMA socket, default to 0
EAL:   probe driver: 15ad:7b0 net_vmxnet3
NES: [INFO] Could not load config file /opt/intel/nev_sdk/nes_root/scripts/nes.cfg.
NES: [INFO] Port 1 initialized and started
NES: [INFO]
Checking link status for 1 port
NES: [INFO] Port 1 Link Up - speed 10000 Mbps - full-duplex
NES: [INFO] done
NES: [INFO] DPDK port: 1 PCI address: 0000:13:00.0 MAC address: 00:0c:29:0a:6b:46
USER1: rte_ip_frag_table_create: allocated of 402653312 bytes at socket 0
NES: [INFO] Port 2 initialized and started
NES: [INFO]
Checking link status for 2 port
NES: [INFO] Port 2 Link Up - speed 10000 Mbps - full-duplex
NES: [INFO] done
NES: [INFO] DPDK port: 2 PCI address: 0000:1b:00.0 MAC address: 00:0c:29:0a:6b:50
USER1: rte_ip_frag_table_create: allocated of 402653312 bytes at socket 0
NES: [INFO] Port 0 initialized and started
NES: [INFO]
Checking link status for 0 port
NES: [INFO] Port 0 Link Up - speed 10000 Mbps - full-duplex
NES: [INFO] done
NES: [INFO] DPDK port: 0 PCI address: 0000:0b:00.0 MAC address: 00:0c:29:0a:6b:3c
USER1: rte_ip_frag_table_create: allocated of 402653312 bytes at socket 0
NES: [INFO] NIS_IO started
NES: [INFO] Starting nes_ctrl_main
NES: [ERR] Missing: section PORT2, entry lbp-mac, in config file.
NES: [ERR] Failed to add dataplane rules.
NES: [ERR] Could not initialize nes ACL.
NES: [ERR] Could not initialize ctrl.
NES: [INFO] No "DNS" in config file, DNS agent won't start
NES: [WARNING] DNS agent not working
NES: [INFO] NTS_IO started
NES: [INFO] KNI initialized
VHOST_CONFIG: vhost-user server: socket created, fd: 1061
VHOST_CONFIG: bind to /var/lib/appliance/nts/qemu/usvhost-1
NES: [INFO] Lcore 4 stopped

Here the nts.cfg file.
As as you can see, PORT0 does not have a field named lbp-mac nor a mac, like PORT1 and 2 have.
[root@nts-edge ~]# cat /var/lib/appliance/nts/nts.cfg
[PORT0]
name = 0000:0b:00.0
description = VMXNET3 Ethernet Controller
pci-address = 0000:0b:00.0
traffic-type = IP
traffic-direction = lbp
egress-port = 2
MTU = 1500

[PORT1]
name = 0000:13:00.0
description = [ens224] VMXNET3 Ethernet Controller
pci-address = 0000:13:00.0
traffic-type = IP
traffic-direction = upstream
egress-port = 2
mac = 00:0c:29:0a:6b:46
MTU = 1500

[PORT2]
name = 0000:1b:00.0
description = [ens256] VMXNET3 Ethernet Controller
pci-address = 0000:1b:00.0
traffic-type = IP
traffic-direction = downstream
egress-port = 1
mac = 00:0c:29:0a:6b:50
MTU = 1500

[VM common]
max = 32
number = 2
vhost-dev = /var/lib/appliance/nts/qemu/usvhost-1

[NES_SERVER]
ctrl_socket = /var/lib/appliance/nts/control-socket

[KNI]
max = 32

In the hope that it might help, I'm adding a video of my steps https://youtu.be/-zyfry2FoeQ

What is missing is a traffic rule for LBP.
LBP traffic rule should be added to LBP interface.
Traffic rules can be added from interface tab.
For upstream or downstream, there’s no problem to pass it without traffic rule. (the case of LTE mode with GTP traffic, upstream and downstream also require traffic rule from interface tab).
For LBP, there’s no way to set traffic rule if it’s not done from interface tab. So, it’s essential.

@nicolacdnll did you get your issue sorted out?

Yes, adding a traffic rule solved that issue. Thanks for your support!
I have reasons to believe that the Preparing set-up for Local Breakout Point (LBP) guide erroneously suggests that the IP filter for the traffic rule should go in the Source, rather than on the Destination, as shown by this image under the Configuration in Controller section.
In fact, if I do like the image - adding the IP filter on the Source - the controller creates a traffic rule on the ue_ip like this route = prio:99,encap_proto:noencap,ue_ip:<IP>. Shouldn't that be on srv_ip? Thus, shouldn't the IP filter be added on the Destination?
[Since this is slightly OT, if you prefer we can close this and create a new issue.]

Good point @nicolacdnll. The screenshot is misleading and the IP filter should be set in the Source section as you suggested. The image will be updated for the next release.

I just wanted to clear any possible misunderstanding:
I'm saying that the IP filter should be set in the Destination and not on the Source.


Add database index on alter_time column (bsc#1111787)

Add new index to velum_production database on table salt_events on column
alter_time. This will improve cleanings for old salt events and salt returns.

We can close it, there was bug which was fixed in https://github.com/kubic-project/velum/pull/688


v. 0.2.2

@JuliaRegistrator register

Registration pull request created: JuliaRegistries/General/84261
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.2.2 -m "<description of version>" 51b4c4c37c68093c8d6eb8a50ff0a6db6eb0d77f
git push origin v0.2.2


behavior when transition() is called before StateMachine.leaveAnyState()

I defined a onleaveSTATE function that returns StateMachine.ASYNC. The asynchronous part maybe does a long running task. If it does not, it calls transition immedeately. This results in onenterstate to run before onleavestate, which is not the behavior I would expect.

The fix for this is going to require changing the ASYNC code to use a Promise based system (which is planned for upcoming v3)

Async behavior has been changed to use a less error prone Promise based system in v3 - https://github.com/jakesgordon/javascript-state-machine/blob/v3/docs/async-transitions.md


python3Packages.systembridge: 1.2.4 -> 2.0.4

Motivation for this change
Update to latest upstream release 2.0.4
Change log: https://github.com/timmo001/system-bridge-connector-py/releases/tag/v2.0.4

Related Home Assistant change: https://github.com/home-assistant/core/pull/53443
Target Home Assistant release: 2021.8.0

Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase notes)

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

[x] Fits CONTRIBUTING.md.

nixpkgs-review happy, macos 10.15 & nixos x86_64

Result of nixpkgs-review pr 131454 at f5d34191 run on aarch64-linux 1

2 packages built successfully:

python38Packages.systembridge
python39Packages.systembridge

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

2 packages built:

python38Packages.systembridge
python39Packages.systembridge


feat: implement repo suggestions in create insight page

What type of PR is this? (check all applicable)

[x] 🍕 Feature
[ ] 🐛 Bug Fix
[ ] 📝 Documentation Update
[ ] 🎨 Style
[ ] 🧑💻 Code Refactor
[ ] 🔥 Performance Improvements
[ ] ✅ Test
[ ] 🤖 Build
[ ] 🔁 CI
[ ] 📦 Chore (Release)
[ ] ⏩ Revert

Description

Extends on the Repo Suggestions List component
Inserts it in Create Insight Page
Adds 3 static suggestions to it

Related Tickets & Documents

closes #711
Mobile & Desktop Screenshots/Recordings

Added tests?

[x] 👍 yes
[ ] 🙅 no, because they aren't needed
[ ] 🙋 no, because I need help

Added to documentation?

[ ] 📜 README.md
[ ] 📓 docs.opensauced.pizza
[ ] 🍕 dev.to/opensauced
[ ] 📕 storybook
[ ] 🙅 no documentation needed

[optional] Are there any post-deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?

I've noticed that we have 2 almost identical components: suggested-repo and repository-result.


Fix DeleteCurveValuesWorkerTest to work with universal time

#Request Template Witsml Explorer
Fixes # (issue)
Description
Please include a summary of the change and which issue is fixed. List any dependencies that are required for this change.
This test failed on my computer, maybe because I'm running Windows instead of Linux. Changing the test to use universal time works on both operating systems.
...
Type of change
Put an x in the boxes that apply. You can also fill these out after creating the PR.

[x] Bugfix
[ ] New feature (non-breaking change which adds functionality)
[ ] Enhancement of existing functionality
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Impacted Areas in Application
List general components of the application that this PR will affect:

Testing

Checklist:
Put an x in the boxes that are fulfilled.

[ ] PR is related to an issue
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have used descriptive naming on components, functions and variables to avoid additional explanatory comments in the code
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

I think for this pull request the most important part is to just get the tests up and running. I will use the extension instead as it does the same as my code so we only need to change the extension if we want to improve upon that in the future.
Expanding on the extension to allow for local time and UTC offsets is a good idea, but should maybe wait until it is going to be used.


383 | Ransom Note | CPP

Title -
RANSOM NOTE
What will change -
Type of Issue -
.

[x] Adding New Code

Programming Language

[x] C++

Self Check

Ask for issue assignment and wait to get assigned to it before making Pull Request.
Add your file in the proper folder
Clean Code and Documentation for better readability
Add Title and Description of the program in the file

:star2: Star it :fork_and_knife:Fork it :handshake: Contribute to it!
Discord server ✉ - https://discord.gg/CXksGdn
Happy Coding,

Please assign this issue to me.

Interested in contributing . Kindly assign this issue to me

@pstreemaze I assigned you to #1120
I'll assign this to @rematchka for now.
@charles-vj you may select other issues you want to work on.

@pstreemaze I assigned you to #1120
I'll assign this to @rematchka for now.
@charles-vj you may select other issues you want to work on.

I wanted all three issues for hacktoberfest, but nevermind


added ability to fire refresh manually, deinit fixed

+1

+1

I added the fixed code in my project it is working fine , but it start pull from a corner I want to show it from centre how can I do that.


rename Result variants Ok/Err to Pass/Fail

I was never too happy with the existing naming approaches for the Result type:
Success/Failure

pro: both names have the same length, like Option's Some/None
con: quite long, concern that people may use Option over Result

Ok/Err

pro: short
con: names don't have the same length, leading to inconsistent indentation when pattern matching
con: Err is not a word

The new naming ticks all the boxes:
Pass/Fail

pro: both names have the same length
pro: same length as Some/None
pro: real words

It's a minor thing, but it's nice to have found a good design even where it doesn't matter that much!
The only concern I have is that I might find a type in a testing-related context where Pass/Fail would fit even better!

After the implementation of https://github.com/core-lang/core/issues/8, Option and Result could actually "share" the good case:
class Some[T](val: T)
class None[T]
enum Option[T] of Some, None
enum Result[T, E] of Some, ???

This still leaves open the question what to call the non-success variant of Result.

Adrian from Coffee Compiler Club mentioned Is and No as a possibility. :-)


Code example in readme uses unknown components

This block of code in the readme uses a <Link/> and <BodyText/> component of unknown origin. I suggest replacing them with standard html tags
import TextLoop from "react-text-loop";

class App extends Component {
    render() {
        return (

                <TextLoop>
                    First item
                    <Link to="/">Second item</Link>
                    <BodyText>Third item</BodyText>
                </TextLoop>{" "}
                and something else.

        );
    }
}

@zeroskillz Thanks for opening the issue and the PR. Sorry it took me so long to reply but I ended up not spending much time around this.
The use of the Link and BodyText components is to highlight the fact that you can use normal HTML elements (the span), something from a 3rd party library or another component in your codebase.
I've made it a bit more explicit in the README so hopefully that will clear any confusion people might have.


how to query order

in omnipay-alipay, i can find https://github.com/lokielse/omnipay-alipay/blob/master/src/AbstractAopGateway.php#L348.
but in omnipay-global-alipay,i can't find

This file:
https://github.com/lokielse/omnipay-global-alipay/blob/master/src/Message/TradeQueryRequest.php


Update android build.gradle template setup

This wraps the buildscripts section in the generated build.gradle file in a conditional, avoiding unnecessary downloads of/conflicts with different Android Gradle plugin distributions in application projects.
Please check https://github.com/facebook/react-native/pull/25569 for a related discussion, specifically this comment: https://github.com/facebook/react-native/pull/25569#issuecomment-532504277
The commit also includes some minor indentation fixes in the same file.
Closes #129

I think the tests are failing.. How do I regenerate the test fixtures @brodybits?

I think the tests are failing.. How do I regenerate the test fixtures @brodybits?

do npx jest -u or yarn jest -u
and I have some feedback coming up
thanks @friederbluemle for the nice contribution!

Thanks @brodybits - Updated and rebased.

@brodybits - As you requested, I lowered the Android Gradle plugin version to 3.4.1 again. We can make the update separately. What do you suggest as a PR title and commit message subject line?

Resolved conflicts and rebased.

I just published version 0.11.0 with this update included.
I think some followup fixes will be needed, will raise one or more issues for the tracking purposes.
/cc @SaeedZhiany

Wonderful update!
Thanks, @friederbluemle, and @brodybits.

I think some followup fixes will be needed

What kind of followup is needed?

What kind of followup is needed?

see #143


Navbar getting cut off

Hey! So I recently started using scrollify and just got it working. I came upon an issue where the horizontal nav bar at the top of the screen was getting cut off. I have 3 panels, and each one have their own name. I would like the navbar to be included in panel 1 but I don't know how to do that other than to put them both in the same div. Thanks!

I seemed to have fixed it by making a div that covers both of them and making that the panel.


[TM] Add spec for UIManager

Summary
part of #24875. Because some of the methods are rewriteable, I dropped the + from the signature, this doesn't feel right to me, but I am not sure if the codegen requires that. If it does, it will probably be better to extend the spec and allow those specific methods to be overriden in a UIManager.js interface. Thoughts on that @fkgozali or @RSNara?
Changelog

[General] [Added] - Add TM spec for UIManager
Test Plan

Flow passes.

If it does, it will probably be better to extend the spec and allow those specific methods to be overriden in a UIManager.js interface.

So, I think the Spec should focus on what exists in native classes. For props that are updated during runtime, you can introduce a wrapper object that provides those methods. For example:
interface Spec extends TurboModule {
  +foo: () => void;
}

const Module = TurboModuleRegistry.getEnforcing<Spec>('UIManager');

type WrapperSpec = {
// extend the spec above here somehow
};

const UIManager: WrapperSpec = {
  foo(): void {
    Module.foo();
  },
  updatedFunc(): void {
    // do something here
  },
}

export default UIManager;

@fkgozali I wrapped the native module that I was originally exporting from UIManager.js in a type that extends the spec, the other overrides are in native code too- so im not 100% sure we want to remove them from the spec, they get modified later and call their originals. not sure if legal.


Report an error

I checked the mod, anyway, when I only open two front mods and tmpe will report the following error, and keep popping up when running the game, resulting in a menu that cannot be opened.Thank you for your trouble.
Method not found: 'PathManager.FindPathPosition'. [System.MissingMethodException]
Details:
No details

Make sure you run the latest version of the game (1.17.1f4) and the latest version of the mod (<IP_ADDRESS>).
Any other combination will not work correctly.
https://github.com/CitiesSkylinesMods/TMPE?tab=readme-ov-file#support-policy

确保运行最新版本的游戏 （1.17.1f4） 和**最新版本的 mod （<IP_ADDRESS>）。**任何其他组合都无法正常工作。
https://github.com/CitiesSkylinesMods/TMPE?tab=readme-ov-file#support-policy

Both versions are correct. Is it possible that I don't have enough memory? I'm 16 gigabytes, and there's one more mistake.
Broken assets:
Commercial Low: Game Store 3x3 Level 1: Object reference not set to an instance of an object
Commercial Low: Game Store 3x3 Level 2: Object reference not set to an instance of an object
Commercial Low: Game Store 3x3 Level 3: Object reference not set to an instance of an object
Commercial High: Future Store 4x4 Level 1: Object reference not set to an instance of an object
Commercial High: Future Store 4x4 Level 2: Object reference not set to an instance of an object
Commercial High: Future Store 4x4 Level 3: Object reference not set to an instance of an object
Monument: Countdown Clock: Object reference not set to an instance of an object [BrokenAssetException]
Details:
No details

There are only two reasons for that error:

outdated version of the game or mod
or completely different mod is incompatible.

You didn't share the log file (output_log.txt) so how can I know what you have installed?
And no, the error about missing method is only caused when trying to run incompatible version of the game or mod.
Broken assets are broken assets, completely unrelated.

output_log.txt
Is this it? I always subscribe to the latest in the creative Workshop, and just a few essential mods. Please take another look at it for me, and I'll change the hardware if I have to. Thank you.

output_log.txt Is this it? I always subscribe to the latest in the creative Workshop, and just a few essential mods. Please take another look at it for me, and I'll change the hardware if I have to. Thank you.

Yes, file is correct.
From the log you sent: Game Version: 1.17.0-f3-epic-win
You said:

"Both versions are correct."

which is clearly not true!
I told you what's the correct version so update the game.

Make sure you run the latest version of the game (1.17.1f4)

output_log.txt就这样吗？我总是订阅创意工坊中的最新版本，以及一些基本的 Mod。请帮我再看一眼，如果需要的话，我会更换硬件。谢谢。

是的，文件是正确的。从你发送的日志中：你说：Game Version: 1.17.0-f3-epic-win

“两个版本都是正确的。”

这显然不是真的！
我告诉过你什么是正确的版本，所以更新游戏。

确保您运行最新版本的游戏 （1.17.1f4）

Thank you so much. I figured out the problem, too. Since I'm epic, I had to import the mod myself, so the game version was automatically lowered. Thank you for your answer.


import.meta.glob批量导入JS，部分生成后的JS文件头部包含“require("vue");”依赖，从而导致浏览器访问报错

Describe the bug
import.meta.glob批量导入JS，部分生成后的JS文件头部包含“require("vue");”依赖，从而导致浏览器访问报错
Reproduction
无
Steps to reproduce
No response
System Info
vite:3.0.4
vue: 3.2.37
mac

Used Package Manager
npm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

sorry,我排查出问题了，是自动导入函数引起的


ADX

When the ADX function is called and run it provides unreasonably high numbers, cuts out on data gaps and never comes back, as opposed to NaN and then going back to evaluated numbers. Not sure what is going on, but something is not right.  Perhaps the issue is with another one of the functions ADX uses

Hi @grich4,
Thanks for reporting the issue, I'll look into this when I have some free time. Could you let me know which python version you are using?

3.6

I've gone through your code and do not understand what is causing the issues I'm experiencing.  I've attached the data i've been using.  Processing the data via this package shows there are no instances where ADX is under 25, when that occurrence should be a bit more frequent, and high values occur around 400, which doesn't make sense. 40 does make sense, but the formulas I checked this against all matched.
five_min.txt

I have run into this problem as well. I've done a little bit of digging and found a few issues:
Most directly, some parenthesis are missing from the equation for ADX leading to erroneous values.
Since ADX depends on ATR and SMMA, I looked at those functions as well. ATR is defined recursively which does not play nice with nan values (one nan value in the middle of a dataset can cause all subsequent values to be nan).
The values for smma are not the same as what I get using other packages such as pandas.
I would be happy to work on fixing these issues if you are open to a pull-request.

@kylejusticemagnuson I noticed that the pyti package only depends on numpy and not pandas, is this by design or is it ok to include pandas as an additional dependency? I ask because there is a nice exponential weighted moving average function in pandas that might be useful when computing smma.

@kylejusticemagnuson Thanks for the quick response! I'll let you know once I have some progress to show.

Thanks a lot!
I have been busy recently and haven't been able to deep dive into any issues/improvements

New version released v0.2.2 with this fix


reload_data=True doesn't work

I'm using version 0.3.4.post2. This is my code:
def draw_table() -> None:
    df_filtered = prepare_data_for_table(st.session_state.df)

    options = GridOptionsBuilder.from_dataframe(
        df_filtered,
        enableRowGroup=True,
        enableValue=True,
        enablePivot=True,
    )
    options.configure_pagination(
        paginationAutoPageSize=False,
        paginationPageSize=50,
    )
    options.configure_grid_options(sideBar={"toolPanels": ['filters']})
    options.configure_selection(
        selection_mode='multiple',
        use_checkbox=True,
        header_checkbox=True,
        pre_selected_rows=st.session_state.pre_selected_rows,
    )
    grid_return = AgGrid(
        df_filtered,
        columns_auto_size_mode='autoSizeAllColumns',
        data_return_mode=DataReturnMode.FILTERED,
        enable_enterprise_modules=False,
        gridOptions=options.build(),
        update_mode=GridUpdateMode.MODEL_CHANGED,
        theme='alpine',
        reload_data=True,
        key='my_data',
        enable_quicksearch=True,
    )
    st.session_state.pre_selected_rows = grid_return['selected_rows']

In the method prepare_data_for_table, The selection logic works and gets returned perfectly from the AgGrid(...) call. I select some rows and then have submit button to send the selected rows to our backend. I need to visually mark the items that were sent to the backend.
However the reload_data=True parameter doesn't seem to work and the new data doesn't seem to reflect in the grid. Anyone else experiencing this? How do I get the updated data frame to be rendered?
Also, 0.3.4.post3 doesn't work either. The selection gets lost in this version on reload.

@PablocFonseca Are you still working on this package? Do you have any sense of what parameter(s) need(s) to change? I can make the PR and send it for your approval if you give me a general idea.

I just got back to st_aggrid development. I'm working on state management and my idea is to deprecate reload_data. The grid will be full responsive to gridOptions and data changes.


Add feedback functionality to new site

e.g. GradePage's survey

moved to jira https://jira.cac.washington.edu/browse/DAWGPATH-16


"Cannot assign int to property" when passing in data

I am getting the error "Cannot assign int to property" when trying to open a modal from a non-livewire component. Generated code:
Livewire.emit("openModal", "my-modal-test", {"user":581});

The modal opens fine without any data.
Is Laravel 9 + PHP 8.1 supported?

Interestingly, it seems to work when this line is commented out:
public User $user;

This doesn't seem right, though, so I won't close the issue.

@jamesblackwell public User | int $user; should do the trick. Livewire first sets the ID before resolving the model.

@PhiloNL Your suggestion does not work for me for some reason (Laravel 10). The only obvious difference here is that I am using UUID id type, so I declare it as string type:
public User | string $user;

Later on when using the user model:
Call to a member function relationship() on string


passing data via stdin does not work anymore after update to 2.4.0

with version 2.3.0 I was able to import a flux script like
cat downsampling_opentsdb_090d.flux |> influx task create --org default-org
after updating to version 2.4.0, I'm getting the error message
Error: 400 Bad Request: failed to decode request: missing flux
but I'm still able to import the same script with
influx task create --org default-org -f downsampling_opentsdb_090d.flux
This means that the script itself works and is not the root cause for this issues. For me it looks like the stdin is not used anymore.

In my usecase I piped a file into a docker container
cat /dataDisk/build/influxdb/downsampling_opentsdb_090d.flux | docker exec -i influxdb sh -c "influx task create --org default-org"
which does not work anymore. My workaround for now, using a tmp file inside the container, is
cat /dataDisk/build/influxdb/downsampling_opentsdb_090d.flux | docker exec -i influxdb sh -c "cat > tmp.flux && influx task create --org default-org -f tmp.flux"
best

I'm encountering this problem as well. Annoying as I just wanted to write a shell script with heredocs


Client-side validation

From: https://github.com/Azure/autorest.csharp/pull/356
Look into what client-side validation make sense.

Not doing client-side validation. Closing.


Pointer type definitions (ex. type SPtrType *S) behave the same as pointer type aliases (ex. type SPtrType = *S), but the definitions produce different types while the aliases produce the same type.
In the below image (click to enlarge) there is automatic conversion between types *main.S and main.SPtrType despite the fact that they are two different types at runtime:

This is the same program, but with a type alias type SPtrType = *S  instead of type definition type SPtrType *S:

Type definitions aren't supposed to behave exactly the same as type aliases.
What did you expect to see?
In the first screenshot, I expected that *main.S and main.SPtrType would behave like two different types. Without casting, I expected a compile error when passing a variable of type *main.S in place of a main.SPtrType.
What did you see instead?
I see pointer type definitions behaving the same as pointer type aliases.
Like imagine that you had type definition type SType S and then you had one variable of type S and another variable of type SType. You couldn't pass in an S in place of an SType. But if you had type definition type STypePtr *S and then you had one variable of type *S and another variable of type SPtrType, you could pass in a *S in place of a SPtrType as if they were the exact same type. This is unintuitive.

@JohnReedLOL You can post code between two ``` lines to get better readibility.

Yes, please post your code as text inside ``` or as links to play.golang.org, not as screenshots.  Thanks.

The different behavior between type Stype S and type StypePtr *S is as described at https://golang.org/ref/spec#Assignability.  The key point is that S is a defined type and *S is not.
This is basically a question about how the language works.  We don't use the issue tracker for questions.  Please ask questions in a forum instead.  See https://golang.org/wiki/Questions.  Thanks.

@ianlancetaylor Oh, I see. S is a defined type but struct {} is not a defined type, just like SPtrType is a defined type but *S is not a defined type. The reason you can't assign an S to an SType is because both of them are defined types. I understand.


Build example config file before packaging.

Closes #435.
Adds 6 more ignorable headers for trust profile.
Adds a WebUI trigger for MDB List.

It would be difficult to run and test.  Please verify then.


[usecase] Cleanup designed tests

Remove all test files for designed cases from master branch,
maintain them in "designed" branch:
https://github.com/crosswalk-project/crosswalk-test-suite/tree/designed/usecase
BUG=https://crosswalk-project.org/jira/browse/XWALK-4606

@ibelem @Honry Pls also review the WRT usecase and cordova usecase

LGTM

LGTM


fix and refactor SetUpForNewFiletype

fix: variables are not set for a new buffer with filetype = s:last_filetype
remove unused parameter and simplify name

By the way I don't know if you saw from a week or so ago I actually invited you to join this org to have a little more free hand in helping out (e.g. being able to close issues, review and even merge other people's PRs, merge your own PRs with just a review approval from another contributor that doesn't have to be me, etc.). Feel free to join, and know that I don't expect that to mean any obligation to contribute, just a few more tools at your disposal when you do feel like it.

@alerque I was a bit hesitant to accept the invitation because I probably won't be able to actively monitor the issues and PRs. But at least I think I can review and merge PRs on nerdcommenter when requested.
BTW the invitation seems to be expired. Can you please send it again? Thanks.

Sure, I guess invitations have a 7 day timeout. I just renewed it.
I fully understand if you can't/don't actively monitory all issues and PRs, but at least if occasion does come up again the barriers to contribution will be a little lower.


[CO] build_geometry labels

Hello,
I think it would make some sense to introduce a default convention for the labels set in build_geometry.
There should also be some convenience function to set and get the labels. So the actual label creation is removed from all build_geometry methods and will be in one 'central' place.
What do you think?
Best regards,
Sebastian

Hello Sebastian,
Great idea ! We already talked a bit about it in #195. We need to take into account the constraints "Several laminations" from #212 and if possible the 30 characters constraint from #202.
I agree that having a method that gather all the labels would be handy. What about a method "get_label_dict" that would return a dict with key the surface type ("Winding", "Lamination", "Magnet", Vent"...) and the value would be the proper label "Stator_0_Lamination", "Wind_Stator_R0_T0_S0"...
I think that this method can be defined in Lamination and always return the complete dict. That way all is defined in a single file.
Then maybe we can remove the "R0_T0_S0" part and let the build_geometry set it.
What do you think of this proposal ?
For this issue, I think that we should first set the label conventions by writing the proper documentation on pyleecan.org, then add this new method and finally update the labels in all the code (in particular FEMM).
Best regards,
Pierre

Done in #412


double quoting string values in SQL WHERE clause gives "org.apache.calcite.runtime.CalciteContextException"

Please provide a detailed title:
double quoting string values in SQL WHERE clauses result in a exception:
"org.apache.calcite.tools.ValidationException: org.apache.calcite.runtime.CalciteContextException: From line 1, column 49 to line 1, column 53:"
single quoting the value yields the expected result. example below
Affected Version
0.20.0 running in a standalone single-server
Description
Very strange issue here. I have a table (schema below) with a varchar 'ticker' column. When I run a query I must use single-quote characters around values for ticker. If I use double quotes I get the following error.
Output copied from dsql:
dsql> SELECT count(*) FROM "stocks-5k" WHERE ticker = "AHH";
org.apache.calcite.tools.ValidationException: org.apache.calcite.runtime.CalciteContextException: From line 1, column 49 to line 1, column 53: Column 'AHH' not found in any table
dsql> SELECT count(*) FROM "stocks-5k" WHERE ticker = 'AHH';
┌────────┐
│ EXPR$0 │
├────────┤
│    692 │
└────────┘

table schema:
__time TIMESTAMP
count BIGINT
sum_adj_close DOUBLE
sum_close DOUBLE
sum_high DOUBLE
sum_low DOUBLE
sum_open DOUBLE
sum_volume BIGINT
ticker VARCHAR

This issue is more relevant to SQL standard.
According to SQL standard, double quoted strings refer to object names, which are usually name of tables or columns while single quoted strings refer to string value which are usually used in an expression or as an input argument of a function.
In your first SQL, "AHH" is double quoted, Calcite(the SQL parser) treats it as a column name which obviously does not exist in table stocks-5k and throws the exception that you see.
Table names or column names are not necessarily double quoted if they only contains letters and underscores.
SELECT count(*) FROM "stocks" WHERE ticker = 'AHH'

is equivalent to
SELECT count(*) FROM stocks WHERE ticker = 'AHH'

But if there're spaces or other special characters like hyphen('-') in the names(stocks-5k in your example), the name must be double quoted or the SQL can't be parsed correctly.

There is a feature in the next release of the web console made especially for you!

Notice:

If you click it it will do it!


npm install installs a Persian translated version

npm install currently installs github:ex7r3me/vue-tour#06ceccedd07b97c6279f71746d4f45348ade89b3 which is localized in Persian.
A workaround is to npm install pulsardev/vue-tour instead.

Hey there!
This is kinda weird, npm view vue-tour returns proper informations:
▶ npm view vue-tour

<EMAIL_ADDRESS>| MIT | deps: 3 | versions: 5
Vue Tour is a lightweight, simple and customizable tour plugin for use with Vue.js. It provides a quick and easy way to guide your users through your application.
https://pulsardev.github.io/vue-tour

keywords: vue, tour

dist
.tarball: https://registry.npmjs.org/vue-tour/-/vue-tour-1.0.1.tgz
.shasum: 70062f265db8391fc25f44da8ffcf80dd72169e6
.integrity: sha512-Eq5Ypm8K98l4/ecUDeFDkSH+YYwZ1VGOybnp9gSgBy1Uz3TllFTjzhAiC+tqP5OVwXFZp054Vr4WlGIVy8fy/A==
.unpackedSize: 2.2 MB

dependencies:
hash-sum: ^1.0.2   popper.js: ^1.12.9 vue: ^2.5.13

maintainers:
- mmorainville<EMAIL_ADDRESS>- outpox<EMAIL_ADDRESS>- pulsar-dev<EMAIL_ADDRESS>
dist-tags:
latest: 1.0.1

published 7 months ago by pulsar-dev<EMAIL_ADDRESS>
Could you provide additional information please?

Hi there Outpox,
Digging further I got more npm packages with odd versions - turns out the project had some tainted files lying around that were messing up with npm. I did a fresh install and everything is okay. Sorry for the red herring!


[swift-4.0-branch] Add workaround for MSVC bug

See #33
The problem is that MSVC doesn't recognise the following syntax, reporting multiple errors here:
  size_t numTrailingObjects(
      typename TrailingObjects::template OverloadToken<SourceLoc>) const {
    return asDerived().hasArgumentLabelLocs()
             ? asDerived().getNumArguments()
             : 0;
  }

For some reason, MSVC ignores the fact that we gave friend access to llvm::TrailingObjects here.
Note that the Swift source code also has to be modified, as this is not enough. This is the new source code, that fully compiles after this change!
  // Work around MSVC bug: can't infer llvm::trailing_objects_internal, even though we granted friend access to it
	size_t numTrailingObjects(
		llvm::trailing_objects_internal::TrailingObjectsBase::OverloadToken<Identifier>) const {
		typename TrailingObjects::template OverloadToken<Identifier>) const {
    return asDerived().getNumArguments();
  }

Is this already in upstream-with-swift?

Nope: https://reviews.llvm.org/D29880 has been accepted but not yet been merged in.

Let's take the upstream fix instead once you get it merged. (If you don't have commit access there you can ask James to merge it for you.)


Query regarding destruction of Async worker's callback

I'm pretty new to nan; pardon my naiveness; I had query regarding the life cycle of Asyncworker.
https://github.com/nodejs/nan/blob/master/doc/asyncworker.md#api_nan_async_worker
In the constructor we are initiating a callback class; but in all examples available I don't see that getting cleaned up in the destructor. Is it left out intentionally ?
eg : https://github.com/nodejs/nan/blob/master/examples/async_pi_estimate/async.cc
class PiWorker : public AsyncWorker {
 public:
  PiWorker(Callback *callback, int points)
    : AsyncWorker(callback), points(points), estimate(0) {}
  ~PiWorker() {}

shouldn't it be something like this
class PiWorker : public AsyncWorker {
 public:
  PiWorker(Callback *callback, int points)
    : AsyncWorker(callback), points(points), estimate(0) {}
  ~PiWorker() {
     delete callback;
}

PiWorker also inherits AsyncWorker's destructor which calls delete callback.  Constructors in C++ are called in least derived -> most derived order, destructors the other way around.  Hope that clears it up.

thank you 👍


Use UserDefaults for heartbeat on tvOS

This will need an accompanying change to any users of heartbeat storage. I could leave the existing initializer for now and wait until the next breaking change though if that's easier - this would break for tvOS apps who update GoogleUtilities but don't update Firebase. Let's discuss!

Closing in favour of #23


Clean up unused non-CMR colors

For some reason, it's not showing the files I changed unless you click on my initial commit 💁♀

@tiffwong is there anything in here that didn't get changed separately? If not we can close this out.

@tiffwong is there anything in here that didn't get changed separately? If not we can close this out.

Closing since changes are present in develop

Closing since changes are present in develop


Inconsistent Edit Behavior When Creating Folders in Symbols View

Version and Platform (required):

Binary Ninja Version: 3.6.4709-dev Personal
OS: macos
OS Version: 14.1
CPU Architecture: arm64

Bug Description:
As shown in the GIF, when we create a folder while filtering, BN does not allow us to rename that folder directly as it does in non-filtered situation.

Fixed in 4.0.4849-dev.


Theme switching

Hi! WhatSie does not remember which theme I've selected. I have to config it every time I boot.
Fedora Linux 36 (Workstation Edition)
GNOME 42, Wayland
Kernel: 5.18.9-200.fc36.x86_64

Strange, make sure you have set the theme in whatsapp web settings to system default in this way whatsie will manage theme switching for you. Also, look in settings if you have auto theme switching turned on. If you don't want auto theme switching based on the day and night time you set, turn it off.


【Musecoco CausalDotProduct.dot none type】

attribute2music stage. I have download the attribute2music.pt checkpoint model from google drive. However when run bash interactive_1billion.sh 0 200
the following ERROR was reported:
Traceback (most recent call last):
File "interactive_dict_v5_1billion.py", line 428, in
cli_main()
File "interactive_dict_v5_1billion.py", line 418, in cli_main
attributes(args)
File "interactive_dict_v5_1billion.py", line 347, in attributes
translations = task.inference_step(
File "/opt/miniconda3/envs/MuseCoco/lib/python3.8/site-packages/fairseq/tasks/language_modeling.py", line 313, in inference_step
return generator.generate(
File "/opt/miniconda3/envs/MuseCoco/lib/python3.8/site-packages/torch/autograd/grad_mode.py", line 27, in decorate_context
return func(*args, **kwargs)
File "/workspace/music_gen/musecoco/2-attribute2music_model/linear_mask/command_seq_generator.py", line 179, in generate
return self._generate(sample, **kwargs)
File "/workspace/music_gen/musecoco/2-attribute2music_model/linear_mask/command_seq_generator.py", line 315, in _generate
lprobs, avg_attn_scores = self.model.forward_decoder(
File "/workspace/music_gen/musecoco/2-attribute2music_model/linear_mask/command_seq_generator.py", line 837, in forward_decoder
decoder_out = model.decoder.forward(tokens, sep_pos, encoder_out=encoder_out)
File "/workspace/music_gen/musecoco/2-attribute2music_model/linear_mask/linear/transformer.py", line 193, in forward
x, extra = self.extract_features(
File "/workspace/music_gen/musecoco/2-attribute2music_model/linear_mask/linear/transformer.py", line 216, in extract_features
return self.extract_features_scriptable(
File "/workspace/music_gen/musecoco/2-attribute2music_model/linear_mask/linear/transformer.py", line 363, in extract_features_scriptable
x, layer_attn, _ = layer(
File "/opt/miniconda3/envs/MuseCoco/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1110, in _call_impl
return forward_call(*input, **kwargs)
File "/workspace/music_gen/musecoco/2-attribute2music_model/linear_mask/linear/transformer_layer.py", line 58, in forward
x, attn = self.run_self_attn(
File "/workspace/music_gen/musecoco/2-attribute2music_model/linear_mask/linear/transformer_layer.py", line 117, in run_self_attn
r = self.self_attn(query, query, query, attn_mask=None, key_padding_mask=key_padding_mask)
File "/opt/miniconda3/envs/MuseCoco/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1110, in _call_impl
return forward_call(*input, **kwargs)
File "/workspace/music_gen/musecoco/2-attribute2music_model/linear_mask/linear/attention_layer.py", line 114, in forward
new_values = self.inner_attention(
File "/opt/miniconda3/envs/MuseCoco/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1110, in _call_impl
return forward_call(*input, **kwargs)
File "/workspace/music_gen/musecoco/2-attribute2music_model/linear_mask/linear/causal_linear_attention.py", line 70, in forward
V = causal_linear(
File "/workspace/music_gen/musecoco/2-attribute2music_model/linear_mask/linear/causal_linear_attention.py", line 23, in causal_linear
V_new = causal_dot_product(Q, K, V)
File "/opt/miniconda3/envs/MuseCoco/lib/python3.8/site-packages/fast_transformers/causal_product/init.py", line 43, in forward
CausalDotProduct.dot[device.type](
TypeError: 'NoneType' object is not callable

This is an issue with the library pytorch-fast-transformers because you haven't compiled the CUDA operators.
You need to clone it from https://github.com/idiap/fast-transformers.git and then install it manually using pip install . This process will compile the .cpp and .cu files.
My CUDA version is 12.2 and I installed the latest torch version (2.1.2+cu121) and replaced setup.py with an absolute path.
@lru_cache(None)
def cuda_toolkit_available():
    try:
        call(["/usr/local/cuda-12.2/bin/nvcc"], stdout=DEVNULL, stderr=DEVNULL)
        return True

After making these changes, the installation was successful.


i am having errors in the make phase something about libX11.so.6, i think i got the package on my system

Linking CXX executable chapter01
/usr/bin/ld: /usr/local/lib/libglfw3.a(x11_window.c.o): undefined reference to symbol 'XConvertSelection'
//usr/lib/x86_64-linux-gnu/libX11.so.6: error adding symbols: DSO missing from command line
collect2: error: ld returned 1 exit status
make[2]: *** [chapter01/chapter01] Error 1
make[1]: *** [chapter01/CMakeFiles/chapter01.dir/all] Error 2
make: *** [all] Error 2

I have the same problem..

Try installing the x11 developer packages for your Linux distro.

I wanted to note that I've run into the same problem on Linux Mint 17.2. I made sure to download every libx11 package I could find from doing a apt-cache search libx11 . @casio101 and @wppr have you guys had any success resolving this? Interestingly, I had no issues building on Ubuntu 14.04. I was also able to build https://github.com/openglsuperbible/sb7code which also depends on GLFW3 and uses X11 on Linux, so I'm trying to see where the difference in building lies between that project and this one.

Thanks @zryan3, want to make a pull request with this fix?

Sure, I can get one up in a jiffy

Fixed with PR #39


checkpoint file

when I try to use function utils.result_file on the checkpoint file, I get key error on KeyError: 'dim_info', is there a way to fix it?
What is the easiest way to convert the best individual to .m maestro file?
Thank you

Hello, I've met with the same error KeyError: 'dim_info'. Have you solved it ?


More Exmple

Please add more practical examples and scenarios as the topic is still vague !  Thank you!

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

ID: f089d928-4091-1873-0d28-c1307187aff3
Version Independent ID: af2356c1-5d4d-6d14-e9d4-9cfdddc37abd
Content: SQL Server Service Broker - SQL Server
Content Source: docs/database-engine/configure-windows/sql-server-service-broker.md
Product: sql
Technology: configuration
GitHub Login: @markingmyname
Microsoft Alias: maghan

@mehrdadalemi -- thank you for your feedback.  Please consider these resources:

SQL Server on Microsoft Q&A
DBA Stack Exchange
Stack Overflow

@markingmyname -- please look into this issue.

very general response !  Seems you are reluctant to help !
On Tue, May 18, 2021 at 2:27 PM William Anton Rohm @.***>
wrote:

@mehrdadalemi https://github.com/mehrdadalemi -- thank you for your
feedback. Please consider these resources:

SQL Server on Microsoft Q&A
https://docs.microsoft.com/en-us/answers/products/sql-server
DBA Stack Exchange
https://dba.stackexchange.com/questions/tagged/sql-server
Stack Overflow
https://stackoverflow.com/questions/tagged/sql-server

@markingmyname https://github.com/markingmyname -- please look into
this issue.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/sql-docs/issues/6414#issuecomment-843575182,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUDTF5JBOPYLVQ6EH5DXITLTOLLTVANCNFSM45BNM3HQ
.

@mehrdadalemi -- thank you for responding.  This repo is solely for documentation issues, with limited support options other than those resources.  I assigned this issue to @markingmyname for followup.

Thank you very much. 👍On May 18, 2021 5:55 PM, William Anton Rohm @.***> wrote:
@mehrdadalemi -- thank you for responding.  This repo is solely for documentation issues, with limited support options other than those resources.  I assigned this issue to @markingmyname for followup.
—You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or unsubscribe.

We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion, and we will reopen the issue. To see what the community is wanting, please visit the SQL Server User Feedback site.


ML 1.3 : Central Limit Theorem (V)

Welcome to 'ML' Team, good to see you here
This issue will helps readers in acquiring all the knowledge that one needs to know about Central Limit Theorem.
To get assigned to this issue, add your Batch Numbers mentioned in the spreadsheet of "Machine Learning", the approach one would follow and choice you prefer (Documentation, Audio, Video). You can go with all three or any number of options you're interested to work on.
Domain : Machine Learning
Points to Note :

The issues will be assigned on a first come first serve basis, 1 Issue == 1 PR.

"Issue Title" and "PR Title should be the same. Include issue number along with it.

Changes should be made inside the Machine Learning/Machine_Learning/Statistics_for_Machine_Learning Branch.

Follow Contributing Guidelines & Code of Conduct before start Contributing.

This issue is only for 'GWOC' contributors of 'Machine Learning' domain.

To be Mentioned while taking the issue :

Full name
Batch Number
GitHub Profile Link
Which type of Contribution you want to make :

[ ] Video
All the best. Enjoy your open source journey ahead. 😎

Hi sir
I would like to contribute to this
Full name: Udit Kumar Jain
Batch Number: 1
github profile: https://github.com/UditJain77
Type: Video

Issue assigned @UditJain77
Happy contributing!

Today's Update
Creating a PPT for CRT!

Today's Update
Creating a PPT for this CLT issue.

What is the update @UditJain77 ? It's been 7 days and you were although you've assigned an issue. Create the PR ASAP, otherwise we can't make further issues!

Hello, I am Deepthi M with serial number:198, Batch-2. I would like to do video on this issue.

@abhisheks008 I am extremely sorry for not updating, i had some issues. By tomorrow i will try to commit it fully.

@UditJain77 update?

Hello, I am Deepthi M with serial number:198, Batch-2. I would like to do a video on this issue.
@abhisheks008 @Isha307 as the deadline has arrived can u please look into the deadline and process after that?

@UditJain77 Please give us update.

@Isha307 I am extremely sorry for the delay. I wont be able to work on this issue for the being due to some problems. I request you to grant this issue to @deepthi1107

No problem @UditJain77. Well wishes.

please do assign me @Isha307 @abhisheks008
Hello, I am Deepthi M with serial number:172, Batch-2. I would like to do a video on this issue

No problem @UditJain77, @deepthi1107 you can work on this.


[Blazor Server-side] architecture question - handle existing code with static call

Hello guys,
I actually rewrite a silverlight app in blazor, and i met some problem with reusing ViewModels from the silverlight app in my app.
The existing ViewModels use a lot of a static acess as a repository, session-like system. In client-side version of blazor, it will not be a problem normally, but on the server side it's obviously a big problem, all clients whill shared same repository wich is not OK.
For the moment i develop only in server side, i need debugging to develop correctly. For the target, we actually don't know if we will use the client or server version, but we want keep possibility to use both.
Refactoring the existing ViewModels is a complicated task because the existing app depend of them and it will be a huge and complicated refactoring, including potential regression on the actual app. I can rewrite the entry point of the static class if needed but i can't really replace the calls to the static everywhere.
So the question, there is any way to manage a static class to "retrieve" for example an session-scoped service
1 - someting like that
//this is the existing repository class, i can rewrite it, but calls like Locator.UiParameter need stay valid
public static class Locator{
public static UiParameterViewModel UiParameter
{
get
{
       UiParameterService myScopedService = ?????;
       return myScopedService.UiParameter;
}
}
}

2 - If i cant, it is possible with a singleton system ? (and i replace all calls to Locator by Locator.Instance for example ?
Thanks for your help !
Julien Gourdon

Hi. It looks like this is a question about how to use ASP.NET Core. While we do our best to look through all the issues filed here, to get a faster response we suggest posting your questions to StackOverflow using the asp.net-core-mvc tag.


Fix direwolf tests

DW got a round of submodule updates recently and should be able to pass most inputs again.
The issue with warnings not being overrid-able got fixed.

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

All fixed, improved testing of HTGR/pbmr400 model


Azure Search: does it support deleting documents?

I've searched but can't find any tests demonstrating that the .NET Azure SDK for Azure Search supports deleting documents in the search service index.
Using Microsoft Azure Search Library 0.9.7-preview .
Intellisense in VS doesn't show any operation/method other than Create.
Was expecting IndexBatch.Delete to be able to state:
await indexClient.Documents.IndexAsync(IndexBatch.Delete(myDocs.Select(doc => IndexAction.Delete(doc))));

Yes. You can use IndexActionType.Delete for this:
var batch =
    IndexBatch.Create(
        new IndexAction(IndexActionType.Delete, new Document() { { "id", "123" } }),
        new IndexAction(IndexActionType.Delete, new Document() { { "id", "456" } }),
        new IndexAction(IndexActionType.Delete, new Document() { { "id", "789" } }));

indexClient.Documents.Index(batch);

Urm...... not with the latest version of the .sdk you can't!!! Might need to update the code snippet above to something like
IndexBatch.Delete(
new Document() { { "id", "123" } },
new Document() { { "id", "456" } },
new Document() { { "id", "789" } });

The answer was valid for the SDK version mentioned in the original issue. Then we changed the SDK to make things easier. :-)
Here is how you would delete a batch with version 1.0 or later:
var batch = IndexBatch.Delete("id", new[] { "123", "456", "789" });
indexClient.Documents.Index(batch);

Or, if you want to mix different operations in the same batch, you can do this:
var batch =
    IndexBatch.New(new[] {
        IndexAction.Delete("id", "123"),
        IndexAction.Delete("id", "456"),
        IndexAction.Delete("id", "789"),
        /* Put IndexAction.Upload, Merge, or MergeOrUpload here */
    });

indexClient.Documents.Index(batch);


Only publish platform neutral outputs on win-x64 release leg

Summary
Currently, each build leg is publishing platform neutral outputs and uploading them to the same build artifact and sub path. This can cause a race condition between two build legs that are publishing at the same time. This change fixes this behavior by only publishing platform neutral outputs on the win-x64 release build leg. Additionally, the publish outputs for this build leg are also uploaded to the Build_Unified_Release artifact so that nupkg creation can consume it without publishing it (currently, the nupkg creation leg is invoking publish, but this will be fixed in a different change).
Example build: https://dev.azure.com/dnceng/internal/_build/results?buildId=2130271&view=results

Release Notes Entry

/backport to release/6.x

/backport to release/7.x


Migrating to 64Bit

Can you upload an 64bit APK file with Mascot Capsule support?
The latest Snapdragon processor had cut down the support for 32bit applications to only allow them run on Cortex-A78 cores
And future processors or Android releases may drop 32bit support entirely at any time point
Thanks

Added for new builds


recordFailure does not record as a request

I noticed that when an error is recorded it does not also record as a request causing the rps to be less than expected.
Both the python client and boomer appear to record errors also as requests however locust4j does not. I believe the difference is here
In boomer a record failure and request are done
https://github.com/myzhan/boomer/blob/e17d364d020f6feae49a06c05f02b19c6dcc6a5e/stats.go#L142-L144
In locust4j only the error is logged
https://github.com/myzhan/locust4j/blob/1.0.12/src/main/java/com/github/myzhan/locust4j/stats/Stats.java#L138-L142

Yes, your PR is welcome.

this will fix the issue https://github.com/myzhan/locust4j/pull/39

Version 2.2.1 has been released.


Steal supports packages

Steal will be able to declare packages like:
steal.packages('contacts','login','filemanager')

This tells the builder to build contacts, login, and filemanager packages that can be progressively loaded later.  In development, when code runs like:
steal('contacts', function(){ ... })

Steal will simply steal contacts/contacts.js and anything it steals that hasn't already been stolen.
But in production, instead of stealing contacts/contacts.js, it will steal app/packages/contacts.js.  app/packages/contacts.js will be comprised of contacts/contacts.js and the steals unique to contacts/contacts.js.
In summary, packages tells steal that other resources will be loaded and to package and minify them in an efficient way for progressive loading.  In the app's production.js, it tells it where to find these packages.
Example
Packages lets you define several parts of your application to progressively load.  In this example, we want to load only what is needed to get the application started and defer the rest until we know if the user has logged in or not.  The following is done with 3.3 code:
// tell steal that you want to build these as packages
steal.packages('contacts','login','filemanager')
      // load the minimum you need to start your app
      .then('jquery/controller/route',
              function(){

// checks if the user logged in or not
// we might get this from the page
$.ajax({
  url: '/curuser',
  success: function(){
    // if the user is, start routing
    new Router(window);
  },
  error : function(){
    // otherwise load the login code
    steal('login')
  }
});

// a router that listens to hashchange and loads modules on demand.
$.Controller('Router',{
  "contacts route" : function(){
    steal('contacts', function(){
      // setup contacts
    })
  },
  "filemanger route" : function(){
    steal('filemanager', function(){
      // setup filemanager
    })
  }
})

})

Impact
This makes progressive loading very simple.  You define the parts of the app you want to load later and steal them naturally in development mode.
Steal's build system will make sure to package them as efficiently as possible.  In fact, dependencies between packages are packaged themselves to improving caching and minimize load times.
When it comes time to build, everything just 'works'. No configuration files!
I believe this will be a game-changer.  It provide the fastest possible load times for complex apps in a straightforward manner.

A short example of what could be done is:

The initial code loads with some basic understanding of routing, if the user is logged in, and the initial data requested for most routes.
The initial code checks the route and user, decides that they are wanting to view a filemanager, they can request the code and the data for the filemanager.
When the data, and the filemanager code are ready, the initial code populates the filemanger with the data, preventing it's first ajax request.

Here's the test app:
https://github.com/jupiterjs/steal/blob/packages/build/packages/test/app.js
It progressively loads the accordion or table scroll as app.

Dealing with large numbers of packages
If the number of packages are large, say 7, there could be many sub-packages that need to get loaded for a single package.  This is not ideal as the overhead would not be worth loading all the scripts.
What's needed is a DEPTH value that is used to limit the number of sub-packages any package needs to load.
Here's one way we can do it ... for each sub-package combination calculate the difference.
Select the min distance and 'combine' into a sub-package.
Repeat until every package only loads DEPTH number of scripts.
Issue

it's possible to combine sub-packages so some packages will load less than DEPTH sub-packages
This might not take into account a package that keeps getting 'screwed' ( repeatedly having packages part of the minimum difference )

A possible solution could be to select the minimum distance based on DEPTH and which packages are getting 'screwed'.

Not sure if this is the right place for this, but as JMVC gets traction and is used more and more in production, there is a clear need for cache busting in steal.
What do you guys think about this approach:
Current way to load an app in prod is:
<script type="text/javascript" src="steal.production.js?myApp"></script>
What if there would be an option to add:
<script type="text/javascript" src="steal.production.js?myApp&_=123456789"></script>
Inspired by jQuery getScript cache busting.
The _ variable would propagate to all dependency calls steal makes.
This way you could change the _ variable to the build number for your app and be sure that
users always get the latest code and we can turn up cache to forever.
(Maybe "v" is a better varible than "_" )

I agree this is needed, but I think Brian has a way to do this.  Brian?

Oh, can you move this to another issue, also one for steal.  This is about packages.


Rails 5 custom devise controller not working

server log:
Started GET "/users/sign_up" for <IP_ADDRESS> at
ActiveRecord::SchemaMigration Load (0.2ms)  SELECT "schema_migrations".* FROM "schema_migrations"
Processing by RegistrationsController#new as HTML
User Load (0.2ms)  SELECT  "users".* FROM "users" WHERE "users"."id" = ? ORDER BY "users"."id" ASC LIMIT ?  [["id", 1], ["LIMIT", 1]]
Redirected to http://localhost:3000/
Filter chain halted as :require_no_authentication rendered or redirected
Completed 302 Found in 46ms (ActiveRecord: 0.9ms)
my custom controller:
class RegistrationsController < Devise::RegistrationsController
def new
super
end
def create
# add custom create logic here
super
end
def update
super
end
end
Rails.application.routes.draw do
devise_for :users, :controllers => { registrations: 'registrations' }
end

@naveen1336  @SuTran  can you please provide a sample application

Sorry a bit late to reply. This is how it worked my ways
"config/routes.rb"
Rails.application.routes.draw do
devise_for :users, :controllers => {: registrations => "users/registrations" }
end
"users/registrations_conroller.rb"
class Users:: RegistrationsController < Devise::SessionsController
end


Economy's Accounts still require a cause for their actions

While checking whether the code examples in SpongeDocs actually compile I stumbled upon some outdated parts. While trying to update these snippets I found that Account still require an explicit cause (argument) for each action.
For the rest of the API this concept has been replaced by Sponge.getCauseStackManager().pushCause(obj).
Is this intentional?

I believe this is intentional to allow economy stuff to be done off the main thread, but I'm not sure.

Pretty sure it was for the ability to have off threaded ness for account management. Since for example, you can have a server account that is being altered by other servers (bungee networking) to get changes.

Just to be sure how is this supposed to work, is the given code correct?
EconomyService economyService = magic();
Account account = economyService.getOrCreateAccount("playerA").get();
Currency currency = economyService.getDefaultCurrency();
CauseStackManager csm = Sponge.getCauseStackManager();
try (StackFrame sf = csm.pushCauseFrame()) {
	csm.pushCause(this);
	economyService.getOrCreateAccount("sdf").get().deposit(currency, BigDecimal.ONE, csm.getCurrentCause());
}

This would only work on the main Thread.
For the async usage we would probably have to use this piece of code:
economyService.getOrCreateAccount("sdf").get().deposit(currency, BigDecimal.ONE,
		Cause.of(EventContext.of(ImmutableMap.of(EventContextKeys.PLUGIN, this)), this));


Typescript definitions don't work

Installed the package (I'm using a ViteJS + React + TypeScript project) and immediately VSCode complains about the type definitions:
Could not find a declaration file for module 'colortranslator'. '/Users/stephen/repos/zara-douyin/live-gradients/node_modules/colortranslator/esm/index.js' implicitly has an 'any' type.
  There are types at '/Users/stephen/repos/zara-douyin/live-gradients/node_modules/colortranslator/index.d.ts', but this result could not be resolved when respecting package.json "exports". The 'colortranslator' library may need to update its package.json or typings.ts(7016)

Hi @anselanza,
It should be easy to solve, I‘ll publish a version and I‘ll let you know.
Regards

Hi @anselanza,
Please, try with the version 3.0.2 of the library and report back.
Regards