"Short press" time sync not working

Describe the bug
App hangs when time sync is requested, via short press, and the time setting fails to be applied.
To Reproduce
Steps to reproduce the behavior:

Open the app
Short press lower right button on watch
App responds with spinner graphic.
No response from watch - reverts to previous state after a few moments.
App hangs with spinner and eventually forces close.

Expected behavior
App to push time setting to watch, and watch to give an OK message.
Smartphone (please complete the following information):

Device: Samsung S21+
OS: Android 13
Watch: DW-B5600
App Version: v10.1

Additional context
App doesn't appear to sync the time to the watch even when connected via bluetooth (long press lower left button), but the app doesn't hang in the attempt.

@geebotron
OK, thanks for the info. Let us first get a baseline of the problem. Could you try the following steps:

Make sure the official app is not running on you phone or another phone within Bluetooth range. This could interfere with the app.
Stop and restart the app form the Android settings.
When the app starts, press the "Forget" button, in case another watch was selected before.
Connect to the watch by long-pressing the lower-left button, go to "Actions" and make sure only "Set Time" action is enabled. If you have too many actions enabled, they may interfere.
Disconnect from the phone by pressing any button on the watch.
Run the time-setting action again by short-pressing in the lower-right button.

Please let me know the result, and we can take it from there.

After looking at this watch a bit further, it is a new and it uses a different module than the GW-B5600, which I am familiar with. If you have tried all the steps above, and still cannot set the time, there might be some difference in the way the watched communicate. I do not have this watch, so cannot try it. We are trying to raise some funds to support more watches. But let me know if you cannot set the time after you have tried the above steps.

I followed your instructions, and the result was the same for the short press time set, as I described above.
Using the long-press,  then he manual set time on the time screen seems to work - but I get no aknowledgement from the watch i.e. no OK message, just the actual adjustment of the time

Hmm. On further tinkering I am not convinced the app is setting the time, even via the set time feature on the time screen. It did appear to do this one time, but I can't get it to set it on subsequent attempts.

Yes, this is what I thought. The GW series has to send all its World Cities first, before setting the time. The DW has no World Cities, to the app gets stuck waiting for results (thus you see the red waiting circle forever). The DW should just send the time. This is what I am trying to do to support the DW.

Could you confirm that these are the specs for your DW-5600B watch? I.e. no
reminders, one alarm, no world time and no DST?
https://shockbase.org/watches/subseries_dyn.php?series=5600&subseries=DW-B5600&modul=3509
Thanks
On Tue., Aug. 29, 2023, 9:30 a.m. geebotron, @.***>
wrote:

Yes no problem - I will help you test this
—
Reply to this email directly, view it on GitHub
https://github.com/izivkov/CasioGShockSmartSync/issues/48#issuecomment-1697451936,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA7M37VLG3JXJ5WSZY7VZVTXXXVI3ANCNFSM6AAAAAA4CETTU4
.
You are receiving this because you commented.Message ID:
@.***>

Yes. Before the model number (DW-B5600) is stamped the digits "3509"

Unfortunately I don't have a google account - and it appears the drive is not accessible to an anonymous user.
Could the apk be uploaded to github?
I have not built Android packages before. What minimum environment would I need?
Sorry to be a pain!

I could not upload it GitHub, but I have given you access to the same APK.
You can try downloading again.
On Wed., Aug. 30, 2023, 4:17 p.m. geebotron, @.***>
wrote:

Unfortunately I don't have a google account - and it appears the drive is
not accessible to an anonymous user.
Could the apk be uploaded to github?
I have not built Android packages before. What minimum environment would I
need?
Sorry to be a pain!
—
Reply to this email directly, view it on GitHub
https://github.com/izivkov/CasioGShockSmartSync/issues/48#issuecomment-1699779489,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA7M37SHOU3YMS6V32MVTE3XX6NT5ANCNFSM6AAAAAA4CETTU4
.
You are receiving this because you commented.Message ID:
@.***>

I installed the debug app on the wife's phone, and the setting of the time seemed to work ok - from the time screen, but not via the "short press" of the lower right watch button.

Ok, so the same problem, the red circle keeps spinning? I'll take a look

Can I ask you to verify that the manual setup actually works. Please set the watch to a different time using the watch buttons, Then connect the app with the Lower-Left button and try setting the time from the main screen. When setting the time, you should see two messages at the bottom: "Setting time" and a few seconds later "Time Set". The time correct time should be set on the watch, and RCVD displayed on it. After that, you should be able to operate the app as usual.
If this works, change time manually using the watch buttons, and try the lower-right button again. If the first method works, the seconds whould work too.
Let me know.
Thanks for your help.

The permissions you have granted are, I imagine, for my wife's google identity: I personally have no google credentials at all.
The APK link still requires google credentials for access.
My wife is not around at this moment, but when she is I will commandeer her phone again!

Ok, sorry about that. I changed the permissions of the file so anybody with
a link can now download it.
The apk have a lot of restrictions, so if I send them via email, they most
likely will be blocked. Anyways, I think you should be able to get it via
the link now.
On Thu., Aug. 31, 2023, 8:05 a.m. geebotron, @.***>
wrote:

The permissions you have granted are, I imagine, for my wife's google
identity: I personally have no google credentials at all.
The APK link still requires google credentials for access.
My wife is not around at this moment, but when she is I will commandeer
her phone again!
—
Reply to this email directly, view it on GitHub
https://github.com/izivkov/CasioGShockSmartSync/issues/48#issuecomment-1700912228,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA7M37TVGBGUYHB6FF2VLUTXYB42NANCNFSM6AAAAAA4CETTU4
.
You are receiving this because you commented.Message ID:
@.***>

Yes - I can now download the debug app to my phone.
I had to un-install the 'release' version in order to debug version.
I can confirm now that the debug version sets the time correctl, in both modes: short-press of lower right button, and long-press connection mode.
I manually set the time on the watch differing to the phone, and the app corrected the time via both mechanisms.

Great! Thanks for your help.
I'll creat a proper release and will update the app on both playstore and
f-driod.
In the meantime, let me know if you find any other functions, like in
settings screen not working or missing. Also, see if alarms are working as
expected. For the DW, I have removed the "events" screen, since it is not
supported AFAIK.
Ivo
On Thu., Aug. 31, 2023, 10:38 a.m. geebotron, @.***>
wrote:

Yes - I can now download the debug app to my phone.
I had to un-install the 'release' version in order to debug version.
I can confirm now that the debug version sets the time correctl, in both
modes: short-press of lower right button, and long-press connection mode.
I manually set the time on the watch differing to the phone, and the app
corrected the time via both mechanisms.
—
Reply to this email directly, view it on GitHub
https://github.com/izivkov/CasioGShockSmartSync/issues/48#issuecomment-1701178157,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA7M37UF2JONT7WVXPULDE3XYCOYDANCNFSM6AAAAAA4CETTU4
.
You are receiving this because you commented.Message ID:
@.***>

And thank you for the swift implementation of the 3509 spec.
I will report any further issues I encounter...

Closing this issue. Support for the DW-B5600 watch will be in version 10.2 of this app.

@geebotron
One thing I thought of, what happens if you try to run action 'set reminders from Google calendar". Does the app crash?

No - the app does not crash: it reports "events sent to watch".
One thing I have noticed, is that setting the time from the time screen seems to introduce a random deviation from the phone time of up to 4 seconds.
Setting the time via the "short-press"  seem to result in a much more precise time sync, from phone to watch.

Another thing: the "find my phone" action (long-press of lower right button) seems to trigger in the app, but produces no ring alert from the phone i.e. phone remains silent.
The volume level setting appears to be normal, i.e. not zero.


[curl] Update Curl to 7.87

Update Curl to version 7.87 to fix CVE-2022-43551

Duplicate of #28977


Maybe a shortcut to toggle invisibility?

The error in after line mode is more intuitive, but the line number of the error will constantly increase or decrease when writing code, which can be distracting. Is it possible to implement a shortcut key to open and close this plugin?

I mean have a shortcut to change after mod and inline mode, when I want more detail error position I can use a shortcut change from inline to after

Hi, do you mean you want shortcut to change from after line mode to under line mode? I'll try my best to implement this in the next update.
Also there can be more useful shortccuts, do you have any  ideas?

I implemented shortcuts to toggle errors visibility & toggle text style (Under/After line).
Also hopefully hints won't stay anymore if error disappears.
Update 1.3.0 will be available after JetBrains checks.
If you will face hints staying after error is fixed, please open issue on it, because it is critical and I cannot reproduce this happening.
Other requested features may be added in a future!


[HUDI-5394] Fix RowCustomColumnsSortPartitioner

Change Logs
RowCustomColumnsSortPartitioner should not use sortWithinPartitions
Impact
May reduce the performance of the cluster when enabled hoodie.datasource.write.row.writer.enable
Risk level (write none, low medium or high below)
low
Documentation Update
None
Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

@boneanxs

LGTM.


Single Profile check function added

This pull request introduces a new function to the existing codebase that allows saving a user's profile data in the profileDiff collection.
Changes Made:
Added a new function to the call-profile package in the main.go file.
The function takes the necessary user data as input in the request body.
Validates the input data for completeness and correctness.
Inserts the user's profile data into the profileDiff collection, creating a single profile entry.
Returns the appropriate response based on the success or failure of the operation.
Reasoning:
The current implementation of the identity service lacks a dedicated mechanism to call all the profiles parallelly. By introducing the call-profile function, we ensure a more efficient and organized approach to call user profiles.
closes #147

We have to add a custom URL and lambda authorizer to our AWS before merging it.


Adjust chat search 'When' field

This forces chat search to also display times, not just dates. Sure it's more noise, but it beats binary-searching to figure out the lobby where an outdated, or even yesterday's, report happened.
(I'm having some issues with dependency setup right now, so I wasn't able to test this one, but it should be simple enough.)

It used to do this but I changed it to reduce the noise. Is there anything else we could do to make it easier to see when the match started maybe?

I don't think that would help much - when I do a chat search, I mostly want to match up the report date from https://server4.beyondallreason.info/moderation/report/<report_id> (which has the full datetime regardless of when the report was made) to a chat entry, to figure out the exact moment someone felt compelled to report someone. Having these visible on the chat search, while I get it's a trade-off vs the noise, would still make it a fair bit easier.


Fix wrong path for registerFactories() in prvoider.stub

Previously, app was search for factories in Module/Providers/Database/factories instead of Module/Database/factories

Whoops, nice catch. Thanks! 👍


add "How to sign unsigned commits" to CONTRIBUTING.md

This is for the case where you already made commits but want to update them to be signed.

Hello..can I add my suggestion for this?


Fix for issues with startup and multiple monitors on windows.

