OracleContainer needs further configuration

The latest version of oracle database needs a slash between the port and the sid of the jdbcUrl: jdbc:oracle:thin@//[hostname]:[port]/[DB service name]
resources:
https://www.oracle.com/dk/database/technologies/appdev/xe/quickstart.html (under java connection)
https://docs.oracle.com/database/121/JJDBC/urls.htm#BEIDHCBA
Also the SID should be configurable (in the same way as username/password is)

@vanwobe I think there is some confusion here between service name and sid.  When using jdbcURLs you should refer to a sid by using a colon:
jdbc:oracle:thin@//[hostname]:[port]:[sid]

and a service name using a slash
jdbc:oracle:thin@//[hostname]:[port]/[DB service name]

In this case, I believe testcontainers is only supporting XE versions of Oracle.  Therefore, our current jdbc configuration is correct.  XE is both the SID and Service Name.  And the is not way to change that in the XE version.
See documentation here: https://github.com/oracle/docker-images/tree/master/OracleDatabase/SingleInstance#running-oracle-database-18c-express-edition-in-a-docker-container

I don't see how the information from your documentation link differs from the links I provided.  There is also a slash in the connect string:
sqlplus sys/<your password>@//localhost:1521/XE as sysdba
sqlplus system/<your password>@//localhost:1521/XE
sqlplus pdbadmin/<your password>@//localhost:1521/XEPDB1
The last line shows an example to connect to a PDB (instead of CDB). In this case we want to change the default XE into XEPDB1.
The use of a ':' is only valid in previous versions like xe 11g:
https://docs.oracle.com/cd/E17781_01/appdev.112/e18805/getconn.htm#TDPJD138
jdbc:oracle:driver_type:[username/password]@//host_name:port_number:SID

Ahh okay so sqlplus connection strings are different than JDBC connection strings.  The oracle JDBC driver will still allow you to connect using a URL with either a SID or Service name, but I understand where you are coming from with this. Let me see if I understand correctly:
So you would want something like a method:
    public OracleContainer withSeriviceName(String serviceName) {
        this.serviceName = serviceName;
        return self();
    }

That would allow you to add a serviceName.  Since a service name is more specific than a SID we could then have logic to say if a serviceName is present return a JDBC URL that in the form:
jdbc:oracle:thin@//[hostname]:[port]/[DB service name]

otherwise, default back to using the SID (for XE containers this will always be XE)
jdbc:oracle:thin@//[hostname]:[port]:[sid]

Indeed that would solve it.

@KyleAure sounds reasonable.
Since you already provided another Oracle PR, do you want to take care of this? I will test the Oracle PRs next week (currently limited bandwidth and therefore can't build the Oracle images for testing).

I was going to open another ticket but this is still open and seems to be related to what I'd like to propose. We're using Oracle XE for test containers and have run into a couple of shortcomings (or so I think, happy to be corrected!)

SID for XE can be either xe for the CDB (cluster level, non user DB) or xepdb1 for the PDB (pluggable database, the location for user data). When doing user-level stuff like creating an empty schema it should be done at PDB level I believe so this needs to be customisable or just hardcoded to xepdb1 for any of the more recent oracle versions.
Having Username and Password hardcoded to system and oracle creates issues when trying to run Flyway migrations before your tests start. This is because Oracle conflates users and schemas and the default system schema contains a lot of DB internal tables which causes Flyway to fail.

Can I suggest allowing configuration of username and password the same way as the other JDBC configurations do it from the ConnectionURL the same way the PosgreSQLContainerProvider does it? As PostgreSQL also has a default container configured with a schema, username and password configured to 'test' I thought it might be useful to provide an OracleXE container build script that does the same.
Happy to implement myself, just want to make sure I'm not doing something silly as I'm relatively new to TestContainers and Oracle.

Any news on this?


TTRF folder adjust

Combined frontend and backend
Added index and form page in server url

attc, buttons, routes


Logging in twice creates duplicate user

If you log in twice with the same account, it creates a duplicate user and the site breaks rather badly. I suspect some weird interaction between auth0 and our ensureUser / preferUser handlers, which have been problematic before.

Weird thing is that the users (both original and duplicate) are getting the wrong date for the join_date (Andrea got April, I got two days ago).
The good thing is that the duplicate and original have the same auth0_user_id so we should be able to prevent this relatively easily.


more social networks

Hi,
When will be more social networks added?

Hi @stojankukrika ,
I would like to get some free time to add new features and more social networks to this package but I'm afraid that I can not say when since I'm really busy...
Anyway, everyone is more than welcome to contribute and make this package bigger and better.


How to prevent azureml-fe pod failing to schedule on 2-node clusters

Hi,
When deploying to an arc-connected cluster with 2 nodes, azuremlfe fails to schedule the 3rd pod.
Is there any way to configure azureml deployment so that it can be deployed to 2- or even 1- node clusters?
I understand the recommendation is 3 nodes, but there are several scenarios where even 1-node clusters are relevant (edge or remote clusters for simpler workloads).
Thanks

Hi @joaocc ,  when you deploy the extension, you can set this config to false explicitly.

Great. Thanks!


Documentation Updates

Updated Documentation on Definitions of Terms used within Google DataStore
Updated Documentation on using annotations for building models

@Id
@Index

limits of Google DataStore

@SearchIndex

This improves #40, and #32


Upgrade pkgs for Carbon

Summary

Updated packages for carbon-components-react, carbon-icons, and @carbon/icons-react and fixed obvious regressions.

At some point we may want to upgrade to Carbon components v10 but this will have a much bigger impact on component visuals.
Related Issues

Closes https://github.com/IBM/carbon-addons-iot-react/issues/154

Acceptance Test (how to verify the PR)

go through component stories and make sure they are looking normal.

Carbon v10 package upgrades will be more fully contained through #304


fix(pstor): increase persistence timeouts

The pstor might might unavailable for some time, for example during upgrade. We should have sufficient timeouts to cope with this. Also pstor can get slow at times as it's writing to disk, so we also should have a larger timeout on store.
todo: should we still carry on trying to persist on separate task after we fail the nexus?
todo: what happens when we try to shutdown nexus first and pstor fails?
We'll revisit this post-release.

lgtm, maybe add a comment stating we want to revisit again.

Done.

bors merge

bors merge


create homepage with HTML/CSS

Home page with links to other pages

created a hompage with apps rendering on page using a django for loop


add connect_with_timeout to set connection timeout

I need set the connection timeout with shorter Duration and R/W timeout with longer Duration, so add new method  to set connection timeout instead of the global timeout.
fn main() {
    let mut session = ssh::create_session()
        .username("root")
        .password("root")
        .timeout(Some(Duration::from_millis(5000)))
        // .connect("<IP_ADDRESS>:22")
        .connect_with_timeout("<IP_ADDRESS>:22", Some(Duration::from_millis(1000)))
        .unwrap()
        .run_local();
    // Usage 1
    let mut shell = session.open_shell().unwrap();
    run_shell(&mut shell);

    // Close channel.
    shell.close().unwrap();
    // Close session.
    session.close();
}

Hi there,
Your idea looks good, but actually, there already is an API connect_bio that provides the feasibility of customising the underlying connection in any way.
I'm wondering if it is necessary to add more APIs to customise only TCP streams.
Is there any user story that connect_bio cannot satisfy?
BRs.

Thank you for you reply.
I think it would be better to add connect_with_timeout method.

There is a curving way to add a connection timeout by customising the TcpStream, including error handling (cannot reuse the error handling with SshError).
The connection timeout config is a very basic and commonly used feature for networking crates.
One parameter configuration timeout indicates two meanings, which may lead to misunderstanding. (yes, that's me)

Also, a alternative way is to add a timeout param for connect API, but it's not appropriate for minor version updates.

fair enough,
I can do the update later this week
BRs.

Hi, sorry for the late,
I've released a new version of ssh-rs 0.5.0, which includes your change.
Unfortunately, during the fix of another issue API change was necessary, so the minor version also increased.
BRs.


[Gally]: master <- dev

Automatically created by Git-Ally

:tada: This PR is included in version 1.13.150 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


bug(docs): sticky navbar broke links

Provide environment information
https://create-t3-app-nu.vercel.app/en/introduction#what-is-this-some-kinda-template
Describe the bug
Clicking the link above is broken cause the navbar is in the way
the header that we link to should be visible when navigating to it.
when fixing this, also add smooth scroll behavior :)
To reproduce
click the link
Additional information
No response

cc @gabrielelpidio @Coyenn

Fixed in #475


Error occurred errorMessage

hi
i have problem for connect local mattermost to open AI
i connect to the OpenAI API key and OpenAI organization ID but cannost search or ask qus in mattermost app
this error is Error occurred errorMessage

Hi,
the exact same thing happens to me as well.

Same issue. I do remember that it had displayed an issue with the colon after the base url

Same here. Mattermost Version 9.0.0 Plugin: v1.1.0 Message: {"timestamp":"2023-09-27 11:57:09.092 Z","level":"warn","msg":"Error occurred while processing the chat completion request","caller":"app/plugin_api.go:983","plugin_id":"open-ai","Error":"errorMessage "}

In my case, its my fault. Payment Plan not configured :)

