maximum number of function calls reached in fmin

While runing the code during the test production, I've encountered a maximum calls reached in fmin (for many SN, usually in R). Here is an example, and the absolute path to the config.json file used (at CC), where the absoulte paths to the cubes have been added.
/afs/in2p3.fr/group/snovae/snprodJob/00-13/cubefit/SNF20070712-003/SNF-0013-CUBEFIT_Rb-20070712-003_config.json
INFO: cubefit started at 2015-09-17 16:14:42
INFO: parameters: mu_wave=0.07 mu_xy=0.001 refitgal=False
INFO: reading config file
INFO: reading 21 data cubes
INFO: setting up PSF and ADR for all 21 epochs
INFO: guessing sky for all 21 epochs
INFO: fitting galaxy to master ref [18]
INFO:         success: 865 iterations, 898 calls, val=218025.65
INFO: fitting position of non-master refs [17, 19, 20]
Traceback (most recent call last):
File "/afs/in2p3.fr/group/snovae/snf/cubefit/bin/cubefit", line 45, in
master_ref=args.master_ref)
File "/afs/in2p3.fr/group/snovae/snf/cubefit/lib/python2.7/site-packages/cubefit/main.py", line 239, in main
(yctr[i], xctr[i]), atms[i])
File "/afs/in2p3.fr/group/snovae/snf/cubefit/lib/python2.7/site-packages/cubefit/fitting.py", line 419, in fit_position_sky
_check_result_fmin(warnflag)
File "/afs/in2p3.fr/group/snovae/snf/cubefit/lib/python2.7/site-packages/cubefit/fitting.py", line 32, in _check_result_fmin
raise RuntimeError("maximum number of function calls reached in fmin")
RuntimeError: maximum number of function calls reached in fmin

Based on Nico's email, only two instances of this error remain after fixing the plan scripts:
PTF10ndc/SNF-0013-CUBEFIT2_Bb-10ndc.err:RuntimeError: maximum number of function calls reached in fmin
SNF20070504-017/SNF-0013-CUBEFIT2_Rb-20070504-017.err:RuntimeError: maximum number of function calls reached in fmin

Looking into these now.

Side note: the config file for SNF20070504-017 seems to contain only 2 cubes, both of which are references. @nicolaschotard, do you know what's going on there?

fmin() is used in the simplest fitting step: fitting the position of each non-master final ref to the master final ref. It is a two parameter fit (x and y position), so we've just been using a simple fitting method. The maximum number of function evaluations in fmin seems undocumented, but looking at the source code the default is 200 * N_params or in our case 400. This does seem a bit low.
In PTF10ndc_B, I get
INFO fitting position of non-master refs [20]
INFO         success: 188 iterations, 377 calls, val=104035.94

In SNF20070504-017, I get
INFO fitting position of non-master refs [0]
INFO         success:  51 iterations,  95 calls, val=87529.73

so I can't reproduce the problem, but in PTF10ndc the number of function calls (377) is quite close to the limit (400).
We could simply up the limit. However, first I'm going to try moving to using fmin_l_bfgs_b as in the other steps. Now that we have the analytic position gradient from #15, it should be easy.

Switched to using fmin_l_bfgs_b, should be fixed.

