ExpansionTile class, add a controller option to pass to its constructor or expose the controller property.

It's doing what it was designed for. I am merely suggesting another property that I find valuable. To allow for expanding and collapsing the widget programmatically. From the source code, this action of expanding and collapsing is being handled by an AnimationController widget. By simply calling reverse or the forward method of the controller, we are able to control the expansion etc. Could we expose the controller property to the widget's API so that we can have access it to.
Consider this scenario:
I want to programmatically collapse ExpansionTile after 10 seconds. Right now this is quite impossible, given the current strucutre of the ExpansionTile code snippet. But it would be possible if we had acccess to the controller as so
ExpansionTile(controller: controller)
We will have to create and dispose the AnimationController ourselves.
or You could also expose the controller. Right now it is a private property

@therenca
Check this solution and see if it works for you.
Thanks.

Here, we still have to interact with the widget to update state. Besides, this solution could have been done with an ExpansionPanel widget. Is there a solution where the expansion tile can expand and collapse by using a timer and a callback without user interacting with the widget?

I recently had the same issue. If the goal is just to expand / collapse the ˋExpansionTileˋ programmatically, wouldn't it be easier to make _ExpansionTileState public and add a setExpanded(bool) method? This could then easily be accessed using a key (see ˋFormState.validate()ˋ).
Any opinions on this? If this is a good idea, I'd like to look into it.

Is this issue still valid? I think ExpansionTileController solves this issue.

This has been fixed, see ExpansionTileController (as noted in https://github.com/flutter/flutter/issues/67724#issuecomment-1838994137)


jnilibs issue

A month ago, I downloaded your code and was able to sucessfully use it in the android device. Then few days back, I updated the android/gradle version from 2.1.0 to 2.1.2. Then onwards the following error is coming :  java.lang.UnsatisfiedLinkError: dlopen failed: cannot locate symbol "sigfillset" referenced by "libcaffe.so"...
After that I even re downloaded it and compiled but no success. The android studio version that I am trying is the following. Could you tell me if I am using a higher version. Also, did you create the jnilibs using caffe-android-libs git repo?

First, please refer to https://github.com/sh1r0/caffe-android-demo/issues/40#issuecomment-256854306. And yes, this project is for demonstrating caffe-android-lib, btw, the prebuilt libs in this project are outdated.

Thanks for your reply :) I forgot to check in the closed issues


test: add integration tests for nft permissions

Task linked: CU-2kbujbr pallet-nft integration tests for basilisk

This tests are not specific to marketplace, I think they should be in separate NFT integration tests


Cannot debug the instance started with runIdea

I am starting Gradle with debug parameters and getting Listening for transport dt_socket at address: 5005 message, but I cannot start at breakpoints inside the IDEA instance started with runIdea
What is the recommended way to debug plugins?

I didn't understand.

I am starting Gradle with debug parameters

What parameters? And where did you add them?

but I cannot start at breakpoints inside the IDEA instance started with runIdea

Have you attached with remote debug to gradle?

What is the recommended way to debug plugins?

In the same way as for other applications: via run configurations (without debug-parameters and remote debug)

https://github.com/JetBrains/gradle-intellij-plugin/issues/9#issuecomment-139237985

I have environment variable GRADLE_OPTS=-agentlib:jdwp=transport=dt_socket,server=y,suspend=n,address=5005
I am attaching remote debugger to Gradle successfully, but it ignores breakpoints.
Is it possible that IDEA is launched in a separate JVM?
P.S. After I configured plugin build to run via Gradle run/debug buttons in the IDE start new IDEA instance without the plugin.


Feature request: parameter to "docker plugin install" to allow permissions

File: engine/extend/index.md
To install a plugin, the "docker plugin install" seems to be interactive.
 If not, can we please have that ?
If not, we have to drop to writing dirty scripts to wrap around this interaction to make it automatic.

@taherv Add the --grant-all-permissions flag :)


How to create my own handler?

Also I found another problem, although it is not the fault of the plugin itself.
For example hello = world becomes hello u003d world.
I'm sure it's an issue with translate-shell .
I was wondering how to create a handler myself? Below is my config file:
require("translate").setup(
    {
        default = {
            command = "translate_shell",
            output = "floating",
            parse_before = "trim",
            parse_after = "no_handle"
        },
        preset = {
            output = {
                split = {
                    min_size = 8
                }
            }
        }
    }
)

I feel that this kind of error is not caused by the plugin itself and should be resolved by the user.
    for index, char in ipairs(data) do
        if char:find("u003d") then
            data[index] = char:gsub("u003d", "=")
        end
    end

Wasn't it mentioned in the help?
require"translate".setup({
  parse_after = {
    cmd = function(lines)
      ...
    end
  }
})

Please see lua/translate/preset/parse_after/

If you use trans on the command line and it does the same thing, then it's not a plugin issue. I can put it in parse_after/translate_shell if it's reproducible.

No matter where I put it, it doesn't work. it won't run...
require("translate").setup(
    {
        default = {
            command = "translate_shell",
            output = "floating",
            parse_before = "trim",
            parse_after = "no_handle"
        },
        preset = {
            output = {
                split = {
                    min_size = 8
                }
            },
            parse_after = {
                -- useless
                cmd = function(line)
                    print("ok", line)
                end
            }
        },
        parse_after = {
            -- useless
            cmd = function(line)
                print("ok", line)
            end
        }
    }
)

I tried it on the command line, is translate-shell itself causing this problem:
$ trans -to en "var hello = 123"
var hello u003d 123

Oh sorry, I made a mistake.
require"translate".setup({
  parse_after = {
    my_handler = {
      cmd = function(lines)
        ...
      end
    }
  }
})

Ok, I understand how to do it, thanks for your answer.


Nodeagent default install not working

I did a helm install with nodeagent.enabled = true - it doesn't seem to work. There are some values (ca address) that are not filled in, I can't see how this is passing the tests.
Are we sure the e2e tests are enabled with node agent ? If we want to ship it in 1.1 we need to make
sure it gets a lot of stability and usability testing - right now I can't even install it...

The features are implemented. Now the node agent doesn't install by default - you still need to set a couple of parameters. We are fixing it. Now we are working on the integration tests. Our goal is to make everything done before 1.1.


Showing BitTorrent v2 files hashes?

Hello, it would be a giant feature leap forward if you could show hashes of files in the corresponding web pages.
Since BitTorrent v2 / hybrid torrents contain root hashes of files in torrent metadata, including fetched from DHT network, it could open a possibility for searching for the same files inside many swarms therefore widening search for users and opening a possibility to revive dead swarms or find active ones with valuable files.
In short you check "meta version" key in info dictionary which equals to 2, and file hashes in file tree ( "pieces root" key ) which are in binary format and can be easily put back into hexadecimal ( for example, in php function for this would be bin2hex() ):

Showing bt v2 magnet links is optional, new clients will catch them from peers automatically.
There're no DHT indexers yet providing this option, so it could be a nice piar for your project.
End users will be able to extract root hashes from .torrent files by simple convenient tools for that already created, including my own.

Also it fixes long standing issue with DHT indexers in form of file masking / corrupting.
Since clients can deceive indexers by fake filenames and sizes to make it more difficult to search or waste time of users.
If DHT indexer validates piece layers agains root hashes before adding to database, it can reject fake ones.
If client knows only one true hash then it can filter out all the illegetimate swarms.

No answer followed


Invasive plant form fixes

[x] move "check form for errors" and "copy form data" buttons to the bottom of the form (FYI we will be adding 1-2 additional buttons as well called "save form" and possibly "submit form" TBD)
[ ] currently there are two headings in the observation forms that say "terrestrial plant information" (with no fields under it), followed by "Observation Data".  The first heading is redundant and confusing - merge the two so there is only one heading here called "Terrestrial Invasive Plant Observation Data".

There is no simple way to change two headings there. Those are bound to our objects and after removing the title from either one, it would show the name of the object instead. An example would be: after removing the "Terrestrial Plant Information" title, it would show activity_subtype_data instead.


Don't use unicode()

toga_gtk/widgets/textinput.py uses unicode(), which is not available in Python 3.
This breaks the second example in the docs: http://toga.readthedocs.org/en/latest/tutorial/tutorial-1.html
For Python 3, I guess it would make sense to use str() instead, but I don't know if this breaks anything on the Python 2 side ...
If it's actually meant to be Unicode in both Python 2 and 3, you could do something like this:
try:
    _unicode = unicode  # doesn't exist in Python 3.x
except NameError:
    _unicode = str

Thanks for the report. That's definitely an oversight. The adaptor you describe already exists in the libs/utils module; I've just neglected to use it here.

This has become a moot point in the 0.2.2 release, which is Python3 only.


Add Instructions for Distrobox

Hi, thanks for the SDK, this definitely makes setting up a development environment much easier!
As a distrobox user, I did something like:
distrobox create --image ghcr.io/wkdev:latest wkdev
distrobox enter wkdev
git clone https://github.com/WebKit/WebKit.git WebKit
cd WebKit
Tools/Scripts/build-webkit --gtk

Would you be open to adding distrobox instructions?

While the image alone is very useful, we also have some custom scripts that are expected to be ran.
Maybe we could document that, someone will need to test and write it all down.

What came to your mind @TingPing?
@danyeaw - did you have any issues building + using WebKit this way?

Hi @nikolaszimmermann, with the steps above things worked great.

What I meant is we have a list of custom permissions:
https://github.com/Igalia/webkit-container-sdk/blob/2bed97b7678ec3ab1dc7f44baa7dd580c5a8cd88/scripts/host-only/wkdev-create#L362
And we have an init script:
https://github.com/Igalia/webkit-container-sdk/blob/main/scripts/container-only/.wkdev-init
Distrobox itself handles some of the things we do but not necessarily all. So running WebKit may behave differently.

Hi @TingPing, thanks! I think we can go ahead and close this one then.


updating to protobuf 3.6.0

Thank you for your pull request. Before we can look at your contribution, we need to ensure all contributors are covered by a Contributor License Agreement.
After the following items are addressed, please respond with a new comment here, and the automated system will re-verify.

User @bagmangood isn't covered by a CLA. They will need to complete the form at https://identity.linuxfoundation.org/projects/cncf

Regards,
CLA GitHub bot

This is going to be handled by https://github.com/grpc/grpc/pull/15597 - there's a few glitches with upgrading protobuf that I am currently looking into. Said PR should resolve those.

@jtattermusch thanks - was looking into getting the ruby_package updates in - I'll rebase it off that branch


Check positive dilation in F.convolution_2d

Close #7304.
This PR make F.convolution_2d check if the supplied dilation is positive integer.

CIs, test this please.

Jenkins CI test (for commit 053772efb15bba8ccf09ac9dd96941c774d72a83, target branch master) succeeded!


TSServer needs some way of specifying editor options

