Test expired metadata from cache

This tests that an expired timestamp/snapshot/targets when loaded
from cache is not stored as final but is used to verify the new
timestamp
Addresses #1681

[x] The code follows the Code Style Guidelines
[x] Tests have been added for the bug fix or new feature
[n/a] Docs have been added for the bug fix or new feature

I'm leaving this with the non-matching description on purpose, as the expected behaviour in
the issue description is to load expired snapshot/targets, which is not the case (and not loading them
is correct from what I understand). So we could ether reword the requirement, of if there's a bug we can fix it.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
4031

Relevant Lines:
4115

💛  - Coveralls

I have some ideas about this pr and will have a conversation with Ivana soon.

the expected behaviour in
the issue description is to load expired snapshot/targets, which is not the case. Not loading them
is actually correct from what I understand from the spec

This may have been clarified already but I'l say it here:
We do load expired metadata with one big caveat: the expired metadata can only be used to validate a newer version of the same type of metadata. In practice:

local, possibly expired, metadata is loaded and used to validate newer downloaded metadata. "validate" here means signature verification for root and various version checks for timestamp, snapshot and targets.
even the downloaded versions may be expired (in practice only root is, because we download all root versions up to the current, hopefully non-expired version)
the python-tuf implementation  will fail the update once the process moves to the next type of metadata and the previous metadata is still expired (e.g. if final root version is expired, refresh() will fail when trying to validate timestamp)

tox is giving really strange proposals here:
-        past_datetime = datetime.utcnow().replace(microsecond=0) - timedelta(days=5)
+        past_datetime = datetime.utcnow().replace(microsecond=0) - timedelta(
+            days=5
+        )

Do we really expect that or it's misconfiguration?

the core issue here is that verifying that local files are opened does not mean they are actually used to verify new downloaded metadata (this is made clear even in the test itself: it opens targets.json but we know that's not used for anything). This does not mean the test is worthless but at least some descriptions in the PR and commit are not describing it correctly. I would like to see a clear explanation of what the test does so that
A) we can be sure we don't already have a test that does the same checks
B) future readers won't be deceived by inaccurate descriptions
I get that this may be the hardest part of this PR: let's talk if you don't have a good idea already
Just to be clear: I think this is a useful test that probably does not exist yet... but test_updater_top_level_update.py has some clearly related tests like test_new_timestamp_expired(): maybe it should have a companion test_local_timestamp_expired()? I don't have strong opinions here but maybe @sechkova has ideas about where to put tests like this -- and whether other tests exist already?
Also (and this is a small detail in comparison), I've been trying to advocate not meddling with the client metadata cache manually: in this case I think it would be better to not do that and instead mock current time (pretend that time now > expiry). Search for mock_time in tests for examples of this.

Just to be clear: I think this is a useful test that probably does not exist yet... but test_updater_top_level_update.py has some clearly related tests like test_new_timestamp_expired(): maybe it should have a companion test_local_timestamp_expired()? I don't have strong opinions here but maybe @sechkova has ideas about where to put tests like this -- and whether other tests exist already?

I admit I find it hard to follow the intentions here, I am sorry if I've put a misleading issue description.
If I understand correctly, test_trusted_root_expired from test_updater_top_level_update.py is doing something very similar and can be used as an example.

@sechkova the idea was to simulate an expired metadata in cache without manually modifying the cache files. I might have misunderstood this mock - the intention is to simulate change in system time, so that what's stored in cache is considered expired, without need of directly touching the local metadata.

I admit I find it hard to follow the intentions here, I am sorry if I've put a misleading issue description. If I understand correctly, test_trusted_root_expired from test_updater_top_level_update.py is doing something very similar and can be used as an example.

It is pretty similar, except that I think we don't actually persist downloaded non-root metadata if it's expired, even though we probably could (rado pointed this out to me -- I had not noticed). This difference isn't that important for actual working repos but for testing it means we need to mock system time to "make the local metadata expired".

I admit I find it hard to follow the intentions here, I am sorry if I've put a misleading issue description. If I understand correctly, test_trusted_root_expired from test_updater_top_level_update.py is doing something very similar and can be used as an example.

It is pretty similar, except that I think we don't actually persist downloaded non-root metadata if it's expired, even though we probably could (rado pointed this out to me -- I had not noticed). This difference isn't that important for actual working repos but for testing it means we need to mock system time (or utcnow() as you point out) to "make the local metadata expired".

Thanks, now I think I got all pieces together. So I agree, test_updater_top_level_update.py could host the additional tests for loading expired snapshot and timestamp and since expired metadata and timestamp are not persisted, mocking seems to be the way.
I think expired local "targets" should be treated the same way as an invalid (unsigned) local metadata and may be part of another set of tests?

I'm marking this a draft since the issue pointed out by Teodora (mocking the wrong thing) is still there: I might have other comments but I've been postponing review until the basics are there. I'm available for a chat if anything is unclear.

@jku I changed to mocking datetime as Teodora advised, so it's now available for review.

I've pushed an update.

Thank you @jku , I added docstrings to all the test methods for clarification.

@jku ah, I resolved by inertia with the microseconds updates... will note to address in following changes. Thanks


page breaks

I noticed many of the pages in the CPK pdf are out of order!  I was faithful to Colop's page breaks in the XML, but I'm sure we'll want to rearrange those chunks at some point (and perhaps rearrange the pdf pages as well, as it's a bit confusing to follow and the text is generally out of order).  Thanks!

Thanks for pointing that out!
On Fri, Nov 6, 2020, 4:59 PM Summer Chambers<EMAIL_ADDRESS>wrote:

I noticed many of the pages in the CPK pdf are out of order! I was
faithful to Colop's page breaks in the XML, but I'm sure we'll want to
rearrange those chunks at some point (and perhaps rearrange the pdf pages
as well, as it's a bit confusing to follow and the text is generally out of
order). Thanks!
You can view, comment on, or merge this pull request online at:
https://github.com/Multepal/popolwuj-colop-poetica-kiche/pull/5
Commit Summary

Update colop-pk.xml

File Changes

M colop-pk.xml
https://github.com/Multepal/popolwuj-colop-poetica-kiche/pull/5/files#diff-18840d609916197bcc7c88adae9991d535f30e137583388a55ed44845fed7fc1
(109)

Patch Links:

https://github.com/Multepal/popolwuj-colop-poetica-kiche/pull/5.patch
https://github.com/Multepal/popolwuj-colop-poetica-kiche/pull/5.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Multepal/popolwuj-colop-poetica-kiche/pull/5, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AACTQ3FAS5B6IIBRQVK3KODSORWUXANCNFSM4TNFZOIQ
.


Logicapp AbuseIPDB-BlacklistIpToThreatIntelligence failes

Describe the bug
my logic app "AbuseIPDB-BlacklistIpToThreatIntelligence" from contenct hub in sentinel is being failed at point it should be injecting data to TI in setinel
Expected behavior
successfully insert the TI data to sentinel
Screenshots

Hello,
Can anyone help me with this?
thanks,
Snehal

Hi @atech-soc, We will update you in few days. Thanks!

Seeing the same issue here too!

Hi @v-amolpatil, I hope you are well. did you get a chance to check this issue?

Hi @atech-soc Can you please check the following points once and confirm for us:

Got to AbuseIPDB solution from content hub then go to playbook AbuseIPDB-BlacklistIpToThreatIntelligence

Select the API connections tab

Select abuseipdb-AbuseIPDB-BlacklistIpToThreatIntelligence connection and go to Edit API Connection and ensure the API key parameter has your required key and save the changes

Again back on API connections tab select the ThreatIntelligence-connection-AbuseIPDB-BlacklistIpToThreatIntelligence connection and in the Edit API tab ensure that you have Authorized the usage of the API save the changes and run again.

If you still face the same issue please let us know, we will discuss further with the team, thanks!

@atech-soc Open the designer and open the Select statement and change the value of targetProeduct from Microsoft Sentinel to Azure Sentinel.

This will get the Submit multiple TI indicators to give you a status of 200 and be successful. However, I do not see the data in the ThreatIntelligenceIndicator table in Log Analytics. @v-vdixit any thoughts on the API returning an incorrect status or possibly missing some mandatory values?

Hi @atech-soc Since we have not received a response in the last 5 days, we are closing your issue #7544 as per our standard operating procedures. If you still need support for this issue, feel free to re-open at any time. Thank you for your co-operation.


fix simplerunner

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 75.00%. Comparing base (dd86f49) to head (5b799f8).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop      #63   +/-   ##
========================================
  Coverage    75.00%   75.00%
========================================
  Files            1        1
  Lines            4        4
========================================
  Hits             3        3
  Misses           1        1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Building Course Data ...

I have Udemy business account. None of courses can be downloaded and following message is displayed
Building Course Data ...

That's because UDemy changed their videos to be DRM. Most likely you won't be able to now.

