Exception when Response is Empty

I'm making a DELETE request to a server and when it success it returns empty string
but in Agera ,the response must not be an empty string because the responseBody will be null so it throws an exception here
  @NonNull
  public static <T> Result<T> success(@NonNull final T value) {
    return new Result<>(checkNotNull(value), null);
  }

is there any workaround for this case ?

Is this done using the net sample extension? If so, do you have a stack trace you can add here?
In general though, a success would always have to contain a non-null value. If you have a value that's null it'd have to use absent or if you have a value that sometimes is null, absentIfNull.  Result would support containing an empty string though (if it's a Result of a string that is).
If the sample extension is trying to put a null value in a success, we'll make sure to fix it, but  that stack trace would be helpful.

@mahmoud-elnaggar do you have any additional input on this?

@ernstsson
I'm very sorry for being late
this is my stack trace

java.lang.NullPointerException
at com.google.android.agera.Preconditions.checkNotNull(Preconditions.java:40)
at com.google.android.agera.Result.success(Result.java:71)
at me.drakeet.retrofit2.adapter.agera.CallSupplier.get(CallSupplier.java:47)
at me.drakeet.retrofit2.adapter.agera.CallSupplier.get(CallSupplier.java:32)
at com.google.android.agera.CompiledRepository.runGetFrom(CompiledRepository.java:340)
at com.google.android.agera.CompiledRepository.runFlowFrom(CompiledRepository.java:301)
at com.google.android.agera.CompiledRepository.run(CompiledRepository.java:540)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1113)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:588)
at java.lang.Thread.run(Thread.java:818)
03-06 18:52:53.156 3509-6069/? E/android.os.Debug: ro.product_ship = true
03-06 18:52:53.156 3509-6069/? E/android.os.Debug: ro.debug_level = 0x4f4c
03-06 18:52:53.156 3509-6069/? E/android.os.Debug: sys.mobilecare.preload = false
03-06 18:52:55.446 4399-4485/? E/ContactsProvider_EventLog: Flush buffer to file cnt : 15 size : 40Kb duration : 412ms lastUpdatedAfter : 152052ms

and this is how I initialize my Repository
`public <R> Repository<Result<R>> getRepository(Supplier<Result<R>> supplier) {
    return Repositories
            .repositoryWithInitialValue(Result.<R>absent())
            .observe()
            .onUpdatesPerLoop()
            .goTo(Executors.newSingleThreadExecutor())
            .attemptGetFrom(supplier)
            .orEnd(Result::failure)
            .thenTransform(Result::success)
            .compile();
}`

it is a generic method as you see to use it with different datatype and thats why I don't know how to use
absenceIfNull
and this is how I use it with retrofit 2
`private CareemAPI getRetrofitAPI(String baseUrl, Class<CareemAPI> uberAPIClass) throws Exception {
    if (baseUrl == null) throw new Exception();

    HttpLoggingInterceptor loggingInterceptor = new HttpLoggingInterceptor();
    loggingInterceptor.setLevel(HttpLoggingInterceptor.Level.BODY);
    OkHttpClient client = new OkHttpClient.Builder()
            .connectTimeout(60, TimeUnit.SECONDS)
            .addInterceptor(loggingInterceptor)
            .addInterceptor(chain -> {
                Request request = chain.request();
                request = request.newBuilder().addHeader("Authorization", Config.CAREEM_TOKEN).build();
                return chain.proceed(request);
            })
            .build();

    Retrofit retrofit = new Retrofit.Builder()
            .client(client)
            .addCallAdapterFactory(AgeraCallAdapterFactory.create())
            .addConverterFactory(GsonConverterFactory.create())
            .baseUrl(baseUrl)
            .build();
    return retrofit.create(uberAPIClass);
}`

the problem when I do a DELETE request which return empty response
thanks for your intention :)

Thanks! The issue is really in https://github.com/drakeet/retrofit-agera-call-adapter that calls Result.success with null here:
https://github.com/drakeet/retrofit-agera-call-adapter/blob/50b2ed6d1ecc89f86a7bcd7c66283df5058dcfe7/agera-call-adapter/src/main/java/me/drakeet/retrofit2/adapter/agera/CallSupplier.java#L47
And possibly also here:
https://github.com/drakeet/retrofit-agera-call-adapter/blob/50b2ed6d1ecc89f86a7bcd7c66283df5058dcfe7/agera-call-adapter/src/main/java/me/drakeet/retrofit2/adapter/agera/CallResponseSupplier.java#L47
I'll file a bug on retrofit-agera-call-adapter, @drakeet looping you in here for ref.


first look and tidy up

why closing the pull request? this might be a valuable contribution?

I was just having a look to see the basics of order matching - having read
the stackoverflow post. I never really meant to create a PR
On Fri, 20 Apr 2018, 17:37 Matthias Zimmermann<EMAIL_ADDRESS>wrote:

why closing the pull request? this might be a valuable contribution?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/matthiaszimmermann/order-matching-demo/pull/1#issuecomment-383153200,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB0E0IDC2LKomKWzxUIvKBF-IRu4FXjAks5tqg7igaJpZM4Tc8Ca
.


Iteroperation with SqlAlchemy broken in 0.9.2

import sqlalchemy as sa

sa.create_engine('vertica+vertica_python://test:test@localhost/db')

Result:
---------------------------------------------------------------------------
KeyError                                  Traceback (most recent call last)
/opt/conda/lib/python3.7/site-packages/pkg_resources/__init__.py in requires(self, extras)
   2731             try:
-> 2732                 deps.extend(dm[safe_extra(ext)])
   2733             except KeyError:

KeyError: 'namedparams'

During handling of the above exception, another exception occurred:

UnknownExtra                              Traceback (most recent call last)
<ipython-input-15-8fe01fb55c4d> in <module>
      1 import sqlalchemy as sa
      2
----> 3 sa.create_engine('vertica+vertica_python://test:test@localhost/db')

/opt/conda/lib/python3.7/site-packages/sqlalchemy/engine/__init__.py in create_engine(*args, **kwargs)
    421     strategy = kwargs.pop("strategy", default_strategy)
    422     strategy = strategies.strategies[strategy]
--> 423     return strategy.create(*args, **kwargs)
    424
    425

/opt/conda/lib/python3.7/site-packages/sqlalchemy/engine/strategies.py in create(self, name_or_url, **kwargs)
     59         kwargs.pop("plugins", None)
     60
---> 61         entrypoint = u._get_entrypoint()
     62         dialect_cls = entrypoint.get_dialect_cls(u)
     63

/opt/conda/lib/python3.7/site-packages/sqlalchemy/engine/url.py in _get_entrypoint(self)
    170         else:
    171             name = self.drivername.replace("+", ".")
--> 172         cls = registry.load(name)
    173         # check for legacy dialects that
    174         # would return a module with 'dialect' as the

/opt/conda/lib/python3.7/site-packages/sqlalchemy/util/langhelpers.py in load(self, name)
    220     def load(self, name):
    221         if name in self.impls:
--> 222             return self.impls[name]()
    223
    224         if self.auto_fn:

/opt/conda/lib/python3.7/site-packages/pkg_resources/__init__.py in load(self, require, *args, **kwargs)
   2431             )
   2432         if require:
-> 2433             self.require(*args, **kwargs)
   2434         return self.resolve()
   2435

/opt/conda/lib/python3.7/site-packages/pkg_resources/__init__.py in require(self, env, installer)
   2454         # requirements for that extra are purely optional and skip over them.
   2455         reqs = self.dist.requires(self.extras)
-> 2456         items = working_set.resolve(reqs, env, installer, extras=self.extras)
   2457         list(map(working_set.add, items))
   2458

/opt/conda/lib/python3.7/site-packages/pkg_resources/__init__.py in resolve(self, requirements, env, installer, replace_conflicting, extras)
    792
    793             # push the new requirements onto the stack
--> 794             new_requirements = dist.requires(req.extras)[::-1]
    795             requirements.extend(new_requirements)
    796

/opt/conda/lib/python3.7/site-packages/pkg_resources/__init__.py in requires(self, extras)
   2733             except KeyError:
   2734                 raise UnknownExtra(
-> 2735                     "%s has no such extra feature %r" % (self, ext)
   2736                 )
   2737         return deps

UnknownExtra: vertica-python 0.9.2 has no such extra feature 'namedparams'

Looks like the change was made in #276
I'm using LocusEnergy/sqlalchemy-vertica-python for connecting from Jupyter and it has [namedparams] in the requirements.

Please report this issue to LocusEnergy/sqlalchemy-vertica-python.

Hi @sitingren - sqlalchemy-vertica-python doesn't have any support right now. One of their former contributors mentioned that they asked if vertica could take over support here, but that there wasn't any bandwidth for that (see convo at https://github.com/LocusEnergy/sqlalchemy-vertica-python/issues/35). We use both vertica_python and the sqlalchemy-vertica-python adapter at my company, but given the lack of updates on the latter, I see the following options moving forward:

vertica takes over the sqlalchemy-vertica-python repo, either by (a) transferring ownership (like you did for this repo) or (b) forking it. This will require release management on your part, but I think users of this library can contribute most of the improvements (and you would mostly be reviewing/approving and uploading to PyPI).
One of the downstream users of the library either (a) transfers ownership or  (b) forks the library and creates a new PyPI entry.

Less interested in (2) because it may fall into the same issue - all the maintainers leave the company and it falls behind on updates. Granted, I may end up doing this for our company, anyway, but it'd be a shame if we end up with a bunch of disparate copies of sqlalchemy-vertica-python (I think there are 2-3 already) because we don't have a central "official" library (ideally, under the Vertica org on GH).

According to conversations at
https://github.com/LocusEnergy/sqlalchemy-vertica-python/pull/37#issuecomment-589158302,
https://github.com/bluelabsio/sqlalchemy-vertica-python is the new fork and continues to publish to https://pypi.org/project/sqlalchemy-vertica-python/.
@lemeryfertitta @vinceatbluelabs We really appreciate your great efforts to make this project continue!


Remove peerDependency "jest-plugins" from jest-plugin-set

Description
Hi! Thanks a lot for these plugins, I'm using the set plugin and it is really awesome so far!
I wanted to ask you whether you could remove the peerDependency to "jest-plugins" for the jest-plugins-set package. It appears it works without it or not? I'd like to only use that plugin for the time being.
P.S.: I created an eslint-rule which no longer marks variables declared via set as undefined:
https://github.com/peterjuras/eslint-plugin-jest-set

Hi @peterjuras!
Thanks for the suggestion, you're absolutely right. I've submitted a PR and will deploy a new version in a few min.
Also, this eslint rule is great! I've been wanting to do that myself for a while, glad someone took the initiative and built it :)

Thank you! That was very quick =)


request to add espic university

request to add espic university

@NazifToure01 First stage of verification passed, official domain is espic.com as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@NazifToure01 Please provide us with an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course. Thank you!

@NazifToure01 You might miss my previous comment. Please react.

@NazifToure01 I am sorry, this request remains unanswered for a while. I have to close it without merging. Please reopen and comment if you have an update regarding the aforementioned issue. Thank you for your understanding!