Hi Kyle,
It appears we have only one observation for this SN, appart from the two refs, taken in a non-photometric night (http://snf.in2p3.fr/agenda/07125/ http://snf.in2p3.fr/agenda/07125/). Since only one of the ref has been taken in photometric condition, we cannot flux calibrate the cubes taken during the two non-photo nights, which is why only the two taken during night 08107 are kept. This SN should thus not be considered in this analysis for now.

On Oct 18, 2015, at 23:33, Kyle Barbary<EMAIL_ADDRESS>wrote:
Side note: the config file for SNF20070504-017 seems to contain only 2 cubes, both of which are references. @nicolaschotard https://github.com/nicolaschotard, do you know what's going on there?
—
Reply to this email directly or view it on GitHub https://github.com/snfactory/cubefit/issues/14#issuecomment-149049073.


additionalInfo empty element in wrong position in otherEntity

I created a brand new document
I uploaded one of the valid dataverse xml files: Japanese Macaque 20478-F1-09_UN_001a,
which correctly clears out the Title, Image, and Slide data - forcing the user to enter.
I added this information and then downloaded the document, it included an empty
element under , which is under the eml xsd, in the WRONG position - after entityType instead of before.
(Since it is not part of mother's xsd, it is not handled by mother_pype and shouldn't be. )
Note that we are reusing eml's otherEntity for the image. Did our code changes make this happen? I had to change xml file extension to txt. The one without _thumbnail is the one from dataverse. The one with _thumbnail was downloaded during our meeting today.
Please investigate.
20478-F1-09_UN_001a_thumbnail.xml.txt
20478-F1-09_UN_001a.xml.txt

This was being caused because existing nodes were being modified in otherEntity instead of deleting and recreating the nodes like seemingly every other form would do. As a result, a new node for additionalInfo was being created after the existing entityType node instead of them all being created in order. This has now been rectified by properly recreating the nodes whenever the form is submitted.


run base tests partially in aws

The idea is to upload OS images already containing latest commit of
osbuild-composer to AWS and boot them for base tests. The Weldr API
socket will be forwarded over SSH just like it was in PR:
https://github.com/osbuild/osbuild-composer/pull/887

superseded by #960


Support custom modifications of parsed test suite before execution

There are many command line options to alter the parsed test suite structure before executing it. You can, for example, use --name and --doc to set new name and documentation, respectively, or use --test, --suite, --include' and --exclude` to select which tests to execute. There are, however, sometimes needs for custom modifications.
For example, a team at Nokia recently wanted to way to run only every Xth test in one test run. This isn't supported using any command line options, but programmatically Robot's public APIs make the basic implementation pretty simple:
import sys
from robot.model import SuiteVisitor
from robot.running import TestSuiteBuilder

class RemoveEveryXthTest(SuiteVisitor):

    def __init__(self, x):
        self.x = int(x)

    def start_suite(self, suite):
        suite.tests = list(suite.tests)[::self.x]

if __name__ == '__main__':
    path, x = sys.argv[1:]
    remover = RemoveEveryXthTest(x)
    suite = TestSuiteBuilder().build(path)
    suite.visit(remover)
    suite.run()

A problem with the above solution is that it only runs tests and doesn't create logs and reports, but that would be pretty easy to implement too. A much bigger is that this script doesn't take same arguments as pybot does, and handling all same arguments would be a really big task.
A pretty simple but generic and powerful solution would be allowing registering visitors like RemoveEveryXthTest above to be used after parsing tests but before running them. This could work similarly as when registering listeners that get notifications during test execution:
pybot --visitor RemoveEveryXthTest:2 tests.robot

The :2 part of the the visitor name would specify arguments passed to the visitors __init__ same way listeners accept them.

Things to consider:

We probably want make it possible to give actual visitor instances, not only visitor names, to robot.run and TestSuite.run. This support is currently planned for listeners in #1969.

Probably also want to support semicolon as an alternative argument separator similarly as with listeners (#1866).

Would this function run exactly once before the tests start running?
Instead, what about making it possible to define a function to be called immediately before running each test? That function could return True or False, and if False, that test is skipped? That would let you do everything described in this feature, plus allow dynamic skipping of tests.

Yes, this is executed once before execution. There are also other use cases than dropping certain tests.
I think modifications during execution are better done using the listener API. It obviously needed to be enhanced somehow to make dropping tests possible.

So, using this interface could you programmatically build suites/tests? That would be extremely cool and something I've wanted to be able to do for a while.
Also, skipping tests/changing the results of a test should definitely be doable from the listener interface.

You can build tests already now, but then you need to also take care of running tests, creating logs, etc. This API is planned to allow pre-run modifications so that execution etc. would be like normally.

Right, that's more what I mean. I know I can build them, but it's very cumbersome to add them in to normal test execution.

What should the command line option to enable this be called? --visitor I used in earlier examples is too generic. My best idea so far is --prerunvisitor or --previsitor.

It's going to be used to initialize the test run, so something along the
lines of "--init" or "--initfunc" or "--initializer" makes sense to me.
On Thu, May 28, 2015 at 4:15 PM, Pekka Klärck<EMAIL_ADDRESS>wrote:

What should the command line option to enable this be called? --visitor I
used in earlier examples is too generic. My best idea so far is
--prerunvisitor or --previsitor.
—
Reply to this email directly or view it on GitHub
https://github.com/robotframework/robotframework/issues/1976#issuecomment-106602034
.

I think --init is a bit too generic. The current implementation uses --prerunvisitor, but that visitor part doesn't really tell much. Perhaps something like --prerunmodifier would be better. It's pretty long, but options can be shortened (e.g. --prerun) when used  as long as they are unique.
I've also been thinking should we add similar --postrunmodifier to allow modifying results before logs and reports are created. It could also be used with Rebot, so perhaps it should be --prerebotmodifier in both sides.

Sounds good to me.

--prebot and --postbot?
--bryan

On May 29, 2015, at 11:42 AM, Pekka Klärck<EMAIL_ADDRESS>wrote:
I think --init is a bit too generic. The current implementation uses --prerunvisitor, but that visitor part doesn't really tell much. Perhaps something like --prerunmodifier would be better. It's pretty long, but options can be shortened (e.g. --prerun) when used as long as they are unique.
I've also been thinking should we add similar --postrunmodifier to allow modifying results before logs and reports are created. It could also be used with Rebot, so perhaps it should be --prerebotmodifier in both sides.
—
Reply to this email directly or view it on GitHub.

The commit above (32e2a04) added support for custom result modifications. Command line options are now --prerunmodifier and --prerebotmodifier. Both User Guide and --help documentation missing but otherwise this functionality ought to be ready. This is a very powerful new feature!
Command line option names can still be updated if someone comes up with really good alternatives. I kind of like Bryan's --prebot and  --postbot but don't think they are generic enough. Especially the latter would be strange when used with Rebot.

Here's an example of a modifier that selects only every Xth test. It would work both with --prerunmodifier and --prerebotmodifier.
from robot.model import SuiteVisitor

class RunEveryXthTest(SuiteVisitor):

    def __init__(self, run_every_xth, start_from=0):
        self.run_every_xth = int(run_every_xth)
        self.start_from = int(start_from)

    def start_suite(self, suite):
        suite.tests = suite.tests[self.start_from::self.run_every_xth]

    def end_suite(self, suite):
        suite.suites = [child for child in suite.suites
                        if child.test_count > 0]

This issue is now ready for review. Especially new docs committed in ce4d5fa (visitor API docs),  b26a664 (User Guide) and 54ad854 (--help) should be proof-read.
Review by anyone is more than appreciated, but I leave it for @jussimalinen to close this issue when he's reviewed the changes.

Looks good.

Could you please clarify what this change currently supports?  You say at one point that this occurs once before exceution, but then you mention the Listener interface needs to be enhanced.  Does that mean that a keyword/test/suite listener can now make modifications on a listener event, or is that something tied to a different enhancement, or something not yet captured?

This functionality is documented in the User Guide:
http://robotframework.org/robotframework/2.9/RobotFrameworkUserGuide.html#programmatic-modification-of-test-data
Could you @davekipfer take a look at it and see is it adequate? If it isn't, please submit a new issue and tell what you don't understand and we'll enhance docs.


Form Inputs not updating after change event is fired

react-testing-library version: 4.1.5
react version: 16.4.1
node version: 8.6.0
npm (or yarn) version: 6.2.0

Relevant code or config:
test('submits form successfully', async () => {

const { getByText, queryByText, getByLabelText, debug } = renderWithRouter(
<SignIn setCurrentUser={jest.fn()} isSignedIn={false} history={{}} />
);

const submitButton = getByText('Sign In');
const emailInput = getByLabelText('Email');
const passwordInput = getByLabelText('Password');

emailInput.value =<EMAIL_ADDRESS>passwordInput.value = '12345678';

fireEvent.change(emailInput);
fireEvent.change(passwordInput);

debug();
/*
In debug email has value of<EMAIL_ADDRESS>and password has a value of ' '
*/

fireEvent.click(submitButton);

const emailErrorNode = queryByText('Email is required');
const passwordErrorNode = queryByText('Password is required')

expect(emailErrorNode).toBeNull();
expect(passwordErrorNode).toBeNull();

});

What you did:
Test a sign in form by setting username to<EMAIL_ADDRESS>and password to '12345678'
What happened:
When updating 2 inputs in a test, only one input gets updated the second input's value is ' ' (empty)
Reproduction:

https://codesandbox.io/s/0xwn98wo1p
Problem description:

I can't test forms with more than one field change
Suggested solution:

Hi @pcfields,
Thanks for this. Please ping in here when you've added the minimal reproduction codesandbox. I definitely want to improve this so testing this kind of experience is more obvious.
FWIW, change is one of the events that we do some trickery for because the change event works differently with React than it does in a real browser. I'd love to do away entirely with this code: https://github.com/kentcdodds/react-testing-library/blob/9667967a085e438919fe3d02bc36761735bb2732/src/index.js#L50-L56

I've updated the codesandbox https://codesandbox.io/s/0xwn98wo1p

Let me know how I can help you fix this?

Sounds great! Could you strip down your example to be as minimal as possible? The less code in there to reproduce the issue the better as it makes it easier for us to determine what could cause the issue.

@kentcdodds I've stripped down the code. Let me know if this is enough.

@kentcdodds Yes. But I remember, we had the discussion on the same lines of code before somewhere, I tried to search our issues, but no luck. Looks like Jomaxx have added these files, when we moved from React Simulate to native events. Not sure the reason behind it. @jomaxx any reasons?

Hi friends! So it doesn't quite do it for us. Test cases definitely fail because even though the value does change, the event handlers are not called. I've figured out a way to fix this for onChange, but still need to work on the other three. I think I want these changes in dom-testing-library instead though.
I've opened a PR for this in react-testing-library to show how it works: #153

@pcfields, if you'd like to open a PR that makes this work in dom-testing-library that'd be great. I think that's the right place for this change because even though it's kind of an implementation detail for react it doesn't impact how things work for other frameworks and I believe that Cypress (another framework-agnostic tool) handles this case.

@kentcdodds I am only now seeing your message. What would you like me to do?

Could you review this: https://github.com/kentcdodds/dom-testing-library/pull/85
Thanks!

Ok, so this bug happened due to the way React works, and now dom-testing-library's fireEvent utility supports setting properties on the target element when you fire an event. This allows you to set the value at the time of the event and dom-testing-library will set it in a way that works around how React tracks input value changes.
Here's how you use it: https://codesandbox.io/s/github/kentcdodds/react-testing-library-examples/tree/master/?module=%2Fsrc%2F__tests__%2Fon-change.js&previewwindow=tests
Here's the basic example:
fireEvent.change(passwordInput, {target: {value: '12345678'}})

:tada:
Thanks! Now we gotta figure out how to help people avoid this in the future 🤔

Is that now the recommended way to change an input value then? Should the readme be updated?

Yes, if you're going to fire a change event then that's how the value should be changed. Otherwise manipulating the value should work fine.

Hmm, should this work with the latest version of this package on npm? It's not for working for me

Please be more specific @weyert. A reproduction would be helpful.

@kentcdodds Sorry, I am was goofing up! Checked the wrong form field!

fireEvent.change(passwordInput, {target: {value: '12345678'}})

This should definitely be added to the README file.

It is in the readme. Search for the string "fireEvent.change" in the README, and you'll find this:
fireEvent.change(comment, {
  target: {value: 'Great advice, I love your posts!'},
})


Optional selection reconciliation

All updates (except readonly and collaboration) are forced selection reconciliation. This is not ideal when the user programmatically wants to update the content without forcing focus.
In this PR I propose that most updates do not reconcile selection by default. DOM events do make sense, at the end of the day they're interacting with the input directly but a toolbar update feels like optional. They can still call editor.focus after.
Given that this is a major breaking change, we can do this via
editor.update(fn, { restoreSelection: boolean }

(not in this PR but I can iterate on this other version)

This PR also fixes (WIP) update bundling when tags are present, in particular a current issue with collab and bootstrap. It is dangerous to merge updates with different tags because of batching, the next update potentially does not want to share the same behavior.
In the case of collab -> selection reconciliation
In the case of bootstrap -> history merge

Won’t this cause selection to be out of sync from the editor state?

@zurfyx I think I'd like to know more about this in maybe a VC at some point.

This PR also fixes (WIP) update bundling when tags are present, in particular a current issue with collab and bootstrap. It is dangerous to merge updates with different tags because of batching, the next update potentially does not want to share the same behavior.

I think this should be another PR, no?

@zurfyx what is the status of this PR, is it dead or can we expect to get it merged?
This is causing us some issues and there don't seem to be any work arounds that don't involve some pretty involved focus management from other elements.

So many changes have happened since, it might be easier to start this PR from scratch. Closing for now.


file links not working anymore

Currently In zulip server version 6.0 the ENABLE_FILE_LINKS generates a link that I can click on but I cannot open the file.
If I use a image with file: i also dont get a preview.
After recognizing this I tried  setting INLINE_IMAGE_PREVIEW = True, too but it does not change the behaviour.

Tried if version 4.6 has working file:// links but no luck, too.

I'm closing this as WONTFIX, as file:// links are insecure, and browsers do not support them anymore.
See also #19720 for removing the implication that we support them.


Feature/update minimalcss dependency

Description
Update minimalcss dependency
💔Thank you!

Wait website deployment before merge.

Results with package updates

Looks good! 💪


Update README.md

Add video links

Make sense.
Can you please add those changes, as part of #301 ?

Added videos to docs index page in related PR #301 through https://github.com/Clay-foundation/model/pull/301/commits/3fe1732a26832e20f5364ab4ab1a6a85611855c2


I had to change 'use-preset' to 'apply'

I think 'use-preset' has been changed to 'apply'
see https://usepreset.dev
/home/nedcol/.npm/_npx/7282/pnpm-global/4:
+ use-preset 0.2.12
npx: command not found: use-preset

Eg
pnpx apply svelte-add/tailwindcss

Seems to mostly work

Thank you for the heads up! I think every occurrence has been updated.

I had the opposite experience--use-preset worked and apply didn't. Using the latest SvelteKit (installed today) built on Vite and NPM 6.14.11:
Apply didn't do anything:
❯ pnpx apply svelte-add/postcss
usage: apply [-a magic] [-d] [-0123456789] command arguments ...

use-preset worked:
pnpx use-preset svelte-add/postcss
 WARN  use-preset<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
Packages: +133
+++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Packages are hard linked from the content-addressable store to the virtual store.
  Content-addressable store is at: /Users/f/.pnpm-store/v3
  Virtual store is at:             node_modules/.pnpm
Progress: resolved 133, reused 29, downloaded 104, added 133, done
<EMAIL_ADDRESS>Running postinstall script, done in 650ms

/Users/f/.npm/_npx/23910/pnpm-global/4:
+ use-preset 0.2.13
npx: command not found: use-preset

I had the opposite experience--use-preset worked and apply didn't. Using the latest SvelteKit (installed today) built on Vite and NPM 6.14.11:
Apply didn't do anything:
❯ pnpx apply svelte-add/postcss
usage: apply [-a magic] [-d] [-0123456789] command arguments ...

use-preset worked:
pnpx use-preset svelte-add/postcss
 WARN  use-preset<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
Packages: +133
+++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Packages are hard linked from the content-addressable store to the virtual store.
  Content-addressable store is at: /Users/f/.pnpm-store/v3
  Virtual store is at:             node_modules/.pnpm
Progress: resolved 133, reused 29, downloaded 104, added 133, done
<EMAIL_ADDRESS>Running postinstall script, done in 650ms

/Users/f/.npm/_npx/23910/pnpm-global/4:
+ use-preset 0.2.13
npx: command not found: use-preset

Please see https://github.com/preset/cli/issues/65. I know this is a problem that I'm going to try to experiment with solving soon.

Thanks. That is it--man apply shows a result for me too.
I should mention, I wasn't able to get tailwind working, like I could previously using this, but I'll try again later since SvelteKit is in flux.


Chatgpt 3.5

Hi there, will you implement chatgpt 3.5 model?

Sorry, didn't notice your comment. Not sure what you exactly mean, it's already implemented here. You can use it with /chat command.


Reverse proxy does not works with trojan protocal (使用trojan协议时无法使用反向代理)

What version of V2Ray are you using?
V2Ray 5.2.0 (V2Fly, a community-driven edition of V2Ray.) Custom (go1.19.4 linux/amd64)
What's your scenario of using V2Ray?
Reverse Proxy
What problems have you encountered?
It does not work with trojan protocal, while it works with vless protocal.
What's your expectation?
trojan can be used as the protocal for the tunnel.
Please attach your configuration here
Server(portal) configuration:
log:
  access: /var/log/v2ray/access.log
  loglevel: debug
reverse:
  portals:
  - domain: aaa.private.test
    tag: aaa-portal
inbounds:
- protocol: vless
  port: 10000
  tag: tunnel-vless
  settings:
    decryption: none
    clients:
    - email: tunnel
      id: 6ba03c02-8fc8-11ed-88c2-1fb064fe3abd
  streamSettings:
    network: ws
    wsSettings:
      path: /my-path
- protocol: trojan
  port: 10001
  tag: tunnel-trojan
  settings:
    clients:
    - email: tunnel
      password: my-password
  streamSettings:
    network: ws
    wsSettings:
      path: /my-path
- protocol: dokodemo-door
  port: 10002
  tag: aaa-ssh
  settings:
    address: <IP_ADDRESS>
    network: tcp
    port: 22
outbounds:
- protocol: freedom
  settings:
    domainStrategy: UseIPv4
  tag: free
routing:
  domainStrategy: AsIs
  rules:
  - type: field
    inboundTag:
    - aaa-ssh
    outboundTag: aaa-portal
  - type: field
    domain:
    - full:aaa.private.test
    outboundTag: aaa-portal

Client(bridge) configuration:
log:
  access: none
  loglevel: debug
outbounds:
- protocol: vless
  settings:
    vnext:
    - address: # address here
      port: 10000
      users:
      - id: 6ba03c02-8fc8-11ed-88c2-1fb064fe3abd
        encryption: none
  streamSettings:
    network: ws
    wsSettings:
      path: /my-path
  tag: tunnel-vless
- protocol: trojan
  settings:
    servers:
    - address: # address here
      password: my-password
      port: 10001
  streamSettings:
    network: ws
    wsSettings:
      path: /my-path
  tag: tunnel-trojan
- protocol: freedom
  tag: local-out
reverse:
  bridges:
  - domain: aaa.private.test
    tag: bridge
routing:
  domainStrategy: AsIs
  rules:
  - type: field
    inboundTag:
    - bridge
    domain:
    - full:aaa.private.test
    outboundTag: tunnel-trojan # it works if we change it to tunnel-vless
  - type: field
    inboundTag:
    - bridge
    outboundTag: local-out

应该跟这个有关https://github.com/XTLS/Xray-core/commit/7427a55ef1e67e4941a007ed1f5daa83a05e36d7 ，v2ray core少了这个修改


Include the request_type when creating a retirement request

Depends on https://github.com/ManageIQ/manageiq-api/pull/439
Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1604016

@d-m-u @tinaafitz If retirement is being done as a request, does Vm even respond to #retire anymore?
I'm asking because I'm wondering if we should remove https://github.com/ManageIQ/manageiq/blob/master/app/models/vm/operations/lifecycle.rb#L21-L25 or is it still valid to call retire directly on a Vm or Service object?

Oh I see, we added a method to the mixin rather than replacing the existing retire method.
So it sounds like this broke when retirement went from a direct method on the Vm object to a request.
Now instead of pushing the retire action over the API for central admin on the resource this PR is giving us the information we need for when we relay the create_request method.

Correct, now that it's hitting create_request which will be relayed to the remote region, we need to include the type of request for the remote region to parse.

When was the retirement-as-a-request added? Does this need to be backported?

I don't think it made it back to g-branch.  @tinaafitz @d-m-u Can you confirm?

@bdunne That is correct. The new retirement as a request changes were not back ported to the g-branch.


Add WalletConnect V2 implementation via Web3Modal Flutter SDK

Fix #88 with implementing WalletConnect V2 via Web3Modal Flutter SDK

@dmccartney Would you please take a look at the failed check. I'm curious about the reason. Is it sth with my last commit?


Fixing Inplace Error that Made Weird Autograd Errors

Finally fixed the weird errors that autograd was throwing about in-place operations. After changing these inplace operations in the NeuralCLBFController() class, the example train_inverted_pendulum.py was able to run with (mostly) no errors.
Feel free to reject these if you don't think that they are useful @dawsonc !

Hi Kwesi! Thanks for the PR! Sorry it took me so long to realize it was here!
Which autograd errors does this fix? Could you please make an issue on this repo with the errors you're getting and the steps to reproduce? Context for my confusion: train_inverted_pendulum.py runs with no errors on my laptop :thinking: . AFAIK the +=, *=, etc. operators aren't in-place operations; they're sugar for x += y => x = x + y. Maybe there's something subtle going on that makes this run differently between machines.
Also, this PR looks like it includes all of the code for the new pusher-slider example. That's great, but it should probably go in a different PR (e.g. 1 for autograd errors, 1 to add the pusher slider).

Lots of interesting things here. Thanks for your thorough response:

Which autograd errors does this fix? Could you please make an issue on this repo with the errors you're getting and the steps to reproduce?

The specific autograd error is the following:
RuntimeError: one of the variables needed for gradient computation has been modified by an inplace operation: [torch.FloatTensor [64]], which is output 0 of ReluBackward0, is at version 1; expected version 0 instead. Hint: the backtrace further above shows the operation that failed to compute its gradient. The variable in question was changed in there or anywhere later. Good luck!
Epoch 0:   0%|          | 0/157 [00:02<?, ?it/s]

Added an extra line so that it shows this error occurred before the first epoch began.
I can create an issue later on today (or tomorrow, depending on when this plane lands)! This error appeared for me when running any of the training scripts in the train directory. I originally thought that it might be due to Pusher-Slider code, but attempted to run the current scripts and received the same error.

Context for my confusion: train_inverted_pendulum.py runs with no errors on my laptop 🤔 . AFAIK the +=, *=, etc. operators aren't in-place operations; they're sugar for x += y => x = x + y. Maybe there's something subtle going on that makes this run differently between machines.

It's possible that the problem is caused by different machines. This forum post leads me to believe that += and *= are inplace operations, though: discuss.pytorch. Regardless of whether or not it is, I am very curious why pytorch doesn't complain on your machine. I'm running the newest versions of the libraries; Are any of yours older?

Also, this PR looks like it includes all of the code for the new pusher-slider example. That's great, but it should probably go in a different PR (e.g. 1 for autograd errors, 1 to add the pusher slider).

This is super surprising! Github marked the Pusher-Slider stuff as having been committed 2 weeks ago, but I committed that stuff yesterday! Now, I know that GitHub includes ALL changes on a branch into a pull request (not just the ones that were available at the time of pull request creation). I'll make a note to fix this.
A possible compromise: If you don't see the issue on your machine with updated libraries, then feel free to close/archive the pull request which will make this discussion visible to any M1 Mac users that may run into the same issues. :)

I'm open to keeping this PR open and tracking down this inplace bug. The first step would be to make an issue that includes steps to reproduce and the full stack trace that the error prints out. Even if it works on my machine, the fact that it doesn't work on your machine is troubling and we should try to dig into it.

This pull request was referenced in a new issue, following Charles' guidelines. This PR will be closed and a new one will be created containing only the proposed "inplace" changes.


⏫ Upgrade to Micrometer 1.13

Describe the feature
Since Micrometer 1.13 introduces a few breaking changes and have deprecated io.micrometer.prometheus.PrometheusMeterRegistry I would like to see javalin-micrometer bumped to this version.
Additional context
The breaking changes in 1.13 can be see here Migration Guide

@bogedal You want to introduce breaking changes on the current (6x) branch?

I have to say @tipsy I see the value in this, we could have an extra module for the newer micrometer until 7.x then drop the old one?

Let's add a Javalin 7 tag and let it age like wine until we bump to jetty12

Hm.. I accidentally merged this from dependabot because the tests passed 🤔

Well we should check our tests and potentially revert the changes

Well, a bit late for that. My bad.


Build issue with fits

0;32m    box: TASK [ansible-samvera/roles/fits : install zip utility] ************************ [0m
 [0;32m    box:  [0;32mok: [<IP_ADDRESS>] [0m [0m
 [0;32m    box: [0m
 [0;32m    box: TASK [ansible-samvera/roles/fits : download fits zip version 0.8.4] ************ [0m
 [0;32m    box:  [0;33mchanged: [<IP_ADDRESS>] [0m [0m
 [0;32m    box: [0m
 [0;32m    box: TASK [ansible-samvera/roles/fits : unpack fits] ******************************** [0m
 [0;32m    box:  [0;31mfatal: [<IP_ADDRESS>]: FAILED! => {"changed": false, "msg": "Failed to find handler for "/home/vagrant/install/fits-0.8.4.zip". Make sure the required command to extract the file is installed. Command "/bin/tar" could not handle archive. Command "/usr/bin/unzip" could not handle archive."} [0m [0m

This is not repeatable. I'm guessing it was a problem with the download and then unzip didn't know what to do with an empty file. This could be fixed upstream in the ansible roles so that they failed with a better message, but I'm going to mark this as out of scope for packer-samvera.


Characters in latin text in vertical writing mode should be rotated.

The rendered images writing-mode-tb-001-1-image.png, writing-mode-tb-001-2-image.png, writing-mode-tbrl-001-1-image.png and writing-mode-tbrl-001-2-image.png are incorrect. They show the characters as if in XSL-FO a glyph-orientation-vertical with 0deg would have been applied. Instead the default value "auto" should apply. This would result in latin text that is rotated sideway right.
Thanks for @palemieux for spotting and clarifying this. See also the minutes from the Timed Text Working Group from 2017-02-16.

The following shows the incorrect rendering of writing-mode-tb-001-2 that needs amendment

The below shows the correct rendering of the same sample:


Update some legacy references to Twitter

Fixed a few that I mentioned in #19, which I've checked off there, but:

unsure of what to touch in regards to licences and author fields
don't know what to point the build status badge at, considering travis-ci is broken
no idea what I should do in regard to the jsdelivr npm package links
can't PR changes to repo description

Feel free to submit your own edits to my branch, considering you're way more experienced here.
Thanks,
Elliott

Any idea what's happening with the stuff I wasn't able to fix?


Bug 2077974: Prevent reconcile error when policy status is missing

Consider the the cluster non compliant when the policy has missing status

/bugzilla refresh

How do you want to change that function? Its job is to return the cluster status and it is doing its job. The bug is not in that function. Also in my opinion the changes to the caller functions is good because I think the extraneous error handling is removed.

/approve

/lgtm

/cherry-pick release-4.11

/cherry-pick release-4.11

/cherry-pick release-4.10

/cherry-pick release-4.10


#1068 Reveresed the sponsors List

Reversed the sponsors List--
Test case used Fossasia 2014 api which gives sponsor level '1' to google. I think for FOSSASIA 2017, Sponsor level should be reversed by organiser.

@dinesh-sirvi Please see, there are some sponsors that are not visible here Ex : Google cloud {"description": "", "level": "7", "url": "http://www.google.com/cloud", "sponsor_type": "Platinum Sponsor", "logo": "https://storage.googleapis.com/eventyay.com/events/6/sponsors/201/logo/R0NTNTBzTE/7eb895c9-90b4-4cda-ab91-70218df29756.png", "id": 201, "name": "Google"} when the app is generated with endpoint https://eventyay.com/api/v1/events/6.
Also you can find all event sponsors here https://eventyay.com/api/v1/events/6?include=sponsors

@dinesh-sirvi It would be great if you can provide a link to demo server or live gh-page.
Use Api endpoint : https://eventyay.com/api/v1/events/6
To generate webapp. Thanks

@dinesh-sirvi It contains commits from many developers. Please clear the commits of the other users.


ReferenceError: window is not defined

This bug is appearing in the console when running the app. Find out what it means and how to fix it.

Window not defined is an error that can be resolved by dynamically calling the MapComponent inside of the ViewTrucks component


Pretraining Datesets

Hi, thanks for the great work.
I have a question you mentioned that NIH and CheXpert datasets have been used for both training and testing. I am curious about whether the images in the test sets have been used in the pretraining stage.

Just the training sets of both. You could check it with the dataloaders. https://github.com/lambert-x/medical_mae/blob/b199e6771f92a82ca524d63334a82e806b6db6d4/util/datasets.py#L65C1-L80C95


WARNING: PMU in use. And the CPU utilization is inaccurate

I try to use perf-collect.py to collect the metrics of "stress" process.
But I meet a warning like this:
ubuntu@inspur-icx-1:~/zhi/PerfSpect$ sudo python perf-collect.py -a "stress --vm 1 -c 1 --vm-bytes 1024M"
2023-05-10 16:57:37,423 INFO: nmi_watchdog disabled
2023-05-10 16:57:37,776 WARNING: PMU in use: cpu cycles
2023-05-10 16:57:37,815 WARNING: output could be inaccurate
2023-05-10 16:57:37,815 INFO: Collecting perf stat for events in : /home/ubuntu/zhi/PerfSpect/events/icx.txt
stress: info: [832408] dispatching hogs: 1 cpu, 0 io, 1 vm, 0 hdd
^C2023-05-10 16:58:18,853 INFO: Collection stopped! Caculating TSC frequency now
2023-05-10 16:58:19,903 INFO: nmi_watchdog re-enabled
2023-05-10 16:58:19,905 INFO: perf stat dumped to perfstat.csv

ubuntu@inspur-icx-1:~/zhi/PerfSpect$ sudo python perf-postprocess.py -r perfstat.csv --html cpu.html
2023-05-10 16:57:17,006 INFO: Generated results file(s) in: /home/ubuntu/zhi/PerfSpect
2023-05-10 16:57:17,467 INFO: static HTML file written at cpu.html
2023-05-10 16:57:17,467 INFO: Done!

If I ignore these warnings, the CPU utilization is inaccurate in the cpu.html file.

Is there any relationship between the warning info and the inaccurate problem?
Thanks!

@changzhi1990 What is the expected CPU utilization?

@changzhi1990 What is the expected CPU utilization?

The expected utilization is 100%

@changzhi1990 What is the expected CPU utilization?

The expected utilization is 100%

Because I use the stress process and I can see the CPU utilization is almost 100% by htop of the specific core.

@changzhi1990 The CPU utilization presented by PerfSpect is the average utilization across all cores/threads. The stress-ng command you are running is running only 2 threads.

Thanks for your reply. I try to add a new feature to collcet the perf events of the specific CPU cores, like this

But the CPU utilization is almost 0% in the html file. So I think maybe I need to modify the code further.

there's two things you could do here.

Run PerfSpect in thread mode sudo ./perf-collect --thread. This cannot generate an HTML file yet
Run PerfSpect in PID mode sudo ./perf-collect --pid <your-stress-ng-process-id>. I also don't think this can generate an HTML file

since these aren't supported in the HTML report yet you'll have to look in the csv for metric_CPU utilization %

closing because PerfSpect is functioning as intended. It shows system-wide cpu utilization which is very low in this case (given that only one thread is occupied at 100%)


Hide projects section

Hi @bchiang7 ,
thanks a lot for your code! How can I hide the "projects" section in the code?
Thanks a lot :-)

You could just comment out this line https://github.com/bchiang7/v4/blob/main/src/pages/index.js#L17

Thanks a lot! :-)
One last question: How is it possible to change the logo at the top and the logo at the loading screen?
Thanks you very much!

@hardcore you'll have to update the following files for this as per your need:
src/components/icons/loader.js
src/components/icons/logo.js

Hope it helps!

Thanks a lot @manitbaser


Improve ci more

This branch improves the CI support.

Make the Eowyn workflow more robust
Update the workflows to use actions/checkout@v3.
Currently, the actions log reported a deprecation warning.
Finally add a job to run the ziglings unit tests.

Added a link to the associated issues.


TCP.getsockname and getpeername not working as expected

Hi,
First of all thanks for the very nice, well documented library!
So far it's working well for me except I can't get seem to get useful data from Luv.TCP.getsockname and Luv.TCP.getpeername. I may well be misunderstanding something in the API though.
I modified the TCP echo example adding some getsockname and getpeername calls: https://github.com/djs55/luv/commit/9134c54a7aae55b2be1c79f6199cfa72ccb2ccf5
If I run the program I see
dave@m1 luv % ./_build/default/example/tcp_echo_server.exe
getsockname server: to_string = None, port = None

Although getsockname server succeeds after the bind, I can't seem to get information from the resulting Sockaddr.t with Luv.Sockaddr.to_string and Luv.Sockaddr.port.
Connecting to the server with nc localhost 7000 I then see:
getsockname server: to_string = None, port = None
getpeername client: to_string = None, port = None
^C

^ in the listen callback I can't get useful info out of either getsockname or getpeername.
My host is a macOS Big Sur Apple Silicon machine. I'm testing current decdaad7511855d012e31f22b35538ddd874c08d as well as 0.5.7. I haven't tried Luv.UDP so far.
Thanks!

Thanks for the issue.
I won't be able to look into this in the coming days. However, you should be able to narrow down exactly what is going on quite quickly, given that you already have a checkout of Luv. You can simply insert prints into the various parts of Luv code. The vendored C code is fully integrated into the OCaml build, so if you need to examine the C side, you can just insert fprintf, and your changes should be immediately picked up when you rebuild the example. The libuv code is quite well-organized, and you should be able to navigate it quickly with some help from grep.


Fix: Adjusted BPN validations on different operators

WHAT

Fixed IS_ALL_OF operator validation. It had the same behavior of EQ operator
Fixed IS_NONE_OF operator validation. It had the same behavior of the NEQ operator
Fixed NEQ operator validation. It was considering the opposite result of the IN operator, and it should be the opposite of the EQ operator
Added a new rule when the BPN groups or assigned groups are empty, and if the operator is NEQ or IS_NONE_OF, it returns true instead of false

WHY
In order to keep the correct behavior on BPNL validations
FURTHER NOTES
Some minor adjusts reported by Sonar, including:

Removed eq() method from tests that were not necessary
Adjusted String format and casting
Adjusted returning empty list instead of null

Closes #https://github.com/eclipse-tractusx/tractusx-edc/issues/1411
Closes #https://github.com/eclipse-tractusx/tractusx-edc/issues/775

TL;DR: don't just change unrelated code, don't use final except for field declarations

Thank you for the comments, I followed them and adjusted the code.


Search doesn't work

Issue Type: Bug
Hello, after the last update the search is not working
VS Code version: Code 1.44.1 (a9f8623ec050e5f0b44cc8ce8204a1455884749f, 2020-04-11T01:48:12.622Z)
OS version: Windows_NT x64 10.0.18363

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8850H CPU @ 2.60GHz (12 x 2592)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: enabledrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.82GB (6.35GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (5)

Extension
Author (truncated)
Version

vscode-eslint
dba
2.1.3

githistory
don
0.6.3

prophet
Sqr
1.3.1

vscode-stylelint
sty
0.84.0

html-preview-vscode
tht
0.2.5

for me the same

for me it was the same. i just reopened project (file -> open folder) and now it works

Yes, looks like it works. It takes some more time for search, but at least it works.
Thanks

Given that you have the Prophet extension installed, I think this is probably #94928.

I will close it since it works already after the reload.


HDR support

If anyone needs HDR support you can add RenderTextureFormat.DefaultHDR as a parameter to the GetTemporaryRT calls in CommandBufferBlur.cs

Why wouldn't you need HDR support?!

I think the issue is more that some platforms don't support HDR.
I've actually been trying to come up with a patch to handle this automatically, but I can't figure out how to approach it: https://answers.unity.com/questions/1520753/how-to-determine-if-rendering-is-actually-hdr.html

well, moments after I posted that, someone replied with a possible answer, so if it was someone here then thank you ;) I've got a pull request here https://github.com/andydbc/unity-frosted-glass/pull/9 that seems to work, although I haven't been totally able to test what happens in situations where HDR is not possible on the hardware. Feedback is welcome.

and https://github.com/andydbc/unity-frosted-glass/pull/9 is accepted!


Feature/7633 start end year

Hey @davidhabib, could you please take a look at this and let me know where else you think changes need to be made to support seasonal addresses start and end year? (There are couple more small things in this pull request, but they are just small metadata changes. I think it's easy to spot them.)

@davidhabib ~ The pull request is finally ready for review. I have made the changes we talked about, fixed couple of bugs, and created a set of tests for the additional functionality. I have also changed the name of a couple of methods and variables, just to make the code easier to read. Hopefully that won't be too confusing. :-) Oh, and I finally decided to remove that section of code that I had initially commented out just for cleanliness. The file is long enough already. I don't thing this makes the diff harder to read.
@daveperry ~ Please feel free to review it as well and make any comments, but please don't merge it. I'd like to get the thumbs up from DH, since he wrote all the original seasonal addresses code. Thanks.

@davidhabib Did my last commit on this pull request make sense? Do you feel it properly addresses the issue without any negative side effects? Thanks.


Make giphy logo optional (when legal)

For legal reasons our giphy bot shows a "Powered by Giphy" logo.
For users that sign up for Giphy accounts (presumably paid) that don't require the logo, we should have an option in the bot that turns off the powered-by logo.
The first step here is to track down the policy from Giphy and understand it to the best of your ability.
Then subsequent steps will be something like this:

add config options that the giphy bot can read
possibly check in smaller versions of the logo
add documentation to the bot (particularly if people want to use giphy images for the avatar)
possibly open server-side issues to the extent that we want powered-by-giphy in avatars (like it could be a bot setting, I don't know)

As per the terms from Giphy, the "Powered by GIPHY" logo is required.
From the docs:

As per our section 5 A of our terms of service, we require all apps that use the GIPHY API to conspicuously display "Powered By GIPHY" attribution marks where the API is utilized.

This applies even if one obtains a "Production API key"( A Production API key can make more number of requests than the normal API key. )
Infact, one of the conditions to obtain a "Production API key" is to display the attribution marks wherever the API is used.
GIPHY Developer docs: https://developers.giphy.com/docs/
GIPHY FAQ: https://developers.giphy.com/faq/

@zulipbot claim


Support for stackit_postgresflex_user password rotation

I would like to request support to rotate the password  for the stackit_postgresflex_user resource, currently, we are unable to do it by terraform, and when it is done by the web console we don't have the output updated. Other option would be the possibility to specify the password.

Hello, thanks for opening the issue.
We will investigate if we can support this in the provider


Javascript API vs Native SDKs

Hello!
I am currently developing an App with Ionic 2 and Cordova for Android and iOS.
It's a Geo-based app, I have tested what I want the app to do using the JavaScript API.
I have encountered a few issues regarding the Js API (mentioned here)

Slow loads
Limited to 25k free map loads per day

Also the directions service API has an even lower limit of 2500 requests a day...
Can someone confirm if it's 2500 requests a day per API or 2500 requests a day per user?
I am using google.maps.DirectionsService(); to calculate a route but I have NOT activated the API for directions service and it's working properly. In the quota graph, 1 is added for every map load but not for direction service request (I think its included in Js API).
Due to the limit of the 25k map loads, I am more inclined to use the native SDKs, although I am concerned about the directions service...
Does the cordova-plugin-googlemaps support directions service?
Would I need to activate the directions service API to use it with the native SDKs?
Thanks in advance!

Does the cordova-plugin-googlemaps support directions service?

Currently not supported

I don't really need directions per se.
When a user selects a level, that level has a certain distance. A polyline is drawn with a distance equal to the levels distance. A marker is placed at the end of this polyline.
The issue is drawing the polyline on roads / paths / routes a user can walk through.
Can this be done using the cordova-plugin-googlemaps?
thanks in advance!

Since this plugin does not provide navigation feature, you need to get (or have) the navigation data.
If you have one, you can draw path on the map.


The health request handler should pre-register

Developers shouldn't have to explicitly register a health request handler unless they want to override the default one. We should pre-register that ourselves, likely using the init() function in the actor module.

fixed in actor sdk v0.1.2


DateParam parsing failing on date with minutes but no seconds  (2.0)

Not a huge problem, but the handling of date parsing for ca.uhn.fhir.rest.param.DateParam does not seem to be inline with the FHIR specification.
From the FHIR spec it appears a date string value with precision only to minutes should be acceptable in a search, but HAPI (2.0) is failing unless seconds are included as well.
http://hl7.org/fhir/2016Sep/search.html#date
A simple test for this:
new DateParam(ParamPrefixEnum.LESSTHAN_OR_EQUALS, "2016-09-06T09:05");
Yields this exception:
ca.uhn.fhir.parser.DataFormatException: Invalid date/time format: "2016-09-06T09:05"     at ca.uhn.fhir.model.primitive.BaseDateTimeDt.throwBadDateFormat(BaseDateTimeDt.java:465)     at ca.uhn.fhir.model.primitive.BaseDateTimeDt.validateLengthIsAtLeast(BaseDateTimeDt.java:526)     at ca.uhn.fhir.model.primitive.BaseDateTimeDt.parse(BaseDateTimeDt.java:289)     at ca.uhn.fhir.model.primitive.BaseDateTimeDt.parse(BaseDateTimeDt.java:39)     at ca.uhn.fhir.model.api.BasePrimitive.setValueAsString(BasePrimitive.java:112)     at ca.uhn.fhir.model.primitive.BaseDateTimeDt.setValueAsString(BaseDateTimeDt.java:461)     at ca.uhn.fhir.rest.param.DateParam.setValueAsString(DateParam.java:263)     at ca.uhn.fhir.rest.param.DateParam.<init>(DateParam.java:94)

We have a similar problem.  The format listed in bold below is reporting an error for us as well.
According to the FHIR Specification:

Any degree of precision can be provided, but it SHALL be populated from the left (e.g. can't specify a month without a year), except that the minutes SHALL be present if an hour is present, and you SHOULD provide a time zone if the time part is present.

HAPI server currently allows following pattern:
2011-03-06T12:15:22.000+05:30
2011-03-06T12:15:22.000
2011-03-06T12:15:22
2011-03-06
2011-03
2011
HAPI server currently does not allows following pattern:
2011-03-06T12:15
2011-03-06T12

This was fixed in #655. Can you please try it with HAPI FHIR 2.5?

Keith, my reading is that hours-only precision is not allowed in search, but hours and minutes is. So your second value should not work, as per the spec.
And just to note, there is a distinction between what is allowed in SEARCH and what is allowed in the DateTime datatype. The Datetime datatype requires seconds to be present if hours and minutes is present.

Confirmed fixed. Sorry for the delay in closing.


[R] Broken links in section 2

It seems all the links in Section 2 are broken.
Below are some examples:

https://arrow.apache.org/cookbook/r/reading-and-writing-data.html
https://arrow.apache.org/cookbook/r/reading-and-writing-data.html#write-a-parquet-file
https://arrow.apache.org/cookbook/r/reading-and-writing-data.html#read-a-feather-file

Hi @andreranza , thanks for reporting this!  Would you mind just clarifying where these links are from?

Sure, these links are from the summary in the cookbook.
So, if I click on those titles, I cannot read the content.

Thanks for this helpful resource.

Oh, interesting, it works for me, but we've had issues in the past on other bits of the site with caching on Macs.  Does it work if you press Shift+Command+R to reload without the cache?

Yes, it works. Thanks a lot! I thought these might have broken somehow since I could open other pages by clicking on different sections - titles.

It's because we updated those sections specifically, so makes sense it looked like something had gone wrong in a different way!


Increase default limits. See GH-209.

See #209 and https://stackoverflow.com/questions/44310987/splash-memory-limit-scrapy/.

Codecov Report

Merging #615 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master     #615   +/-   ##
=======================================
  Coverage   87.27%   87.27%
=======================================
  Files          42       42
  Lines        5539     5539
  Branches      758      758
=======================================
  Hits         4834     4834
  Misses        534      534
  Partials      171      171

Impacted Files
Coverage Δ

splash/defaults.py
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


[WIP] Compound/Gibbs step and discrete suppot

Here are the examples from
@pm.model
def categorical():
    var1 = yield pm.Categorical("var1", probs=[0.2, 0.4, 0.4])
    var1 = yield pm.Categorical("var2", probs=[0.1, 0.3, 0.2, 0.2, 0.2])
    var1 = yield pm.Categorical("var3", probs=[0.1, 0.1, 0.1, 0.1, 0.1, 0.5])

Som results with different proposal generations for Poisson. I'm note sure about the poisson proposal generation though. Now it will be able to discover proper samples when scale is provided.
@pm.model
def model():
    var1 = yield pm.Poisson("var1", 4) # gaussian with round dist proposal
    var1 = yield pm.Poisson("var2", 4) # poisson dist proposal
    var2 = yield pm.Poisson("var3", 4) # continous space

trace = pm.sample(model(),
                  sampler_type="compound",
                  sampler_methods=[
                      ("var1", RandomWalkM, {"new_state_fn": gaussian_round_fn}),
                      ("var2", RandomWalkM, {"new_state_fn": functools.partial(poisson_fn, scale=4)}), # not sure if this is the right way to define proposal
                  ],
                  trace_discrete=["var1", "var2"])

@junpenglao Can you also suggest something better with the trace_discrete argument I've added to sample(). I don't like any additional argument, but it is the only way I can have discreteness in the plot for Poisson (see the Picture above). (tfd.Poisson can sample dtype tf.float32 points only, so delta should also be the same dtype)

@rrkarim is this ready for review? if so could you resolve the conflicts?

@rrkarim is this ready for review? if so could you resolve the conflicts?

Done, fixing tests.

This is mostly ready - I suggest we merge this after https://github.com/pymc-devs/pymc4/pull/311.

This is awesome contribution! Thanks and great job :-)


Cannot connect to MQTT Message bus

Below is Golang code,
package main

import (
	"context"
	"fmt"
	"github.com/edgexfoundry/go-mod-core-contracts/clients/coredata"
	"github.com/edgexfoundry/go-mod-core-contracts/clients/urlclient/local"
	"github.com/edgexfoundry/go-mod-core-contracts/models"
	"github.com/edgexfoundry/go-mod-messaging/messaging"
	"github.com/edgexfoundry/go-mod-messaging/pkg/types"
	"log"
	"time"
)

func main() {
	var msgConfig2 = types.MessageBusConfig{
		PublishHost: types.HostInfo{
			Host:     "<IP_ADDRESS>",
			Port:     1883,
			Protocol: "tcp",
		},
		Optional:map[string]string{
			"ClientId": "0001_client_id",
		},
		Type:messaging.MQTT,
	}
	if msgClient, err := messaging.NewMessageClient(msgConfig2); err != nil {
		log.Fatal(err)
	} else {
		if ec := msgClient.Connect(); ec != nil {
			log.Fatal(ec)
		}
		client := coredata.NewEventClient(local.New("test1"))
		var testEvent = models.Event{Device: "demo1", Created: 123, Modified: 123, Origin: 123}
		var r1 = models.Reading{Pushed: 123, Created: 123, Origin: 123, Modified: 123, Device: "test device name", Name: "Temperature", Value: "20"}
		var r2 = models.Reading{Pushed: 123, Created: 123, Origin: 123, Modified: 123, Device: "test device name", Name: "Humidity", Value: "30"}

		testEvent.Readings = append(testEvent.Readings, r1, r2)

		data, err := client.MarshalEvent(testEvent)
		if err != nil {
			fmt.Errorf("unexpected error MarshalEvent %v", err)
		} else {
			fmt.Println(string(data))
		}

		env := types.NewMessageEnvelope([]byte(data), context.Background())
		env.ContentType = "application/json"

		if e := msgClient.Publish(env, "application"); e != nil {
			log.Fatal(e)
		} else {
			fmt.Printf("pubToAnother successful: %s\n", data)
		}
		time.Sleep(1500 * time.Millisecond)
	}
}

Make sure that the MQTT broker is available, but it reports timeout error.
telnet <IP_ADDRESS> 1883
Trying <IP_ADDRESS>...
Connected to <IP_ADDRESS>.
Escape character is '^]'.
^]
telnet> quit
Connection closed.
RockydeMBP-2:kuiper rockyjin$ xsql/test/test
2020/03/31 15:02:04 Timeout occured while performing a 'Connect' operation: Unable to connect

@jinfahua, I think you need to add call to client.Connect() before attempting to publish. @AnthonyMBonafide is this correct?

@jinfahua, I think you need to add call to client.Connect() before attempting to publish. @AnthonyMBonafide is this correct?

That is correct, you need to call connect before publishing or subscribing. However, it seems like that is being done on the first line of the outermost else statement.
@jinfahua , can you provide and MQTT broker logs? While developing the MQTT support I found that the reasons a connection could fail were not always provided to the client. For example, in some cases I would see the same thing you shared simply because there was no ClientId provided. To find that I  needed to get the logs from the MQTT server to see what the error for the closed or rejected connection is. I was using Mosquitto to test so it might be a little different. Which MQTT broker are you using? I can try to reproduce the issue.

I tested in both mosquitto & EMQ, and it reported with the same error info. Below is the log file for mosquitto.
Apr  1 07:55:20 liufen mosquitto:<PHONE_NUMBER>: New connection from <IP_ADDRESS> on port 1883.
Apr  1 07:55:20 liufen mosquitto:<PHONE_NUMBER>: Client <unknown> disconnected due to protocol error.
Apr  1 07:55:20 liufen mosquitto:<PHONE_NUMBER>: New connection from <IP_ADDRESS> on port 1883.
Apr  1 07:55:20 liufen mosquitto:<PHONE_NUMBER>: Client <unknown> disconnected due to protocol error.

@jinfahua, I think you need to add call to client.Connect() before attempting to publish. @AnthonyMBonafide is this correct?
Lenny, below line connects to the broker, and the error is reported by this line.

if ec := msgClient.Connect(); ec != nil {
			log.Fatal(ec)
		}

@jinfahua , It looks like the client ID is not being propagated to the server. That is the error I would see when the underlying client attempted to connect without specifying an ID. I will try to reproduce with your example. Thank you for the response.

I was able to run your code with a locally running Mosquitto MQTT server. The only thing I changed was the host address. I tried both localhost and <IP_ADDRESS> and was able to get the client to connect. Here are the logs from my MQTT server:
1585750355: mosquitto version 1.6.8 starting
1585750355: Config loaded from /mosquitto/config/mosquitto.conf.
1585750355: Opening ipv4 listen socket on port 1883.
1585750355: Opening ipv6 listen socket on port 1883.
1585750364: New connection from <IP_ADDRESS> on port 1883.
1585750364: New client connected from <IP_ADDRESS> as 0001_client_id (p2, c1, k0).
1585750365: Socket error on client 0001_client_id, disconnecting.

1585750450: New connection from <IP_ADDRESS> on port 1883.
1585750450: New client connected from <IP_ADDRESS> as 0001_client_id (p2, c1, k0).
1585750451: Socket error on client 0001_client_id, disconnecting.

I am using version v0.1.16 of go-mod-messaging are you doing the same?

Those errors that are present in my previous comment's logs are due to the program exiting and not properly disconnecting the client, which is expected since we are not calling Disconnect

@AnthonyMBonafide I used v0.1.14 previously. After switching to v0.1.16, it works. Thank you!
What's the go-mod-messaging version number shipped with EdgeX Geneva? I think it would be better to keep same version in Kuiper as with EdgeX.

@jinfahua , Geneva has not shipped yet. We are aiming for a code freeze later this month which will give is a better idea as to which version will be used in the release. There will most likely be at least one more patch version release to the module once the work to add support for RedisStreams is completed.

Thanks, the issue could be closed now.


Does apollo client support transactions?

I have a nuxtjs app that uses apollo module for graphql communication with FaunaDB.
I want to make multiple mutations in one transaction to FaunaDB. So I don't want to send a different request for every single mutation to FaunaDB, I want to make a single request that contains all the mutations. Is it possible?

Put multiple mutations in on mutation document, just like you would ask for multiple fields or multiple root queries.

If you want to have true transaction with rollbacks then you have to create special mutation on the GraphQL backend server.


0.10 release

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@372402e). Click here to learn what that means.
The diff coverage is 1.01%.

@@           Coverage Diff            @@
##             master     #44   +/-   ##
========================================
  Coverage          ?   0.78%
========================================
  Files             ?      39
  Lines             ?    3062
  Branches          ?       0
========================================
  Hits              ?      24
  Misses            ?    3038
  Partials          ?       0

Impacted Files
Coverage Δ

lib/util/polyline.js
0% <ø> (ø)

lib/display/d3.geo.tile.js
0% <ø> (ø)

lib/point/multipoint.js
0% <0%> (ø)

lib/util/interpolate-line.js
0% <0%> (ø)

lib/point/place.js
0% <0%> (ø)

lib/renderer/renderer.js
0% <0%> (ø)

lib/core/patterngroup.js
0% <0%> (ø)

lib/labeler/segmentlabel.js
0% <0%> (ø)

lib/core/path.js
0% <0%> (ø)

lib/renderer/renderedsegment.js
0% <0%> (ø)

... and 29 more

Continue to review full report at Codecov.

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


Setting Path Mappings in the Portal Does Nothing

From what I can tell, setting the Path Mappings in the Azure Portal doesn't have any affect on the container. My app 100% requires a configuration file to be in a known location (as well as custom log output). This is trivial to do with local Docker and should be straightforward in the app. The Path Mappings look exactly like what I need but no amount of twiddling of the settings results in the configured Storage Containers getting mounted in my app.
The various logs I've looked at DO NOT show the expected mount points in the "docker run" command. However, checking the following command:
az webapp config storage-account lilst -n [appname] -g [resourcegroupname]
shows the two mappings that I've configured. Everything LOOKS good. Nothing WORKS.
What is going on? Is there some prerequisite configuration setting that I'm missing here?

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

ID: 1e25899c-1ed8-7cfc-2c37-73bb010a87f0
Version Independent ID: 8b366c42-43f8-d5eb-b619-b356e75a9916
Content: Configure apps in the portal - Azure App Service
Content Source: articles/app-service/configure-common.md
Service: app-service
GitHub Login: @cephalin
Microsoft Alias: cephalin

On a hunch I removed the spaces [and capitalization] from the "Name" field entries in my configuration. Suddenly the path mappings (and my app) started to work.
The documentation currently has this for the description:

Name: The display name.

It does not mention that this name is actually used by the system and must not have spaces. Please adjust the documentation to reflect this (or let me know what other voodoo I may have happened across to suddenly make it work!).
In other news, if the formatting of the field was indeed the source of the issue, please update the UI to require valid input.

@ericdrobinson, Thanks for the feedback! We are taking a look into this and will get back to you soon.

@ericdrobinson, I have assigned the issue to the content author to review further and update the document as appropriate.

@apwestgarth can you take a look at the portal feedback?

On a hunch I removed the spaces [and capitalization] from the "Name" field entries in my configuration. Suddenly the path mappings (and my app) started to work.
The documentation currently has this for the description:
Name: The display name.
It does not mention that this name is actually used by the system and must not have spaces. Please adjust the documentation to reflect this (or let me know what other voodoo I may have happened across to suddenly make it work!).
In other news, if the formatting of the field was indeed the source of the issue, please update the UI to require valid input.

@apwestgarth Here it is.

Fixed in documentation and portal warning. #please-close


Opening png files corrupts them

Describe the bug
Opening png files corrupts them
To Reproduce
Steps to reproduce the behavior:

open png file
it's corrupted

Expected behavior
Either it doesn't corrupt it or it just disallows png and other image formats
Screenshots
on phone rn so i can't get but it's very obvious (image files drop to like 7 bytes)
Desktop (please complete the following information):

OS: Windows 10
Version [e.g. v1.0.3] v1.1.1

Same for literally any binary file.

I think its because of the way griddycode handles opening files. It loads files as a string so that may be the problem.


[Request] Split manga-mode in two options

Hi,
As far as I can tell, the manga mode currently does two things:

Change reading order (next page is left)
Determine which is the first image when splitting horizontal images

The second feature is pretty much necessary if you're reading manga, but I'm not really interested in the first one (because page turn is now a different gesture than when reading traditional book). Would it be possible to split the two into two different options?
Thanks.

Technically I could be possible but I'm not going to do that.

Okay, I don't mind keeping my own patch. If anybody else is not interested in right-to-left page turns, you should modify the code (here)[https://github.com/ciromattia/kcc/blob/master/kcc/comic2ebook.py#L274]

Would you maybe accept a pull request with the proposed feature, or are you opposed to it conceptually?
I'd keep it CLI only.

There are two reasons why I don't want to touch it:

Some of casual users are already confused by number of options and I don't want to dilute options even more.
KindleGen is a black box and I already had some anomalies that discourage me to experiment with new header combinations. It is QA nightmare.


App is turning to black screen

@malik-at-work @gstanlo
Whenever I tried to launch my app it turns to black screen then after sometime I get camera visible. What could be the issue.
regards

Wouldn't it be better to ask this with a lot more information (code) on stackoverflow?

@ManuelTS  I was thinking it might be an AR issue. So thats why I  posted over here. Don't you think so?

Can you please include a video? What device is this happening on? Can you post the code for your activity class? Possibly simplified to the smallest activity that you are able to reproduce the issue with?
Thanks!

Closing due to inactivity.


Cut a 2.1.x release which includes the From<Option> for SingularPtrField.

Would be awesome to have a 2.x release which includes the impl<T> From<Option<T>> for SingularPtrField<T> code.

I think that's a good idea. I'll do that. Thanks!

I think it should be 2.2 release, because of API changes (backwards compatible, but still).

Yea, I can agree with 2.2. Seems logical. Thanks for the quick response.
I’ve been using these projects for a long time now. Thanks for all of the awesome work.

Published 2.2.0. Thank you for the feedback!


Add Bittorent Bundle

https://bundles.bittorrent.com/
For most film the site provide a trailer.
The url is in audio/video tag, you just have to change the end of the url with the filename you want (which are at the end of the page, kindly provided by the site) to retrieve the full film/album.
Example:
https://bundles.bittorrent.com/bundles/killswitch

Which should give you this url:
 https://s3.amazonaws.com/content-bundles/production-df0ec56d-0fbb-bc2c-11e7-354ff3af9c4e/445f077458b8583394423c53b88b5019f861ab0c9753b398dbf2396a88df1f66/xcode/Killswitch/Killswitch - Documentary Trailer.mp4

This only the trailer, now if you want the full film, just take the name of the file at the bottom of the page and replace it in the url like so:
 https://s3.amazonaws.com/content-bundles/production-df0ec56d-0fbb-bc2c-11e7-354ff3af9c4e/445f077458b8583394423c53b88b5019f861ab0c9753b398dbf2396a88df1f66/xcode/Killswitch/Killswitch_BT-Launch_20151019-8.5Mbs.mp4

If this is a site support requests you must provide an example URL.

Ok I have updated the issue.

API URL - https://bundles.bittorrent.com/api/v1/bundles/ +  {bundleId}
Ex - https://bundles.bittorrent.com/api/v1/bundles/killswitch

Perhaps tag this issue site-support-request?


Fix 'issues' URL in CONTRIBUTING.md

The URL for "our GitHub issue system" didn't point to NTLK's issues, but instead to that of the repository that the text was copy-pasted from. That is now fixed.
(I added my name to AUTHORS.md because it said to in CONTRIBUTING.md, but I can remove it if this contribution is too small)

Thanks @goodmami


[Python] Implement arrow JSON writer

Add support for JSON writer in python
the docs only state JSON reading is supported: https://arrow.apache.org/docs/python/json.html
Reporter: Rishabh Rana
Related issues:

[C++] JSON table writer (requires)

Note: This issue was originally created as ARROW-15230. Please see the migration documentation for further details.

Duplicate of #21529


dts: nordic: nrf54h20_enga: add new peripherals

Add new peripherals to the nRF54H20 devicetree. Also add necessary changes to the hal_nordic module glue code or SoC Kconfig files.

@nordicjm can I merge this?

@nordicjm can I merge this?

yes

@nordicjm can I merge this?

yes

note it slightly touches boards


Update package.json with jest option

Add --notify option to Jest to show test results in a pop-up window (AFAIK working in Win10)
It can be useful when test are running in a background window.

Hello @alexjoverm
Do you know what is going on with this PR? I only added a couple of options in scripts section in package.json. It should not affect builds, but reading Travis logs i can't understand what is wrong and think it could be useful to have jest notifications. Thanks.

Your PR description says add --notify but all your changes add --jest
I don't understand the point of calling jest --jest. I don't see it in their CLI options:
https://facebook.github.io/jest/docs/en/cli.html#notify

Also, from the Travis logs I'm suspecting you didn't break the build, it was already broken.  The previous commits to master which are updating package.json without updating package-lock.json which cause them to be out of sync.  This means the npm ci command will fail because it's designed to ensure package-lock.json is source of truth, where as npm install uses package.json as source of truth and will update package-lock.json.
For instance the logs say:
npm ERR! Invalid: lock file's<EMAIL_ADDRESS>does not satisfy<EMAIL_ADDRESS>
and there was PR #209 which only updated package.json and causes the issue.

Merge pull request #209 from<EMAIL_ADDRESS>https://github.com/alexjoverm/typescript-library-starter/commit/cae1aac9091102084f071eaf4266ed7cd444a5e5

It looks like he fixed it but manually updating package-lock.json here:
https://github.com/alexjoverm/typescript-library-starter/commit/0e9e3831c1360b6eab73c83fbc4da36f7a44311e
So if you re-base your changes and re-run build I suspect it would work.

Sorry, I think tests are running as expected already. Build was broken as @mattmazzola said but was fixed

Excuse me, It was an error. I made the changes quickly online. You are right, the added option should be jest --notify. That must be the reason why package.json and package-lock.json are out of sync. Do you think is it worth I try to make fork in my pc and send a new PR?

That must be the reason why package.json and package-lock.json are out of sync

No, command flags have nothing to do with package versions.
Do you think is it worth I try to make fork in my pc and send a new PR?
I don't think it's clear that this should be a default.  I assume there are people who would not want the notification distracting them.

I understand. BTW to be precise I was refering to files out of sync due to a pull request made online, not that the cause was a command flag, keep in mind English is not my first language, and congratulations for the repository. It has been very useful for me.


TFLite Interpreter.invoke()- gather index out of boundsNode number 935 (GATHER) failed to invoke.

Getting this error with the following sample code for prediction:
Model weights: https://drive.google.com/file/d/1SXbMtxQKiU-Iavh6U7lwzEZS0lYYNpJy/view?usp=sharing
import tensorflow as tf

def run_tflite_model(tflite_file,image_data,y_inp):
    #print('image_data',image_data)
    #print('y_inp',y_inp)
    #feed_f = dict(zip(['input_2', 'input_shape'], (image_data, y_inp)))
    interpreter = tf.lite.Interpreter(model_path=str(tflite_file))
    interpreter.allocate_tensors()
    #print(interpreter.get_input_details())
    input_details = interpreter.get_input_details()
    output_details = interpreter.get_output_details()
    print(input_details)
    print(output_details)
    print(image_data.shape)
    print(y_inp.shape)
    interpreter.set_tensor(input_details[0]["index"],image_data)
    interpreter.set_tensor(input_details[1]["index"],y_inp)
    print('tensor details:',interpreter.get_tensor_details())
    interpreter.resize_tensor_input(input_details[1]['index'], [1,2])
    interpreter.invoke()
    t1=time.time()
    print('yes:',interpreter)
    output = interpreter.get_tensor(output_details[0]["index"])[0]
    print(output)
    t2=time.time()
    print('time:',t2-t1)
    pred = torch.Tensor(output)
    prediction = output.argmax()
    #print(len(prediction))
    return prediction,pred

`import cv2
if name == 'main':
converted_model = "kvp_converted_model_fp16.tflite"
image_path = "kvp_test/test_1.png"
seg_image_path="kvp_test/seg_test_1.png"
#good_image_path = "datasets/experiment/good/g.png"
img1 = cv2.imread(image_path)
img2=cv2.imread(seg_image_path)
#img = img[None, :, :, :]
#resized=img
resized_img = cv2.resize(img1, (608,608)).astype('float32')
resized_seg_img = cv2.resize(img2, (608,608)).astype('float32')
#resized=img.astype('float32')
resized_img = resized_img / 255.
resized_seg_img = resized_seg_img / 255.
input_image = np.expand_dims(resized_img, axis=0)
seg_image= np.expand_dims(resized_seg_img, axis=0)
#print(test_img)
image_data = np.concatenate([input_image,seg_image], axis = -1).astype(np.float32)
y_inp = np.expand_dims(np.array([img1.shape[0], img1.shape[1]]), axis = 0).astype(np.float32)

prediction,pred = run_tflite_model(converted_model,image_data,y_inp)
print(pred)`

Error:

@mohantym
I was able to reproduce the  issue on Colab using TF v2.10. Please find the gist here for reference.
Thank you!

@divyajaincs !
Sorry for the late response.
I changed the order of resizing-set_tensor-invoke to resolve the issue.
Could you check the required changes in this gist for reference.
Thank you!

@mohantym when i'm testing with your updated code, colab session is crashing. any idea why this is happening?

@divyajaincs !
You input size in lite model  is (608,608) so I resized the tensor input accordingly , But while using a low resolution image, it crashed the session.
Thank you!

@mohantym I'm testing with original image size of (760,1000), with gpu it's working fine but with cpu it's crashing.

ok @divyajaincs !
Thanks for the update.
It is a bit strange though as I ran the above the code in a cpu-runtime only and it did not surface in my case.
@sachinprasadhs !
Could you look at this issue.
Thank you!

Hi @divyajaincs, it seems like your model link is no longer working, can you respond with your updated model link? Thanks!


Context

This includes standing PostCrafter's PR (sorry, it was in the way :))
Also:

switch to run(context)
updated test bundle (unhooked to anything though, but runs in my kernel!)
optional rootImage/startContext because it is nice to be able to say:

import { bundle as TestBundle } from "./posis/test.bundle";

posisKernel.loadBundle(TestBundle);

if (TestBundle.rootImageName)
	posisKernel.startRootProcess(TestBundle.rootImageName, TestBundle.defaultStartContext);

Other than those two things, looks good to me, although I still see no use whatsoever in rootImageName


New Tags feature implemented in provider version 3.32.0 not adding tags to Cloudflare records

Confirmation

[X] My issue isn't already found on the issue tracker.
[X] I have replicated my issue using the latest version of the provider and it is still present.

Terraform and Cloudflare provider version
Terraform v1.3.7
on windows_amd64
Using cloudflare/cloudflare v3.32.0
Affected resource(s)
cloudflare_record
Terraform configuration files
terraform {
  required_providers {
    cloudflare = {
      source = "registry.terraform.io/cloudflare/cloudflare"
      version = ">= 3.32.0"
    }
  }
}

provider "cloudflare" {
  # email   = var.cloudflare_email
  api_token = var.cloudflare_api_token
}

resource "cloudflare_record" "record" {
  zone_id = var.clouddflare_zone_id
  name    = var.record_name
  value   = var.record_value
  type    = var.record_type
  proxied = var.proxied
  tags    = var.tags
  ttl     = var.ttl
}

variable "tags" {
  description = "Tags for the record"
  type        = set(string)
  default     = (["testtag1"])
}

Link to debug output
n/a
Panic output
There are no errors displayed.
Expected output
Tags to appear on the Cloudflare records
Actual output
The code runs successfully but the tags section of the Cloudflare record are blank. Subsequent runs don't see the tags and want to apply them again.
module.cloudflare_cname.cloudflare_record.record: Refreshing state...

Terraform used the selected providers to generate the following execution
plan. Resource actions are indicated with the following symbols:
  ~ update in-place

Terraform will perform the following actions:

  # module.cloudflare_cname.cloudflare_record.record will be updated in-place
  ~ resource "cloudflare_record" "record" {
        id              = "xxxxxxxxxxxxxxxxxxxxxxxxxxxxx"
        name            = "cfdnstagtest"
      ~ tags            = [
          + "testtag1",
        ]
        # (11 unchanged attributes hidden)
    }

Plan: 0 to add, 1 to change, 0 to destroy.
module.cloudflare_cname.cloudflare_record.record: Modifying... [module.cloudflare_cname.cloudflare_record.record: Modifications complete after 1s

Apply complete! Resources: 0 added, 1 changed, 0 destroyed.
Finishing: Terraform Apply

Steps to reproduce

Deploy a record with tags.
Tags do not apply to the record.

Additional factoids
This was attempted with both set of string (as indicated in the documentation) as well as list of strings. Both ran but didn't add tags.
variable "tags" {
  description = "Tags for the record"
  type        = set(string)
  default     = (["testtag1"])
}

variable "tags" {
  description = "Tags for the record"
  type        = list(string)
  default     = ["testtag1"]
}

References
No response

already addressed via #2148


Fix returning error  when tracing is off

We should not use RETURN_IF_ERROR  for TRITONBACKEND_RequestTrace call, since when tracing is disabled, core side returns error.

Guys this error wasted 3 weeks of a mid-level engineer ...


Unable to add fauna in Syrma IV

Impossible to add a plant species, the software blocks two even though there are three.
Planet Syrma IV on Syrma system.

Good catch! Let me fix this. The next version will also allow you to unlock it entirely, since some mods may be adding more aliens.

Fixed in v1.3.0!


cpu-flex: Issue width constraint

Change-Id: Ib60b9c72bc54ae7276bef94f8a68ba3062e6d8fd
Signed-off-by: Bradley Wang<EMAIL_ADDRESS>

As for advanceInst, the original semantic (which I think is still valid at this point in time) is that it's the step taken to begin all the steps necessary to complete exactly one instruction. For every instruction for which we try to complete these steps, we call advanceInst to place a receptacle for the instruction's set of changes to committed state into the buffer (InflightInst).
As for the places we call advanceInst, our dataflow inspired logic goes thus: As soon as you have enough information to begin the first step of an upcoming instruction, make a call to advanceInst. So before we had branch prediction, we did it after the instruction was issued, since it would be done on the same tick that the instruction was decoded, which provided us with the next PCState. Now that we've potentially delayed issue, it may actually be called slightly after that. It would be possible to call advanceInst at decode time still, but we would need to (as you've suggested) change some logic that's currently in issue and move it into decode. We would then need to add a new constraint onto advanceInst or decode, in addition to issue, because it would mean that the front of the front-end would be able to run independently of the issue unit again.
As for the other places where advanceInst is called, if the predictor is not there, or is wrong, we don't know the real following PCState until execute instead of just until decode, so in that case, we would have to advanceInst in execute since we were unable to do it properly earlier. With a branch predictor, we can get a PCState early which is likely to be correct, and advanceInst early using that PCState, but if it's wrong, we have to undo the work done by that advanceInst (which is logically simple with the ROB style buffer, by just clearing after a point), and then we would call advanceInst with the correct state after execute. If the prediction was correct, then we could just use the state that the branch predictor had created.


Meanings of detection result

Hi thanks for your work.
Can you specify the meaning detection result in txt files.
Thanks

for 3D detections
  0,2,298.3125,165.1800,458.2292,293.4391,8.2981,1.9605,1.8137,4.7549,-4.5720,1.8435,13.5308,-2.1125,-1.7867   which respectively mean
  fram  type(2 means car) 2D BBOXz(x1,y1,x2,y2) score 3D BBOX(h,w,l,x,y,z,rot_y)  alpha.
May you need to see the KITTI's data format for evaluation：https://github.com/JonathonLuiten/TrackEval/blob/master/docs/KITTI-format.txt
for 2D detections
  0,296.021000,160.173000,452.297000,288.372000,0.529230  which respectively mean
  fram  2D BBOXz(x1,y1,x2,y2) score.
------------------ 原始邮件 ------------------
发件人:                                                                                "wangxiyang2022/DeepFusionMOT"                                                                                @.>;
发送时间: 2022年2月28日(星期一) 下午5:11
@.>;
@.***>;
主题: [wangxiyang2022/DeepFusionMOT] Meanings of detection result (Issue #1)
Hi thanks for your work.
Can you specify the meaning detection result in txt files.
Thanks
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

For 3D detections
0,2,298.3125,165.1800,458.2292,293.4391,8.2981,1.9605,1.8137,4.7549,-4.5720,1.8435,13.5308,-2.1125,-1.7867   which respectively mean
fram  type(2 means car) 2D BBOXz(x1,y1,x2,y2) score 3D BBOX(h,w,l,x,y,z,rot_y)  alpha.
For 2D detections
0,296.021000,160.173000,452.297000,288.372000,0.529230  which respectively mean
fram  2D BBOXz(x1,y1,x2,y2) score.

@wangxiyang2022
Thanks for your reply.
As I read through the code. I found that calib_file is not used: https://github.com/wangxiyang2022/DeepFusionMOT/blob/a09ad79e1c0235c7a97d9c6c716c21efd94e5a65/tracking/matching.py#L157

I have the same question. Could you resolve this problem?

For 3D detections 0,2,298.3125,165.1800,458.2292,293.4391,8.2981,1.9605,1.8137,4.7549,-4.5720,1.8435,13.5308,-2.1125,-1.7867 which respectively mean frame type(2 means car) 2D BBOXz(x1,y1,x2,y2) score 3D BBOX(h,w,l,x,y,z,rot_y) alpha.
For 2D detections 0,296.021000,160.173000,452.297000,288.372000,0.529230 which respectively mean frame 2D BBOXz(x1,y1,x2,y2) score.

So, what's the meaning of score in 3D detection results? If it means confidence level, Why is its value greater than 1 ？
Could you please answer this question?
Thanks

Hi @BrickDanD , did you get clarity on this. Can someone or  @wangxiyang2022  help with this?
Thanks!

For 3D detections 0,2,298.3125,165.1800,458.2292,293.4391,8.2981,1.9605,1.8137,4.7549,-4.5720,1.8435,13.5308,-2.1125,-1.7867 which respectively mean frame type(2 means car) 2D BBOXz(x1,y1,x2,y2) score 3D BBOX(h,w,l,x,y,z,rot_y) alpha.
For 2D detections 0,296.021000,160.173000,452.297000,288.372000,0.529230 which respectively mean frame 2D BBOXz(x1,y1,x2,y2) score.

So, what's the meaning of score in 3D detection results? If it means confidence level, Why is its value greater than 1 ？ Could you please answer this question? Thanks

Hi @BrickDanD , did you get clarity on this. Can someone or  @wangxiyang2022  help with this?
Thanks!


WIBNI request - support duplicate query parms on expand/collapse

Originally reported on Google Code with ID 187
I can create an expanded request and enter the same query parms multiple times (see
image arc_expandedQuery);  I do this so my eyes can see all the content without scrolling,
and pull out the shortcut text I can use on other parms.

If I collapse it and then expand again, it does not return (see second image arc_badExpandedQuery).

This is the full request/query string:
https://localhost:9443/ccm/oslc/contexts/_v_oywIqdEeKqPKwLyTGuHg/workitems?oslc.where=dcterms:modified>="1910-08-01T21:51:40.979Z"
and dcterms:identifier>"173"&oslc.select=dcterms:title,oslc:serviceProvider,oslc_cm:status,dcterms:contributor,oslc_cm:approved,oslccm:status,rtc_cm:filedAgainst,oslc_cmx:priority,oslc_cmx:severity,oslc:shortTitle&oslc.prefix=dcterms=<http://purl.org/dc/terms/>,oslc=<http://open-services.net/ns/core#>&oslc.prefix=oslc_cm=<http://open-services.net/ns/cm#>,oslc_cmx=<http://open-services.net/ns/cm-x#>&oslc.prefix=rtc_cm=<http://jazz.net/xmlns/prod/jazz/rtc/cm/1.0/>

What steps will reproduce the problem?
1. create request with repeating query parm keys when in expanded mode
2. collapse request/query
3. re-expand request/query

What is the expected output? What do you see instead?
> would like to see the expand tolerate the repeating query parm keys, so my control
of the content remains (easier on the eyes, less scroll).

On what operating system, browser and browser version?
> Windows 7; Chrome Version 27.0.1453.116 m, ARC Application version: 3.1.5

Please provide any additional information below.
> A little bit of an intersection with reported issue 153; the parsing control area
of your sweet little tool. If you can tolerate my strategy for old eyes, big thanks.
 Oh, and the summary - WIBNI = Wouldn't It Be Nice If.... (Project killers, a long
list of WIBNIs  :-}.

Reported by PatMc2Go on 2013-06-25 13:52:30

 * *Attachment: arc_expandedQuery.png![arc_expandedQuery.png](https://storage.googleapis.com/google-code-attachments/chrome-rest-client/issue-187/comment-0/arc_expandedQuery.png)*
 * *Attachment: arc_badExpandedQuery.png![arc_badExpandedQuery.png](https://storage.googleapis.com/google-code-attachments/chrome-rest-client/issue-187/comment-0/arc_badExpandedQuery.png)*

It is long fixed.


Including paths symlinked to node_modules

node_modules is ignored by default it seems.
Assuming I have symlinked my lib directory to node_modules, is there a way to still run that path through 6to5-browserify?
Also open to other suggestion on how to avoid require('../../../lib/stuff') than with symlinks to node_modules.

As @sebmck suspected, this is default Browserify behavior:

@raine's second question, about long relative paths, is also a Browserify issue, not 6to5. You can use the "browser" field to a point; or you can use symlinks; or you can try remapify or something similar.

@davidtheclark Great, thanks for looking into this!


draft: As an Attestation Issuer, I can use the DefaultPortal of Verax as my Portal

What does this PR do?
This is a draft PR. This is just to get early opinion on the implementation.
Tests needs to be added.
Related ticket
Fixes #63
Type of change

[ ] Chore
[ ] Bug fix
[x] New feature
[ ] Documentation update

Check list

[ ] Unit tests for any smart contract change
[ ] Contracts and functions are documented

Codecov Report

Merging #70 (cd8f144) into dev (223189a) will decrease coverage by 28.41%.
The diff coverage is 13.79%.

@@             Coverage Diff              @@
##               dev      #70       +/-   ##
============================================
- Coverage   100.00%   71.59%   -28.41%
============================================
  Files            5        6        +1
  Lines           61       88       +27
  Branches        20       24        +4
============================================
+ Hits            61       63        +2
- Misses           0       25       +25

Files Changed
Coverage Δ

src/ModuleRegistry.sol
68.42% <0.00%> (-31.58%)
:arrow_down:

src/portal/DefaultPortal.sol
0.00% <0.00%> (ø)

src/PortalRegistry.sol
85.00% <57.14%> (-15.00%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Proposal to remove the bytes attestation payload in favor of a sturct AttestationPayload or AttestationRequest. I think it is cleaner here to clearly define and enfoce the parameters passed for the attestation creating


question : how to trigger search programmatically (action search)

hello ,
i want to trigger the search key action search , programmatically .
any pre defined method ?

Hi @mhdjazmati , that would surely be out of the scope of this library.
Thanks,
Ari


PowerManager algorithm that can shift or limit bounds

What's needed?
Currently the power manager has a 2-group approach, where each group has a Matryoshka implementation that supports only limiting of bounds.  And the first group shifts the second group's bounds.
We need a more flexible implementation where each actor is able to decide whether they want to shift or limit the bounds of the lower priority actors.
The following changes are necessary to achieve this:

The priority becomes optional.
A proposal with just a target power and without bounds would shift the other actors.
A proposal with target bounds and optional target power would limit other actors with no priority or lower priorities.

Proposed solution
No response
Use cases
No response
Alternatives and workarounds
No response
Additional context
No response

It might be good to add some examples on how we envision this new algorithm to work so we don't forget, as I'm not sure when we'll have time to actually implement it.


Create a worker pool that supports job streaming

So many of the examples, documentation and other go packages do not support the model where the issuance of jobs to a worker pool can be made in bursts and at anytime up to and including some known end point. A lot of the examples discovered assumes the client knowns the full job stream up front; ie we see examples where a client will create a slice containing all the jobs which are then subsequently dispatched to the pool and the channel immediately closed. These are really noddy examples that don't reflect the complexity of the real world. So we need to roll our own.
The worker pool must have the following features/properties:

must contain a means of reporting errors, via an error channel
must have an output channel off which the client can receive results
must be observable compatible; this is because this needs to be able to be wrapped inside an observable to support the reactive model
must be able to support reactive operators (only a small set of operators, ie only the ones required by extendio)
must be able to support reactive options
appropriate channels must be based upon generics rather that reflection/interface{}. But we also need a specific type let's call this envelope, which defines fixed properties for internal requirements and a client defined generic parameter, let's call this the payload.

Channels in play:
🔆 jobs (input)
🔆 results (output)
🔆 errors (output)
🔆 cancel (signal)
🔆 done (signals no more new work)
▶ ProducerGR(observable):
- writes to job channel
▶ PoolGR(workers):
- reads from job channel
▶ ConsumerGR(observer):
- reads from results channel
- reads from errors channel
Both the Producer and the Consumer should be started up immediately as
separate GRs, distinct from the main GR.
* ProducerGR(observable) --> owns the job channel and should be free to close it
when no more work is available.

* PoolGR(workers) --> the pool owns the output channels

So, the next question is, how does the pool know when to close the output channels?
In theory, this should be when the jobs queue is empty and the current pool of
workers is empty. This realisation now makes us discover what the worker is. The
worker is effectively a handle to the go routine which is stored in a scoped collection.
This collection should probably be a map, who key is a uniquely generated ID
(see "github.com/google/uuid"). When the map is empty, we know there are no more
workers active to send to the outputs, therefore we can close them.
TODO:

[ ] #286
[ ] add error reporting channel to worker pool
[ ] add ability to define functional worker pool options
[ ] add ability to define functional worker pool operators

This is being spun out into a new package, lorax


[Feature] Visual filters

Device: TV
OS: Android TV
Browser: Kiwi

I want to suggest this feature:
... Visual options such as grain filter, high contrast, black and white, or maybe HDR

Not possible with JS & CSS.


:art: add f-strings and convert .format() to f-strings

resolves #6241

Description

[x] Adding a missing f-string to https://github.com/dbt-labs/dbt-core/blob/68d06d8a9cea28600ed9c3f375605c9467247316/core/dbt/events/types.py#L539-L543 as noted in #6241
[x] Found a few other functions that had some incomplete f-string formatting and fixed those
[x] Converted most .format() strings to f-strings to have consistency throughout the core/dbt/events/types.py file

Checklist

[x] I have read the contributing guide and understand what's expected of me
[x] I have signed the CLA
[x] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[x] I have opened an issue to add/update docs, or docs changes are not required/relevant for this PR
[x] I have run changie new to create a changelog entry

sorry for the assign/notification - thought converting draft -> PR would refresh the cla-bot as i fixed my git config issue locally (forgot to set it on a new machine 😅 ). i believe i'll need to close/re-open a new PR to ensure my email is consistent across each commit. sorry about that!


Endpoints tracking

Should endpoints be defined and tracked in README.md?
If yes, I propose following table:

op.
URL
impl
tests

GET
/api/students
✅
❌

POST
/api/students
❌
❌

If no, I'll remove it.

Yes, we want endpoints to be defined and tracked in the README.md, and your proposal looks pretty. We like pretty things.


grafana: made phantomjs2 optional

On servers especially, phantomjs2 pulls graphical dependencies which is unecessary.
This pathes enable the package to be linked/installed without
phantomjs2.

Motivation for this change
On systems with environment.noXlibs = true;, grafana pulls a lot of dependencies with the current package because phantomjs2 depends on X libraries.
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"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

I pushed the default to false.
Where can I find the info to write a release note ?

You can find it in nixos/doc/manual/release-notes/rl-2009.xml (probably add it to Backward Incompatibilities). You can test the change by running nix build --print-build-logs -f nixos/release.nix --arg supportedSystems '[ builtins.currentSystem ]' --option builders '' manual and then $BROWSER result/share/doc/nixos/release-notes.html.

Done!

Thank you for your help, I couldn't have found it with the logs...