Thank you Faisal for your message.
Its a heart breaking news.
Is there any chance we will be able again to download videos from Udemy again in the future if you update Udeler accordingly.
Thanks
Shahid
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10
From: @.>
Sent: 29 April 2021 02:10
To: @.>
Cc: Shahid @.>; @.>
Subject: Re: [FaisalUmair/udemy-downloader-gui] Building Course Data ... (#581)
That's because UDemy changed their videos to be DRM. Most likely you won't be able to now.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/FaisalUmair/udemy-downloader-gui/issues/581#issuecomment-828879629, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AENHO2VI2DB6IGGVPL3BL6LTLCWZDANCNFSM43XATJ6A.

Unfortunately not. You won't be able to download videos from UDemy any longer.  I live in an area with very, very slow internet. It was my only way to watch UDemy videos. UDemy will be losing a few hundred dollars a month from me, from now on.

Its very sad news but I have to admit that you have done a brilliant job. I was using your software for a long time.
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10
From: @.>
Sent: 29 April 2021 02:21
To: @.>
Cc: Shahid @.>; @.>
Subject: Re: [FaisalUmair/udemy-downloader-gui] Building Course Data ... (#581)
Unfortunately not. You won't be able to download videos from UDemy any longer.  I live in an area with very, very slow internet. It was my only way to watch UDemy videos. UDemy will be losing a few hundred dollars a month from me, from now on.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/FaisalUmair/udemy-downloader-gui/issues/581#issuecomment-828882359, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AENHO2UUMPSPVWABU32YC33TLCX7VANCNFSM43XATJ6A.

Unfortunately not. You won't be able to download videos from UDemy any longer.  I live in an area with very, very slow internet. It was my only way to watch UDemy videos. UDemy will be losing a few hundred dollars a month from me, from now on.

Dude there is always a solution to a problem when it comes to software engineering. I advice you stop using Udeler Gui and go for more complex tools using Python and Perl.

It seems that Udemy is adding DRM to its contents everyday. Some of my courses show DRM icon on the browser and some other don't. However, I can't download any with existing option.

I can download all videos with a Chrome Extension, although is all manual (video by video).

What is the name of chrome extention please?

I can download all videos with a Chrome Extension, although is all manual (video by video).

What is the name of chrome extention please?

Of course, it's VideoOctopus

OMG that is too bad I hate how Udemy makes the transition from one video to another one looks for me like an eternity one reason I download videos and also I share my internet with 5 people

There's a "udemy downloader" extension for chrome too which downloads whole course, but the problem is it downloads automatically in 1080p ( or perhaps highest available quality)
link is here :   https://chrome.google.com/webstore/detail/udemy-downloader/oojlfdpeghhdkdbcajpmbnkbmecbdejm?hl=en

There's an extension for chrome called "udemy downloader" too which downloads whole course, but the problem is it downloads automatically in 1080p ( or perhaps highest available quality)
link is here : https://chrome.google.com/webstore/detail/udemy-downloader/oojlfdpeghhdkdbcajpmbnkbmecbdejm?hl=en

I saw that one but I have biz account and does not work with biz accounts :/

There's an extension for chrome called "udemy downloader" too which downloads whole course, but the problem is it downloads automatically in 1080p ( or perhaps highest available quality)
link is here : https://chrome.google.com/webstore/detail/udemy-downloader/oojlfdpeghhdkdbcajpmbnkbmecbdejm?hl=en

sadly that addon extension also fails to download

I would recommend we don't open any further issues regarding this, and instead focus on one. We can link and close the other issues later.
I'll investigate the issue over this weekend and see if I can find a cause. I can confirm that I can still download videos using other more traditional slower methods (find the .m3u8 file in the network tab of your browser and use an HLS downloader) as well as some Extensions (HLS Downloader).

Please can you explain in detail using the slower traditional method for those that are not developers. Thanks 🙏

My mistake, after further inspection I can confirm they are using Widevine. For all videos that have public previews, they are streaming them as HLS, unencrypted, thus why my notes above work. Once you get into the videos that require payment, they begin using Widevine and streaming it as MPEG-DASH.
This would also explain why people are reporting that they are able to download a few videos, but then it stops. I'd have to dig more into Widevine, which I don't have time for at the moment, but from my initial reading, it doesn't seem likely we'd get this fixed.

I can download all videos with a Chrome Extension, although is all manual (video by video).

What is the name of chrome extention please?

Of course, it's VideoOctopus

I would not use that software.....Looks to be some shady-ass info-grabbing software.


TabView里面放ListView滚动有问题

Use English please.
TabView isn't part of CardsView nuget, this is just controls for test.
But you can solve all problems, if look through workaround section https://github.com/AndreiMisiukevich/CardView#workarounds


Event with no root object.

I'm getting the following error:
[2018-01-16 16:02:02,265] ERROR {org.wso2.extension.siddhi.map.json.sourcemapper.JsonSourceMapper} - Default json message {"eventName":"START_NODE","eventCode":"2"} contains an invalid event identifier. Required "event", but found "eventName". Hence dropping the message.

It seems that JSON mapper doesn't support json objects with no root element. I tried to experiment with different configuration of enclosing.element parameter with no result.
Whole Siddhi script:
@App:name("CargoWeightApp")

@sink(type='log', prefix='LOGGER')
define stream OutputStream(eventName string);

@source(type ='rabbitmq',
    uri = 'amqp://guest:guest@localhost:5672',
    exchange.name = 'device-event',
    routing.key= '#' , @map(type='json', fail.on.missing.attribute='false')
)
define stream MetricsStream(eventName string);

@info(name='q1')
from MetricsStream
select eventName
insert into OutputStream;

We have resolved this with #47  and released v4.0.24.

Fixed with above PR


Fix system & everyone role deletion issue

Purpose

Related Issues

https://github.com/wso2/product-is/issues/18390

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/7165087841

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/7165087841
Status: success


Tab scrollbuttons disabled status with CSS

I use SCSS for the styling option with this MUI framework. When I use scrollable tabs, I cannot select the disabled scroll buttons to add the styles but I can add the styles of default scroll buttons.
import styles from './index.scss'
....
<Tabs
          ....
          className={styles.tabs}
          classes={{ scrollButtons: styles.tabScrollButton }}
>
....

At index.scss file
.tabScrollButton {
      width: 24px;   // this works well

      &.disabled {
        width: 0;        // this is not working
      }
    }

I've tried with &.Mui-disabled and even with.MuiTabScrollButton-root.Mui-disabled but they didn't work for me.
Please let me know how I can select that disabled status of the scroll button to add style with SCSS or CSS.

Tip the .MuiTabs-scrollButtons.Mui-disabled CSS selector should work.


Right-associative :|, proper tests and PureScript 0.8 updates

This PR includes a few things:

Updates for PureScript 0.8 (fix warnings and use new operator alias syntax)
Make :| right-associative
Add test suite (previously, Test.Main just printed a few things to the console)

If you would prefer a PureScript 0.7 - compatible PR, let me know.
Remark: it's slightly unfortunate that we have to (?) define a synonym nonEmpty = NonEmpty in order to define the operator :|. It would be nice if we could use:
infixr 6 NonEmpty as :|

Thanks very much!


Avoid to call 'go' with empty -tags argument

go1.10.1 linux/amd64 complains: flag provided but not defined: -tags
when compiling with: "make build-cross dist". This seems to fix it.

Thanks! Once the CLA is signed we'd be happy to merge this.

thanks!


Getting nothing when i use the same with polygon

import { Component } from '@angular/core';
@Component({
selector: 'app-root',
templateUrl: './app.component.html',
styleUrls: ['./app.component.css'],
})
export class AppComponent {
public sdspolygons;
ngOnInit() {
this.sdspolygons = [
{points:"150,75 258,137.5 258,262.5 150,325 42,262.6 42,137.5"},
];
}
ngOnDestroy() {
}
onAnimationFrame() {
}
}
@Component({
selector: '[a-polygon]',
template:      <svg:polygon points="200,10 250,190 160,210"></svg:polygon>
})
export class PolygonComponent {
}
app.component.html

<svg:g a-polygon
*ngFor="let polygon of sdspolygons">
/svg:g

Im not able to figure what is missing .Could you please help me .

Sorry, I can't make out from this code what the problem is. Can you share your whole repository on GitHub so that I can see the whole thing?

@teropa  Could you please update your existing repository to angular 2 latest version and try it.i tried your repo with the latest angular2 the svg is not rendering anything.

Unfortunately I'm not going to have time to do that any time soon. But I can take a look at your version if you can share it.

@teropa  you can mark this  as closed as i developed a library for svg  https://www.npmjs.com/package/angular-svg


Ruby install error - Encoding error

Hello,
I am trying to install ruby and ruby dev kit using chocolatey cookbook but running into the below issues -
         * chocolatey[ruby] action install

           =====================================================================
===========
           Error executing action `install` on resource 'chocolatey[ruby]'
           =====================================================================
===========

           Encoding::UndefinedConversionError
           ----------------------------------
           "\xEF" to UTF-8 in conversion from ASCII-8BIT to UTF-8 to UTF-16LE

           Resource Declaration:
           ---------------------
           # In C:/Users/vagrant/AppData/Local/Temp/kitchen/cache/cookbooks/ruby/recipes/_configure-windows.rb

            22: chocolatey 'ruby' do
            23:   action :install
            24:   version node['ruby']['version']
            25: end
            26:

           Compiled Resource:
           ------------------
           # Declared in C:/Users/vagrant/AppData/Local/Temp/kitchen/cache/cookb
ooks/ruby/recipes/_configure-windows.rb:22:in `from_file'

           chocolatey("ruby") do
             action [:install]
             retries 0
             retry_delay 2
             default_guard_interpreter :default
             declared_type :chocolatey
             cookbook_name "ruby"
             recipe_name "_configure-windows"
             version "2.3.0"
             package "ruby"
           end

           Platform:
           ---------
           x64-mingw32

       Running handlers:
       [2016-06-21T07:54:01-07:00] ERROR: Running exception handlers
       Running handlers complete
       [2016-06-21T07:54:01-07:00] ERROR: Exception handlers complete
       Chef Client failed. 43 resources updated in 05 minutes 34 seconds
       [2016-06-21T07:54:01-07:00] FATAL: Stacktrace dumped to C:/Users/vagrant/
AppData/Local/Temp/kitchen/cache/chef-stacktrace.out
       [2016-06-21T07:54:01-07:00] FATAL: Please provide the contents of the sta
cktrace.out file if you file a bug report
       [2016-06-21T07:54:01-07:00] FATAL: Encoding::UndefinedConversionError: ch
ocolatey[ruby] (ruby::_configure-windows line 22) had an error: Encoding::
UndefinedConversionError: "\xEF" to UTF-8 in conversion from ASCII-8BIT to UTF-8
 to UTF-16LE
$$$$$$ C:/opscode/chef/embedded/lib/ruby/2.1.0/x64-mingw32/dl.so: warning: alrea
dy initialized constant DL::RUBY_FREE
$$$$$$ C:/opscode/chef/embedded/lib/ruby/gems/2.1.0/gems/net-ssh-3.1.1/lib/net/s
sh/authentication/pageant.rb:16: warning: previous definition of RUBY_FREE was h
ere
$$$$$$ DL is deprecated, please use Fiddle
>>>>>> Converge failed on instance <default-windows-2008R2>.
>>>>>> Please see .kitchen/logs/default-windows-2008R2.log for more details
>>>>>> ------Exception-------
>>>>>> Class: Kitchen::ActionFailed
>>>>>> Message: WinRM exited (1) for command: [
$env:PATH = [System.Environment]::GetEnvironmentVariable("PATH","Machine")

& $env:systemdrive\opscode\chef\bin\chef-client.bat --local-mode --config $env:T
EMP\kitchen\client.rb --log_level auto --force-formatter --no-color --json-attri
butes $env:TEMP\kitchen\dna.json --chef-zero-port 8889]
>>>>>> ----------------------

Running this on Windows 7 OS with Chefdk 0.12.0. Chef client version - 12.8.1 and kitchen version 1.8.0
Not sure if this is the right place to ask about it but would appreciate if you can help with this or redirect to correct forum.

Thanks for reporting! Can you add the stacktrace either by running with -l debug or pasting the stacktrace.out file?

[Uploading default-windows-2008R2.txt…]
Please find attached the logs generated by using -l debug option

For some reason Test-Kitchen only has the stacktrace from the chef failure hit by chef.bat and not the inner error. I think you'd be able to get that from C:/Users/vagrant/AppData/Local/Temp/kitchen/cache/chef-stacktrace.out

I don't see any chef-stacktrace.out file generated on my system. Not sure what else can I check

closing due to inactivity.


Fix bug: compatibility with rails > 5.1

Thanks folks! I haven't used this gem for a couple years, but this look good to me. It someone wanted to contribute a spec suite, that'd be even better! :p


Add support for database types other than MaxMind

There are now a number of alternative providers of MMDB databases (including the free Geoacumen database generated from ASN data), but they are unusable with the geoip2 library because it explicitly checks for a MaxMind-specific reader.metadata.DatabaseType.
This pull request maintains the current behavior while adding the option to specify a custom database type when needed by calling NewXxxReaderType() instead of NewXxxReader(). It also accepts Geoacumen-Country as a valid database type for NewCountryReader() by default.

Hi, thanks for your interest.
I think, you can just add Geoacumen-Country to supported types, without other changes. How I can see, Geoacumen contains only one field in country - iso_code, this field is supported in the library, but anyway test should be written for this database.
This library has hardcoded available fields, because of that it's not possible to permit user to use any types of database.

Thanks for the response. My thought (and the reason I didn't create tests specifically for Geoacumen) is that there are actually a number of other providers of MMDB databases -- DB-IP is one, IP2Location is another, and there were a number of others that I did not make note of as they didn't fit my specific needs when I was compiling my list. There are also a number of tools available to create your own MMDB databases.
In my (admittedly subjective) experience, they tend to use the same field names as the MaxMind databases, they just don't use a DatabaseType that contains MaxMind's trademarked product names. So the existing DatabaseType checks become an artificial limitation on using third-party databases.
I'm not suggesting you aim to offer formal support for all database types; it would just be nice if we as developers had the option to swap in other databases on a "try it and see" basis without maintaining a fork just for that purpose.

Looks reasonable.
But I think it would be better to use names for functions with suffix like (By|With)Type and add documentation for this case, when developer takes responsible for correctness of database up to himself.

Sounds good -- I've made the requested changes.

Conflicts should be resolved.

Looks ok, conflicts should be resolved before merge.


Let MxNet Support CuDNN v2.

HI, guys, I am going to make MxNet support CuDNNv2.
Do you have any good suggestions?
@tqchen @piiswrong

Why should we support cuDNN v2? Is there any benefit from that?
On Mon, Feb 1, 2016 at 4:25 PM, Zhang Ming<EMAIL_ADDRESS>wrote:

HI, guys, I am going to make MxNet support CuDNNv2.
Do you have any good suggestions?
@tqchen https://github.com/tqchen @piiswrong
https://github.com/piiswrong
—
Reply to this email directly or view it on GitHub
https://github.com/dmlc/mxnet/issues/1392.

--
HONG Chuntao
System Research Group
Microsoft Research Asia

TK1 only support CuDNNv2.

I see. 😅
On Mon, Feb 1, 2016 at 4:46 PM, Zhang Ming<EMAIL_ADDRESS>wrote:

TK1 only support CuDNNv2.
—
Reply to this email directly or view it on GitHub
https://github.com/dmlc/mxnet/issues/1392#issuecomment-177855174.

--
HONG Chuntao
System Research Group
Microsoft Research Asia

Look into cudnn_covlution-inl.h. It shows how to use CUDNN_MAJOR to determine cudnn version and activate different code.

thanks @piiswrong
do you have any further suggestion?

if do not consider the backward operation, it is easy to make mxnet use cudnn_v2.

Have you make it to support cudadnn v2？

yes, but only for prediction.

While compiling with cudnn v2, I got the error:
/mxnet/mshadow/mshadow/./base.h:276:45: error: 'CUDNN_DATA_HALF' was not declared in this scope
static const cudnnDataType_t kCudnnFlag = CUDNN_DATA_HALF;
^

@starimpact  Have you managed to make mxnet support cudnn v2? I have two TK1 lying around and I would like to use them to train small sets for screening models and it would be great if you already did that :)

@el-oso I am sorry, I just did the prediction on TK1.

@starimpact Well, that is a start :). Could you share your code? Maybe I can manage to also update the training part.

@starimpact Could you please share your Cudnnv2 version? a fork or a gist file would be really appreciated by the community :)

no problem.i will share it tomorrow.

@marcociccone
git@github.com:starimpact/mxnet_tk1.git
you need to remove the -msse2 and -msse3 which should be in dmlc-core and mshadow by yourself.
if there is any problem , you can ask me for help.

Thank you! I’ll let you know if I have some troubles

Il giorno 14 ott 2016, alle ore 05:27, Zhang Ming<EMAIL_ADDRESS>ha scritto:
@marcociccone https://github.com/marcociccone
<EMAIL_ADDRESS>mailto:git@github.com:starimpact/mxnet_tk1.git
you need to remove the -msse2 and -msse3 which should be in dmlc-core and mshadow by yourself.
if there is any problem , you can ask me for help.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/dmlc/mxnet/issues/1392#issuecomment-253700680, or mute the thread https://github.com/notifications/unsubscribe-auth/ABECRHMl452YrwviTPuYc4ej88PocGhUks5qzvaugaJpZM4HQWDc.

@marcociccone @starimpact Could you succeed in compiling mxnet for cuDNN? I couldn't compile it. In fact, I've noticed GPUs which I may access have compute capability less than 3. So they do not support cuDNN. However, I feel it may require sometime :)

Got the same problem as @horsepurve ,
/mxnet/mshadow/mshadow/./base.h:276:45: error: 'CUDNN_DATA_HALF' was not declared in this scope
static const cudnnDataType_t kCudnnFlag = CUDNN_DATA_HALF;

@horsepurve
Just missing CUDNN_DATA_HALF definition, modify your cudnn.h.
cd /usr/local/cuda/include;
find the cudnn.h file, edit it
find this code block：
/*

CUDNN data type
*/
typedef enum
{
CUDNN_DATA_FLOAT  = 0,
CUDNN_DATA_DOUBLE = 1
} cudnnDataType_t;

Add CUDNN_DATA_HALF = 2
After changes：
typedef enum
{
CUDNN_DATA_FLOAT  = 0,
CUDNN_DATA_DOUBLE = 1,
CUDNN_DATA_HALF = 2
} cudnnDataType_t;
good luck
hhhhhh

No, I still get an error :(. I can build MXNet without CUDA support, but then I don't use the full potential of the board, i.e. no GPU. I just want to build it for prediction with CUDA 6.5/CUDNN v2 (max available in the TK1 afaik), is there a way to do that?
@crabriver did you try your fix yourself? I tried it and compilation goes further but then it gets stuck in
In file included from src/operator/cudnn_convolution.cc:7:0:
src/operator/./cudnn_convolution-inl.h:36:13: error: ‘cudnnConvolutionBwdDataAlgo_t’ has not been declared
cudnnConvolutionBwdDataAlgo_t *bwd,

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!


Change encoding to UTF-8 for minimal.css

CSS files should be UTF-8: https://www.w3.org/International/questions/qa-css-charset

https://github.com/boostorg/filesystem/commit/9df2bfab58aadf35b4e01f2fd6252e1521409779#diff-36873c6a5a238ba4df45a47d97d80ead5ed664b3cb7e5a8fbe918020cc6889a4 replaced another ISO-8859-1 copyright symbol with (c), which would also be an option here. I just want to get rid of the ISO-8859-1 character in a file that doesn't specify its @charset, because it causes warnings for distro packagers.

I think changing to '(c)' is the simplest fix.

OK thanks, I'll re-push with that change then...

Now done consistently with https://github.com/boostorg/detail/commit/8c603a1894ec1d8abc7a0a538297923b0ac98b33

Ping


Any example implementation on ios?

Does anyone know an ios equivalent of the android example in the repository? Thanks

https://github.com/bjnortier/whisper-tflite-ios/tree/main


⚠ Bofrost.fr has degraded performance

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

HTTP code: 200
Response time: 2458 ms

Resolved: Bofrost.fr performance has improved in c1480db.


slurm: <IP_ADDRESS> -> <IP_ADDRESS>

Motivation for this change
Version bump.
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

Success on x86_64-linux (full log)
Attempted: slurm
Partial log (click to expand)
gzipping man pages under /nix/store/5bpiax51vkz2kdalj526nddm1zw9k02n-slurm-<IP_ADDRESS>/share/man/
strip is /nix/store/vcc4svb8gy29g4pam2zja6llkbcwsyiq-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/5bpiax51vkz2kdalj526nddm1zw9k02n-slurm-<IP_ADDRESS>/lib  /nix/store/5bpiax51vkz2kdalj526nddm1zw9k02n-slurm-<IP_ADDRESS>/bin  /nix/store/5bpiax51vkz2kdalj526nddm1zw9k02n-slurm-<IP_ADDRESS>/sbin
patching script interpreter paths in /nix/store/5bpiax51vkz2kdalj526nddm1zw9k02n-slurm-<IP_ADDRESS>
checking for references to /build in /nix/store/5bpiax51vkz2kdalj526nddm1zw9k02n-slurm-<IP_ADDRESS>...
moving /nix/store/5bpiax51vkz2kdalj526nddm1zw9k02n-slurm-<IP_ADDRESS>/sbin/* to /nix/store/5bpiax51vkz2kdalj526nddm1zw9k02n-slurm-<IP_ADDRESS>/bin
shrinking RPATHs of ELF executables and libraries in /nix/store/awgqzm0f0bf9xh5mdicb8wvf1vwmj2fs-slurm-<IP_ADDRESS>-dev
strip is /nix/store/vcc4svb8gy29g4pam2zja6llkbcwsyiq-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/awgqzm0f0bf9xh5mdicb8wvf1vwmj2fs-slurm-<IP_ADDRESS>-dev
checking for references to /build in /nix/store/awgqzm0f0bf9xh5mdicb8wvf1vwmj2fs-slurm-<IP_ADDRESS>-dev...


QoL Setting: OoT Gerudo Fortress Behavior

Like the title says. A drop-down setting that consists of the following:
Gerudo Fortress:
-Default Behavior (current functionality)
-Rescue One Carpenter (so only need to rescue the first carpenter, one key, and the Gerudo Card check activates after freeing the one carpenter)
-Open (the bridge is constructed, the player essentially starts with the Gerudo Card)
Can't find a PR where this was added to OOTR but the current setting can be looked at for OOTR in the main stable branch anyway.

How does it work with keysanity?
Right now the hideout keys are shuffled (as if it was a dungeon) even if obviously the shuffle doesn't change anything outside of keysanity.

In OOTR, for Open Fortress, they block you from shuffling the keys because the Open fortress setting basically gives you the Gerudo Card so the check is basically completed from the onset.
With One Carpenter, it's just the first carpenter that everyone normally gets. You have only 1 key and once you free that carpenter, you get the Gerudo Card check.

How does it work with keysanity? Right now the hideout keys are shuffled (as if it was a dungeon) even if obviously the shuffle doesn't change anything outside of keysanity.

In OOTR, for Open Fortress, they block you from shuffling the keys because the Open fortress setting basically gives you the Gerudo Card so the check is basically completed from the onset.
With One Carpenter, it's just the first carpenter that everyone normally gets. You have only 1 key and once you free that carpenter, you get the Gerudo Card check.

Done


ctags detection should be only called when needed

but then it's required

@gf712 right?

Is it possible to set this CTAGS_FILE manually while running cmake?

this should fix #5003


Avoid multiple problem locations in report

Fixes https://github.com/gradle/gradle-private/issues/4497
Reviewing cheatsheet
Before merging the PR, comments starting with

❌ ❓must be fixed
🤔 💅 should be fixed
💭 may be fixed
🎉 celebrate happy things

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

Pull Request Feedback (Trigger) (Check)

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

Pull Request Feedback (Trigger) (Check)

The merge queue build has failed. Click here to see all failures.


Not getting any display in Magic Mirror

Full disclosure - I might just be dumb - but even when loading this module in barebones (api_key only under config: - using the Main API key from my newly minted uptimerobot account) - I don't get any squawking errors, but I also don't get... well, anything, really.
Is there perhaps a very obvious thing I missed?

Hi,
had the same issue - renaming uptimerobot.js to MMM-uptimerobot.js plus changing how the module registers worked for me.
Run this in your cloned project folder:
mv uptimerobot.js MMM-uptimerobot.js
Then change MMM-uptimerobot.js to this in line 8, right after the header comment:
Module.register("MMM-uptimerobot", {
This error is not dumped in the shell when executing MagicMirror but rather in the Electron app console... to see it, press e.g. on a Mac CMD-ALT-I when viewing your MagicMirror app - same combination for pulling up dev tools in Chrome.
Hope that helps!

@hnoesekabel: You are right, there was a typo using MMM-template. Fix will be ready shortly.
BTW: Right now there is an issue with using icons (config useIcons: true); also this issue will be fixed with next commit.

@hnoesekabel -
a very thorough explanation, and right on the nose! Thanks very much for pointing me in the right direction - was definitely getting a bit lost in the weeds. I am getting the display
@mrVragec - thanks for your work on this module - I love the idea of the Magic Mirror project, and I find your project to be extremely practical for my intended use. Now that the modules displays, I am having success with useIcons (at least the 'Online' icon) - 'useColours: true' doesn't appear to have any effect, though. (I don't know if it's meant to stack with the icons, but it seems to have no effect on the text-only view or the icon view).

@tgabben with option 'useIcons: true' you should see icons instead of text and with option 'useColors: true' icons or text should be green (online) or red (offline) (and other colors for different status). I just added screenshots so you can see a different between useIcons and useColors options.

@mrVragec that's what I figured - just was allowing for the possibility that the options were mutually exclusive.
Again - I'm finding that the 'Online' icon is displaying (let's hope i rarely, if ever, have to see any other icon) - but I even double-checked to be sure I hadn't used the Canadian/British spelling of 'colours' (as in my comment above) - but the colour display isn't showing either for icons or text.

This is resolved for me - just noticed that you'd posted new instructions w/the code, to rename the folder and the module title in the main config (the inverse of @hnoesekabel's fix from earlier than got the module to load in the first place) - must have been another case where the MMM-uptimerobot vs. uptimerobot thing wasn't calling the right .css file, because both icons and colours are now working as intended!
rm -rf'd for good measure within /modules, re-cloned from git to get your fixes, and after re-naming the cloned folder to 'uptimerobot', all is well.
Thanks!


random guest name not working

we might not care, but something with the integration of the comments with questions broke the random guest default if author is not provided.

feature removed for release


ModuleNotFoundError: No module named 'utils.console'

This is the following error that I recieve -
Traceback (most recent call last):
File "eagle-eye.py", line 8, in
import utils.console as console
ModuleNotFoundError: No module named 'utils.console'
i looked at the file and it seems like console and config isn't in classes.

Use python 3.6 or higher.


[WFLY-19150] Clean up server.jvm.args handling

Fix places that were not passing through the 'server.jvm.args' maven property value to surefire

Remove some (but not all) redundant -Dmaven.repo.local=${settings.localRepository} setting, as it's in server.jvm.args via surefire.system.args

Remove crufty -Dorg.wildfly.unsupported.skip.jakarta.transformer=true settings

Remove overridden setting of the surefire 'server.jvm.args' system property that just does the same as the global config setting in testsuite/integration/pom.xml

https://issues.redhat.com/browse/WFLY-19150

Thanks @luck3y


Issue in converting to Verilog.

Hello,
I'm very new to FIRRTL. I'm just trying to convert the following FIRRTL to Verilog, but it fails with error

line 8:18 mismatched input '(' expecting DEDENT

circuit mod_example:
  module mod_example:
    input  indata: UInt<16>
    input  dem: UInt<16>
    output outdata: UInt<16>

    outdata <= mod(indata, dem)

Command I used is

./utils/bin/firrtl -tn top -i mod_example.fir -o mod_example.v

Could you tell me what's wrong with my FIRRTL? I'll appreciate.
Thank you.

It appears that the spec is quite out of date. Instead of mod, try rem.

I see. Thank you.


addition assignment behaves surprisingly with union types

TypeScript Version:  2.4.0 / nightly (2.5.0-dev.201xxxxx)
Code
let x: string | number = 0;
x += ''; // ERROR(2,1): Type 'string' is not assignable to type 'number'.

Expected behavior:
Since this is ok
let x: string | number = 0;
x = x + '';

similarly (showing that this has to do with type-narrowing)
function f(x: string | number) {
  x += '';
}

and finally TS allows string and number additions, thus I expect the original to also be accepted by the compiler.
Generally treating x = x + 1 and x += 1 differently from type checking perspective is surprising, but maybe this is byproduct of other designed behaviors.
Actual behavior:
Error - Type 'string' is not assignable to type 'number'.

I'm not really sure if this is a bug per se or not, but it doesn't seem like something that needs time spent on it either way.


azure-app-configuration: add replication latency metric

@mcleanbyron
IMPORTANT: 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

@mcleanbyron We are signing off. PR is ready to be signed off and merged :)

#sign-off


V3.0.0 - replace python mqtt script with mqtt.js

replaced python mqtt script with mqtt.js

checkTVStatus still needs to run periodically to check if the tv is online/offline
but the Accessory now opens a mqtt connection if the tv is online, and updates states in real time

there is now a node executable used to authenticate the homebridge instance (instead of the py script)
Add option for TVs with always on display
WOL Packages get send multiple times to account for packet loss

Most changes should already be reflacted in the README
TODO

Add option for TVs with always on display
Send WOL Package multiple times to account for network packet loss
Add apps (as source)
Remove duplicated code

still work in progress (but feedback is always welcome)

Hey @LeLunZ ! It seems you're still working on this PR, so I converted it to a draft to avoid accidental merges. Please let me know when you'd like for me to review 😊
Thank you for pushing this forward!

I want to give a update on "Always On TVs" #18 :
I read through the home-assistent thread mentioned here
And it seems there are two different ways Always On TVs can show the Sleep State:

if statetype (in the state topic we already listen to) is set to 'fake_sleep_0'
topic .../platform_service/data/picturesetting sends out Settings object with menu_items which one could change depending on if the TV is On or Off.

To get 1. to work, a user just need to change "TV Type" in the settings to Fake Sleep.
To get 2. to work, someone really must look up what menu_item changes and add the id, and the value it has in the off state to the config.

From my side this is finally done ;)
@MrAsterisco if you have the time and want to take a look at the code please do.
I am not that skillful in writing README files so if you find any weird wordings there please tell me :)
I also released the first v3.0.0 beta so that this is testable while in review.

Furthermore just some information about the changes:
Changes in homekit are now mostly triggered reactive based. (Means we only do polling if 100% necessary. For every other state updates, we get mqtt messages and then update the homekit state based on that)
For normal TVs we can never fully remove polling, because they shut down their mqtt server if turned off.
The mqtt client could do the polling itself, but I didn't change the interval (60 seconds) to something lower, as its probably more resource intensive than just opening the socket for a very short time???

I guess I should also add more information to the  CHANGELOG.md so users who just upgrade without looking into the repo see that there are some breaking changes.

Well finally done... Saw that the homebridge-plugin-template migrated to esmodules so i included that.
Also removed the comment from above to put it here again:
From my side this is finally done ;)
@MrAsterisco if you have the time and want to take a look at the code please do.
I am not that skillful in writing README files so if you find any weird wordings there please tell me :)
I also released the first v3.0.0 beta so that this is testable while in review.

Furthermore just some information about the changes:
Changes in homekit are now mostly triggered reactive based. (Means we only do polling if 100% necessary. For every other state updates, we get mqtt messages and then update the homekit state based on that)
For normal TVs we can never fully remove polling, because they shut down their mqtt server if turned off.
The mqtt client could do the polling itself, but I didn't change the interval (60 seconds) to something lower, as its probably more resource intensive than just opening the socket for a very short time???

Also regarding the previously known issues:

Switching input to "TV" might not work properly. Home will not display any error, but the next TV state refresh will bring the input back to the previous one (which is also the one displayed on the TV).

Is now fixed. That was a problem with always creating the mqtt client (when publishing), where the connection probably wasn't 100% there or stable. Now when pressing a input it always works as the mqtt connection is now always open :)

Making changes to the TV state (turning on/off, changing input) while the Home app is opened will not trigger a live update. This is theoretically supported by the plugin, but it seems to not work properly.. Just switching to another app and then going back to Home will trigger a refresh.

Is now also fixed with the migration to the newer homebridge version and the reactive approach of updating states.


small fix prerender https issue #1202

Fix of Issue https://github.com/TelescopeJS/Telescope/issues/1202

Thanks! I actually fixed something else in that code so I'll just work in your changes manually.

@SachaG is the new fix on a branch somewhere that I can cherry pick for now till you update the package on github?

It's on the devel branch.
On Wed, Nov 25, 2015 at 4:01 PM, Truong Nguyen<EMAIL_ADDRESS>wrote:

@SachaG https://github.com/SachaG is the new fix on a branch somewhere
that I can cherry pick for now till you update the package on github?
—
Reply to this email directly or view it on GitHub
https://github.com/TelescopeJS/Telescope/pull/1205#issuecomment-159519647
.

got it, thanks :)


not issue ! your reply bot really work !

You are the only one bot in python really working and comment to a post thank you

cool


Base16 support has already been developed and you just have to merge it

Is your feature request related to a problem? Please describe.
@pardom has already developed support for base16 in https://github.com/EdenEast/nightfox.nvim/pull/354
Describe the solution you'd like
Make any edits you want and then merge it. The pull request has been waiting for 7 months, why?

Make any edits you want and then merge it. The pull request has been waiting for 7 months, why?

I am not sure you understand what it means to be a maintainer of an open source project. This is something I initially wrote for myself, then shared it for others if they liked it. The why is that life changes (health, jobs, family obligations) and what someone spends their free time on will also change.
From your profile it looks like you are new to the space of open source. In the future please keep in mind these things and be careful when asking why something has not been done in someone's own project.


actions/deploy-pages v2 failing with HttpError: Invalid artifact url

Hi. I have a GitHub Pages deployement workflow that is failing with

It seems like this is recurring issue for this incident which has since been closed. What's more interesting to me is while a new deployment fails, my site shows a 404, shouldn't it just show the previous version?

👋 can you share the workflow you are using?

Hey yea sure.
# Sample workflow for building and deploying a Hugo site to GitHub Pages
name: Deploy Hugo site to Pages

on:
  # Runs on pushes targeting the default branch
  push:
    branches:
      - main

  # Allows you to run this workflow manually from the Actions tab
  workflow_dispatch:

# Sets permissions of the GITHUB_TOKEN to allow deployment to GitHub Pages
permissions:
  contents: read
  pages: write
  id-token: write

# Allow only one concurrent deployment, skipping runs queued between the run in-progress and latest queued.
# However, do NOT cancel in-progress runs as we want to allow these production deployments to complete.
concurrency:
  group: "pages"
  cancel-in-progress: false

# Default to bash
defaults:
  run:
    shell: bash

jobs:
  # Build job
  build:
    runs-on: ubuntu-latest
    env:
      HUGO_VERSION: 0.120.2
    steps:
      - name: Install Hugo CLI
        run: |
          wget -O ${{ runner.temp }}/hugo.deb https://github.com/gohugoio/hugo/releases/download/v${HUGO_VERSION}/hugo_extended_${HUGO_VERSION}_linux-amd64.deb \
          && sudo dpkg -i ${{ runner.temp }}/hugo.deb
      - name: Install Dart Sass
        run: sudo snap install dart-sass
      - name: Checkout
        uses: actions/checkout@v4
        with:
          submodules: recursive
          fetch-depth: 0
      - name: Setup Pages
        id: pages
        uses: actions/configure-pages@v3
      - name: Install Node.js dependencies
        run: "[[ -f package-lock.json || -f npm-shrinkwrap.json ]] && npm ci || true"
      - name: Build with Hugo
        env:
          # For maximum backward compatibility with Hugo modules
          HUGO_ENVIRONMENT: production
          HUGO_ENV: production
        run: |
          hugo \
            --gc \
            --minify \
            --baseURL "${{ steps.pages.outputs.base_url }}/"
      - name: Upload artifact
        uses: actions/upload-pages-artifact@v2
        with:
          path: ./public

  # Deployment job
  deploy:
    environment:
      name: github-pages
      url: ${{ steps.deployment.outputs.page_url }}
    runs-on: ubuntu-latest
    needs: build
    steps:
      - name: Deploy to GitHub Pages
        id: deployment
        uses: actions/deploy-pages@v2

Try updating actions/upload-pages-artifact to v3 and actions/deploy-pages to v4.
We publish this starter workflow as a good reference to follow: https://github.com/actions/starter-workflows/blob/666350e29b10d665a82d5c6d1501a29e50d63c29/pages/hugo.yml.

Sounds good. I updated and now I'm getting a Validation error. seems like a me issue though, thank you for your help. I'll try to figure this out on the weekend.


MYNEWT-292; Don't allow two tasks to have the same priority

Add an assert checking that same prio is not used by two different
tasks.

Cool. Can you also update the unit tests for kernel/os and sys/flash_map? They start failing after I pull this in. I assume the fault is within the tests, not with the code.

Ok, should be fixed now!
There were basically two issues:

Duplicated calls of os_init and sysinit. Since tu_init already calls those functions, I removed them from some of the testcases which were creating duplicated tasks.

Some tasks where being created with same priorities as tasks that already existed. At least one I did not find where the original task was created (with prio=2) but I changed the tests to start at some other initial prio.

The calls to os_start I also had to remove because those tasks functions don't have an infinite loop in them and there is an assert that checks that started tasks never finish! Someone should do some careful review of this particular change (if deemed necessary).


bump ntp-client revision to 1.9.1

Issue Number

N/A
Overview

470183eb832dc270b2fb11830edea30ab7777da6
:round_pushpin: bump ntp-client revision to 1.9.1
This revision includes the Win32 library which makes it work on Windows.

Comments

bors r+


logged user name

Hi Im not sure this is a bug I have logger install on two servers one for dev the other production. But for some reason I get different users names when logging. 1 always has the user a APEX_PUBLIC_USER as the user name (production 19.1) while dev server shows the user_name as the LDAP user who's logged into apex.
I just created a quick test and I called logger but got different user when the same use was logged into both.

Is this a bug Im not sure why they would be different the Oracle Apex workspace is the same on both hosts I copied from dev to production when we created the new workspace so they both have the same work_space_id and both have the same setting and apex version the only difference is that Oracle Database  is 12.2c and the other 19.1c both are on apex 20.1

As I said Im not sure if its me just making a mistake or a bug as I have the same settings on both hosts the only difference is the database version not Apex and all Logger version are the same
APEX 20.1
ORDS 19.2
Oracle DB, 12.2c & 19.1c
logger version 3.1.1

@apps4u I don't think this is a Logger problem. When you see the APEX_PUBIC_USER being logged what is the APP_USER in APEX showing?


Improving Aegir test speed

TLDR;
Are y'all interested in some refactors?
That would...

reduce testing time (from 4000ms to 45ms per test)
utilize the extra time to introduce more tests for more descriptive coverage
save money on CI
remove side effects from ./src/utils.js (making it easier to add new commands)

PS: I will break code examples here into smaller more manageable chunks for commits / PR reviews. The linked code was made as a proof.
Is your feature request related to a problem? Please describe
When running the tests for Aegir, most tests take 4+ seconds to finish
Describe the solution you'd like
Ideally, Aegir should not take several minutes to run its test suite
Describe alternatives you've considered

using command.parse in tests over execa
remove reliance on outside modules through dependency injection (more context below)

Additional context
Using command.parse over execa
Running tests with execa forces additional overhead by opening a new node instance for each test.
By using command.parse we use the existing instance and dodge this overhead. The caveat/blocker is src/utils.js . It has side affects on import that make it difficult to test commands in isolation.
See: https://github.com/ipfs/aegir/commit/506644efdb1d4a41369fc2d2103fe9df947dd5eb
Results
We gain almost 2 seconds back per test by doing this.
Default tests with execa and module dependencies
  build
    esm
      ✔ should build an esm project (4005ms)
    ts
      ✔ should build a typescript project (4031ms)

Tests run from command.parse
  build
    esm
      ✔ should build an esm project (2394ms)
    ts
      ✔ should build a typescript project (2287ms)

Removing reliance on outside modules from tests
Completely removing them wouldn't be great since we still want to make sure that it works from end to end.
But we could still pull some dependency injections to test aegir independently.
See: https://github.com/ipfs/aegir/commit/bfa18e43c73c560870a4ceb42113281704f9ecd0
Results
Running the typescript command on its own takes 2-3 seconds for a small project. If you look back to the tests run with command.parse you'll notice that's about how long running command.parse takes.
With some dependency injection, that type drops to a negligible value. Compared from the original 4 seconds, it drops down to 45ms.
Default tests with execa and module dependencies
  build
    esm
      ✔ should build an esm project (4005ms)
    ts
      ✔ should build a typescript project (4031ms)

Tests run from command.parse and dependency injections
  build
    esm
      ✔ should pass the appropriate options to injected dependencies (45ms)
      ✔ should build an esm project (2342ms)
    ts
      ✔ should build a typescript project (2280ms)

I've since cleaned up the examples I made into more consumable chunks to be easier to understand.
Here is a draft pr (on my fork of aegir) to showcase the changes. If this seems like something y'all are interested in, I'll drop the PRs over here too.
https://github.com/RonaldZielaznicki/ipfs-aegir/pull/5
Here is what a test run looks like with these changes:
  build
    should pass the appropriate options to dependency
      tsc
        ✔ passes preferLocal (257ms)
        ✔ passes stdio (149ms)
      esbuild
        cli options
          ✔ passes build to metafile (158ms)
        default config options
          ✔ passes conditions (129ms)
          ✔ passes format (99ms)
          ✔ passes minify (108ms)
          ✔ passes sourcemap (104ms)
        derived options
          ✔ passes pascalCased globalName from package.name (110ms)
          ✔ passes entrypoint "/src/index.js" for none-typescript project (102ms)
          ✔ passes entrypoint "/dist/src/index.js" for typescript project (106ms)
          ✔ passes outfile (105ms)
          ✔ passes umdPre as js banner (102ms)
          ✔ passes umdPost as js footer (100ms)

Closing due to a lack of input.


Is it possible to make nonzero jittable

Currently, it's not possible to do things like
from jax import jit
import jax.numpy as jnp

jit(jnp.where(a>1))(jnp.arange(3))

because nonzero is not jittable. Is there some trick that would accomplish the same thing as jnp.where(condition)?

Suppose also that you know how large the output should be, e.g. jnp.where(a=jnp.max(a)) where a has a unique maximum.

I guess something like,
#jnp.where(a=jnp.max(a))
jnp.unravel_index(jnp.argmax(a.flatten()), a.shape)

So one way to use this for a general jnp.where(condition) is to use unravel_index on all elements and then to use a mask in any downstrea operations.

Good question! There isn't a good trick to get the same thing as "unary-where" jnp.where(condition), though I expect we'll add support for it to jax.mask soon.
Usually there is a good way to re-express the overall computation involving the unary-where to avoid its use, but it's a case-by-case thing.
For that reason I'm not sure if we'll be able to provide a more general answer to this question (though we're happy to help look at specific examples!). WDYT?

This seem reasonable. My need was to select largest location in an array with one unique maximum, which I was able to do without unary-where as able. I don't know about jax.mask. What is its intended purpose?

Indeed, we haven't documented jax.mask, because it's just a prototype! It's a tool for supporting certain kinds of shape polymorphism. We'll post a design doc or tutorial at some point in the next few months, so keep an eye out!
In the meantime I'll close this issue, but please don't hesitate to open others!


feat: support disable dialect's feature

Issue: #1045

I'd say this should be done one the dialect level, e.g.:
myDialect := pgdialect.New(pgdialect.WithoutFeature(...))

And then the db will just use the features from the dialect:
db := bun.NewDB(sqldb, myDialect)

The problem with your current approach is that now we have bun.DB with one set of features and dialect with another set features. It might be that we currently don't use dialect.features anywhere, but it will probably create a problem in the long term.

And i dont want to add method for every dialect.

We can try to create BaseDialect and extract this functionality to the dialect package, but TBH it looks easier to just copy/paste 2 simple methods in each package.

@vmihailenco
Let use remove DB.features, and use only the dialect?

Looks good :+1:
I still think that options should be named WithFeature and WithoutFeature, not AddFeature and RemoveFeature.


Using Simba SQL expressions on Spark 2.1

Hi guys,
Am I correct in understanding that it's currently not possible to use Simba SQL mode on a Spark 2.1 cluster? I'm much more comfortable using Spark SQL than scala, so would love to hear how you guys are going in migrating this feature.
Thanks!

Yeah, the SQL mode is not available on the standalone version. It is basically because we need to extend the SQL parser to support that and the new antlr based parser is not quite extensible.
Anyway, I will try to find another way around when I have time (maybe make a separate parser path in Simba rather than reuse the parser in Spark SQL).
Thanks for your interest to our project.
Dong
On 5/30/2018 8:36:04 AM, Gilles<EMAIL_ADDRESS>wrote:
Hi guys,
Am I correct in understanding that it's currently not possible to use Simba SQL mode on a Spark 2.1 cluster? I'm much more comfortable using Spark SQL than scala, so would love to hear how you guys are going in migrating this feature.
Thanks!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub [https://github.com/InitialDLab/Simba/issues/92], or mute the thread [https://github.com/notifications/unsubscribe-auth/AAu4JgrkIskHGfDVGVFeeG7yOaJoiqrPks5t3rxhgaJpZM4UThr3].

Thanks, I appreciate that!
I'll start experimenting with the example code you've put up in the mean time. Is any overview available of all functions with their purpose and required parameters?
Cheers,
Gilles

Sorry for late response. I think the best way is to refer the DataFrame or DataSet API. I believe most of them are already in the examples located in the example source folder.


[FEATURE REQ] Distributed tracing for EventHubs: add enqueueTime to Process span links

Is your feature request related to a problem? Please describe.
One of the interesting questions for EventHubs tracing is how much time message spent in EventHubs (or when it was enqueued). It helps to detect worker scaling issues.
Today, we trace when message was sent and also when processing has started. This, however, only helps if the same tracing tool is used on both sides AND if users have permissions to access telemetry from both sides.
Also, assuming heavy batching scenarios, it's hard for analysis/visualization tool such as Azure Monitor to calculate and visualize aggregated information about 'time in queue'.
Describe the solution you'd like
EventData provides information about EnqueuedTime already and we should populate it on the telemetry to help customers easily understand how much time message spent in the hub.
Please add enqueuedTime attribute on each link when processing messages: unix epoch time with milliseconds precision representing when message was enqueued (x-opt-enqueued-time system property). Attribute value should have long type.
Describe alternatives you've considered

Populate time in queue as a metric. this requires heavy investment into metrics but otherwise a valid long term approach. However metric does not help answer questions about particular message.

Do not populate, let users log Enqueued time. this won't allow tracing tools to analyze/visualize this value in cases when telemetry from just consumer side is available. It will also complicate analysis for cases  when telemetry from producer and consumer is available.

Additional context
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[x] Description Added
[x] Expected solution specified

@lmolkova
Sorry for the delay, we're addressing this in #8745.
One comment brought up was about casing for the enqueuedTime property. Does this need to be the same casing across languages and if so, is camelCase the intended format?

This change is now available in @azure/event-hubs version 5.2.1:
<EMAIL_ADDRESS>Here's some more info on all the changes that went into this version:
https://github.com/Azure/azure-sdk-for-js/blob/master/sdk/eventhub/event-hubs/CHANGELOG.md#521-2020-06-08


Support cjs/amd export without global export

For instance in Backbone.Radio "Radio" is not attached to root, but you can still:
var Radio = require('backbone.radio')
With this boilerplate if you export it is all or nothing.
Not sure if this is a limitation of esperanto or what, but it'd be a nice to have option.

?

If you were to convert backbone.radio to babel boilerplate,  and added it via a script tag it would be available via backbone.radio and window.radio.  The latter is not true with radio in its current state.

You sure? I think I do what you're describing here.

So correct me if I'm wrong here, but with this, while its not available globally via a <script> tag and only attached to moment, this does not allow you to do:
var something = require('moment-business');

With Radio you can both require it into a variable, and not expect it to be in the global namespace if added via script

Oooh, it's finally coming together. I see what you mean. Good point.

Not sure if this is a limitation of esperanto or what, but it'd be a nice to have option.

Would you care to look into whether Esperanto supports it or not?

Turns out this is relatively easy and already supported.
https://github.com/esperantojs/esperanto/issues/126
For my use-case for instance this was simply:
"babelBoilerplateOptions": {
    "entryFileName": "marionette.toolkit",
    "exportVarName": "Marionette.Toolkit",
...


Encode Hackathon - Bug bounty for Harmony Marketplace SDK (Coding errors / Bugs)

Description
Related to the Encode Hackathon, this bounty is for promotion of the Harmony Marketplace SDK
Context
Identify software / api / coding errors in the Harmony Marketplace SDK and do a PR to fix the errors. Limited to 5 submissions
Acceptance Criteria

[ ] Document the error found in the SDK on this bug bounty
[ ] Link to the PR that is pushed to the Marketplace SDK solving the bug

Reward
We will reward based on the errors found and may bundle a few errors together for payment
5 bounties up to $150 USD in ONE per bounty

@jacksteroo please review and confirm

Confirmed

This bounty still active?


Mypy error importing plugin

Summary
I'm not able to use the mypy plugin
Reproduction Steps
First I created a venv and installed disnake by this command
pip install git+https://github.com/DisnakeDev/disnake.git

And then I installed mypy
pip install mypy

I created the mypy.ini file with the content that the documentation suggests
[mypy]
plugins = disnake.ext.mypy_plugin

And finally, I made a very simple file, just to have something for mypy to analyze
# main.py
from disnake.ext import commands

bot = commands.Bot(command_prefix="...")

print(bot)

And when I run mypy, it doesn't find the plugin
$ mypy .
mypy.ini:2: error: Error importing plugin "disnake.ext.mypy_plugin": No module named 'disnake.ext.mypy_plugin'
Found 1 error in 1 file (errors prevented further checking)

Expected Results
Expected is that mypy will find the plugin
Actual Results
Mypy can't find the plugin
Intents
without intent
System Information

Python v3.10.1-final
disnake v2.3.0-beta

disnake pkg_resources: v2.3.0

aiohttp v3.8.1
system info: Linux 5.15.16-1-MANJARO # 1 SMP PREEMPT Thu Jan 20 13:58:19 UTC 2022

Checklist

[X] I have searched the open issues for duplicates.
[X] I have shown the entire traceback, if possible.
[X] I have removed my token from display, if visible.

Fixed by #316, thanks for the report!
side note: mypy needs the --namespace-packages flag to properly check types in disnake.ext.* (unless you just omitted that in the example)

Fixed by #316, thanks for the report!
side note: mypy needs the --namespace-packages flag to properly check types in disnake.ext.* (unless you just omitted that in the example)

OK thanks. I didn't know I had to use this flag


ngx-jodit: icons always showing text, not SVG / PNG

Package versions

package
version

Angular
16.2.6

jodit
4.0.0-beta.87

ngx-jodit
^3.0.0-beta.8" (same behavior for v2)

The toolbar for Jodit is always showing text buttons, never icons for the buttons. Group buttons display icons correctly, but buttons such as (e.g.) bold show the word 'bold', not the icon.
The actual bold functionality works fine.

This occurs when i supply any list of buttons to the jodi [options] input, e.g.:
public joditOptions: any = {
    buttons: ['bold', 'underline'],
    textIcons: false
}

----

<ngx-jodit [(value)]="text" [options]="joditOptions"></ngx-jodit>

Setting textIcons to true transforms the 'menu' icons to text.
Any ideas what i'm doing wrong ?
thanks

@ivan-price-acted thanks for reporting this issue! Maybe related to #8 . I'll have a look at it asap.

@ivan-price-acted I just released new versions of ngx-jodit 2.x and 3.x. Please update to the latest version.
The issue occurred because typing/esm issues with the jodit package. I don't know why but the Jodit developers don't get the typings working like other packages. That's why we have to use the compiled JS files from Jodit. That's a breaking change, please have a look on the new installation instructions. It's related to #9.

@julianpoemp thanks for your rapid response !
i can confirm that after bumping to version 3.0.0-beta.9, and adding "node_modules/jodit/es5/jodit.fat.min.js" to the angular.json scripts list the icons are back.. however i'm concerned that including jodit.fat.min.js (>700K !) will increase the weight of our app with functionality we won't use.
it seems a high price to pay for icons in the toolbar, we would probably prefer text icons and save the bandwidth for our network-sensitive application. Is there any other workaround, are Jodit aware of this limitation ? We are planning on paying for the pro version of Jodit, should we take it up with them ?
thanks very much for your work on this project !
-i

@ivan-price-acted did you try one of the other jodit js files? Perhaps there is a smaller one.
At the moment I don't recommend to use Jodit if you are looking for well written Typescript libraries. I didn't get the import of Jodit working for ngx-jodit because there are any issues every time (like your issue with the icons).
Btw Jodit v4 and jodit-pro v2 are in beta, I don't recommend to use it on production.
I really don't know what Jodit is doing wrong. Perhaps something with their build system.
The problem of the large size is because we are not using ESM modules so Angular imports commonjs builds without the option to select only code it needs. Jodit should get ESM working.

Hi there,
i tried out a custom build, and got it down to 300K.. but that still seems a lot considering we only want very simple functionality.. and we'd be obliged to re-build manually and no longer manage the version via our package.json which i don't love.
i thank you very much for this library and your work on this subject, it may be that we end up using the custom build if we can't find anything smaller / mode modular.

@ivan-price-acted I reported this issue to Jodit (#1029). Perhaps a solution will be found soon.

update: Jodit is working on a solution. It's confirmed that this issue is a bug. I'm confident that we can use the ESM version again. As soon as I can confirm it's working I'll release a new version of ngx-jodit 2.x and 3.x .
Reopening this issue.

ok thanks @julianpoemp we'll take a look at that !


Matplotlib

There are several references to a "Matprolib library" in this article where I believe the correct name is Matplotlib.

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

ID: 9c6b7b4f-9f34-cec1-687c-2f5c708fe99f
Version Independent ID: e66e6e43-157d-8331-20e2-89d1379c5251
Content: Create Power BI visuals using Python in Power BI Desktop - Power BI
Content Source: powerbi-docs/desktop-python-visuals.md
Service: powerbi
Sub-service: powerbi-desktop
GitHub Login: @otarb
Microsoft Alias: otarb

Thanks for your patience and your good eye, @agustingc ! These are fixed, will be published at the latest Mon Apr 6.

Thanks for your patience and your good eye, @agustingc ! These are fixed, will be published at the latest Mon Apr 6.

Welcome!


Add license headers where missing.

Codecov Report

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

@@           Coverage Diff           @@
##           master      #82   +/-   ##
=======================================
  Coverage   78.87%   78.87%
=======================================
  Files          22       22
  Lines        1505     1505
=======================================
  Hits         1187     1187
  Misses        244      244
  Partials       74       74

Impacted Files
Coverage Δ

daisy/workflow/step.go
91.95% <ø> (ø)
:arrow_up:

daisy/workflow/instance.go
60% <ø> (ø)
:arrow_up:

daisy/workflow/resources.go
95% <ø> (ø)
:arrow_up:

daisy/workflow/sources.go
67.64% <ø> (ø)
:arrow_up:

daisy/workflow/image.go
60% <ø> (ø)
:arrow_up:

daisy/workflow/disk.go
60% <ø> (ø)
:arrow_up:

daisy/compute/test_client.go
96.42% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


[CSL-1565] Refactor TxError

This is needed for CSM-399.

Please ping me once it stops being WIP.

@Martoon-00 can you approve if everything is ok now?


"Arbore" is listed in the drop-down menu

"Arbore" is listed in the drop-down menu alongside its corresponding county according to the law
Precondition:
The website is up an running.

Step 1
Type "Arbore" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Arbore, SUCEAVA".


gat: 0.17.0 -> 0.18.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/koki-develop/gat/releases.
meta.description for gat is: Cat alternative written in Go
meta.homepage for gat is: https://github.com/koki-develop/gat
Updates performed

Golang update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.18.0 with grep in /nix/store/dlx70n9i64ga81ppp4b8dlx6kby9a4vq-gat-0.18.0
found 0.18.0 in filename of file in /nix/store/dlx70n9i64ga81ppp4b8dlx6kby9a4vq-gat-0.18.0

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

gat

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/dlx70n9i64ga81ppp4b8dlx6kby9a4vq-gat-0.18.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A gat https://github.com/r-ryantm/nixpkgs/archive/a456f43e0b5fef812261720d86c38f43d3dfeda4.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/a456f43e0b5fef812261720d86c38f43d3dfeda4#gat

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/dlx70n9i64ga81ppp4b8dlx6kby9a4vq-gat-0.18.0
ls -la /nix/store/dlx70n9i64ga81ppp4b8dlx6kby9a4vq-gat-0.18.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

gat

Maintainer pings
cc @TheMaxMur for testing.

@TheMaxMur you can run the merge bot command if you are OK with the changes.
Write this in a reply message:
@NixOS/nixpkgs-merge-bot merge

@NixOS/nixpkgs-merge-bot merge


builder: Use docker cp instead of rsync when possible

... because docker cp is much faster than rsync, especially with a remote Docker host.
Reviewer: Please check my use/non-use of quotation marks.

Yes

Cool. :)


Multithreading CC Selfplay

I was looking at the C++ code for the self play and I was wondering if it could be enhance to be multithreading to generate more output. I guess a solution for multiple OS has to be found or is it to be assumed that it runs only on Linux (unix) clients?

Yes, we plan on adding multithreading support soon.
On Wed, 30 May 2018, 04:30 marcocalignano<EMAIL_ADDRESS>wrote:

I was looking at the C++ code for the self play and I was wondering if it
could be enhance to be multithreading to generate more output. I guess a
solution for multiple OS has to be found or is it to be assumed that it
runs only on Linux (unix) clients?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/minigo/issues/250, or mute the thread
https://github.com/notifications/unsubscribe-auth/AHPFxVKZ7ZoLojNwu2hPHIxowsgPW11iks5t3oLGgaJpZM4UTGHv
.

Although I should clarify that we're planning on adding multithreading specifically for Cloud TPUs.
We generally see >95% GPU utilization during single threaded selfplay, so it's not clear that adding more threads will improve GPU selfplay performance significantly.

Unfortunately I cannot compile tensor flow on my machine otherwise I would implement that.

@marcocalignano Tom's got a PR in progress for the multithreading; what problems are you having compiling tensorflow on your machine?

Basically I have Kubuntu 18.04 and the default compiler is gcc-7 which apparently is not really liked by the configure_tensorflow.sh script.
I tried to change the default gcc links to point to gcc-6 but I still have some problem with c++ code, I guess I didn't change enough. I will have to try a bit more!

Can you try compiling with Bazel 0.11.1 (and gcc-6)? We have experienced some issues with later versions: https://docs.bazel.build/versions/master/install.html


Recommending Hooks API before it is has been accepted in React

First off, thank you for the great library.
I realize that @acdlite is a core dev on the react team, but Is it smart to announce no future maintenance and recommending hooks when the feature is still a proposal and has not been accepted into the core react library? Would also love to see some examples on how you believe the new hooks api solves the same problems, as I personally am still not seeing the api to be as powerful as the composability of a library like recompose.

Have u read this https://reactjs.org/docs/hooks-custom.html ?

You would have much more freedom with hooks

I had not, you are right this could help a lot.
Isn't this still just a proposal? Which means it may not be added to React core.

I think it's coming no matter what, but the names / implementation might change slightly.
Facebook said they've been using it in prod for a month already.

Nothing is changing in practice. Andrew’s wording was too strong but he didn’t mean the library would suddenly become unsupported. Your code will continue to work, and new versions for newer React updates will continue to be released.
See my reply in https://github.com/acdlite/recompose/issues/756#issuecomment-438674573.

Thanks @gaearon. I am going to close this issue as #756 seems to be discussing the same thing.


create a blog entry in the wiki

Added content to the sections of the blog, as created.

great, i think the editing team wants these to be a bit longer, but this is good enough for a first pass! closing this issue


Tooltip

Any reason kivy doesn't have tooltip? Yes, can be done using a workaround. Would be nice to have tooltip as a standard feature.

This is a duplicate of https://github.com/kivy/kivy/issues/6717

This is a duplicate of https://github.com/kivy/kivy/issues/6717

Feature #7658 enables writing of hover behavior and that's not far from tooltip behavior mixin.
You can find working hover event manager and hover behavior at https://github.com/pythonic64/hover.


Feature: --listen

Is there a way to add the --listen flag to access the ui from other computers on the net. Would it be as simple as me changing line 63 in util.py
from    app.run(host="localhost", port=config["port"], use_reloader=False)
to    app.run(host="<IP_ADDRESS>", port=config["port"], use_reloader=False) ?

I tried changing the host and port in app.run() to make it accessible to the internet but unfortunately it did not work for me.
As you suggested, I think it is a feature that can be considered.

Got it to work by doing this:

open chatdocs.yml and add ip: <IP_ADDRESS> above the port line.

Open ui.py and replace localhost on line 63 with config["ip"]

then open a cmd window like your going to run chatdocs ui but instead do:
pip uninstall chatdocs
pip install ./
This will reinstall / install with the new changes and it will be able to be accessed via the network.


Add scrollable-image

task: [scrollable-image] scrollable-image component
task: [scrollable-image] scrollable-image embedded code builder

First make build-dist would occur error.
webpack didn't create dist folder for BundleListPlugin.

You can see the error here
https://d.pr/free/i/QxKvK2

First make build-dist would occur error.
webpack didn't create dist folder for BundleListPlugin.

Updated. Create a dist folder  if it doesn't exist.

Updated.

Updated.

update the heading of some commits (fix -> refactor)
address review comments since commit 7085249

LGTM

This PR has rebased https://github.com/twreporter/orangutan-monorepo/pull/7.
Please have a see on commit a2d1dc0

LGTM

LGTM 👍 💯


ValueError: too many values to unpack (expected 5)

Traceback (most recent call last):
File "kickthemout.py", line 754, in
defaultInterface = getDefaultInterface()
File "kickthemout.py", line 154, in getDefaultInterface
network, netmask, _, interface, address = max(iface_routes, key=lambda item:item[1])
ValueError: too many values to unpack (expected 5)
OS : Deepin v15.7
Python : 3.6.5
pip : 9.0.1
python3-nmap : 0.6.1
python3-scapy : 0.23
python3-netifaces : 0.10.4

I use Arch linux , i removed packages python2-scapy , python2-netifaces and python2-nmap and uninstall Kickthemout from AUR, next git clone from here and sudo -H pip3 install -r requirements.txt

Unfortunately yes .

Can you try running pull req #294. Let me know if that works for you. Thanks!

thank it's work for me .
by adding
try:         network, netmask, _, interface, address = max(iface_routes, key=lambda item:item[1])     except:         network, netmask, _, interface, address, _ = max(iface_routes, key=lambda item:item[1])

Glad to hear that!


The OpenAI module cannot be imported.

Sorry for late reply.
The lib does not export OpenAPI class but ChatOpenAI. Please refer to the example


Savaradh containerregistry microsoft.container registry 2024 11 01 preview

ARM (Control Plane) API Specification Update Pull Request

[!TIP]
Overwhelmed by all this guidance? See the Getting help section at the bottom of this PR description.

PR review workflow diagram
Please understand this diagram before proceeding. It explains how to get your PR approved & merged.

Purpose of this PR
What's the purpose of this PR? Check the specific option that applies. This is mandatory!

[ ] New resource provider.
[x] New API version for an existing resource provider. (If API spec is not defined in TypeSpec, the PR should have been created in adherence to OpenAPI specs PR creation guidance).
[ ] Update existing version for a new feature. (This is applicable only when you are revising a private preview API version.)
[ ] Update existing version to fix OpenAPI spec quality issues in S360.
[ ] Convert existing OpenAPI spec to TypeSpec spec (do not combine this with implementing changes for a new API version).
[ ] Other, please clarify:

edit this with your clarification

Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:

[x] I confirm this PR is modifying Azure Resource Manager (ARM) related specifications, and not data plane related specifications.
[x] I have reviewed following Resource Provider guidelines, including
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can proceed to the diagram Step 2, "ARM API changes review", for this PR.

Additional information

 Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.

Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
suppressions guide to get approval.

Getting help

First, please carefully read through this PR description, from top to bottom. Please fill out the Purpose of this PR and Due diligence checklist.
If you don't have permissions to remove or add labels to the PR, request write access per aka.ms/azsdk/access#request-access-to-rest-api-or-sdk-repositories
To understand what you must do next to merge this PR, see the Next Steps to Merge comment. It will appear within few minutes of submitting this PR and will continue to be up-to-date with current PR state.
For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
For help with ARM review (PR workflow diagram Step 2), see https://aka.ms/azsdk/pr-arm-review.
If the PR CI checks appear to be stuck in queued state, please add a comment with contents /azp run.
This should result in a new comment denoting a PR validation pipeline has started and the checks should be updated after few minutes.
If the help provided by the previous points is not enough, post to https://aka.ms/azsdk/support/specreview-channel and link to this PR.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Microsoft.ContainerRegistry

The first commit needs to be an exact copy of the previous API version. All new changes should only be added in the subsequent commits.
This allows the reviewer to get a clear understanding of the actual changes being introduced. With the way the PR is raised now, it is not possible for the reviewer to tell what the changes are. Please either abandon the PR and raise another one with the recommendation or create a new set of commits on this PR following the recommendation. If you are doing the later option please indicate which commit is the exact copy of the previous version.

@razvanbadea-msft I was following the new API guidelines and the first three commits were added by the script Copy-ApiVersion.ps1. I noticed that the script messed up the new API directory structure and the readme file. Had to fix those in the next two commits.
Sixth commit is the one that shows all the new properties and changes. Hope it is clear and as expected. Please let me know if you still want me to abandon the PR and recreate a new one. Honestly, the script does a really bad job. IMO, simple copy/paste and a search/replace is a much better option.

Raised an issue for python sdk failue - https://github.com/Azure/autorest.python/issues/2924
All the other SDKs work as expected

Closing this pull request as a new PR is created as requested - https://github.com/Azure/azure-rest-api-specs/pull/31612


Upgrade auto-conversion hook to remove deprecation

The current hook proposed in the best practices section of
CONTRIBUTING.md to auto-convert notebooks to .py and .html formats
has a slightly minor problem: it uses a deprecated command, as stated
in the stdout every time the hook gets called:
[TerminalIPythonApp] WARNING | Subcommand `ipython nbconvert` is
deprecated and will be removed in future versions.
[TerminalIPythonApp] WARNING | You likely want to use `jupyter
nbconvert` in the future

This PR fix the recommendation, orientating users to use the new
approach (jupyter nbconvert) instead of the old one. There should not
be any side effects.

Good catch @samuelgrigolato.


Replace console.log with a different way of showing how a function works

Challenge Write Reusable JavaScript with Functions has an issue.
We might want to use something other than console.log for the function example so we don't have to worry about trying to show them how to open developer tools (which depends on their browser).
Developer tools are covered in the video challenges - there's no need to cover them here in the JavaScript section.

I'd be interested in tackling this. Would using an alert (albeit very annoying) be a better idea over the console.log?

Beginners like me trying the other ways will hate alert sometimes. For example:
var data = {a: 'foo', b: 'bar'};
console.log(data);
alert(data);
You'll see that console.log shows you the object, while alert gives you [object Object], which isn't useful

why don't we just return the output than console.log .. Any particular reason for console.log??

I had a look at this yesterday and my thought was to use alert in the first lesson and switch the order of the lessons so that "Return a Value from a Function Using Return" is the lesson after this. That way, alert only gets used in one lesson (and with a string), they don't have to use console.log, and the return statement will also be explained immediately when introduced.

What about document.write("Heyya, World"). Would it work?

What about just adding an "as well" to the end of:
/**
  * Your output will go here
  * Any console.log() -type
  * Statements will appear in
  * Your browser's DevTools
  * JavaScript console
  */

since the results do indeed show up there.
And maybe add a "For more info about DevTools" link to the video challenges?

@QuincyLarson I got this weird idea today.. i am not sure this is possible or not.. we got some  empty space at bottom of every challenge.. Maybe we could have a proxy Console the one which works in every browser.. like the one CodePen and Jsbin  are having..  CodePen had mentioned in their blog Thanks to JS Bin for some of the code that they used the code from the open source code... There could be some way or some one could build this.. then we don't have to worry about trying to show them how to open developer tools

@ChaituVR We have thought about that, but we don't want to clutter up the UI. It's tempting to fill up empty space, but it makes things much more complicated for new comers who are like "what's this box down here? So complicated." and then leave.
I agree with @ChadKreutzer that we should update the copy to say "as well."
Regarding @vcooley's suggestion, the intention of the challenge is to introduce the console.log function, which is commonly used in debugging, so I don't know whether alert() or @darwinrc's suggestion of document.write() will suffice either.
I think since we are expanding our challenges and creating new quality assurance/testing challenges, we will ultimately get rid of this challenge or introduce the concept of logging in a more comprehensive, proper way. So I will close this issue for now and open a new one for "as well".


Updating the get kubeconfig command

There is no get-kubeconfig.sh, could see only  get-admin-aks-kubeconfig.sh get-admin-kubeconfig.sh
under Dev RP ./hack, updating the command with the correct script name.

I'm not sure what "Dev RP" means. The document is referring to this script in the ./hack/ subdirectory of this project (openshift/hive). Am I missing something?

I was referring to Dev RP Hack
Just rechecked it, so that is a different get-kubeconfig specifically for the hive hack. Closing this PR. Thanks :)


GRpc error: DNS resolution failed

<main>: Error: GRpc error: (14): DNS resolution failed for grpc://<endpoint>: UNKNOWN: Name or service not known

The error occurs when trying to make a request from the dst cluster to the src cluster via ydb_proxy. Need to figure out and fix it.

It turned out that the endpoint should be specified without grpc[s]://. The ability to specify a scheme is only available in connection string. Will be continued here: #2494


Per label and global metrics return 0

Describe the bug
During training most metrics return 0 instead of actual value.
To Reproduce
Steps to reproduce the behavior:

Use this metrics setting in config.yaml:

metrics:
- classification_accuracy
- f1:
    average: weighted
- accuracy
- balanced_accuracy
- per_label_accuracy

Run training of imagenet_vgg16 (binary classification task)
Check job output
See error

Expected behavior
Metrics should return actual values instead of 0.
Screenshots

GaNDLF Version
0.0.16-dev
Version information of the GaNDLF package in the virtual environment.
Desktop (please complete the following information):

OS: Linux

@sarthakpati please assign it to me


Some operators (e.g., and, or, xor, is) are missing from autocomplete within the conditions of an if-statement

1. Describe the bug
If you type a if-statement and want to combine multiple conditions, autocomplete does not provide these operators:

and
or
xor
is (Preview feature)
maybe more, which I didn't consider

2. To Reproduce
see screenshot
procedure Foo(): Text
var
    Bool: Boolean;
    Bool2: Boolean;
begin
   if Bool and Bool2 then
     exit('');
end;

3. Expected behavior
I expect that the autocomplete provides these operators.
4. Actual behavior
Autocomplete does not provide these operators.
5. Versions:

AL Language: v14.0.1055126 (pre-release)
Visual Studio Code: 1.90.2

I just noticed that "and", "xor", "or" and some other keywords are also not shown for c#. So I think this is "by-design".


Cannot grant "browse" scope on new authorizations

Here's my exact authorization URL: https://www.deviantart.com/settings/applications?state=%2F&scope=user+browse&redirect_uri=https%3A%2F%2Fmlpvc-rr.lc%2Fda-auth&response_type=code&client_id=2786 (Note: redirect_uri points to my local testing domain, it won't load an actual page after authorizing)
Note the browse in the scope parameter, yet the page does not display the bullet point for "Browse public information" nor will the token have that scope set once I get a response from the API.
Personally I'd be the happiest if user/whois did not require the browse scope and could be used with a simple client access token, but if the scope is a must-have requirement then please make sure I can actually request that scope from the users.

Try to use https://www.deviantart.com/oauth2/authorize instead of  https://www.deviantart.com/settings/applications.
First one redirects user to second one but with updated scope.

I am actually using that, I just grabbed the URL from the browser which was visible after the redirection.


Add artifacts to .gitignore

I think it is best practice to add the build results to .gitignore to prevent accidental commits and repository bloat.
To make discovery of the current artifacts easier they could be attached to a github release and the 'release' folder removed from git. (You may want to rewrite the history to reduce the size of the git repository.)

thanks @everflux  - I will try this

This is done @everflux  Thanks for the idea - Closing the issue
https://github.com/AKSarav/Kube-Node-Usage/releases/tag/v3.0.1


Use overlapped interface with NamedPipe

This pull request has two primary changes:

Using GetOverlappedResult instead of lpNumberOfBytes{Read,Written} in ReadFile and WriteFile for read_overlapped and write_overlapped, respectively.  According to https://docs.microsoft.com/en-us/windows/desktop/api/fileapi/nf-fileapi-readfile and https://docs.microsoft.com/en-us/windows/desktop/api/fileapi/nf-fileapi-writefile, if the file was opened with FILE_FLAG_OVERLAPPED,

The lpNumberOfBytesWritten parameter should be set to NULL. To get the number of bytes written, use the GetOverlappedResult function.

Noting that all NamedPipes are created with FILE_FLAG_OVERLAPPED, their read and write implementations should use the lpOverlapped parameter.  Create a variant that will wait for the completion before returning to satisfy the simple read and write interface.

This seemed the simplest change that didn't break the interface defaults but am open to suggestions / alternate solutions that use the overlapped interface with handles opened using FILE_FLAG_OVERLAPPED.

Thanks! The only suggestion I'd have is that for the blocking read/write impls could this perhaps use a thread local cached even to avoid recreating one on each call?

good suggestion -- I've made it use a threadlocal!

Thanks!

Do you mind cutting a new release with this included when you get a chance?  Not a huge rush for me :)

Sure thing, done now!