Kdump.ps1 issue, After kdump was triggered, wait for reboot and check vmcore, but Kdump_Result.sh wasn't pushed

Kdump.ps1

For Kdump_Config.sh, Kdump_Execute.sh, Kdump_Results.sh, put them into Kdump_Tests.xml , they can be pushed with utils.sh together by framework. Will reduce rate of failed pushing in Kdump.ps1 script.

Thank you for the suggestion @HiRoySoft
I'm testing the changes now and submit them once validated.
Have you seen failures in the scripts being pushed?
From our experience, as long as the VM is configured properly for lisa (keys for eg) and it starts after the kdump crash, it works as designed.
Other VM failures should be handled by LISA itself.

Yes. Have seen these failures several times in stress test.
As different distributions have the different booting time(I am focused on RHEL6 and RHEL7, even RHEL7 booting speed is faster than RHEL6).
Also time SSHD service prepared in every booting is different too, LISA hasn't try-again policy when push failed.
Anyway, put them into Kdump_Test.xml to push. that is will be better before test kdump.
Thanks.


Update ShipManifest-ShipManifest.<IP_ADDRESS>.3.ckan

Correct version to allow proper detection.

If this is generated via netkan then the upstream (in this case kerbalstuff) has the wrong version entered.
It is much better to contact the mod author and have him/her change the version on KS.

This is generated via NetKan.  I have since removed the download from KerbalStuff.   I’m hoping that action combined with removing that version from the meta will resolve the issue.
I have placed a notice on KerbalStuff and on my KSP Forum thread
Thanks!
From: Hakan Tandogan
Sent: Sunday, January 18, 2015 6:16 PM
To: KSP-CKAN/CKAN-meta
Cc: Joseph Korinek
Subject: Re: [CKAN-meta] Update ShipManifest-ShipManifest.<IP_ADDRESS>.3.ckan (#267)
If this is generated via netkan then the upstream (in this case kerbalstuff) has the wrong version entered.
It is much better to contact the mod author and have him/her change the version on KS.
—
Reply to this email directly or view it on GitHub.

This PR solved the wrong problem. I'll close it and remove the offending file from CKAN-meta (by the way, another case for the "deprecated" tag we should be considering...)


Add volume to postgres container to persist refreshed data

Currently, the data loaded by ./refresh-data.sh doesn't persist across container shutdown (docker-compose down). This adds a volume to fix that. Now data will persist across both container shutdown and docker shutdown.
Data is updated as normal, running ./refresh-data.sh again will freshen the data and the new data will be persisted.
If for some reason this data needs to be wiped directly, you can just docker-compose down && docker volume remove postgres_data and an empty volume will be newly created on your next docker-compose up, but I doubt this will be a common workflow.

I have no problems with this PR, but I don’t think I’ve encountered this as a problem — my changes to the database (with refresh-data.sh and changes made by hand) seem to persist?


chore(release): 0.21.0

This release includes #24
@dsherret @bartlomieju sorry to bother you, can you release a new version by merging it and then maybe tagging?

Done.

Thanks! ❤


Eliminate PCRE recursion crashes

Set PCRE recursion limit to match stack size.
Increase stack size on Mac (default is 512k) and Windows (default is 1MB) to 8MB (as on Android).

Seems that std::thread doesn't support stack setting. May be replace it with some other code?

/core/pull-requests/872


Full accounting of double-photoelectron emission

Currently flamedisx accounts for double-PE emission approximately, by increasing the mean and width of the Gaussian S1 resolution. By the central limit theorem, this reaches any desired level of accuracy above some number of photons -- in practice it's reasonable for more than ~5.
... but of course, DM searches use < 5 PE signals, and it wouldn't be particularly hard to add an extra binomial step in flamedisx to account for DPE emission as best we can.

On the shorter term, we might want to check if our approximate formula is correct for non-unity mean PE gains: it's easy to miss this factor in one of the formulas since it is usually 1.


Introduce jvm.thread.daemon and jvm.thread.state attributes

Fixes #101
Changes
This PR does two things:

Removes the general thread.daemon attribute and introduces jvm.thread.daemon instead
Introduces jvm.thread.state attribute and adds it to the jvm.thread.count metric

It's definitely debatable whether these two attributes should live in the jvm.thread.* or thread.* namespace.
My primary reason for proposing that they live under jvm.thread.* is that I couldn't find anything to support these being general cross-language concepts, and JVM metric stability will depend on any attributes it uses also being marked stable.
I think we could view this like garbage collection, which is something that some other languages have, but we are not prepared to try to unify under a single set of attribute definitions prior to initial stability.
Note that while ECS defines thread.name and thread.id, they do not have anything like thread.daemon or thread.state.
Merge requirement checklist

[x] CONTRIBUTING.md guidelines followed.
[x] CHANGELOG.md updated for non-trivial changes.
[x] schema-next.yaml updated with changes to existing conventions.

In other hand, the "isDaemon" concept can be extracted from Python and .NET, so maybe we should consider actually keeping it in the existing thread namespace? In other words, what is the drawback of keeping daemon where it is today vs adding in the jvm namespace?

Per @joaopgrassi's comments on DAEMON and python/.NET, I'm not sure we have the same meaning between them all. Would be good to do a quick comparison, but I do think isolating Java's threading model from others would be healthy at this point.

Java "daemon threads" == .NET "background threads"
I couldn't find any other languages that have a similar concept (other than Java, Python and .NET)
I'm personally in favor of keeping jvm.thread.daemon langauge specific, and if .NET wants to introduce similar they would probably name it .background


pv scan createScan command should be implemented

It is more convenient to have a command pv scan createScan to copy scans between Purview accounts.

Agree, there's not much meaning for me to create datasource via CLI, if I can't create scans.
Seem like an obvious miss.

Here is the related endpoint definition https://docs.microsoft.com/en-us/rest/api/purview/scanningdataplane/scans/createorupdate

Thanks @nakazax  and @th0ger. Just updated purviewcli (0.1.15) to align with the recent documentation on the Scanning Data Plane. Please see updated scan notebook which includes a section on Scans.
Example command to create or update a scan
pv scan putScan --dataSourceName "DataSource2" --scanName "myScan" --payload-file "C:\Users\tarifat\Downloads\purviewcli\samples\json\scan.json"

Sorry for delayed comments in closed issue, but I've confirmed that pv scan putScan and some other commands work correctly with the purviewcli 0.1.17. And thanks to your fix, I've been able to create the following migration helper scripts for Purview. Thank you so much 😊
https://github.com/nakazax/purview-migration-helper


Incorrect crawl-delay value when robots content does not contain crawl-delay

The SimpleRobotRulesParser implementation produces an invalid crawl delay value (-9223372036854775808) when crawl-delay is not set.
SimpleRobotRulesParser parser = new SimpleRobotRulesParser();
BaseRobotRules rules = parser.parseContent("http://domain.com", "".getBytes(), "text/plain", "Mozilla/5.0");
System.out.println("Crawl Delay: " + rules.getCrawlDelay());

Result: -9223372036854775808
Expected: -1

The value being returned is UNSET_CRAWL_DELAY from BaseRobotRules, which is Long.MIN_VALUE. Thus I don't think this is a bug.
If there's documentation somewhere that refers to -1, we should fix that.

@guadenine do you have any source for the value being -1?

No, I dont have the source, im now aware that it returns Long.MIN_VALUE, but I dont see the reasoning for it. The naturally expected return value of a missing crawl-delay is -1.


[Q] Windows IP Multicast, no output? Won't terminate?

Hi, I am attempting to use your program to troubleshoot some IPTV issues.
I am running this on windows, and for example when I the following command, I never seem to get any output in the console window, and also when I attempt to crtl-c, TSP never seems to actually terminate no matter how long I wait, I have ukill it via task manager.
I can confirm the multicast group is joined via IGMP and data is coming in via wireshark capture on the interface.
I sorry if I am missing something obvious.
C:\util\TSDuck\bin>tsp --debug -I ip -l <IP_ADDRESS> <IP_ADDRESS>:50000 -P until --seconds 10 -P analyze --interval 1 --service-analysis -O drop
tsp: Debug: debug level set to 1
tsp: Debug: ====> tsp --debug
* Debug: debug level set to 1
* Debug: ip: debug level set to 1
* Debug: ip: ====> tsp -I ip -l <IP_ADDRESS> <IP_ADDRESS>:50000
* Debug: registering processor plugin "until", status: ok
* Debug: drop: debug level set to 1
* Debug: drop: ====> tsp -O drop
* Debug: registering processor plugin "analyze", status: ok
* Debug: trying to load "C:\util\TSDuck\bin\tsplugin_until.dll"
* Debug: until: debug level set to 1
* Debug: until: ====> tsp -P until --seconds 10
* Debug: trying to load "C:\util\TSDuck\bin\tsplugin_analyze.dll"
* Debug: analyze: debug level set to 1
* Debug: analyze: ====> tsp -P analyze --interval 1 --service-analysis
* Debug: tsp: buffer size: 89,240 TS packets, 16,777,120 bytes
* Debug: ip: setting socket reuse address to 1
* Debug: ip: binding socket to <IP_ADDRESS>:50000
* ip: joining multicast group <IP_ADDRESS>:50000 from local address <IP_ADDRESS>
* tsp: user interrupt, terminating...

Hi @Napsterbater
I tried the very same command (local interface address excepted of course) on a Windows 10 system and it works. The IP stream was sent from a Linux VM.
So, there must be something on your Windows system that prevents the application from receiving the packets.
Try --debug=2. At this level, all incoming UDP packets are traced like this:
* Debug[2]: ip: received UDP packet, source: <IP_ADDRESS>:42200, destination: <IP_ADDRESS>:50000, timestamp: -1

But, because everything in tsp is triggered on TS packet processing, I suspect that you don't receive any UDP packet for two reasons:

The plugin until should trigger a termination on the reception of a TS packet exactly 10 seconds after receiving the first one. If there is no termination, there is probably no packet flowing through the chain of plugins.
The Ctrl-C interrupt is caught by tsp and synchronously processed on the next TS packet processing to trigger a graceful termination. Again, if it does not terminate, there is probably no packet. In this case, a second Ctrl-C abruptly terminates the process (regular system interrupt). This works on Linux and macOS but not on Windows for some usual mysterious Windowish reason. But this specific Windows issue is of course unrelated to your UDP reception.

Looks like you were right on both accounts.
Windows firewall, tsp was allowed for the Domain profile,, but not for Public and Private, which the 2nd NIC that I am using to the data with was attached.
Sorry for the noise on a simple problem, But thanks for the quick response.

Forgot to close this.

Forgot to close this.


Support saving and loading workflows from video metadata

Like with pngs, the workflow used to generate a video can now be restored by dragging and dropping it into the webui, or by clicking the load button.
The method used for saving the metadata has a maximum length of ~32 KB on Windows and ~137 KB on Linux. I don't think hitting this would happen under normal working conditions, but embedding will be skipped with a warning if it ever occurs.
Because of this limit, a lack of support for animated pictures like gif/webp, and safety in case I've missed an edge case, the metadata is still saved in a png of the first frame.
As with loading metadata from a png, the video files are processed in javascript on the client side, and a video that lacks or has invalid metadata fails silently.

Nice! Do you know if there are any unique length limits on macs?
I am finishing up a huge refactor in my code that will make the loaders and unloaders behave predictably and avoid running when unnecessary. I should have my changes in dev in the next 6ish hours, and then we can try to merge this in.

I'm not finding any good sources on what the limit for macs would be. My best guess is it's the same as the limit on Linux, but the current code limits it to the same value of windows.
I've since taken the time to forcefully test the exact limit and found that I was off by 1 on Linux. A fix for this and the commented form of the quick testing code has been comitted and I'll try and quickly test the limit on windows.
The other alternative is to instead read metadata from a temporary file, but I try real hard to avoid those where possible.

Windows results were not great. In addition to the lower limits, those limits apply to all args. I'll want to do some more thorough testing tomorrow that includes some stock workflows to see if it's reasonably possible to hit the limit without copy+pasting dozens of text prompts and if so, I'll look into using a temporary metadata file as a fallback

Gotcha, sounds good. My changes for the refactor + fixes have been pushed to develop. They should not mess with any of the code you're working on, but just wanted to give you a heads up in case you want to test it with that. I still need to make the beta_schedule on the AnimateDiff Loaders actually apply, and then I'm good to go

I think things are in a pretty good spot now. The file fallback should automatically apply and I've been able to load restore some metadata of extreme length.
It doesn't seem like there's an easy way to drag and drop a video from a webpage, but I'll keep digging
AnimateDiff_00012_.webm

Sweet, I'll take a closer look and test it on my machine when I wake up.

Slightly unrelated to this PR, but could you look into having the option to toggle if the produced videos in the combined node are displayed? Peeps with a lot of the combine nodes are having their browsers lag with all the displayed videos at once.
Also, I think there might be a potential issue with "save image" option on the combine nodes when doing non-gof option, the if the setting is off, it doesn't save the video at all.

Adding some sort of toggle to disable video previews should be pretty easy, but from my quick testing, clicking the minimize dot on the top left of the node seems to work properly already and should be the preferred solution to prevent option bloat. I'll try and hunt down the issue in after this to see if there's more I'm missing.
I'm not having success reproducing the error you describe with "save image" , but my gut is that there's some major issue causing the ffmpeg command to fail. Are you getting any relevant console output? I should have the logging properly filtered to display any error message serious enough to prevent output.

I agree about avoiding option bloat, but imo for the video saving nodes, it'll end up being an important option. Having to minimize only certain nodes to avoid your chrome from crashing, then frantically unminimizing and minimizing again to change some setting, but now knowing which node is the mp4 node vs gig node, etc., makes for a bad experience.
I'll post some more info about the save issue soon, maybe it was just something on my end, but i'll describe how to replicate it if I can.
And, I think it would be a good idea for you and I to make a separate repo for all video-helper nodes, so that it can be used universally for any video-related workflow. We'll have nodes that do the video saving and previews, but also latent splitting/recombining, image splitting/recombining, loading images from gifs/videos, having all those nodes output (and take input) with paths of where they should save/load from, etc. Just all stuff related to video workflows, but only related to video workflows. I can make you a maintainer of it too, so both you and I can push to and maintain the repo without waiting on anyone to do PRs. If you're down, I think I'll call it ComfyUI-VideoHelperSuite (VHS for short lol). And then we can reference that repo in the sample workflows here.

That sounds reasonable. It better consolidates issues and would minimize the amount of extra work I'm creating for you.
I've done some testing and I believe I've figured out the required git voodoo to keep the commit history of affected files when the migration is done.

I've created the repo, ComfyUI-VideoHelperSuite, and invited you to be collaborator. I'm going to be afk for the next few hours, so I won't have a chance to put in the scaffolding for a bit.


Unable to build TensorFlow targeting Raspberry Pi

Please make sure that this is a build/installation issue. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:build_template
System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): MacOS 10.15.7
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: N/A
TensorFlow installed from (source or binary): Source
TensorFlow version: v2.3.0
Python version: N/A
Installed using virtualenv? pip? conda?: N/A
Bazel version (if compiling from source): N/A
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version: N/A
GPU model and memory: N/A