I have the same issue.. :(

I have the same issue.. :(

what is mattermost version?

I'am using Mattermost 9.0.1 (server) and 5.5.1 as client.
I tried lot of different api url (i have chatgpt4) starting by https://api.openai.com/v1

Same here

This may help you all. I had to restart mattermost after everything was configured. Then it started working.
Remember to setup your billing too.


CursorHoldI events trigger at unexpected time when there are timers firing

install python3 and its neovim package
extract this and put it in ~/.vim (containss nvim-yarp, vim-hug-neovim-rpc and some minimal code to reproduce the bug; you can install the plugins and copy the simple code under pack/general/start/deoplete too)
run gvim (terminal vim doesn't have this problem; Linux with GTK 2 GUI, don't know about others)
in insert mode, type some text quickly
try to undo several times

You'll see that the insertion is broken into many steps for undoing, because CursorHoldI has been called too many times.
I tracked this down from deoplete (see https://github.com/Shougo/deoplete.nvim/issues/598).
I think the problem happens because, in gui.c:2980, when gui_wait_for_chars_or_timer(p_ut) returns FAIL, Vim fires CursorHold* events. But in gui_wait_for_chars_or_timer, when a timer calls feedkeys, it may return FAIL soon.
This is the timing I have logged:
[
    ['NOTIFY:', [1513338040, 179913]],
    ['NOTIFY:', [1513338040, 431771]],
    ['BACK:', [1513338040, 474465]],
    ['BACK:', [1513338040, 494892]],
    ['feedkeys:', [1513338040, 524562]],
    ['NOTIFY:', [1513338040, 527912]],
    ['BACK:', [1513338040, 537664]],
    ['feedkeys:', [1513338040, 544299]],
    ['NOTIFY:', [1513338040, 557622]],
    ['NOTIFY:', [1513338040, 569784]],
    ['BACK:', [1513338040, 583423]],
    ['BACK:', [1513338040, 585836]],
    ['feedkeys:', [1513338040, 586457]],
    ['NOTIFY:', [1513338040, 601140]],
    ['BACK:', [1513338040, 631092]],
    ['feedkeys:', [1513338040, 633510]],
    ['feedkeys:', [1513338040, 635674]],
    ['NOTIFY:', [1513338040, 658948]],
    ['NOTIFY:', [1513338040, 666987]],
    ['BACK:', [1513338040, 674234]],
    ['BACK:', [1513338040, 675218]],
    ['feedkeys:', [1513338040, 680711]],
    ['feedkeys:', [1513338040, 723942]],
    ['feedkeys:', [1513338040, 723991]],
    ['NOTIFY:', [1513338040, 798839]],
    ['NOTIFY:', [1513338040, 802242]],
    ['BACK:', [1513338040, 806270]],
    ['BACK:', [1513338040, 808379]],
    ['feedkeys:', [1513338040, 856461]],
    ['feedkeys:', [1513338040, 858759]]
]

My vim version is 8.0.1387.

Hi Bram, with 8.0.1406 I can still reproduce the problem.

Hi Bram, with 8.0.1406 I can still reproduce the problem.

Do you have a (simple) example?  Without the whole plugin.
Hmm, I guess something may put K_IGNORE into the input stream, which is
hard to detect.  We can at least check that 'updatetime' has passed.
I looked into unifying the gui_inchar() and mch_inchar() functionality,
but there are too many differences.  Especially that the GUI uses a
timeout for waiting and polling channels.
--
"Microsoft is like Coke.  It's a secret formula, all the money is from
distribution, and their goal is to get Coke everywhere.  Open source is like
selling water.  There are water companies like Perrier and Poland Spring, but
you're competing with something that's free."   -- Carl Howe
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///


Update update regression case for index change

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall


“Block prove” tx fails while following tx for the same block prove was successfull

There are 2 x “Block prove” txs regarding block 515570:

https://sepolia.etherscan.io/tx/0x54327bc7a27c41d9b983390a710cb0cc06141fb0f3deaec9f76a3590f616e150
Sent in block: 3812149 with status: “failed” execution reverted
https://sepolia.etherscan.io/tx/0x5304bfca9e4dfb8bf38877d6acd631030160478c20cab9896f69456ba2b25b2b
Sent in block: 3812153 with status: “success”

Could you pls clarify the reason the 2nd TX got success while it was submitted after the 1st one which failed.

@adaki2004 could u also help looking at this question?

How to figure out - which fork will be rewarded?


Update USAGE-DETAILS.md

@rivkap , please help review the words

don't forget to also add the comment in the separate example storageclass yaml


Remove legacy anchor nodes acting as documentation [phab:T280]

Created by schroederc at 2017-10-20 20:54:30:

schroederc wrote at 2017-10-20 22:07:01:
Reassigning for C++ indexer work

revision #2438


build(deps): update scalatest from 3.2.12 to 3.2.13

Updates org.scalatest:scalatest from 3.2.12 to 3.2.13.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalatest", artifactId = "scalatest" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "org.scalatest", artifactId = "scalatest" }
}]

labels: test-library-update, early-semver-patch, semver-spec-patch, commit-count:1

Codecov Report

Merging #85 (114c5b1) into main (1fe4e91) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #85   +/-   ##
=======================================
  Coverage   85.36%   85.36%
=======================================
  Files           3        3
  Lines          41       41
  Branches        3        3
=======================================
  Hits           35       35
  Misses          6        6

Help us with your feedback. Take ten seconds to tell us how you rate us.


Remove unused PredicateFailure constructors

This commit removes the following Failure* constructors that do not occur outside of their own definition.
$ git grep -e ' Failure' 5d7e2705 -- 'shelley/*hs'
5d7e2705:shelley/chain-and-ledger/executable-spec/src/Shelley/Spec/Ledger/STS/Newpp.hs:    = FailureNEWPP
5d7e2705:shelley/chain-and-ledger/executable-spec/src/Shelley/Spec/Ledger/STS/PoolReap.hs:    = FailurePOOLREAP
5d7e2705:shelley/chain-and-ledger/executable-spec/src/Shelley/Spec/Ledger/STS/Rupd.hs:    = FailureRUPD
5d7e2705:shelley/chain-and-ledger/executable-spec/src/Shelley/Spec/Ledger/STS/Snap.hs:    = FailureSNAP
5d7e2705:shelley/chain-and-ledger/executable-spec/src/Shelley/Spec/Ledger/STS/Updn.hs:  data PredicateFailure (UPDN crypto) = FailureUPDN
$ git grep -e FailureNEWPP -e FailurePOOLREAP -e FailureRUPD -e FailureSNAP -e FailureUPDN  5d7e2705 -- 'shelley/*hs'
5d7e2705:shelley/chain-and-ledger/executable-spec/src/Shelley/Spec/Ledger/STS/Newpp.hs:    = FailureNEWPP
5d7e2705:shelley/chain-and-ledger/executable-spec/src/Shelley/Spec/Ledger/STS/PoolReap.hs:    = FailurePOOLREAP
5d7e2705:shelley/chain-and-ledger/executable-spec/src/Shelley/Spec/Ledger/STS/Rupd.hs:    = FailureRUPD
5d7e2705:shelley/chain-and-ledger/executable-spec/src/Shelley/Spec/Ledger/STS/Snap.hs:    = FailureSNAP
5d7e2705:shelley/chain-and-ledger/executable-spec/src/Shelley/Spec/Ledger/STS/Updn.hs:  data PredicateFailure (UPDN crypto) = FailureUPDN

Well, I had to spell out the NoUnexpectedThunk instances, since cardano-prelude does not provide an instance for V1, the generic representation of empty data types.
@mrBliss Is whnfNoUnexpectedThunks _ _ = return NoUnexpectedThunks the right choice for empty types? On the one hand, it can't be any thing other than a thunk/exception. On the other, it has no business being stored in a reference cell, which is the only place we actually check for thunks, if I understand correctly. Thanks.
Should we add a V1 instance to cardano-prelude?

I rerequested review, since this now updates the cardano-prelude dependency. (Also, I've never made that change in this repo before: is updating stack.yaml and cabal.project sufficient?)

I rerequested review, since this now updates the cardano-prelude dependency. (Also, I've never made that change in this repo before: is updating stack.yaml and cabal.project sufficient?)

yep, looks good, you just need to sign the commit. thanks again!


ads1x15.cxx: fails to compile with gcc7

float
ADS1X15::getThresh(ADSTHRESH reg){
     if( THRESH_HIGH && THRESH_LOW) return getLastSample(reg);
     else return 0.0;
}

fails to compile with gcc7 because THRESH_HIGH and THRESH_LOW are enums:
           /**
             * @enum ADSTHRESH
             * @brief uint8_t enum containing register addresses
             * used for setting HI and LOW threshold values as
             * well as setting conversion ready and set to default.
             *
             * @var ADSTHRESH::THRESH_LOW     = 0x02
             * @var ADSTHRESH::THRESH_HIGH    = 0x03
             * @var ADSTHRESH::CONVERSION_RDY = 0x04
             * @var ADSTHRESH::THRESH_DEFAULT = 0x05
             */
            typedef enum ADSTHRESH {
                THRESH_LOW      = ADS1X15_REG_POINTER_LOWTHRESH,
                THRESH_HIGH     = ADS1X15_REG_POINTER_HITHRESH,
                CONVERSION_RDY  = 0x04,
                THRESH_DEFAULT  = 0x05

            } ADSTHRESH;

Leading to:
ads1x15.cxx:119:25: error: enum constant in boolean context [-Werror=int-in-bool-context]
       if( THRESH_HIGH && THRESH_LOW) return getLastSample(reg);
                          ^~~~~~~~~~
ads1x15.cxx:119:25: error: enum constant in boolean context [-Werror=int-in-bool-context]

I don't know what the if check was meant to achieve. As it stands, older compilers probably just evaluated it to "true" at build time.

Hi, this was already fixed with c64d04d084f47b4cb3e1cdee6e96348fcc6a756e, and will be part of the next tag. See #560 also. Thanks!

Patch was merged and this should no longer be an issue with UPM 1.5.0


Will chrome automatically convert webp to avif?

I use these environment variables, but when chrome requests, the response header is image/avif,I did not enable AVIF support detection
darthsim/imgproxy  v3.18
ENV IMGPROXY_ENABLE_WEBP_DETECTION=true
ENV IMGPROXY_PREFERRED_FORMATS=webp,jpeg,png,gif

image link
edge:

chrome:

Wait, I got an error using compose.yaml build file

Well, it was my misconfiguration of compose.yml that caused the environment variables to be overwritten.


update Gearpump's performance metrics in doc

I hava a silly question : How to add label ? thank you ~~

If you mean add:

Or apply a label:


Support for Visual Studio 2017

Currently Graph Engine VSExtension only supports Visual Studio 2012, 2013, 2015

I've been using it in vs 2017 for months now.

@TaviTruman How are you using it in VS 2017?  The extension doesn't install.

@PhilPJL please try to build the extension from master

@Yatli.  Ok thanks, will do. What's the ETA on the next preview?

We're now packing all the nuget packages into the SDK vsix so we've got to make sure everything is functioning. Cannot be so sure about a release ETA, sorry :)

@yatli.  Ok.  If want to ask various dumb questions, is emailing<EMAIL_ADDRESS>a good place?

feel free to send any query to us! or you can talk to me directly here<EMAIL_ADDRESS>we've got a GitHub bot connected to our Teams channel, so posting in the issues section will also get me notified.

Thanks, I'll get some questions together and send them over next week.

Will be there.
I'm closing this issue now, as VS2017 support is getting ready.


added proper handling of download actions #244

Hi I added better handling of the download actions, instead of just a message box now it actually does: launch, unpack+show, show, none (nothing). Show requests the desktop to show the location/ file in the browser.
People seems not to not know how to get to the downloaded stuff or where it end up even though we show the location.

I added a dialog for confirmation before it begins to "install" it actually only launches the installer so it would also be cancelled from there.
Concerning unzip a.k.a unpack it works as intended on mac but it does not open "install_downloads" but "install" where is unpacks the content to.

You are right. For some reason on windows, it opens into-cps-projects with the install folder selected.
Anyway, the unpack is also working. The windows folder thing seems minor enough that we can leave as is.


Mast: Blacklist SPIRITUAL[\W_]*+GRANDMASTER

Mast requests the blacklist of the keyword SPIRITUAL[\W_]*+GRANDMASTER. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

SPIRITUAL[\W_]*+GRANDMASTER has been seen in 7 true positives, 0 false positives, and 0 NAAs.

Approved by Daniil in Charcoal HQ


Question on controller frequency

Hello! and thanks for the great work.
One curiosity about the frequency of the vehicle control:
From what i see in the _simulate() function, an high level action (e.g. LANE_LEFT/LANE_RIGHT) is performed at a policy frequency and kept for a certain number of simulation steps.
If i understood correctly, the low level steering and speed controller are also updated at a policy frequency. Wouldn't be better to make them faster compared to the high level decision?

Maybe #63 helps you.

From what i see in the _simulate() function, an high level action (e.g. LANE_LEFT/LANE_RIGHT) is performed at a policy frequency and kept for a certain number of simulation steps.

That's right.

If i understood correctly, the low level steering and speed controller are also updated at a policy frequency. Wouldn't be better to make them faster compared to the high level decision?

No, low-level steering and speed control is performed at simulation frequency (higher than policy/decision). I can imagine how it's not really obvious from reading the code, but here's how it works:

In MDPVehicle.act(action) when the action is None, the current speed/lane targets are unchanged, and the action is forwarded to the ControlledVehicle.act() method, which computes the steering and acceleration controls.
ego_vehicle.act(action) can thus be called at policy frequency for high-level decisions, but ego.vehicle.act() also needs to be called at simulation frequency for control.
this is achieved in the _simulate() function by calling road.act(), which itself broadcasts vehicle.act() to every vehicle in the scene.

Thanks for the answer. I did not see that ego.vehicle.act() was called also in road.act().


Change IPC to use full name for interface.

/** The `\0` terminated `az_span` with the capability name. */

Remove

Refers to: inc/az_ulib_descriptor_api.h:45 in 0edc5ee. [](commit_id = 0edc5eec906c52559bd99c82ac16bceea536d10e, deletion_comment = False)

This API initialize the IPC. It shall be called only once, at the beginning of the code

nit: intializes

Refers to: inc/az_ulib_ipc_api.h:73 in e295408. [](commit_id = e2954085c4fbf3c52cacb9453d8761839168dd03, deletion_comment = False)

     initialization process is completely done.

nit: "This API is not thread safe. The other IPC APIs shall only ve called after the initialization process is complete."

Refers to: inc/az_ulib_ipc_api.h:77 in e295408. [](commit_id = e2954085c4fbf3c52cacb9453d8761839168dd03, deletion_comment = False)

     this init.

nit: "This API is not thread safe. No other IPC API may be called during the execution of this init."

Refers to: inc/az_ulib_ipc_api.h:86 in e295408. [](commit_id = e2954085c4fbf3c52cacb9453d8761839168dd03, deletion_comment = False)

@retval #AZ_OK                              If the IPC initialize with success.

nit: initializes

Refers to: inc/az_ulib_ipc_api.h:89 in e295408. [](commit_id = e2954085c4fbf3c52cacb9453d8761839168dd03, deletion_comment = False)

     this deinit, and no other IPC API shall be running during the execution of this API.

nit: "This API is not thread safe. No other IPC API may be called during the execution of this deinit and no other IPC API shall be running during the execution of this API."

Refers to: inc/az_ulib_ipc_api.h:107 in e295408. [](commit_id = e2954085c4fbf3c52cacb9453d8761839168dd03, deletion_comment = False)

                               success.

nit: "... up to the point the interface is successfully unpublished."

Refers to: inc/az_ulib_ipc_api.h:155 in e295408. [](commit_id = e2954085c4fbf3c52cacb9453d8761839168dd03, deletion_comment = False)

     this init.

This is actually duplicated. I'll remove it.

In reply to: 876719970

Refers to: inc/az_ulib_ipc_api.h:86 in e295408. [](commit_id = e2954085c4fbf3c52cacb9453d8761839168dd03, deletion_comment = False)

in.max_sum = 10000;

should have commented this in the original PR. Is this a hard limit for ipc?

Refers to: tests/tests_e2e/az_ulib_ipc_e2e/az_ulib_ipc_e2e.c:150 in 2e4b0b4. [](commit_id = 2e4b0b498dcf236d1516a735b16f3a14c2733b87, deletion_comment = False)

in.max_sum = 10000;

No, this is a randon number for an e2e test.

In reply to: 877226924

Refers to: tests/tests_e2e/az_ulib_ipc_e2e/az_ulib_ipc_e2e.c:150 in 2e4b0b4. [](commit_id = 2e4b0b498dcf236d1516a735b16f3a14c2733b87, deletion_comment = False)


Radio buttons are not updating reactive data

Version & browser
Vue: 2.5.13
UIV: 0.18.0
Chrome: 64.0.3282.186
Which component
Btn / BtnGroup
Steps to reproduce the problem
Test A:
<btn-group>
    <btn input-type="radio" input-value="1" v-model="profile">None</btn>
    <btn input-type="radio" input-value="2" v-model="profile">Use Existing</btn>
    <btn input-type="radio" input-value="3" v-model="profile">Create New</btn>
</btn-group>

Test B:
<btn input-type="radio" input-value="1" v-model="profile">None</btn>
<btn input-type="radio" input-value="2" v-model="profile">Use Existing</btn>
<btn input-type="radio" input-value="3" v-model="profile">Create New</btn>

Data is declared in host component
data: ()=>({
    profile: '1'
})

Expected behavior
With Test A, this.profile value is updated when pressing on the buttons, as it reactive and bound with the v-model.
Actual behavior
With Test A, this.profile is NOT updated. And the first "radio" is NOT selected (as value is "1").
With Test B,  this.profile is updated (but the layout is scrambled without the enclosing <btn-group>).

Btw, the <template> in the examples in the docs are not valid because there is not single root element. If I'm right, that should go in another issue.

The same happens with input-type="checkbox"

The radio or checkbox ui is working (toggles as expected, it's just the model that is not updated. Another thing I noted is that it initializes correctly with the model data. So if data is set to profile:  '2' , the correct element is selected. But does not changes after when interacting.

With breakpoints on node_modules/uiv/src/components/button/Btn.js, I see that the  change event is not triggered, when the Btn is enclosed in an BtnGroup and does when alone.
          on: {
            change () { // <---- does not trigger
              if (props.inputType === INPUT_TYPE_CHECKBOX) {
                if (isInputActive) {
                  props.value.splice(props.value.indexOf(props.inputValue), 1)
                } else {
                  props.value.push(props.inputValue)
                }
              } else {'COUCOU', console.log(props.inputValue);
                listeners['input'](props.inputValue)
              }
            }
          }

This does not work as well:

    <btn input-type="radio" input-value="1" v-model="profile">None</btn>
    <btn input-type="radio" input-value="2" v-model="profile">Use Existing</btn>
    <btn input-type="radio" input-value="3" v-model="profile">Create New</btn>

But this does...

    <btn input-type="radio" input-value="1" v-model="profile">None</btn>
    <btn input-type="radio" input-value="2" v-model="profile">Use Existing</btn>
    <btn input-type="radio" input-value="3" v-model="profile">Create New</btn>

..so I fear I have some side-effects from bootstrap javascript file! It should not be there, but maybe I'm wrong, so I'll verify that.

Dammit, found this require('bootstrap-sass');... probably for nav dropdowns that were there before. I'll use UIV components for that and get rid of that dependency!
...all ok now ..my baaad

@isometriq I am having this problem, I can reproduce your exact behavior, but I don't understand the solution.

If I recall, the require('bootstrap-sass'); statement introduce the necessary javascript to use the bootstrap components and this was interfering with UIV. Removing it allows UIV to manipulate the elements as intended.

Cant find it, 'require('bootstrap-sass');', I don't have this on my project. Cant make them work

It's possible that was something I added in the past ..I'm pretty sure it was included in the Laravel project. where I had the issue. Try to see if there is another library or script that should not be there or that is interfering...


[Event Request] Page 233 Apply Vendor Entries OnQueryClosePageAppliesToDocNoOnBeforeVendEntryEdit

Please Add an Event OnQueryClosePageAppliesToDocNoOnBeforeVendEntryEdit in Page 233 Apply Vendor Entries
trigger OnQueryClosePage(CloseAction: Action): Boolean
    var
        RaiseError: Boolean;
    begin
        if CloseAction = ACTION::LookupOK then
            LookupOKOnPush();
        if ApplnType = ApplnType::"Applies-to Doc. No." then begin
            if OK then begin
                RaiseError := ApplyingVendLedgEntry."Posting Date" < Rec."Posting Date";
                OnBeforeEarlierPostingDateError(ApplyingVendLedgEntry, Rec, RaiseError, CalcType.AsInteger(), PmtDiscAmount);
                if RaiseError then begin
                    OK := false;
                    Error(
                      EarlierPostingDateErr, ApplyingVendLedgEntry."Document Type", ApplyingVendLedgEntry."Document No.",
                      Rec."Document Type", Rec."Document No.");
                end;
            end;
            if OK then begin
                if Rec."Amount to Apply" = 0 then
                    Rec."Amount to Apply" := Rec."Remaining Amount";

                OnQueryClosePageAppliesToDocNoOnBeforeVendEntryEdit();  <--- NEW EVENT

                CODEUNIT.Run(CODEUNIT::"Vend. Entry-Edit", Rec);
            end;
        end;

        if CheckActionPerformed then begin
            Rec := ApplyingVendLedgEntry;
            Rec."Applying Entry" := false;
            if AppliesToID = '' then begin
                Rec."Applies-to ID" := '';
                Rec."Amount to Apply" := 0;
            end;
            CODEUNIT.Run(CODEUNIT::"Vend. Entry-Edit", Rec);
        end;
    end;

[IntegrationEvent(true, false)]
    local procedure OnQueryClosePageAppliesToDocNoOnBeforeVendEntryEdit()
    begin
    end;

Thank You!

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


Appstudio update mvp-test-component

Pipelines as Code configuration proposal

Pipelines as Code CI/mvp-test-component-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
5 seconds
init

✅ Succeeded
15 seconds
clone-repository

✅ Succeeded
3 minutes
build-container

✅ Succeeded
17 seconds
deprecated-base-image-check

✅ Succeeded
19 seconds
sanity-inspect-image

✅ Succeeded
12 seconds
sbom-json-check

✅ Succeeded
19 seconds
clair-scan

✅ Succeeded
1 minute
clamav-scan

✅ Succeeded
15 seconds
sanity-optional-label-check

✅ Succeeded
14 seconds
sanity-label-check

✅ Succeeded
5 seconds
show-summary

Pipelines as Code CI/mvp-test-component-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
5 seconds
init

✅ Succeeded
15 seconds
clone-repository

✅ Succeeded
3 minutes
build-container

✅ Succeeded
19 seconds
sanity-inspect-image

✅ Succeeded
17 seconds
deprecated-base-image-check

✅ Succeeded
19 seconds
clair-scan

✅ Succeeded
1 minute
clamav-scan

✅ Succeeded
12 seconds
sbom-json-check

✅ Succeeded
15 seconds
sanity-optional-label-check

✅ Succeeded
14 seconds
sanity-label-check

✅ Succeeded
5 seconds
show-summary

Pipelines as Code CI/mvp-test-component-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
5 seconds
init

✅ Succeeded
15 seconds
clone-repository

✅ Succeeded
3 minutes
build-container

✅ Succeeded
17 seconds
deprecated-base-image-check

✅ Succeeded
19 seconds
sanity-inspect-image

✅ Succeeded
1 minute
clamav-scan

✅ Succeeded
19 seconds
clair-scan

✅ Succeeded
12 seconds
sbom-json-check

✅ Succeeded
15 seconds
sanity-optional-label-check

✅ Succeeded
14 seconds
sanity-label-check

✅ Succeeded
5 seconds
show-summary


add kubectl-armadactl to goreleaser.yml

This PR builds another archive named kubectl-armadactl in our Armada project which will be later used to integrate armadactl with kubectl.
The contents of the new archive are exactly same as that of armadactl.
┆Issue is synchronized with this Jira Task by Unito

Change looks good to me but I think there is a syntax error:
Error: yaml: unmarshal errors:
  line 177: field rename not found in type config.Archive

I thought so this might give out error, is there any other way we can change binary name cause if we don't do this a new archive will be created kubectl-armadactl but will have binary with the name armadactl which we don't want.


"uwfmgr.exe Registry Commit/Commit-Delete" command does not work for HKCU registry

Describe the bug
Attempts to commit changes to keys and values under the HKCU registry with the "uwfmgr.exe Registry Commit/Commit-Delete" command fail. The command does not support modifying HKCU registry keys and values, but its description is not mentioned anywhere in the documentation.
https://docs.microsoft.com/en-us/windows-hardware/customize/enterprise/uwfmgrexe
https://docs.microsoft.com/en-us/windows-hardware/customize/enterprise/uwf-registryfiltercommitregistry
https://docs.microsoft.com/en-us/windows-hardware/customize/enterprise/uwf-registryfiltercommitregistrydeletion

To Reproduce
Steps to reproduce the behavior:

Run "uwfmgr.exe Registry Commit/Commit-Delete" command to values under HKCU registry keys.

Example:
uwfmgr.exe registry commit HKCU\test test

See Err.

Error: Could not commit changes on "HKCU\test" since it is not allowed to be committed by Unified Write Filter.

Expected behavior
Please write in the documentation the registry to which the commit does not apply, eg under HKCU.

Desktop (please complete the following information):

OS: Windows 10 IoT Enterprise

Additional context
Some users use this command and may be asked to explain it, so please document it as soon as possible.

Same issue on:

OS: Windows 10 Enterprise 2016 LTSB

Is it expected and just to be documented?

Documentation MUST HAVE THE DATA or NEEDS TO BE ACCESSIBLE | REFERRABLE.


Truncated Backpropagation Through Time

Is your feature request related to a problem? Please describe.
Training very long sequences using RNNs is not easily feasible. Thus, limiting the number of steps for forward and backward path, while keeping the last state for the next iterateion, speeds up training significantly.
Describe the solution you'd like
Have an option in trainer like bptt_steps
Describe alternatives you've considered
Overwrite the trainers ´__run_training_batch´ method.

Here is an implementation for 2 variable length sequences. Input is a packed sequence, so that the zero padding does not need copied to GPU, padding can be done after copying.
import torch.nn as nn

n_features = 1
out_size = 2
tbptt_steps = 3

seq_len_1 = 3
seq_len_2 = 5

batch_first = True
split_dim = 1 if batch_first else 0

x = [torch.rand(seq_len_1, n_features), torch.rand(seq_len_2, n_features)]
y = [torch.rand(seq_len_1, out_size), torch.rand(seq_len_2, out_size)]
x = nn.utils.rnn.pack_sequence(x, enforce_sorted=False)
y = nn.utils.rnn.pack_sequence(y, enforce_sorted=False)

model = nn.RNN(
    input_size=n_features, hidden_size=out_size, num_layers=2, batch_first=batch_first
)

opt = torch.optim.Adam(model.parameters())

# In lightning module
x, x_lengths = nn.utils.rnn.pad_packed_sequence(x, batch_first=batch_first)
y, y_lengths = nn.utils.rnn.pad_packed_sequence(y, batch_first=batch_first)

for i, (x_, y_) in enumerate(
    zip(x.split(tbptt_steps, dim=split_dim), y.split(tbptt_steps, dim=split_dim))
):
    print(f"{i}: {x_.shape}")
    # Forward path
    y_pred, state = model(x_)
    # Compute loss
    loss = nn.functional.mse_loss(y_, y_pred)
    # Backward path
    loss.backward()
    # Update weights
    opt.step()

Output with batch_first = True:
0: torch.Size([2, 3, 1])
1: torch.Size([2, 2, 1])

Output with batch_first = False:
0: torch.Size([3, 2, 1])
1: torch.Size([2, 2, 1])

Yes this sounds good.
The state however is not handled in the current trainer right now.

The state needs to be passed from a tbptt step to the next
Before providing it to the model, it has to be detached

I think the following should be done in the LightningModule and not in the dataloader due to several reasons:
x, y = batch
if input_is_packed:
    x, x_lengths = nn.utils.rnn.pad_packed_sequence(x, batch_first=batch_first)
    y, y_lengths = nn.utils.rnn.pad_packed_sequence(y, batch_first=batch_first)

Only the meaningful data and no zeros of the padding need to be copied
batch might not be a tuple but a custom type, e.g.

batch = {
  'input': input_tensor,
  'target': target_tensor,
  'meta': some_meta_dict,
}

So we cannot do ´x, y = batch´.
Both can be solved if split_each_element_in_batch_fx is implemented in the LightningModule. Here the user can pad, process the custom input batch and split the data. We could also provide a default implementation like above via x.split(tbptt_steps).


Connect Mobile Wallet from Blitz menu failed with exceptions

Bug
Today i tried to test connect my Zapp Android app with my blitz. This failed with some exceptions i want to share.
Steps to reproduce:

From Raspiblitz menu choose: Mobile
ZAPP_ANDROID
Continue

blitz version running
1.6.1
logs
At this point an exception was thrown based on blitz.subscriptions.ip2tor.py:
# dynDomain ON
creating install info ...
# OK - file exists: /home/admin/raspiblitz/pictures/app_zeus.png
Traceback (most recent call last):
  File "/home/admin/config.scripts/blitz.subscriptions.ip2tor.py", line 16, in <module>
    from lndlibs import rpc_pb2 as lnrpc
  File "/home/admin/config.scripts/lndlibs/rpc_pb2.py", line 5, in <module>
    from google.protobuf.internal import enum_type_wrapper
[...]
  File "/usr/lib/python3/dist-packages/pkg_resources/__init__.py", line 1413, in get_metadata
    return value.decode('utf-8') if six.PY3 else value
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xe3 in position 4286: invalid continuation byte
Traceback (most recent call last):
  File "/home/admin/config.scripts/blitz.subscriptions.ip2tor.py", line 16, in <module>
    from lndlibs import rpc_pb2 as lnrpc
[...]
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xe3 in position 4286: invalid continuation byte
# Installing lndconnect..
rfkill: cannot open /dev/rfkill: Permission denied
rfkill: cannot read /dev/rfkill: Bad file descriptor
Check Framework: Go
*** Installing Go v1.13.3 for armv6l ***

This got followed by the announced go installation. Another Exception was thrown and hangs on go installation with some serious errors:
/usr/include/asm-generic/errno.h:91:2: error: invalid preprocessing directive #d
 #d�fine ESHUTDOWN 108 /* Cannot send cfter transport endpoint shutdown */
  ^
/usr/include/asm-generic/errno.h:97:8: error: macro names must be identifiers
 #definEALREADY 114 /* Operation already in progress */
        ^
/usr/include/asm-generic/errno.h:98:2: error: invalid preprocessing directive #definE; did you mean #define?
 #definE EINPROGRESS 115 /* Operation"now in progress�*/
  ^~~~~~
  define
/usr/include/asm-generic/errno.h:100:2: error: invalid preprocessing directive #defmne; did you mean #define?
 #defmne EECLEAN  �17 /* Str5cture needs aleaning */
  ^~~~~~
  define
/usr/include/asm-generic/errno.h:101:8: error: macro names must be identifiers
 #defin�NOTNAM  118 /* Not a XENIX namet type file *+
        ^
/usr/include/asm-generic/errno.h:104:2: error: invalid preprocessing directive #Define; did you mean #define?
 #Define �REMOTEIO 121 /* Remote I/O error */
  ^~~~~~
  define
/usr/include/asm-generic/errno.h:119:2: error: invalid preprocessing directive #denine; did you mean #define?
 #denine ERFKILL  132 /* Operation not possible due to RF-{ill */N#definm EHWPOISON 133 /* Memory page has hazdware errOr �/
  ^~~~~~
  define
/usr/include/asm-generic/errno.h:119:89: error: unterminated comment
 #denine ERFKILL  132 /* Operation not possible due to RF-{ill */N#definm EHWPOISON 133 /* Memory page has hazdware errOr �/
                                                                                ^
In file included from /usr/include/arm-linux-gnueabihf/asm/errno.h:1,
                 from /usr/include/linux/errno.h:1,
                 from /usr/include/arm-linux-gnueabihf/bits/errno.h:26,
                 from /usr/include/errno.h:28,
                 from cgo-gcc-prolog:30:
/usr/include/asm-generic/errno.h:2: error: unterminated #ifndef
 #ifndef _ASM_GENERIC_ERRNO_H

make: *** [Makefile:4: dep] Error 2

!!!!!!!!!!!!!!!!!!!!!!!!!!
FAIL: Was not able to install/build lndconnect
Retry later or report to developers with logs above.
/home/admin/config.scripts/bonus.lndconnect.sh: line 55: lndconnect: command not found
!!!!!!!!!!!!!!!!!!!!!!!!!!

After some time and enter was pressed, i was thrown back to raspi blitz menu. Everything seems fine except my mobile wallet connect
looks like a still corrupted system after some recently happened Control-D issues. I will reflash my sd card soon and give it a retry.

Hm.. I just tried this on a most recent v1.6 checkout and everything is fine. Please reflash and let us now if you still have issues.

Tested again with 1.6.1. The initial issue is not reproducible and seems related to a corrupted filesystem


error overflowing RenderFlex has an orientation of Axis.vertical

I have an error when closing the search engine by closing it with the gesture of sliding the finger down

DETAILS
The overflowing RenderFlex has an orientation of Axis.vertical.
The edge of the RenderFlex that is overflowing has been marked in the rendering with a yellow and black striped pattern. This is usually caused by the contents being too big for the RenderFlex.
Consider applying a flex factor (e.g. using an Expanded widget) to force the children of the RenderFlex to fit within the available space instead of being sized to their natural size.
This is considered an error condition because it indicates that there is content that cannot be seen. If the content is legitimately bigger than the available space, consider clipping it with a ClipRect widget before putting it in the flex, or using a scrollable container rather than a Flex, like a ListView.
The specific RenderFlex in question is: RenderFlex#dfe04 relayoutBoundary=up10 OVERFLOWING

@StillGC Please check this with the latest version. Thanks.


Secrets can be leaked in debug and on error

Secrets that are passed as values using the values option can be leaked when using debug mode or in case of error as the whole helm command will be printed.
I don't know if there's a good fix for this as I don't know if it can be determined which of those values come from drone env variables, e.g. DRONE_TAG or from custom secrets, e.g SECRET_PASSWORD in the README example

Hi @ipedrazas May I ask why is this not an issue or how can I work around it?

The only way of not leaking secrets is not to set those values. If you need to inject values that are secrets, you have to accept that those values might be leaked.
I know it's not great, but I'd rather fix the process that patch a tool that should not be doing what it does.
Happy to leave the issue open, but I don't have any bandwidth to solve any issues with this repo right now.

Since leaking secrets is not an option for me as repos are public, for the moment the workaround is to create secrets in k8s beforehand, modify the chart to use them instead of the bare values and refer to those secrets from helm

I know it's not great, but I'd rather fix the process that patch a tool that should not be doing what it does.

What do you mean by "fix the process"?

Happy to leave the issue open, but I don't have any bandwidth to solve any issues with this repo right now.

I'd say it is better to leave it opened, as it would mark in a clearer fashion that issue is not resolved. I will send a PR to add a cautionary message in the README.md
As for the solution, I'm happy to give it a try. If you have any ideas of how this should be fixed in the plugin, please share.

That's exactly what I do, all the secrets are created by a different chart and pods consume them.

@ipedrazas I do it the same way too, secrets come from the different chart. it also allows easier to maintain you main chart releases.


Add case-insensitive header name comparison in HeaderExtractor

Headers are sent with lower case names by Google Chrome since version 60 [1].
This change seems to be a conformity to the rfc7320 spec, where header fields
are described as 'case-insensitive'. [2]
Unfortunately, this renders client authentication via DirectBasicAuthClient unusable.
This patch extends the header look up in HeaderExtractor by implementing a fall back
mechanism to look for a lowercase named header name.
[1] https://bugs.chromium.org/p/chromium/issues/detail?id=749086
[2] https://tools.ietf.org/html/rfc7230#section-3.2

Excellent! Thanks

np, cheers! 🍺


Alpaca API Inconsistent Portfolio Information

Expected Behavior
Alpaca Brokerage returns consistent portfolio contents
Actual Behavior
Share quantity inconsistent. Potentially missing partial share fills?
Line 22490: 2019-11-13 15:11:00 New Order Event: Time: 11/13/2019 15:11:00 OrderID: 20 Symbol: FB Status: Invalid Message: Error placing order: insufficient qty available for order (requested: 15, available: 11)
Line 30063: 2019-11-13 18:27:00 New Order Event: Time: 11/13/2019 18:27:00 OrderID: 26 Symbol: FSLY Status: Invalid Message: Error placing order: insufficient qty available for order (requested: 87, available: 48)
Line 250942: 2019-11-15 15:11:00 New Order Event: Time: 11/15/2019 15:11:00 OrderID: 30 Symbol: FB Status: Invalid Message: Error placing order: insufficient qty available for order (requested: 15, available: 11)
Line 256420: 2019-11-15 18:23:00 New Order Event: Time: 11/15/2019 18:23:00 OrderID: 31 Symbol: FB Status: Invalid Message: Error placing order: insufficient qty available for order (requested: 15, available: 11)

Potential Solution
Review and debug
Reproducing the Problem
Likely placing large trades.
Checklist

[x] I have completely filled out this template
[x] I have confirmed that this issue exists on the current master branch
[x] I have confirmed that this is not a duplicate issue by searching issues
[x] I have provided detailed steps to reproduce the issue

Closed by https://github.com/QuantConnect/Lean/pull/3946


Unable to set workingDirectory and targetPath separately for 32 and 64 bit arch.

This allows me to have bin and bin64 directories where first has 32 bit dll files and second has 64 bit dll files.
This makes program simpler because there is no difference in running 32 or 64bit executable since dlls are in the same location relative to executable location.
This can be archieved by supporting platform suffixes in targetPath and workingDirectory build settings.
Then I can do in dub.json
"targetPath" : "bin",
"workingDirectory" : "bin",
"targetPath" : "bin64",
"workingDirectory" : "bin64",

Not sure why it's explicitly unsupported, but I guess it would be hard to map to generators like visuald.
You should use copyFiles to copy your DLLs into the target path, this setting does support version suffixes.

The reason why it's not supported is simply that it would lead to possible ambiguities:
"targetPath-windows": "windows/bin",
"targetPath-windows-x86": "windows/bin32"

The easiest solution would be to simply let the outcome of ambiguous specifications be explicitly undefined.

Ok, copyFiles worked.
Here is what I've got
    "targetPath" : "bin",
    "workingDirectory" : "bin",

    "sourcePaths" : [ "source" ],

    "copyFiles-x86" : ["lib/FreeImage.dll", "lib/freetype.dll", "lib/glfw3.dll", "lib/libenet.dll"],
    "copyFiles-x86_64" : ["lib64/FreeImage.dll", "lib64/freetype.dll", "lib64/glfw3.dll", "lib64/libenet.dll"],

    "configurations": [
        {
            "name" : "client",
            "targetName" : "client",
            "targetType" : "executable",
            "excludedSourceFiles" : ["source/voxelman/server/*", "source/servermain.d"],
            "mainSourceFile" : "source/clientmain.d"
        },
        {
            "name": "server",
            "targetName" : "server",
            "targetType": "executable",
            "excludedSourceFiles" : ["source/voxelman/client/*", "source/clientmain.d"],
            "mainSourceFile" : "source/servermain.d"
        }
    ]

This would make a great first entry for our new dub cookbook. Would you mind adding it?
https://github.com/D-Programming-Language/dub/wiki/Cookbook

Didn't knew, that we have cookbook, will work on that tomorrow.

Didn't knew, that we have cookbook, will work on that tomorrow.

I just added it ;).

I found a little issue with this approach. As I have server and client configuration, I always start server then client, but dub tries to copy dll files even when they are identical. So when server is running and client is started, dub fails to copy dll files and client cannot start. Dub should not copy files if they are the same and/or run executable even if copying failed.

Btw, having separate bin64 folder is convenient for doing binary releases.

Btw, having separate bin64 folder is convenient for doing binary releases.

Maybe use a release script?

related https://github.com/D-Programming-Language/dub/issues/242

Well, here is the function used to copy the files.
Any idea how to efficiently implement the comparison on Windows?
https://github.com/D-Programming-Language/dub/blob/a8475bb0a645711120426a55c67e886dd0bf935d/source/dub/internal/vibecompat/core/file.d#L143

Can you check whether that works for you?
https://github.com/MartinNowak/dub/commit/bfc41dfed17cd1476cb3b2482dd419a1ac45cc30

Checked. If applied to latest dub it works, files aren't copied.
(With yours branch got a bunch of deprecations and errors, since it is 0.9.12)

Added entry to cookbook https://github.com/D-Programming-Language/dub/wiki/Cookbook#creating-windows-package-that-has-32-and-64-bit-dlls

OT, but I don't know if you get notifications for edits to the wiki. I
added another entry to the cookbook. Very good idea btw :)
Mathias LANG
Epitech 2015
2015-02-16 23:25 GMT+01:00 Andrey Penechko<EMAIL_ADDRESS>
Added entry to cookbook
https://github.com/D-Programming-Language/dub/wiki/Cookbook#creating-windows-package-that-has-32-and-64-bit-dlls
—
Reply to this email directly or view it on GitHub
https://github.com/D-Programming-Language/dub/pull/511#issuecomment-74581913
.


add support for empty build-cmd/clean-cmd, as a noop

This commit changes the behavior of empty build-cmds in project.yamls. As reported in #24, empty was not distinguished from missing. In this change, a zero length string is treated as an intentional noop.

lgtm


Vulnerability - will it be in Windows update /sccm?

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
https://msrc.microsoft.com/update-guide/vulnerability/CVE-2021-41355
The above security risk needs patching - the Q&A just describe updating one machine by downloading a new setup file but surely, PowerShell is installed on every server and PC
Do Microsoft intend to patch this from windows update?
If not, how do they plan to assist rolling out to every known machine
Why hasnt it had more focus and a KB/hotfix via normal channels?
Expected behavior
Not applicable

Actual behavior
Not applicable

Environment data
Not applicable

OS Data
Not applicable

Windows update log
Not applicable

Visuals
No response

There are updates to 7.0.x and 7.1.x. And the daily build is updated as is, I think, the latest 7.2 preview.  I've had those for a couple of days. Whenever there's an updated version, I get the notification the next time I start PowerShell.
There is community awareness. I've seen tweets pointing here. And I've posted a blog entry and a post over on Spiceworks.
Also, this is not a vulnerability in WIndows Powershell, so it's only relevant where you have installed PowerShell 7.x.


Split Typha metrics by syncer type

Description

This PR splits Typha's Prometheus metrics by syncer type. Many of Typha's statistics are only meaningful within the scope of a syncer but previously, all the syncers wrote to the same metrics resulting in metrics that would bounce around between the values associated with the different syncers.
Related issues/PRs

Todos

[x] Tests
[x] Documentation
[x] Release note

Release Note

Many of Typha's Prometheus metrics are now split by syncer (client) type, represented by a label "syncer" on the metrics. This prevents cross-talk where the syncers would all share the same metrics and the last writer to the metric would "win".

Reminder for the reviewer
Make sure that this PR has the correct labels and milestone set.
Every PR needs one docs-* label.

docs-pr-required: This change requires a change to the documentation that has not been completed yet.
docs-completed: This change has all necessary documentation completed.
docs-not-required: This change has no user-facing impact and requires no docs.

Every PR needs one release-note-* label.

release-note-required: This PR has user-facing changes. Most PRs should have this label.
release-note-not-required: This PR has no user-facing changes.

Other optional labels:

cherry-pick-candidate: This PR should be cherry-picked to an earlier release. For bug fixes only.
needs-operator-pr: This PR is related to install and requires a corresponding change to the operator.

/merge-when-ready

OK, I will merge the pull request when it's ready, leave the commits as is when I merge it, and leave the branch after I've merged it.

Removing "merge-when-ready" label due to new commits

Failures look to be from tests that don't use typha in any way so writing off as flakes.


deleteLink and deleteList are broken

If you try to delete a list of links, it only deletes part of it and you can only get rid of it by deleting it multiple times. It's a problem in deleteLink inside tools.py now.

branch fixlistdelete created for the changes..

fixed with #4


Not log DnsQuery EventID 22

OS: Ubuntu 20.04
Installation instruction: https://github.com/Sysinternals/SysmonForLinux/blob/main/INSTALL.md#ubuntu-1804-2004--2104
sysmon config:
<Sysmon schemaversion="4.21">
  <EventFiltering>
    <DnsQuery onmatch="exclude">
    </DnsQuery>
  </EventFiltering>
</Sysmon>

command for making dns query:
ping www.google.com
checking event id:
sudo cat /var/log/syslog | grep -oP "EventID>\d+<" | sort -u
result:
EventID>1<
EventID>16<
EventID>4<
EventID>5<

Same on Debian 11

Yep...even on a Ubuntu 20.04 Server LTS system...it does not log anything.
Considering all the other bugs (broken in RHEL systems), wrong man page (they use Windows stuff on a Linux system)...SysmonForLinux seems to be in alpha stadium ...and I don't get it why the Sysinternals team has all those "features" in it that don't work at all.

@PoundXI Try without specifying the config file. Does sysmon generate any events in that scenario?

Just process create & terminate events

Thanks for checking. I've tagged this as a bug for now and added to backlog.

Observing same issue with sysmon 1.2.0 with some variations on debian 11.7 and ubuntu 22.04.
Any way to troubleshoot?
expecting more in both case (RawAccessRead for both and file/network/service for first one):
debian11# journalctl -xeu sysmon -l --no-pager | /opt/sysmon/sysmonLogView |grep Event | sort | uniq -c | sort -nr
    630 Event SYSMONEVENT_PROCESS_TERMINATE
    370 Event SYSMONEVENT_CREATE_PROCESS
ubuntu22# journalctl -xeu sysmon -l --no-pager | /opt/sysmon/sysmonLogView |grep 'Event' | sort | uniq -c | sort -nr
     95 Event SYSMONEVENT_PROCESS_TERMINATE
     67 Event SYSMONEVENT_CREATE_PROCESS
      7 Event SYSMONEVENT_NETWORK_CONNECT
      5 Event SYSMONEVENT_FILE_DELETE
      5 Event SYSMONEVENT_FILE_CREATE
      1 Event SYSMONEVENT_SERVICE_STATE_CHANGE
      1 Event SYSMONEVENT_SERVICE_CONFIGURATION_CHANGE

Config based on
https://github.com/microsoft/MSTIC-Sysmon/tree/main/linux/configs
https://github.com/juju4/ansible-sysmon/blob/main/templates/config.xml.j2
Not seeing any DNS catch in
https://github.com/Sysinternals/SysmonForLinux/blob/main/sysmonforlinux.c#L848
but have SYSMONEVENT_NETWORK_CONNECT_EVENT_value and SYSMONEVENT_RAWACCESS_READ_EVENT_value

Thanks for reporting this. I've been a bit back logged but hopefully I can look into this in the next couple of weeks.

Any updates on this?

Hi @0xab3d - Thanks for checking in. We haven't implemented this yet as we're currently busy with other infrastructure work. I will keep everyone updated once we get to this.


Port: 30000 already in use

I have a class with a plc member. I have a thread that sends a counter to MAIN.iCamClk. I have another node which writes to a struct. The codes works well without the clock thread but as soon as I try to run the thread I get:
2024-09-13T13:52:04-0700 Warning: Port: 30000 already in use as 0x7f282fffdf80

Hi can you please provide a sample of your code that can reproduce the error, thanks.

Here's my python code:
import threading
import pyads
import time

REMOTE_ADS = "X.X.X.X.1.1"
PLC_PORT = 851
REMOTE_IP = "X.X.X.X"

class PlcServer(threading.Thread):
    def __init__(self):
        super().__init__()
        self._stop_event = threading.Event()
        print("PLC Server initiated!")

        self.plc = pyads.Connection(REMOTE_ADS, int(PLC_PORT), REMOTE_IP)
        self.clkCounter = 0
        self.sequence = 0

        self.structure_def = (
                ("arrSeqData", pyads.PLCTYPE_INT, 2),
                ("bActivated", pyads.PLCTYPE_BOOL, 1)
            )

        try:
            self.plc.open()
            print(f"PLC connected at AMS Address {REMOTE_ADS}")
            time.sleep(1)
            self.stLinuxData     = self.plc.get_symbol("MAIN.stLinuxData", structure_def=self.structure_def,  array_size=1)
            self.iLinuxClk       = self.plc.get_symbol("MAIN.iLinuxClk")
            self.isPlcConnected = True

        except Exception as inst:
            d = inst
            print(d)
            print(f"AMS Address {REMOTE_ADS} not found!")
            self.isPlcConnected = False

    def send_cmd(self):
        if self.isPlcConnected:
            try:
                self.stLinuxData.write([
                    {
                        "arrSeqData": [self.sequence, self.sequence],
                        "bActivated": True
                        }
                    ])
                self.sequence += 1

            except:
                print(f"Connection lost with {REMOTE_ADS}!")
                print("Trying to reconnect ...")
                self._reconnect()
                time.sleep(0.5)

    def _reconnect(self):
        try:
            self.plc.close()
            self.plc.open()
            self.plc.read_state()
            self.isPlcConnected = True
            print(f"PLC reconnected at AMS Address {REMOTE_ADS}")
        except:
            self.isPlcConnected = False

    def stop(self):
        self._stop_event.set()

    def stopped(self):
        return self._stop_event.is_set()

    def run(self):
        while not self._stop_event.is_set():
            self.clkCounter += 1
            time.sleep(0.005)
            self.iLinuxClk.write(self.clkCounter)
            if self.clkCounter == 60000:
                self.clkCounter = 0

    def __del__(self):
        try:
            self.plc.close()
            if self.is_alive():
                self.join()
        except:
            pass

if __name__ == "__main__":

    plc_server = PlcServer()
    plc_server.start()

    try:
        while True:
            plc_server.send_cmd()
            time.sleep(0.5)

    except KeyboardInterrupt:
        print("Shutting down")

    finally:
        plc_server.stop()
        plc_server.join()

And here's the Plc stuct ST_LinuxData:
{attribute 'pack_mode':='1'}
TYPE ST_LinuxData :
STRUCT
	arrSeqData : ARRAY[0..1] OF INT;
	bActicated : BOOL;
END_STRUCT
END_TYPE

And my PLC.MAIN:
PROGRAM MAIN
VAR
	stLinuxData : ST_LinuxData;
	iLinuxClk	: UINT;
	bStart : BOOL;
END_VAR

Everything works fine as long as I have TwinCAT open on a laptop, however, as soon as I open the TwinCAT on PLC or connect an HMI as a new ads client, I get 'Port: 30000 already in use as 0x7ffe9a7eba50' error.


Update dispatch-config.md

Description & motivation
Some code examples seem wrong. fix them.
Pre-release docs
Is this change related to an unreleased version of dbt?

[ ] Yes: please

update the base branch to next
add Changelog components: <Changelog>[New/Changed] in v0.x.0</Changelog>
add links to the "New and changed documentation" section of the latest Migration Guide

[x] No: please ensure the base branch is current
[ ] Unsure: we'll let you know!

Thank you for contributing to the dbt product docs! This has been fixed in a different PR, but if you see any other changes for the dispatch-config page then we'd appreciate you  opening a new PR! 🙇🏻♀


Error pq: relation "gateway" does not exist (code: 2)

After upgrading loraserver to 0.16.1 and lora-app-server to 0.7.0 i get the following error on the "Manage Gateway" page, it appears like the gateway table in the database is not created correctly in the migration.
Error pq: relation "gateway" does not exist (code: 2)

Please see the --db-automigrate / DB_AUTOMIGRATE config setting, which will create all tables.

Hi, thanks for a quick response. I have that option enabled in the configuration and have also tried running it manually.
INFO[0000] starting LoRa App Server                      docs="https://docs.loraserver.io/" version=0.7.0
INFO[0000] connecting to postgresql
INFO[0000] setup redis connection pool
INFO[0000] handler/mqtt: connecting to mqtt broker       server="tcp://localhost:1883"
INFO[0000] connecting to network-server api              ca-cert= server="<IP_ADDRESS>:8000" tls-cert= tls-key=
INFO[0000] applying database migrations
INFO[0000] handler/mqtt: connected to mqtt broker
INFO[0000] handler/mqtt: subscribling to tx topic        topic="application/+/node/+/tx"
INFO[0000] migrations applied                            count=0
INFO[0000] starting application-server api               bind="<IP_ADDRESS>:8001" ca-cert= tls-cert= tls-key=

Sorry, I meant you need to add that config setting to the LoRa Server config (as LoRa Server will keep track of your gateways and stats).

Okay, I have enabled it in the loraserver configuration as well, but same result unfortunately.
level=info msg="applying database migrations"
level=info msg="migrations applied" count=0
```

Would you mind sending me a private message on https://gitter.im/loraserver/loraserver, then we can together go over your config and see if it is a config issue or if there is a bug somewhere.

The problem was that I used the same database for both loraserver and lora-app-server, changing loraserver to its own database solved the problem, thanks @brocaar


chore: integration test with ssl

Issue #, if available:
Description of changes:
Work in progress: Currently, running into crt errors when creating tls context.
Supply HiveMQ with server certs so we can exercise CRT tls setup in integration tests.
Why is this change necessary:
How was this change tested:
Any additional information or context required to review the change:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Error currently running into:
[DEBUG] [2023-03-30T03:32:56Z] [000070000e790000] [tls-handler] - static: certificate and key have been set, setting them up now.
[ERROR] [2023-03-30T03:32:56Z] [000070000e790000] [tls-handler] - static: failed to import certificate and private key with error 0.

Based on results from https://github.com/aws-greengrass/aws-greengrass-mqtt-bridge/pull/120, test should pass after https://github.com/aws-greengrass/aws-greengrass-mqtt-bridge/pull/119 is merged


Why the result of object detection of coco dataset has 545 object categories?

As we all know, coco dataset has 80 object categories. But your code result of "object detection" of coco has 545 object categories. So, can you explain why? Thanks

I am using the detector trained on OpenImage.
https://github.com/tensorflow/models/blob/master/research/object_detection/g3doc/detection_model_zoo.md


Feat: Event View Backend

Improved backend of the get method endpoint of "eventview" to show actual event details instead of placeholders.
On the way:

Fixed up get_event_by_id (previously by_id), including error handling
Fixed delete_event and update_event to fit new get_event_by_id function
Moved logger to dependencies so can be used outside of main
Fixed up the event_test_client
Added and fixed up event tests

Codecov Report

Merging #159 (0cec1bc) into develop (bb388ce) will decrease coverage by 1.07%.
The diff coverage is 80.00%.

@@             Coverage Diff             @@
##           develop     #159      +/-   ##
===========================================
- Coverage    99.49%   98.42%   -1.08%
===========================================
  Files           30       30
  Lines          989     1014      +25
===========================================
+ Hits           984      998      +14
- Misses           5       16      +11

Impacted Files
Coverage Δ

app/routers/event.py
87.35% <78.84%> (-12.65%)
:arrow_down:

app/dependencies.py
100.00% <100.00%> (ø)

app/main.py
97.14% <100.00%> (-0.23%)
:arrow_down:

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 bb388ce...d47b925. Read the comment docs.

Codecov Report

Merging #159 (0cec1bc) into develop (bb388ce) will decrease coverage by 1.07%.
The diff coverage is 80.00%.

@@             Coverage Diff             @@
##           develop     #159      +/-   ##
===========================================
- Coverage    99.49%   98.42%   -1.08%
===========================================
  Files           30       30
  Lines          989     1014      +25
===========================================
+ Hits           984      998      +14
- Misses           5       16      +11

Impacted Files
Coverage Δ

app/routers/event.py
87.35% <78.84%> (-12.65%)
:arrow_down:

app/dependencies.py
100.00% <100.00%> (ø)

app/main.py
97.14% <100.00%> (-0.23%)
:arrow_down:

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 bb388ce...d47b925. Read the comment docs.


IRGen: Fix enum.sil test on watchos simulator

The flags field is different. I can't really be bothered decoding
the difference at this point, probably extra inhabitants.

@swift-ci Please test macOS

@swift-ci Please smoke test Linux


pyyaml breaks the build in macOS for Python version 3.7-dev (3.7.0-rc1+)

I've disabled the macOS build for python version 3.7-dev until we decide what to do, or until pyyaml releases a new version that will work.
See yaml/pyyaml#126 and yaml/pyyaml#152 for details and linked issues/info.
It appears there is a workaround (to get pyyaml directly from git instead) which we can consider, but it's only a stopgap.

No, we can just wait until packages catch up to 3.7.

PyYaml seems to have been updated.
Trying a build to see if the problem is resolved on macOS

The new package is working with Python 3.7-dev + macOS. Closing.


Improve the installation for windows

Instead of anaconda, it is better to use venv
Even better if this code will be executed by itself. I have a problem with running commands in venv is it possible to fix it?
Installation code for windows
# Install python 3.10
python -m venv venv
cd .\nunif\venv\Scripts
activate.bat
cd .. # move one directory back
cd ..
pip3 install torch torchvision torchaudio torchtext --index-url https://download.pytorch.org/whl/cu116
pip install -r requirements.txt
python -m waifu2x.download_models
python -m waifu2x.web.webgen

Startup code for windows
cd .\venv\Scripts
activate.bat
cd .. # move one directory back
cd ..
python -m waifu2x.web --no-size-limit --tile-size 400

I usually run using the commands
F:
cd F:\nunif
cd .\venv\Scripts
activate.bat
cd ..
cd ..
python -m waifu2x.web --no-size-limit --cache-ttl 120 --cache-dir F:\nunif\waifu2x\cache --tile-size 400

Yes, it is better not to use the conda package manager if you can avoid using it. However, there are some modules that are only available on conda repository (It is not currently in nunif's dependency modules).

I have a problem with running commands in venv is it possible to fix it?

What is the problem?

cd .\nunif\venv\Scripts
activate.bat
cd .. # move one directory back
cd ..

This part can be done on one line. nunif\venv\Scripts\activate
Here is an example of run-web.bat
cd /d F:\nunif
.\venv\Scripts\activate
python -m waifu2x.web --no-size-limit --cache-ttl 120 --cache-dir F:\nunif\waifu2x\cache --tile-size 400

The command is not executed python -m waifu2x.web --no-size-limit --cache-ttl 120 --cache-dir F:\nunif\waifu2x\cache --tile-size 400
How do I make run-web.bat not close?

It seems that "call" is needed to run activate.bat.
cd /d F:\nunif
call .\venv\Scripts\activate
python -m waifu2x.web --no-size-limit --cache-ttl 120 --cache-dir F:\nunif\waifu2x\cache --tile-size 400
pause

https://github.com/nagadomi/nunif/pull/46

https://github.com/nagadomi/nunif/pull/48


Access level modifications

Closes #186.
A thought regarding the set(encodable:forKey:) interface:
/// Encodes passed `encodable` and saves the resulting data into the user defaults for the key `key`.
/// Any error encoding will result in an assertion failure.
func set<T: Encodable>(encodable: T, forKey key: String) {
    do {
        let data = try JSONEncoder().encode(encodable)
        set(data, forKey: key)
    } catch {
        assertionFailure("Failure encoding encodable of type \(T.self): \(error.localizedDescription)")
    }
}

This PR makes this interface public, without adjusting its error handling behavior. One could argue this is okay, because when using the DefaultsCodableBridge, the same error handling is done (crash on encoding failure).
However, this poses the question, whether the DefaultsCodableBridge should include error handling itself... Maybe make the save function of DefaultsBridge throwing (allowing the user to catch errors if needed), while force-trying on the subscript? (Throwing subscripts are not a thing as far as I know and if they were, they would definitely not look that swifty). Maybe introduce customizable failure fallback strategies as a param on DefaultsCodableBridge initialization?
Maybe, an unexpected failure encoding is practically impossible, if that'd be the case, we should not think too much about it...

Thanks for the PR @fredpi! Gonna review it this week, sorry for the delay.


Agrega automatic package discovery

Description
Agrega automatic package discovery al composer.json
Motivation and context
N/A
How has this been tested?
Se ha probado en una instalación con laravel 5.5*
Screenshots (if appropriate)
Types of changes
What types of changes does your code introduce? Put an x in all the boxes that apply:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:
Go over all the following points, and put an x in all the boxes that apply.
Please, please, please, don't send your pull request until all of the boxes are ticked.

[x] I have read the CONTRIBUTING document.
[x] My pull request addresses exactly one patch/feature.
[ ] I have created a branch for this patch/feature.
[ ] Each individual commit in the pull request is meaningful.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.
[ ] If my change requires a change to the documentation, I have updated it accordingly.
[x] My code follows the code style of this project.

If you're unsure about any of these, don't hesitate to ask. We're here to help!

@kikemarto Recuerda actualizar el paquete en packagist  o configura el hook para que se haga automáticamente :smiley:


Website nav bar is being cut off

The navigation bar is being cut off under certain conditions:

An easy way to replicate is to zoom in on the web page.
The nav bar needs to be fixed so that it never cuts off and all links are always visible.

It might be time to just remove the games from the nav bar and include a table with links on the home page of the website instead of shoving every game in the nave bar. The number of games has increased to the point where adding them all to the nav bar makes it too long and thus not very user friendly.


Cannot log in with an account not in your Bitwarden vault

Clicking Login (or hitting enter) clears the password field if the typed username is not in your Bitwarden vault

Added in 856ca5b9e5b678c9b4953117522b5e132775a1b2


Export native function pointer through natives

This can be used by third-party library like scipy.LowLevelCallable to
avoid conversion costs.

@maartenbreddels can you confirm it works for you test case? I still need to implement proper support for raw pointers as parameters in Pythran but that's for another PR.


function: implement NOW function

Closes #566
Didn't add integration tests for this for obvious reasons.

C# is misbehaving in CI.


[EKA-143] Filtering out linked care contexts during discovery

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 1

Complexity increasing per file
==============================
- src/In.ProjectEKA.HipService/Discovery/PatientDiscovery.cs  2

Clones added
============
- src/In.ProjectEKA.HipService/Link/LinkPatientRepository.cs  2

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- src/In.ProjectEKA.HipService/Discovery/PatientDiscovery.cs  2

Clones added
============
- src/In.ProjectEKA.HipService/Link/LinkPatientRepository.cs  2

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- src/In.ProjectEKA.HipService/Discovery/PatientDiscovery.cs  2

Clones added
============
- src/In.ProjectEKA.HipService/Link/LinkPatientRepository.cs  2
- test/In.ProjectEKA.HipServiceTest/Discovery/PatientDiscoveryTest.cs  4

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- src/In.ProjectEKA.HipService/Discovery/PatientDiscovery.cs  2

Clones added
============
- test/In.ProjectEKA.HipServiceTest/Discovery/PatientDiscoveryTest.cs  4
- src/In.ProjectEKA.HipService/Link/LinkPatientRepository.cs  2

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- src/In.ProjectEKA.HipService/Discovery/PatientDiscovery.cs  2

Clones added
============
- src/In.ProjectEKA.HipService/Link/LinkPatientRepository.cs  2
- test/In.ProjectEKA.HipServiceTest/Discovery/PatientDiscoveryTest.cs  4

See the complete overview on Codacy


Only first page of search results are accessible

When calling the OntoPortalClient.search method the response indicates that there are more results:
>>> client = BioPortalClient()
>>> response = client.search("swim")
>>> response["totalCount"]
575
>>> len(response["collection"])
50

There doesn't appear to be any way to access all 575 results in the above example.

This came up as part of addressing Chris' comment here. I suppose there are any number of different ways to accomplish this. The most obvious to me was to have search return a Generator. That's what I've implemented in #8, but if you have a different design in mind feel free to disregard that PR.


Support for DropWizard 1.3.x.

This updates dropwizard-hk2 to be compatible with Dropwizard 1.3+

This was merged as 8607fe36ba2b7f1e987370ebd1927e6bdd520619


Create Pages “teaching”

Automatically generated by Netlify CMS

👷 Deploy Preview for koytak processing.

Name
Link

🔨 Latest commit
e49084b4d3ec6e09bfc5d563d5a64fde8767faba

🔍 Latest deploy log
https://app.netlify.com/sites/koytak/deploys/6515cd927c29fa000806594f


mark-active and mark-inactive seem to affect the active user and not the selected one

Hey there,
I tried to mark a user inactive (user -u<EMAIL_ADDRESS>update --mark-inactive), got the response that the update went through but can't use the API afterwards. So one of my colleagues checked within their account and could confirm, that my account has been set inactive instead of the one that was planned to be set inactive.
However my colleagues are not able to reactivate my account, all tags with --mark-(in)active seem to affect the active user only, not the one selected with -u username.
Affected versions seem to be 1.2.x up to 1.4.0
Thank you,
CERT@VDE
/jochen

Also worth noting that perhaps using interactive mode would have prevented this:
> cve -i user update -u<EMAIL_ADDRESS>--mark-inactive
You are about to update the<EMAIL_ADDRESS>user with the following changes:

- active: false

Do you want to continue? [y/N]: n
Exiting...

> cve -i user -u<EMAIL_ADDRESS>update --mark-inactive
You are about to update the<EMAIL_ADDRESS>user with the following changes:

- active: false

Do you want to continue? [y/N]: n

If you have ideas of how to improve the user experience, do share :wink:

Thank you for clarification @mprpic !
My idea would be to force the -u after update to when using --mark-inactive or --mark-active. That should prevent users that are not able to understand the help (like me 😉) to accidentally lockout themselves.
Another option would be to use something like --target-user instead of -u after update...
However, with your help a colleague was able to mark my account as active again, thank you.
/Jochen

See linked MR where I made the -u option required so that it's more explicit about which user is being updated.


Access the directory as a signed in user

Why the "Access the directory as a signed in user" permission to be enabled for this. Does the "sign-in and read user profile" permission is not enough.

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

ID: 0fdac5f7-cd43-34c5-5905-dceeb9813952
Version Independent ID: 6fd53f84-dbb2-2a9c-2745-ea1999779113
Content: Embed Power BI content into an application for your customers - Power BI
Content Source: powerbi-docs/developer/embed-sample-for-customers.md
Service: powerbi
GitHub Login: @markingmyname
Microsoft Alias: maghan

@BharathiAN You are correct, there is no difference. The "Sign in and read user profile" permission would work the same.  Thank you for your feedback.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

Hi,
Thanks for your response. In that case, whether this article will be modified accordingly.


fix - semicolon followed by block statement doesnt have new line

Description

[X] Source branch in your fork has meaningful name (not main)

This change makes sure that a new line is added everytime a semicolon is followed by a block statement
Fixes Issue:
fixes #1852
Before Merge Checklist
These items can be completed after PR is created.
(Check any items that are not applicable (NA) for this PR)

[X] JavaScript implementation
[X] Python implementation (NA if HTML beautifier)
[X] Added Tests to data file(s)
[ ] Added command-line option(s) (NA if
[ ] README.md documents new feature/option(s)

This could be considered a breaking change but I believe it is correct now


Ignore Some Messages When Transforming

Why are these changes needed?
This PR implements some utilities that can be useful when applying TransformMessages to GroupChatManager. This is also part 1 of 2.
The changes in this PR allow specific messages to be included or excluded during the transformation process based on user-defined criteria.

filter_configs was improved so the user can have finer control when filtering configs. If the user decides to exclude=False (default), then whichever config matches the filter_dict will be returned (the behavior that is currently in main). If exclude=True then the opposite happens; whichever config matches the filter_dict will be excluded from the final result.

Some transforms like TextMessageCompressor and MessageTokenLimiter can use the filter_dict concept present in autogen to ignore some messages from being transformed.

Example:

Let us say we want to ignore all system messages from being compressed, we can pass in a filter_dict={"role": ["system"]}, exclude=True.
How about when we want to compress messages that only come from a specific agent, filter_dict={"name": ["compress_me_agent"]}, exclude=False.

Future Work:

PrefixName transform: Prepends every message with a text template, e.g. "The agent {agent_name} said this:". It is useful when API providers don't use the name key in the message.
FilterMessage: Drops all messages based on the filter_dict you pass in. E.g. we want to drop all system messages from the context.

Related issue number

Checks

[ ] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[x] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[x] I've made sure all auto checks have passed.

Amazing work @WaelKarkoub, I'll test it out asap :)

@WaelKarkoub, works really well. I used it in some test code as well as using it in the select speaker process using both role and name based filtering - and it worked as described. This takes the select speaker a good step forward in maintaining context (by keeping system and other key messages as is) while trying to keep context counts down.
Thanks!

@marklysze I appreciate the help with testing the PR out :pray:!
For PrefixName, that's actually a good insight. For the first iteration of this transform, we can have a template that only includes the sender (e.g. "{sender} said this:"), and then in a future PR we could expand it to (e.g. "{sender} said to {receiver}"). Given this new behavior, we should probably rename it to something more generic like PrefixTextMessage. At this moment, I'm not worried about optimizing these transforms, but more of a tool to gauge what users want out of autogen.

@marklysze I appreciate the help with testing the PR out :pray:!
For PrefixName, that's actually a good insight. For the first iteration of this transform, we can have a template that only includes the sender (e.g. "{sender} said this:"), and then in a future PR we could expand it (e.g. "{sender} said to {receiver}"). Given this new behavior, we should probably rename it to something more generic like PrefixTextMessage. At this moment, I'm not worried about optimizing these transforms, but more of a tool to gauge what users want out of autogen.

Sounds good, I'd definitely use it for testing out prompts!

@ekzhu good call! I added tests to cover those code paths. I also added a text compressor mock to allow us to test TextMessageCompressor even when llmlingua is not installed

Codecov Report
Attention: Patch coverage is 6.45161% with 29 lines in your changes are missing coverage. Please review.

Project coverage is 16.04%. Comparing base (11d9336) to head (0312d01).
Report is 5 commits behind head on main.

Files
Patch %
Lines

...togen/agentchat/contrib/capabilities/transforms.py
0.00%
20 Missing :warning:

autogen/oai/openai_utils.py
18.18%
9 Missing :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #2661       +/-   ##
===========================================
- Coverage   33.60%   16.04%   -17.56%
===========================================
  Files          87       87
  Lines        9336     9359       +23
  Branches     1987     1992        +5
===========================================
- Hits         3137     1502     -1635
- Misses       5933     7803     +1870
+ Partials      266       54      -212

Flag
Coverage Δ

unittests
16.04% <6.45%> (-17.56%)
:arrow_down:

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

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


lon lat x y

Hi,
Mapsui.Projection.Transformer
Mapsui.Projection.Mercator
Mapsui.Projection.SphericalMercator

are ok
but
Mapsui.Projection.MinimalTransformation
in constructor is bug I think
_toLatLon["EPSG:4326"] = (x, y) => new Point(x, y);
_fromLatLon["EPSG:4326"] = (x, y) => new Point(x, y);
_toLatLon["EPSG:3857"] = SphericalMercator.ToLonLat;
_fromLatLon["EPSG:3857"] = SphericalMercator.FromLonLat;
LatLon = Y,X
LatLon != LonLat
Regards,
Łukasz

I don't think this is a bug. It does not produce unwanted behavior, or does it?
But you do point out bad naming.  _toLatLon and _fromLatLon are internal fields with a confusing name. Renaming them will fix the problem but won't change te behavior.
Did you find this code through the API documentation?

in short, for me:
Point->FromLonLat : lon = x, lat = y
Point->ToLonLat : x = lon, y = lat

so, when I saw this
_toLatLon : Point(x,y)
_fromLatLon : Point(x, y)
_toLatLon : SphericalMercator.ToLonLat
_fromLatLon : SphericalMercator.FromLonLat

I was confused.
That's all (for now ;) )

It is good to point it out. I will fix it.

Done.

2.0 is ok,
What with 1.4 ?


42 add prop all child variables

closes #42

I know we agreed to have a design finishing round later but this toggle is just huge. So I would like to ask for a quick fix to reduce the size to 32x16 px with a 12x12 px circle inside. Please ensure that the label is still aligned after the change.
Expected size of 32x16 px:

Animation is already included. I'm okay with it.

Done. I agree that it looks much better


TypeScript/Flow as options for plugin ecosystem

This is purely curiosity, and I know someone is bound to ask eventually, but are there plans to support the plugin ecosystem using a JavaScript type system such as TypeScript, JSDoc, or Flow?
It is mentioned that one of the motivations of using Rust for the core editor is that it has been shown over time that dynamic languages such as JavaScript (without Flow/JSDoc) take at least a small amount of effort more to maintain than a statically typed language like Rust.
Quoted from Readme "Core Application Logic is Written in Rust"

All of the core application code other than the view logic should be written in Rust. This will ensure that it has a minimal footprint to load and execute, and Rust's robust type system will help us maintain it more efficiently than dynamically typed code.

So, are there plans to leverage existing typing tools for the JavaScript components of this project?
This is purely curiosity, as I understand there are many valid arguments on both sides which are each weighed differently from person to person. And it is also prompted by recent significant improvements to JSX typing support in TypeScript.
Thanks!

We definitely plan to support JavaScript for extensions and it's my goal to also publish type definitions for our public APIs. I am a fan of static typing and we will probably use a typed JS variant in the UI code, but we have deferred it in favor of making progress on areas of the system that represent more risk.

Definitely makes sense!
Thanks
On Fri, Mar 9, 2018, 2:28 PM Nathan Sobo<EMAIL_ADDRESS>wrote:

Closed #31 https://github.com/atom/xray/issues/31.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/atom/xray/issues/31#event-1514073257, or mute the
thread
https://github.com/notifications/unsubscribe-auth/ACyjU0S_NloSZyklN5Ej8xeWEYmchSt0ks5tcuXggaJpZM4SjIKW
.


"A Javascript error occured in the main process" windows install

OS version and name:  Windows 10
Hyper.app version: 1.0.0
Link of a dump with the contents of your .hyper.js:  dump

Issue
I'm trying to install Hyper by running hyper-Setup-1.0.0.exe and I get the above error. Also here's a screenshot of the full error.
Hyper.exe also stays running as a process after this dialog closes. If I run the set up again after closing the dialog I get a different error, although if I kill the Hyper process through Task Manager and run the setup.exe again then I get the same error included in the screenshot.

@bitttttten can you try to run the setup as an admin ?

I ran npm cache clean and I was already running it as an Administrator.
This is the log I get if I try to run it again without closing the hyper.exe instances that are lefting hanging.


Multiple custom routes

https://github.com/Nugine/s3s/pull/209 has added the ability to add a custom route. Would it be possible to extend this to add more than one route? If so, I could give it a try!
Ok, looking at it more closely. We could have a generic /admin and handle all the subpaths ourselves.
BR

You can use a router like matchit inside the custom route, which is the recommanded method.
I'm also open to accept a PR that adds multiple custom routes if you like this way.
BTW, the Req/Resp types of custom route seem strange due to historical reasons.
A (huge) breaking change is scheduled for these types (#210). It is not happening soon.

Thanks for the answer, I will try the matchit idea and if I miss something I will come back to the multiple routes option.


bump compat for "CxxWrap" to "0.8"

Here it is https://github.com/JuliaFEM/MFrontInterface.jl/pull/29
The problem relates to CxxWrap, because it goes hand in hand with libcxxwrap. When I use BinaryBuilder.jl approach the binaries are in the different repo (https://github.com/TeroFrondelius/mgisBuilder), which needs to be released first in order to be able to test new CxxWrap version. I hope my explanation makes sense. Anyway, this is the reason why I have fixed the CxxWrap = "= 0.8.1"
Maybe CompatHelper shouldn't make a pull request when compat pins exactly one version using "=" sign. What do you think?

Thats just a bug, nothing should happen there.

Probably caused by https://github.com/bcbi/CompatHelper.jl/pull/33

Thanks @fredrikekre, I thought so too.

So, technically, this is the intended behavior. CompatHelper wants you to support the latest release of all of your dependencies.
The latest release of CxxWrap is CxxWrap version 0.8.2.
Your current compat entry is:
[compat]
CxxWrap = "= 0.8.1"

The compat entry "= 0.8.1" does not include the version 0.8.2. So CompatHelper wants you to broaden your compat entry to include 0.8.2. There are two possible ways to do this:

CxxWrap = "= 0.8.1, 0.8"
CxxWrap = "0.8"

By default, CompatHelper will suggest option 1. (This is the functionality implemented in #33.)
However, if you change your CompatHelper configuration (via keyword arguments to the CompatHelper.main function), you can tell CompatHelper to do any of the following:

Suggest only option 1 (default)
Suggest only option 2
Suggest both option 1 and option 2 and let you pick

In my opinion, this is the correct behavior. The purpose of CompatHelper is to make sure that you always support the latest release of a dependency. You currently do not support the latest release of CxxWrap, so CompatHelper opened a pull request.

So, what do you do if you don't agree with CompatHelper? Easy! Just take the following steps:

Apply the wontfix label to the pull request, or whatever labels you want. (This step is not required. I just think it's a good idea.)
Leave the pull request open. As long as the pull request is open, CompatHelper will see it and will not open another pull request.

In general, if you don't agree with CompatHelper for whatever reason, that's totally fine. It's your repo, and you should make the final decision. Just leave the pull request open and do not merge it. As long as the pull request is open, CompatHelper will not open another pull request that does the same thing.

Sorry, I missed the =.

I see it's a feature rather than a bug. I will close this. Thanks for the advice @DilumAluthge.


feat(injected-css): optionally export css for consumer to import

Description
This is a POC for removing all of the inline styles for CSP safe alternatives. This is tied to this discussion.
This converts inline styles into css class equivalents, and exports them as part of the npm package. The idea is that consuming apps would import these css files as part of their build. I left this as a progressive enhancement where by providing skipStyleInjection it's assumed you're importing these CSS files, but consumers who are not can still use the injected inline styles.
Looking for input on how to have one source of truth between the css file and the inline styles. I don't think there's an easy way to do it but was hoping to not have to change two places at once.

I appreciate the contribution and consideration of the tradeoffs here! I'm going to close this PR for now because a) there's a huge backlog I'm trying to clear out, and b) I'm not sure I fully understand the problem and solution at this point. Glad to continue the discussion here to see how we can move forward.