Currently, the options are hard-coded (see here).
We need some way of letting the client specify their editor options to fix issues like https://github.com/Microsoft/TypeScript-Sublime-Plugin/issues/382

We need some way of letting the client specify their editor options to fix issues like

How about using tsconfig.json : https://github.com/TypeStrong/atom-typescript/blob/master/docs/tsconfig.md#formatcodeoptions :rose:
At some point I imagined seeing TSServer do this. Must have been mistaken.

:+1: for adding editor options to the tsconfig in the default host.

It should be really cool too if tsserver could support editorconfig http://editorconfig.org/ which is a lot of used and supported by many IDE.

On a second thought, I think putting editor options inside tsconfig is not an optimal solution. For example, you would have to have a tsconfig file to set editor options, which might bring side effects if you don't previously have one. In addition, it would overlap with the settings in editors in vscode / editorconfig file, and things get complex if there are conflicts. Also, there would be no way to change the "default" behavior (loose file scenario).
As tsserver already provides APIs for the host to configure editor options here, I think the support for editorconfig files can be done by VSCode or other hosts. As VSCode already has workspace specific setting files, that solves per-project setting scenario too. The reason why it doesn't work today is we didn't honor the configure options passed in, so as long as we fix that we should be good.

I think we still need this issue open to track the progress of honoring the editor options passed by the editor, and the issue remains a bug because currently it doesn't work as expected. Reopening.

@zhengbli we have https://github.com/Microsoft/TypeScript/issues/6923

@mhegazy makes sense. And it seems that other than the new line character, nothing else is hard-coded. I'll send a PR for that separately. Closing this.

On a second thought, I think putting editor options inside tsconfig is not an optimal solution

Agreed. This time with alm I went with a bit more staggered approach. EditorConfig for the options that it provides : https://basarat.gitbooks.io/alm/content/config/editorconfig.html and then tsconfig.formatCodeOptions for the ones specific to TypeScript : https://basarat.gitbooks.io/alm/content/config/tsconfig.html#formatcodeoptions :rose:

@basarat nice to hear about your solution. Though does that mean if the user wants to change the indent size, he/she needs to have a dedicated editorconfig file?

he/she needs to have a dedicated editorconfig file

Yes. But the good news is that if they edit it they get the same experience for all their team mates that might use some other editor http://editorconfig.org/#download :rose:

Indeed. With so many different requirement to meet, each solution is a double-edged sword.