Describe the problem
I am following the instructions on https://www.tensorflow.org/install/source_rpi to build a .whl package that I can install on a Raspberry Pi 3B, but the build process is failing. I can clone TensorFlow, check out a revision, and begin the build in the Docker container without issue, but the build fails with the following output at the end:
Removing intermediate container 8f1ac04cb8a3
 ---> 9c723ba9afae
Step 14/14 : COPY install/.bazelrc /etc/bazel.bazelrc
 ---> 8dba22ce896f
Successfully built 8dba22ce896f
Successfully tagged tf_ci.pi-python37:latest
Running 'tensorflow/tools/ci_build/pi/build_raspberry_pi.sh' inside tf_ci.pi-python37...
Reading package lists...
Building dependency tree...
Reading state information...
sudo is already the newest version (1.8.16-0ubuntu1.9).
0 upgraded, 0 newly installed, 0 to remove and 4 not upgraded.
addgroup: Please enter a username matching the regular expression configured
via the NAME_REGEX[_SYSTEM] configuration variable.  Use the `--force-badname'
option to relax this check or reconfigure NAME_REGEX.

If I instead try to build v2.3.1, I get the following output:
Step 15/15 : ENV TF_ENABLE_XLA=0
 ---> Using cache
 ---> e7647e308801
Successfully built e7647e308801
Successfully tagged tf_ci.pi-python37:latest
Running 'tensorflow/tools/ci_build/pi/build_raspberry_pi.sh' inside tf_ci.pi-python37...
Reading package lists...
Building dependency tree...
Reading state information...
sudo is already the newest version (1.8.16-0ubuntu1.9).
0 upgraded, 0 newly installed, 0 to remove and 4 not upgraded.
addgroup: To avoid problems, the username should consist only of
letters, digits, underscores, periods, at signs and dashes, and not start with
a dash (as defined by IEEE Std 1003.1-2001). For compatibility with Samba
machine accounts $ is also supported at the end of the username

This happens well before the 30 minutes the documentation suggests the process should take, and there is no .whl in the tensorflow/output-artifacts directory.
I am experiencing this issue when trying to compile v2.3.0, but I have encountered when trying to build v1.15.0 as well. I would expect the use of containers and Bazel to make the build reproducible, but it would seem that something has changed that broke the build process. Please let me know if there are any additional logs I could provide to help diagnose this.
Provide the exact sequence of commands / steps that you executed before running into the problem

Note: you must have docker installed for the following steps

$ git clone https://github.com/tensorflow/tensorflow.git
$ cd tensorflow
$ git checkout v2.3.0
$ tensorflow/tools/ci_build/ci_build.sh PI-PYTHON37 \
    tensorflow/tools/ci_build/pi/build_raspberry_pi.sh

At this point the build will fail, with no output .whl generated.

This seems to be related to the with_the_same_user script and the environment variable CI_BUILD_USER_FORCE_BADNAME, but these don't appear to be well documented, and it's not clear to an end user that a workaround might exist.

Hi @TylerADavis ! Could you try with instructions in this repo? Thanks!

I don't have an rpi to test on at the moment, but it does look like that repo would allow for one to get a prebuilt wheel of full (non-lite) tensorflow for an aarch64 linux system.
https://github.com/PINTO0309/Tensorflow-bin#usage

Ok @TylerADavis ! Is this issue good to close then?

Sounds good to me. Thank you for referring me to that repo @mohantym !


Fix toggle font

Undo the change for <ToggleBadge> to let it change text font-weight on selection. Now it will be regular always.

Codecov Report

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

@@          Coverage Diff          @@
##           master   #336   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          41     41
  Lines         475    475
=====================================
  Hits          475    475

Impacted Files
Coverage Δ

src/ToggleBadge.js
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Unicorn Is Displayed on exceljs GitHub Main Project Page

🐛 Bug Report
There seems to be something wrong with:
https://github.com/exceljs/exceljs
When I try to navigate to this page, I get the dreaded Unicorn (too long to load):

I have tried multiple browsers and VPN and I get the same issue.  It is possible to navigate to the issues, however, as evidenced by this bug report.  This has been an issue over several days so it is not transitory.
More information in Chrome DevTools Console:

Looks like it's somehow solved.


refactor: move connection api to Spanner client lib

EXPERIMENTAL
Move the generic Connection API to the Spanner client library to make it re-usable. This refactor does the following:

Move all non-JDBC classes, tests and resources from the JDBC driver to the Java client library. All files are placed in the com.google.cloud.spanner.connection package.
Make the following classes and interfaces public in order to make them usable for the JDBC driver and other users of the package:

Interface: com.google.cloud.spanner.connection.Connection
Interface: com.google.cloud.spanner.connection.TransactionRetryListener
Interface: com.google.cloud.spanner.connection.StatementResult
Class: com.google.cloud.spanner.connection.ConnectionOptions
Class: com.google.cloud.spanner.connection.ReadOnlyStalenessUtil
Class: com.google.cloud.spanner.connection.StatementParser
Enum: com.google.cloud.spanner.connection.TransactionRetryListener.RetryResult

@olavloite Can you please request @elefeint for the review as well?
She's leading the Spanner R2DBC effort.

@olavloite Can you please request @elefeint for the review as well?
She's leading the Spanner R2DBC effort.

I'd love to, but apparently she is not a member of this repository, so I can't request her review. I would certainly appreciate her input, though.

I just want to say that I'm comfortable with this overall approach, and excited for the connection api to be made available for other libraries besides JDBC.
I would love for there to be something explicit about the level of support (and API non-breaking-ness) we are promising for this API. i.e. is every method exposed in the public interface going to be supported until the next major version? Do we want to claim less-than-that to start out with? It just seems important that we don't accidentally make this a fully supported/unchanging API without being clear.

@olavloite Can you explain the decision for not moving the connection API to a separate Maven module in the repo?

@meltsufin The honest answer is that I didn't think of that possibility as well :-). I automatically interpreted the suggestion to 'add it to the client library repository' as 'add it to the client library module'.
That being said, I would say that the following argument favors putting them together:

Ease of versioning: The connection API only relies on the public API of the client library, so it could just have a dependency on the client library module. The two are however in real life somewhat more interconnected, and there have been changes in the past where a version of the connection API (or actually; the JDBC driver) depended on a specific minimum version of the client library. Users who would exclude the transitive client library dependency brought in by the connection API and import their own version of the client library would in those cases run into subtle bugs. (Of course one could argue that anyone who excludes a transitive dependency is responsible for knowing what they're doing.)

And the following would argue against putting them together:

It adds build time to the client library.
Anyone who imports the client library automatically also imports the connection API. This might confuse users as to which API to use.

I personally don't really feel very strongly either way, and I'm certainly open to additional arguments and views on this.

Regarding versioning, users should really be using the Spanner BOM the ensure compatibility between the Connection API and the Client Lib.
In addition to the benefits of separating into a separate module that you already outlined, another one is that it will ensure that the Client Library does not accidentally rely on something in the Connection API.
I guess the decision ultimately depends on how decoupled you want the Connection API to be from the client library.

Thank you! Looks good from my side.
To Cliff's point, perhaps we could have a README in the connection package explaining that this is internal only, "so please do not use this API by itself", and then relax this constraint over time if necessary?
I am out on Monday, so I'll get back to this on Tuesday.

Could we also add the @InternalApi annotation to make it explicit in the code?

Just my 2 cents on the two modules vs one module discussion. I feel that the connections API is strongly tied to the client library implementation and I can't think of any use cases where someone might want to use a different version of the connections API to that of the client library. So in terms the ease of us managing the modules/version and for users to not get into a confusing versioning situation, I'm in favor of having both being in the same module.

I haven't reviewed this line by line since this is PR is just moving code. @olavloite please feel free to merge.


Possible to redact partially?

Is it possible to redact partially, like the way credit card numbers or SSN number are done?
E.g.
{ secret: "1234567890" }

// outputs

{ secret: "xxxxxx7890" }

This will be very helpful to identify potential issues without delving the entire secret.
I think the easiest and most flexible way would be to accept a function? Something like this:
censor = (value) => {
// apply mask
return maskedValue
}

I understand this will cause a dip in performance which should be acceptable since this will be opt-in. And if there's a way to do it without sacrificing performance, even better!
Thoughts?

yes absolutely – I like it, we just need to ensure that there's no incidental performance hit on full redaction (e.g. ensure having to do a type of existence check doesn't reduce speed)

if you'd like to submit a PR (where censor can be a function) then it would be most welcome


changing the charge rate

It would be very nice (I didn't see that anywhere in other app, or mobile app) to be able to change Charge speed (changing the Charge Current) to better plan for charging end-time, etc...

This would be nice but I don't think this is available through the API:
https://tesla-api.timdorr.com/vehicle/commands/charging


fix!: ops::Module now empty struct rather than unit struct

Fixes #1270
A simpler test json might be better but kept the original just to show it is fixed.
BREAKING CHANGE: ops::Module no longer unit struct, must be constructed with new() or default()

Converting to draft in case there is some alternative fix for: https://github.com/serde-rs/serde/issues/2775


Update ens registry contract address

The new contract is located at the same address on all networks.
Fixes #237

@angerborn  closing in favor of #243
thanks for your PR anyway. Much apreciated!


Handle bar still moving even when dealer is disabled.

Hi, I had a situation where the I wanted the dealer to be activated only when the handle was held. What happened was that the transform styles were still being applied to the handle even when the dealer was disabled.

Okay so I fixed it by adding this
if(this.disabled)       return;
on line 682 on the renderHandlePosition method.


Fix recreated-database bug and issue with -D General

This should fix Issue #4 (both pieces). It may be inelegant, but it solves the issues I had.

What is the status of the PR? It is quite old.

As far as I know, it hasn't been merged.

Of course it isn't merged because I can see that here on GitHub. ;)
The question is if someone is working on it? Or is there a reason why isn't it merged?

Heh, as the one submitting the patch, I can only give a "the pull-request I submitted solved the issues I was having", so you're welcome to go rogue like me and run a patched version. I haven't heard anything privately from @iamaziz beyond the silence in this issue.  And that said, while TermFeed is nice I was mostly tinkering with it and have since settled on rss2email which just puts them in my inbox (affording me all the power of my mail-filters for processing RSS).


Track balance for specific addresses

Related to ecadlabs/go-tezos#3
This metric will consume the go-tezos get balance method
Tracking balance of a wallet overtime is useful for alerting and visualization. A timeseries of balance is a facsimile, and shouldn't be used for accounting. The value of exposing address specific values comes from using prometheus tooling for tracking and alerting.
The exporter would need to have a config file where the operator can list addresses of interest.
curl localhost:8732/chains/{chain_id}/blocks/{block_id}/context/delegates/{pkh}/balance
chain_id is a symbolic chain identifier, e.g. main or test, or their equivalent Base58-encoded hash, e. g. NetXzGDuYoAawjv.
block_id is a symbolic block identifier, e.g. head of genesis, or any Base58-encoded block hash, e.g.
Concrete example against mainnet
curl localhost:8732/chains/main/blocks/head/context/delegates/tz1Yju7jmmsaUiG9qQLoYv35v5pHgnWoLWbt/balance

@jevonearth Do you want this functionality only for delegate balances or also for contract balances ([...]/context/contracts/<contract_id>/balance)?

Closing. We will stick to operational node metrics, and avoid on-chain metrics.


dem2tin failure w/o actionable error msg

I downloaded tin-terrain today and built the Docker image on my Mac.
When I run dem2tin, I see the following failure:
root@f481e479e8a9:/home# tin-terrain dem2tin --input /data/LMI_Haedarlikan_DEM_16bit/LMI_Haedarlikan_DEM_16bit-Mercator.tif --output /data/LMI_Haedarlikan_DEM_16bit/test.obj
subcommand_dem2tin
Opening raster file /data/LMI_Haedarlikan_DEM_16bit/LMI_Haedarlikan_DEM_16bit-Mercator.tif with GDAL...
reading raster data...
Killed
root@f481e479e8a9:/home#
My dataset:
root@f481e479e8a9:/home# gdalinfo /data/LMI_Haedarlikan_DEM_16bit/LMI_Haedarlikan_DEM_16bit-Mercator.tif
Driver: GTiff/GeoTIFF
Files: /data/LMI_Haedarlikan_DEM_16bit/LMI_Haedarlikan_DEM_16bit-Mercator.tif
Size is 56541, 39914
Coordinate System is:
PROJCS["WGS 84 / Pseudo-Mercator",
GEOGCS["WGS 84",
DATUM["WGS_1984",
SPHEROID["WGS 84",6378137,298.257223563,
AUTHORITY["EPSG","7030"]],
AUTHORITY["EPSG","6326"]],
PRIMEM["Greenwich",0,
AUTHORITY["EPSG","8901"]],
UNIT["degree",0.0174532925199433,
AUTHORITY["EPSG","9122"]],
AUTHORITY["EPSG","4326"]],
PROJECTION["Mercator_1SP"],
PARAMETER["central_meridian",0],
PARAMETER["scale_factor",1],
PARAMETER["false_easting",0],
PARAMETER["false_northing",0],
UNIT["metre",1,
AUTHORITY["EPSG","9001"]],
AXIS["X",EAST],
AXIS["Y",NORTH],
EXTENSION["PROJ4","+proj=merc +a=6378137 +b=6378137 +lat_ts=0.0 +lon_0=0.0 +x_0=0.0 +y_0=0 +k=1.0 +units=m +nadgrids=@null +wktext +no_defs"],
AUTHORITY["EPSG","3857"]]
Origin = (-2764161.621573575772345,10052026.112180622294545)
Pixel Size = (23.488512876275621,-23.488512876275621)
Metadata:
AREA_OR_POINT=Area
DataType=Generic
Image Structure Metadata:
INTERLEAVE=BAND
Corner Coordinates:
Upper Left  (-2764161.622,10052026.112) ( 24d49'51.19"W, 66d37'55.54"N)
Lower Left  (-2764161.622, 9114505.609) ( 24d49'51.19"W, 63d 3'29.08"N)
Upper Right (-1436097.615,10052026.112) ( 12d54' 2.46"W, 66d37'55.54"N)
Lower Right (-1436097.615, 9114505.609) ( 12d54' 2.46"W, 63d 3'29.08"N)
Center      (-2100129.618, 9583265.861) ( 18d51'56.83"W, 64d54'16.26"N)
Band 1 Block=56541x1 Type=Int16, ColorInterp=Gray
NoData Value=-10000
Metadata:
BandName=Band_1
SourceBandIndex=0
root@f481e479e8a9:/home#
Since I downloaded rather than cloned, not much here. But I did this today:
root@f481e479e8a9:/home# tin-terrain version
name: tin-terrain
copyright_notice: Copyright (C) 2018 HERE Europe B.V.
git_description:
git_hash:
build_timestamp: 20181130T2116Z
root@f481e479e8a9:/home#
I've tried the verbose switch and the log option, but there is no additional information to help me decide what to try next. Any suggestions would be much appreciated.

Hi @moosehill,
thank you for reaching out and trying tin-terrain. I would like to understand more about the reason for tin-terrain to be killed by the OS.
The dataset you are using, is it this one I found on this Icelandic page? https://vefthjonustur.wordpress.com/thrividd/ ?
Second, how much memory does your machine have?
I suspect that tin-terrain runs out of memory, but I want to test this first and eventually rule that possibility out, before we start digging deeper.
Thanks for providing this information.

I was able to reproduce the issue and it is lies in how Docker runs on Mac. The VM that contains Docker (docker-machine) only has 2GB memory in its default mode. Therefore tin-terrain runs out of memory quite fast.
You can increase the memory by following these instructions here: https://docs.docker.com/docker-for-mac/#advanced
Or you try compiling tin-terrain natively on the Mac without Docker. If you need help with that, let me know.

@adrpar Thanks for the additional information -- I've increased Docker memory to 24GB and it still fails, but it is getting much further than previously. I will continue to play with Docker settings to see if I can get it to complete. (Also, I need new glasses because I thought the dataset was 5000x4000 when it is actually 100X larger than that. Now I understand why you immediately suspected memory was the problem.)


fix conflict

What problem does this PR solve?
What is changed and how does it work?
fix conflict
Check List
Tests

Unit test
E2E test
Stability test
Manual test (add detailed scripts or steps below)
No code

Code changes

Has Go code change
Has CI related scripts change
Has Terraform scripts change

Side effects

Breaking backward compatibility

Related changes

Need to cherry-pick to the release branch
Need to update the documentation

Does this PR introduce a user-facing change?:

NONE

Codecov Report

:exclamation: No coverage uploaded for pull request base (release-1.1-997286b692bd@121975d). Click here to learn what that means.
The diff coverage is n/a.

@@                     Coverage Diff                     @@
##             release-1.1-997286b692bd       #2   +/-   ##
===========================================================
  Coverage                            ?   52.02%
===========================================================
  Files                               ?      153
  Lines                               ?    14893
  Branches                            ?        0
===========================================================
  Hits                                ?     7748
  Misses                              ?     6408
  Partials                            ?      737

Flag
Coverage Δ

unittest
52.02% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

/merge

/merge


New project references "place-my-order"

My project is called "scraplog" and somehow my index.stache references "place-my-order" in the production
      {{/case}}
      {{#default}}

      {{/default}}
    {{/switch}}

Actually, I'm not sure this is a bug.  Did you re-init this folder?
It seems like what you are showing is impossible from the current state of the index.stache template:
https://github.com/donejs/generator-donejs/blob/master/app/templates/src/index.stache#L17

Also, which version of donejs are you using?  0.6?

Just ran donejs init foobar and got this for an index.stache:

    <title>{{title}}</title>
    {{asset "css"}}

    <can-import from="foobar/styles.less!" />
    <can-import from="foobar/app" as="viewModel" />

{{message}}

    {{asset "inline-cache"}}

    {{#switch @env.NODE_ENV}}
      {{#case "production"}}

      {{/case}}
      {{#default}}

      {{/default}}
    {{/switch}}

My donejs version is 0.6.0.

What are your steps to create the project? Are you doing:
donejs init project-name

or
mkdir project-name
cd project-name
donejs init

Both are valid.

donejs init project-name

what version of generator-donejs is in your node_modules folder? Can you check that file justin referenced above?

My generator-donejs index.stache looks like @justinbmeyer's. The main attribute looks like this:
main="<%= name %>/index.stache!done-autorender"

Which version is in the package.json?

"version": "0.4.3",

This looks like an issue of an older DoneJS version. The generator should use the project name for index.stache (see https://github.com/donejs/generator-donejs/blob/master/app/templates/src/index.stache#L17).

I can't verify this with the latest versions. Please reopen if it still happens after npm cache clean and reinstalling everything.


go get github.com/docker/swarm broken

It was working yesterday but today:
root@zhlpeer:/opt# export GOPATH=/opt/golang
root@zhlpeer:/opt# go get github.com/docker/swarm
# github.com/docker/swarm/api/nopclient
golang/src/github.com/docker/swarm/api/nopclient/nop.go:226: undefined: client.RequestPrivilegeFunc
golang/src/github.com/docker/swarm/api/nopclient/nop.go:231: undefined: client.RequestPrivilegeFunc
golang/src/github.com/docker/swarm/api/nopclient/nop.go:241: undefined: client.RequestPrivilegeFunc

I think if go get worked, it was by chance that all the dependencies were at compatible. The official way to build is with godep and documented in contributing.

By chance or not, go get github.com/docker/swarm certainly is a much simpler way to build than the "official" way.

@gongsu832 it's very common to have this issue when the project's dependencies evolve so quickly.
This particular issue is going to be solved when we are going to switch to go 1.6 with the new vendoring

@vieux good to know. Thanks!

This is due to https://github.com/docker/engine-api/pull/162 being merged today. This should be fixed the next time we update Godeps for engine-api, preferably after #1983.

@gongsu832 this will be fixed by #2124


[FR] Local Differential Privacy Methods

Is your feature request related to a problem? Please describe.
Currently there is only one implementation of local differential privacy (LDP): RAPPOR[1], implemented in https://github.com/TL-System/plato/blob/main/plato/utils/unary_encoding.py and it is not decoupled with algorithm implementation.
https://github.com/TL-System/plato/blob/fac44a6bdbe64d3060ae290e4633b316b02a1474/plato/algorithms/mistnet.py#L52-L64
https://github.com/TL-System/plato/blob/fac44a6bdbe64d3060ae290e4633b316b02a1474/plato/algorithms/mindspore/mistnet.py#L44-L48
https://github.com/TL-System/plato/blob/fac44a6bdbe64d3060ae290e4633b316b02a1474/examples/nnrt/nnrt_algorithms/mistnet.py#L60-L65
This feature request calls for a modular LDP plugin interface and a number of different other methods e.g. [2][3]
Describe the solution you'd like

[x] Unified data exchange format between clients and server.
[x] A modular interface for plugging in data processing modules into the server-client data exchange.
[x] A config entry for enabling specific data processing modules.
[ ] LDP modules implementation.
[ ] Test on the theoretical property of modules i.e. ε-LDP

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
To be filled.
Additional context
Add any other context or screenshots about the feature request here.
[1] Ú. Erlingsson, V. Pihur, and A. Korolova. Rappor: Randomized aggregatable privacy-preserving ordinal response. In Proceedings of the 2014 ACM SIGSAC conference on computer and communications security, pages 1054–1067. ACM, 2014.
[2] Differential Privacy Team, Apple. Learning with privacy at scale. 2017.
[3] B. Ding, J. Kulkarni, and S. Yekhanin. Collecting telemetry data privately. In Advances in Neural Information Processing Systems 30, December 2017.

To combine with adding model/feature compression methods later, I also propose to implement it as a middleman or preprocessor interface that can accept a number of data preprocessing modules before data is sent between clients and server

Challenging to do but makes sense.

In investigation of the data transferred, we can standardise two types of data format:

features, in the form of a list of numpy arrays, with each numpy array represents a feature extracted from an input.
model parameters, in the form of a ordered dict of (layer name) to (numpy array of the parameters of that layer).

The algorithm class should be responsible in providing the formatting to and from the framework format into our standard format in the method extract_weights, load_weights and extract_features. Loading of features in servers can be implemented in the feature.DataSource class.

A new class called DataProcessor is proposed. Each DataProcessor class should have a process method that processes the data.
Two new config entries can be added for both client and server, one for data receiving and one for sending. Each entry should be a list of the names of DataProcessor classes in the order of processor application.
For example when a client receives a piece of data from the server, the data is first processed by the list of DataProcessor in the order of the config list., and then passed to the client's remaining handling.
For example when a client is ready to send a piece of data, the data is first processed by the list of DataProcessor in the order of the config list., and then sent out to the server.
Same can be said for the server.

A serializer/deserializer DataProcessor class can also be mandated as the last/first DataProcessor to provide different transfer encoding than Python pickle.

I like the design of the DataProcessor class. For now, however, I think we should continue to use pickle for data transfers, as this is challenging to replace -- it would require changing a lot of existing data transfer code that is field tested.

Individual client can have different privacy requirement and might opt to set their own LDP parameter i.e. epsilon. Is the performance of federated learning model under varying privacy parameter a valid research question?
Support for variable privacy parameter for individual clients? Or a randomised privacy parameter distribution for clients?

Fact is, even the performance of federated learning with the same set of parameters across all clients is not well understood.

I briefly studied the other two LDP methods, they seem to be applied to statistical data e.g. counters and frequencies.
I cannot determine if they are suitable, and how are they applied to the FL scenario.

Beyond randomized response (using unary encoding, which is implemented already in Plato), one can implement other, more mainstream mechanisms of differential privacy. These include (but not limited to) the Laplace mechanism, the Gaussian mechanism, the Exponential mechanism_, and the sparse vector technique. More detailed descriptions of these differential privacy mechanisms can be found here:
Programming Differential Privacy
These mechanisms should all be fairly easy to implement. For example, the Laplace mechanism involves simply calling the function np.random.laplace() in Python's numpy framework.
For a more detailed coverage, you can also find a comparison between randomized response and the Laplace mechanism in this research paper:
http://ceur-ws.org/Vol-1558/paper35.pdf

Not sure about the applicability of exponential mechanism and sparse vector technique as exponential mechanism is more suitable for querying for a discrete/finite result e.g. actual inferencing, and sparse vector technique seems to be applied to finding a query that has result above the threshold. Not exactly a data feeding process into the model training.
Also, is applying clipping to features and model parameters make sense? I assume its ok for features, but not sure for model parameters.

No, I agree that the exponential mechanism and sparse vector technique do not make much sense.
We actually don't apply Gaussian or Laplacian mechanisms on model parameters — we apply them on gradients during training inside the training loop (where clipping is done as well). So some of your recent code needs to be redone. I am working on this in the 'gradient_dp' branch.


Fix failing Travis CI build

The go.mod file has the appropriate dependencies, but they don't appear to be resolved.

Fixed in f63ca3c0


Worker nodes not able to delete message from azure storage queue.

I am trying dramatiq-azure with the following code:
@dramatiq.actor(queue_name="default", store_results=True)
def count_words(url):
response = requests.get(url)
count = len(response.text.split(" "))
print(f"There are {count} words at {url!r}.")
return count
@app.route("/")
def hello_world():
urls = [
"https://google.com",
"https://facebook.com",
"https://rabbitmq.com",
"https://bmw.com",
"https://apple.com",
"https://amazon.com"
]
x = [count_words.send(url) for url in urls]
return "Successfully sent all the messages"
But the workers are throwing this error:
There are 481 words at 'https://google.com'.
[2023-07-18 18:01:50,882] [PID 15824] [Thread-3] [dramatiq.worker.ConsumerThread(default)] [ERROR] Unhandled error during post_process_message(count_words('https://rabbitmq.com')).  You've found a bug in Dramatiq.  Please report it!
Although your message has been processed, it will be processed again once this worker is restarted.
Traceback (most recent call last):
File "C:\Users\danish.kapoor\Desktop\Flask APP\myproject\env\Lib\site-packages\dramatiq\worker.py", line 352, in post_process_message
self.consumer.ack(message)
File "C:\Users\danish.kapoor\Desktop\Flask APP\myproject\env\Lib\site-packages\dramatiq_azure\asq.py", line 112, in ack
self.__remove_from_queue(message)
File "C:\Users\danish.kapoor\Desktop\Flask APP\myproject\env\Lib\site-packages\dramatiq_azure\asq.py", line 107, in __remove_from_queue
self.q_client.delete_message(message._asq_message)
File "C:\Users\danish.kapoor\Desktop\Flask APP\myproject\env\Lib\site-packages\azure\core\tracing\decorator.py", line 76, in wrapper_use_tracer
return func(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\danish.kapoor\Desktop\Flask APP\myproject\env\Lib\site-packages\azure\storage\queue_queue_client.py", line 956, in delete_message
process_storage_error(error)
File "C:\Users\danish.kapoor\Desktop\Flask APP\myproject\env\Lib\site-packages\azure\storage\queue_shared\response_handlers.py", line 189, in process_storage_error
exec("raise error from None")   # pylint: disable=exec-used # nosec
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "", line 1, in
File "C:\Users\danish.kapoor\Desktop\Flask APP\myproject\env\Lib\site-packages\azure\storage\queue_queue_client.py", line 949, in delete_message
self._client.message_id.delete(
File "C:\Users\danish.kapoor\Desktop\Flask APP\myproject\env\Lib\site-packages\azure\core\tracing\decorator.py", line 76, in wrapper_use_tracer
return func(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\danish.kapoor\Desktop\Flask APP\myproject\env\Lib\site-packages\azure\storage\queue_generated\operations_message_id_operations.py", line 279, in delete
map_error(status_code=response.status_code, response=response, error_map=error_map)
File "C:\Users\danish.kapoor\Desktop\Flask APP\myproject\env\Lib\site-packages\azure\core\exceptions.py", line 112, in map_error
raise error
azure.core.exceptions.ResourceNotFoundError: The specified message does not exist.
RequestId:351e1b3c-7003-006b-4773-b938f3000000
Time:2023-07-18T12:31:50.5462703Z
ErrorCode:MessageNotFound
Content: MessageNotFoundThe specified message does not exist.
RequestId:351e1b3c-7003-006b-4773-b938f3000000
Time:2023-07-18T12:31:50.5462703Z
Any suggestions what needs to be changed?

could you please provide any update on this issue. We need this urgently.

Can you create a ASQ-only test case that reproduces the issue? There are too many external factors here (i.e. flask, network calls).

@bidossessi We have also encountered this problem. In our case, it is reproduced every time when the message processing by the actor exceeds the set timeout of message visibility in the queue.
For example, the message visibility timeout is 10 minutes.

The actor "A_1" starts processing message "M". And does so for e.g. 15 minutes.
After 10 minutes, message "M" becomes visible in the queue again and another actor "A_2" starts processing it. At this stage we have two actors processing the same message "M".
Now one of the actors finishes processing message "M" and removes this message from the queue.
The other actor also finishes processing the message "M" and gives the command to delete the message. This is where the error occurs. Azure says that no message was found and Dramatiq logs "You've found a bug in Dramatiq". After that the worker hangs, it just stops processing the next messages.

Hopes it make sense.

Is the expected behaviour that the worker does not hang? This might probably better be handled in the worker code?
The following bugs might be relevant:

https://github.com/Bogdanp/dramatiq/issues/445
https://github.com/Bogdanp/dramatiq/issues/211


Which Scheme implementation does the Makefile rely on?

Hi, I'm keen to run the tests.
Could you make a note in the README about which version of Scheme was used?

Ah, it's in the Dockerfile 😊 -> Chez Scheme.

Feel free to send a pull request to clarify README.

@daveduthie I wrote that Makefile, I should have been more precise. 🤦♂

Fix it. :)


Using TSL with default netTcpBinding

Hi,
With default netTcpBinding (Mode=Transport + clientCredentialType= Windows + protectionLevel=EncryptAndSign) TLS used secure the transport channel? or something else used to encrypt the channel?
I found below link. I need to confirm this,
https://stackoverflow.com/questions/4187604/how-does-nettcpbindingread-windowsstreamsecuritybindingelement-encrypt-sign-me
If default netTcpBinding does not use TLS then, the only when it can be used for transport level security by using Certificate (Mode=Transport + clientCredentialType= Certificate + protectionLevel=EncryptAndSign)?

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

ID: 028a8be0-9ff5-f253-aa43-d02fc1c8e9be
Version Independent ID: 75ce978f-aecd-5b60-be3f-54c78a6a71aa
Content: Transport Security Overview
Content Source: docs/framework/wcf/feature-details/transport-security-overview.md
Product: dotnet-framework
GitHub Login: @Lxiamail
Microsoft Alias: wcfsrvt

Dupe of #10592.


DIP25: parsing

This starts implementing:
http://wiki.dlang.org/DIP25
the parsing, name mangling, and pretty-printing. Also sets up the type, and handles covariance.
Adds -DIP25 switch which will later be used to enable the semantics of it. With this PR, adding return attributes is 100% backwards compatible.

Scope is a much more descriptive variable and switch name than DIP25.

The switch will only be temporary for transitional purposes. Hence a name that is not likely to collide with some future use for -scope.

Added some basic checking and inference semantics.


Implement ping/pong

Are you planning to implement ping/pong messages to the slack api such that the client can monitor its connection and reconnect when it is dropped?
I'm having some issues with my connection and it looks like the client stays active after a connection drop, but it is dropped from slack's point of view.
A similar sollution worked in another slack-bot project

+1

+1

I don't think I'll be implementing this feature, as I don't have enough resources to actively develop this project, but I am definitely open to someone else implementing it. I can offer advice and guidance on ReactPHP if that is a barrier for anyone looking into it themselves.

For anyone interested, I hacked a solution into my rather hacky bot script:
$nReconnects = 0;
do {
	$oClient->connect()->then(
		function () use ( $oClient, $sDevelopmentChannelId, &$oChannel ) {
			echo "Connected!", PHP_EOL;

			$oClient->getChannelById( $sDevelopmentChannelId )->then(
				function ( \Slack\Channel $oObtainedChannel ) use ( $oClient, &$oChannel ) {
					$oChannel = $oObtainedChannel;
				}
			);
			$oClient->send( 'Reconnected...', $oChannel );
		}
	);

	$oReactLoop->run();

	$nReconnects++;
	sleep(10); // so as not to hammer it, if there are genuine connection issues.
}
while ( $nReconnects <= 10 && !$bDisconnect );

The $bDisconnect is a boolean that is set when the bot receives a disconnect message, so that I can break out of that reconnection loop. Also, the $nReconnects is just a safety for not forever running while you play with it, otherwise just remove it for a forever thing.
This might help some of you without actually implementing anything into the client library. You could also implement a ping/pong in a similar fashion.


New Android Webview 2.3.3 prompts to autofill the password

When an Ionic is built with a form that has a password input type, the new Webview is offering to save the password to Google account. I know this is probably a Chrome behaviour but I suggest that we should enforce the webview to disable this behaviour since it saves it under http://scheme:PORT_NUMBER not normal behaviour for "apps"

Hi,
Any news on this problem ?
It's really strange to have the password popup on localhost showing up for login forms, not "native" at all.
I had to create a fork to add webView.setImportantForAutofill(View.IMPORTANT_FOR_AUTOFILL_NO); in the android part, it's working but I don't know about the side effects.

Can you provide a sample app reproducing the issue?
Also, does it still prompts if you add  autocomplete="off" to the field?

@jcesarmobile It prompts with autocomplete="off" and also autocomplete="new-password"
You can see a screenshot here: https://forum.ionicframework.com/t/autofill-label-problem/160580
On this post they say it might not show on all devices and I've got a P10 with android 8.
I removed the label by adding this to src/android/com/ionicframework/cordova/webview/IonicWebViewEngine.java, but I'm not sure of the side effects :
// Remove google autofill for Android > 8
if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) {
webView.setImportantForAutofill(View.IMPORTANT_FOR_AUTOFILL_NO);
}

Hi. Any updates on this issue?

autocomplete doesn't work

Did anyone found a solution for the ugly autofill labels on android login and passwords fields ?

Hi, Any updates on the issue?

@jcesarmobile Any updates?

Any solution in 2022 to either hide this or successfully save the password for the app?


Default values

I think it would be a great feature if we could add default values to dotenv. This way if an environment variable is not set, dotenv could set a default value. In some cases  it would be useful instead of requiring a specific variable to be set. For example: application environment: dev by default, can be staging, production.

You can achieve default values with ?: operator.
$foo = getenv("FOO") ?: "BAR";

Yeah, that's obvious.......and ugly. Also. default values are set when the environment is loaded. You cannot do it inside your application or at least it is ugly again.

Yeh, that's not really a proper solution to the problem here.

BTW what I want would look like something like this:
if (!getenv('FOO')) {
    putenv('FOO=bar');
}

The best way to achieve this would be, in my opinion, to use the bash standard for this, so we can keep the behavior when sourcing the dotenv file from the shell.
hchinchilla@bowser ~/projects/dockerlib$ echo $foo
hchinchilla@bowser ~/projects/dockerlib$ echo ${foo:-my_default_value}
my_default_value

@sagikazarmark , I would sugest the Laravel syntax in this case. Something like:
getenv('APP_ENV', 'production')
And I fully agree, this would be a great addition.

That's the same as @tuupola's solution and the problem is the same: I want to set defaults at bootstrap time, not in the middle of my application. Environment is not something dynamic, it should be fixed, even if it is a default. And the correct place for it is at bootstrap time.

:+1: default values in bootstrap time will good I think.

I second @mattboban's suggestion, because other frameworks do it like this, and I think it's a good idea to harmonize approaches across - and best practices - frameworks.
Also Python does it like Laravel. Add a second parameter to retrieve a default value in case the value is not set. Without default value the call can fail with an exception (good feedback). Specify a default value to allow robust, self-healing, retrieval.

As a workaround I used the following functions to create my own version of the env() function.
`
/**

Check an env. var is set, else return default value

@param $param
@param $value
@return mixed
*/
function getEnvValue($param, $value)
{
return env($param, $value);
}