This fixes an issue with the qt application sometimes disappearing at startup. This was my original code and has been accepted by unlimited and core. I am just making certain that classic gets the fixes too :-).
This change is discussed at length in the following PRs:
https://github.com/BitcoinUnlimited/BitcoinUnlimited/pull/419. https://github.com/bitcoin/bitcoin/pull/10156.

First, thanks a lot for this PR. I just tried it and I like the idea. Quite simple solution.
Well. Hmm, if its already been accepted by other clients, then should I be ashamed or proud that I found a bug in this code?
Its a subtle one, but I think you'll agree we should account for it.
The Screen variable is a QRect and you use it like it were a QSize. In other words, you ignore the x()/y() of the screen.
In case you have two screens where the size (in pixels) of those two are not the same, you can end up with an offset.  As an example:
QRect(1920x 1200+0+1000);
Your code forgets to add the y pos of 1000 and thus it doesn't end up centred on this screen. On high-res laptops with a main screen being the relatively low-res flatscreen TV (like the example I choose) it may even be partly off-screen.
To fix this I suggest using the QRectF::center() instead. Or, in code;
QPoint defaultPos = screen.center() - QPoint(defaultSize.width() / 2, defaultSize.height() / 2);

Is this something you want to update? Or should I merge it and fix it after? Your choice :)

Not sure if this is something you are willing to do, but it would be awesome if you could make this fix against the 1.2 branch as that is the bugfix branch.

@zander. I shall add your idea and push it into the 1.2 branch. It may take a day or two.

thank you!


support for running in docker container and access through SSL (https://)

The simple instructions to install and run razzle are specific to running AND browsing the app on localmachine:3000. i am evaulating this for the first time for three days with no progress and no notable error logs, though everything that can fail has failed (hot module reload not working, errors adding new npm packages, and bundle.js mixed-content errors failing to load styling as I am running https). There are no useful error logs to suggest anything is misconfigured and it was broken on installation (into a docker container behind an nginx proxy). Has anyone had any success running from a docker container and accessing the site through a remote browser that does not have direct access to localhost on the razzle server?
thanks

Check out: https://github.com/fivethreeo/react-ssr-django-jwt-docker, only hmr on client not working, yet hopefully.

Thanks I would like to try to start with the most basic installation alongside a set of working Next and CRA live reload apps.
Is there some docker config missing from the documentation? I can't see any notes on docker support or for installation prerequisite . I cant find any notes for using on non-localhost and a bit puzzled by why the home page is trying to pull svg files and bundle.js files from an http address?
My setup: I am using nginx to redirect Razzle, CRA and Next domain requests to respective ports and this automatically takes care of 3000 redirect. To enable 3001 access I have had to (i) publish 3001 port in docker-compose (ii) add Razzle's container hostname as a DNS record, and (iii) add the hostname to the .env file containing Razzles env vars.
I am glad of your assistance, thank you for your patience.

For the address it is trying to pull the svg from. Look here https://github.com/fivethreeo/react-ssr-django-jwt-docker/blob/master/docker-compose.staging.yml#L51, theese are env vars that need to be set at build time. For 3001 port it is a bit more cumbersome since it is always process.env.PORT at build time + 1 or current port in browser + 1. You could put 3001 at another path and use sub filter to replace the port in the code https://github.com/jaredpalmer/razzle/blob/master/packages/razzle-dev-utils/webpackHotDevClient.js#L62 or fork razzle. I wish there was a better way. ;)

thanks based on your info i have set up an .env file in the project root directory (same level as package.json)
HOST='apps.syntapse.co.uk'
PORT=3000
VERBOSE='true'
NODE_ENV='production'
BUILD_TARGET='server'
PUBLIC_PATH: 'https://razzle.syntapse.co.uk'
CLIENT_PUBLIC_PATH: 'https://apps.syntapse.co.uk'

the log is still pointing to http address.
0|razzle   | ℹ ｢wds｣: webpack output is served from undefined
0|razzle   | ℹ ｢wds｣: Content not from webpack is served from /app/razzle/razzle
0|razzle   | ℹ ｢wds｣: 404s will fallback to /index.html
0|razzle   | ✔ Client: Compiled successfully in 6.79s
0|razzle   | ℹ Compiling Server
0|razzle   | ✔ Server: Compiled successfully in 375.09ms
0|razzle   | * TRACE: js = http://apps.syntapse.co.uk:3001/static/js/bundle.js
0|razzle   | * TRACE: css=undefined
0|razzle   | * TRACE: public dir=/app/razzle/razzle/public
0|razzle   | ✅  Server-side HMR Enabled!
0|razzle   | 🚀 started
0|razzle   |
0|razzle   | ><EMAIL_ADDRESS>start /app/razzle/razzle
0|razzle   | > razzle start
0|razzle   |

0|razzle  |  WAIT  Compiling...
0|razzle  | ℹ Compiling Client
0|razzle  | ℹ ｢wds｣: Project is running at http://apps.syntapse.co.uk:3001/
0|razzle  | ℹ ｢wds｣: webpack output is served from undefined
0|razzle  | ℹ ｢wds｣: Content not from webpack is served from /app/razzle/razzle
0|razzle  | ℹ ｢wds｣: 404s will fallback to /index.html
0|razzle  | ✔ Client: Compiled successfully in 4.43s
0|razzle  | ℹ Compiling Server
0|razzle  | ✔ Server: Compiled successfully in 375.51ms
0|razzle  | Warning: Please use `require("react-router-dom").Route` instead of `require("react-router-dom/Rout0|razzle  | Warning: Please use `require("react-router-dom").Switch` instead of `require("react-router-dom/Swi0|razzle  | * TRACE: js = http://apps.syntapse.co.uk:3001/static/js/bundle.js
0|razzle  | * TRACE: css=undefined
0|razzle  | * TRACE: public dir=/app/razzle/razzle/public
0|razzle  | ✅  Server-side HMR Enabled!
0|razzle  | 🚀 started

and bundle.js remains blocked on the SSL page.
Request URL: http://apps.syntapse.co.uk:3001/static/js/bundle.js
Referrer Policy: no-referrer-when-downgrade
Razzle is not respecting the client path.

working! @fivethreeo sincere and massive thanks for you help.

Got a fix for you here :)
https://gist.github.com/fivethreeo/48558e2bf0ec228af4b4763d4199df0c


KOGITO-4523 - Set dmn model version in cloud event

Related PR on kogito-apps: https://github.com/kiegroup/kogito-apps/pull/672
Many thanks for submitting your Pull Request :heart:!
Please make sure that your PR meets the following requirements:

[ ] You have read the contributors guide
[ ] Your code is properly formatted according to this configuration
[ ] Pull Request title is properly formatted: KOGITO-XYZ Subject
[ ] Pull Request title contains the target branch if not targeting master: [0.9.x] KOGITO-XYZ Subject
[ ] Pull Request contains link to the JIRA issue
[ ] Pull Request contains link to any dependent or related Pull Request
[ ] Pull Request contains description of the issue
[ ] Pull Request does not include fixes for issues other than the main ticket

The Linux check has failed. Please check the logs.

The Linux check has failed. Please check the logs.

The Linux check has failed. Please check the logs.

Jenkins retest this please

The Linux check has failed. Please check the logs.

The Linux check has failed. Please check the logs.

The Linux check has failed. Please check the logs.

Jenkins retest this please

The Linux check has failed. Please check the logs.

Jenkins retest this please

The Linux check has failed. Please check the logs.

Jenkins retest this please

The Linux check has failed. Please check the logs.

Jenkins retest this please

The Linux check has failed. Please check the logs.

The Linux check has failed. Please check the logs.

Jenkins retest this please

The Linux check is successful.

The Linux check is successful.


Why we need to pin app service to a zone using ASE?

Can you please help me to understand this feature?
What I need is zone redundancy to achieve high availability for app service.
Are we saying that the only option is to use ASE and pin the app service in a specific zone and create multiple app services in multiple zones.

What about the app services running in premium app service plan? Azure behind the scene does not support zone redundant infrastructure?
[Enter feedback here]

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

ID: d05795f9-5027-e37d-7862-29c72f5c4edb
Version Independent ID: 7ccc48be-64dd-10ed-a0bf-e0c90db04bed
Content: Availability Zone support for App Service Environments - Azure App Service Environment
Content Source: articles/app-service/environment/zone-redundancy.md
Service: app-service
GitHub Login: @ccompy
Microsoft Alias: ccompy

@RonnieQuan Thank you for your feedback! We will review and update as appropriate.

@RonnieQuan

An Availability Zone is a high-availability offering that protects your applications and data from datacenter failures. Availability Zones are unique physical locations within an Azure region. Each zone is made up of one or more datacenters equipped with independent power, cooling, and networking. To ensure resiliency, there's a minimum of three separate zones in all enabled regions. The physical separation of Availability Zones within a region protects applications and data from datacenter failures. Zone-redundant services replicate your applications and data across Availability Zones to protect from single-points-of-failure. With Availability Zones, Azure offers industry best 99.99% VM uptime SLA. The full Azure SLA explains the guaranteed availability of Azure as a whole. Source

In more layman's terms, availability zones allow for you to achieve higher availability without losing performance in the event of a failover. If a majority of your customers are located near the Central US datacenter and you are using Azure Traffic Manager for failover routing to a backup copy of your web app in West US, this could add additional latency and requires you to not only make your web app available in a second region but also your other resources as it's not a best practice to have your web app and database in different regions. Azure regions are located at least 500 miles apart to ensure resiliency. This distance can lead to latency or slower performance for your users. So one advantage is that you get to keep your backup web app close to the location of your primary web app so your customers shouldn't perceive a difference in their experience.
Using availability zones also ensures both instances of your app will not be updated at the same time. That way, if there is an issue with the update, you should still have a functional backup web app.
You do not need a load balancer for availability zones. The ILB ASE already includes a load balancer that will manage routing.
As of right now, multi-tenant web apps (non App Service Environment web apps), such as those on the premium tier that you mentioned, are not able to use availability zones. With multi-tenant web apps, you would need to deploy a primary and back up web app and use something like Azure Traffic Manager to achieve resiliency.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Bug:  'vaex.ext.bqplot' has no attribute 'get_class'

Hi,
The plot_bq doesn't seem to work. See code and error message below. There doesn't appear to be a get_class function in the repo.  I have noticed that you mentioned in the code comments that the bqplot in Vaex is experimental so perhaps I should just be using matplotlib for now.  However, the bqplot demo in the 'Billion stars in Jupyter' presentation you did is very impressive!  I tried to replicate it but I can't.
import vaex as vx
dataset = vx.datasets.helmi_de_zeeuw.fetch() # get a cup of coffee while this downloads
dataset.plot_bq("x", "y", f="log1p", show=True)

AttributeError                            Traceback (most recent call last)
 in ()
----> 1 dataset.plot_bq("x", "y", f="log1p", show=True)
c:\users\doug\documents\github\vaexdr\vaex\vaex\dataset.py in plot_bq(self, x, y, grid, shape, limits, what, figsize, f, figure_key, fig, axes, xlabel, ylabel, title, show, selection, colormap, grid_limits, normalize, grid_before, what_kwargs, type, scales, tool_select, bq_cleanup, **kwargs)
1991 			 **kwargs):
1992                 import vaex.ext.bqplot
-> 1993                 cls = vaex.ext.bqplot.get_class(type)
1994 		plot2d = cls(dataset=self, x=x, y=y, grid=grid, shape=shape, limits=limits, what=what,
1995                                 f=f, figure_key=figure_key, fig=fig,
AttributeError: module 'vaex.ext.bqplot' has no attribute 'get_class'

Hi,
ah, I see where this snippet comes from, I should update that. Try this:
dataset.plot_widget("Lz", "E", f="log1p")

cheers,
Maarten

It works! Thankyou.

Thanks again, fixed by 002c9c6bf7f52d2b755ef43bb022a91b0f28c3ae


Training process fails immediately with no useful output details

Problem encountered on https://dotnet.microsoft.com/learn/ml-dotnet/get-started-tutorial/train
Operating System: windows
I've tried to train an image classification model with the 60000 MNIST datasets. I've put the data in the folder structure as written, and in fact, the data preview showed that the data had been read correctly.
The problem occurs when I push "Start Train". After one second it fails with the following message on the status field: "Failed. See more in Output Pane". But in the output it shoes only this:
"|     Trainer                              MicroAccuracy  MacroAccuracy  Duration #Iteration                     |".
Nothing useful for taking further action.

Sorry you've run into this problem Renato! I know the MNIST dataset isn't downloadable in a standard image format.  Based on the fact you got a data preview working it sounds like you unpacked the files.  What image file type did you save them as in the folders?
I'll try to pull this dataset and try it on my machine.

Yes, I unpacked it. The format is jpeg. Anyway, it worked using 10000 images instead of 60000.
The resulting folder from the bigger dataset after the unpacking occupies almost 250 MB against the 40 MB fo the smaller one. Maybe there is a limit in image recognition that I missed.

@LittleLittleCloud Do you know anything about size limits for image recognition?

As far as I know, there's no such limitation in that. I'm gonna double check though

I tried using the corefx-issues-train.tsv file from the MulticlassClassification-GitHubLabeler sample code on my console app. I am getting the same error. No useful information on the Output pane.

Hi,
i'm trying to make a binary-classification model. Data come form a csv file (small sample here dati-avvisi.zip) that has 5 columns:
ID | CODICE-HOST | DESCRIZIONE | TITOLO | Inserito
Inserito should by my label column, and Descrizione and Titolo are my features.
Immediately after i hit train in the model builder i have this:

Can you help my to track down what is the error? Is it a bug?
I've tried with the sentiment analysis example and it work. It work even with another similar dataset I tried yesterday. I really can't see where is the problem....
I use Visual studio 2019 updated to the latest version.

Related to #402

@KodaPOwER Sorry for the delayed response!
Are you saying it worked for Sentiment Analysis but didn't work for Custom -> Binary Classification?

@KodaPOwER I'm able to reproduce the bug you report. I will take a look, thanks for your feedback

@LittleLittleCloud ok, thank you for the hint!
I already tried with a tab separated dataset but i get the same error, maybe the problem is the \n chars and the ones with 6 cols. Especially this one is strange because the dataset it's an export directly from JetBrains DataGrip results window...
Maybe can also be useful to have a meaningful error message from the tool :)
Regards, and thanks again for your help

