Warning: Tried to initialize from a previous compilation

Compiler message:
Warning: Tried to initialize from a previous compilation (.../testfile.dill), but the file was not self-contained. This might be a bug.

On: Dart VM version: 2.3.0-dev.0.3.flutter-1f1592edce (Tue Apr 23 00:32:54 2019 +0000) on "linux_x64"
initialize_from.dill.gz

Thank you for the report.
This is caused by the previous compilation being compiled on the basis of an older version of the sdk where the new version (the one now included in flutter) has changed (specifically in https://github.com/flutter/engine/commit/3e47b4bb39bb4993f03a278ea7b1c11ee6459b06 the getter "left" on class "Rect" was removed and became a field instead.)
This means that the old compilation tries to reference something that no longer exists. This is detected, and we're saying that we cannot initialize from it. This is perfectly fine. Ideally we wouldn't even try to initialize from it, because we would know that the sdk versions doesn't match up, but we cannot do that (yet). #36032 tracks that.


Fix pytest --pyargs napari test on pip install.  Add CI test

Description
fixes #2108  by

fixing the pytest error TypeError: expected str, bytes or os.PathLike object, not NotSetType reported by @jni when pytest tries to calculate test ids from an empty list.  (now, it will simply skip the test if the list of examples is empty)
adds a dedicated test to both the PR tests and "full tests" that does not checkout the full repo, but rather directly pip installs the commit from github.

as a side conversation, I still think point #3 in https://github.com/napari/napari/issues/2108#issuecomment-760199377 could be interesting: we could include napari_examples as a separate package in our sdist, allowing something like napari demo add_image after pip install
Type of change

[x] Bug-fix (non-breaking change which fixes an issue)

How has this been tested?

[ ] example: the test suite for my feature covers cases x, y, and z
[ ] example: all tests pass with my change

Final checklist:

[ ] My PR is the minimum possible work for the desired functionality
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works

working.  here is the

working.  here is the

I added a minor comment about the name

thanks! included.

I like shipping examples too, but maybe we merge this PR first and then add that in a follow-on? Up to you @tlambert03

Yeah, I think that could be cool.  definitely a follow up (if @jni likes it)

I added a minor comment about the name

thanks! included.

I like shipping examples too, but maybe we merge this PR first and then add that in a follow-on? Up to you @tlambert03

Yeah, I think that could be cool.  definitely a follow up (if @jni likes it)

ok, this looks great - will merge now!

ok, this looks great - will merge now!

I like shipping examples too, but maybe we merge this PR first and then add that in a follow-on? Up to you @tlambert03

Yeah, I think that could be cool. definitely a follow up (if @jni likes it)

As long as it's in its own 'extras' package, ie pip install napari[demos]... It would be obnoxious otherwise and also we are very lax with requirements with our examples...
Even with that, though, I'm not sure it's the best way to distribute examples... Typically people want to look at code in examples, as well as run it, not just run it. So I feel like a web gallery is a better format. Of course, napari demo [name] could also print the code to the console, or copy it to the clipboard 😂. But we are getting very fancy for some rather niche value...

I like shipping examples too, but maybe we merge this PR first and then add that in a follow-on? Up to you @tlambert03

Yeah, I think that could be cool. definitely a follow up (if @jni likes it)

As long as it's in its own 'extras' package, ie pip install napari[demos]... It would be obnoxious otherwise and also we are very lax with requirements with our examples...
Even with that, though, I'm not sure it's the best way to distribute examples... Typically people want to look at code in examples, as well as run it, not just run it. So I feel like a web gallery is a better format. Of course, napari demo [name] could also print the code to the console, or copy it to the clipboard 😂. But we are getting very fancy for some rather niche value...

As long as it's in its own 'extras' package, ie pip install napari[demos]

I'm not sure it can be an extra (that would be for external deps)... I was thinking more something like an additional package (something to the "find_packages" parameter in setuptools).  Installed as _napari_demos.

So I feel like a web gallery is a better format

100% ... but both would be possible.

But we are getting very fancy for some rather niche value...

yep 😂 let's ditch it

As long as it's in its own 'extras' package, ie pip install napari[demos]

I'm not sure it can be an extra (that would be for external deps)... I was thinking more something like an additional package (something to the "find_packages" parameter in setuptools).  Installed as _napari_demos.

So I feel like a web gallery is a better format

100% ... but both would be possible.

But we are getting very fancy for some rather niche value...

yep 😂 let's ditch it


Custom view DSL examples

Description
Hello
I was trying to understand how Custom Views work and their capabilities, but I can't find any examples.
There are two documentation pages:

This page has an example of a rendered diagram, but not the source code.
DSL Reference is also very terse and does not have examples.

Would it be possible to add this to the documentation?
Priority
I have no budget and there's no rush, please add this feature for free
More information
No response

Examples have been added to the DSL cookbook:

Custom elements
Custom views


Error messages seem to point to the wrong place

Error in program python_tea.py, module _weakref, line 5:
	Attempt to load module: _weakref
	Can't find any of:
		/Users/boxed/Projects/pythonista-wip/__javascript__/_weakref.mod.js
[etc]

My file python_tea.py doesn't have an import of _weakref, so the error message pointing me to "line 5" isn't helpful. It seems like I import a lib, that imports a lib, that imports a lib that imports _weakref. The error message should give me such a traceback I think, or at least point to the place the import is done, not my code.

That's not as it should be indeed.
We'll look into that.
Thanks for the feedback!

I've committed fix for this.
Could you confirm that it works for you?
Kind regards
Jacques

It's a bit weird the new error in my case at least:
Error in program python_tea.py, module pyrsistent.pyrsistent._pmap.collections._collections_abc.abc._weakrefset, line 5:

but it's understandable and workable at least.

Actually, after looking at this a bit more I don't think the error is good enough. A stack trace style error would be vastly superior. The problem now is that you get the error message only on the last file.

With regard to the error message in your example:

Error in program python_tea.py, module pyrsistent.pyrsistent._pmap.collections._collections_abc.abc._weakrefset, line 5:

what exactly do you mean by:

you get the error message only on the last file

It tells you that the error is in line 5 of module _weakrefset,
which can be reached from your main module by the qualified name:
pyrsistent.pyrsistent._pmap.collections._collections_abc.abc._weakrefset
Seems to me that exactly pinpoints the error.
But probably am I missing something important.
What information exactly to your opinion is missing here that you need to locate (and correct) the problem?
Could you perhaps give an example of what in your view the output should look like in this case.

Well it's sort of like an ImportError so you'd want the lines of the imports leading up to it. So something like:
foo.py:5
pyrsistent.py:8
_pmap.py:2
Etc
Hopefully with full paths to the files. Another problem I have is that the code I'm trying to use does try/importerror but the transcrypt compiler goes through it all of course. I have to create stub files with "raise ImportError" in them :P

On 1 Feb 2017, at 10:57, Jacques de Hooge<EMAIL_ADDRESS>wrote:
With regard to the error message in your example:
Error in program python_tea.py, module pyrsistent.pyrsistent._pmap.collections._collections_abc.abc._weakrefset, line 5:
what exactly do you mean by:
you get the error message only on the last file
It tells you that the error is in line 5 of module _weakrefset,
which can be reached from your main module by the qualified name:
pyrsistent.pyrsistent._pmap.collections._collections_abc.abc._weakrefset
Seems to me that exactly pinpoints the error.
But probably am I missing something important.
What information exactly to your opinion is missing here that you need to locate (and correct) the problem?
Could you perhaps give an example of what in your view the output should look like in this case.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

The stacktrace-style error reporting has been added and committed.
About the 'raise ImortError': Indeed import is resolved compile time, so unfortunately exceptions in it cannot be caught at runtime.


little typo in code

Inside Window_Loaded function, AxWMPLib should be renamed to WmpAxLib

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

ID: 37c83f79-91e4-e89f-5791-ad0ec41fc622
Version Independent ID: e87aa8be-b171-df3c-5eea-d54645116c95
Content: Walkthrough: Hosting an ActiveX Control in WPF
Content Source: docs/framework/wpf/advanced/walkthrough-hosting-an-activex-control-in-wpf.md
Product: dotnet-framework
GitHub Login: @dotnet-bot
Microsoft Alias: dotnetcontent

@convexbytes, thanks for letting us know. Would you like to make the change and submit a PR for this?

Thanks for the quick response. I have created a PR.


Tools and Applications

Please list any tools or applications developed to use Phenoscape, ontologies, or machine learning that you have developed or would like to learn.
Please share links and documentation for these tools or applications.

I have been using Phenex to annotate a morphological matrix with ontology terms from the Coleoptera Anatomy Ontology (COLAO).


NFD_PickFolder GTK crashing

Version
3.3.0 (nightly)
Platform
Linux x64
JDK
OpenJDK 17.0.2
Module
NFD
Bug description
When I'm using NFD_PickFolder function with Swing Window, it crash with following error. It crashing only when I use GTK+ look and feel for Swing window. Without it works normally.
System: Linux Mint 20.3 x64
Desktop: Cinnamon
(LWJGL version is 3.3.1 but can't select it in issue template)
Stacktrace or crash log output
(java:7263): GLib-GObject-WARNING **: 18:25:03.260: cannot register existing type 'GdkDisplayManager'

(java:7263): GLib-CRITICAL **: 18:25:03.260: g_once_init_leave: assertion 'result != 0' failed

(java:7263): GLib-GObject-CRITICAL **: 18:25:03.260: g_object_new_with_properties: assertion 'G_TYPE_IS_OBJECT (object_type)' failed

Hey @rafi612,
I don't think there's much we can do about this, but could you please post a simple code snippet that reproduces the crash?

import javax.swing.*;

import org.lwjgl.PointerBuffer;
import org.lwjgl.system.MemoryStack;

import static org.lwjgl.util.nfd.NativeFileDialog.*;

public class NFDTest {

	public static void main(String[] args) {

		try {
			UIManager.setLookAndFeel("com.sun.java.swing.plaf.gtk.GTKLookAndFeel");
		} catch (ClassNotFoundException | InstantiationException | IllegalAccessException
				| UnsupportedLookAndFeelException e) {
			e.printStackTrace();
		}

		JFrame frame = new JFrame("NFD Test");
		frame.setSize(500,500);

		JButton test = new JButton("Test");

		test.addActionListener((actionEvent) -> {
			try (MemoryStack stack = MemoryStack.stackPush()) {

				PointerBuffer pointer = stack.mallocPointer(1);
				if (NFD_OpenDialog("Test", "Test", pointer) == NFD_OKAY)
					System.out.println("Selected path is:" + pointer.getStringUTF8());
			}
		});

		frame.add(test);
		frame.setVisible(true);
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

	}
}

Hey @rafi612,
Recent GTK versions require that all GTK functions are called from a single thread. Afaict, AWT interacts with X and libraries like GTK via the AWT-XAWT thread, which is not the same as the event dispatch thread used to call the NFD_OpenDialog function. This breaks GTK's thread rules and probably causes a deadlock.
Why not use Java's file chooser in the first place?

 But now I use tinyfd from LWJGL instead and it's work very well.


Web component usage

Hey @ije. I'm still a fan of your lib. For this reason, I've created a Web component library that consumes yours.
You can check it @ https://github.com/karesztrk/md4wc
I hope it ok, if you are interested we can move it into a monorepo or something.

I love it! Nice job👍
Would you like to create PR that merges your idea into this repo? or you more perfer md4wc?
I'm happy to help.

You mean to restructure this repo to a monorepo? I would like to have a nice chiming name for the whole thing. It can be md4w if you insist.

or we can put the 'md4wc' link in the readme, what do you think?

I will try to create a monorepo as soon as possible

thanks!

Can you please allow me to create a Pull request? I have lack of permission.

you need to fork the repo for creating pr, however i just added you as collaborator

https://github.com/ije/md4w/pull/12


Not working with graphql actual version

When i want to install ( npm install --save merge-graphql-schemas ), I am getting the error:
+-- UNMET PEER DEPENDENCY<EMAIL_ADDRESS>`--<EMAIL_ADDRESS>npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.

Thanks for logging, @schmetzyannick. Can you try installing the package, and let us know if it works?
npm install --save<EMAIL_ADDRESS>

This may be fixed by #103.

@schmetzyannick What version of merge-graphql-schemas do you have installed?

Closing for now. This is only warning you to install graphql.  This package expects you to have installed graphql. PR #103 should resolve the warning by bumping the graphql versions.


Regions need updating

If I get a moment I will try to add support for more regions.
We're missing:

us-east-2
ca-central-1
ap-south-1
ap-northeast-2
eu-west-1
eu-west-2

ref: http://docs.aws.amazon.com/general/latest/gr/rande.html#ec2_region

Actually it looks like this fork has already done the work... https://github.com/chanakyad-cuelogic/awscosts

Sorry for the delay.  Pull requested merged.


pull request for e2e test at 12/18/2016 13:56:18

: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.


Make validator selection API so validation libraries can be wrapped

Enhancement
As a user/developer, when I use this I should be able to ...

use the same schema validator on my nodejs backend (to reuse custom formats and keywords). Unfortunately, due to poor performance (see comparison of benchmark results for json-schema validators) we can't use tv4 on our backend.
use features from the latest (draft-06) json-schema spec

@json-schema-form/angular-schema-form-lead What are the chances of switching to use ajv instead of tv4 to get these enhancements in a future major version?
Compared to tv4, it seems pretty active (see number of Issues, number of PRs, Graphs > Commits on the github page for ajv vs for tv4) and pretty popular (see Stats on the npm page for ajv vs for tv4). It also makes me feel warm and fuzzy that it's used by Webpack.

@mattbrunetti making validator selectable is definitely something I am looking at, it is something we need given the lack of support now for tv4. But I don't want to repeat the mistake of lock-in and select ajv, for example djv appears to be 25% faster, just not as up to date, but if it was supporting the latest draft it would be superior performance wise. So yes the plan is to make it possible to pick a validator, make a wrapper for it and plug it in. I just can promise anything on timelines, although it is definitely something I want myself so is a higher priority than many other things. :)

Even better! This is great news!


Connection to HiveMQ or DIoTY works for only a few minutes

Hi! I have tried to connect from the testing environment ( https://testing.opensensemap.org/ ) to HiveMQ over MQTTS and after a minute or two published messages aren't saved anymore or the client disconnects. I ran the project locally and the connection isn't closed, all the messages sent to the broker are received, even after 15+ minutes running. The same behavior happens when trying to connect over MQTT to https://dioty.co/ from the prod environment ( https://opensensemap.org/ ). Changing the topic or connection options and saving the config makes the connection work for the next 1-2 minutes. The integration with MyQttHub works perfectly in the prod env, the client doesn't disconnect. Is this a testing environment specific issue?

Hi @Wdestroier,
thanks for your feedback. I am going to investigate it.
The changes are not deployed to the prod environment. So I guess it is a testing specific issue.

Sorry for the late reply.
I am back on this issue and tried a MQTTS connection to HiveMQ on the testing environment for several minutes without any connection issues.
dioty.co is not existing anymore right?
@Wdestroier maybe you can share your local project with me.

Hi @mpfeil,
I just tested and the messages aren't processed after 5 minutes...
Steps to reproduce the problem:

Access http://www.hivemq.com/demos/websocket-client/
Fill the authentication form and login
Access openSenseMap and configure the senseBox's MQTT integration
Url<EMAIL_ADDRESS>Topic: /devices/5
Message format: json
Decoding options:  empty/blank/none
Connection options: { "clientId": "openSenseMap-1" }
Set the HiveMQ client's topic and message
Topic: /devices/5
QoS: 0
Message: { "sensorId": "value" }

I've already made the communication with AWS instead of HiveMQ, but this feature may be useful to others...


Parse header with proper hash (tendermint-rpc + proto-ts)

Proto-ts defines Header.time (from tendermint/types) as Date, which has millisecond precision. The rpc returns nanosecond precision, which we can capture in tendermint-rpc (this is for the Header object). But we will loose it when constructing the ts-proto object. This must be identical for the header hash to match the signatures. (At least I think so, I need to ensure they do hash the nanosecond result).
This thought came up when I hit some failing ibc tests with:
"header failed basic validation: commit signs block 9E8DAB852CE462075B06FE258B0B84E66E480F54644E30256BC76EFE0C44E119, header is block 6C3DE71B5E0376E364B7CEBD22D0917F4B0EC391A3A1430CD847DB59EF52E4D1"
I am also a bit confused by some queries:
curl http://localhost:26658/commit?height=3725
{
  "jsonrpc": "2.0",
  "id": -1,
  "result": {
    "signed_header": {
      "header": {
        "version": {
          "block": "11"
        },
        "chain_id": "simd-testing",
        "height": "3725",
        "time": "2021-02-10T22:47:15.968851173Z",
        "last_block_id": {
          "hash": "949532143488917A7D920A92A599CD3C90AD5979AFCE07AE790EE36AEEF0CCCA",
          "parts": {
            "total": 1,
            "hash": "D79BB4BED310C0A67656B8B30D1FEA35FC105D23660098BBF0F4608D5E9AB753"
          }
        },
        "last_commit_hash": "F9E6B97A12EF491B46B039F827A8CA0FB121B1959DEA76357FD29659AD58C91B",
        "data_hash": "E3B0C44298FC1C149AFBF4C8996FB92427AE41E4649B934CA495991B7852B855",
        "validators_hash": "C73C1C6A829C9BF3F574CD00FC1D9EB54FC2E378FAC4D0A2F57878CDF74DFEF7",
        "next_validators_hash": "C73C1C6A829C9BF3F574CD00FC1D9EB54FC2E378FAC4D0A2F57878CDF74DFEF7",
        "consensus_hash": "048091BC7DDC283F77BFBF91D73C44DA58C3DF8A9CBC867405D8B7F3DAADA22F",
        "app_hash": "297B39D7C25115AB92A28D70FBD3608D59167EDD88AE1126A4C0D0EEA14D7DB2",
        "last_results_hash": "E3B0C44298FC1C149AFBF4C8996FB92427AE41E4649B934CA495991B7852B855",
        "evidence_hash": "E3B0C44298FC1C149AFBF4C8996FB92427AE41E4649B934CA495991B7852B855",
        "proposer_address": "E359FA31E749FFDC571AA4E5F629F70CD53D328A"
      },
      "commit": {
        "height": "3725",
        "round": 0,
        "block_id": {
          "hash": "9E8DAB852CE462075B06FE258B0B84E66E480F54644E30256BC76EFE0C44E119",
          "parts": {
            "total": 1,
            "hash": "0C3528C46715A4775C44DF1C76184145EE0A2581794FF08F74B90DBD8FD2EA2E"
          }
        },
        "signatures": [
          {
            "block_id_flag": 2,
            "validator_address": "E359FA31E749FFDC571AA4E5F629F70CD53D328A",
            "timestamp": "2021-02-10T22:47:17.015519194Z",
            "signature": "GEn7xg/fxcVYNVBMVYlzu0ZGMXEUrt8JtjIJic7vdqIBeDF3FiWpgF52/FLhEnGL24AviNptuM15LJxdnq/8CQ=="
          }
        ]
      }
    },
    "canonical": true
  }
}

Well, the header doesn't ever show it's own block_id, just the last one (building the chain). Let's see what the next header claims that 3725 was:
curl http://localhost:26658/commit?height=3726 | jq .result.signed_header.header.last_block_id
{
  "hash": "9E8DAB852CE462075B06FE258B0B84E66E480F54644E30256BC76EFE0C44E119",
  "parts": {
    "total": 1,
    "hash": "0C3528C46715A4775C44DF1C76184145EE0A2581794FF08F74B90DBD8FD2EA2E"
  }
}

Yes, it agrees. And what was the header object we got (parsed):
{
  chainId: 'simd-testing',
  height: Long { low: 3725, high: 0, unsigned: false },
  version: {
    block: Long { low: 11, high: 0, unsigned: false },
    app: Long { low: 0, high: 0, unsigned: true }
  },
  time: 2021-02-10T22:47:15.968Z,
  lastBlockId: {
    hash: Uint8Array(32) [
      148, 149,  50,  20,  52, 136, 145,
      122, 125, 146,  10, 146, 165, 153,
      205,  60, 144, 173,  89, 121, 175,
      206,   7, 174, 121,  14, 227, 106,
      238, 240, 204, 202
    ],
    partSetHeader: undefined
  },
  lastCommitHash: Uint8Array(32) [
    249, 230, 185, 122,  18, 239,  73, 27,
     70, 176,  57, 248,  39, 168, 202, 15,
    177,  33, 177, 149, 157, 234, 118, 53,
    127, 210, 150,  89, 173,  88, 201, 27
  ],
  dataHash: Uint8Array(32) [
    227, 176, 196,  66, 152, 252,  28,
     20, 154, 251, 244, 200, 153, 111,
    185,  36,  39, 174,  65, 228, 100,
    155, 147,  76, 164, 149, 153,  27,
    120,  82, 184,  85
  ],
  validatorsHash: Uint8Array(32) [
    199,  60,  28, 106, 130, 156, 155,
    243, 245, 116, 205,   0, 252,  29,
    158, 181,  79, 194, 227, 120, 250,
    196, 208, 162, 245, 120, 120, 205,
    247,  77, 254, 247
  ],
  nextValidatorsHash: Uint8Array(32) [
    199,  60,  28, 106, 130, 156, 155,
    243, 245, 116, 205,   0, 252,  29,
    158, 181,  79, 194, 227, 120, 250,
    196, 208, 162, 245, 120, 120, 205,
    247,  77, 254, 247
  ],
  consensusHash: Uint8Array(32) [
      4, 128, 145, 188, 125, 220,  40,
     63, 119, 191, 191, 145, 215,  60,
     68, 218,  88, 195, 223, 138, 156,
    188, 134, 116,   5, 216, 183, 243,
    218, 173, 162,  47
  ],
  appHash: Uint8Array(32) [
     41, 123,  57, 215, 194,  81,  21,
    171, 146, 162, 141, 112, 251, 211,
     96, 141,  89,  22, 126, 221, 136,
    174,  17,  38, 164, 192, 208, 238,
    161,  77, 125, 178
  ],
  lastResultsHash: Uint8Array(32) [
    227, 176, 196,  66, 152, 252,  28,
     20, 154, 251, 244, 200, 153, 111,
    185,  36,  39, 174,  65, 228, 100,
    155, 147,  76, 164, 149, 153,  27,
    120,  82, 184,  85
  ],
  evidenceHash: Uint8Array(32) [
    227, 176, 196,  66, 152, 252,  28,
     20, 154, 251, 244, 200, 153, 111,
    185,  36,  39, 174,  65, 228, 100,
    155, 147,  76, 164, 149, 153,  27,
    120,  82, 184,  85
  ],
  proposerAddress: Uint8Array(20) [
    227,  89, 250, 49, 231,  73,
    255, 220,  87, 26, 164, 229,
    246,  41, 247, 12, 213,  61,
     50, 138
  ]
}

The header hash discrepancy seems to either be due to missing nanoseconds on the time, or missing partSetHeader on blockId (or both?)

This is the code for hashing the header (to get the canonical hash): https://github.com/tendermint/tendermint/blob/059d42866c9971838446611524edb8d75adf926f/types/block.go#L441-L483
Notably, these lines (time and blockId):
pbt, err := gogotypes.StdTimeMarshal(h.Time)

pbbi := h.LastBlockID.ToProto()
bzbi, err := pbbi.Marshal()

gogoproto/StdTimeMarshal is here: https://github.com/gogo/protobuf/blob/deb6fe8ca7c6d06584bfbd40ca407bf69d9fd2aa/types/timestamp_gogo.go#L68-L81
Which does take into account nanoseconds - https://github.com/gogo/protobuf/blob/deb6fe8ca7c6d06584bfbd40ca407bf69d9fd2aa/types/timestamp.go#L111-L120
I guess ts-proto need the new DateWithNanoseconds type?

I found this in ts-proto:

With --ts_proto_opt=useDate=false, fields of type google.protobuf.Timestamp will not be mapped to type Date in the generated types

I guess I need this to compile the tendermint types. And a DateWithNanoseconds type that converts to/from this might be useful as well.
I still need to check about the missing info in blockId

With --ts_proto_opt=useDate=false, fields of type google.protobuf.Timestamp will not be mapped to type Date in the generated types

This sounds like a reasonable default. With this set, timestamps become a nested structure like this: https://github.com/protocolbuffers/protobuf/blob/master/src/google/protobuf/timestamp.proto#L136. Would be good to have a DateWithNanoseconds constructor that can easily convert from this.

I will try this issue - both the blockId parts (in tendermint-rpc) as well as the timestamp (only in ts-relayer).
When it works, happy to share my changes for a proper implemention in CosmJS

Btw, looking at the json output from the rpc, I am amazed they still have the mixed number and string representations for integers. Seems like "smallish numbers" go to int? :facepalm:

Okay, so the blockId.parts was set by tendermint-rpc. It was just names `partSetHeader in the protobuf file.
    const header = Header.fromPartial({
      ...rpcHeader,
      lastBlockId: {
        hash: rpcHeader.lastBlockId.hash,
        partSetHeader: rpcHeader.lastBlockId.parts,
      }
    });

fixes is

ts-proto is an issue, but can be compiled in the apps that need it for their protobuf types.
There is a bug in ts-proto however... which needs to be fixed upstream.
Closing this as no changes are needed in CosmJS for this.

ts-proto is an issue, but can be compiled in the apps that need it for their protobuf types. There is a bug in ts-proto however... which needs to be fixed upstream.
Closing this as no changes are needed in CosmJS for this.

was this ever fixed in ts-proto? i'm still running into this header validation issue with the latest version of @confio/relayer :/

ts-proto has been fully replaced by telescope type generation and a lot of this low level code changes on both the RPC side and the generated code. Could you try debugging getSignedHeader in src/lib/ibcclient.ts and open a ticket in ts-relayer with the latest insights around the issue?

ts-proto has been fully replaced by telescope type generation and a lot of this low level code changed on both the RPC side and the generated code. Could you try debugging getSignedHeader in src/lib/ibcclient.ts and open a ticket in ts-relayer with the latest insights around the issue?

i will fix it 🫡

ts-proto has been fully replaced by telescope type generation and a lot of this low level code changed on both the RPC side and the generated code. Could you try debugging getSignedHeader in src/lib/ibcclient.ts and open a ticket in ts-relayer with the latest insights around the issue?

i will fix it 🫡

🤝
https://github.com/confio/ts-relayer/pull/285


Manage Date / duration type

the dicom standard uses massively temporal types (encoded in String), it can be useful to add methods to obtain directly time.Time or time.Duration in Go.
Proposal (in element.go) :
func MustGetDates(v Value) (t []time.Time, err error) {
	if v.ValueType() != Strings {
		err = fmt.Errorf("MustGetDates expected ValueType of Strings, got %v", v.ValueType())
		return
	}
	vl := v.GetValue().([]string)
	for _, s := range vl {
		d, err := time.Parse("20060102", s)
		if err != nil {
			return t, fmt.Errorf("MustGetDates are unable to parse date in %s", s)
		}
		t = append(t, d)
	}
	return
}

func MustGetDatesOrZeo(v Value) (t []time.Time) {
	t, err := MustGetDates(v)
	log.Print(err)
	return
}

func MustGetTimes(v Value) (t []time.Time, err error) {
	if v.ValueType() != Strings {
		err = fmt.Errorf("MustGetTimes expected ValueType of Strings, got %v", v.ValueType())
		return
	}
	vl := v.GetValue().([]string)
	for _, s := range vl {
		d, err := time.Parse("150405.000000", s)
		if err != nil {
			return t, fmt.Errorf("MustGetDates are unable to parse date in %s", s)
		}
		t = append(t, d)
	}
	return
}

func MustGetTimesOrZeo(v Value) (t []time.Time) {
	t, err := MustGetTimes(v)
	if err != nil {
		log.Print(err)
	}
	return
}

func MustGetAge(v Value) (t time.Duration, err error) {
	if v.ValueType() != Strings {
		log.Panicf("MustGetAge expected ValueType of Strings, got %v", v.ValueType())
	}
	vl := v.GetValue().([]string)
	if len(vl) != 1 {
		return t, fmt.Errorf("found %d value(s) in GetAge (expect 1)", len(vl))
	}
	vlu := strings.ToUpper(vl[0])
	if len(vlu) != 4 {
		return t, fmt.Errorf("%s is not on age string format (nnn[D-W-M-Y])", v)
	}
	value := vlu[:3]
	intValue, err := strconv.Atoi(value)
	if err != nil {
		return t, fmt.Errorf("can't parse %s as interger", value)
	}
	kind := vlu[3:4]
	switch kind {
	case "D":
		return time.Duration(intValue * 24 * 3_600 * 1_000_000_000), err
	case "W":
		return time.Duration(intValue * 7 * 24 * 3_600 * 1_000_000_000), err
	case "M":
		return time.Duration(intValue * 2_629_746 * 1_000_000_000), err
	case "Y":
		return time.Duration(intValue * 31_556_952 * 1_000_000_000), err
	}
	return
}

func MustGetAgeOrZero(v Value) (t time.Duration) {
	t, err := MustGetAge(v)
	if err != nil {
		log.Print(err)
	}
	return
}

Hi there! Thanks for the suggestion, I think it's a good idea to help folks work with native Go dates and durations.
I'd like to think a little about where this should live. I'm wondering if maybe this should be inside a utility subpackage (something like pkg/dcmtime) since the top level library API is already becoming somewhat complicated, and I'd like to simplify. Also, this logic may be useful for folks working with dicom date strings that may not come directly out of a dicom (e.g. maybe from some third party place like Google cloud dicom store).
The other option is to treat Date Strings as special ValueTypes during parsing, and parse them directly into native Go containers. Because without generics this is a little cumbersome, I'm somewhat partial to maybe avoiding too many additional ValueTypes if possible.
I'm leaning towards a dcmtime package with a different API, perhaps something like:
// FromDateString converts a DICOM date string (from VR=DT or VR=DA) to a Go time.Time.
FromDateString(dateStr string) (time.Time, error)

// FromAgeString ...
FromAgeString(ageStr string) (time.Duration, error)

Agree, look better to keep only Go base types in default package and move time.Time, time.Duration, etc.. in an extension.


Update analysis to work with integer timestamp

🚀 Feature Request
Update analysis to work with integer timestamp.
Proposal
Update signature for parameters start, end and add error check like in #146 for TSDataset.plot:

decomposition/plot_time_series_with_change_points
decomposition/plot_change_points_interactive
eda/plot_holidays: to discuss
eda/plot_imputation
outliers/plot_anomalies
outliers/plot_anomalies_interactive

Fix signatures:

outliers/plot_anomalies_interactive:

parameter method

outliers/get_anomalies_density
outliers/get_anomalies_hist
outliers/get_anomalies_median
outliers/get_anomalies_prediction_interval

Update code:

eda/plot_holidays

It shoudn't work if holidays is str
It should work if holidays is df, in that case ds should be integer
Alternative: always fail on integer timestamp

Check all indexing of ts.df and ts.to_pandas().df and make sure that by timestamp it is done with .loc

Otherwise it could be interpreted as .iloc index

Test cases

Add tests on error for start, end methods
Add tests on plot_holidays

Additional context
Blocked by #146.

Closed by #161.


unknown field "spec.scaleTargetRef.metrics"

Dear team,
When running the command kubectl apply -f aks-store-quickstart-hpa.yaml, the following issue appears:

Error from server (BadRequest): error when creating "aks-store-quickstart-hpa.yaml": HorizontalPodAutoscaler in version "v1" cannot be handled as a HorizontalPodAutoscaler: strict decoding error: unknown field "spec.scaleTargetRef.metrics"

Which led me to run the following command to check the 'scaleTargetRef' parameters:
kubectl explain --api-version=autoscaling/v1 HorizontalPodAutoscaler.spec.scaleTargetRef
and the result is :
GROUP:      autoscaling
KIND:       HorizontalPodAutoscaler
VERSION:    v1

FIELD: scaleTargetRef <CrossVersionObjectReference>

DESCRIPTION:
    reference to scaled resource; horizontal pod autoscaler will learn the
    current resource consumption and will set the desired number of pods by
    using its Scale subresource.
    CrossVersionObjectReference contains enough information to let you identify
    the referred resource.

FIELDS:
  apiVersion	<string>
    apiVersion is the API version of the referent

  kind	<string> -required-
    kind is the kind of the referent; More info:
    https://git.k8s.io/community/contributors/devel/sig-architecture/api-conventions.md#types-kinds

  name	<string> -required-
    name is the name of the referent; More info:
    https://kubernetes.io/docs/concepts/overview/working-with-objects/names/#names

And, as you can see, the metrics parameter isn't referenced.
Instead, I've used the targetCPUUtilizationPercentage under the spec section and it worked as expected.
The aks-store-quickstart-hpa.yaml file should look like this:
apiVersion: autoscaling/v1
kind: HorizontalPodAutoscaler
metadata:
  name: store-front-hpa
spec:
  maxReplicas: 10 # define max replica count
  minReplicas: 3  # define min replica count
  targetCPUUtilizationPercentage: 50 # target CPU utilization
  scaleTargetRef:
    apiVersion: apps/v1
    kind: Deployment
    name: store-front

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

ID: 6ab10c96-1f2f-e48f-a084-b1715156e33f
Version Independent ID: a113b8e9-259a-2a65-2e54-a2661c78ddb5
Content: Kubernetes on Azure tutorial - Scale applications in Azure Kubernetes Service (AKS) - Azure Kubernetes Service
Content Source: articles/aks/tutorial-kubernetes-scale.md
Service: azure-kubernetes-service
GitHub Login: @MGoedtel
Microsoft Alias: magoedte

@khalilG
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

#reassign:schaffererin

Hi, @khalilG! Thank you for your feedback. I tested for verification purposes, confirming the issue and resolution suggestion. I created a PR with the updated aks-store-quickstart-hpa.yaml. The changes will be live in the AKS documentation soon. I will now close this GitHub issue. Thanks again :)
#please-close


console spam ProjectPortals-0.13.6

Hello, i noticed frequently in console i will get:
[22:04:23] [HikariPool-1 housekeeper/DEBUG] [com.zaxxer.hikari.pool.HikariPool/sponge]: HikariPool-1 - Pool stats (total=9, active=0, idle=9, waiting=0)
https://gist.github.com/ktole1999/af9487d9fba4d7e8933858f9c308a6fa

Your viewing the log with debugging enabled. this is normal output, displaying the pool stats of the H2 db connections


Direct TTY to specific process?

Is there a way to direct the active TTY to a specific process?
We regularly use Foreman in development and have the need to pry/byebug debug them often.
Thanks

second that. one could redirect the tty to the process running the debugger, in my case the development web server. could be a command line switch/config option.

Sorry, Foreman is an output-only tool right now.


bump: v0.6

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

This bumps the package to v0.6.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

I dont know what is going on with the CI here? The Azure pipelines has long finished.

The pipeline is currently going to fail because yacs >=0.1.8 doesn't exist yet on anaconda.org. I created an MR to do this in the yacs-feedstock.

@conda-forge-admin, please rerender

Looks like the build is going to succeed! Ready to merge!

Tested locally. Works like a charm! :D


Unstable Version Detected

Hi! I would like to use these plugins

path_provider_android
url_launcher_android

in my Flutter application.
My CI/CD procedure checks for any vulnerable dependencies. For these plugins, an unstable version is detected due to the version indicated in the build.gradle file of the native Android components (1.0-SNAPSHOT). Is it possible to align the version in the build.gradle file with the same version as the Dart side plugin?

Hi @fabionegro-it, what exactly is the vulnerability reported? You should be able to use the plugins without any issue in your application

Hi @danagbemava-nc, thanks for your reply. I confirm that both plugins work correctly.
My application, however, is undergoing a software composition analysis during the CI/CD procedure and some dependencies are identified as possible unstable versions due to the word "snapshot" in the "version" property of the build.gradle file.

some dependencies are identified as possible unstable versions

It's likely that every single plugin in the Flutter ecosystem has this, because it's part of the template. Convincing every plugin author to maintain multiple duplicate copies of version information forever seems like an uphill battle; is there an issue with just ignoring this warning given that it's not usefully applicable to Flutter plugins, for which the Gradle version information is not how the actual versioning is managed?

Thank you for the response. I understand that the “1.0-SNAPSHOT” version is part of the template and that convincing every plugin author to maintain duplicate copies of version information is a challenging task.
However, I work in an environment where security issues are highly scrutinized. Therefore, our CI/CD procedure requires all dependencies to be stable to comply with security and compliance policies, which is a strong requirement for us. Ignoring this warning is not a feasible option for us without accumulating technical debt that will need to be addressed sooner or later.
In practical terms, during the CI procedure, three sbom.xml files are generated (one for dart, one for gradle, and one for cocoapods) and dependency analysis is performed on these sbom files at every level (direct and transitive).
It is strange, however, that the “1.0-SNAPSHOT” label is present only in the build.gradle template for android-kotlin and not for android-java. Also, for the iOS module, the “SNAPSHOT” label is not present in the podspec template file. Wouldn’t it be appropriate to start addressing this issue?

However, I work in an environment where security issues are highly scrutinized.

To clarify, by "ignore the warning" I don't mean ignoring all warnings about all dependencies for the entire project, I mean tracking specific instances and explicitly marking those you have scrutinized and found to be false positives as ignored.

Therefore, our CI/CD procedure requires all dependencies to be stable to comply with security and compliance policies, which is a strong requirement for us.

This warning is not inconsistent with that requirement in the context of a Flutter plugin, however. The version that matters for a Flutter plugin is the version in the plugin's pubspec.yaml. The arbitrary fixed numbers in Flutter plugin's build.gradle or podspec file have no bearing on the stability of the dependency, or its actual version or versioning policy.

Ignoring this warning is not a feasible option for us without accumulating technical debt that will need to be addressed sooner or later.

What is the technical debt? The issue here is not actually "path_provider_android is unstable" it is "some specific CI is looking at the wrong value to try to determine whether path_provider_android is stable".

In practical terms, during the CI procedure, three sbom.xml files are generated (one for dart, one for gradle, and one for cocoapods) and dependency analysis is performed on these sbom files at every level (direct and transitive).

It sounds like this system has enough information to be able to cross-reference which gradle and cocoapods files are from Flutter plugins, and thus to avoid this entire class of false positive in the first place by being adjusted to not look at the placeholder version files when analyzing Flutter plugins. I'm still not clear on why having the CI adjust to the realities of how the Flutter plugin ecosystem works is not the better outcome here than having the entire ecosystem change to accommodate specific CI. Is this a widely used CI system that cannot be changed for some reason?

@stuartmorgan, thank you again for the response. Yes, I am adopting widely used CI systems.
So, the solution you are proposing is to ignore the word "SNAPSHOT" for Gradle and CocoaPods dependencies that are directly related to a Dart plugin?
What do you think about removing the word “SNAPSHOT” from the build.gradle template for android-kotlin?
In any case, we must continue to analyze transitive Gradle and CocoaPods dependencies because they can potentially have vulnerabilities.
Thanks a lot.

Yes, I am adopting widely used CI systems.

What is the system that is doing this check, so that we can reach out to them and see if they can update this check?

So, the solution you are proposing is to ignore issues due to the word "SNAPSHOT" for Gradle and CocoaPods dependencies that are directly related to a Dart plugin?

From the description you have provided, the warning does not appear to provide any useful information in that context. If that's correct, then I can't see any reason not to ignore it.

What do you think about removing the word “SNAPSHOT” from the build.gradle template for android-kotlin?

The Android team will need to weigh in on that, as I'm not familiar with the Gradle subtleties that may be involved here. For instance, I don't know if that would potentially regress local plugin development by causing stale build artifacts while editing plugin source.


Calling model.load_weights() on model built with Sequential API throws ValueError error - TensorFlow Save and loading APIs Guide

System information

Have I written custom code: No. Following a TF Guide and adding a line of code mentioned in its comments - no custom code/implemention.
OS Platform and Distribution: Code is running on Google Colab from Google Chrome (Version 81.0.4044.129 (Official Build) (64-bit))
TensorFlow installed from (source or binary): Binary - TF is running in Google Colab
TensorFlow version: 2.2.0-dev20200508 (tf.version.GIT_VERSION = v1.12.1-31489-g6047d50555)
Python version: 3.6.9
Bazel version: NA
GCC/Compiler version: NA
CUDA/cuDNN version: NA
GPU model and memory: NA

Describe the current behavior
I am running the Save and loading APIs Guide by TensorFlow and came across an error that should not be raised as per the documentation provided in the same Notebook.
I was exploring the behavior of model.load_weights('pretrained_ckpt') in this part of the guide. The code cell contains the following:
# Example 2: Sequential model
# Recreate the pretrained model, and load the saved weights.
inputs = keras.Input(shape=(784,), name='digits')
x = keras.layers.Dense(64, activation='relu', name='dense_1')(inputs)
x = keras.layers.Dense(64, activation='relu', name='dense_2')(x)
pretrained_model = keras.Model(inputs=inputs, outputs=x, name='pretrained')

# Sequential example:
model = keras.Sequential(
    [pretrained_model, keras.layers.Dense(5, name='predictions')])
model.summary()

pretrained_model.load_weights('pretrained_ckpt')

# Warning! Calling `model.load_weights('pretrained_ckpt')` won't throw an error,
# but will *not* work as expected. If you inspect the weights, you'll see that
# none of the weights will have loaded. `pretrained_model.load_weights()` is the
# correct method to call.

The comment at the end states that,

# Warning! Calling `model.load_weights('pretrained_ckpt')` won't throw an error

However, if I add a new code cell right after the mentioned cell with the following line, it throws an error.
model.load_weights('pretrained_ckpt')
The error is:
WARNING:tensorflow:Inconsistent references when loading the checkpoint into this object graph. Either the Trackable object references in the Python program have changed in an incompatible way, or the checkpoint was generated in an incompatible program.

Two checkpoint references resolved to different objects (<tensorflow.python.keras.engine.training.Model object at 0x7f4335f43ac8> and <tensorflow.python.keras.layers.core.Dense object at 0x7f4334612748>).
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-22-d50d92a829e2> in <module>()
----> 1 model.load_weights('pretrained_ckpt')

10 frames
/usr/local/lib/python3.6/dist-packages/tensorflow/python/framework/tensor_shape.py in assert_is_compatible_with(self, other)
   1126     """
   1127     if not self.is_compatible_with(other):
-> 1128       raise ValueError("Shapes %s and %s are incompatible" % (self, other))
   1129
   1130   def most_specific_compatible_shape(self, other):

ValueError: Shapes (5,) and (64,) are incompatible

Why is this happening? Has something changed recently causing this to break?
Describe the expected behavior
As the comment states, the function call must not throw an error and return a function to be used.

# Warning! Calling `model.load_weights('pretrained_ckpt')` won't throw an error

Standalone code to reproduce the issue
Please see the sample code provided in the notebook linked below to reproduce the issue:
Google Colab Notebook
Other info / logs
NA

I am able to replicate this issue, please find the gist here. Thanks!

@Saduf2019
Thanks for the links. I went through them and I'm afraid I could not use them. However, scrutinizing the lines, I can see why calling the load_weights() method would fail. Below is a brief explanation.
model is built using Sequential API and is based on pretrained_model followed by Dense(5) layer.
pretrained_model: Input((784,)) -> Dense(64) -> Dense(64)
model: [pretrained_model, Dense(5)] = Input((784,)) -> Dense(64) -> Dense(64) -> Dense(5)
However, the save_weights() method is called on pretrained model which has the following architecture:
pretrained: Input((784,)) -> Dense(64) -> Dense(64)
Since model and pretrained have different architectures (and the extra layer in model does have weights), calling load_weights() method on model will/should pretty much throw an error.
So, basically, changing this line
model = keras.Sequential([pretrained_model, keras.layers.Dense(5, name='predictions')])
to this
model = keras.Sequential([pretrained_model])
allows the code to run without an error (Google Colab Notebook).
According to the remaining part of the comment, calling model.load_weights('pretrained_ckpt') should not have left the weights unchanged. But they do change.

# Warning! Calling model.load_weights('pretrained_ckpt') won't throw an error,
# but will not work as expected. If you inspect the weights, you'll see that
# none of the weights will have loaded. pretrained_model.load_weights() is the
# correct method to call.

The following assertion test did not throw an error which indicated the weights in model were actually updated.
for a, b in zip(pretrained.weights, model.weights):
  np.testing.assert_allclose(a.numpy(), b.numpy())

(Why are we comparing model.weights against pretrained.weights? Because the weights were saved from pretrained model by calling pretrained.save_weights('pretrained_ckpt'))
After a bit of digging, I realized that model is inheriting its weights from pretrained_model when building the model calling Sequential([pretrained_model]). Since pretrained_model weights were pre-loaded a few lines earlier with a call to pretrained_model.load_weights('pretrained_ckpt'), model ends up having the same weights as pretrained model and the assertion above will not throw any errors.
To actually verify that the model.load_weights('pretrained_ckpt') fails to change the weights in model, simply comment out pretrained_model.load_weights('pretrained_ckpt') line and watch everything fall apart.
I guess it was a case of unclear documentation where the writer had a different idea in mind and the writing did not convey that.
Note to any future readers:
The variable names in this example are very confusing. We have pretrained, pretrained_model, and model variables which are variables of type Model which means we have pretrained model, pretrained_model model and, of course, model model. This makes the last paragraphs a tad confusing, but I've done my best to ensure the variable names are correct.


enterprise: make ingress.class configurable

Ensure that user could customize the ingress.class to other values such as "alb" or "nginx". The default "nginx" value is not changed.

This looks great - could you bump the chart version in Chart.yaml for the enterprise chart? Currently we cut a release on every commit
Take a look at this for an example
https://github.com/buildbuddy-io/buildbuddy-helm/commit/bbe660919b493d01f49141e791f8fe759153adf3
We could probably look at automating this with a github action - but you can do it manually for now.

bumped chart version to 0.0.173


Allow pass options on install plugin

Vue install plugin hook allows pass a second argument.

If the Idea is approved I can work on docs (readme) update

Nice, thanks! A readme update PR would be most welcome :)


Using Micropython 1.2.0

I'm trying to get this working on a Wemos LOLIN S3 mini. https://www.wemos.cc/en/latest/s3/s3_mini.html#
Unfortunately it was released after your recommended build of micropython. The first version available is 1.19.1 which doesn't seem very stable. However, the device works well with V1.20.0 and I've managed to load all the code onto the device and it connects to my MQTT server fine. I've had to change the tools.py file to use  UART0 together with the correct pins. This all works fine until I connect it to my Truma Combi (confirmed compatible). I've attached a log file. I'm not sure whether these issues are caused by the micropython version or by the differences in the ESP32S3.
Are there any tests I can do to confirm the compatibility of micropython 1.20.0 or suggestions on how to debug.
Thanks
Richard
ESPs3 Connection Logs.txt

I have already had experience with the S3, but not with the Wemos design. The chip only has 2 UARTs. In principle, the software works, but not the full flash prints as is possible with the ESP32. The problem is that part of the python code must already be in the firmware, otherwise there is not enough memory. In addition, micropython does not currently support memory paging. That's why the bigger chips with more memory don't make sense. Try a normal ESP32 and if you like a smaller design, take an RPI pico and you will be successful.

Here is an S3 version for testing. After flashing, you must download all modules that are not already loaded on the chip under 'help modules()'.
flash_pekaway_inetbox2mqtt_v251_2M.bin.zip
Allow me to point out that I do not support the version

Many thanks, I will give that a try and report back.

To start from scratch...
esptool.py erase_flash
esptool.py write_flash 0 flash_pekaway_inetbox2mqtt_v251_2M.bin

Fails with
esptool.py v4.5.1
Found 2 serial ports
Serial port /dev/cu.usbmodem14201
Connecting...
Detecting chip type... ESP32-S3
Chip is ESP32-S3 (revision v0.1)
Features: WiFi, BLE
Crystal is 40MHz
MAC: f4:12:fa:a8:d7:74
Uploading stub...
Running stub...
Stub running...
Configuring flash size...
Unexpected chip id in image. Expected 9 but value was 5. Is this image for a different chip model?

A fatal error occurred: flash_pekaway_inetbox2mqtt_v251_2M.bin is not an ESP32-S3 image. Use --force to flash anyway.

I've moved over to using the esphome component which works fine with my device so I'll close this request.
Thanks
Richard

Closed


help: Tighten content on /help/printing-messages.

Current page: https://zulip.com/help/printing-messages
Revised:

@karlstolley would be great to get your review on this one.

This looks like great rewrite, Alya! I looked hard, but I don't have any suggestions.
Sorry I missed this until Tim DM'd me about it. I've changed up how I monitor my @-mentions on here so hopefully this won't happen again.

Merged, thanks @alya!


Proposal: .php_cs.lock with list of fixers being (and not) applied

It would be helpful to have this file dumped after each CS Fixer run (similar to composer.lock file).
E.g. for Php Inspections (EA Extended) to not apply "last comma in array" inspection to source code - just read the file and don't bother developers with extra setting management.
Some rough ideas for structure:
header_comment=yes
long_array_syntax=yes
strict=no

Very interesting idea. I'd vote for having it json formatted, like composer does.
Ping @bobthecow - you were concerned over bc breaks due to us introducing new fixers. This might be a good solution for you.

@kalessil A better way to do this as far as I'm concerned is for the lock file to just have the fixers to apply, rather than the whole yes/no deal, so we can just skip straight to that list instead of going through the config resolver like we do at the moment.

@GrahamCampbell : for the format it would be better ini/xml format as PHP/Java has functions from the box to work with these formats. To keep only applied fixers in lock file will work for me.
For file structure it can be more useful options: excluded resources (folders and files), preset id (psrN, Symfony). Maybe something else to state version/BC meta-data.

Update: JSON is fine, there is lightweight library for java - just 20KB.

:+1: Lock file sounds good to me, that'd fix all my concerns :)

Regenerating this file after each PHP CS Fixer run is pointless - it must be generated once and then used. yaml is good with me too.
@GrahamCampbell , maybe create one such lock file and then no need to have it duplicated into StyleCI configuration?
One more thing: sb configured fixers inside .php_cs, then store it in lock file and change configuration. When running a fixer we will taki list from lock file even when config change ? And what if one run a fixer with --fixers option?
Such a lock file must be generated on demand, not always. Remember that there will be users that want to use full level like PSR2 or Symfony - even when new fixer appear.

Something like npm shrinkwrap then?

Maybe one step further: declarative cs_fixer.json as composer again, and then lock-file just to state BC meta-info?
Then I can just use manifest and adjust my inspections accordingly, without any workarounds/

Maybe one step further: declarative cs_fixer.json as composer again, and then lock-file just to state BC meta-info?

You say to replace .php_cs with cs_fixer.json ?
No. One is PHP, one is JSON. You can't do all with JSON.

@keradus : in .php_cs it's possible to specify preset, exclude some directories/files, add extra fixers. What else can be there what is not possible to make "declarative"?
I can even make a prototype dumping declarations from .php_cs to cs_fixer.json and loader for it.

Fiddle with Config and it properties, eg applying sth on finder (like filtering), conditional settings, including other files (like .php_cs.local when we don't have .php_cs.dist). And so on so on...
https://github.com/zendframework/zf2/blob/master/.php_cs#L23
https://github.com/bobthecow/mustache.php/blob/dev/.php_cs#L15

Affirmative, then only lock file is option for 3rd parties.
It's actually means regeneration of the lock file after each run to be sure it's up to date.
VCS shall be fine with this and overhead shall be not so big one.

:-1: for regenerating on each run - then it will be no benefits to have both files, if config is always used and lock is always regenerated
:-1: for VCS

Ok, even if it's generated on demand, it's better than nothing at all =)

:-1: for replacing the .php_cs file. I actually see this being useful as an actual lock file, like composer has, so that we can guarantee the same fixers are used between versions.

Also lock files would have nothing to do with StyleCI?

@GrahamCampbell thinking about sth that can be common between PHP CS Fixer and StyleCI. And that lock file could be thath thing

Structure .styleci.yml would fit as for me (but in JSON as I mentioned already)
preset: name
version: 1.6
enabled:
  - strict
  - strict_param
  - others expanded from preset

disabled:
  - short_array_syntax

@GrahamCampbell : what about excluded resources? How it make sense to define them?

I am thinking about sth that can be common between PHP CS Fixer and StyleCI. And that lock file could be that thing

Nah. We're totally ignoring anything php-cs-fixer does with config atm. We're rolling out own system, and have no plans to change.

what about excluded resources? How it makes sense to define them?

That's exactly the problem. You can't atm.

I see.
Then I can't see any benefit of that lock file. Everything that could be possible with lock file is possible with current config file.

Everything you can do with the Composer lockfile you can do with composer.json.

@keradus I see the lock file as being useful to lock the fixers to use, and nothing more. IE, it's not the same as composer's lock file that contains all the information needed.

yeah, but one can lock fixers with .php_cs as well.

@keradus Yes, but that's not convenient. They;d have to manually go and check what fixers are new with each release, instead of just running a single command to update the lock file.

OK. If anyone willing to take it go ahead ;)

yeah, but one can lock fixers with .php_cs as well.

one can lock dependencies with composer.json as well … but that doesn't mean it's the most effective and convenient way to do it ;)

If it will be no volunteers, I could start with it, but it will be earliest mid of April (is_null() fixer first).

Anyone willing to create PR for this issue?
After introducing RuleSet and FixerFactory is should be a bit easier.

what about excluded resources? How it makes sense to define them?

That's exactly the problem. You can't atm.

Is excluding resources covered by this issue or is there a separate one? It's a feature I really miss.

Is excluding resources covered by this issue or is there a separate one? It's a feature I really miss.

what kind of resources would you like to exclude which cannot be excluded by the current configuration file?

I was looking for some docs of the config file but found nothing helpful.
Can I create a .php_cs file with
exlude=somedir

What's the exact syntax?

Maybe this is an handy example; for this project we use the following to exclude the fixtures directory;
https://github.com/FriendsOfPHP/PHP-CS-Fixer/blob/1.13/.php_cs#L31
The way this works is that the returned object by the config extends the Symfony Finder class.

Thank you! I will try this. Didn't exepect .php_cs to be a PHP-file :)

Reading the discussion I don't see any benefit of having some lock file like Composer does.
IMO the solution is to lock the version of PHP CS Fixer that is installed with composer.lock so that one can choose when to update it and update the config file together, if needed. A specific constraint (e.g. 2.16.*) in composer.json can be used if the lock file is not versioned like in most lib projects.


remove cookie-cutter dependency

we remove the top-level cookiecutter instructions and migrate the project to amlrt_project instead

@jerpint : I do not think I will have the time necessary to review this code.
I do agree with @marondeau-mila that a general review can be enough..
To be safe, I would remove my review request, and add @hbertrand .
Two positive opinions will be enough to merge (so we speed up this PR).


How to get RK result

I have some question about RK, what is the meaning, and how can I find the paper introduce this method?

RK stands for re-ranking, there are various papers proposing re-ranking techniques. This repo implements this paper:
Zhong Z, Zheng L, Cao D, et al. Re-ranking Person Re-identification with k-reciprocal Encoding[J]. 2017.
code here:
https://github.com/RQuispeC/top-dropblock/blob/master/torchreid/utils/rerank.py
To get RK results you should change the config file (e.g. this):
test:
  rerank: True


Create ru_ru.json

Why stop on mine cells when we can also translate this one) Btw brown leaf pile translation probably got lost while reorganizing, so i added it back.

That's awesome! ❤ Thank you once again!!
One thing tho - do you mind also translating "Cherry Leaf Pile"? You can comment the translation here, I'll be sure to include it for the 1.20.1 release 😁

Yeah, sure) The translation is "Ворох вишнёвых листьев"


Include UserID in the patterns reponse JSON

Signed-off-by: Utkarsh Srivastava<EMAIL_ADDRESS>Description
This PR changes the response type of /api/experimental/patternfile to include user id fields.
Sample response type for remote provider (meshery remote provider):
{
  "page": 0,
  "page_size": 10,
  "total_count": 1,
  "patterns": [
    {
      "id": "336814b9-96c0-47dd-8793-7b0163704640",
      "name": "istio-with-rollout.yaml",
      "user_id": "d87945d1-c968-42c2-844b-1a67f93c2e71",
      "pattern_file": "// Pattern File Content Here",
      "created_at": "2021-04-16T01:35:54.276926Z",
      "updated_at": "2021-04-16T01:35:54.276929Z"
    }
  ]
}

Sample response type for local provider:
{
  "page": 0,
  "page_size": 10,
  "total_count": 3,
  "patterns": [
    {
      "id": "14910b32-a26c-4ce1-9bdb-3567d993431a",
      "name": "istio-with-rollout.yaml",
      "pattern_file": "// Pattern File Content Here",
      "user_id": null,
      "updated_at": "2021-04-08T19:32:42.940009933+05:30",
      "created_at": "2021-04-08T19:32:42.941020336+05:30"
    },
    {
      "id": "e8fecfc5-c700-41bd-90e6-a7f107b4f6b5",
      "name": "istio-demo.yaml",
      "pattern_file": "// Pattern File Content Here",
      "user_id": null,
      "updated_at": "2021-04-08T19:30:38.506767011+05:30",
      "created_at": "2021-04-08T19:30:38.508806216+05:30"
    },
    {
      "id": "69decb5c-7c56-4eba-9e2c-5757f2ac4df8",
      "name": "istio-demo.yaml",
      "pattern_file": "// Pattern File Content Here",
      "user_id": null,
      "updated_at": "2021-04-01T20:40:23.81920176+05:30"
    }
  ]
}

Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

@aneeshnema @s1ntaxe770r FYI


onSwipeNext or previous then wait

When swipe swiping next or previous then the change date I pass to API and get the events.
when users Swipe many times at once the date change at any time also called the API function takes some time then git returns the wrong result.
I want that when swiping next or previous then wait until the data is not gotten from API, once data is gotten the users can swipe next or back

As you watch in the video when I swipe many times at once.
the date change but UI can't update
from the API return 4-jun date data but the UI date is 3-jun for that it can't show.
https://user-images.githubusercontent.com/58082294/171629074-f52963e1-a5c6-4149-9644-1835726c9213.mov

Can you provide the code for reproduction?


Does AutoGPTQ currently support Ascend NPUs？

Ascend NPUs are Huawei's neural network chips. Does AutoGPTQ currently support inference/training on Ascend NPUs?

Hi @Dbassqwer, I don't think it does.

Ascend NPUs are Huawei's neural network chips. Does AutoGPTQ currently support inference/training on Ascend NPUs?

Do you have any other solutions on Ascend NPU？


query operation "not equal"

I would like to select all rows, which dont have specific values in a column. Therefor a query option for not equal "!=" would be very helpful.
thanks in advance.

this query option already exist

ah, I'm sorry. Apparently it isn't written in the docs = https://danfo.jsdata.org/api-reference/dataframe/danfo.dataframe.query


Hardware Supported Keychain for Lollipop+

Can we get a KeyChain class that implements the hardware keystore instead of shared preferences?

We'll look into the keychain from Lollipop. Do you have any docs of where the storage of keys changed to hardware backed keys? The last time i checked the android keystore stored keys protected by the unlock password.

http://developer.android.com/training/articles/keystore.html

👍 for the feature.
Looks like we can generate the key for the user and store it safely in the Android KeyStore.
https://medium.com/@stelianmorariu/sharedpreferences-and-android-keystore-c4eac3373ac7
Min API is 18+ though

Yeah most of the Keystore stuff is pretty late API, we could have it fallback to shared preferences.  A lot of the keystore stuff, as I understand it, is broken even on APIs that support it. From my experimentation it's very difficult to rely on a particular handset having the proper encoding or keytype support. There is going to have to be a lot of testing and graceful fallback to ensure operation.

Android 4.3 is vulnerable https://securityintelligence.com/android-keystore-stack-buffer-overflow-to-keep-things-simple-buffers-are-always-larger-than-needed/
Is it actually safe to use the default SharedPrefsBackedKeyChain to store the KeyChain? If someone roots the device, wouldn't he be able to access the keys and be able to read the encrypted values?
Or does Conceal covers a safe storage of the keys? Like in http://developer.android.com/intl/es/training/articles/keystore.html
Cheers!

I can see a beginning of answer in this thread:
https://news.ycombinator.com/item?id=7140497
and this one: https://www.reddit.com/r/androiddev/comments/1x6owd/use_facebooks_conceal_library_to_store_api_keys/
Answer being no, right?
I am more interested in the Api keys encryption scenario and I believe I am not the only one.
Conceal has the advantage to greatly simplifying the encryption process, making the difficult choices for us, compared to the pure Android method...
Now with the fragmentation we can see in http://developer.android.com/intl/es/training/articles/keystore.html I struggle to find a simple way to store encryption keys.

Using the system keystore probably is not a good idea. The fragmentation is one issue, but there's also the design goal of "Extraction Prevention" - key material is not available to the application process.  So if you're not using the "standard" android/java API's for encryption, you can't access the key. This means alternative implementations like Conceal or the crypto used by Realm, are pretty much out of luck - http://developer.android.com/training/articles/keystore.html#SecurityFeatures

Thanks, so is there any way to use Conceal with a safe way to store the encryption key?
I would imagine the Facebook devs who uses Conceal for the Facebook app store the encryption key in a safe place, encryption is worth nothing otherwise.
Are there any good suggestions?
Android devs jump on Conceal because it makes encryption easier, but if the default setup is not secured, there must be tons of app not properly secured out there which use Conceal.
Or am I missing something regarding the encryption key storage?

Based on previous discussion, I assume that the purpose of Conceal is to provide easy encryption mechanism, but the burden to store keys is purely on user shoulders ?
The case I want to protect users against: phone is stolen - data should not be retrieved (assuming that phone was already rooted).
Let's imagine, that keys are in SharedPrefsBackedKeyChain: then patched application is installed (which does not require password) and attacker can see all stored data.
It seems to me that hardware-backed storage does not protect against this scenario either.
One can say that password-derived encryption key is a good idea, but then you have to cache password hash somewhere in order to avoid necessity of user login on each application launch.
So, the solution I see (for Android) is to have user data encrypted by password-derived key which is cached on the file system and thus vulnerable.
But, the key itself is additionally encrypted with short pin-code which user must enter each time he launches app. Pin-code is never stored and validated by the fact of success keys decryption.
I'm not a security specialist, so want to hear thoughts from you guys :)
Maybe, @fabiendem already have some good solution for his app

Hi @Vandalko, I am trying to cover the same scenario than you but moved onto other stories for now.
I will have to dig back into this later.
Though now I think it's a matter of finding the right balance between the security level you want to achieve and the amount of time you want to spend on this. (and the sensitivity of the data...)
In the end even the hardware backed encryption gets hacked by researchers at some point.
Still, having the PIN code when you open the app is probably the safest solution as nothing is stored.

What do you all think about this idea?
Using the AndroidKeyStore, create an RSA key pair (see https://developer.android.com/training/articles/keystore.html#GeneratingANewPrivateKey).  This will work on SDK 18+, and gives you a public and private key, stored in the secure hardware (assuming the device has the secure hardware).  We can't get the private key bytes back out, and we can't use RSA keys with AES, but we can get the public key as a byte array, as not being able to share that would be... problematic given the whole concept of asymmetric encryption.
As long as the public key isn't shared anywhere, the byte[] of the public key can then be used as input into PBKDF2 (https://android-developers.googleblog.com/2013/02/using-cryptography-to-store-credentials.html) to generate an appropriate AES key as needed.
And then you can use Android's built-in require authentication for key use features (https://developer.android.com/training/articles/keystore.html#UserAuthentication).

Hello, that sounds interesting.
It's true that how Facebook manages log-in (user is not challenged if already logged in) makes difficult to protect data for current logged in user in case of a stolen phone. If you have other app that behaves differently then you could use the user-challenge (pin, password, something) to generate a key which protects the real data key.
There's a project I'm on but wasn't first on my priorities for now. It involves leveraging the Android KeyStore when possible. The idea is similar but as everything else in computing involves another step of indirection.
KeyStore doesn't allow the user to retrieve data from it. But it can generate RSA pairs and you can use it (it doesn't allow symmetric encryption in old versions at least). But you can use that to encrypt the data key in internal storage. Let use names:
DEK = data encryption key, this is what Conceal would use for the data you want
KEK = key encryption key, this encrypts the DEK, in this case it'll be the KeyStore key pair
Therefore:

a KEK pair is generated for the app
a random DEK is generated in memory
EDEK (DEK encrypted with KEK) is stored in internal storage
So storage is not readable except using KeyStore which is only for the app.

When app needs the key (at least first time, later it can cache it in memory), it can ask the KeyStore to decrypt the EDEK from disk with its KEK.
The issue with this is that supposedly the Android KeyStore might be wiped if user changes the phone screen lock. That means app should need a backup mechanism to restore the DEK. If you're ok with asking user to login again, then the same mechanism of having a EDEK can be applied with a server provided KEK, or with a key derived from the user password (using PBKDF).
But yes, if you are ok with having a backup method of challenging the user in case the KeyStore is wiped out, this scheme would be totally usable.


Advanced Feature No.2 Mockup

This is a sub-task of issue #3.

The mockup for a advanced feature of the Simple-Camera app has been demonstrated with steps
inside the feature mockups wiki page:
Feature Mockups
The feature mockups wiki page will be updated as we progress along with our sprints.


Fix typos in /service-accounts-admin.md

content/en/docs/reference/access-authn-authz/service-accounts-admin.md

Preview: https://deploy-preview-37844--kubernetes-io-main-staging.netlify.app/docs/reference/access-authn-authz/service-accounts-admin/

/approve


How to run on the windows

How to run on the windows，

first download classyshark.jar   open cmd  window use commandline like this:
java -jar <ClassyShark.jar path> -open

get the latest jar from releases ==> https://github.com/google/android-classyshark/releases
run the jar on Windows according to this ==> http://stackoverflow.com/questions/394616/running-jar-file-on-windows
Please let us know if there is a problem


Paginators

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue, please leave a comment

Tell us about your request
Implement paginators in the AWS SDK for Rust

shipped in 0.4.1


Cannot navigate with VoiceOver

I noticed during testing that VoiceOver:

Doesn't recognize that a list has been opened, only the input field.
Doesn't recognize that when entering text the list is updated
Doesn't recognizeNavigating the dropdown list with the arrows is not reported either

I might be able to contibute code to fix this.

Thanks for pointing this out. Any improvements in this regard are very welcome.


Turbospectrum

I have never tried  Turbospectrum but it is supposed to treat molecules better than MOOG. It would be nice to implement it. I have no idea of the format of the input data required. Any help appreciated.

Interesting idea. Are we sure we want to go in that direction? I think it's worth weighting the pros and cons before starting this. It seems like a major work (I might be wrong about that).

I think Turbospectrum is more suitable in the analysis of M dwarfs compared to MOOG. That is the only advantage I can think of. At the moment I am not going to work on that. I think the tricky part is the format of line lists and model atmospheres so it would be better a person with experience to help us.

Okay, let's keep the issue open, if there is an opportunity

Actually, we just got a new student to work on M dwarfs in NIR. It might be something he can help with.


Add Section/chapter 4 opening

Create Chapter 4 section Opening.

Related issue: https://github.com/ModDota/dota-tutorial/issues/39


Make EVM chains injectable through genesis and make them params

Background
What the title says :)
Done when

[ ] TODO

This is a duplicate and it was also done :)


[Attach link] Attach button is active though required field is not filled in

EPAM AI DIAL chat version
0.10.0
What happened?

Create new conversation based on any model or app which supports an attachment of a link, for example DIAL RAG
Click on Clip icon and select Attach link
Note that Attach button is active -> bug

Expected: Attach button should be disabled
Issue2:
Tooltip is shown unexpectedly in the middle of the form

Confidential information

[X] I confirm that do not share any confidential information

Verified on staging env successfully


How do you work this?

Hi, i'm using a mac with no experience of coding.... How do I actually get this program to work?
Thank you!

Hi, I'm just a member.
The program was live on a site that no longer works.
In order to run it local in your machine I think you will need a google map
API key. The problem is that google ask you for a credit card in case you
exceed some quota, which is annoying.
Greetings.
On Thu, Nov 1, 2018 at 12:15 PM Tanveer-Saini<EMAIL_ADDRESS>wrote:

Hi, i'm using a mac with no experience of coding.... How do I actually get
this program to work?
Thank you!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/TeehanLax/Hyperlapse.js/issues/43, or mute the thread
https://github.com/notifications/unsubscribe-auth/ADxg1b7DhoM3CB6w1PS6Ahyrp4iPRCcrks5uqxAGgaJpZM4YGoXq
.


Portfolio: Fix details view animation

This has been written by someone who definitely had no idea what he's doing... What a mess...
picocms/Pico#358

Nice work on fixing that CPU usage.  It also feels a lot smoother now, probably because it was lagging more than I realized before. :+1:
I did find one bug though... :sweat:
If the fullsize image isn't already loaded/cached, the size of the content area is calculated wrong.  Instead of fitting the image, it fits to the text, which results in the image being cut off on an item with shorter text.
After the image is loaded or if it is in the browser cache, it functions properly from then onward.  Unfortunately, this happens for every item, as the images are loaded on demand.  It results in a very broken first visit to the page.
This may also be a result of my slower connection at work, but either way needs to be fixed.
Here's a screenshot of the issue (complete with Network Panel :smile:)

If I were to revert this merge, does it return to its "Ready to Merge" status?  Or will you have to submit a new PR?

I'll open a new PR.

So that's a "No"? :unamused:
Guess I should have tested it first. :sweat_smile:

What do you mean by "Ready to Merge" status?

If I click "Revert", does it go back to how it was, letting me choose to merge it again later on, or does it get rejected?

No, it adds another commit reverting the merge and keeps the PR closed. You can't reopen a PR once it has been merged. As said, I'll open a new PR.

and keeps the PR closed. You can't reopen a PR once it has been merged.

Okay, that's what I was wondering. :disappointed:


Modify the benchmark output file name with hours, minutes information

The current all-in-one benchmark save the csv file with the name only have date.
if we run multi test in the same day, the older test data will be overwrite by the new one.
Suggest to modify the file name with hours and minutes.
Here is the suggested code change, and the new file name example
../../../dev/benchmark/all-in-one/pipeline_parallel_gpu-results-2024-06-22_22-20-51.csv
diff --git a/python/llm/dev/benchmark/all-in-one/run.py b/python/llm/dev/benchmark/all-in-one/run.py
index 2036f6f9ee..df7ae3539e 100644
--- a/python/llm/dev/benchmark/all-in-one/run.py
+++ b/python/llm/dev/benchmark/all-in-one/run.py
@@ -25,6 +25,7 @@ import csv

 import numpy as np
 from datetime import date
+import datetime

 import os
 current_dir = os.path.dirname(os.path.realpath(__file__))
@@ -1826,7 +1827,9 @@ def run_pipeline_parallel_gpu(repo_id,
 if __name__ == '__main__':
     from omegaconf import OmegaConf
     conf = OmegaConf.load(f'{current_dir}/config.yaml')
-    today = date.today()
+    # today = date.today()
+    now = datetime.datetime.now()
+    today=now.strftime("%Y-%m-%d_%H-%M-%S")
     if 'exclude' in conf:
         excludes = conf['exclude']
     streaming = False

This change will affect our nightly performance test's post data processing. @lalalapotter will look into this.

Fixed in PR #11444 , please try it.


Create Topic specifying # of partitions

Currently, there is no way to create a Topic in Kafka and specify the number of Partitions for the Topic using this library.  This feature is available through the Kafka API.  This requires one to do it through scripts on the Kafka server or ensure the default setting for the servers is the number of partitions you want and they'll have to be the same for all topics.
Not having this feature limits the usefulness of this library in a production environment.

Meant to respond sooner.
After looking briefly at the Kafka docs, I think you're right.  I've created a docker image that will do it for me now and hopefully soon I can publish it for others to use.


Unable to Apply Imported AWS Security Groups due to Cycle

Currently AWS Security Group are imported in its standalone form with inline ingresses and egresses, due to that not always the imported resources are applicable.
Terraform has aws_security_group_rule resource that could be used instead of inlined rules to avoid this problem.

I know this issue, but aws_security_group_rule it's very ugly solution for many groups.
aws provider send API request for each rule.....

I've tried to parse the generated HCL and extract the ingresses and egresses to aws_security_group_rules but I've failed. But even if I had succeeded the terraform.tfstate file would still be the inlined form and I don't know it's feasible to do that on this file as well.

Let’s then compute a spanning tree and only create rules on edges

Is it possible to force terraformer to import aws_security_group_rule instead of inlined?
We need this because you can't use both aws_security_group and aws_security_group_rule. For instance, we have aws_security_group in one module, but we add rules (aws_security_group_rule) in others.
Changing it manually after an import is tedious.

@RobertoUa, you should file another issue for that


Fix testing as Java application

Hello, I'm currently working on integrating Mill over the BSP plugin for IntelliJ (https://github.com/JetBrains/hirschgarten - now the BSP plugin code is merged with Bazel-related plugins). I have stumbled upon an issue, where the Test <module.name> as Java application action did not work with a testing suite. It only performed a Query JVM environment task and tests were not run (later on it turned out that it was caused by the zincWorker in MillJvmBuildServer#jvmRunTestEnvironment, which could not find the correct test main class). After digging into firstly the BSP plugin's code and later on Mill's code, I discovered that in the MillJvmBuildServer#jvmRunTestEnvironment there is testing logic lacking. It did not manage to fetch the correct test-specific main class, classpath and main arguments necessary for the testing as a local Java application. I figured out that in order to make it work, I needed to leverage the TestModule's logic - I reproduced the MillJvmBuildServer#testTask logic of creating the java process' arguments and used it in a new TestModule#getTestEnvironmentVars function, which is then used in the MillJvmBuildServer#jvmRunTestEnvironment in case of a TestModule with JavaModule module (note: the code reproduction I've done might not be the most graceful solution - please let me know if I can do it better). Then the test-specific main class, classpath and main arguments I mentioned are assigned to adequate fields of the function's result - JvmEnvironmentItem, which is finally used on the BSP plugin side, which launches the actual local Java application.
If you need any help with e.g. setting up the BSP plugin, I'm happy to help

@AnthonyGrod can you revert all non-essential changes in the PR so it is easier to review exactly what you are trying to do

@lihaoyi sorry, done.

Thanks @AnthonyGrod !

I think this looks reasonable, would be good if @lefou could take a look before I merge it

@AnthonyGrod can you update the PR description to include how you tested this? Manual testing is fine, but we should at least write down how it was done in case someone needs to reproduce the test in future. After that I can merge this

@lihaoyi I have updated the description.

Thanks!


Get TravisCI building consistently

Use this: http://robolectric.org/

This has resolved itself.  Will open again if issues crop up in the future...


Redirect works incorrectly when redirecting site is hosted on a non-standard port

This is caused by a bug in Url code for relative addresses - the port is taken from baseUrl.
This assertion fails:
Assert.Equal("https://hosted-domain.com/signin",new Url(new Url("http://localhost:12345/account/login"), "https://hosted-domain.com/signin").ToString());

Thanks for the report. This is definitely a bug.


missing import for annotations on parameters of methods of MapperFactory interface

The DAMapping framework does not generate the import for annotations on parameter of method annotated with @MapperFactory.
Given the following class:
@Mapper
public class SimpleMapperFactory {
  @MapperFactory
  public SimpleMapperFactory(@Nullable boolean flag) {
    // implementation does not matter
  }

  public Float fooBar(BigDecimal inBigD) {
    return 1f; // implementation does not matter
  }
}

I get the following compilation errors:
[ERROR] [...]/SimpleMapperFactoryMapperFactory.java:[9,36] cannot find symbol
[ERROR] symbol:   class Nullable
[ERROR] location: interface fr.javatronic.damapping.test.mappermethod.SimpleMapperFactoryMapperFactory
[ERROR] [...]/SimpleMapperFactoryMapperFactory.java:[6,8] Type fr.javatronic.damapping.test.mappermethod.SimpleMapperFactoryMapperFactory generated by DAMapping has unresolved references ([Nullable]). Compilation will fail
[ERROR] [...]/SimpleMapperFactoryMapperFactory.java:[9,36] cannot find symbol
[ERROR] symbol:   class Nullable
[ERROR] location: interface fr.javatronic.damapping.test.mappermethod.SimpleMapperFactoryMapperFactory

fixed and pushed to master


Go Test failing for go1.20

https://github.com/golangFame/dbresolver/actions/runs/4986849183/jobs/8928043909

@bxcodec
What do u think is the reason?

closing since tests are passing now


zk prover failed

I started the zk server and eth nodes normally, but zk prover kept reporting errors while aggregating proofs.It looks like:
Recursive circuit proof is invalid

Then, I change the command to 'zk prover 3', it doesn't work.

Hey @IversonHan, is this still the case? we could not replicate it locally

Hey @IversonHan, is this still the case? we could not replicate it locally

Well, it's a long time ago, it does not matter anymore.I will close thie issue.Thanks.


[SPARK-3880] HBase as data source to SparkSQL

Can one of the admins verify this patch?

@yzhou2001, we need rebase our branch

and we need check the coding styles.

Hi - thanks for working on this... it looks interesting. I'd like to close this issue (i.e. the PR) and discuss more on the JIRA/dev list rather than having a big pull request like this. For very large features this is the way we do it. If you look on your wiki it says "If you are proposing a larger change, attach a design document to your JIRA first (example) and email the dev mailing list to discuss it."
https://cwiki.apache.org/confluence/display/SPARK/Contributing+to+Spark

Also one thing that would help is if you could create a standalone project for this on github (see spark-avro).


npm install fails

npm install

><EMAIL_ADDRESS>install /Users/timothee/git_clone/temp/h264-live-player/node_modules/bufferutil
> node-gyp rebuild

  CXX(target) Release/obj.target/bufferutil/src/bufferutil.o
../src/bufferutil.cc:25:30: error: no template named 'Handle' in namespace 'v8'
  static void Initialize(v8::Handle<v8::Object> target)

actually https://github.com/131/h264-live-player/pull/76 seems to fix it

I've merged 76, is is okay for you now ?

yes, thanks!


Implement client to request shared unlock key from Locker

Done in parent ticket https://github.com/trustbloc/hub-kms/issues/22


feat: add cache-control headers

Following Gatsby's recommended configuration:

HTML cache-control: public, max-age=0, must-revalidate
app-data.json cache-control: public, max-age=0, must-revalidate
page-data.json cache-control: public, max-age=0, must-revalidate
un-cacheable files (/misc, /sw.js, /css/bootstrap.min.css) cache-control: public, max-age=0, must-revalidate
other css and js public, max-age=31536000, immutable

@raisedadead NGINX didn't work out well for this.  I could not see a way to match against HTML files without hard-coding each path /learn,/ etc. that we needed.
On the plus side, if we go this route we can dramatically simplify the NGINX config by using serve to provide headers for the fonts, too.

Is there a way we can confirm all this works on say staging before I merge and deploy this?

Sure, it's easy enough to test on staging.  Just let me know when it's fine for staging to break and I'll update it to use this config.

Sure go ahead, I will defer any deployments until you are done.

Okay, cool.  I'll let you know how it goes.
I'd welcome any thoughts about the paths we're caching forever, too.  Just because of the risk of it getting stuck in a user's cache for years!

As long as you confirm they have hashed name, we want the immutable behavior.

Right, I'm worried about the ones that aren't hashed.  This makes exceptions for them, but if someone's working on learn and adds a new script in the wrong place it will get cached incorrectly.  I feel like that's quite likely.
Anyway, I've tested this on staging and it all looks good, but other eyes should look at it!

Let's start with a lower max-age like say 2 days


Remove Amazon Drive support

Amazon has changed their policy to not supüport third parties acessing Amazon Drive.
So I will comploetely remove Amazon Drive support.

Releases v2.0.0 today.


Update readme on airbag after 1st test run

Updated the readme after @gestchild took it for the first test run.

@gestchild How\s this look?


Installed on Fedora 24 but "net::ERR_INSECURE_RESPONSE"

I've managed to install Ghost-Desktop on Fedora 24. Here is what I did:
sudo alien -r ghost-desktop-0.7.0-debian.deb
rpm -ivh --nodeps /root/rpmbuild/RPMS/x86_64/Ghost-0.7.0-2.x86_64.rpm

Here many errors occurred:
file / from install of Ghost-0.7.0-2.x86_64 conflicts with file from package filesystem-3.2-37.fc24.x86_64
...

What I did to solve this I run:
rpmrebuild -ep Ghost-0.7.0-2.x86_64.rpm

and deleted all lines that had the directories that pop up at the previous errors. Save and exit. Rerun:
rpm -ivh --nodeps /root/rpmbuild/RPMS/x86_64/Ghost-0.7.0-2.x86_64.rpm

And voila!
The app starts successfully.
Problem is that when I type my url, the input area becomes  red saying:

We can't reach the given URL. Ensure that you are connected to the internet and that it is spelled correctly!

After that i opened Developer Tools and there was an error on the console:
vendor.js:13900 GET https://url net::ERR_INSECURE_RESPONSE

Ghost runs on a home server and I can access it from the browser. It is configured properly (I suppose) behind nginx as reverse proxy with self signed ssl certificates.
Thanks in advance :)

Hi! Thanks for taking a look into this. Unfortunately, we don't really have the bandwidth to support every variation of Linux, but would really appreciate the help in getting this fixed for those who do want this. If you've got time to poke around and submit a pull request with a fix, it would be very appreciated!

I ran into this issue too (the net::ERR_INSECURE_RESPONSE, not the 'repacking a .deb to a .rpm issue).
For me, it boiled down to the fact I use a self-signed certificate for my admin interface in Ghost.  The ghost-desktop app wants to verify the SSL certificates.
So the options for self-signers are to get your cert trusted on your system or run ghost-desktop with --ignore-certificate-errors (...or look into Let's Encrypt)
On the 'convert an rpm to deb' front, I've found that doing it without alien may be a better route.


Fix explore by area button mouseover

I'm seeing this in Safari but not Firefox"

For the life of me I can't figure this one out. Seems to be an issue only in Safari 11--works fine in Edge, Chrome, Firefox, and Safari 10. It has something to do with the icons. The bug goes away if any text is included in the button.

Ok, tried a hack fix for this. I still have no idea what was causing it. Works on Safari 11 on browserstack.


Requesting maven publish for recent changes

Please publish the latest changes from main into maven so that I can take advantage of the visitor transform.

https://repo1.maven.org/maven2/org/partiql/partiql-ir-generator/0.3.2/
https://repo1.maven.org/maven2/org/partiql/partiql-ir-generator-runtime/0.3.2/


Add grunt and version bump task

Moin,
when this project is installed via bower, one gets the old version 0.0.2. That is because the version number in bower.json is outdated. (See #50)
Here comes a fix to this along with a proposal to prevent such mismatches in the future.
With  grunt bump the versions in bower.json and package.json are updated together, commited, a git tag is created and everything will be pushed to origin.

Awesome, thanks!

Cool, thanks for the fast merge.
Would be awesome if you could release v0.2.1 (npm install && grunt bump) in order to get the version numbers in sync again.

done

perfect :) thanks.


[Feature] K line support no rise no fall no color scene

What problem does this feature solve?
Hope K line can support not rise and not fall when given white
What does the proposed API look like?
Hope K line can support not rise and not fall when given white

Is borderColorDoji what you want?

The color will change to gray when you drag.
https://user-images.githubusercontent.com/50894935/230524680-f56ada73-7332-43ce-ad4c-79f1537086d8.mp4


Attribute application@name at AndroidManifest.xml:10:9-42 requires a placeholder substitution but no value for  is provided.

I cannot run my app because of this error.

[✓] Flutter (Channel dev, 2.3.0-24.0.pre, on macOS 11.1 20C69 darwin-arm, locale en-PH)
• Flutter version 2.3.0-24.0.pre at /Applications/Development/flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision 2b9537c783 (8 months ago), 2021-06-11 22:11:53 -0700
• Engine revision ddbac024d1
• Dart version 2.14.0 (build 2.14.0-188.0.dev)
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
• Android SDK at /Users/aikbusmeon/Library/Android/sdk
• Platform android-31, build-tools 30.0.3
• Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7772763)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 12.5.1, Build version 12E507
• CocoaPods version 1.11.2
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 2021.1)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7772763)
[✓] VS Code (version 1.64.2)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.34.0
[✓] Connected device (2 available)
• sdk gphone64 arm64 (mobile) • emulator-5554 • android-arm64  • Android 12 (API 31) (emulator)
• Chrome (web)                • chrome        • web-javascript • Google Chrome 98.0.4758.80
• No issues found!
I'm using Macbook Pro M1 and the latest Android Studio Bumblebee.

Can you show your AndroidManifest.xml file ?

Removing the android:name=${applicationName} line works for me.

Hi @aikonics, please provide your AndroidManifest.xml.
Thank you

Hello guys, thanks for your response. I have fixed it by migrating my code to a new flutter project. I think my android files are outdated.  Sorry I have deleted the old version already so I cannot show you the AndroidManifest.xml. that you are asking for. Have a nice day and God bless!

Hi @RicardoKurashiki, kindly file a new issue a provide the relevant steps to reproduce.
Given that the original reporter has fixed his issue, I'm going to close this as solved.
Closing based on https://github.com/flutter/flutter/issues/98316#issuecomment-1040361210.
Thank you

Sorry for reviving this thread, but I see that @RicardoKurashiki did not open up a new issue for this and at the same time I have just encountered this.
If you'll get this kind of error, just change:
defaultConfig {
        // ...
        manifestPlaceholders = [
                'appAuthRedirectScheme': 'xxxxxx'
        ]
}

to
defaultConfig {
        // ...
        manifestPlaceholders['appAuthRedirectScheme'] = "'xxxxxx"
}

This will ensure that Flutter-generated manifest placeholders won't be overwritten:


perlPackages.MetaCPANClient: 2.028000 -> 2.029000

Motivation for this change
https://metacpan.org/source/MICKEY/MetaCPAN-Client-2.029000/Changes
Things done

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

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[X] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

perl530Packages.MetaCPANClient
perl532Packages.MetaCPANClient

@GrahamcOfBorg build perlPackages.MetaCPANClient

Result of nixpkgs-review pr 107279 run on x86_64-darwin 1

2 packages built:

perl530Packages.MetaCPANClient
perl532Packages.MetaCPANClient

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

2 packages built:

perl530Packages.MetaCPANClient
perl532Packages.MetaCPANClient