/**

Gets the value of an environment variable. Supports boolean, empty and null.

@param  string  $key
@param  mixed   $default
@return mixed
*/
function env($key, $default = null)
{
$value = getenv($key);
if ($value === false) {
return value($default);
}
switch (strtolower($value)) {
case 'true':
case '(true)':
return true;
case 'false':
case '(false)':
return false;
case 'empty':
case '(empty)':
return '';
case 'null':
case '(null)':
return;
}
if (startsWith($value, '"') && endsWith($value, '"')) {
return substr($value, 1, -1);
}
return $value;
}

/**

Return the default value of the given value.

@param  mixed  $value
@return mixed
*/
function value($value)
{
return $value instanceof Closure ? $value() : $value;
}

/**

Determine if a given string starts with a given substring.

@param  string  $haystack
@param  string|array  $needles
@return bool
*/
function startsWith($haystack, $needles)
{
foreach ((array) $needles as $needle) {
if ($needle !== '' && substr($haystack, 0, strlen($needle)) === (string) $needle) {
return true;
}
}
return false;
}

/**

Determine if a given string ends with a given substring.

@param  string  $haystack
@param  string|array  $needles
@return bool
*/
function endsWith($haystack, $needles)
{
foreach ((array) $needles as $needle) {
if (substr($haystack, -strlen($needle)) === (string) $needle) {
return true;
}
}
return false;
}
`

There's a PR for this feature already open: #229 — maybe this issue should be closed, and discussion continued over there?

That PR has been open since April alongside PR dating back to October 2016.
Is this project dead? Sure looks like it.

It seems it might be. :-(


SpriteRenderer excepts on WebGLRenderer destruction

WebGLRenderer.prototype.destroy calls destroyPlugins, which due to bc54b1b73ae8c2f09a1752bcb61201d1fcf5476a has  SprireRenderer.prototype.destroy call ObjectRenderer's destroy method on itself, which is actually WebGLManager.prototype.destroy. This sets renderer to null, which causes an exception here.
Simplified jsFiddle from #1968 with pixi compiled from f5abaa7060bd1347cfd0506ab2b60db9fc8c9f19.
I may take a stab at PRing fixes for this and #1968 because I need renderers to destroy cleanly.

Ah, nice catch; sorry about that. Thanks for the coming PR!


PWN-3748 - Swap/Send screen selection mechanism / Part 2

Jira Ticket
https://p2pvalidator.atlassian.net/browse/PWN-3100
https://p2pvalidator.atlassian.net/browse/PWN-3748

don't review guys, please, yet


Why the function part can use number?

[Enter feedback here]
What does this mean
SELECT ROUND(150.75, 0, 1);

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

ID: 7b61f3f8-eb79-944e-3241-e53dcaa294fa
Version Independent ID: dd7552a3-5917-84e3-2746-cb3489fabaf5
Content: ROUND (Transact-SQL) - SQL Server
Content Source: docs/t-sql/functions/round-transact-sql.md
Product: sql
Technology: t-sql
GitHub Login: @MashaMSFT
Microsoft Alias: mathoma

@MashaMSFT Hi Masha, At a brief glance, all I can glean about the third parameters is that.....
_the numeric value of 1 for the third parameter seems to mean "truncate", rather than "round".....
.....per example C.  So in example C, the ".75" portion is chopped off, leaving nothing to round up or down regarding the 1's place-value unit.
IF instead the Select statement had been.....
SELECT ROUND(150.75, 1,1);
..... the result would have been "150.7", I presume, as the second parameter would mean Do not chop off the first digit after (to the right of) the decimal point.  So in both cases, the third parameter value of "1" means truncate (chop off), at the position indicated by the second parameter (I think).
The preceding explanations probably answer the specific question asked by @kind03, but overall the article seems lacking in its explanation of the third parameter.
Maybe 0=Round, 1=Truncate, but other values I dunno?

@kind03 Oh now I see. The article Does explain "function" parameter. 0=Round, and any other value means =Truncate.  Example C shows the nonZero value of "1" performing truncation.
The answer is now complete.
So I will Close this Issue 1927 now, as Fixed (answered).
Thank you for inquiring.
CC: @MashaMSFT


Add Expiration Date to Posts

We need to add expiration_date to the Post model and the Post endpoints.
The frontend will then filter out posts that have expired when it renders the feed.

I think the backend will apply filters based on query params that are sent by the frontend? Well that makes more sense to me anyway... maybe I didn't understand the requirements exactly, haven't looked at the designs for the feed in detail yet.

That makes sense, thanks Robin. Updated the ticket

When creating the post, we set the expireAt (I suggest that name to keep the consistency with createdAt and updatedAt timestamps) at X time after the current date and then filtering it by the fronted params should be easy at the backend as @robinv85 said.
What is still not clear to me about this feature @dfrancks is when editing, what is the reference date? If the user edits a post and sets to expire "1 week", should we set it to 1 week from the date he is editing or 1 week from the date the post is created?

I think one week from editing makes more sense. I'll update the ticket.

I think one week from editing makes more sense. I'll update the ticket.

@Naraujo13 this is also implemented on your posts PR right?

@Naraujo13 this is also implemented on your posts PR right?

Yes, I've added the expireAt at both posts endpoints, this should be solved with the posts migration.

@Naraujo13 should we close this ticket?

It's not done yet, but I will assign this issue to me and link it with the PR for Posts migration since it is being done there.
It is currently being reviewed.

@Naraujo13 - Can we please move to Done so that PM's can do UAT?

Sure, I moved it to QA thinking that was the state  for that


Enable operation with Self-Managed instances

Here is a complete credentials set for a Self-Managed Instance:
export ZEEBE_SECURE_CONNECTION=false
export ZEEBE_ADDRESS='localhost:26500'
export ZEEBE_CLIENT_ID='zeebe'
export ZEEBE_CLIENT_SECRET='zecret'
export ZEEBE_AUTHORIZATION_SERVER_URL='http://localhost:18080/auth/realms/camunda-platform/protocol/openid-connect/token'
export ZEEBE_TOKEN_AUDIENCE='zeebe.camunda.io'
export CAMUNDA_CREDENTIALS_SCOPES='Zeebe,Tasklist,Operate'
export CAMUNDA_OAUTH_URL='http://localhost:18080/auth/realms/camunda-platform/protocol/openid-connect/token'

This is optional, for Zeebe Node, at least:
export ZEEBE_TENANT_ID='<default>'

Depends on https://github.com/camunda-community-hub/camunda-8-credentials-from-env/issues/3

This is implemented in the new client: https://github.com/camunda-community-hub/camunda-8-js-sdk


Namespace cattle-global-nt not assigned to System project

Running a HA cluster of 3 nodes, fresh install of Rancher 2.3.5.
For some reason namespace cattle-global-nt is not assigned to System project:

Same on Rancher helm v2.4.3

Component
Version

Rancher
v2.4.3

User Interface
v2.4.22

Helm
v2.16.5-rancher1

Machine
v0.15.0-rancher36

Same in Rancher v2.4.4

Seems to be fixed, in v2.4.8 cattle-global-nt is assigned to the System project

This was added via https://github.com/rancher/rancher/pull/27056, let us know if this happens in recent versions.


Update version number

Hello,
The latest commit on master fixes an issue with "Division by zero" but the commit is not tagged as a release so i can't download the fixed version through composer

Oops, I thought I had done that. I'll fix that now 😃

Thanks, can I close this issue?


[Event Request] table 83 "Item Journal Line" - OnAfterSetReservEntrySourceFilters

Describe the request
    internal procedure SetReservEntrySourceFilters(var ReservEntry: Record "Reservation Entry"; SourceKey: Boolean)
    begin
        if IsSourceSales() then
            ReservEntry.SetSourceFilter(Database::"Item Journal Line", "Entry Type".AsInteger(), "Document No.", "Document Line No.", SourceKey)
        else
            ReservEntry.SetSourceFilter(Database::"Item Journal Line", "Entry Type".AsInteger(), "Journal Template Name", "Line No.", SourceKey);
        ReservEntry.SetSourceFilter("Journal Batch Name", 0);
        OnAfterSetReservEntrySourceFilters(ReservEntry, SourceKey);
    end;

    [IntegrationEvent(false, false)]
    local procedure OnAfterSetReservEntrySourceFilters(var ReservEntry: Record "Reservation Entry"; SourceKey: Boolean);
    begin
    end;

Additional context
The change is needed as the function SetReservEntrySourceFilters has been introduced in BC24. This function did not exist prior to BC 24 and the filtering was always on Journal Template Name and Line No. With introduction of the function in BC24 the solution stopped working as the error did appear as no Reservation Entries were found during posting of inventory entries involved in sales process. Therefore, we need a possibility to fallback to old behavior in certain scenarios when Serial No.s are posted in sales area.
ReservEntry.SetSourceFilter(Database::"Item Journal Line", "Entry Type".AsInteger(), "Journal Template Name", "Line No.", SourceKey);

Internal work item: AB#538874

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 21140.


Add tests to validate templates generated by bicep

This would likely be a subset of all the examples and should focus on complex scenarios.

much like terraform validate, this would be handy


Populate partialFingerprints for results in SARIF reports

Expected Behavior
partialFingerprints is populated for results in SARIF reports.
Current Behavior
No fingerprint is generated.
Context
SARIF report should provide all data that's "required" by GitHub.
This attribute is useful:

to avoid seeing duplicate alerts
to accurately identify which results are the same across commits and branches

Related with #7661

This link is very interesting to know how to implement this https://docs.oasis-open.org/sarif/sarif/v2.1.0/errata01/os/sarif-v2.1.0-errata01-os-complete.html#_Toc141791182
To me this is the most important part:

It is difficult to devise an algorithm that constructs a truly stable fingerprint for a result. Fortunately, for practical purposes, the fingerprint does not need to be absolutely stable; it only needs to be stable enough to reduce the number of results that are erroneously reported as “new” to a low enough level that the development team can manage the erroneously reported results without too much effort.

We should be aware that it's impossible to create a bullet proof implementation for the partialFingerprint.


Please update the Image in the Readme file before the pre-requisites section

The the  image before the per-requisites sections in the readme file leads to resource not found

Readme page screenshot

@Raveendiran-RR I have fixed the URLs. That should work for you.

Closing this issue.


Merge dev to master (minus commit 71bf2a5)

This is everything from dev except my lookAt changes in commit #71bf2a5

Tests with the current release of Altspace. LookAt was functional, so I think I'm good to merge this.


Add an explicit link to the source license in the README

This change is

Following the example of VS Code: https://github.com/Microsoft/vscode/#license

@liminzhu does this look alright?

LGTM :).


Get form fields before form submission

Hi,
Is there any built in functionality that allows getting the Form field values as the user interacts with the form?
Thanks a lot!

@danielweinmann thank you. Guga has also replied about the watch. I've mistakenly used watch by importing it from react-hook-form instead from within remix-forms.


Saving file and quickly closing app can result in the file getting emptied

I have lost a few files after saving and closing the application.  The file becomes zero bytes.
Steps to Reproduce:

File must not be dirty
Save file
Close application.  Must follow save by less than a second
Save onto slowish USB flash device
Could not re-produce with faster internal SSD
All extensions disabled

If file is dirty, the normal file save location might still become zero bytes, but vscode can restore the contents using magic journalling or something.
Similar to #50942 and 51016
Test machine:
Windows 10
PNY 64GB USB 3.0 FD flash drive
Version: 1.27.2 (system setup)
Commit: f46c4c469d6e6d8c46f268d1553c5dc4b475840f
Date: 2018-09-12T16:17:45.060Z
Electron: 2.0.7
Chrome: 61.0.3163.100
Node.js: 8.9.3
V8: 6.1.534.41
Architecture: x64

@Scott-Cooper would you be willing to conduct a small experiment and setup your machine where you see this to run VSCode out of its sources so that I can give you some modifications to see if it helps? The instructions are here: https://github.com/Microsoft/vscode/wiki/How-to-Contribute

@Scott-Cooper great, can you do this to try out if it helps:

open fileService.ts line 580
change the code if (!exists || !isWindows) { to if (!exists || isWindows) { (remove the ! from isWindows)
recompile typescript
save on that USB drive

Does it reproduce?

@Scott-Cooper ping


onyo: remove dry-run

Removes --dry-run everywhere, including commands, main/shared_arguments, tests and docs.
The flag is out of date and mostly removed anyways -- this PR just removes the last references to it.

LGTM

Merging. :-)


Learning Material for Cluster-API and MetalCubed

Learning Material for Cluster-API and MetalCubed

You should just have to rebase your branch now that https://github.com/SUSE-Cloud/socok8s/pull/663 has merged.


Telegram bot with TLS 1.2 & TinyGsm & Universal Telegram Bot - collaboration?

Is not an Issue, is an Enhancement.
As Telegram don't work anymore with SIM800L, can you talk with TinyGSM and Universal Telegram bot to make an integration?

No plans at the moment, may have to rely on others
But you can send me a link to see what it is, I haven't heard of it.

Good evening  Mr,
They create library  for serial communication with Simcom. Modules. As your LilyGo are as an compact product, is very good one for different applications. PS: In future can you add Ring Indicator tied to a thruhole somewhere. What i saw in your diagram RI is not connected on sim7000 also not in sim800. https://github.com/vshymanskyy/TinyGSM

PS: In future can you add Ring Indicator tied to a thruhole somewhere.
What does it mean ?
What i saw in your diagram RI is not connected on sim7000 also not in sim800.
I personally do not recommend SIM800 and SIM7000. SIM800 is no longer available in my country because we do not have 2G here. SIM7000 is unless you use NBIOT. If you use 2G, I recommend A7670. A7670 is easier to use than SIM800 and SIM7000.
https://github.com/vshymanskyy/TinyGSM
vhymanskyy has no maintenance on TinyGSM, so what are you referring to?


[BEAM-2662] Fix Java quickstart for Spark to include Spark version

Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[x] Each commit in the pull request should have a meaningful subject line and body.
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Run mvn clean verify to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

R: @iemejia @jbonofre

Coverage decreased (-0.01%) to 70.579% when pulling a7c18495f91c9d7537271c9595b3ac4d24003ece on lukecwik:beam2662 into f39874860027832fdeff957afdc5cc8326de6d30 on apache:master.


Greedy Method -Activity Selection Problem

🛠 Issue (Number) #330
Issue no #330
👨💻 Changes proposed
In Greedy-Algorithms Folder an activity selection problem added in C++,C,Java,Python
✔ Check List (Check all the applicable boxes)

[✔] My code follows the code style of this project.
[✔] This PR does not contain plagiarized content.
[✔] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers
I have added Activity Selection Problem in Greedy-Algorithms Folder in all the languages(C,C++,Java,Python)

📷 Screenshots
->Number of processes is the activities which is given dynamically by user.
->Start and finish time is taken by the user for each activity.
->According to the algorithm the output is given as the activities which will be selected.

Add comments to explain your approach

I have added the comments to explain the approach just now. If there is a problem in the code then please tell .

Please include the issue number of Matrix Chain

It is issue number #189 and implemented in #329

I have added the python,C++ as well as Java Code

If you have removed the code of Matrix Chain Multiplication, then remove the issue number from description

yes removed

code of matrix multiplication not removed yet

Removed .Please check

@shailja2727 Please add a output screenshots to depict the result.

Added with steps. Please check the topmost comment where i've added a screenshot and respond is any changes needed

You can try adding the code for C too.

Just added the code in C too. Please Check

Done with all changes requested, @Kumar-laxmi @Arun9739 Please review it

If you have removed the code of Matrix Chain Multiplication, then remove the issue number from description

Sir I've done all the changes and my branch is updated to upstream .Please review @Kumar-laxmi


  I cloned over https.  Not sure if it matters.
Tried cloning using ssh - same problem.  I also can't seem to use the example project
File "build/bdist.macosx-10.10-x86_64/egg/table/tests/test_columns.py", line 124
column1 = MonthsColumn(None, date(2012, 12, 18), date(2013, 07, 01))
^
SyntaxError: invalid token
byte-compiling build/bdist.macosx-10.10-x86_64/egg/table/tests/test_tables.py to test_tables.cpython-34.pyc
byte-compiling build/bdist.macosx-10.10-x86_64/egg/table/tests/test_views.py to test_views.cpython-34.pyc
byte-compiling build/bdist.macosx-10.10-x86_64/egg/table/tests/test_widgets.py to test_widgets.cpython-34.pyc
byte-compiling build/bdist.macosx-10.10-x86_64/egg/table/urls.py to urls.cpython-34.pyc
byte-compiling build/bdist.macosx-10.10-x86_64/egg/table/utils.py to utils.cpython-34.pyc
File "build/bdist.macosx-10.10-x86_64/egg/table/utils.py", line 45
except Exception, e:
^
SyntaxError: invalid syntax

Well, it's really weird. I built this application on my Macbook as well.
Try the following commands, let me know the output.
$ cd django-datatable/example
$ python manage.py runserver


Vm Crash caused by unaligned float64 read on ARM

It looks like the DartVM makes unaligned memory access for doubles in certain cases.
We were recently seeing a VM crash with exit code SIGEMT ("Emulator trap; this results from certain unimplemented instructions which might be emulated in software, or the operating system's failure to properly emulate them."). Looking at it with gdb:
(gdb) bt
#0  0x00649f84 in dart::BootstrapNatives::DN_TypedData_GetFloat64(_Dart_NativeArguments*) ()
#1  0xb5dff17c in ?? ()
#2  0xb5dff17c in ?? ()
(gdb) disas
Dump of assembler code for function _ZN4dart16BootstrapNatives23DN_TypedData_GetFloat64EP21_Dart_NativeArguments:
...
   0x00649f7c <+728>:   add     r3, r3, r2, asr #1
   0x00649f80 <+732>:   vldr    d0, [r3]
=> 0x00649f84 <+736>:   bl      0x2b557c <_ZN4dart6Double3NewEdNS_4Heap5SpaceE>
   0x00649f88 <+740>:   mov     r2, r0
...
End of assembler dump.
(gdb) info registers
r0             0x0      0
r1             0x8      8
r2             0xa      10
r3             0xb5611a5d<PHONE_NUMBER>
r4             0xb5dabd4c<PHONE_NUMBER>
r5             0xb5dab928<PHONE_NUMBER>
r6             0xb5dabf90<PHONE_NUMBER>
r7             0xb5dabd54<PHONE_NUMBER>
r8             0xb5dab920<PHONE_NUMBER>
r9             0x914bd0 9522128
r10            0x81ab60 8498016
r11            0xb5dabf8c<PHONE_NUMBER>
r12            0x1      1
sp             0xb5dab918       0xb5dab918
lr             0x649f6c 6594412
pc             0x649f84 0x649f84 <dart::BootstrapNatives::DN_TypedData_GetFloat64(_Dart_NativeArguments*)+736>
cpsr           0x60000010<PHONE_NUMBER>
(gdb) p *(double*)0xb5611a5d
$1 = 42.420000000000002

The instruction which caused the SIGEMT signal is vldr which tries to load from 0xb5611a5d which is not aligned.
The causing dart code called most likely this function:
static double readDoubleFromBuffer(Uint8List buffer, int offset) {
  return buffer.buffer.asByteData(buffer.offsetInBytes)
      .getFloat64(offset, commandEndianness);
}

There seems to be a linux kernel option named CONFIG_ALIGNMENT_TRAP which, when enabled, will cause a software emulation of the unaligned access, thereby not causing any issues. The option is apparently enabled by default in newer kernels. See here.
=> Therefore I'm not sure if our buildbots would catch the issue.
Might be related to https://github.com/dart-lang/sdk/issues/23810 and https://github.com/dart-lang/sdk/issues/12868 .
$ dart --version
Dart VM version: 1.12.0-edge.b2ccdc92f8f87e8ccdac6a7468e2e69ca726fbc9 (Mon Jun 29 13:24:14 2015) on "linux_arm"

cc possibly interested people: @zanderso, @johnmccutchan, @iposva

If the typed data API is meant to support arbitrarily aligned accesses to ByteData buffers, then we'll have to emulate this on ARM, probably both in the functions generated by the TYPED_GETTER_SETTER macro in the ExternalTypedData class defined in //runtime/vm/object.h, and in {Load,Store}IndexedInstr. Since 32 and 64 bit integer accesses will be fixed up by either hardware or the kernel, we can probably just cast to/from an int type before the access. John, WDYT?

@zanderso I don't think we can guarantee alignment of the loads / stores. We will need some fallback for unaligned I/O.

Another duplicate: #22151

I was able to work around the crash using this: https://github.com/dart-lang/fletch/commit/cad1d1ff5997bda69582c05c81c04ff3345686a9
(Note to people attempting to copy my work around: be aware that it assumes that offset is a multiple of 8).

Since 32 and 64 bit integer accesses will be fixed up by either hardware or the kernel

As a side question: Does anyone know what big of a performance drop it is if we rely on the kernel to fix it?  [I would naively assume it's quite big -- in which case a programmer might be very surprised if he adds e.g. one more byte in front of a messages and suddenly everything gets much slower.]

@mkustermann I'd guess that doing something to prevent kernel handling of unaligned accesses would entail a big penalty to aligned accesses. I'll see if it's easy to convenient to get some numbers as I'm working on a fix.

@fsc8000 would https://github.com/dart-lang/sdk/commit/fb75e1e5e481334260b2fd467393a367fc74389a help with this?

My CL fixed the discrepancy in class member offsets between the host- and the target-toolchain. So I don't think it will help here. The ALIGN8 macro just aligns offset of the member - so there would still be unaligned access if the object start is misaligned.

@sortie Not sure if the slow down you experience is related to this, but sometimes the kernel traps unaligned access and emulates them which is significantly slower.

We saw a Flutter customer hitting this in the wild in https://github.com/flutter/flutter/issues/10701.

Closing in favor of #22151.


Add code to save model checkpoints

Maybe every 5 epochs?

Added code to save model weights every 10 epochs.


Kubelet logs would be more useful if standard form for namespace/name.container

We have a NamespacedName type, which helps some.  Kubelet might benefit from something like:
func FormatPodName(pod *api.Pod) string {
    return NamespacedName{pod.Namespace, pod.Name}.String()  // sort of inefficient, maybe a util func between them
}
func FormatContainerName(pod *api.Pod, ctr *api.Container) string {
    return fmt.Sprintf("%s:%s", FormatPodName(pod), ctr.Name)
}

Previous discussion: #3731

Related: https://github.com/kubernetes/kubernetes/issues/23338

Closing in favor of #23338, which has a wide scope and a longer discussion.


feat(publisher): allow platform level config for publish targets

Also allows for multiple publish targets to be run simultaneously
Fixes #118
Have you read the section in CONTRIBUTING.md about pull requests?
Yes
Are your changes appropriately documented?
No 😆 (will do soon)
Do your changes have sufficient test coverage?
Yes
Does the testsuite pass successfully on your local machine?
Yes
Summarize your changes:

Ready for :eyes:


Installer does not run on Win7 x86

From https://www.reddit.com/r/halospv3/comments/oly5qb/can_not_run_the_installer_tt/

Is SPV3.3 installer run on 64 bit only? If It's true, then is there a manual way to install It because I'm using win7-32bit.

The installer successfully installed SPV3 on a win7 x86 VM. I'm not sure what the issue was.


Glitch?

Found this glitch while playing music, downloaded the latest build from the play store. The app doesn't crash when I click on rewind button, but it sure is weird, I removed the music details, but the rewind button is glitchy, its the same in the notifcations too. Reboot didn't fix it, nor did clearing data.

There's actually a bug in the 5.0 platform that causes some Vector Drawables to render incorrectly like that. I think it has to do when you close a path and then relatively move to another area to begin a second path in the same data tag (which the sample material design icons that Google provides do from time to time). This had been working correctly until I recently switched to the gradle build tools to generate raster icons for older versions of Android (which supports the relative move operation, but doesn't support the reflection transformation I used to make it work previously), meaning that this only happens on devices running 5.0.
This should be resolved as of 8559d26.


Support Istio

Currently the starter job doesn't support istio. This means the test job pods launch, starter pod launches, and then immediately errors. Either because it can't communicate with the test job pods, it is waiting for the istio side car, etc.
Currently you have to setup a peer auth to open up the specific port that the starter job uses, ensure that there is no side car, and manage the tls/http settings.
Instead, I'd like to propose that k6 operator starter image simply uses a tool like: https://github.com/redboxllc/scuttle
Scuttle provides a wrapper around the shell command to wait for istio sidecar to launch and then quit afterwards properly.
This would involve a small modification to the containers/curl.go, from:
		Command: []string{
			"sh",
			"-c",
			strings.Join(parts, ";"),
		},

to:
		Command: []string{
			"scuttle",
			"sh",
			"-c",
			strings.Join(parts, ";"),
		},

And then it would require the building of a second docker image:
FROM radial/busyboxplus:curl

COPY --from=redboxoss/scuttle:latest /scuttle /bin/scuttle

and then obviously using that image. This could be put behind an istio flag, if desired, but I don't think there is any side effect to use scuttle in a non-istio environment, from the docs:
This application, if provided an ENVOY_ADMIN_API environment variable, will poll indefinitely with backoff, waiting for envoy to report itself as live, implying it has loaded cluster configuration (for example from an ADS server). Only then will it execute the command provided as an argument.

Another change required is to not use IP routing, but DNS based routing:
The specific change in this case is that istio doesn't support IP address routing. So the starter fails to actually tell the runners to do their job. Because the parrallelism is important, the solution is to have the operator stand up a k8s service per runner and use the DNS name of the service instead.
This would work both inside and outside the istio environment, so no breaking changes, and would probably make the operator work with other service meshes like linkerd, etc. That's something we could test later down the line.

I have a POC working for this in istio. I'll submit a PR so people can see the changes