The latest released ModelBuilder doesn't include the error handler PR, therefore it won't through an error window to notify users of the fail reason.  That PR will be included in the next release.


annotating vcf with intersect

I'm trying to annotate a vcf with a number of bedfiles. Not hard but it would be good to confirm the strategy with bedtools.
Objective
a) The annotations are sorted but NOT collapsed/merged to avoid annotating positions with regions that don't overlap the position
b) The output should have the same number of lines and order as the original vcf
c) Multiple hits for a position should have distinct values collapsed
d) Only chr, start,stop,annocol1,annocolx are required in the output. i.e. no score,name in -a and no chr, start, stop in -b
Challenge

intersect results in multiple hits/rows for a single position in the vcf when the annotations aren't merged. Seems like the easiest thing to do is do a merge on the output, right?
Intersect on a bed3 (just the index) and bed4 results in a ragged output (8 columns where there is no  match and 7 where there is). Is there a way to avoid defaulting to a bed5 internally or should I add dummy columns to a bed3? See example.
merge requires enumeration of all the columns but these are dynamically changing. Any way to default to all columns if -o is set?

Strategy
tmp1.bed
tmp2.bed
# bcftools view a.vcf |  awk '{OFS="\t";print $1, $2-1, $2}'  > tmp1.bed
bedtools intersect -loj  -a tmp1.bed -b tmp2.bed > out1.bed
bedtools merge -i out1.bed -o distinct -d delim -c 5 > out2.bed

One suggestion would be to have a look at the vcfanno project from our lab: https://github.com/brentp/vcfanno

One suggestion would be to have a look at the vcfanno project from our lab: https://github.com/brentp/vcfanno


Add properties metadata

linked to issue #20, allow adding units and description (optional) to properties and show them in the map and the info table

FYI, if you write fix #20 in the first comment, this will automatically close the issue when the PR is merged. https://docs.github.com/en/free-pro-team@latest/github/managing-your-work-on-github/linking-a-pull-request-to-an-issue#linking-a-pull-request-to-an-issue-using-a-keyword

Also TBD before merging this:

[ ] Add a visual indication that the description is available by hovering over the name in the properties panel
[ ] Update the example files with unit & description
[ ] rebase/squash unnecessary commits. I can help with this if you want!

@cbenmahm updated the example files to include description & unit, and they are now online!

FYI, I'm liking a lot the ISO standard that prescribes that units in axes
should be indicated as quantity/unit (e.g. volume/angstrom3).
Incidentally, do we have support for subscripts/superscripts?
On Mon, 26 Oct 2020 at 16:13, Guillaume Fraux<EMAIL_ADDRESS>wrote:

@Luthaf commented on this pull request.
In src/map/map.ts
https://github.com/cosmo-epfl/chemiscope/pull/88#discussion_r512038197:

     layout.xaxis.type = this._options.x.scale.value;

     layout.yaxis.type = this._options.y.scale.value;

   layout.scene.xaxis.title = this._options.x.property.value;

   layout.scene.yaxis.title = this._options.y.property.value;

   layout.scene.zaxis.title = this._options.z.property.value;

   layout.scene.xaxis.title = this._title(this._options.x.property.value);

   layout.scene.yaxis.title = this._title(this._options.y.property.value);

   layout.scene.zaxis.title = this._options.z.property.value; // using this._title(this._options.z.property.value) here seems to cause troubles
   layout.coloraxis.colorscale = this._options.colorScale();
   layout.coloraxis.cmin = this._options.color.min.value;
   layout.coloraxis.cmax = this._options.color.max.value;

We may also want to display the unit in the color bar title, so below in
layout.coloraxis.colorbar.title.text; and the in callbacks. What do you
think?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/cosmo-epfl/chemiscope/pull/88#pullrequestreview-516881188,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIREZYLJMP6OFXVOCBLUPTSMWGZBANCNFSM4SHCNMQA
.

Incidentally, do we have support for subscripts/superscripts?

We don't interpret the "unit" string at all, so A^3 will not be rendered as A³. However, we pass unicode characters through without touching them, so users can use unicode superscript

can plotly format labels using latex? I think that it might be easier for
people (and similar to what you do in matplotlib) than having them type
unicode characters.
On Tue, 27 Oct 2020 at 14:26, Guillaume Fraux<EMAIL_ADDRESS>wrote:

Incidentally, do we have support for subscripts/superscripts?
We don't interpret the "unit" string at all, so A^3 will not be rendered
as A³. However, we pass unicode characters through without touching them,
so users can use unicode superscript
https://en.wikipedia.org/wiki/Unicode_subscripts_and_superscripts
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/cosmo-epfl/chemiscope/pull/88#issuecomment-717241339,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIREZ5GV7H5EDCAQFJZJ3DSM3C6TANCNFSM4SHCNMQA
.

TIL that plotly supports latex: https://plotly.com/javascript/LaTeX/. So it should work fine for axis titles, and maybe colorbar title. The remaining issue concerns the other places where we would like to output units, i.e. the property table and maybe the <select> in options. There we would have to render LaTeX manually.
I don't know if we can piggy-back on Plotly's LaTeX support or if we need to include another dependency such as KaTeX to enable this, I'll have a quick look.
Another thing is that if we start supporting LaTeX in units, why not support it in meta.references, meta.description and friends? Overall, I think that LaTeX support might be best left to a future PR.

So looks like plotly is using with MathJax for latex rendering but do not included it by default. This means we could render latex manually/automagically by setting up MathJax and maybe KaTeX manually ourself.
However, Latex support inside the plots seems a bit hit or miss: https://github.com/plotly/plotly.js/issues/4605 and https://github.com/plotly/plotly.js/issues/608. In particular, LaTex rendering does not work at all for 3D plots, and would be very hard to fix on the plotly side.
Given that this seems non-trivial to implement and ensure that it keeps working, I think we should leave latex support for another PR.

sounds good, but would be a very nice feature to have - perhaps could you
open a feature issue for that?
On Tue, 27 Oct 2020 at 15:40, Guillaume Fraux<EMAIL_ADDRESS>wrote:

So looks like plotly is using with MathJax for latex rendering but do not
included it by default. This means we could render latex
manually/automagically by setting up MathJax and maybe KaTeX manually
ourself.
However, Latex support inside the plots seems a bit hit or miss:
plotly/plotly.js#4605 https://github.com/plotly/plotly.js/issues/4605
and plotly/plotly.js#608 https://github.com/plotly/plotly.js/issues/608.
In particular, LaTex rendering does not work at all for 3D plots, and would
be very hard to fix on the plotly side.
Given that this seems non-trivial to implement and ensure that it keeps
working, I think we should leave latex support for another PR.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/cosmo-epfl/chemiscope/pull/88#issuecomment-717291004,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIREZ27GAF2R6WNCDQQUB3SM3LW7ANCNFSM4SHCNMQA
.

Thanks a lot for implementing this @cbenmahm! And sorry for the delay in review


Grid Filters - Projects

we need the same filter functionality across all application as ones implemented in the Workplan Grid