feat: Add view for get/set ERC20 balances (#39)

vid: https://twitter.com/dittoproj/status/1699904944898240966?s=20

[x] button to take account view
[x] add tokens
[x] view bal
[x] remove tokens
[x] set bal
[x] handle tokens that change balanceOf like rebasing token (stETH)
not doing erc721/1115 in this pr

@jxom said we can add a new issue to embed revm to do this better but my workaround might work for now for most tokens.
This setERCBalance is a bit like fuzzing! By assuming the balanceOf mapping is usually near slot 0, we can iterate through each and randomly check values to set we are setting the right storage slot.

loop through each slot, starting from 0 (maybe stop at some point, I just hardcoded it to 10)
setStorageAt() with a special value (if we used the user value, it might be 0 or a number that a slot might already be so I want to make sure it's weird enough that it's obvious we calculated it right I think?)
check the val of balanceOf == value
if not, guess++ (get the next slot, from 0 to 1, etc)
if yes, profit! (we've found it!, maybe this could be saved in the tokens store locally, or could just hardcode special tokens)

Note: one issue is ERC20 tokens that use special slots (assembly, diamond storage, etc) - this stops checking at slot 10. Those would have to be hardcoded or we could read the source code to find the storage constant, not sure how easy that is?

q:

should erc20 be stored separately from 721/1155? Or put all into one as diff types
copied the abi file since not importing from wagmi

Note: one issue is ERC20 tokens that use special slots (assembly, diamond storage, etc) - this stops checking at slot 10. Those would have to be hardcoded or we could read the source code to find the storage constant, not sure how easy that is?

I think we can merge w/o this, document as a known limitation, and we can add an issue for future usage. Running with the revm in WASM & capturing the storage slots accessed during balanceOf is useful here. I'll see if I can get us a nice library for this.


Category sync

With this PR we are able to update categories to any supported update action - currently there are 6 actions.
@butenkor and @emmenko  please review.

Looks good, clean and simple ;)
Look at my comments, then you can merge :+1:

So sync will handle category updates only? Creates and deletes will be handled by https://github.com/sphereio/sphere-category-sync?


Custom conditionals?

Hi @sirlantis
I'm porting a project from Ruby to Liquid, and using this library to do it. So far, so good. Thanks for writing it!
I'm hitting one issue, though. We have a custom conditional that looks like this:
{% if page.version ver_gt "2.10" %}
   - The appliance is part of a Geo-replication configuration, but is an inactive replica.
{% endif %}

Is it possible to register a custom conditional like this? I see in the README that there's support for "asynchronous variables, tags, functions and filters (helpers)", but I'm not sure exactly what each these things are. Maybe a custom function? Any guidance is appreciated, and I will happily open a PR to help clarify documentation about whatever I learn here.
Thanks for reading!

I dug into the code and discovered it's possible to attach functions to the Liquid.Condition.operators object. I'm not sure if that object is intended to be private, but this works:
const Liquid = require('liquid-node')

Liquid.Condition.operators.ends_with = (cond, left, right) => {
  return left.endsWith(right)
}

module.exports = new Liquid.Engine()

I'd be happy to open a PR documenting this in the README. @sirlantis let me know if you're open to that.

I actually never thought of that people would want to add custom operators.
So, yes, your proposal would be the a way to do it.  The object is indeed intended to be private. So ultimately, it would be nice if one could do this without modifying the global Liquid.Condition but rather by passing custom operators to the Liquid.Engine (like one can do with custom filters). That would allow for engines with different custom operators in the same project.


[To rel/0.13] [IOTDB-2754] SessionPool auto redirect IoTDB instance (#5267)

Coverage increased (+0.01%) to 67.906% when pulling 3cb106386cf1d6ce05b7e2624b3095fb3295ec9e on CRZbulabula:SessionPool_redirect into 39027b52f0c3479ff6dac5f4709c4d79564a50b1 on apache:rel/0.13.


Update for CUDA 12.3.2

xref: https://github.com/conda-forge/cuda-feedstock/issues/15

@conda-forge-admin, please rerender

Canceled and restarted the Travis CI job as it looked stuck

Looks like that did the trick


Update MD file on new branch 2017-17-16_00-11-58

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestRef.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Feature/Registration Schema Relationship [PLAT-1001]

Purpose
The going-forward ember method of connecting related resources is by manipulating the relationship instead of setting attributes, and this endpoint is just now being used by ember.
Changes

Specify the registration_schema for a draft registration via relationships instead of an attribute.

Before
POST localhost:8000/v2/nodes/<node_id>/draft_registrations/

{
    "data": {
        "type": "draft_registrations",
        "attributes": {
            "registration_supplement": "<schema_id"

        }

    }
}

After (empty attributes until PLAT-1000 gets rid of this)
POST localhost:8000/v2/nodes/<node_id>/draft_registrations/

{
    "data": {
        "type": "draft_registrations",
        "attributes": {},
        "relationships": {
        	"registration_schema": {
        	    "data": {
        	        "type": "registration_schemas",
        	        "id": "<schema_id>"
        	    }
        	}
        }

    }
}

Errors in relationship field are serialized under source > pointer as error being under relationships instead of attributes

{
    "errors": [
        {
            "source": {
                "pointer": "/data/relationships/registration_schema"
            },
            "detail": "This field is required."
        }
    ],
    "meta": {
        "version": "2.0"
    }
}

QA Notes

API tests passing are sufficient checks.  Will be better tested on front end.
Documentation
Will wait until approved before writing docs
Side Effects

Ticket
https://openscience.atlassian.net/browse/PLAT-1001

This is going to staging3.

addresses an issue found on staging3 by @Johnetordoff while testing https://github.com/CenterForOpenScience/osf.io/pull/8565.
This PR attempts to properly pinpoint the location of the incorrect field in your error message (something like error in data/attributes/title).  Now that we will sometimes be posting to relationships, code was added to look at the serializer and figure out if the errored field is a RelationshipField or not.  This doesn't if the serializer to make the request is different than the serializer to make the response (which is in the case in UserSettings).
This is a rare enough case that I am assuming the error was in attributes by default, unless your field is a RelationshipField.  You can't assume the errored field is on the serializer in question.


TEST Env - Admin Page - Promoted Users cannot see action button

Describe the bug
Dev level users can edit permissions from the Admin Page.
System Admin, Instance Admin, and Superusers should also be able to edit permissions.
right now, Instance admins cannot see the action button.
we need to check to see if the other roles are also broken and fix any that are not functioning correctly.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
any role above viewer can see the admin option in the menu.
any role above viewer can promote a user with lesser permissions to a max of that role
any role above viewer can demote a user with lesser or equal permissions to a min of viewer.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

superusers should be able to see the admin menu option

Fixed! thanks @dinesh-aot!


Add a new data drift detection method to Evidently

This is an open issue. We invite you add new data drift detection tests or metrics to the Evidently library.
Here are general instruction on how to add a new test:
https://github.com/evidentlyai/evidently/wiki/Hacktoberfest
We invite you contribute new drift detection methods that you consider useful. If you want to add a new method, you can create a separate issue to let us know that you are working on it. You can also directly send the pull-request.
Please do share some details about the new method and when it can be useful (or when you are using it!)
Each contribution should contain:

Test implementation
Unittest
Docs update
Example update

Our goal is not to implement as many statistical tests as possible (as the choice can be confusing to the users!) but to add tests that have different properties and might be chosen over already existing ones for a specific reason.
We highly welcome additions that come from your practice: e.g., if you, as a data scientist, already use a different drift detection method in your work.
Here are some ideas for methods you can explore:

Bhattacharyya distance https://en.wikipedia.org/wiki/Bhattacharyya_distance
Energy distance https://en.wikipedia.org/wiki/Energy_distance
Total variation https://en.wikipedia.org/wiki/Total_variation
Matrix or vector norm https://numpy.org/doc/stable/reference/generated/numpy.linalg.norm.html

You can also study implementations that exist in research papers and propose ones that could be a valuable addition. Please cite the paper if you implement any of the approaches proposed by the authors so that we reflect it in the documentation.

Mahalanobis distance is also another candidate.
Here is a paper that discusses about Mahalanobis distance and its applications in Outlier detection
https://core.ac.uk/download/pdf/233075917.pdf


native worker output

moves our API closer to rspec's actual API and allows for real RSpec test output on the workers

thanks for the helpful nudge, bin/check_yard


⚠ Bofrost.fr has degraded performance

In c8e5b7c, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 972 ms

Resolved: Bofrost.fr performance has improved in 101c801.


Hotfix

Codecov Report
Attention: Patch coverage is 63.63636% with 4 lines in your changes are missing coverage. Please review.

Project coverage is 87.73%.  Consider uploading reports for the commit 3723555 to get more accurate results

Additional details and impacted files

@@             Coverage Diff             @@
##           develop      #24      +/-   ##
===========================================
- Coverage    88.38%   87.73%   -0.66%
===========================================
  Files           11       11
  Lines          482      489       +7
===========================================
+ Hits           426      429       +3
- Misses          56       60       +4

Files
Coverage Δ

lyncs_setuptools/__init__.py
100.00% <100.00%> (ø)

lyncs_setuptools/cmake.py
91.96% <100.00%> (ø)

lyncs_setuptools/pylint.py
77.35% <55.55%> (-5.26%)
:arrow_down:


Swift package manager support + compile from command line

Fixes #4

import Cocoa is needed only if you compile the project for macOS platform; without it the compiler laments with "use of undeclared type 'NSError'" type errors. Sorry for not checking the build break on iOS.


Update to newest android embedding

The example won't run because of migration issues with the version of the Android
embedding.
https://flutter.dev/go/android-project-migration

+1

+2

+3

+4

+5


luci-base: update Japanese translation

sync translations
updated Japanese translations.

Signed-off-by: INAGAKI Hiroshi<EMAIL_ADDRESS>

Merged. Thanks - as always, much appreciated! :)


Expired Invoice - Wrong Redirect

Allow an invoice to expire
Displays “What happened?  Your invoice expired.  Invoices only last 15 minutes.  What can you do now?  You can return to Online Health Networks Inc, and try your purchase again…”
Online Health Networks Inc links to my “redirect” URL:   /shop/checkout/onepage/success/
It should not redirect to this page.  This page is for successful orders.  It should redirect to the cart page so they can try the payment again.

Fixed in release 2.1.18


Solved: Function components cannot be given refs. Attempts to access this ref will fail. Did you mean to use React.forwardRef()?

Here, you can also try this solution its work fine for me.
It will not generate an error of Function components cannot be given refs.
/* Child.jsx */
import React from 'react'

class Child extends React.Component {
  componentDidMount() {
    const { childRef } = this.props;
    childRef(this);
  }
  componentWillUnmount() {
   const { childRef } = this.props;
    childRef(undefined);
  }
  alertMessage() {
    window.alert('called from parent component');
  }
  render() {
    return
Hello World!

  }
}

export default Child;

/* Parent.jsx */
import React from 'react';
import Child from './Child';

class Parent extends React.Component {
  onClick = () => {
    this.child.alertMessage(); // do stuff
  }
  render() {
    return (

        <Child childRef={ref => (this.child = ref)} />
        <button onClick={this.onClick}>Child.alertMessage()</button>

    );
  }
}

Originally posted by @skychavda in https://github.com/reactjs/reactjs.org/issues/1730#issuecomment-508056509

thank you really helped

Hi,
It is obvious that if we are using class component, there will be no error. But if you check react webside doc, one of the examples is with functional component, so something else is the problem or the doc is not correct!
Can someone check that?
Tnx

Here is the another solution regarding the forwardRef problem in functional compnent
/* Class Component */
import React from 'react';
import FuncComponent from './func_com';

class ClassComponent extends React.Component {

  changeFunCompText = (text) => {
    this.funcComRef(text);
  }

  render() {
    return (

class component

        <button onClick={() => this.changeFunCompText('from class component')}>function component click</button>
        <FuncComponent forwardRef={c => { this.funcComRef = c }} />

    )
  }
}

export default ClassComponent;

/* Functional Component */
import React, { useState } from 'react';

const FuncComponent = ({ forwardRef }) => {
  const [text, setText] = useState('functional');

  forwardRef((text) => renderText(text));

  function renderText(text) {
    setText(text);
  }

  return (

this is {text} component

  )
}

export default FuncComponent;

Hello,
I think this solution is a workaround solution. Because both Child and Parent component are class component. Not the function component.
So I think react should support function component use ref the same as class component.
Do you think so???

A functional component isn't backed by an "instance" of anything, so it can't handle ref's passed to it automatically.
BUT, if you wrap your functional component in forwardRef, then you get the ref and can do whatever you want with it. As a matter of fact, the FancyButton example in react's documentation does exactly this:
const FancyButton = React.forwardRef((props, ref) => (
  <button ref={ref} className="FancyButton">
    {props.children}
  </button>
));

// You can now get a ref directly to the DOM button:
const ref = React.createRef();
<FancyButton ref={ref}>Click me!</FancyButton>;

To just make it crystal clear, you don't have to take the ref and pass it to a DOM node, you can capture it and do whatever you like. I typically do something like this:
// this component gets a second argument which is the ref ONLY if it's wrapped in React.forwardRef
const MyFunctionalComponent = React.forwardRef((props,ref) => {

   const someInternalRef = useRef('someValue').current;

   useEffect(() => {
      if(!ref) return;
     // expect that if a ref has been given to this component, it is either a function or an object created by React.createRef();
     typeof ref === 'function' ? ref(someInternalRef) : (ref.current = someInternalRef);

     //  clean up (hint: 99.999% of the time the only time the clean up function will be called is when this component unmounts)
     return () => typeof ref === 'function' ? ref(null) : (ref.current = null);

    // this is to satisfy the exhaustive dependency eslint rule of hooks. In practice, it's **likely** this hook will only ever get fired twice - when the component mounts and when it unmounts as the `ref` and someInternalRef will (again, **likely**) never change.
   },[someInternalRef,ref])

  return (
    ...
  )

});

In this example:
import React, { useState } from 'react';

const FuncComponent = ({ forwardRef }) => {
  const [text, setText] = useState('functional component');

  forwardRef((text) => renderText(text));

  function renderText(text) {
    setText(text);
  }

  return (

this is {text}

  )
}

export default FuncComponent;

why don't you just do:
import React, { useState } from 'react';

const FuncComponent = ({ forwardRef }) => {
  const [text, setText] = useState('functional component');

-  forwardRef((text) => renderText(text));
+  forwardRef((text) => setText(text));

-  function renderText(text) {
-    setText(text);
-  }

  return (

this is {text}

  )
}

export default FuncComponent;

Is it on purpose or we must wrap the setText ?

@kopax You are right! It can be that way also when I tried that program to solve the Ref problem it will work for me and I've posted that not checked with react hooks. Thanks for correcting me for a better solution. 😁

@skychavda you are welcome. Your solution didn't work for me, because of forwardRef using different param, I don't see what you are trying to solve with useState to be honest, I just removed the useless lines.
To whom may be interested, this is how I did with functional components and react-redux:
import React, { forwardRef } from 'react';
import PropTypes from 'prop-types';
import { connect } from 'react-redux';
import {
  NavigationContainer as NavigationContainerDefault,
} from '@react-navigation/native';

const NavigationContainer = forwardRef(({ theme, ...rest }, ref) => (
  <NavigationContainerDefault theme={theme} ref={ref} {...rest} />
));

NavigationContainer.propTypes = {
  /** @ignore */
  theme: PropTypes.object.isRequired,
  /** Settings allow you to use other icons than MaterialCommunityIcons */
  settings: PropTypes.object,
};

NavigationContainer.defaultProps = {
  settings: undefined,
};

const mapStateToProps = (state) => ({
  theme: state.layout.themes[state.preferences.colorScheme],
});

const ConnectedNavigationContainer = connect(mapStateToProps, null, null, { forwardRef: true })(NavigationContainer);

export default ConnectedNavigationContainer;

Notice that with<EMAIL_ADDRESS>you must stack the forward ref in connect and with the functional component.

This solution is worked for me, use both forwardRef and  useImperativeHandle at the same time:
import React, {
  useImperativeHandle,
  forwardRef,
} from 'react';

const MyComponent: React.FC = forwardRef((props, ref) => {
  useImperativeHandle(ref, () => ({
    alertMessge: () => {
      alert('Hello world');
    }
  }));

  return (

MyComponent

  )
});

@komelabbbas your solution works for me thanks

wrap your element with a  tag.
<Link>

      {...}

</Link>

I don't understand what I'm doing but replacing useRef for createRef did the job for me (I'm using the camera module of React Native, I don't care about persistency of data)


bug: tuple type is not interacting with length assertions (.length >= 2 and [] | [number] | [number, number] type)

Affected options:
0.42.0 on Mac OS X,
0.42.0 and master on https://flowtype.org/try/
Code example:
const shape: [number, number] | [number] | []  = [1,2];

if (shape.length === 2)
  shape[1]

Expected behavior:
possible types gets limited to the ones that infers with .length === 2 rule, and act accordingly
Actual behavior:
4:   shape[1]     ^ access of computed property/element. Out of bound access. This tuple has 0 elements and you tried to access index 1 of
4:   shape[1]
     ^ tuple type
4:   shape[1]     ^ access of computed property/element. Out of bound access. This tuple has 1 elements and you tried to access index 1 of
4:   shape[1]
     ^ tuple type

Other notes:
I'm afraid I cannot use number[] as type, as long I'm working on multi-dimensional lib array and actual behavior is more like Matrix<[number, number]> to validate the matrix operations, e.g. vector can only be multiplied with vector, and 3d matrix - only with 3d matrix. Also this is nice approach to limit the amount of max dimensions (5d is max possible dimensions for my case)

I guess this should work, as long as tuples have exact length

Issue still there for why I come, Try Flow.

It works now, you can do length refinements on tuples


Videos / Audios stop playing when switching source or running multiple videos on iOS

Bug
Videos / Audios stop playing when switching source or running multiple videos on iOS. The following is logged into the debug console through XCode.
In Android everything works as intended.
'''
[AVAudioSession.mm:-[AVAudioSession setActive:withOptions:error:]: Deactivating an audio session that has running I/O. All I/O should be stopped or paused prior to deactivating the audio session
'''
Environment info
React Native: 0.59.9
react-native-video: 4.2.0
react-native-video pod version: 4.4.0
Steps To Reproduce
1 - Have multiple <Video paused={true} /> components on a screen (different sources for each).
2 - change paused prop to false in one of them.
3 - while the first still plays, make a second video play as well (by changing paused to false)
Expected Result
The first and second videos should play accordingly
Actual Result
Videos stop playing until components are unmounted or app restarted
Possibly related issues:
#1727

Can you test with this branch?
https://github.com/24i/react-native-video/tree/feature/ios-player-deinit
if it works I will make the PR

Hi, thanks for investing some time into this
It didn't work, unfortunately, same error.
I did try to isolate the case and found out that if the video/audio files are bundled with the app, everything runs fine.
In my app, though, the files are first downloaded to an internal directory (the documents dir). The  component is then provided with the prop source={{file:///path/to/file}}
I will create a reproduction case and update this issue

I managed to track down the issue. It is not a problem with this library. Sorry for the inconvenience.
In the case someone runs into anything similar, here is what happened
My app depends on react-native-sound-recorder to record audios, and sometimes that library, not this one, would hang the whole avaudiosession thing.
The issue was resolved in this pull request (not yet on npm)
Lesson here is: Always try to create a simple reproduction case of a bug before bothering the library maintainers.


feat: introduce iceberg-rust catalog

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
iceberg-rust now is the iceberg rust SDK maintained by upstream. It's under more active development now and we will graduate migrate to it from icelake gradually.
For now, iceberg-rust has powerful support for reading ability. The #17277 is aimed at introducing iceberg-rust and using it in our batch iceberg scan operator.
This PR is the first part of #17277. This PR:

introduce the iceberg-rust crate
introduce the catalog for iceberg-rust. Due to our aim at first stage is to read, so the catalog implementation mainly focuses on loading table interface and lefting other interfaces todo. We can implement them in the future.

rest catalog (upstream full support)
jni catalog (only implement load_table)
storage catalog (only implement load_table)
the upstream is not plan to support storage catalog so we need to implement it in rw.

Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added test labels as necessary. See details.
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[ ] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

@BugenZhao @xxchan Any concerns on the Cargo.toml update?

https://github.com/risingwavelabs/risingwave/issues/17548


Play and pause animation of the morphology viewers

@ddelpiano is there any specific requirement about when animation should play/pause?
Anything that the user can trigger? Or an automatic mechanism that plays/pause only when viewer is selected or mouse inside of the viewer frame?


ADD Binance exchange overwriting CRO exchange

In this PR

Binance was added but all the CRO API was disabled
CRO pair checks removed because they don't work in the Binance library
The buy_market method for the ETH_BTC in CRO is weird/inverted. For Binance the right amount is used

TODO: Abstract exchange values and method calls into configurable functions and make the script support both exchanges

@sebaslogen I've abstracted calls to clients to a class that handles everything. I've tested Crypto.com and works fine. For Binance, I always get an error when trying any private method with the following error:
binance.exceptions.BinanceAPIException: APIError(code=-1021): Timestamp for this request is outside of the recvWindow
I checked my computer's unix time and Binance's server unix time and indeed there seems to be an issue since my computer seems to be 1 hour ahead (GMT+1)... Have you tried the binance API for private methods? Does it work for you?

Awesome job Tapi!
I didn't try to buy/sell before so I just tried and I only faced this error:
Exception occurred -> 'APIError(code=-1013): Filter failure: LOT_SIZE'.
The problem is that max_decimals_buy should be the same as max_decimals_sell in Binance (and any other normal pair/exchange because you buy and sell in the same denomination for a pair, the decimals of the core_number_currency, in our case ETH, are 3). Once I changed it to 3 it worked fine and made a sell of BTC (a ETH buy).
As for your GTM problem, I'm not sure, my computer is also on GTM+1 and it worked, did you check that your API key supports trading?

@sebaslogen I've abstracted calls to clients to a class that handles everything. I've tested Crypto.com and works fine. For Binance, I always get an error when trying any private method with the following error:
binance.exceptions.BinanceAPIException: APIError(code=-1021): Timestamp for this request is outside of the recvWindow
I checked my computer's unix time and Binance's server unix time and indeed there seems to be an issue since my computer seems to be 1 hour ahead (GMT+1)... Have you tried the binance API for private methods? Does it work for you?

Awesome job Tapi!
I didn't try to buy/sell before so I just tried and I only faced this error:
Exception occurred -> 'APIError(code=-1013): Filter failure: LOT_SIZE'.
The problem is that max_decimals_buy should be the same as max_decimals_sell in Binance (and any other normal pair/exchange because you buy and sell in the same denomination for a pair, the decimals of the core_number_currency, in our case ETH, are 3). Once I changed it to 3 it worked fine and made a sell of BTC (a ETH buy).
As for your GTM problem, I'm not sure, my computer is also on GTM+1 and it worked, did you check that your API key supports trading?

Hmmm, it looks like my API keys are fine. In the Binance website API settings I have the options for spot trading enabled... though I don't have enabled the withdraw option and also I don't have IP's whitelist enabled (so any IP can be used). Maybe that is the reason? What settings do you have?

@sebaslogen I've abstracted calls to clients to a class that handles everything. I've tested Crypto.com and works fine. For Binance, I always get an error when trying any private method with the following error:
binance.exceptions.BinanceAPIException: APIError(code=-1021): Timestamp for this request is outside of the recvWindow
I checked my computer's unix time and Binance's server unix time and indeed there seems to be an issue since my computer seems to be 1 hour ahead (GMT+1)... Have you tried the binance API for private methods? Does it work for you?

Awesome job Tapi!
I didn't try to buy/sell before so I just tried and I only faced this error:
Exception occurred -> 'APIError(code=-1013): Filter failure: LOT_SIZE'.
The problem is that max_decimals_buy should be the same as max_decimals_sell in Binance (and any other normal pair/exchange because you buy and sell in the same denomination for a pair, the decimals of the core_number_currency, in our case ETH, are 3). Once I changed it to 3 it worked fine and made a sell of BTC (a ETH buy).
As for your GTM problem, I'm not sure, my computer is also on GTM+1 and it worked, did you check that your API key supports trading?

Hmmm, it looks like my API keys are fine. In the Binance website API settings I have the options for spot trading enabled... though I don't have enabled the withdraw option and also I don't have IP's whitelist enabled (so any IP can be used). Maybe that is the reason? What settings do you have?

I'm also getting the same problem today 😥

@sebaslogen I've abstracted calls to clients to a class that handles everything. I've tested Crypto.com and works fine. For Binance, I always get an error when trying any private method with the following error:
binance.exceptions.BinanceAPIException: APIError(code=-1021): Timestamp for this request is outside of the recvWindow
I checked my computer's unix time and Binance's server unix time and indeed there seems to be an issue since my computer seems to be 1 hour ahead (GMT+1)... Have you tried the binance API for private methods? Does it work for you?

Awesome job Tapi!
I didn't try to buy/sell before so I just tried and I only faced this error:
Exception occurred -> 'APIError(code=-1013): Filter failure: LOT_SIZE'.
The problem is that max_decimals_buy should be the same as max_decimals_sell in Binance (and any other normal pair/exchange because you buy and sell in the same denomination for a pair, the decimals of the core_number_currency, in our case ETH, are 3). Once I changed it to 3 it worked fine and made a sell of BTC (a ETH buy).
As for your GTM problem, I'm not sure, my computer is also on GTM+1 and it worked, did you check that your API key supports trading?

Hmmm, it looks like my API keys are fine. In the Binance website API settings I have the options for spot trading enabled... though I don't have enabled the withdraw option and also I don't have IP's whitelist enabled (so any IP can be used). Maybe that is the reason? What settings do you have?

Ok, I've found this problem is very common and 2 possible solutions, both involve adjusting the time difference in the client, not the OS:

Wrapping the lib we currently use with this class to adjust the time diff https://github.com/sammchardy/python-binance/issues/249#issuecomment-392771703
Switching the python lib for Binance by this one https://github.com/ccxt/ccxt which supports a setting to automatically adjust the time diff (https://github.com/ccxt/ccxt/issues/850#issuecomment-361112005) and also supports tons of exchanges

The last option is way more work but it's actually very good news for this project because it'll mean corecito could support way more exchanges with minimal changes.

@sebaslogen I've abstracted calls to clients to a class that handles everything. I've tested Crypto.com and works fine. For Binance, I always get an error when trying any private method with the following error:
binance.exceptions.BinanceAPIException: APIError(code=-1021): Timestamp for this request is outside of the recvWindow
I checked my computer's unix time and Binance's server unix time and indeed there seems to be an issue since my computer seems to be 1 hour ahead (GMT+1)... Have you tried the binance API for private methods? Does it work for you?

Awesome job Tapi!
I didn't try to buy/sell before so I just tried and I only faced this error:
Exception occurred -> 'APIError(code=-1013): Filter failure: LOT_SIZE'.
The problem is that max_decimals_buy should be the same as max_decimals_sell in Binance (and any other normal pair/exchange because you buy and sell in the same denomination for a pair, the decimals of the core_number_currency, in our case ETH, are 3). Once I changed it to 3 it worked fine and made a sell of BTC (a ETH buy).
As for your GTM problem, I'm not sure, my computer is also on GTM+1 and it worked, did you check that your API key supports trading?

Hmmm, it looks like my API keys are fine. In the Binance website API settings I have the options for spot trading enabled... though I don't have enabled the withdraw option and also I don't have IP's whitelist enabled (so any IP can be used). Maybe that is the reason? What settings do you have?

Ok, I've found this problem is very common and 2 possible solutions, both involve adjusting the time difference in the client, not the OS:

Wrapping the lib we currently use with this class to adjust the time diff sammchardy/python-binance#249 (comment)
Switching the python lib for Binance by this one https://github.com/ccxt/ccxt which supports a setting to automatically adjust the time diff (ccxt/ccxt#850 (comment)) and also supports tons of exchanges

The last option is way more work but it's actually very good news for this project because it'll mean corecito could support way more exchanges with minimal changes.

I have tried the first solution but it does not seem to work on my end either...
About the second option -> Yes, I also saw the ccxt lib. Also for the arbitrage project I am working on seems to be a better solution... I guess we could go ahead and try the ccxt solution, since it can be interesting for the future of Corecito 😄. I will try the lib separately to find out if Binance actually works for me (let me know if you give it a try also).
So, if ccxt seems to work fine, I would switch to that lib ( I believe it does not have Crypto.com library, but we can use what we already have).

@sebaslogen I've abstracted calls to clients to a class that handles everything. I've tested Crypto.com and works fine. For Binance, I always get an error when trying any private method with the following error:
binance.exceptions.BinanceAPIException: APIError(code=-1021): Timestamp for this request is outside of the recvWindow
I checked my computer's unix time and Binance's server unix time and indeed there seems to be an issue since my computer seems to be 1 hour ahead (GMT+1)... Have you tried the binance API for private methods? Does it work for you?

Awesome job Tapi!
I didn't try to buy/sell before so I just tried and I only faced this error:
Exception occurred -> 'APIError(code=-1013): Filter failure: LOT_SIZE'.
The problem is that max_decimals_buy should be the same as max_decimals_sell in Binance (and any other normal pair/exchange because you buy and sell in the same denomination for a pair, the decimals of the core_number_currency, in our case ETH, are 3). Once I changed it to 3 it worked fine and made a sell of BTC (a ETH buy).
As for your GTM problem, I'm not sure, my computer is also on GTM+1 and it worked, did you check that your API key supports trading?

Hmmm, it looks like my API keys are fine. In the Binance website API settings I have the options for spot trading enabled... though I don't have enabled the withdraw option and also I don't have IP's whitelist enabled (so any IP can be used). Maybe that is the reason? What settings do you have?

Ok, I've found this problem is very common and 2 possible solutions, both involve adjusting the time difference in the client, not the OS:

Wrapping the lib we currently use with this class to adjust the time diff sammchardy/python-binance#249 (comment)
Switching the python lib for Binance by this one https://github.com/ccxt/ccxt which supports a setting to automatically adjust the time diff (ccxt/ccxt#850 (comment)) and also supports tons of exchanges

The last option is way more work but it's actually very good news for this project because it'll mean corecito could support way more exchanges with minimal changes.

I have tried the first solution but it does not seem to work on my end either...
About the second option -> Yes, I also saw the ccxt lib. Also for the arbitrage project I am working on seems to be a better solution... I guess we could go ahead and try the ccxt solution, since it can be interesting for the future of Corecito 😄. I will try the lib separately to find out if Binance actually works for me (let me know if you give it a try also).
So, if ccxt seems to work fine, I would switch to that lib ( I believe it does not have Crypto.com library, but we can use what we already have).

I'm not touching code at least until net year 😉 so go ahead

@sebaslogen I've abstracted calls to clients to a class that handles everything. I've tested Crypto.com and works fine. For Binance, I always get an error when trying any private method with the following error:
binance.exceptions.BinanceAPIException: APIError(code=-1021): Timestamp for this request is outside of the recvWindow
I checked my computer's unix time and Binance's server unix time and indeed there seems to be an issue since my computer seems to be 1 hour ahead (GMT+1)... Have you tried the binance API for private methods? Does it work for you?

Awesome job Tapi!
I didn't try to buy/sell before so I just tried and I only faced this error:
Exception occurred -> 'APIError(code=-1013): Filter failure: LOT_SIZE'.
The problem is that max_decimals_buy should be the same as max_decimals_sell in Binance (and any other normal pair/exchange because you buy and sell in the same denomination for a pair, the decimals of the core_number_currency, in our case ETH, are 3). Once I changed it to 3 it worked fine and made a sell of BTC (a ETH buy).
As for your GTM problem, I'm not sure, my computer is also on GTM+1 and it worked, did you check that your API key supports trading?

Hmmm, it looks like my API keys are fine. In the Binance website API settings I have the options for spot trading enabled... though I don't have enabled the withdraw option and also I don't have IP's whitelist enabled (so any IP can be used). Maybe that is the reason? What settings do you have?

Ok, I've found this problem is very common and 2 possible solutions, both involve adjusting the time difference in the client, not the OS:

Wrapping the lib we currently use with this class to adjust the time diff sammchardy/python-binance#249 (comment)
Switching the python lib for Binance by this one https://github.com/ccxt/ccxt which supports a setting to automatically adjust the time diff (ccxt/ccxt#850 (comment)) and also supports tons of exchanges

The last option is way more work but it's actually very good news for this project because it'll mean corecito could support way more exchanges with minimal changes.

I have tried the first solution but it does not seem to work on my end either...
About the second option -> Yes, I also saw the ccxt lib. Also for the arbitrage project I am working on seems to be a better solution... I guess we could go ahead and try the ccxt solution, since it can be interesting for the future of Corecito 😄. I will try the lib separately to find out if Binance actually works for me (let me know if you give it a try also).
So, if ccxt seems to work fine, I would switch to that lib ( I believe it does not have Crypto.com library, but we can use what we already have).

I'm not touching code at least until next year 😉 so go ahead

Hahaha, ok. You know what, I've made it work with the first solution... or I think I believe so. In that case, I will prepare a solution using the first solution now that it is almost done. If it works fine after testing, I suggest we can merge this PR and then calmly we can work on a next version with ccxt, ok?

@sebaslogen Ok, it seems to be fine now with my latest fixes. Give it a look if you have a sec (and maybe some more testing in Binance on your end). If there are no other issues, we could merge this one and next year (😄 ) work on the cctx version.

@sebaslogen Ok, it seems to be fine now with my latest fixes. Give it a look if you have a sec (and maybe some more testing in Binance on your end). If there are no other issues, we could merge this one and next year (😄 ) work on the ccxt version.

I just quickly tried with the latest code and got the same error 😭
BinanceAPIException: APIError(code=-1021): Timestamp for this request was 1000ms ahead of the server's time.

@sebaslogen Ok, it seems to be fine now with my latest fixes. Give it a look if you have a sec (and maybe some more testing in Binance on your end). If there are no other issues, we could merge this one and next year (😄 ) work on the ccxt version.

I just quickly tried with the latest code and got the same error 😭
BinanceAPIException: APIError(code=-1021): Timestamp for this request was 1000ms ahead of the server's time.

That error is slightly different, I mean, the error code is the same but the issue itself is different. I believe that in your case now, the issue can be solved if you manually set the time of your computer a few seconds back in time. I had to do that for the crypto.com api as I was having the same problem and now both apis are working fine for me.

@sebaslogen Ok, it seems to be fine now with my latest fixes. Give it a look if you have a sec (and maybe some more testing in Binance on your end). If there are no other issues, we could merge this one and next year (😄 ) work on the ccxt version.

I just quickly tried with the latest code and got the same error 😭
BinanceAPIException: APIError(code=-1021): Timestamp for this request was 1000ms ahead of the server's time.

That error is slightly different, I mean, the error code is the same but the issue itself is different. I believe that in your case now, the issue can be solved if you manually set the time of your computer a few seconds back in time. I had to do that for the crypto.com api as I was having the same problem and now both apis are working fine for me.

Yes, you're right, it's working like a charm now 😃
I'll finish the review and we can merge this 🚀

@sebaslogen Ok, it seems to be fine now with my latest fixes. Give it a look if you have a sec (and maybe some more testing in Binance on your end). If there are no other issues, we could merge this one and next year (😄 ) work on the ccxt version.

I just quickly tried with the latest code and got the same error 😭
BinanceAPIException: APIError(code=-1021): Timestamp for this request was 1000ms ahead of the server's time.

That error is slightly different, I mean, the error code is the same but the issue itself is different. I believe that in your case now, the issue can be solved if you manually set the time of your computer a few seconds back in time. I had to do that for the crypto.com api as I was having the same problem and now both apis are working fine for me.

Yes, you're right, it's working like a charm now 😃
I'll finish the review and we can merge this 🚀


cimgui.dll is linked against a debug redistributable

Dependency Walker suggests that CIMGUI.DLL redistributed with bindbc-imgui is linked against debug VC++ runtime. This causes a lot of issues, as expected. I think it should be built and linked against a "release" VC++ runtime

Yeah this is currently a manual process, so it's very easy for one of us (almost certainly always me) to slip when we're debugging Inochi and committing a debug build, we really need all the binaries to be generated and committed as a GitHub Action rather than from our dev machines. Ideally it'd actually be uploaded as a Release, but I don't know anything about how the D package manager works and if it can grab stuff from there. @LunaTheFoxgirl set that stuff up.

Please take a look to make sure the newest release looks right. I did check myself using Dependencies (which I would recommend over Dependency Walker these days), but it's possible there's still an issue there.

Please take a look to make sure the newest release looks right. I did check myself using Dependencies (which I would recommend over Dependency Walker these days), but it's possible there's still an issue there.

Dub hadn't quite yet registered the package update, though cloning the repo and using a local dependency works as expected now. I had some issues with freetype.dll not being copied over, but I'll have to see that once dub recognizes there's a new version. Thank you for a quick fix!

Of course, please let us know if you see an issue with freetype. I can't promise I can figure it out but I can try :) I'm also going to try to figure out the CI thing, but I might have to wait until this weekend if I can't figure out how to make commits from the CI.
And like I said, sorry for the trouble here. I'd say we still kinda have a lot in flux, but ideally we don't make errors like this going forward at least.

I have tested the 0.6.1 release on a fresh dub project and everything works perfectly, Thank you again :)


not letting me sign in

when i try to sign in it goes to open the google popup but instantly closes it - https://twitter-clone-jade-zeta.vercel.app/

Fix: add your domain to authorized domains on Firebase Authentication settings


Module not found - '../config/examplesApi.json'

File is currently called examplesAPI.json.
Easiest fix would be renaming the file

Do you have a question or issue? If required could you please elaborate and give more context?
Thanks

Downloaded the code and ran npm install
Running npm start throws an error Error: Cannot find module '../config/examplesApi.json'

Current file is called examplesAPI.json
Renaming the file to examplesApi.json fixed it for me

Sounds like a bug.
I raise it to our Engineering team. Meanwhile you can continue using the workaround you've found.
Thank you for your message

Fixed in https://github.com/docusign/code-examples-node/pull/44


will it work for windows ?

will it work for windows ?

Hi, blocking implementation should work on Windows, but I think nobody has tested it. Please try and let us know (preferably by note in README or in this conversation).
Non-blocking implementation is Unix-like specific: https://github.com/jakac/subwabbit/blob/master/subwabbit/nonblocking.py#L25

@Sandy4321 did you try on Windows? What was the result?


Limit visible records to published / authoritative records

This will build on the Public website access
Will also need to be able to see their own records

No longer required for LG release


Forward all labels to the snapshotter

Ideally we would have separate snapshotter labels such as with the --snapshotter-label option to ctr container create. The containerd daemon will automatically filter out labels that don't match a snapshotter and I really don't want to add a new field to the server API right now. /shurg

After this change the command
docker run --rm -it mcr.microsoft.com/windows/servercore:ltsc2022 powershell -Command "(gdr C).Free + (gdr C).Used"
Prints the default 20G:<PHONE_NUMBER>6
Adding the windows snapshotter label with the command
docker run --label "containerd.io/snapshot/windows/rootfs.sizebytes=32212254720" --rm -it mcr.microsoft.com/windows/servercore:ltsc2022 powershell -Command "(gdr C).Free + (gdr C).Used"
Prints:<PHONE_NUMBER>0


Is there ways to render image on 3dtiles like texture on runtime

Is there ways to rander image on 3dtiles like texture on runtime

Yes! See the textured material option in this code example:
https://sandcastle.cesium.com/index.html?src=Classification Types.html
For general questions like this please use the Cesium forum.


Exporting an image form working when it's not valid

Context
Export an image of a map.
Steps to Reproduce

Set basemaps to Google (it has a lower limit)
Set the size of the image to something bigger than limits (1000x1000 for a Google basemap, for instance).
The form warns the user that the limit has been surpassed.

Current Result
The button to export the image is enabled, allowing the user to export the map.
Expected result
The button is disabled, disallowing the user to export the map.
Browser and version
Seen in Chrome 57.

This is important, and in my opinion, we should fix it before release the feature.
cc @noguerol @saleiva

Absolutely.
BTW is that 640 x 640 limitation something fixed? I'm afraid that will not work for some use cases

640x640 is a Google Maps limitation, it is ok.

but... our clients are paid Google clients, so we can go to bigger sizes, right?

It depends if they are premium plans, if so, yes, it works of course. It is defined by a paramater. All info is here, but @javierarce already did a nice analysis and implemented with that https://developers.google.com/maps/documentation/static-maps/intro.

ok!
El lun., 10 abr. 2017 a las 15:29, Javier Álvarez Medina (<
<EMAIL_ADDRESS>escribió:

It depends if they are premium plans, if so, yes, it works of course. It
is defined by a paramater. All info is here, but @javierarce
https://github.com/javierarce already did a nice analysis
https://github.com/CartoDB/cartodb/issues/11775#issuecomment-288425323
and implemented with that
https://developers.google.com/maps/documentation/static-maps/intro.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/CartoDB/cartodb/issues/11949#issuecomment-292949410,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAIENTa86Cv-obiqBXpebDUOxbmcOIHuks5rui7JgaJpZM4M4lPS
.

Closing since it is in production.


Wrong words flip showed in web app

Hi, I have submitted flips but when I check on web app, it's showed a wrong selected words
on desktop app:

on web app:

i have bug like this but vice versa
i have created flips in idena-web (with my own shared node) and also i run idena-desktop (connected to same node) and see created flips without images, ok
but one flip are with WRONG keywords:
invalid in idena-desktop: Penguin / Crazy hypnotist (i skip this when created flip) but right idena-web: Skipping rope / Gloves
restarting app does not change anything

i think it's this flip https://scan.idena.io/flip/bafkreigm2h2fe4oehdwbhpnxfwbm2sgu4kxuhsyrmclg35ejsxbncupo74

Please, open the issue again if the problem still exists.


Issue uploading notebook

@malev
sean$ binstar notebook upload Test\ Data.ipynb -u sean
Using binstar api site http://localhost:4990/api
Test Data.ipynb has been uploaded.
[TypeError] 'bool' object has no attribute '__getitem__'
Traceback (most recent call last):
  File "/Users/sean/anaconda/bin/binstar", line 9, in <module>
    load_entry_point('binstar==0.10.2.post5', 'console_scripts', 'binstar')()
  File "/Users/sean/Documents/workspace/binstar/binstar_client/scripts/cli.py", line 94, in main
    description=__doc__, version=version)
  File "/Users/sean/Documents/workspace/binstar/binstar_client/scripts/cli.py", line 76, in binstar_main
    return args.main(args)
  File "/Users/sean/Documents/workspace/binstar/binstar_client/commands/notebook.py", line 111, in upload
    log.info("You can visit your notebook at {}".format(notebook_url(upload_info)))
  File "/Users/sean/Documents/workspace/binstar/binstar_client/utils/notebook/__init__.py", line 33, in notebook_url
    parsed = urlparse(upload_info['url'])
TypeError: 'bool' object has no attribute '__getitem__'

seems to be working for me. closing


Made navigation bar not translucent

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


Cannot deploy to oxygen

Hello I can build the site locally but when I try to build the example with github connected to my Hydrogen app on shopify i have this error.
Do you know what could be the issue ??
Run shopify/oxygenctl-action@v4
Run TOKEN="***"
Token is in V2 format
Run echo "Deploying to Oxygen..."
Deploying to Oxygen...
npm WARN exec The following package was not found and will be installed<EMAIL_ADDRESS>✔ Build initiated successfully with id gid://oxygen-hub/Build/150005.

> build
> turbo run build

• Packages in scope: @demo-ecommerce/sanity, storefront, studio
• Running build in 3 packages
• Remote caching disabled
storefront:build
studio:build

 Tasks:    2 successful, 2 total
Cached:    0 cached, 2 total
  Time:    [32](https://github.com/dindonstudio/albertine/actions/runs/7308966176/job/19916105753#step:6:34).047s

Build failed with: Path not found: dist/worker, cancelling build.
Build with id gid://oxygen-hub/Build/150005 cancelled.
Path not found: dist/worker
╭─ error ──────────────────────────────────────────────────────────────────────╮
│                                                                              │
│  EEXIT: 1                                                                    │
│                                                                              │
╰──────────────────────────────────────────────────────────────────────────────╯

Error: Process completed with exit code 1.

You may need to adjust the Github workflow file to add the correct path:
https://github.com/sanity-io/demo-ecommerce/blob/main/.github/workflows/oxygen-deployment-1000007465.yml#L45


I am developing a very similar skill to this one

Hey @goldyfruit,
a few days ago I stumbled over your skill while browsing on GitHub. I’m currently working on a very similar skill (https://github.com/QW3RTZUI0P/sonos-music-controller-skill) and figured that it might be unneccessary to develop two skills for the same purpose.
I installed your skill but wasn’t able to get it functioning. My skill is currently working (at least for me) and supports Spotify and Apple Music. Here’s my post on the Mycroft forum concerning my skill: https://community.mycroft.ai/t/testing-and-feedback-sonos-music-controller-skill/10313
Maybe some of my workarounds / solutions can help you. My skill doesn’t need authentication with Spotify and handles almost everything through the speakers itself, but I tried to explain this in my README. Perhaps my issue in the SoCo package can help, too (https://github.com/SoCo/SoCo/issues/812).
Also, I’m very new to Mycroft and I’ve absolutely no experience with python and GitHub at all.
Hope I can help,
QW3RTZUI0P

Hey @QW3RTZUI0P
Glad to see someone with Sonos devices and Mycroft AI, thanks for reaching out and share your project! :+1:
This skill works for me but maybe we have different setup and version of Mycroft, I'm using multiple Sonos devices with Boost, etc... and the latest Mycroft Core dev branch. What was your issue?
I'm curious about the fact that you don't need authentication with Spotify, we are working with the SoCo team to bring back the music service (Spotify, Amazon Music, etc...) which requires authentication (https://github.com/SoCo/SoCo/pull/763).
I have a pretty big music library based on vinyl records converted to MP3 which is why I'm having the "Music Library" part in my code.
Feel free to contribute anytime, you have pretty nice stuff in there (mycroft-playback-control, iTunes, Radio).

Hey, thanks for the quick answer,
I was now able to install your skill (previously I just forgot to change my language to English 😖), but I had some errors along the way:

the pairing code given by Mycroft isn’t valid for a long period of time or is it?
the first time I tried the skill it didn’t find the room. I don’t know whether the reason was that I spelled the room with capital letters (I was using mycroft-cli) or that I didn’t say before “discover sonos speakers” because after I had said that it worked (with lower case letters)
the skill didn’t play the track or album I intended
the skill threw an error when I was trying to play a track and an album. If you’d like I can give them to you.

My skill plays songs without any authentication by giving the soco speaker’s play_uri() method an URI. The URIs are service-dependant and I found them using jishis Node HTTP API.
E.g. the URI for Spotify (at least on my system, I don’t know how the sid value changes on other systems) would be x-sonos-spotify:TRACK_ID?sid=9&flags=0&sn=19 where TRACK_ID is the URI returned by the Spotify API (e.g. “spotify:track:0tgVpDi06FyKpA1z0VMD4v”).
I’m accessing the Spotify API using the public token they give you when you visit open.spotify.com. In the source code you can search for accessToken and you’ll find one (with very limited permissions).
iTunes works similarly (URI = x-sonos-http:${id}.mp4?sid=204&flags=8224&sn=4) and the iTunes Search API doesn’t even require an access token.
Is this very workaround-y or could this be a potential solution for the SoCo music service problem?

The way how SoCo works for Spotify is that it's looking to the Sonos Music services you registered from your Sonos App, but right now the only supported branch for this is from the PR I pasted above.
For the discovery, you need to ask the skill to find you devices and then you will know the names.
It's still under heavy development and not yet ready for production (I'm mostly working on it everyday and commit the code).
Thanks for your feedback.


testID is defined in propTypes, but not added as a prop to the native component

testID is defined in propTypes, but not added as a prop to the native component. So, when testing with tools like Detox, it can't find it.

any fix for this found? I believe my issue may be related to this.


Add automation for AWS -> downstream consumers

Create automation to create issues in downstream consumers of pulumi-aws whenever a release occurs:

pulumi-awsx
pulumi-eks
pulumi-aws-apigateway

Avoid the use of Zapier if possible because it requires manual config outside of IaC.

this work needs to be scalable for any component - e.g. kubernetes-ingress-nginx etc


delete a router or registry pod when both are running on the same node

Closing this. The logic to determine when it's safe to delete a pod is not fully derived, and about to change dramatically as we provision extra infra nodes. Implementing daemonsets and/or affinity would reduce the volume of these alerts to something a human could deal with, but that's also tabled pending third infra node.


CIFAR-100 HP

Thanks for this interesting work! I am just wondering if you can provide the command line for training the cifar-100 dataset?

Thank you for your interest.
We normally used the same hyperparameters as CIFAR-100. This means that the following is a good place to start:
python main.py \
       --dataset cifar100 \
       --model cct_7 \
       --conv-size 3 \
       --conv-layers 1 \
       --warmup 10 \
       --batch-size 128 \
       --checkpoint-path /path/to/checkpoint.pth \
       path/to/cifar100

More details on our newest results are in the latest arXiv preprint, as we train these models for longer and with a different weight decay.
If you have any questions, please do not hesitate to ask.

I'm closing the issue now, if you still have questions feel free to open it back up.


Update sotu.md - Add ormolu and brittany code formatters

Basically, the title. Both seem to be reasonably well-maintained at the moment.

@lierdakil: Thank you! 🙂


Add the Weltraumschaf

Please add my personal blog about security, software architecture and clean code.

thanks nice blog!


Webpack 4 migration?

Any plans of migrations to webpack 4?

I'm currently working on a Pull Request at the moment.
All is functioning in development mode, but still working on a patch for the "extract-text-webpack-plugin" which is causing an error in production mode.

@larkin-nz I realised it as well some warnings with extract-text-webpack-plugin. Existing webpack 4 upgrade pull request works with similar warnings but it produces the bundle correctly. What error do you receive?

The issues that I'm having occurring in production mode are causing Webpack not to serve the CSS files to the browser.

This sounds slightly similar issue I was having when I upgraded webpack-dev-middleware to 3.x but had to revert back to 2.x. The issue I was having was on Windows OS and it was for all static files not being served because of some path malformed.
But yours might be a different issue.

Interesting, I'm running a Unix-like architecture and it seems to run fine in development mode.
I'll do some debugging tomorrow and have a look at that.

I'm also looking forward to this working with Vue 4.x

Any updates on this ? Looking forward to it.


New version: StandardNotes.StandardNotes version 3.9.15

ARP entries are correct.
Auto-updated by vedantmgoyal2009/winget-pkgs-automation in workflow run 1786
Microsoft Reviewers: Open in CodeFlow

/azp run


Autopin gov contracts

Closes #702
Simply calls pin after creation if we create it via a gov proposal

@alpe This should be fine. I just don't cover the case of PinCode returning an error, thus the patch cover issue

Changelog entry added. This should also be ready to merge (merge conflicts on CHANGELOG with #727 but we can just keep both lines)


Challenge 14: Unclear Instructions

It is unclear that you are supposed to enter props.username into {}, because that part of code is above "change code below this line".

Ok I've enclosed that line in comments as well.


Fleet unit reports active/running but still pulls image from registry

May 11 10:14:48 ip-10-152-74-171 systemd[1]: Started masterfirefoxos-prod_v330.cmd.2.
May 11 10:14:48 ip-10-152-74-171 sh[3933]: Error: No such image or container: <IP_ADDRESS>:5000/masterfirefoxos-prod:v330
May 11 10:14:48 ip-10-152-74-171 sh[3933]: Unable to find image '<IP_ADDRESS>:5000/masterfirefoxos-prod:v330' locally
May 11 10:14:52 ip-10-152-74-171 sh[3933]: Pulling repository <IP_ADDRESS>:5000/masterfirefoxos-prod

fleetctl list-units reports
masterfirefoxos-prod_v330.cmd.2.service	44836d36.../<IP_ADDRESS>	active		running

After a while it will change to activating/auto-restart.
Full fleetctl log https://pastebin.mozilla.org/8833052

Hmm...
Get http://<IP_ADDRESS>:5000/v1/repositories/masterfirefoxos-prod/images: read tcp <IP_ADDRESS>:5000: i/o timeout

I've reported the timeout here: https://github.com/deis/deis/issues/3479

closing as irrelevant for v2.


NameError: global name 'db' is not defined

I've set the library path but when I try to search I'm getting Error NameError: global name 'db' is not defined.
Here is what I get from the debug model.
Starting debug for 'Quiver Search'
[2017-09-22 12:38:36][ERROR: input.scriptfilter] Code 1: 12:38:36 workflow.py:1692 DEBUG    Data library_location stored in cpickle format
12:38:36 workflow.py:1707 DEBUG    Stored data loaded from : /Users/cogito/Library/Application Support/Alfred 3/Workflow Data/com.cook.quiver/library_location.cpickle
12:38:36 workflow.py:1803 DEBUG    Loading cached data from : /Users/cogito/Library/Caches/com.runningwithcrayons.Alfred-3/Workflow Data/com.cook.quiver/__workflow_update_status.cpickle
12:38:36 workflow.py:2379 DEBUG    update_data : {u'available': False}
12:38:36 workflow.py:2167 DEBUG    Workflow version : 0.4.0
12:38:36 workflow.py:1568 DEBUG    Reading settings from ~/Library/Application Support/Alfred 3/Workflow Data/com.cook.quiver/settings.json ...
12:38:36 workflow.py:2451 DEBUG    Update check not due
12:38:36 workflow.py:1692 DEBUG    Data library_location stored in cpickle format
12:38:36 workflow.py:1707 DEBUG    Stored data loaded from : ~/Library/Application Support/Alfred 3/Workflow Data/com.cook.quiver/library_location.cpickle
12:38:36 workflow.py:2185 ERROR    global name 'db' is not defined
Traceback (most recent call last):
File "XXX/MEGAsync/Application/Alfred.alfredpreferences/workflows/user.workflow.9A9FF46C-2CBC-499D-A87B-53DCEB90A3CF/workflow/workflow.py", line 2178, in run
func(self)
File "run.py", line 44, in main
db.connect()
NameError: global name 'db' is not defined
12:38:36 workflow.py:2205 DEBUG    Workflow finished in 0.018 seconds.
Please help.

Same thing for me.

Same issue for me.

Hey guys - I'm sorry i've been MIA. I believe the PR above fixes this.

Thanks for fast response! Still no luck.

@keeprock did you clone the repo into your workflows folder?

@danielecook Yes, I did. At first I just edit a line, as PR recommends.
After that, completely replace workflow folder. Check if commits changes was there. And still DB error.
Maybe there is something else is going on, because my Quiver.db file contains only indexes and empty tables.

I've also check if it works at all with newly Quiver DB. It was. But then I connect with my working library - all went out.

Hey keeprock - can you add notes to the quiver db? It might need data...I'm not sure I ever tested it with a fresh database...

Excuse me. Daniel of course. Typo.

Has this been fixed? I used the fixum workflow to fix Python libraries that were incompatible with my version of macOS (10.12.6), and that solved that problem but still getting the global name 'db' name is not defined error.

I can confirm it solved the issue for me. That would be great to package a new release. Thx !

Is the workflow accustomed to the nested folder structure introduced in version 3.1 of Quiver? That's what might be throwing mine off.

Same error:
workflow.py:2266 ERROR    global name 'db' is not defined
using Quiver 3.1.1

I'm also having this issue -- could it be due to the fact that my Quiver Library is stored in my "Dropbox (Personal)" folder, instead of merely Dropbox?

Can it automatically adjust to nested notebooks? I believe that's the main issue.

Same issue here
Starting debug for 'Quiver Search'

[2018-02-15 15:41:33][ERROR: input.scriptfilter] Code 1: 15:41:32 workflow.py:1692 DEBUG    Data `library_location` stored in `cpickle` format
15:41:32 workflow.py:1707 DEBUG    Stored data loaded from : /Users/jklein/Library/Application Support/Alfred 3/Workflow Data/com.cook.quiver/library_location.cpickle
15:41:33 workflow.py:1803 DEBUG    Loading cached data from : /Users/jklein/Library/Caches/com.runningwithcrayons.Alfred-3/Workflow Data/com.cook.quiver/__workflow_update_status.cpickle
15:41:33 workflow.py:2379 DEBUG    update_data : {u'available': False}
15:41:33 workflow.py:2167 DEBUG    Workflow version : 0.4.0
15:41:33 workflow.py:1568 DEBUG    Reading settings from `/Users/jklein/Library/Application Support/Alfred 3/Workflow Data/com.cook.quiver/settings.json` ...
15:41:33 workflow.py:2451 DEBUG    Update check not due
15:41:33 workflow.py:1692 DEBUG    Data `library_location` stored in `cpickle` format
15:41:33 workflow.py:1707 DEBUG    Stored data loaded from : /Users/jklein/Library/Application Support/Alfred 3/Workflow Data/com.cook.quiver/library_location.cpickle
15:41:33 workflow.py:2185 ERROR    global name 'db' is not defined
Traceback (most recent call last):
  File "/Users/jklein/Library/Application Support/Alfred 3/Alfred.alfredpreferences/workflows/user.workflow.7849748C-0D50-444B-AAD0-B8E60BB82B12/workflow/workflow.py", line 2178, in run
    func(self)
  File "run.py", line 44, in main
    db.connect()
NameError: global name 'db' is not defined
15:41:33 workflow.py:2205 DEBUG    Workflow finished in 0.063 seconds.
[2018-02-15 15:41:35][ERROR: input.scriptfilter] Code 1: 15:41:34 workflow.py:1692 DEBUG    Data `library_location` stored in `cpickle` format
15:41:34 workflow.py:1707 DEBUG    Stored data loaded from : /Users/jklein/Library/Application Support/Alfred 3/Workflow Data/com.cook.quiver/library_location.cpickle
15:41:34 workflow.py:1803 DEBUG    Loading cached data from : /Users/jklein/Library/Caches/com.runningwithcrayons.Alfred-3/Workflow Data/com.cook.quiver/__workflow_update_status.cpickle
15:41:34 workflow.py:2379 DEBUG    update_data : {u'available': False}
15:41:35 workflow.py:2167 DEBUG    Workflow version : 0.4.0
15:41:35 workflow.py:1568 DEBUG    Reading settings from `/Users/jklein/Library/Application Support/Alfred 3/Workflow Data/com.cook.quiver/settings.json` ...
15:41:35 workflow.py:2451 DEBUG    Update check not due
15:41:35 workflow.py:1692 DEBUG    Data `library_location` stored in `cpickle` format
15:41:35 workflow.py:1707 DEBUG    Stored data loaded from : /Users/jklein/Library/Application Support/Alfred 3/Workflow Data/com.cook.quiver/library_location.cpickle
15:41:35 workflow.py:2185 ERROR    global name 'db' is not defined
Traceback (most recent call last):
  File "/Users/jklein/Library/Application Support/Alfred 3/Alfred.alfredpreferences/workflows/user.workflow.7849748C-0D50-444B-AAD0-B8E60BB82B12/workflow/workflow.py", line 2178, in run
    func(self)
  File "run.py", line 44, in main
    db.connect()
NameError: global name 'db' is not defined
15:41:35 workflow.py:2205 DEBUG    Workflow finished in 0.090 seconds.

Anyone found a way to use nested notebooks with this workflow? Id really like to get this working

@AlexanderKvamme @austinjreilly I do not have a tremendous amount of time to maintain this but I will add it to my to do list. I've removed a try statement that prevented the db from being imported (or throwing an error when imported)... so that should shed some light on what is going on there.
@AlexanderKvamme Is the main issue browsing nested folders?

Also @austinjreilly the workflow you mention is useful but does not offer the ability to browse tags and folders and in fact this workflow provides the same functionality.

@danielecook I cant say for sure. I was trying to make it work for a while, but ended up succeeding with another workflow (https://github.com/noelcody/quiver-fastsearch-alfred), so I just stuck with that!
Happy to help debug but im new to development so might need some help :)


ValueError: The provided metric AccuracyNovel for keeping the best model is not computed by the evaluation routine.

Can you give me some advice about this Error?
2019-07-31 08:51:00,403 - algorithms.Algorithm - INFO   - Training: miniImageNet_Conv32CosineClassifier
 20%|█▉        | 197/1000 [00:05<00:21, 37.46it/s]2019-07-31 08:51:06,434 - algorithms.Algorithm
100%|██████████| 1000/1000 [00:27<00:00, 35.99it/s]
2019-07-31 08:51:28,193 - algorithms.Algorithm - INFO   - ==> Training stats: {'loss': 2.469}
2019-07-31 08:51:28,200 - algorithms.Algorithm - INFO   - Evaluating: miniImageNet_Conv32CosineClassifier
2019-07-31 08:51:28,200 - algorithms.Algorithm - INFO   - ==> Dataset: MiniImageNet_val [2000 batches]
  0%|          | 0/2000 [00:00<?, ?it/s]/data1/zjj/meta-code/withoutForgetting/algorithms/FewShot.py:185: UserWarning: volatile was removed and now has no effect. Use `with torch.no_grad():` instead.
  images_test_var = Variable(images_test, volatile=is_volatile)
/data1/withoutForgetting/algorithms/FewShot.py:190: UserWarning: volatile was removed and now has no effect. Use `with torch.no_grad():` instead.
  images_train_var = Variable(images_train, volatile=is_volatile)
100%|██████████| 2000/2000 [01:14<00:00, 26.36it/s]
2019-07-31 08:52:43,083 - algorithms.Algorithm - INFO   - ==> Results: {'loss': 3.1131, 'AccuracyNovel_cnf': 0.4092}
2019-07-31 08:52:43,084 - algorithms.Algorithm - INFO   - ==> Evaluation stats: {'loss': 3.1131, 'AccuracyNovel_cnf': 0.4092}
Traceback (most recent call last):
  File "train.py", line 110, in <module>
    algorithm.solve(dloader_train, dloader_test)
  File "/data1/withoutForgetting/algorithms/Algorithm.py", line 288, in solve
    self.keep_record_of_best_model(eval_stats, self.curr_epoch)
  File "/data1/withoutForgetting/algorithms/Algorithm.py", line 359, in keep_record_of_best_model
    .format(metric_name))
ValueError: The provided metric AccuracyNovel for keeping the best model is not computed by the evaluation routine.

The error occurred in：
    def keep_record_of_best_model(self, eval_stats, current_epoch):
        if self.keep_best_model_metric_name is not None:
            metric_name = self.keep_best_model_metric_name
            if (metric_name not in eval_stats):
                raise ValueError('The provided metric {0} for keeping the best '
                                 'model is not computed by the evaluation routine.'
                                 .format(metric_name))
            metric_val = eval_stats[metric_name]

Thanks a lot！

change back to pytorch version==0.4

I think it has nothing to do with pytorch version.
You can modify line 36 of FewShot.py
 self.keep_best_model_metric_name = 'AccuracyNovel_cnf'
# self.keep_best_model_metric_name = 'AccuracyNovel'

This issue is caused by different versions of pytorch. Higher versions of Pytorch will return a tensor and lower version will return the type of float for this line:
https://github.com/gidariss/FewShotWithoutForgetting/blob/master/algorithms/FewShot.py#L20
Then in the update method(https://github.com/gidariss/FewShotWithoutForgetting/blob/master/utils.py#L154), AccuracyNovel will be ignored.

nice!!!

This issue is caused by different versions of pytorch. Higher versions of Pytorch will return a tensor and lower version will return the type of float for this line: https://github.com/gidariss/FewShotWithoutForgetting/blob/master/algorithms/FewShot.py#L20 Then in the update method(https://github.com/gidariss/FewShotWithoutForgetting/blob/master/utils.py#L154), AccuracyNovel will be ignored.

Hi, bro, i use the same pytorch version like u, but i still get the error ,could you help me ?


Does not include use information

New developers will have no clue where to find the nuget package to install gRPC into their console app much less any web api or mvc app.  The documenters did not spend any time considering other people and are heartless.

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

ID: 0db2d29c-ebfd-9c79-4b08-61fc5192f3c3
Version Independent ID: 048d1977-93aa-f28a-60ec-6806dec3f70a
Content: Introduction to gRPC on .NET
Content Source: aspnetcore/grpc/index.md
Product: aspnet-core
Technology: aspnetcore-grpc
GitHub Login: @JunTaoLuo
Microsoft Alias: johluo

This is an intro to gRPC, not a tutorial. It links to https://docs.microsoft.com/en-us/aspnet/core/tutorials/grpc/grpc-start


Create ARC-ISA-example

@xiaoranzhou Could you format it with indentation? Than we can track line changes better


TS/JS general missing features

Type: Bug
I don't know if you are adding missing features manually, but you better make a system that automates it.
Even though I opened this topic under the TS/JS title, I think this topic is also valid for other languages.
// @ts-expect-error - Property 'getCanonicalLocales' does not exist on type 'typeof Intl'.(2339)
console.log(Intl.getCanonicalLocales('EN-US')); // ["en-US"]

type INTL = keyof typeof Intl;
/*
| "Collator"
| "NumberFormat"
| "DateTimeFormat"
| "PluralRules"
| "RelativeTimeFormat"
| "Locale"
| "DisplayNames"
| "ListFormat"
| "Segmenter"
 */

VS Code version: Code 1.70.2 (e4503b30fc78200f846c62cf8091b76ff5547662, 2022-08-16T05:35:13.448Z)
OS version: Windows_NT x64 10.0.22000
Modes:

Duplicate of https://github.com/microsoft/TypeScript/issues/29129


Add CFP-30788 Managing Hubble Metrics cardinality

This proposal aims at adding a new dynamic mechanism to manage Hubble metric cardinality.

Overall idea makes sense to me. We've got a few people on the hubble team who are on holiday this week; so hopefully next week they can take a look and we can move forward.

@kaworu: @chancez While I agree conceptually, disabled: false is difficult to read and would be inconsistent with the Hubble flow exporter.

Fair enough, so then should we just remove the option from the dynamic metrics config entirely? Then to disable a given metric option it just needs to not be configure in the list all? Eg to have the feature enabled, but no metrics configured:
       hubbleMetrics:
            metrics: [] # no metrics enabled currently.

That sounds reasonable, check for the presence of this new configmap and/or the lenght of metrics to determine if the feature is enabled. Removing the enabled/disabled section from CFP.

@kaworu  Thank you for the review,

Did you consider a CRD instead of a ConfigMap to configure metrics? The question was discussed for the Hubble exporter, and I feel like an explanation of why the CFP propose one or the other (or something else) would make sense in the "Implementation Alternatives" section.

Ack, i want building on the previous precedence of config maps, but open to discussion on this. I believe configmaps are better solution compared to CRDs as lined out in @chancez 's comment. Added my views to alternatives section.

It's unclear to me whether the CFP plan to deprecate/remove the cilium's config hubble-metrics. IMO we don't want to maintain the two long-term, and thus we're missing a migration path (or at least suggestion) from the current hubble-metrics mechanism to the proposed hubble-metrics-config mechanism in the "Impacts / Key Questions". Are we going to ship / document hubble-metrics-config for the existing dashboards?

That is a great point, we should deprecate the older path, but i am not familiar about the deprecation policy/process of cilium? If there are any pointers on how this is normally handled, i am happy to add it to "Future Milestones". My assumption is that it will take atleast one or two minor version upgrades for existing behavior get deprecated. I.e. if this support lands in 1.16 cilium, by 1.18 existing configuration via cilium-config will be deprecated?

This is a fair assumption, I think a couple of version should be enough to validate the configMap approach and implementation, and at that point we could start on deprecating the cilium-config metrics configuration.

thank for the review, I added the same at the bottom of the proposal stating, we will re-visit deprecation in 2 versions' time.

Last thing before merging, can you create and put this in a /hubble folder rather than the /cilium folder?

Good idea. @xmulligan @vakalapa once it's moved I'll merge the CFP.

Done, moved CFP to hubble folder. @chancez

Heads up that I moved this under cilium/ to match the documented process: https://github.com/cilium/design-cfps/pull/35 .

Ah @xmulligan I didn't see your request there. If we're going to split PRs up like that, then it'll make sense to update the README.md to describe exactly how we expect developers to make those decisions so we can consistently apply the rule :)

ahh you are right sorry about that. I guess I expected the issue to be in the Hubble repo. Thanks for changing


Support deserialization of ApiErrorResponse

In my application, I have some automated tests that verify the API error response. I tried to deserialize the error response to an instance of ApiErrorResponse with Jackson
ApiErrorResponse errorResponse = objectMapper.readValue(responseBody, ApiErrorResponse.class);

But deserialization fails with this error

com.fasterxml.jackson.databind.exc.InvalidDefinitionException: Cannot construct instance of io.github.wimdeblauwe.errorhandlingspringbootstarter.ApiErrorResponse (no Creators, like default constructor, exist): cannot deserialize from Object value (no delegate- or property-based Creator)
at [Source: (String)"{"code":"VALIDATION_FAILED","message":"Validation failed for object='sourcingOrganisationRequest'. Error count: 1","fieldErrors":[{"code":"REQUIRED_NOT_BLANK","message":"must not be blank","property":"name","rejectedValue":null,"path":"name"}]}"; line: 1, column: 2]

The reason it can't be deserialized "automatically" is because ApiErrorResponse is neither a record nor a Java beans and similarly for the fields of this class like ApiFieldError.
AFAIK, my only option is to implement a custom deserializer, i.e.
import com.fasterxml.jackson.databind.JsonDeserializer;
import io.github.wimdeblauwe.errorhandlingspringbootstarter.ApiErrorResponse;

@JsonComponent
public class new ApiErrorResponseDeserializer extends JsonDeserializer<ApiErrorResponse>() {
    @Override
    public ApiErrorResponse deserialize(JsonParser jsonParser, DeserializationContext context) throws IOException {
        // TODO implement me
    }
}

I checked the project's source code in case a deserializer already exists, but I didn't find one.

My mistake. If you use the ObjectMapper Spring bean instead of constructing one withnew ObjectMapper() it can deserialize the response to an instance of ApiErrorResponse


Remove "sphinx.ext.ifconfig" from docs/source/conf.py

remove "sphinx.ext.ifconfig" from docs/source/conf.py, see #530

As you wish - I've removed the line.


Why smaller batch size results  better results?  With the epochs increased, the G loss went up?

There are 8k+ 256*256 images in my datasets, I set batchsize=32 and trained with 4 12GB GPUs, but it's not as good as I set batchsize = 4 and trained with one GPU,neither the training speed nor the image quality. Why a smaller batch size results  better results? Is there a best batchsize?
Beside,I set batchsize=4, when epoch>70, g-loss went up obviously,and qualities of generated images went worse.Why did this happen? (Training process was interrupted when epoch = 54, I reloaded weight files and optimer states from epoch 53 )
Thanks!

Well, there are some known issues regarding the multi-gpu training since the last couple of updates of PyTorch. I will have to investigate this, but it's quite difficult given my schedule right now. The training instability in this resumed training could be because of improper loading of state/weights.


Spark email address bounces

"The team collaborates on Cisco Spark and GitHub Issues. Send an email to<EMAIL_ADDRESS>and we’ll add you to our Cisco Spark room!"
This email address does not exist.
Final-Recipient<EMAIL_ADDRESS>Action: failed
Status: 5.0.0 (permanent failure)
Remote-MTA: dns; [<IP_ADDRESS>]
Diagnostic-Code: smtp; 5.1.0 - Unknown address error 550-'5.1.1<EMAIL_ADDRESS>User unknown' (delivery attempts: 0)

Hmmm let me check.

@bbaassssiiee check out https://github.com/CiscoCloud/microservices-infrastructure/commit/29a1775593df93359890565eb4633a4f6ff494f0 ... we're on Gitter now!