As a (type of user)
I want (goal)
so that (reason/ why/ value)
Acceptance Criteria
Scenario 1:
List Management > Projects

Name (Search as in the Workplanpage with yellow highlight)
Type (Multiselect)
Subtype (Multiselect)
ENV Region (Multiselect)
Active (Title 'Active' should be changed to 'Status' (Multiselect - Select All, Active, Inactive)

Scenario 2:
xxxxxx
Validation Rules?
(If yes, list here)
Screenshot
(Link above will always be the most up-to-date version.)

There might be a scalability issue regarding filters and tables . I noticed some lag when filters are applied in the projects list, it's a long list. We might need to discuss this @Connor1892


InvalidOperationException

Hello all,
I get this error when running 1.3.1 on a NET 4 DLL compiled with /CLR.

[ERROR] Unknown error occurred.
Exception: System.InvalidOperationException: Operation not valid due to the current status of the object.
in dnlib.DotNet.Writer.MethodBodyChunks.WriteReusedMethodBodies(DataWriter writer, Int64 destStreamBaseOffset)
in dnlib.DotNet.Writer.NativeModuleWriter.WriteFile()
in dnlib.DotNet.Writer.NativeModuleWriter.WriteImpl()
in dnlib.DotNet.Writer.ModuleWriterBase.Write(Stream dest)
in Confuser.Core.ConfuserEngine.WriteModule(ConfuserContext context)
in Confuser.Core.ProtectionPipeline.ExecuteStage(PipelineStage stage, Action1 func, Func1 targets, ConfuserContext context)
in Confuser.Core.ConfuserEngine.RunPipeline(ProtectionPipeline pipeline, ConfuserContext context)
in Confuser.Core.ConfuserEngine.RunInternal(ConfuserParameters parameters, CancellationToken token)
Failed at 09:54, 0:23 elapsed.

Can you help on this? any hints?

I never tested ConfuserEx with C++/CLR assemblies. It's only meant to handle pure managed assemblies and not mixed mode assenblies. This error looks like a problem when writing the assembly back out so I am guessing the problem is Confuser either corrupting the assembly or dnlib not reading it correctly.
The question is actually why you want to obfuscate the assembly. The optimization of the C++ compiler should make the code sufficiently unreadable.

Yes, no problem. I can use ConfuserEx 1.0 (from the original project) to get this.
The error sound strange to me too, it seems there's a writing problem but, as I said, previous version works perfectly.

Okay. I'll honestly can't tell you what goes wrong here. I'll need to setup a test project and see what's what. Is there any chance you could provide a minimal example producing the issue?

I looked in some more detail into this issue and found the tamper protection to be the only one leading to this error. Currently that protection does not work with the C++/CLI assembly, all others work fine it seems. I only used a very simple project as reference for the tests, so it may be different for you. It's a thing you can try in order to work around your issue.

Thank you so much for your help, I'll try to exclude tamper protection

Excluding tamper protection worked, thanks. I'm wondering why there was no need to exclude it with ConfuserEx 1.0. can you tell something on that?


Unify browser global names

There are a few conventions in use right now, and only a few are in line with guidelines. I think people prefer not to have heavily namespaced global names as the guidelines presently state. Therefore, I will update the guidelines to match with what most feature crews seem to have arrived at:

start with the package name
replace @azure with az
remove dashes

Examples:

azkeyvaultkeys
azstorageblob

Is this still a requirement now that we don't bundle the browser versions of the SDK ourselves?


Bye Bye tracer paket

Task/Issue URL: https://app.asana.com/0/0/1201710091654544/f
Description
We fixed the memory leak with the tracer packet in https://github.com/duckduckgo/Android/pull/1702.
But because the tracer packet is highly invasive and we've seen how can affect the health of the VPN and because we haven't seen it results in true positives while testing internally, we're removing it for now not to spend more time debugging it and focus on the other metrics that relate to real system errors.
See here for more context.
Why fixing and then removing?

In case we want to come back to it later, it is simpler to just revert this PR and be sure the memory leak is not there anymore

Steps to test this PR
Feature 1

[ ] Install from this branch
[ ] enable AppTP
[ ] verify that the diagnostrics screen doesn't have any tracer packet metrics reported
[ ] verify in the logcat that nothing comes back when filtering by tracer
[ ] smoke tests on AppTP, open apps, see they load, verify AppTP blocks trackers

Current dependencies on/for this PR:

develop

PR #1703  (this PR)

This comment was auto-generated by Graphite.


Add Google Analytics

Could it be possible to add Google Analytics to track the section of the documentation they are viewing?

Probably? The existing code that determines the currently scrolled section is in source/javascripts/app/_toc.js


Disabled Menuitem, disables nested Tooltip within Select

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Steps to reproduce 🕹
Link to live example:
Steps:

Current behavior 😯
I have a Select component with nested options. When one of the MenuItems is disabled, it also disables the Tooltip that I would like to render only when the MenuItem is disabled. Currently, when the mouse hover's over the disabled selection, no Tooltip is shown.
  <Select
        label="Sort......."
        value={sort}
        onChange={handleSortChange}
      >
//...
        <MenuItem
          value="priceAscending"
          disabled={!hasStartLocation}
        >
          <SortTooltip
            title="select starting location for price sort"
            disabled={hasStartLocation}
          >
            Price: Low to High
          </SortTooltip>
        </MenuItem>

        <MenuItem
          value="priceDescending"
          disabled={!hasStartLocation}
        >
          <SortTooltip
            title="select starting location for price sort"
            disabled={hasStartLocation}
          >
            Price: High to Low
          </SortTooltip>
        </MenuItem>
//...
      </Select>

      function SortTooltip({ title = '', children, sx, disabled = false, ...rest }) {
  return (
    <Tooltip
      title={title}
      arrow
      followCursor
      TransitionComponent={Zoom}
      placement="left"
      disableFocusListener={disabled}
      disableHoverListener={disabled}
      disableInteractive={disabled}
      disableTouchListener={disabled}
      sx={{ ...sx }}
      {...rest}
    >
      {children}
    </Tooltip>
  );
}

Expected behavior 🤔
The idea is when an option is disabled, a Tooltip pops up to show the user how to interact with the Select Menuitem
Context 🔦
No response
Your environment 🌎
attempted running npx @mui/envinfo and returned this error
npm ERR! code ENOENT
npm ERR! syscall lstat
npm ERR! path /Users/Kirill/Documents/VSCode
npm ERR! errno -2
npm ERR! enoent ENOENT: no such file or directory, lstat '/Users/Kirill/Documents/VSCode'
npm ERR! enoent This is related to npm not being able to find a file.
npm ERR! enoent```

npm ERR!      /Users/Kirill/.npm/_logs/2023-03-19T22_14_49_104Z-debug-0.log

  npx @mui/envinfo

Don't forget to mention which browser you used.
Output from npx @mui/envinfo goes here.

Can you please provide a CodeSandbox? You can fork this template: https://material-ui.com/r/issue-template-latest

This is the basic premise of the component. For simplicity, I defined hasStartLocation to false, as the MenuItems should be disabled when hasStartLocation comes back undefined, and ideally, this is when the Tooltip is available as the user hovers over the disabled MenuItems
https://codesandbox.io/s/vigilant-aryabhata-jep8ut?file=/src/App.jsx

Hey @hbjORbj, did that sandbox help clarify the bug at all?

any updates on this issue, team?

Material UI's Select does not support these scenarios.
However, we offer an unstyled version of the Select in Base UI that should work fine in these cases. The downside is that you have to style it on your own.
See https://codesandbox.io/s/vigilant-joji-wq8d97?file=/Demo.js for an example.
We will implement the Material UI's Select with Base UI in the future version. Until then, we likely won't have the capacity to fix it. But if someone from the community would be interested in creating a PR with the fix, we'll gladly accept it.

Ok good to know thanks, we'll wait for it!


Consider using Mima instead of Shrinkwrap Resolver

When checking out the JResolve repository, I've found the following comment:

Shrinkwrap OTOH moved to MIMA https://github.com/maveniverse/mima and new upcoming CLI is https://github.com/maveniverse/toolbox

(https://github.com/bowbahdoe/jresolve/issues/3)

@Thihup Go for it!

See #3795 for why this is being closed as won't fix


Episode "Supper Club × Bramus Van Damme on CSS" show notes not loading

Probably because the API is returning null instead of the show number.
Great show!

Oh dang - looks like I forgot to include the episode number on that one. I'll fix.


Please be advised - The final example is, IMO, incorrect

Mr. Pickett:
First of all, thank you for this tutorial.  It is helping me.
I am NO expert in Web development, but I have been a coder for 45 years.
I ran into an issue on the final part of this page. Here Rick is discussing the use of parameters.
After working with the "HelloWorld" Controller, the tutorial discusses the subject of Parameters.  It states that the "id" can be utilized as a parameter and therefore returned by default.  Unfortunately, I don't think it does.  I could not get this to work correctly until I moved this "Welcome" GET method to the "Home" Controller and matched the call to reflect this.
As in
"https://localhost:XXXX/Home/Welcome/3?UserName=Thom"
instead of
"https://localhost:XXXX/HelloWorld/Welcome/3?UserName=Thom"
What is shown next is exactly what is in the default app.MapControllerRoute in Program.cs.
app.MapControllerRoute(
name: "default",
pattern: "{controller=Home}/{action=Index}/{id?}");
Notice that this still points to "Home".  As written, the GET always returns 0.  Creating a "Welcome" GET in the "Home" Controller, I receive a non-zero value.
Respectfully,
Thom Simone
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 20264150-9ac7-2b21-3314-64ffa114410f
Version Independent ID: 3477519b-1bc2-73be-33e3-63067383d426
Content: Part 2, add a controller to an ASP.NET Core MVC app
Content Source: aspnetcore/tutorials/first-mvc-app/adding-controller.md
Product: aspnet-core
Technology: aspnetcore-tutorials
GitHub Login: @wadepickett
Microsoft Alias: wpickett

@ThomSimone, thanks for the heads up!  We will get it fixed.

I would like to contribute. 😃

@wadepickett  @postmeback  the instructions are correct, this has worked since ASP.NET MVC (pre core).

@ThomSimone, thanks for the heads up! We will get it fixed.

Refrain from acknowledging a problem until you've confirmed it.


Plugin does not work with Android plugin

I'm using the plugin in a project and it works great with java projects, however, it fails when I try to apply it to a project with the android plugin:
Error:The 'java' plugin has been applied, but it is not compatible with the Android plugins.
I see that the Java plugin is enforced in FormatPlugin.groovy, is this necessary? Is there any workaround?

I enforced the usage of Java plugin to directly retrieve the Java sourcesets, this could probably be removed because it is very likely that you will explicitly apply the Java plugin in your project anyway.
Then the files to format, could be by default if not set and if Java plugin is available the Java sourcesets, we could also check for the Android plugin and use those if the plugin is applied too.
I did those changes in the android branch of the project, you can check it out and run gradle install
then you'll have to use the version 1.3 of the plugin, with mavenLocal() added to your buildscript repositories.


Upgrade Newtonsoft.Json from 13.0.1 to 13.0.3

.NET repos currently have a mix of Newtonsoft.Json versions that we're trying to get consolidated into the latest version. This is particularly necessary for the .NET's source-build which can only reference one version of a dependency.
This takes the commit from https://github.com/microsoft/vstest/pull/4680 since I don't have permission to push directly to the PR's branch.
Related to https://github.com/dotnet/source-build-externals/pull/195

cc @MichaelSimons for review

cc @Evangelink for review

Hi @mthalman,
We will have to discuss this change with the team. Because the newtonsoft dll is shipped in the user bin, any update is considered as a breaking change so I am not sure we can move this forward right now.

Hi @mthalman,
We will have to discuss this change with the team. Because the newtonsoft dll is shipped in the user bin, any update is considered as a breaking change so I am not sure we can move this forward right now.

If this change isn't accepted, then SourceBuildPrebuiltBaseline.xml will need to be updated to include Newtonsoft.Json/13.0.1. That will prevent it from being reported as a prebuilt which is currently causing #4680 to fail.

Alternatively would it be an option to conditionally reference 13.0.3 for source-build?  The benefit of this would be the repo leg would mimic the product source-build.

We will have to discuss this change with the team. Because the newtonsoft dll is shipped in the user bin, any update is considered as a breaking change so I am not sure we can move this forward right now.

@Evangelink can you please explain why upgrading an assembly to the latest patch version is considered a breaking change? I totally get your point for major/minor version upgrades but patch version increments should be non-breaking.

The risk here is that we get included into the test project if the tested library also depends on Newtonsoft.Json, we will update the version of the dependency to a newer one that has more fixes, but the deployed product will not have that fixed dll.
So we should be doing those upgrades cautiously. If this was an update from 13 to 14 I would be more careful.
How often do you plan to update your consolidated version? Because I would prefer to not always push for the latest version of newtonsoft json, unless there is a vulnerability reported.

Interesting. I would have assumed that with the correct isolation, dependencies shouldn't overlap, i.e. by using AssemblyLoadContext to load the test assembly by the runner. Isn't that possible here? One more side question, why do you still use Newtonsoft.Json instead of System.Text.Json?

Interesting. I would have assumed that with the correct isolation, dependencies shouldn't overlap, i.e. by using AssemblyLoadContext to load the test assembly by the runner.

If the architecture was different this would be true, but for .NET the testhost is installed into the project as a nuget, and it claims dependencies on newtonsoft.json, so you will end up with only 1 copy of the dll in the output folder, the newer one.
If the testhost was shipped with vstest.console and loaded from a different location (and if assembly load context was used in the first place), then we might be able to isolate it.  But it is not the case, and we don't plan to change it.

One more side question, why do you still use Newtonsoft.Json instead of System.Text.Json?

There are features that are missing from system.text.json (e.g. generic attributes for annotations that don't depend on the library), it is not built into .net framework, and types from Newtonsoft.Json are part of our public communications api, and we don't know what would break if we remove it.
This is a thread where this question was also asked:  https://github.com/microsoft/vstest/issues/2488#issuecomment-1155072927

Is it okay if this is merged in to 17.9 and go into net9, instead of doing this change in 17.8, and potentially breaking net8 which is nearing release?

Is it okay if this is merged in to 17.9 and go into net9, instead of doing this change in 17.8, and potentially breaking net8 which is nearing release?

@MichaelSimons or @mthalman would need to speak to that but I'm pretty sure that this needs to happen as part of .NET 8 because Newtonsoft.Json in https://github.com/dotnet/source-build-externals was updated in the main branch which applies to .NET 8.

We can use the suggestion from @MichaelSimons to conditionally use 13.0.3 only for source-build and make that change for .NET 8. For .NET 9, we can have it use 13.0.3 for everything.
@nohwnd - Which branches should I be targeting to make these two changes then? It doesn't look like there's a 17.9 branch. And, AFAIK, builds from the main branch here are flowing into .NET 8 SDK.

@mthalman  yes that is true, I wanted to snap to 17.8, and make main 17.9, but I was told that it is too early for that. So please target main.

@mthalman yes that is true, I wanted to snap to 17.8, and make main 17.9, but I was told that it is too early for that. So please target main.

Ok. Already targeting main so should be good.


profiler-troubleshooting.md: add note about consumption plan not being supported

Because I ran into this problem.
It'd be nice if the portal called this out too.

@hhunter-ms
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


docs: update Terraform.gitignore

Reasons for making this change:
Adding tfplan option for command terraform plan -out=tfplan -input=false used in automation.
Links to documentation supporting these rule changes:
https://learn.hashicorp.com/terraform/development/running-terraform-in-automation

@nguyendviet thanks!


[1.16.5] Server crash on startup with Infernal Expansion

Describe the bug
The server crashes when I start a fresh new server.
On a client, the mods works just fine.
When I delete Crossroads 2.6.5 from the server and keep Essentials 2.11.1, the server starts just fine.
When I infernal-expansion 2.4.0 and keep Crossroads and Essentials, the server starts just fine.
To Reproduce [Optional]
Steps to reproduce the behavior:

add the folowing mods to a minecraft forge 36.2.35 server:

Crossroads 2.6.5
Essentials 2.11.1
Infernal Expansion 2.4.0

Start the server and watch it fail

Delete Crossroads or Infernal Expansion

Start the server and watch it succeed

Expected behavior [Optional]
compatibility between the 2 mods so it can start on a server (client works just fine)
I also made an issue on their github: https://github.com/infernalstudios/Infernal-Expansion/issues/366
latest.log
debug.log

Apologies for the issue, but the user is using an outdated version of Infernal Expansion. The issue should be fixed in the latest version of Infernal Expansion!

@Dariensg thank you for the heads up!


Rewrite attention_net forward in torch way

In the original code, forward() in attention_net is written using numpy manipulations and customized hard_nms. If we do torch tracing, it will generate constant node, therefore the generated torchscript is not working for a different input (it sticks with the constant node). To fix this, we need write the model in torch way - we use torchvision.nms and torch.topk with some other manipulations to match the original model.
Have done validation test on this. With the same input, the original model and this fixed model get the same result.

@nicolalandro could you please review on this? Thanks!

Tanks it seems correct and torch way is better than numpy, tank you very much!


Mysql INT type field, the data default value is 0

What steps will reproduce the problem?
Mysql INT type field, the data default value is 0, insert new data using REST, if you do not specify the value, then the default is null, not 0, which is very unfriendly in the interface communication.
What is the expected result?
What do you get instead?
Additional info

Q
A

Yii version
<IP_ADDRESS>

PHP version
7.1.12

Operating system
windownsNT

the data default value is 0

Default in MySQL only? Do you have default validator in your model?

which is very unfriendly in the interface communication

Generally in this case it makes sense to mark MySQL column as NOT NULL

@SilverFire Like some counter fields, I did not set the required fields.That if set in the Model validation rules, I do not know whether to circumvent this situation?

Add ['col', 'default', 'value' => 0] to validation rules

@SilverFire  thx.


Update the notebook image

closed by https://github.com/project-codeflare/codeflare-sdk/commit/06a3a59b1acb532a4b4dd3276c6432801485605f


Incorrect markup in markdown exercise

Currently, the tests and example both specify that the the HTML "bold" tag is <em>. This is incorrect; it would actually render as italic. <strong> renders as bold.
Markdown parsers render bold as <strong>, and italics as <em>; this PR brings the exercise in line with this - <i> has been changed to <em>, <em> has been changed to <strong>.

👍LGTM

Thanks for the catch! This is actually in line with what the exercise describes in the tests in the x-common repo at the moment, so it looks like the data there is incorrect as well. @DanCouper - if you're up for it, it would be good to open a PR there as well so others don't make this same mistake when implementing this exercise! Here's a link to that info: https://github.com/exercism/x-common/blob/master/exercises/markdown/canonical-data.json
All looks good to me, but I'll leave it open for another day or so in case anyone has any other questions or comments 👍

Ach, I forgot to check that, was half asleep this morning, I'll check and PR against that tomorrow when I've got time, then I'll update here depending on feedback. There are a few other things about the test conditions I'd say need changing - I know it's a simplified version of Markdown, but it seems a bit off to have it operate under different rules to the actual thing...I'll have a look at it tomorrow anyway.

@parkerl Yeah, we still need to update this to be a refactoring exercise, but we might as well have correct tests before we do that!


Add contributing guidelines

Added contributing guidelines that cover:

Git branching
Git commit messages
Getting started (e.g., forking)
Building the project (including notes on differences)
Issue reporting
Link to license
Contact information (Discord server and GitHub issues)
File naming conventions

Sources include examples found in the git history and commit messages.
Changes made:

Added CONTRIBUTING.md to the repository.
Updated README.md to include build steps from CONTRIBUTING.md.

That should resolve all the spacing issues. It's frustrating when the Markdown preview in VS Code doesn't match how GitHub renders Markdown.

Thank you @SimonNyvall!


Export ParseBool function from strconv package to template

btw: the first lines of Go in my life :)

Nice work!

Thanks @n-rodriguez!

 This is a very nice tool to have :)


Lens turns blue when channel is deactivated

lens turns blue (unprocessed tiles) whenever a channel is deactivated and then stays like this no matter what is done.

Perhaps also a caching issue. Although similar outcomes have occurred when the viewer is either waiting for new tiles to be loaded or is overwhelmed by voluminous tile requests and simply reverts to the blue/green view. Will investigate.

Simplest fix - inc. tile cache max (from 50 to 100).


[Docs] Add caution about updating helm

What type of PR is this?:
/kind documentation
What this PR does / why we need it:
When attempting to update helm template, we should update /hack/helm_patches/update-helm.sh instead of /charts/hnc/templates/*.yaml.
However there are no indication about that and some contributors was confused.(eg. #312, #380)
In this PR, I've added caution to contributing doc.
Tested:
none

/retest


[Bug]: id_token_hint invalid for user session after some idle time

Preflight Checklist

[X] I could not find a solution in the documentation, the existing issues or discussions
[X] I have joined the ZITADEL chat

Environment
Self-hosted
Version
2.42.0
Database
PostgreSQL
Database Version
15
Describe the problem caused by this bug
https://discord.com/channels/927474939156643850/1178715038136148028
SPA OIDC client integration
I caught this error with my client that integrates with zitadel. The client is a JS library oidc-client-js (ts) with token update functionality in the background.

SilentRenewService._tokenExpiring: Error from signinSilent: Unfortunately, the user may not be logged in and/or additional interaction is required.

Apparently the bug is on the zitadel side as stated in this issue. Could you give me a hint, is it fixed?
https://github.com/dotnet/aspnetcore/issues/50812
Logs 1:
2023/11/28 12:54:49 WARN auth request oidc_error.description="Unfortunately, the user may be not logged in and/or additional interaction is required." oidc_error.type=interaction_required
2023/11/28 12:54:56 WARN auth request oidc_error.description="Unfortunately, the user may be not logged in and/or additional interaction is required." oidc_error.type=interaction_required
2023/11/28 12:55:02 WARN request error oidc_error.parent="token has expired" oidc_error.description="id_token_hint invalid" oidc_error.type=invalid_request

Logs 2 (I'm not sure if this is related, but it may well be):
2023/12/06 07:29:04 WARN request error oidc_error.parent="invalid signature (invalid signature: no possible keys matches)" oidc_error.description="id_token_hint invalid" oidc_error.type=invalid_request
time="2023-12-06T07:56:13Z" level=warning msg="token verifier repo: decrypt access token" caller="/home/runner/work/zitadel/zitadel/internal/authz/repository/eventsourcing/eventstore/token_verifier.go:266" error="ID=APP-ASdgg Message=invalid token"
time="2023-12-06T07:56:13Z" level=warning msg="token verifier repo: decrypt access token" caller="/home/runner/work/zitadel/zitadel/internal/authz/repository/eventsourcing/eventstore/token_verifier.go:266" error="ID=APP-ASdgg Message=invalid token"
time="2023-12-06T07:56:13Z" level=warning msg="token verifier repo: decrypt access token" caller="/home/runner/work/zitadel/zitadel/internal/authz/repository/eventsourcing/eventstore/token_verifier.go:266" error="ID=APP-ASdgg Message=invalid token"
time="2023-12-06T07:56:15Z" level=warning msg="token verifier repo: decrypt access token" caller="/home/runner/work/zitadel/zitadel/internal/authz/repository/eventsourcing/eventstore/token_verifier.go:266" error="ID=APP-ASdgg Message=invalid token"
time="2023-12-06T07:56:15Z" level=warning msg="token verifier repo: decrypt access token" caller="/home/runner/work/zitadel/zitadel/internal/authz/repository/eventsourcing/eventstore/token_verifier.go:266" error="ID=APP-ASdgg Message=invalid token"
2023/12/06 08:39:09 WARN request error oidc_error.parent="invalid signature (invalid signature: no possible keys matches)" oidc_error.description="id_token_hint invalid" oidc_error.type=invalid_request
2023/12/06 08:39:12 WARN request error oidc_error.parent="invalid signature (invalid signature: no possible keys matches)" oidc_error.description="id_token_hint invalid" oidc_error.type=invalid_request

To reproduce

Create project, create app with user-agent SPA settings

Redirects URIs
http://localhost:8080/auth/signinpop/main
http://localhost:8080/auth/signinwin/main
http://localhost:8080/auth/signinsilent/main
then copy client_id

Create a sample vue app and integrate it with package: https://github.com/soukoku/vue-oidc-client
Type login & password, close tab
Wait a some time, may be 1-3-5 hours
Open a new tab with vue sample project and catch an error:

Screenshots
No response
Expected behavior
working normally
Operating System
OS: Mac 13.2.1
Browser: Chrome Version 121.0.6140.0 (Official Build) canary (arm64) or Firefox or any browser
Relevant Configuration
No response
Additional Context
No response

@muhlemmer @livio-a can you help here?

Looks like we just forgot about it: https://github.com/zitadel/zitadel/issues/5232
There are two reasons for this:
Either the token has expired (bug in our OIDC library)
or the the token was signed with a key, that is not available anymore.
The session (incl. access_token) will therefore not be valid anymore.
@hifabienne But looks like more and more people run into this and we should try to find a solution. With the integration of v3 we should now be able to handle it (custom implementation of EndSession)

hei @roquie
I would close this issue to not have to handle the duplicates, i just asked for estimation for the other issue, so we can plan it for a future sprint.

still getting this error , is it solved?  i wonder why it has been closed when there is no solution , everyone is just assuming here what might be the problem

This was closed as duplicate. Refer to the linked issue to see the resolutions we took. #5232


emqttd-v2.0 ssl connection issue

Hello,
I am using emqttd-v2.0-rc.3 and I am using MQTTjs to connect to the broker over SSL.
var client = mqtt.connect({
    port: config.broker.port,
    protocol: 'mqtts',
    host: config.broker.host,
    clientId: config.broker.clientId,
    reconnectPeriod: 6000,
    keepalive: 300,
    rejectUnauthorized: false
});

client.on('connect', function() {
    console.log(arguments);
    console.log('MQTT Connected');
});

client.on('close', function() {
    console.log(arguments);
    console.log('MQTT Disconnected');
});

If i run the above, I get
{ '0': { [Error: read ECONNRESET] code: 'ECONNRESET', errno: 'ECONNRESET', syscall: 'read' } }
MQTT Disconnected

After a few changes in the client setting, with no luck, I ran, emqttd in console mode (./bin/emqttd console) and attempted a connect and this is what I get
(emqttd@<IP_ADDRESS>)1> 05:20:46.314 [error] CRASH REPORT Process <0.1127.0> with 0
 neighbours crashed with reason: {ssl_error,timeout} in esockd_connection:upgrad
e/1 line 113
05:20:46.317 [error] Supervisor 'esockd_connection_sup - <0.1062.0>' had child c
onnection started with emqttd_client:start_link([{max_clientid_len,1024},{max_pa
cket_size,65536},{client_idle_timeout,30}]) at <0.1127.0> exit with reason {ssl_
error,timeout} in context connection_crashed

Then I though that the issue may have been with SSL certificate and ran
$ openssl s_client -connect broker.host:broker.port
and I get
depth=1 C = CN, ST = ZheJiang, L = HangZhou, O = emqtt.io, CN = emqtt.io, emailAddress =<EMAIL_ADDRESS>verify error:num=19:self signed certificate in certificate chain
write:errno=54
---
Certificate chain
 0 s:/C=CN/ST=ZheJiang/L=HangZhou/OU=emqtt.io
<EMAIL_ADDRESS> 1<EMAIL_ADDRESS><EMAIL_ADDRESS>---
Server certificate
-----BEGIN CERTIFICATE-----
MIICZjCCAc+gAwIBAgIJAO89PfgaeHB2MA0GCSqGSIb3DQEBBQUAMHcxCzAJBgNV
BAYTAkNOMREwDwYDVQQIEwhaaGVKaWFuZzERMA8GA1UEBxMISGFuZ1pob3UxETAP
BgNVBAoTCGVtcXR0LmlvMREwDwYDVQQDEwhlbXF0dC5pbzEcMBoGCSqGSIb3DQEJ
ARYNaG9uZ0BlbXF0dC5pbzAeFw0xNjEwMTMwOTEzMTFaFw0xNjExMTIwOTEzMTFa
MEYxCzAJBgNVBAYTAkNOMREwDwYDVQQIDAhaaGVKaWFuZzERMA8GA1UEBwwISGFu
Z1pob3UxETAPBgNVBAsMCGVtcXR0LmlvMIGfMA0GCSqGSIb3DQEBAQUAA4GNADCB
iQKBgQDaOI1oasKjo0JGk5bMIxGInlxbvTuJZ8436u8HY4q8jZ+a4G12+UdTVHRF
d94/ClHWn8WvOvzbxvmSkhninlzdWm1rBLWis3Z2kStmhL77kITEfIImus9pjm5l
OgBxY4+Q7LSEsKYYH+ClYVaLlzO8PILEkBk6xxxq0X7AnCfDfQIDAQABoyswKTAJ
BgNVHRMEAjAAMAsGA1UdDwQEAwIF4DAPBgNVHREECDAGhwR/AAABMA0GCSqGSIb3
DQEBBQUAA4GBAGS1yw1w9H7F4uOaK02mUCHZiV+EBB3gkBBqtAx7TXsmoGgT6ySA
7DwrbX6IH82bhZT4TjouhaPlUPE9pin88d/2kNbRrbZoZDMYGq02mVVRxfLzJqM2
GVlsxebsFFPbYhOaf9TuRR3v13ebga0FrXNke+IGLsYZSM2PZ+F4EvIA
-----END CERTIFICATE-----
subject=/C=CN/ST=ZheJiang/L=HangZhou/OU=emqtt.io
<EMAIL_ADDRESS>---
No client certificate CA names sent
Peer signing digest: SHA512
Server Temp Key: ECDH, P-256, 256 bits
---
SSL handshake has read 1564 bytes and written 126 bytes
---
New, TLSv1/SSLv3, Cipher is ECDHE-RSA-AES256-GCM-SHA384
Server public key is 1024 bit
Secure Renegotiation IS supported
Compression: NONE
Expansion: NONE
No ALPN negotiated
SSL-Session:
    Protocol  : TLSv1.2
    Cipher    : ECDHE-RSA-AES256-GCM-SHA384
    Session-ID: 91B888D43D81E181B2EA0BEEEBBBE9D709DB90223B3651CC8013BA8018F4C409
    Session-ID-ctx:
    Master-Key: F4A3DE559FBC85C53C2B87160CA6F7557032FAA45115CB992DC37D112C3A9E52F0AFF80CC29586570E3447CCA1EE30C7
    Key-Arg   : None
    PSK identity: None
    PSK identity hint: None
    SRP username: None
    Start Time:<PHONE_NUMBER>
    Timeout   : 300 (sec)
    Verify return code: 19 (self signed certificate in certificate chain)
---

Everything looks clean, but not sure why I am not able to connect to MQTTS. MQTT works without issues though.
PS: I have another broker running emqttd with v< 2.0 and the SSL with same certs works fine.
Any help is highly appreciated.
Thanks,
Arvind.

hi
config.broker.port?
Check if the listeners for https is configured in the emq.conf file.

Thank @turtleDeng config.broker.port is matching with mqtt.listener.ssl in emq.config. Or should it match with mqtt.listener.https?

ws match mqtt.listener.http
wss match mqtt.listener.https

mqtt.listener.ssl = 1111 matches MQTTS?

hi
Here is how to configure the https documentation.
http://emqtt.com/docs/v2/config.html#mqtt-websocket-ssl-8084

Thanks. Will try and redo the setup.


Update README links to new documentation

Updated the README file to replace old documentation links with the latest ones, ensuring users have access to the correct resources.

Fixes Issue #448

Thanks @TheDarkW3b ❤


Data Element link to terminology (version numbers)

Data Element link to terminology fixed (#396), however, on selecting a terminology, it displays all terminologies for each branch.
This means that a terminology could be displayed many times.
James is going to try and add branch numbers and also try to see if the list can be shortened.

This looks much better now thanks:

@jamesrwelch - just to confirm; if I wanted to link to CARDIOPULMONARY EXERCISE TEST TYPE, would I select it from main or the finalised branch 2022.7.0?

@AngelaFaulding Always link to items in the same branch you're editing - each branch should be a self-contained instance of the dictionary and shouldn't point to anything in another branch.

Thanks James.


Mortgage Calculator - Issue : #125

Description 📜
This script aims to calculate mortgage interest. It is useful in measuring mortgage payments for banking websites (Indian Banks).
Fixes: #125

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[ ] My changes generates no new warnings.
[x] I'm GSSOC'22 contributor
[ ] Have you added a working video of your calculator ?

Screenshots / Gif (Optional) 📸

That's not a binary file. It's a .png file of a screenshot that's been called into README. So is it not needed? @Chayan-11

I added it up. @Chayan-11

Output is not showing!!!! @prathimacode-hub
ps:) @Chayan-11 Please check the project once! Don't Rush!!

I could see it from my link : https://prathimacode-hub.github.io/Mortgage-Calculator/index.html. Could you check it here with this link? I made some changes to the code and pushed it here too. @vasu-1 @Chayan-11

closing due to inactivity!!


Setting up codescanning for ansible-datadog-callback

Static code analysis on ansible-datadog-callback using Github's Code Scanning.
This PR has Github workflow configs to trigger the Code Scanning on every PR and on push to master branch. As of now only default security based CodeQL queries are configured. We can gradually add queries for Code Quality later. It is a part of Datadog compliance requirements to regularly perform code-scanning on all the customer installed code. As the repository is small, no security alerts were found during the scan.

Thanks for the PR, merging.


Export as PNG not work

I click Export as PNG then open a new window,
but no PNG file.
Why? is it my mistake?
My system environment:
XP Pro SP3
Google Chrome V.29.0.1547.66 m

thanks @Merenon

Works now by downloading


Fixed NullReferenceException in OperationComplexityMiddleware (#5278)

Fixes the operation complexity cache key
Change the comparison to the SyntaxComparison

Closes #5278

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

The fix introduces a new bug as the cache id is for the request but we need one for the operation.

Actually, it was correct as it was and this introduces a bug.

Also, can you add a test that shows how to create the NullRefException?

I have a test of issue one .... in the invoke method the cache ID is wrong...

I have explored the issue more and there is more to it ... I am closing this issue in favor of another PR I have open that will tackle the caching issue more broadly.

#6100

We will still give you credits for the fix, thats why I merged you initial attempt.

Thanks Michael, really appreciate your quick response.


Incorrect PNG file format for Viewer?

Required Info

Camera Model
SR300

Firmware Version
<IP_ADDRESS>

Operating System & Version
Windows 10

Kernel Version (Linux Only)

Does anyone else have this problem?  The 2D PNG files written by the Intel RealSense Viewer (v2.8.1) appear to have an incorrect format. The cam works fine for me and both 2D and 3D images look correct in the viewer. The 3D .PLY files also appear correct. But I cannot get the PNG files generated (sample attached) to open correctly in any program. ImageJ claims that they are RGB images with all R/G/B values being equal and values ranging only from 228-255 (i.e. 8-bit and of limited range). A low-level PNG viewer such as TweakPNG says that the image has 8 bits per sample with grayscale+alpha data. Gimp2 also thinks there's an alpha layer. Nothing I've tried loads a 16-bit grayscale image, as expected. Using TweakPNG I edited the PNG image header forcing the data to load as a 16-bit grayscale image, but the image is still grossly incorrect (values along what is a flat surface swing from 1,000-60,000).
I am a user and a rusty programmer, and have not looked at the source code for the Viewer.

Hello @MichaelHanssonPH ,
The Depth image PNG format is as follows - [Depth Width (pixels), Depth Height (pixels) , Compression (2 bytes), Stride (Width*2)].
Note that in the 'realsense-viewer' the depth data is post-processed to generate a colorized image with palettes like the Matlab's "Jet" and "Classic" to make it "human-readable".
The PNG snapshot, however,  preserves the original raw data (Depth, 16bit per-pixel,unsigned) that can be directly imported into 3rd party tools (e.g Matlab), and therefore is best suited for machine processing.
In case you need the colored snapshot of the depth data you can use  'rs-save-to-disk' demo that does precisely this.

Hi @ev-mp ,
Sorry if I wasn't clear. I understand that the depth data in the viewer is "artificially" colored. I just need to access the depth data using 3rd party tools for analysis to, for example, evaluate potential algorithm steps for edge detection, filtering, etc. I would have thought that the images could be opened directly (as 16-bit grayscale images) in software such as ImageJ but it seems they do not. I tried Python+OpenCV and I have the same problem. I have not tried Matlab.

Hi @MichaelHanssonPH ,
When looking into the polarized snapshot attached, it could actually be an issue of endianness as it looks like that the upper and the lower bytes data are swapped.
The depth raw data that is stored arrives in LSB order, while according to this spec PNG expects the 16bit pixel to be MSB-aligned

...16-bit samples are stored in network byte order (MSB first).

We'll look further into this as it seem to be a software issue. Meanwhile you can try to reverse the endianness or just treat the raw data as a binary stream rather than PNG image.

Hi @ev-mp ,
Thank you. I was beginning to suspect the same thing. Reading the raw data might be challenging, considering the compression, but I suppose I can read it as a PNG and then swap the bytes over. I will try it.

Hi again @ev-mp ,
I can confirm that the IHDR chunk of the PNG file is wrong, indicating an 8-bit grayscale+Alpha sample instead of a 16-bit grayscale sample. I can also confirm that the MSB and LSB bytes for each value is incorrectly ordered.
If I edit the IHDR with TweakPNG and then swap the bytes over (in my case with .byteswap() in Python), the images now appear correct.

I also found the realsense viewer saves the depth image as 32bit depth instead of 16bit. Use the opencv imwrite to save the depth image as "CV_16U" without the issue.

For version 2.8.3 we went back to saving depth snapshots as 24-bit RGB PNG files. Hopefully this is more convenient for most users.

Hi, I also have an issue with the endianness of depth raw data. Reading it as little endian gives the correct depth values for each pixel but doesn't display correctly, and vice versa for big endian.
Little endian gives the correct depth value (499 mm) but displays the image incorrectly (with colormap jet):

Big endian displays the image correctly but gives the wrong depth value (6221 mm >> 499 mm!):

RealSense Viewer snapshot:

So how can you actually read the .raw file in python?


MQTT Messages stop getting published!

Hi,
I'm relatively new to Mqtt and I'm using your MQTTnet for the communication between my Simulation (VB.NET) and a Production System (ThingWorx), however when I run the program from Visual Studio 2013 it will run good but when I use the .exe from \bin\Release the software will be faster and therefore the published MQTT messages will duplicate/triple within the same time (but still not that much, it is around 25-50 MQTT messages per second) and then my program will stop producing after a few seconds because MQTT messages will stop getting published. Even when I run it within VS it will run longer for like 2-3 Minutes and after that MQTT messages will stop getting published... is there  something in the protocol implementation that is time sensitive, like a race condition; that the probability it breaks raises with the speed of the software? Any suggestions how to solve this problem?
Thanks in advance for your help.
Best regards,
Morhaf

Hi @Morhafmass. Two things to do: Can you provide some example code and can you please test this with the latest version of MQTTnet? Thank you.

I'm closing this due to inactivity. Feel free to re-open this if needed.


CSS Modules and @nest

I've recently run into an issue where our cssModules scoped name does not get generated after @nest.
CSS written in src/Button/Button.css
.container {
  padding: .5rem;
}

.value {
  size: 1.5rem;
}

.primary {
  background-color: blue;
  @nest .container & .value {
    color: pink
  }
}

CSS expected
.Button__container__12nb-{padding:.5rem}
.Button__value__3i9_C{size:1.5rem}.Button__primary__3ReFs{background-color:#00f}
.Button__container__12nb- .Button__primary__3ReFs .Button__value__3i9_C{color:pink}

CSS returned
.Button__container__12nb-{padding:.5rem}
.Button__value__3i9_C{size:1.5rem}.Button__primary__3ReFs{background-color:#00f}
.container .Button__primary__3ReFs .value{color:pink}

Please see example repo here: https://github.com/juliewongbandue/nesting_demo

@juliewongbandue this looks like it's actually an issue with https://github.com/egoist/rollup-plugin-postcss and how it's handling CSS Modules.
As a quick test, if you disable modules for rollup-plugin-postcss and add a separate postcss-modules in the plugins list,
postcss({
  extensions: ['.css'],
  extract: true,
  modules: false,
  minimize: true,
  autoModules: false,
  plugins: [
    nesting(),
    modules({
      generateScopedName: '[name]__[local]__[hash:base64:5]'
    })
  ]
})

you end up with the expected CSS:
.Button__container__12nb-{padding:.5rem}
.Button__value__3i9_C{size:1.5rem}
.Button__primary__3ReFs{background-color:#00f}
.Button__container__12nb- .Button__primary__3ReFs .Button__value__3i9_C{color:pink}

I would open an issue with rollup-plugin-postcss as the expectation is that rollup-plugin-postcss  also generates a map of the module names so using postcss-modules directly isn't going to solve things completely.


Feature add node GPU card detail to the deployment details

Description
Add Node card details to the deployments details when deploying a VM with GPU if possible, it's helpful to know which card is selected if there's more the one card to select from when deploying.

the used cards should be excluded in the listing

update:

added a new filed in the deployment dialog to view card info in the mentioned pr

Verified, Devnet a11c9c0.
Card name was added the deployment info.


Add Dashboard filters (group type) and sorting (group name)

what kind of group-type filters do you want?

what kind of group-type filters do you want?

We discussed about the possibility to filter groups by type, but we need to better understand what we mean by "type". We should have two types of groups now:

Permissioned off-chain groups
Permissioned on-chain groups

How do they sound?

if we use the flow that we talked about, Scenario B, users will have two different sign in paths. One to see on-chain groups, one to see off-chain groups. So we will not need the ability to filter those groups.

The only other group differentiator that comes to mind for me is the tree depth/capacity of the group.

The only other group differentiator that comes to mind for me is the tree depth/capacity of the group.

Makes sense!


mxnet homework 2-conv_layer function

Comparing to mlp_layer, I need help in defining conv_layer.
As Chuck said in the class, we can add convolution/activation/pooling layers in "def conv_layer", but I found the similar function in mlp is realized in "def get_mlp_sym()", not in "def mlp_layer".
So my question is what we should input in "def conv_layer".
I attached the origin mlp_layer and my get_conv_sym() below for reference.
def mlp_layer(input_layer, n_hidden, activation=None, BN=False):
# get a FC layer
l = mx.sym.FullyConnected(data=input_layer, num_hidden=n_hidden)
# get activation, it can be relu, sigmoid, tanh, softrelu or none
if activation is not None:
l = mx.sym.Activation(data=l, act_type=activation)
if BN:
l = mx.sym.BatchNorm(l)
return l
def get_conv_sym():     // I created by myself since here
"""
:return: symbol of a convolutional neural network
"""
data = mx.sym.Variable("data")
# todo: design the CNN architecture
# How deep the network do you want? like 4 or 5
# How wide the network do you want? like 32/64/128 kernels per layer
# How is the convolution like? Normal CNN? Inception Module? VGG like?
data_f = mx.sym.flatten(data=data)
l = mx.sym.Convolution(data=data_f, kernel=(3, 3), num_filter=20)
l = mx.sym.Activation(data=l, act_type="relu")
l = mx.sym.Pooling(data=l, pool_type="max", kernel=(2, 2), stride=(2, 2))
# first fullc
l = mx.sym.FullyConnected(data=l, num_hidden=10)
# softmax loss
l = mx.sym.SoftmaxOutput(data=l, name='softmax')
return l

@jianglong98
Thank you for the great question.
If I understood your question correctly,  you are asking if we need to implement pool (or any other function) either in the "def conv_layer" or directly in "def get_conv_symbol".
It is really based on you design to make yourself less coding.
Think about this, you have 1000 conv layer, you prefer calling "conv_layer" 1000 times, OR you write 1000 time "mx.sym.Convolution" AND "mx.sym.Activation" AND "mx.sym.Pooling"?
The only reason that we want to have mlp_layer with a lot of if-else option, is that we don't want to repeat ourselves.
So for the design of "def conv_layer", you can put convolution, activation, pool, BN as optional controlled by input parameters.
Or you can also repeat yourself by copying and pasting the same code (pooling) in "def get_conv_symbol", as there are just few lines of the codes.
Hope it helps.

Week 2 Homework
This is Xiao's homework.


Fix GdipIsRegionEmpty for negative rects

Found when running the region tests against Windows GDI+

@marek-safar this one should be simple to merge and unblocks some other PRs :)

build


Facing error while trying to use vhive as Go module

I'm trying to use vhive as a Go module in a separate Go program but facing the following error.
go: finding module for package github.com/ease-lab/vhive
go: downloading github.com/ease-lab/vhive v1.4.1
go: found github.com/ease-lab/vhive in github.com/ease-lab/vhive v1.4.1
go<EMAIL_ADDRESS>requires
<EMAIL_ADDRESS>invalid version: unknown revision<PHONE_NUMBER>00
Can anyone help with this?

Hi @juno-kim, I apologize for the delay.  For simplicity, we use replace pragmas in go.mod files in few places.
Could you please describe your use case for using vHive as a module? We can discuss code improvements

Some of the vHive features seems useful so I just wanted to reuse them in a separate go program. I think I can work around this problem with replace pragmas as well. Thanks for the answer.

no problem, closing the issue.


Copier updated

Use newer python versions.
Add extra older python versions when required by the hooks.
Fix tests.
Unpin in pyproject.toml, so all you need to update the lockfile is to run poetry update from now on.

Closes #158.

This is now ready to merge IMHO. Please @sbidoul could you review?

Thanks. About .gitignore: do you confirm copier supports the symlink? Why not doing different files instead? What we ignore in the template is unrelated to what we ignore in the template itself.

If you prefer we can have separate files, but I have tested that this works as expected in both scenarios.


Confusing V2 Private Listener Statement

"For the Application Gateway v2 SKU, you can only choose Public frontend IP configuration. Only Private frontend IP configuration is currently not enabled for this v2 SKU. You can have both Public and Private frontend IP configuration."
Please re-read this.  It first says that you cannot have private frontend IP Configuration (confusing grammar in this sentence).  Then it says you can have both Public and Private.
This paragraph needs a clarity re-write.

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

ID: 1b1e0765-a7aa-6e76-3aca-751dc8a531dc
Version Independent ID: e95861b3-cfd3-2b75-95e8-1c0176ba79f2
Content: Quickstart: Direct web traffic using the portal - Azure Application Gateway
Content Source: articles/application-gateway/quick-create-portal.md
Service: application-gateway
GitHub Login: @vhorne
Microsoft Alias: victorh

@ElanShudnow , You cannot deploy Application gateway with Private frontend IP configuration. In other words only ILB mode is not supported. If you need to use Private frontend IP you need to add Public frontend as well.

I understand that.  Just saying that it's very confusing in the documentation and it could use an edit to make it more clear.

https://github.com/MicrosoftDocs/azure-docs-pr/pull/99311


cut a new release?

Hey! Can you cut a new release? Stable cannot be used with Rails5 because of '~> 4.0' activemodel requirement.
I can use master, but I try to minimize github gem usage
Thanks!

Hey!
Sorry, I have been neglecting this for a while. Give me a bit of time over the weekend, and I'll try to cut it out. There is some work done already, but it's still not finished. ✌

that's ok :) if any help needed i'd be glad to dive in

@dreyks I have release 1.0.0 you may try it out and see if it works for you. 🍻

yup, this works. Thanks!


Namespace 'React' has no exported member 'SFC'. TS2694

Hi @MaTeMaTuK
I am facing this error while installing this library. This is happening while installing view-switcher
cc  @Janlaywss @VikariiCGI @lisaavikarii
TypeScript error in D:/TraxiemCB/traxiem/traxiem-ui/src/components/Gantt/view-switcher.tsx(9,34)
:
Namespace 'React' has no exported member 'SFC'. TS2694

```

7 | onViewModeChange: (viewMode: ViewMode) => void;
8 | };
> 9 | export const ViewSwitcher: React.SFC<ViewSwitcherProps> = ({
| ^
10 | onViewModeChange,
11 | onViewListChange,
12 | isChecked,`

Hi @MaTeMaTuK I am facing this error while installing this library. This is happening while installing view-switcher
cc @Janlaywss @VikariiCGI @lisaavikarii
TypeScript error in D:/TraxiemCB/traxiem/traxiem-ui/src/components/Gantt/view-switcher.tsx(9,34)
:
Namespace 'React' has no exported member 'SFC'. TS2694

7 | onViewModeChange: (viewMode: ViewMode) => void;
8 | };

9 | export const ViewSwitcher: React.SFC = ({
| ^
10 | onViewModeChange,
11 | onViewListChange,
12 | isChecked,`

What's your react and react-dom version?

Thanks. It worked.

Hi,
We are able to render switcher tool bar but still there is an error in the file. It is saying React has no exported member of FC and tool bar is having very bad css rendered.

Hi @suniltayal ,
What is the react version ?
Have you use import "gantt-task-react/dist/index.css"; ?

Hi MaTeMaTuk,
My react version is "react": "^16.14.0",
and yes I have imported "gantt-task-react/dist/index.css"

I have updated react version. Please check


typo: determinine -> determine

I can't wait to see the finished dictionary you're building. I'm impressed (or depressed) by the number of issues you've found and fixed so far.


"Import" section unexpectedly at top of page, and often presents incorrect information

The "Import" section is intended to be at the bottom of resource docs pages, and for most resources it is.  However, on https://www.pulumi.com/docs/reference/pkg/docker/service/, the "Import" section is at the top of the page - which looks very out of place.  This is true for 2 of the 10 resources in this package.

Related: https://github.com/pulumi/pulumi-docker/issues/1278


[FEATURE] Use of visible_columns should be limited to when materialize=True

Is your feature request related to a problem? Please describe.
When indexing a subset of columns in a DataPanel, we are currently always returning a view of the DataPanel with visible rows set. We should only be using visible columns when materialize is False, otherwise we should clone a new DataPanel and only have it contain a subset of columns. See
Describe the solution you'd like
if isinstance(index[0], str):
      if not set(index).issubset(self.visible_columns):
          missing_cols = set(self.visible_columns) - set(index)
          raise ValueError(f"DataPanel does not have columns {missing_cols}")
      dp = self.view()
      dp.visible_columns = index
      return dp

Should become
if isinstance(index[0], str):
                if not set(index).issubset(self.visible_columns):
                    missing_cols = set(self.visible_columns) - set(index)
                    raise ValueError(f"DataPanel does not have columns {missing_cols}")

                if materialize:
                    dp = self._clone(
                        data = {
                            k: self._data[k] for k in index
                        }
                    )
                else:
                    dp = self.view()
                    dp.visible_columns = index
                return dp

#104 closes


Feature request: Quite a strange behavior with the selected file after transferring it in Single recipient mode

Is your feature request related to a problem? Please describe.
I also noticed that when sending to a Single recipient, the file remains selected after the transfer. I saw that this was intended, but it only works on the phone. When transferring through the app on Windows, the file is deselected after the transfer.
This is a somewhat strange default behavior that I couldn't understand. Why leave the file selected after transferring it to the phone? If it's to transfer it again to another device, then that's what the Multiple recipients mode is designed for. The most common use case is probably still transferring a file to a single recipient. Therefore, it would be much more convenient to deselect the file after the transfer, because if you need to transfer another file that you forgot about, you'll have to manually deselect the previous file to avoid transferring it again. I personally ran into this situation today.
Describe the solution you'd like

Describe alternatives you've considered
No response
Additional context
No response

What kind of files did you sent? This may be related to #293

In my case, it's media files, namely images.

What LocalSend version do you use on Windows?

What LocalSend version do you use on Windows?

The latest version of the program is 1.8.0 (27) and it is the same on both the phone and the computer.
But the issue with selection occurs not when send from computer to phone, but the other way around, from phone to computer.
If send from the computer to the phone, the selection disappears after the transfer, but if from the phone to the computer, it does not.

What LocalSend version do you use on Windows?

Tomorrow I'll try to send file it from another Android device with a different OS and version. Maybe it's not affecting all of them.

What LocalSend version do you use on Windows?

Tomorrow I'll try to send file it from another Android device with a different OS and Android version. Maybe it's not affecting all of them.

What LocalSend version do you use on Windows?

I confirm that this is a local problem, specifically some kind of glitch, as everything works fine when send from another device.
It can be resolved by simply reinstalling the application on the device and clearing its data and cache.
And the selection not disappears because for some reason the device connection page itself was not displayed (see screenshot), and the transfer was made directly from the Send tab.
Therefore, you can close this issue.

Thanks!


Add #pragma: no cover to abstract methods which should never be tested.

See #57 for example

A single .coveragerc would be better solution than having the codebase full of pragma: no cover

Ok