hwloc: 2.9.1 -> 2.9.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/hwloc/versions.
meta.description for hwloc is: Portable abstraction of hierarchical architectures for high-performance computing
meta.homepage for hwloc is: https://www.open-mpi.org/projects/hwloc/
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.9.2 with grep in /nix/store/a5s0crfxyfahjsdr8z4hyxp3igxnc39y-hwloc-2.9.2
found 2.9.2 in filename of file in /nix/store/a5s0crfxyfahjsdr8z4hyxp3igxnc39y-hwloc-2.9.2

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

282 total rebuild path(s)

282 package rebuild(s)

First fifty rebuilds by attrpath
EBTKS
apacheHttpdPackages.mod_tile
apacheHttpdPackages_2_4.mod_tile
bicgl
bicpl
cdo
cloudcompare
cntk
conglomerate
cp2k
dbcsr
dl-poly-classic-mpi
eccodes
elmerfem
elpa
entwine
ezminc
febio
fftwMpi
freecad
frigate
gdal
getdp
globalarrays
gmt
gnudatalanguage
gplates
gpt2tc
grass
gromacs
gromacsCudaMpi
gromacsDouble
gromacsDoubleMpi
gromacsMpi
haskellPackages.haskell-mpi
haskellPackages.hgdal
haskellPackages.mpi-hs
haskellPackages.mpi-hs-binary
haskellPackages.mpi-hs-cereal
haskellPackages.mpi-hs-store
haskellPackages.tensorflow
haskellPackages.tensorflow-core-ops
haskellPackages.tensorflow-logging
haskellPackages.tensorflow-ops
hdf5-mpi
highfive-mpi
hp2p
hpcg
hpl
hpx

Instructions to test this update (click to expand)

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

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

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

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review took longer than 45m and timed out

Maintainer pings
cc @fpletz @markuskowa for testing.

https://raw.githubusercontent.com/open-mpi/hwloc/v2.9/NEWS


Intervention Dose and Adminstration

Intervention, dose, administration route etc

Actually kind of a biomedical drug concept: as it is a new drug this information cannot be retrieved as a concept code from existing dictionaries
Also for the reference drug, the dose and route cannot be specified. Dictionaries covering this will often also cover market information which can vary from country to country so we cannot specify in that detail. In the study design phase

Erin/Craig: The eCPT contains a large table with multiple attributes associated with an administered interventions (see section 6.1., table 1 in eCPT (attached). Previous discussions with Chris/Gaston/Jarod indicated that these should really be modeled as attributes of a separate/new class of something like 'Administered Investigational Intervention' and not added as attributes to the existing class.
The table contains things like:

Route of Administration
Dosage Level
Dose Formulation
Dose Strength
Name/Description
etc.
Consider using that table as a guide to create a new class and attributes?
Also consider that there may be more than one compound within the study, and/or each attribute of each intervention (even if only one compound is used in the study) may have more than one response (e.g., different arms with different routes of administration for the same compound). The model must be flexible enough to accommodate this.

CPT_CoreBWE-v009.docx

Seems like a large task so we can defer this for a while but now we have a framework to work from.

Hi team,
I compared M11 variables from Daves diagram, SDTM TS, CT.gov, CTIS/IDMP in attached spreadsheet. A good basis for adding it to the model.
(https://app.zenhub.com/files/534339959/d151b479-13e4-483a-9954-7965ff9e101a/download).
As discussed with Dave last Tuesday:

If in M11 or SDTM TS - please add to model / rest can be deferred to later phase but maybe added if univocal and complementary.
See  picture for proposed model logic. I will explain the potential issues during the meeting

Comment from Mikkel at the SME meeting of 01/08 based on the above diagram: The study builder links to elements instead of arms. This is more precise. The relationship would be the same (many-many). I agree that it is more precise. Other data models (like ct.gov) refer to arms but the arms can be indirectly linked via the elements as well.

Good comment from Diana Wold: duration of treatment can be added as well with standard coding.

Initial sketch design @BSnoeijerCD

@dih-cdisc
To allow for the automated calculation of InterventionDuration I suggested a link to Procedure as an administration will be often recorded as an activity-procedure in the SoA as well. This might be complex and we can also consider this for the next phase.
We need a use parameter to indicate what TS parameter should be mapped to the treatment. We have the following TS parameters: TRT/CMPTRT/PLACEBO/TCNTRL. I thought this roughly aligned with the M11 Use parameter and thus can be added for both purposes.

Based on the Ct.gov review: the following Intervention Types are included in their list:

Drug: Including placebo
Device: Including sham
Biological/Vaccine
Procedure/Surgery
Radiation
Behavioral: For example, psychotherapy, lifestyle counseling
Genetic: Including gene transfer, stem cell and recombinant DNA
Dietary Supplement: For example, vitamins, minerals
Combination Product: Combining a drug and device, a biological product and device; a drug and biological product; or a drug, biological product, and device
Diagnostic Test: For example, imaging, in vitro
Other
This is different from investigational/non-investigational product etc.

@dih-cdisc To allow for the automated calculation of InterventionDuration I suggested a link to Procedure as an administration will be often recorded as an activity-procedure in the SoA as well. This might be complex and we can also consider this for the next phase.
We need a use parameter to indicate what TS parameter should be mapped to the treatment. We have the following TS parameters: TRT/CMPTRT/PLACEBO/TCNTRL. I thought this roughly aligned with the M11 Use parameter and thus can be added for both purposes.

@dih-cdisc: Can we combine Role and Use with regard to the above? So add the Treatment, comparator treatment, placebo to the role list? Control type (TCNRL) can then be directly derived from that.

@dih-cdisc @EMuhlbradt @czwickl
First step: changing names and adding attributes to StudyIntervention Class. I made some notes with questions. Note: Two additional changes from Daves diagram:

I also changed the relationship name from out of studyDesign class. in alignment with the change in class name.
I change minimumresponseduration from string to quantity. Is that ok?

I will remove the notes in the final 2.6 version again.

Another version with added arrows from source to destination relationship. This might be more clear to Erin and Craig:

And we can evaluate whether this would be good to do for the whole diagram.

Here is the complete provisional version. I think it is good to go through the questions on our scrum meeting tomorrow.

Additional notes / questions:

For Role - I would not include a role called intervention as that is the whole class name. Better to use investigational (as compared to rescue, prohibited etc).
For Role - should we add comparator then?
For Estimand class: should we change the relationship name treatment to intervention?
See notes in diagram for additional suggested changes. All is added now provisionally.

@dih-cdisc @BSnoeijerCD @EMuhlbradt
Notes from scrum meeting discussion on 2023-10-19 (Berber, Erin, Craig):

we will use Berber's latest diagram above as base UML for CT development

StudyIntervention class is meant to be very broad and not limited to the investigational treatment. We will let the 'role' and 'type' attributes define an instance of the class

a codelist needs to be developed for the 'role' attribute in the StudyIntervention class which contains values needed to align with M11 'use'

codelists for 'role' and 'type' attributes in the StudyIntervention class should be co-developed with the class to ensure everything works together (makes logical sense) for an instance of the class

we will add 'method' (administration method concept) attribute to the Administration class

we will remove 'name' and 'label' attributes from the AdministrationDuration class, but leave them in the Administration class (i.e., it is very unlikely that a duration would ever be named)

the AdministrationDuration class is meant to be across administrations and not within a single instance of an administration.

Please comment if any of these miss-represent our decisions or if anything is missing from the list.
We ask Dave to review for agreement or comment on any items of concern that will need further discussion.

@dih-cdisc @BSnoeijerCD @czwickl
Sprint 8 draft file of changes to review:
DDF.Terminology_Sprint.8.changes_2023-10-25pm.xlsx

@dih-cdisc @BSnoeijerCD @czwickl : for the class StudyIntervention:
For the propose attribute 'designation': Given the specificity of the intended meaning of the term (whether the study intervention is a IMP or an AxMP, we think the proposed attribute 'designation' is maybe too vague. Berber and Dave to consider an attribute name update should take place. Something like 'medicinalProductDesignation' or 'mpDesignation'?

"productDesignation"? just a bit shorter.

@EMuhlbradt @czwickl @BSnoeijerCD will pick up on the new CT spreadsheet once we have resolved few queries above and the renames in the other ticket for release 2.6 for the API changes

@dih-cdisc @BSnoeijerCD
DDF.Terminology_Sprint.8.changes_2023-10-26.xlsx
Updated draft CT file listing all changes (new and changes) with updates from today's discussions

@EMuhlbradt @daveih @czwickl
Checked Excel file and found the following issues:

AS indicated before, I renamed the relationship that goes from studyDesign to studyIntervention class to studyInterventions instead of investigationalInterventions. This is in alignment with the class name change. It is also more appropriate as we do not only want the include investigational Interventions in this new class but all study related intervention. Can you align the CT with that new name?
the class name was only partly removed from the following attributes in CT. Dave and I indicated to remove the full name in the Excel file:

propertyResponseCodes should be responseCodes in CT
surrogateReference should be reference in CT

CT missed these changes for previous/next relationships

previousEncounter, nextEncounter should be previous and next
previousActivity, nextActivity should be previous and next

CT missed the following name changes for children relationships (as discussed during one of the scrums this week and indicated in the Excel, we were planning to change them as well )

bcCategoryChildren should be children
contentChildren should be children

I send out an E-mail about the re-added content relationship to studyDesign. I think that is not correct as it was moved from studyDesign to studyProtocolDocumentVersion (see UML). Version 2.5 already includes a new relationship from studyDesign to this new studyProtocolDocumentVersion class called documentVersion.

@BSnoeijerCD @dih-cdisc @czwickl : All draft CT updates have been made in the 2-6 release branch on Github. I look forward to QC comments.

@EMuhlbradt @daveih @czwickl
Checked Excel file and found the following issues:

AS indicated before, I renamed the relationship that goes from studyDesign to studyIntervention class to studyInterventions instead of investigationalInterventions. This is in alignment with the class name change. It is also more appropriate as we do not only want the include investigational Interventions in this new class but all study related intervention. Can you align the CT with that new name?

the class name was only partly removed from the following attributes in CT. Dave and I indicated to remove the full name in the Excel file:

propertyResponseCodes should be responseCodes in CT
surrogateReference should be reference in CT

CT missed these changes for previous/next relationships

previousEncounter, nextEncounter should be previous and next
previousActivity, nextActivity should be previous and next

CT missed the following name changes for children relationships (as discussed during one of the scrums this week and indicated in the Excel, we were planning to change them as well )

bcCategoryChildren should be children
contentChildren should be children

I send out an E-mail about the re-added content relationship to studyDesign. I think that is not correct as it was moved from studyDesign to studyProtocolDocumentVersion (see UML). Version 2.5 already includes a new relationship from studyDesign to this new studyProtocolDocumentVersion class called documentVersion.

@BSnoeijerCD -> Thanks for these; all changes made in the CT deliverables in 2-6 branch release.

@dih-cdisc Uploaded CT aligned UML version.

@EMuhlbradt @czwickl
While preparing the presentation for the SME meeting, I found that during my check I missed the new relationship from procedure to StudyIntervention.
Could you add this relationship to the CT? The relationship is called studyIntervention.
This relationship allows users to link a procedure to a specific study intervention. In that way the intervention can be  is directly related to the study timeline.

@EMuhlbradt @czwickl
While preparing the presentation for the SME meeting, I found that during my check I missed the new relationship from procedure to StudyIntervention.
Could you add this relationship to the CT? The relationship is called studyIntervention.
This relationship allows users to link a procedure to a specific study intervention. In that way the intervention can be  is directly related to the study timeline.

Summary of changes for Intervention and Dosing. Also includes name changes


Allow to handle reuse time limit

Some SMTP server like postfix have a reuse time limit:
http://www.postfix.org/postconf.5.html#smtp_connection_reuse_time_limit
After this time, which is non-infinite by default in postfix, any command sent will raise a 4.4.2 *host* Error: timeout exceeded Exception, QUIT and RSET too.
The only way to keep using the same smtp object instance, is to force the session end by quitting without QUIT message, and fclose the stream resource, to immediately reopen it. This is the use case:
$transport = new Smtp();
$transport->send(new Message());

// A long running script, likely a server-side only tool script
sleep(600);

$transport->send(new Message());

Since we do not have an internal transport timer between the two calls, we need to track the time spent saving the connect start time, and checking it in the second send call.
By default, the reuse time limit I set is -1 which corresponds to infinite and mantain backward compatibility.
0 can be set to restart the connection on every message.
To tackle the postfix configuration, 270 seconds are enough.
As soon as you accept and merge the PR, I'm going to PR against https://github.com/zendframework/zf2-documentation with explanation on why and how.

Travis fails with Array to string conversion on Php 5.6.5 + PHPUnit 4.8.7, but I don't know why and on my system is passing either with and without coverage clover.
Any idea?

Ping @Maks3w :)

There is no need of modify AbstractProcotol.
This is very specific scenario and its better if you implement your own class.

Yup, no need to change AbstractProtocol, we can create a Protocol\TimeoutSmtp and a Transport\TimeoutSmtp like I did in my production app if you prefer
Postfix is pretty common, long running scripts are a bit less common, but still common, I hope more users can vote it

I'm wondering why this fix was not excepted. Long running processes that sent email are not a 'specific scenario' IMHO, So if this PR fixes it, why not use it?
I have been using zend-mail for years with this problem and have jumped through several hoops to work around it. (crontab, supervisor restarts, queue systems). Every so often I revisit it hoping the latest version would solve the problem.

I'm wondering why this PR wasn't accepted. I don't agree with "This is very specific scenario and its better if you implement your own class.". Long running PHP process to sent email is not a very specific scenario IMHO.
I have been looking into this issue for years everytime I wasn't happy with the work arounds so a fix would be very welcomed...

@basz https://github.com/zendframework/zend-mail/pull/117 and https://github.com/zendframework/zend-mail/pull/131 are enough to handle nicely this behaviour.
I do this in my app flawlessly; delegators of ServerManager help a lot.

@basz you can have a look at https://github.com/Slamdunk/zend-mail-extensions for how to do the things.

thanks! Would you happen to have a config example? @Slamdunk

Soon, need same spare time first :/

Got it to work, no need to do delegator config as your thing does it. Just needed to inject a service manager. It works great! Thanks again.


Fix Entrypoint typo

The dockerfile entrypoint has bash misspelled.
Signed-off-by: Steve Sloka<EMAIL_ADDRESS>

No, this is supposed to be ash (alpine shell)


adjusting ampere to solar power does not work on 1.0.36

Hi
now I am on 1.3.6
Hardware V2 FW 042.0
Adapter version: 1.0.36
JS-Controller version: 6.0.3
Node version: 10.8.1
Operating system: QNAP NAS: Docker Buanet
No adjustment...
No difference between "force 1 phase" or "automatic" for phase switch
My car can only use 1 phase
Solar only is set to true
It stays at 6A when I choose 6A in app or it stays at 12A when I choose 12A
It worked on older versions. When I now go back to 1.029 it jumps from 6A immediately to 20A
Debug from 1.036:
go-e.1
2024-06-18 13:38:58.894	debug	{"version":"B","tme":"1806241338","rbc":"62","rbt":"2659627092","car":"2","amp":"6","amx":"6","err":"0","ast":"1","alw":"1","stp":"0","cbl":"32","pha":"63","tmp":"16","tma":[35,34.38,36,38.75,-0.13,-0.13],"amt":"32","dws":"441901","dwo":"0","adi":"0","uby":"0","eto":"38980","wst":"3","txi":"0","nrg":[223,226,225,1,53,0,0,12,0,0,0,122,100,14,0,52],"fwv":"042.0","sse":"023193","wss":"MundM","wke":"","wen":"1","cdi":"0","tof":"101","tds":"1","lbr":"255","aho":"3","afi":"6","azo":"1","ama":"32","al1":"6","al2":"8","al3":"10","al4":"12","al5":"32","cid":"255","cch":"65535","cfi":"65280","lse":"1","ust":"1","wak":"c12aa319ee","r1x":"2","dto":"0","nmo":"0","sch":"AAAAAAAAAAAAAAAA","sdp":"0","eca":"0","ecr":"0","ecd":"0","ec4":"0","ec5":"0","ec6":"0","ec7":"0","ec8":"0","ec9":"0","ec1":"0","rca":"","rcr":"","rcd":"","rc4":"","rc5":"","rc6":"","rc7":"","rc8":"","rc9":"","rc1":"","rna":"","rnm":"","rne":"","rn4":"","rn5":"","rn6":"","rn7":"","rn8":"","rn9":"","rn1":"","loe":0,"lot":0,"lod":65535,"lts":0,"lom":0,"lop":0,"log":"","lon":0,"lof":0,"loa":0,"lch":0,"mce":0,"mcs":"<IP_ADDRESS>","mcp":1883,"mcu":"","mck":"","mcc":0,"pco":"","cmf":0,"ath":1200,"psp":11500,"cco":180,"map":123,"frm":1,"los":"{"chg":[["053644",10,0,1,0,[1,2,3],6]]}"}
go-e.1
2024-06-18 13:38:58.566	debug	Start ajust by foreign Object with 2522 Watts. (4513 solarPower - 300 Buffer - 1690 House consumption + 0 House battery)
go-e.1
2024-06-18 13:38:58.528	debug	Start ajust by foreign Object with 2522 Watts. (4513 solarPower - 300 Buffer - 1690 House consumption + 0 House battery)
go-e.1
2024-06-18 13:38:58.525	debug	Start ajust by foreign Object with 2522 Watts. (4513 solarPower - 300 Buffer - 1690 House consumption + 0 House battery)
go-e.1
2024-06-18 13:38:58.525	debug	Start ajust by foreign Object with 2522 Watts. (4513 solarPower - 300 Buffer - 1690 House consumption + 0 House battery)
go-e.1
2024-06-18 13:38:58.431	debug	Start ajust by foreign Object with 2522 Watts. (4576 solarPower - 300 Buffer - 1753 House consumption + 0 House battery)
go-e.1
2024-06-18 13:38:53.343	debug	Start ajust by foreign Object with 2602 Watts. (4576 solarPower - 300 Buffer - 1673 House consumption + 0 House battery)
go-e.1
2024-06-18 13:38:53.342	debug	Start ajust by foreign Object with 2602 Watts. (4576 solarPower - 300 Buffer - 1673 House consumption + 0 House battery)
go-e.1
2024-06-18 13:38:53.342	debug	Start ajust by foreign Object with 2602 Watts. (4576 solarPower - 300 Buffer - 1673 House consumption + 0 House battery)
go-e.1
2024-06-18 13:38:53.341	debug	Start ajust by foreign Object with 2602 Watts. (4576 solarPower - 300 Buffer - 1673 House consumption + 0 House battery)
go-e.1
2024-06-18 13:38:53.300	debug	200 with message: OK
go-e.1
2024-06-18 13:38:53.235	info	Set value 16 of id amx
go-e.1
2024-06-18 13:38:53.234	debug	Current used 1115 Watts with 5 Ampere (sum) by 1 Phases and adjusting this with 2602 watts by 223 Volts (avg) to new max of 16 Amperes per Phase; PhaseSwitchLevel: from 1 at >4500
go-e.1
2024-06-18 13:38:53.228	debug	Start ajust by foreign Object with 2602 Watts. (4584 solarPower - 300 Buffer - 1681 House consumption + 0 House battery)
go-e.1
2024-06-18 13:38:48.078	debug	Start ajust by foreign Object with 2593 Watts. (4584 solarPower - 300 Buffer - 1690 House consumption + 0 House battery)
go-e.1
2024-06-18 13:38:48.078	debug	Start ajust by foreign Object with 2593 Watts. (4584 solarPower - 300 Buffer - 1690 House consumption + 0 House battery)
go-e.1
2024-06-18 13:38:48.038	debug	Start ajust by foreign Object with 2593 Watts. (4584 solarPower - 300 Buffer - 1690 House consumption + 0 House battery)
go-e.1
2024-06-18 13:38:48.037	debug	Start ajust by foreign Object with 2593 Watts. (4584 solarPower - 300 Buffer - 1690 House consumption + 0 House battery)
go-e.1
2024-06-18 13:38:47.984	debug	Start ajust by foreign Object with 2593 Watts. (4796 solarPower - 300 Buffer - 1902 House consumption + 0 House battery)
go-e.1
2024-06-18 13:38:43.861	debug	{"version":"B","tme":"1806241338","rbc":"62","rbt":"2659612065","car":"2","amp":"6","amx":"6","err":"0","ast":"1","alw":"1","stp":"0","cbl":"32","pha":"63","tmp":"16","tma":[35,34.5,35.88,39,-0.13,-0.13],"amt":"32","dws":"440067","dwo":"0","adi":"0","uby":"0","eto":"38980","wst":"3","txi":"0","nrg":[223,227,225,1,53,0,0,12,0,0,0,123,100,6,9,55],"fwv":"042.0","sse":"023193","wss":"MundM","wke":"","wen":"1","cdi":"0","tof":"101","tds":"1","lbr":"255","aho":"3","afi":"6","azo":"1","ama":"32","al1":"6","al2":"8","al3":"10","al4":"12","al5":"32","cid":"255","cch":"65535","cfi":"65280","lse":"1","ust":"1","wak":"c12aa319ee","r1x":"2","dto":"0","nmo":"0","sch":"AAAAAAAAAAAAAAAA","sdp":"0","eca":"0","ecr":"0","ecd":"0","ec4":"0","ec5":"0","ec6":"0","ec7":"0","ec8":"0","ec9":"0","ec1":"0","rca":"","rcr":"","rcd":"","rc4":"","rc5":"","rc6":"","rc7":"","rc8":"","rc9":"","rc1":"","rna":"","rnm":"","rne":"","rn4":"","rn5":"","rn6":"","rn7":"","rn8":"","rn9":"","rn1":"","loe":0,"lot":0,"lod":65535,"lts":0,"lom":0,"lop":0,"log":"","lon":0,"lof":0,"loa":0,"lch":0,"mce":0,"mcs":"<IP_ADDRESS>","mcp":1883,"mcu":"","mck":"","mcc":0,"pco":"","cmf":0,"ath":1200,"psp":11500,"cco":180,"map":123,"frm":1,"los":"{"chg":[["053644",10,0,1,0,[1,2,3],6]]}"}
go-e.1
2024-06-18 13:38:42.946	debug	Start ajust by foreign Object with 2659 Watts. (4796 solarPower - 300 Buffer - 1836 House consumption + 0 House battery)
go-e.1
2024-06-18 13:38:42.905	debug	Start ajust by foreign Object with 2659 Watts. (4796 solarPower - 300 Buffer - 1836 House consumption + 0 House battery)

Hi.
Can you say it bit more about the expected behavior?
Currently I don't get the issue.

AMX is available since 033.
Can you please check what happened if you set amx manually within the objects.


session_id is 00000000-0000-0000-0000-000000000000 for 20% of the log entries in Android

Hello,
I am working on a React Native application for both Android and iOS. When inspecting logs in DD dashboard I noticed that the 'session_id' field on Android logs is '00000000-0000-0000-0000-000000000000' 20% of the times (on a sample of 700k logs entry I analyzed 122.500 shows the issue). On iOS this is not happening.

Steps to reproduce the issue:
In my project Datatog is initialized with the following code.
`const datadogConfig = new DatadogProviderConfiguration(
'CLIENT_TOKEN,
'ENV',
'RUM_APPLICATION_ID',
true,
true,
true,
);
datadogConfig.verbosity = SdkVerbosity.INFO;
datadogConfig.sessionSamplingRate = 10;
datadogConfig.resourceTracingSamplingRate = 10;
datadogConfig.firstPartyHosts = ['abcdefg.com'];
datadogConfig.initializationMode = InitializationMode.ASYNC;

...

`
and logs are sent with
   DdLogs.info('logging something', { ....     });
Describe what you expected:
I expect session_id to not be 00000000-0000-0000-0000-000000000000.
Additional context

Version of the SDK
"@datadog/mobile-react-native": "1.2.0",
"@datadog/mobile-react-navigation": "1.2.0"

Hi @AleCatSS, thanks for reaching out and reporting this.
We're able to see this problem on our end as well, we're going to investigate and let you know our findings.

We're having the same issue, unfortunately.

Any update here?

Hi @tmadej, we haven't yet reached a conclusion on what is the cause of this issue.
We'll update the thread once we have news.

I also had experienced this, might this be related to when the user has no internet connection? Not sure where these ids are generated.


Add Conan package manager usage example

Update README.txt to show how cpr can be retrieved using Conan package manager.

Overall great work. Thanks.

Nice. Thanks.


{Graph} Re-record related tests of resource module

Related command
See https://github.com/Azure/azure-cli/issues/22174
Re-record resource module tests that include az ad or az role.
Description

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Graph


Users/ramukaritik/co presence vscode web

Add description about what changes we are pushing as part of this PR and update the heading as well.


Edit mode keeps opened when clicking in other field

This happens in verison 0.1.9:
http://jsfiddle.net/qq628hpm/2/
Click on field 1, then click on field2.

same

Same issue here..

Is there any workaround or should we wait for new release for it to be fixed and use 1.8.0 meanwhile?
Any downsides in using 1.8.0?

Can you try this in the latest release?

@eugef this should be fixed with the latest release, please close.


Distributed deployment - Improvements and hardening

Description:
Following Improvements can be made to current distributed deployment implementation.

Improve testcases with semantic validations
Improve Kubernetes artifacts for Distributed deployment to meet installation experience guidelines.
Add support for existing Kafka topics and provide users the ability to decide on Kafka topic creation.
Improve manager nodes with ability to be fronted by a LB.
Improve allocation algorithm to be based on on resource availability.
Add support for sources other than Kafka and introduction of Receiver node types.
Improve rebalancing algorithm when nodes leave and re-join.
Improve websocket sink to work in distributed environment.
Suggested Labels:
Improvements

Affected Product Version:
4.2.0

Fixed with PR #1289, #1287, #1270, #1261, #1215, #1291, #1266, #1264 https://github.com/wso2/kubernetes-sp/pull/2


Dockstore branch setting helx UI

Restructures static assets and updates populate_env script

BRANDS          := braini cat heal restartr scidas eduhelx argus
BRANDS          := braini bdc heal restartr scidas eduhelx argus

Should tracs be in this list or is that for later?

BRANDS          := braini cat heal restartr scidas eduhelx argus
BRANDS          := braini bdc heal restartr scidas eduhelx argus

Should tracs be in this list or is that for later?

tracs related changes are on ohdsi_in_a_pod branch. This should be merged first i guess.

I think we should update the helx-ui image in the Dockerfile, if there is a latest one.


Update auto-generated CLI output to separate command and output

I'm starting to go through the book and the auto generated CLI output doesn't copy and paste well into the terminal because the, "$" is prepended to the command and the output is in the text box.
I was thinking I could remove the, "$" from commands and separate them from their output so they can be directly copy and pasted into the terminal.
It's an opinionated change, but I thought it might make the UX a little better so I thought I'd pitch it. What do y'all think?

I personally like it better as a "listing", i.e. just like it is now. Rough rationalization of it is that it's much easier to lookup something you need to refresh in your understandings when revisit the docs; and facilitation of copy-paste of examples could have even negative value taken broadly.
On July 9, 2022 5:24:14 AM GMT+03:00, soyboy @.***> wrote:

I'm starting to go through the book and the auto generated CLI output doesn't copy and paste well into the terminal because the, "$" is prepended to the command and the output is in the text box.
I was thinking I could remove the, "$" from commands and separate them from their output so they can be directly copy and pasted into the terminal.
It's an opinionated change, but I thought it might make the UX a little better so I thought I'd pitch it. What do y'all think?
--
Reply to this email directly or view it on GitHub:
https://github.com/foundry-rs/book/issues/433
You are receiving this because you are subscribed to this thread.
Message ID: @.***>
--
Sent from my Android device with K-9 Mail. Please excuse my brevity.

@skaunov that makes sense, I personally even learn more when I write out the commands. I appreciate you sharing that perspective.


Could not find 'php-cs-fixer'. The program may not be installed.

Could not find 'php-cs-fixer'. The program may not be installed.
See php-cs-fixer installation instructions at https://github.com/Glavin001/atom-beautify#beautifiers or go to https://github.com/FriendsOfPHP/PHP-CS-Fixer
Your program is properly installed if running 'where.exe php-cs-fixer' in your CMD prompt returns an absolute path to the executable.

same problem here ~

same issue here

Worst implementation of beautifying code - why not making packages just like linter-<lang>.
Didn't make it work for any language - from C/C++ to PHP.
Just keeps saying 'program may not be installed' - though it's on $PATH, and totally functional

@scpketer we looked into doing that in the new Unibeautify implementation, but we are scope naming our packages and Atom doesn't recognize them in that way.
For your error, I think you still need to have the absolute path to the exe in Atom Beautify settings. Please try that. @Glavin001 do you know if it is supposed to work without the path in settings, but in the environment variable?

@stevenzeck I don't have 'exe', I'm on Unix :)
Besides, absolute path to the lang fixer is 'deprecated' and gives me a warning.

got this issue some time ago ,
just go to beautify setting,
remove cs-fixer phar path from exutable, and put into php setting,
at this point beautify will notice you that is deprecated,
just go to to you user folder (i'm assuming you are uing win10) ".../.atom\packages\atom-beautify\src\beautifiers" and edit file "php-cs-fixer.coffee" at line 46 from "cs_fixer_path: true" to "cs_fixer_path: false"
and it should work again


test

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[ ] Make sure the commit message is formatted like:
[GEARPUMP-<Jira issue #>] Meaningful description of pull request
[ ] Make sure tests pass via sbt clean test.
[ ] Make sure old documentation affected by the pull request has been updated and new documentation added for new functionality.

Codecov Report

Merging #179 into master will increase coverage by 0.21%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #179      +/-   ##
==========================================
+ Coverage   71.63%   71.85%   +0.21%
==========================================
  Files         191      191
  Lines        6086     6086
  Branches      527      527
==========================================
+ Hits         4360     4373      +13
+ Misses       1726     1713      -13


Security vulnerability with<EMAIL_ADDRESS>

Describe the bug
Dependabot alerts for a high severity vulnerability:
Dependabot cannot update is-svg to a non-vulnerable version
The latest possible version that can be installed is 3.0.0 because of the following conflicting dependency:
<EMAIL_ADDRESS>requires<EMAIL_ADDRESS>via a transitive dependency on<EMAIL_ADDRESS>The earliest fixed version is 4.2.2.
CVE-2021-28092
Suggested dependabot remediation
Upgrade is-svg to version 4.2.2 or later. For example:
"dependencies": {
  "is-svg": ">=4.2.2"
}

or…
"devDependencies": {
  "is-svg": ">=4.2.2"
}

Should be fixed now. New postcss-svgo patch release dropped is-svg.

Should be fixed now. New postcss-svgo patch release dropped is-svg.

Thanks, I just did  npm update postcss-svgo --depth=5 --dev.

Hey, any idea when this will be addressed?

Hey, any idea when this will be addressed or is there a workaround for now?

@cmacdonnacha This was fixed by a bugfix release from postcss-svgo. Bugfix releases can be automatically picked up by your application without a corresponding release from react-scripts. You can either delete and rebuild your package-lock.json to pick it up, or as suggested by @jiridanek, run npm update postcss-svgo --depth=5 --dev.

Excellent thanks @nj314. Will give that a go.

not sure why, but this npm update postcss-svgo --depth=5 --dev is not doing anything for me. 🤔

A new vulnerability has been found on postcss too.
Remediation
Upgrade postcss to version 8.2.10 or later. For example:

"dependencies": {
  "postcss": ">=8.2.10"
}
or…
"devDependencies": {
  "postcss": ">=8.2.10"
}

react-scripts: 4.0.3 has "resolve-url-loader": "^3.1.2" dependency, which resolves to 3.1.3.
resolve-url-loader: 3.1.2 lists a dependency: "postcss": "7.0.21" which is a fixed version dependency, so I am unable to get rid of postcss security issue even if i add "postcss": ">=8.2.10" in my package.json.
Sounds like react-script needs some updates to fix all these vulnerabilities.

Still an issue.

There is, and has not been, an actual vulnerability here.
See https://github.com/facebook/create-react-app/issues/11174.


[csrng/rtl] added tracking sm fields to reg for debug

The csrng command tracking state machine values have now been attached to an
observation register.
Signed-off-by: Mark Branstad<EMAIL_ADDRESS>

is this ready for a merge?


clean up inspect representation for confusing types

some types, when output, might be confused for things they aren't, like binaries and maps.  These should be prepended with _ ::  when that happens.

note that literals like :atom, 47, or [] don't need this because if you copy/paste them into the REPL they are okay.
we might want:

lists of things
maps
bitstrings

completed a7234c0


Components indexing an array model lose sync

If the model state is an array, and a grid of Control components each index one cell, such as in the following example ...
<Control.text
  model={ `.grid[${ rowIndex }][${ colIndex }]` }
  className="form-control code"
/>

... the state is synced perfectly as long as there is no change in array dimensions. If the array is mutated (e.g. by adding an additional row), the mechanism breaks down.
A full minimal working example, as a project based on create-react-app is available at https://github.com/FelixHenninger/rrf-bug-demo/tree/grid-demo (please note the branch!)
It might very well be that I have introduced a mistake, so I'm hesitant to open an issue here. Nonetheless, I would very much appreciate any hints, and am glad to assist with debugging. Thank you very much!

Have you tried using the getDispatch prop to get the internal dispatch for the <LocalForm>'s store, in order to dispatch an actions.change with the new data?

Thanks a lot for this suggestion! (and a very happy new year!) Indeed I hadn't tried that, but I've updated the online example so that the internal dispatch can be tested easily and it still doesn't seem to work.
Poking around in the form internals a bit, I noticed that the connected Form component in React's component tree looks unexpected after updating througth the form dispatch method: The changed values are stored in an entry with an empty key. Might this be part of the solution?

Thanks a lot for this suggestion! (and a very happy new year!) Indeed I hadn't tried that, but I've updated the online example so that the internal dispatch can be tested easily and it still doesn't seem to work.
Poking around in the form internals a bit, I noticed that the connected Form component in React's component tree looks unexpected after updating througth the form dispatch method: The changed values are stored in an entry with an empty key. Might this be part of the solution?

Ah! I figured it out: By prepending 'local' to every model id, the changes get propagated correctly. So if I use the following action/dispatch combination
this.formDispatch(actions.change('local.title', 'I just changed via the form\'s dispatch method!')

Instead of just targeting .title, everything works perfectly. This is fantastic, thanks for pointing me in the correct direction! I'm unsure whether this is intended behavior, though?

Ah! I figured it out: By prepending 'local' to every model id, the changes get propagated correctly. So if I use the following action/dispatch combination
this.formDispatch(actions.change('local.title', 'I just changed via the form\'s dispatch method!')

Instead of just targeting .title, everything works perfectly. This is fantastic, thanks for pointing me in the correct direction! I'm unsure whether this is intended behavior, though?

That is the intended behavior - I'll make the docs clearer on this!

That is the intended behavior - I'll make the docs clearer on this!

Awesome, thank you again for your excellent work on the library! I'd offer to help with the docs, but I'm not sure I know the code well enough. Hope that day will come, though!
(closing the issue for now, please feel free to reopen if, for example, you'd like to track the doc changes)


Resize large color image to improve rendering time

Please review this request.

It will make #7041 worse

@yurydelendik that's weird because this PR's code path is not used when it tries to render #7041 https://github.com/mozilla/pdf.js/files/150290/default.0002.pdf

@Hengjie: I don't think @yurydelendik has tried this, but rather refers to the weak scaling algorithm that is used (see discussion in #7047).


Http Spans are not linked / does not set parent span

Hello,
I'm manually implementing tracing within my NodeJs server and I'm running into an issue with the span traces are not linking or setting the parent span.
i.e
let requestHeaders = request.headers["x-request-id"]; // Passes traceId within the request headers to another server that also has tracing implemented

let parentSpan = {
    traceId: requestHeaders,
    spanId: prevSpan?.context().spanId,

  };
  span = tracer.startSpan("testSpan ", { parentSpan});

In Jaeger I know that all spans needs to share the same traceID so I'm trying to start spans with a the traceId that is being passed through services.

[ ] This only affects the JavaScript OpenTelemetry library
[ ] This may affect other libraries, but I would like to get opinions here first

I've been trying to figure this out, and I'm still not sure really. But I do believe that your current approach is flawed in that startSpan doesn't take the options you are passing it, there is no parentSpan value so far as I can tell.
I think you will need to set a new Context in order to establish a parent. I can't find the documentation for how to properly create a context object, however. The tracer.startSpan() method does take a context object as the third optional arg. Without the context arg being passed, it will pull the current global context and use that to determine if there is a parent set or if the current span is the root.

I've seen a lot discussion on how to set parent spans but I haven't came across a solution to set. In some examples, I've seen they just pass the parent span into the creation of the new span.
i.e
const span = tracer.startSpan('doWork', {
    parent, attributes: { attribute1 : 'value1' }
  });

But this doesn't seem to set the parent span, for the context object it doesn't accepts a Span Context for this so I'm not sure passing the parentSpan will work.
In the documentation it says passing a link will link spans together but I didn't have any luck with this ( Links ). This is my implementation

The API how to specify a parent has changed over time. In the past it was possible via an explicit parentSpan property in options but this was removed. Now it works only via context (third argument of startSpan).
See https://github.com/open-telemetry/opentelemetry-js/blob/611a30f19145e7294efdc587efc863410c900604/examples/basic-tracer-node/index.js#L48-L49 for a sample.

I see, I gave this a try and it sets the parent span, if I want to link spans from one service to another how can I do that manually?
I am sending the traceId from one Node Server to the other but how do I go about linking it to the spans so in Jaeger they show up.
Do I need to spend the whole span context then assign the parent of the span in the other server to the passed span context?

Propagation between processes is done via PropagationApi.
On client side you call inject() to inject a context into some object which is sent along with the actual message. On server side you call extract() which returns a context you then pass to startSpan().
see for example the http instrumentation client and server side.

I'm using the HTTP instrumentation for both of my servers so shouldn't it be doing that already in that case?
I'm creating a package that is using opentelementry for tracing and other services that can be installed on my other servers. So in the package, it's using HTTP instrumentation that will get each request and trace it.
For each incoming request, it would get the request start a span so since it's using the HTTP instrumentation should it be handling Propagation within it?

Yes, if you install and initialize the HTTP instrumentation spans should be created on server and client side and they should be linked.
At least as long as the ContextManager is able to propagate the context through the async operations you use in your code between start of incoming request and the outgoing request.
see the http sample which shows linked http spans.

I think I'm missing something. My spans are still not showing up together or propagating the span id.
 const tracer = require("./JaegerExporter")(options);

 server.ext("onRequest", (request, h) => {
   if (!prevSpan) {
      prevSpan = recordRequest(tracer, request, h);
   } else {
      span = recordRequest(tracer, request, h, prevSpan);
      prevSpan = span;
   }

span = recordRequest(tracer, request, h, prevSpan);

recordRequest Method

 const ctx = opentelemetry.setSpan(opentelemetry.context.active(), prevSpan);

 const headers = propagation.extract(ROOT_CONTEXT, request.headers)

 span = tracer.startSpan("testSpan", {
    kind: 1, // Server
    links: [
      requestHeaders,
      {
        traceId: prevSpan?.context().traceId,
        spanId: prevSpan?.context().spanId,
        traceFlags: 1,
        traceState: undefined,
      },
    ],
  }, headers);

   return span;
}

This should get each request and extract the context from the header to the incoming requests.
Does anything look off?

@djknowles21 Not certain it will help you, but here is an adapted example meant to demonstrate auto propagation w/ Jaeger and the http auto instrumentation. Service A and B are run in their own processes, as is the client.
It's meant to be a proof of concept that shows that propagation happens automatically when all services are using the http instrumentation library.
Here is the trace as it shows up in Jaeger:

I know this doesn't answer your question about manual instrumentation, but I hope it is a sanity check to show that out of the box http propagation is working with Jaeger.

I would advise to use a NonRecordingSpan that wraps your span context, there is a method for this in the API: https://github.com/open-telemetry/opentelemetry-js-api/blob/e1745d19b9bdfd1af7d805a6b2c32a5a5dbf4d15/src/trace/context-utils.ts#L63
import { trace, context } from '@opentelemetry/api'

const handler = (request) {
   // you'll need to have a propagation component implemented that pickup the trace id from the headers
   const ctx = propagation.extract(ROOT_CONTEXT, request.headers)
   const spanContext = trace.getSpanContext(ctx)
   return context.with(setSpanContext(context.active(), spanContext), () => {
     // continue whatever you want to do,  new span should get the correct remote span
   })
}

Thank you guys for your recommendations, I'm trying to figure them out now.
@jonchurch  Thank you, it gives me a bit of sanity check. For the Http Instrumentation examples, they are making the http call from one server to the other within the implementations themselves. With my package, I won't be making the http calls within it. I'm trying to implement it in a way that when each request/response comes to any server using the package it will come and trace every request made in the server.
I've set the parent span for the traces but then in Jaeger not all the traces come up.  i.e, I make a call and 10 request are made in the one call, in my console all 10 show up and have the same parent span id but only 3 show in Jaeger. Then if I make the request again the same 3 show up under the same parent span.

@djknowles21 hello, any updates on this? I'm having similar troubles

Guys. I had the same issue. But I working with golang. And I added this code in place where I initilize trace provider
otel.SetTextMapPropagator(propagation.NewCompositeTextMapPropagator(propagation.TraceContext{}, propagation.Baggage{}))  and worked perfectly for me. I sure what simillar can be made in js code

I'm using auto instrumenting pkg, the auto instruction already creating span and link the child span with parents.
Then I myself try to create some child span but still not find any way to link them with the span already available of the context.
    const data = tracer.startActiveSpan('AvroSerialize', span => {
        const data =  DoYourStuff();
        span.end();
        return data;
    });

Most of the example said you're simply doing stuff like this but then I just end up with an "root level" span.

Any update on this?
I am also trying to figure out how we can set parent span manually b/w services


feat: util/log support log to stderr and add log-to-stderr flag for kusion compile

fix: https://github.com/KusionStack/kusion/issues/75

Coverage decreased (-0.2%) to 73.088% when pulling 8bf82eb552eced83a6ad96853b7c8a24d20be475 on howieyuen:silent-log into 28e6f7162fd66e028e51f4f5033eaf86d3f057d2 on KusionStack:main.

/lgtm


docs(guide/Conceptual Overview): $http.success is deprecated

Using the standard then method instead success, because success is deprecated

I signed it!


Move observers into one service

Rather than implementing our an pattern for each service/component using the $q library, which is a bit of a hack, we should have an observer/subscription service which you can subscribe to and broadcast events.
subscription-service.js
subscriptionService.subscribe  = function(eventName, callback) {
// register callback with event
}

subscriptionService.broadcast = function(eventName, args) {
 // invoke all callbacks with args
}

some-controller.js
subscriptions.subscribe('selectedElementUpdate', function(selectedElements) {
 // update view
}

some-service.js
// update selectedElements
subscriptions.broadcast('selectedElementUpdate', [selectedElements]);

Merged into develop.


All links color turns to grey inside the tabs?

Note the Login URL text and the Edit button inside the tab content:

https://github.com/cristijora/vue-tabs/blob/master/themes/vue-tabs.css#L4-L6
.vue-tabs a {
  text-decoration: none;
color: gray; }

I feel that those lines should not be there...
It looks better too without it tbh:

(Sorry for the red box censored texts btw, but I hope you get the point.)

Fixed
PR


New Website Logo: beget.com

Organization name: Beget
Homepage url: https://beget.com
Logo url: https://beget.com/img/logo.png
How you are using Rust: as part of backup and resource management systems
Url describing Rust usage: -
Organization contact<EMAIL_ADDRESS>

You're logo is up on the page. Thanks, and have fun with your Rusting!


Fix #1: add horse names in database

attention there wil be ) missing

/tina - can you have a look at this?

/tina - done.


Feature/2633

Related issues
#2633
Short description and why it's useful
Updated vue-offline with babel
Upgrade Notes and Changelog

[x] No upgrade steps required (100% backward compatibility)
[x] I've updated the Upgrade notes and Changelog on how to port existing VS sites with this new feature

Contribution and currently important rules acceptance

[x] I read and followed contribution rules
[x] I read the TypeScript Action Plan and adjusted my PR according to it
[x] I read about Vue Storefront Modules and I am aware that every new feature should be a module

I've one issue with Reason: ReferenceError: window is not defined. Please don't marge it before @filrak approval

@bartdominiak check with 2.0.8

@bartdominiak could you check it with onlineHelper?
import { onlineHelper } from '@vue-storefront/core/helpers'
and then onlineHelper.isOnline

@bartdominiak are you willing to get back to this PR?

Closing due to long inactivity

Sorry for my inactivity. I'll be back to this PR ASAP.


Update clients.md

added in a new R library that is on CRAN

thanks, @zacdav!
Can we make sure to maintain the list in alphabetical order? I'll merge once that's taken care of.

Should be fixed @mlc.
Thanks


About TLV encoding object instance When send lwm2m put-write request

I have problem, ask for help. Thanks!
Scenario as follows, when I want to modify the value of lifetime (path: /1/1/1) and minimum period( path:/1/1/2)，will send a put-write request, the uri path of request is /1/1, and payload data is a object instance, and the content-format is set to TLV. And find that type of indentifier are both 0b11.
I think it is caused by the following handler function。
/leshan-core/src/main/java/org/eclipse/leshan/core/node/codec/tlv/LwM2mNodeTlvEncoder.java
        public void visit(LwM2mObjectInstance instance) {
            LOG.trace("Encoding object instance {} into TLV", instance);

            Tlv[] tlvs;
            if (path.isObjectInstance() || instance.getId() == LwM2mObjectInstance.UNDEFINED) {
                // the instanceId is part of the request path or is undefined
                // so the instance TLV layer is not needed.
                // encoded as an array of resource TLVs
                tlvs = encodeResources(instance.getResources().values(),
                        new LwM2mPath(path.getObjectId(), instance.getId()));
            } else {
                // encoded as an instance TLV
                Tlv[] resources = encodeResources(instance.getResources().values(),
                        new LwM2mPath(path.getObjectId(), instance.getId()));
                tlvs = new Tlv[] { new Tlv(TlvType.OBJECT_INSTANCE, resources, null, instance.getId()) };
            }

            try {
                out.write(TlvEncoder.encode(tlvs).array());
            } catch (IOException e) {
                throw new RuntimeException(e);
            }
        }

But this is described about write request on page 50 of the protocol document (OMA-TS-LightweightM2M_Core-V1_1-20180710-A.pdf)
If no Resource ID is indicated, the Resource Instance ID MUST NOT be indicated , then the value included payload is an Object Instance containing the Resource values.
I think this approach conflicts with the document ，but I don't know why.

I see you reference the v1.1 version, keep in mind that Leshan currently aim the v1.0.x.
(see https://github.com/eclipse/leshan/issues/563 for more details about Leshan and v1.1)
Anyway, there is maybe an ambiguity in the specification. (And the ambiguity seems to concern both version of the specification), but for me Leshan is behavior is OK.
Look at this to get more details : https://github.com/OpenMobileAlliance/OMA_LwM2M_for_Developers/issues/452

@sbernard31
Thank you for your patient reply.

OpenMobileAlliance/OMA_LwM2M_for_Developers#452 confirms that Leshan Behavior sounds OK.
So I close this issue.


Update project.json to newer versions

Update project.json to newer versions (1.0.0 -> 1.1.0)
Cleared migrations folder since they depend on the chosen EntityFramework provider
Added content root configuration (didn't work without it on MacOS, VS Code)

Great! Thanks a lot for this!
Any changes you can add these changes also to the empty-template and empty-template-typscript? They don't use database stuff, just references.


Fix (Next Theme): Prefer light text on filled primary buttons

Description

Update the chooseLightOrDarkText() fn (only in next theme) to prefer light text as long as the contrast ratio is >3.5
Update the Button OUI docs page to provide all the color variants (except ghost) in the initial example

Note that, because primary is the same color for next dark and light, it now always has the light text.
The current contrast ratio between primary and ghost in the next theme is 3.33.

Issues Resolved

Fixes https://github.com/opensearch-project/oui/issues/935
Check List

[ ] New functionality includes testing.
[ ] New functionality has been documented.
[ ] All tests pass

[ ] yarn lint
[ ] yarn test-unit

[ ] Update CHANGELOG.md
[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

"Note that, because primary is the same color for next dark and light, it now always has the light text.
The current contrast ratio between primary and ghost in the next theme is 3.33."

What is Ghost set to at this point? Is it# FCFEFF?
Based on color alone, I get 3.33. But, based on font weight & color, I get a passing contrast ratio (would need to also test the final version in browser once this is merged). Does the rating not take font weight and size into consideration?
I dont love that dark mode now also has the same light text on the button. This doesnt seem to make much sense to me, and causes visual noise (higher contrast buttons than others).

What is Ghost set to at this point?

Next light and dark: #FCFEFF

Based on color alone, I get 3.33.

Yeah, it's just the pure color contrast ratio. It's not a rating, it just gets factored in along with other factors to various a11y ratings.

I dont love that dark mode now also has the same light text on the button. This doesnt seem to make much sense to me, and causes visual noise (higher contrast buttons than others).

All the other dark mode colors are noticeably lighter than the light mode equivalents, except for primary in the next theme. Do you want a variation of primary between next light and next dark, or would you like us to make the text calculation button use the underlying theme to preferentially choose dark or light text?

I am a little confused -  In https://github.com/opensearch-project/oui/issues/935#issuecomment-1681220338 - you mentioned primary wasn't being transformed for the buttons in "current" dark mode - but I am seeing what you are seeing in "Current" theme, that they are lighter in dark mode ("All the other dark mode colors are noticeably lighter than the light mode equivalents"). I think we should follow the same convention, although I had initially provided that primary color becuase it was stable across light and dark modes while providing sufficient contrast for buttons with light text at the cont weight/size specified.

I believe this PR should be closed in favor of https://github.com/opensearch-project/oui/pull/981


Monaca Preview has errors

Bug description
Build in Monaca Preview does not work. The following error is encountered:
Bug screenshot:

According to Google, this error is caused because of the Node.js version, but I don't know how to change it in Monaca preview.
Resources:
https://stackoverflow.com/questions/69665780/error-err-unsupported-esm-url-scheme-only-file-and-data-urls-are-supported-by
https://github.com/vercel/next.js/discussions/29080
https://www.angularfix.com/2022/01/error-errunsupportedesmurlscheme-only.html
Correct behaviour
This error should not appear and Monaca Preview should build the app without errors.
How to reproduce
Upload the project to Monaca Cloud and check the Preview.

@pablomatgar i confirmed it is working fine on Monaca Cloud IDE if we upgrade Node version to 16. Since we are planning to release Cordova 11 support (Node 16) this week, we don't have to fix this issue. Let's do following

upgrade this project to Cordova 11
wait to release this Github/blog after Monaca release Cordova 11 support (Node 16)

Okay, let's do it then


Add New in shortcut keys tab and one other misc

Type of installation
Setup (.exe)
Type of reader (if reader related)
None
Steps to reproduce
Open Yomikiru settings and go to shortcut keys tab.
1. 'Add New' is not a button type and is not linked to button values in theme maker.

The change in mouse when hovered over a button is not replicated on 'Add New' button.
And the text color of it is darker than other button colors.
Example - 'x' button in shortcut keys is linked to all button values in theme maker and change in mouse can be seen on it.

https://github.com/mienaiyami/yomikiru/assets/127277950/0879013b-ad83-4384-8f31-f8a04c18794f

2. Assigned shortcut key highlight.

Disable the highlighting of assigned keys.
It doesn't do anything, so highlighting is unnecessary.

https://github.com/mienaiyami/yomikiru/assets/127277950/8f3a4ae4-e4c0-4dac-9952-ab35986b32eb
✔ Expected Behavior

'Add New' should be a button type and should be linked to the button values in theme maker.
Assigned shortcut keys should not be highlighted.

❌ Actual Behavior

'Add New' is not a button type and is not linked to the button values in theme maker.
Assigned shortcut keys can be highlighted.

This is because they are "text inputs" and not buttons, previous version was also "text inputs". Using buttons here is not a great idea because buttons are meant to do some action on click, and there is none here. Here "Add new" is capturing many key clicks so its not a button.
As for the highlight, its not possible to remove it without disabling the whole element's mouse capture.

It is the input placeholder color, which you might also see in search inputs.
I will see if there is a better way.

Tried using button for "add new" and replacing it with input on click, but results in many unwanted behaviors. So for now I will keep it same.


feat: ✨ add prefix/suffix slots to the tab-group component

Pull Request
📖 Description

Adds two new slots to the tab-group, prefix and suffix.
This implementation is good enough for what I'd need it for, but there's probably room for improvement. :)
🎫 Issues

No relevant issues exist
👩💻 Reviewer Notes

Do you think it makes more sense for the prefix/suffix to be inside or outside the scrollable area?
Do you think it makes more sense for the prefix/suffix to also have the border-bottom that the tabs do or not?
📑 Test Plan

✅ DoD

[x] I have tested my changes manually.
[ ] I have added automatic tests for my changes (unit- and visual regression tests).
[x] I have updated the project documentation to reflect my changes (e.g. CHANGELOG, Storybook, ...).
[ ] I have added documentation to the DaVinci migration guide (if need be)
[x] I have made sure to follow the projects coding and contribution guides.
[ ] I have made sure that the bundle size has changed appropriately.
[ ] I have validated that there are no accessibility errors.
[x] I have used design tokens instead of fix css values

Thanks @xkabylgSICKAG for your valuable effort – highly appreciated, that you took the time for this feature!

While we understand, that this could help you in your specific case, we currently have big hesitation to introduce that especially from a design perspective, as this opens up the door for  patterns, that make it harder to identify the tabs as... tabs. As we're enforcing an unified approach wherever possible, we therefore don't want to introduce this feature in development as well.
In general it would help us a lot, if you would first open up an issue or even a discussion, describing specific use cases, where you see potential for a feature. This helps us to identify, if we should cover a need as a component feature, a template, a CodePen or not at all.
We're working with a so co-called vendorism approach, therefore the change would have to be done in e. g. this file, as otherwise our test pipeline would fail

All in all we would like to close the PR for the moment. Two recommendations:

As already shared internally, this is how you could approach this via CSS: https://codepen.io/mariohamann/pen/jOjYmbp
It could make more sense, to use syn-prio-nav in combination with syn-navigation-item instead of tabs in your cases, see https://synergy-design-system.github.io/?path=/docs/components-syn-prio-nav--docs This makes it already possible to show as many navigation points as possible.


Some comparisons of the LFC results between GI Mapping and gimap

This is an Rmd and its rendered html to compare the LFC results from GI Mapping and gimap. This assumes that the CRISPR_score and lfc_adj columns correspond to one another.

@cansavvy the R-CMD-check for windows-latest was failing because it didn't have the Achilles_common_essentials.csv file, so I added it. It's not failing because it doesn't have the CCLE_expression.csv file. But that file seems to be in the .gitignore. Should I un-add the Achilles file from github and add it to the list in the .gitignore?

@cansavvy the R-CMD-check for windows-latest was failing because it didn't have the Achilles_common_essentials.csv file, so I added it. It's not failing because it doesn't have the CCLE_expression.csv file. But that file seems to be in the .gitignore. Should I un-add the Achilles file from github and add it to the list in the .gitignore?

It's in the gitignore file because the CCLE_expression.csv is too big to be stored on GitHub or included in an R package.

@cansavvy the R-CMD-check for windows-latest was failing because it didn't have the Achilles_common_essentials.csv file, so I added it. It's not failing because it doesn't have the CCLE_expression.csv file. But that file seems to be in the .gitignore. Should I un-add the Achilles file from github and add it to the list in the .gitignore?

It's in the gitignore file because the CCLE_expression.csv is too big to be stored on GitHub or included in an R package.

So should I remove the Achilles_common_essentials.csv file and add it to the .gitignore as well? It's not as large of a file of course.

So should I remove the Achilles_common_essentials.csv file and add it to the .gitignore as well? It's not as large of a file of course.

Yeah I think I included it in there because it wasn't too big but I also like the idea of being consistent. Up to you.

I'm working on comparing the old code and the new code side-by-side for each related chunk....
When I ran this
lfc_df <- dataset %>%
    as.data.frame() %>%
    dplyr::mutate(pg_ids = pg_ids) %>%
    tidyr::pivot_longer(-pg_ids) %>%
    # Adding on metdata
    dplyr::left_join(gimap_dataset$metadata$sample_metadata, by = c("name" = "col_names")) %>%
    dplyr::select(-name) %>%
    tidyr::pivot_wider(values_from = value,
                       names_from = c(timepoints, replicates))

I got this warning message:
Warning messages:
1: Using an external vector in selections was deprecated in tidyselect
1.1.0.
ℹ Please use `all_of()` or `any_of()` instead.
  # Was:
  data %>% select(timepoints)

  # Now:
  data %>% select(all_of(timepoints))

See <https://tidyselect.r-lib.org/reference/faq-external-vector.html>.

2: Using an external vector in selections was deprecated in tidyselect
1.1.0.
ℹ Please use `all_of()` or `any_of()` instead.
  # Was:
  data %>% select(replicates)

  # Now:
  data %>% select(all_of(replicates))

See <https://tidyselect.r-lib.org/reference/faq-external-vector.html>.

I've also figured out that plasmid_df is empty (and is used later). Working on rectifying.

I've also figured out that plasmid_df is empty (and is used later). Working on rectifying.

I'm sorry. This was incorrect. I missed a code block

@cansavvy Is this good to merge now? The R-CMD-check failure appears to be an error on the vignette that's a 403 error during the chunk with a string of steps. Probably the annotate step.

@cansavvy Is this good to merge now? The R-CMD-check failure appears to be an error on the vignette that's a 403 error during the chunk with a string of steps. Probably the annotate step.

The vignette will continue to fail until I update the docker image.
The window failure is a known thing: #48
So yes! We can merge!


Make static keybindings configurable

Make the flexibility of dynamic keybindings available to static
keybindings. You can now customize static keybindings and even bind
multiple keys to one function. This is important especially when keys
such as p are used, which is a default key binding in mpv.

Ah sorry I didn't notice this PR. There was an issue related to this so I made a somewhat similar commit.
In your PR a benefit is allowing binding multiple keys for the non dynamic keybinds as well(I was lazy and skipped this). I decided to remove all the default keybinds for static functions, the only one that I left is the toggle to open playlist.
If you'd like you could rebase and I will merge the PR.

I'm sorry for being late as well, had a somewhat long vacation.
I rebased the commit. Hopefully this should work.


Missing hover effect for menu items

Hi @nickcernis, first of all, thanks for this awesome theme. I've recently started using it and so far I like it. The only issue I've found is that in some menus, (for example the menu to create new files or directories), there is no focus/hover effect to the menu items which makes it hard to know which item you have selected. If you do not have the time to fix it, you can indicate me how to do it as I've never worked on a Jetbrains theme before.

Check the attached screenshot.

It also happens in other menus:

It also happens in other menus:

Thanks for the kind words and the report, @jaumecapdevila. I'll take a look.

@jaumecapdevila I submitted version 0.2.1 to JetBrains with a fix for this. It should be reviewed and available within 48 hours, but usually sooner.

Awesome, thanks for your time @nickcernis 😄

0.2.1 is out now.


File edits do not show up on Windows device after saving on OS X device (in Team folder)

I was testing file edits on a Word doc with someone who has a Macbook, and although I saw the first save, once I started adding to the doc and also saving on my end (on Windows), I no longer saw other subsequent edits from the Macbook.

Example of doc contents:
Hey guys! [saved from Macbook - showed up on Windows]
Testing another comment [saved from Windows]
[waited for subsequent saves from Macbook to show up after closing and reopening doc multiple times on Windows.....]
Comment1 from Macbook [did not show up on Windows]
One more comment [saved from Windows - did not show up on Macbook]
Comment2 from Macbook [did not show up on Windows]
[waited again for subsequent saves to show up but they never did]

There were also no new conflicting files in the team folder. Anyone else experience issues with collaboration on files using team folders? This is super important to us as we'd like to use Keybase for file collaboration on a regular basis.
my log id: 4fe024ce600d02d680bfc01c

If it helps, this is the file on Windows with some greyed out temp files (which I can't seem to open)

@localmisfit can you also please do a keybase log send from the macbook?  I'll look at the logs together soon, likely in the morning (PT).

@strib I'll be able to get that to you by PT afternoon. Is there any personal info from device (MacBook) that is contained within log?

The logs contain file names and file metadata (like sizes and modification times), but none of the actual data from inside the file.

The macbook log would still be useful when you get a chance and are ok with it, but I did find an issue in the Windows log.  It seems like you hit a corner case in our conflict resolution protocol, and Windows gave up trying to resolve the conflict and is now stuck with its current view of the folder.  I'm still investigating the bug, but you can fix it by doing this on the Windows machine:

If there's any data in the team folder that only shows up on the Windows machine that you'd like to save, copy it to a safe location on your local disk.
Quit Keybase completely, so that the K drive unmounts.
Move or rename this directory to something else, say backup-7eab6342d9a0d402: %appdata%/Local\Keybase\kbfs_journal\v1\012007f42bd75bc7b3270d51f35f1b28f131-7eab6342d9a0d402.
Restart Keybase.

Now your Windows machine should be in sync with the latest version of the folder.
I'll continue to look into why conflict resolution failed.  For the searchable record here's the error in the log:
2018-04-11T14:43:27.316740-07:00 - [DEBU kbfs(CR 7eab6342) conflict_resolver.go:3235] 4ef4 Finished conflict resolution: No chain found for BlockPointer{ID: 0108195d87143200f6e0662d777262be5887df415c52af2566110a748e7b797eb4, KeyGen: 2, DataVer: 3, Context: Context{Creator: d34b4cc458ac3a081cd1804f8d9e1d24}, DirectType: direct}

Alright a restart seems to have cleared things up.
Here's log file from Macbook: b064394ed802a56ecfa1021c
Also interesting that I deleted some folders that I created in the team folder, and although they disappeared from the Windows machine, they still showed up on the Macbook.

Also interesting that I deleted some folders that I created in the team folder, and although they disappeared from the Windows machine, they still showed up on the Macbook.

@localmisfit did you delete them from the Windows machine or the macbook?

@localmisfit the logs you just sent are from a brand new device that you just provisioned today -- it couldn't have been involved in any of the tests from yesterday.  Can you do a log send from yesterday's device, or is it lost?

(Or maybe you sent the logs as a user that was just added to the device today -- if it's the same physical macbook, maybe log in as the user you were using yesterday and try keybase log send again?)

@localmisfit did you delete them from the Windows machine or the macbook?

I deleted from the Windows machine.

I deleted from the Windows machine.

That's why -- the Windows machine was stuck in a state where it wasn't sharing any updates with anyone else.  The deletes never made it to our servers, and so your macbooks won't be able to see them.

Let me see if I can get a log send from the Macbook from yesterday.

It's ok if you can't, I'm not sure it's going to give me any more useful info.  There's only some small chance it does.
More interesting will be when you get a chance to run the steps above on your Windows machine and see if that fixes the issues.

Ok let me try the steps above. Didn't see that earlier.

That did the trick! I can see updates to the file and "deleted" folders. Will do some more testing to see if I get any more conflicts.

Ok, so I just tested again (basically entering different data from two different machines - Macbook and Windows and pressing Save simultaneously) and now I'm seeing the conflict file. Looks like whatever the bug was is not showing up now.

Great!  Yeah it must be pretty rare, as I haven't seen one like that lately.  I'll file an internal issue and hope to make some progress on it.

Great! Thanks a bunch. I'll let you know if it happens to come up again.


Compilation performance: use different target directory while compiling through hax

For a few months, we are now enabling the hax feature when compiling through hax.
Enabling a feature at the level of cargo enables the feature for every single dependency: cargo will invalidate all compiled artifacts and rebuild with the new feature set.
The common dev cycle while using hax is:

write some Rust with a LSP server, that yields cargo build calls;
at some point, call cargo hax into ... (or, in the future, automatically call cargo hax through some dedicated lsp);
go back to 1.

This means things are getting recompiled a lot.
This is a well-known issue with other Rust libraries, e.g. when using leptos with server-side rendering (leptos constantly builds first a native binary and then a WASM one for one same crate).
leptos solves that by having a setting (package.metadata.leptos.separate-front-target-dir = true) in Cargo.toml to compile WASM to a different target directory (that different target is actually just a subdir in the current target directory).
Solution
Add a separate-target-dir option to the CLI and Cargo.toml

The current target directory is cargo metadata | jq '.target_directory'
CARGO_TARGET_DIR sets the target directory

Was done in #743


Add path filter for unit tests

Description

This PR adds a path filter to check if any changes to the unit tests code, thus the unit tests will not run if only docs are changed.  The paths to trigger the unit tests include (Please confirm @miguelgfierro @pradnyeshjoshi):

examples/
recommenders/
tests/
setup.py

Related Issues

Checklist:

[X] I have followed the contribution guidelines and code style for this project.
[ ] I have added tests covering my contributions.
[ ] I have updated the documentation accordingly.
[X] This PR is being made to staging branch and not to main branch.

it seems there are some conflicts. Simon one question, how can we check that this is effective? do you need the code to be in staging or main? for example, if you change a markdown now, are the tests not triggered?

I forgot you just merged the PR.  No wonder all unit tests are triggered.

@miguelgfierro I know what's going on.  As said for the pull_request_target event, it will run in the context of the base of the pull request, rather than in the context of the merge commit, as the pull_request event does.
https://github.com/microsoft/recommenders/blob/e16c72ed9d0feb448c399b697dce7eb8cf1fd9c0/.github/workflows/azureml-unit-tests.yml#L8-L21
I think the path filter at the workflow level works:
https://github.com/microsoft/recommenders/blob/4a6754fd28f9b41556bdc882bd448fb4964d3c04/.github/workflows/azureml-unit-tests.yml#L22-L28
And the path filter action at the job level works as well:
https://github.com/microsoft/recommenders/blob/cd9946a4b2d2ee6ea8ffec24f7ab3d423e5e177e/.github/workflows/azureml-unit-tests.yml#L33-L60

@simonzhaoms have you tried this code? do you know if it is not triggered if there is a change in a readme?


[Feature][seatunnel-connectors-v2][connector-kafka] Kafka supports custom schema

Search before asking

[X] I had searched in the feature and found no similar feature requirement.

Description
English:
The original Kafka connector supports custom schema. The current V2 connector does not support custom schema
中文：
原始的Kafka连接器支持自定义模式。当前的V2连接器不支持自定义schema
Usage Scenario
Kafka Source user-defined data structures
Related issues
none
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Yes, this is very import function we should implement. If you can do this will be very helpful

I want to discuss how to define fields schema configuration for connector in seatunnel
I see use case from flink:
create table x ( field1 bigint, field2 string, ...... ) with ( format: 'xxx' )
Connector selectd xxx format factory to convert message data format to  row[fleid1<bigint>,field2<string>].
How should it be defined in seatunnel?
e.g:
source {   KafkaSource {      format = xxx      format_schema = [{"name": "field1", "data_type": "bigint"}  , {"name": "field2", "data_type": "string"}]    } }

Yes, it is a important thing that decide how define schema. Please send a discuss mail in<EMAIL_ADDRESS>And attach your idea.

Yes, this is very import function we should implement. If you can do this will be very helpful

cc @ashulin

https://github.com/apache/incubator-seatunnel/issues/2299

cc @TyrantLucifer

This is a common feature, for example, kafka, HTTP, File, Pulsar, etc;

This is a common feature, for example, kafka, HTTP, File, Pulsar, etc; My opinion is to conform to the habit of SQL;
# conf file
Source {
    schema {
        field = "STRING"
        field2 = "INT"
        field3 = "DECIMAL(30, 3)"
    }
}

I think it's a good idea that add a common feature in connector to support user-defined schema. In my option, I think we can add a new config option schema in all source connector source configs and for each connector they can parse their own schema in getProducedType method. The implement function of parsing schema we can add it in module seatunnel-common. And I agree with your advice @ashulin to conform schema type to the habit of SQL the same as defined in code.

Another problem is missing metadata of mq connector rowdata.
e.g:  partition、timestamp、key、headers
public ProducerRecord(String topic, Integer partition, Long timestamp, K key, V value, Iterable<Header> headers)
I see use case from flink:
https://github.com/apache/flink/blob/master/flink-connectors/flink-connector-kafka/src/main/java/org/apache/flink/streaming/connectors/kafka/table/DynamicKafkaRecordSerializationSchema.java#L119

This is a common feature, for example, kafka, HTTP, File, Pulsar, etc; My opinion is to conform to the habit of SQL;
# conf file
Source {
    schema {
        field = "STRING"
        field2 = "INT"
        field3 = "DECIMAL(30, 3)"
    }
}

I think it's a good idea that add a common feature in connector to support user-defined schema. In my option, I think we can add a new config option schema in all source connector source configs and for each connector they can parse their own schema in getProducedType method. The implement function of parsing schema we can add it in module seatunnel-common. And I agree with your advice @ashulin to conform schema type to the habit of SQL the same as defined in code.

By the way, the row data format information also should defined in source connector. The final source config as the following:
Source {
   Kafka {
     shema {
       fields {
          field = "STRING"
          field2 = "INT"
          field3 = "DECIMAL(30, 3)"
       }
        format = "json" // format = "text"
     }
  }
}

In most use cases, I think we can support json or text, if we assign text, we also assign delimiter in config file.

Where should the TXT type column delimiter be defined here

This feature is in development, please waiting pr.

This feature is in development, please waiting pr.

@eyys Please refer to #2439 #2436, look forward to your contribution.


create faith-kovi.md

Notes for Reviewers
This PR fixes #
Signed commits

[x] Yes, I signed my commits.

🚀 Preview for commit 49d811159169bafa22d0bba046a4cdfb9b231338 at: https://66fea061d9202ca313db053d--meshery-docs-preview.netlify.app

Thank you @FaithKovi 😎


Issue with Hackers keyboard

It seems, that the newest version, 1.6.0 has problems with Hacker's Keyboard. Tested on Android 12 and 13, Samsung Galaxy S22 ultra and Samsung Galaxy A70.
The arrow keys do not work (the cursor cannot be moved).

Clear app data to fix it.
On Wed, Jul 27, 2022, 2:41 AM icefront @.***> wrote:

It seems, that the newest version, 1.6.0 has problems with Hacker's
Keyboard. Tested on Android 12 and 13, Samsung Galaxy S22 ultra and Samsung
Galaxy A70.
The arrow keys do not work (the cursor cannot be moved).
—
Reply to this email directly, view it on GitHub
https://github.com/deadlyjack/Acode/issues/461, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJHIHI4J7TMZ2ZUQYLTRB5TVWBH6LANCNFSM54XLBGQA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

There are countless FTP sites configured for this app. Clearing the data will delete all connection configurations. There should be a way to save these configurations, resetting the app is always the most destructive way to fix things...

Go to settings, backup, after resetting app restore settings.
On Wed, Jul 27, 2022, 1:18 PM icefront @.***> wrote:

There are countless FTP sites configured for this app. Clearing the data
will delete all connection configurations. There should be a way to save
these configurations, resetting the app is always the most destructive way
to fix things...
—
Reply to this email directly, view it on GitHub
https://github.com/deadlyjack/Acode/issues/461#issuecomment-1196380800,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJHIHI64LVZSGHPIKG5VIWTVWDSWJANCNFSM54XLBGQA
.
You are receiving this because you modified the open/close state.Message
ID: @.***>


Ran dartfmt, added linting via analysis_options, updated travis.sh

Let me know if you have any questions, a quick summary of changes:

Added a analysis_options.yaml file, which configures dartanalyzer
Added strong mode, the more strict variant of Dart which has a promise of better future performance characteristics and compiler optimizations.
Made small casting changes to the code base to resolve all analyzer errors
Added a nice set of default lints without anything too strict
Ran dartfmt, the automatic source-code formatter, and enforced in travis.sh

I also moved the timeout logic to discord.dart so contributors that don't have timeout installed still can run the test cases.

Just reopen this into indev, as all new commits go there. :)


What's the parameter of run and why this can influence the result?

(sort Node)
(function mk (i64) Node)
(relation edge (Node Node))
(relation path (Node Node))
(rule ((edge x y))
((path x y)))
(rule ((path x y) (edge y z))
((path x z)))
(edge (mk 1) (mk 2))
(edge (mk 2) (mk 3))
(edge (mk 5) (mk 6))
(union (mk 3) (mk 5))
(run)
(check (edge (mk 3) (mk 6)))
(check (path (mk 1) (mk 6)))

when I was running this example in your paper, I need to set the number of run, I see in your example egglog/tests
/path-union.egg that you set this number to 2.
when I set this number to 2, the last check will be fail.
I don't know why? and where can I get the tutorial about why this situation.

Ok, thank you


How/Can I use this with jquery masonry?

I was wondering can this be integrated with jquery masonry so that when more items are loaded the masonry is reapplied.
I know jquery masonry has append but not sure how i'd integrate it so that it's triggered when the new items are loaded.
Thanks any direction is appreciated.

This extension merely helps you register javascript code to initialize plugin. You can register masonry behavior by configuring behavior to be InfiniteScrollPager::BEHAVIOR_TWITTER (a similar example is included in the documentation).
I have never used it with masonry, so cannot answer your question. You should raise all plugin-related questions directly in infinite-scroll repository.

Okay thanks for pointing me in the right direction, but how do I add the custom call back if the javascript is added via the extension?
For example I have found the following link
$container.infinitescroll({
 navSelector  : '#paging',
nextSelector : '#paging a',
itemSelector : '.col',
behavior: 'twitter',
loading: {
  msgText: 'Loading...',
      finishedMsg: 'Loaded all!'
    }
  },
  // call Isotope as a callback
  function( newElements ) {
    $container.isotope( 'appended', $( newElements ) );
  }
);

It is described in the readme - set contentLoadedCallback property


web3js: Bump sendSignedTransaction from web3-eth

This PR re-implements web3.eth.sendSignedTransaction

based on the web3.js<EMAIL_ADDRESS>which includes a large refactoring (https://github.com/web3/web3.js/pull/6410)

Added new code in src/send_signed_tx/*.ts

Copied from web3-eth https://github.com/web3/web3.js/tree/v4.3.0/packages/web3-eth/src
See diff https://github.com/blukat29/web3klaytn/compare/085092dac5a2e1f6fbee090749e08e11b7a8d479...blukat29:web3klaytn:web3js/rework-sendsignedtx
I only changed a little bit from the original web3-eth code.

Removed old code in src/send_transaction.ts

Continued in #572.


gives error for newer version of k8s

What changes were proposed in this pull request?
for newer versions of k8s produces a error
Error: unable to build kubernetes objects from release manifest: unable to recognize "": no matches for kind "PodDisruptionBudget" in version "policy/v1beta1"
How was this patch tested?
tested it on latest versions

integration step is failing - https://github.com/confluentinc/cp-helm-charts/pull/651/checks?check_run_id=14803793948


luci doesn't follow odhcpd's leasefile option

If I change odhcpd's leasefile option to a different file, for example:
config odhcpd 'odhcpd'
	option maindhcp '0'
	option leasefile '/var-ext/hosts/odhcpd'
	option leasetrigger '/usr/sbin/odhcpd-update'

(in my case, /var-ext is non-volatile storage so as to keep DHCP leases in-between reboots)
odhcpd uses the file pointed to by the above configuration, but luci still monitors the original, hardwired file at /tmp/hosts/odhcpd
I believe the culprit is at line 51 of modules/luci-base/luasrc/tools/status.lua
local fd = io.open("/tmp/hosts/odhcpd", "r")
The equivalent code for dnsmasq (from line 13) seems to read the config file for the lease file, whereas odhcpd simply assumes it is using the one in /tmp/hosts.

This should do the trick:
--- /usr/lib/lua/luci/tools/status.lua
+++ /usr/lib/lua/luci/tools/status.lua
@@ -48,7 +48,17 @@
                fd:close()
        end

-       local fd = io.open("/tmp/hosts/odhcpd", "r")
+       local lease6file = "/tmp/hosts/odhcpd"
+
+       uci:foreach("dhcp", "odhcpd",
+               function(t)
+                       if t.leasefile and nfs.access(t.leasefile) then
+                               lease6file = t.leasefile
+                               return false
+                       end
+               end)
+
+       local fd = io.open(lease6file, "r")
        if fd then
                while true do
                        local ln = fd:read("*l")

That seems to work for me and reads the leases also from a non-standard file.


Simplifies block ridding of one if-else branch met only once.

Changes in this pull request
Simplifies block ridding of one if-else branch met only once.
Solves issue #68.
Pull request checklist

[*] All tests pass. Demo project builds and runs.
[ ] I added tests, an experiment, or detailed why my change isn't tested.
[*] I have reviewed the contributing guide

Awesome! I just put up a diff internally to do the same fix but didn't remove the last param from the block. This is 👌 great stuff!

That block was too dynamic even for Objective-C.


Point to the official conan recipe repo

The original gregmedd/up-conan-recipes repo has been deprecated.

Forgive me: this is a minor change that mirrors changes in up-cpp. If it isn't fixed, it will cause problems for other PRs. I'm going to go forward with merging it without a review


Fix DeepSeekV2Tokenizer bug that padding_side is not set to right

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.同润 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


MudTable: The sortable headers don't sort anymore after a row click

Bug type
Component
Component name
MudTable
What happened?
The MudTable sortable headers don't sort anymore after a row click.
Expected behavior
When enabled, the MudTable sortable headers should always work.
Reproduction link
This is from the docs:
https://try.mudblazor.com/snippet/camvFOFAUiStuWPN
Reproduction steps

Click a couple of times on any header, any column: Sorting works;
Click any row;
Click a couple of times on any header, any column: Sorting does not work;

Relevant log output
No response
Version (bug)
5.2.0-rc2
Version (working)
No response
What browsers are you seeing the problem on?
Microsoft Edge
On what operating system are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Closing due to age of issue and example not building.


Show a spinner while node details are loading

I am getting the spinner but the details pane color is not matching the node's color. Wasn't that fixed?

Also, as you show in your screenshot, the details panel is grey while showing the spinner. Why aren't you showing the final color? Is there a technical limitation preventing you from doing that?

The color matching was done in a concurrent PR so it's broken in this one, but was now merged to master in #689 and #690.
But already coloring the panel in the final color is a good idea. will do that.

cool

@2opremio PTAL.

@davkal LGTM, however, I've noticed that if you click on nodes while they are in the background, their detail requests end up somehow queuing (?) and displaying when you are no longer interested.

You are right, when clicked, the details panel should go back to Loading mode.

Will test this tomorrow. Why are tests failing BTW?

Edge test ...

yeah, unrelated.

It still suffers from the problem I mentioned, this video makes it more clear (wait until the end):

Ah now I know what u mean. Will see how/if this can be fixed.

Much better now, although the details pane takes longer to load (it could be that my system is slower today due to the number of containers) and the Internet node for some reason is brown while it loads (although it's final color is purple):

LGTM


User Engagement in Service Delivery blog

@gabrielcossette @gcharest the length of the card is too long on the webpage for this blog, can one of you help me to fix it please? Thanks

Oh, yes! Absolutely, I think it's due to the fact we haven't identified any "excerpt" part. What would you like to display on the card? The first two sentences only?

@gcharest Yes please, first 2-3 sentences as long as the card with this blog is the same length as others. Thank you

Making a few changes to adjust the blog card!

Thanks for fixing this Guillaume, I forgot about it! 🙂

FR translation review completed 🙂


Inconsistent result in browsers, with "inline-flex" and variable content

I don't really think that issues should be just a "redirecting machine" to questions on Stack Overflow, but I have described the issue there and gave an example to play with, and, since it's already described there I thought it's better not to copy-paste it here...
Basically, Internet Explorer 11 is not expanding "inline-flex" containers when the width of the content dynamically expands (through JS). Running the example in the latest versions of Chrome, Firefox and Opera seams to work. If someone could test on Safari, please let me know.
Here's the Stack Overflow link: http://stackoverflow.com/questions/42510904/inline-flex-item-does-not-grow-with-its-content-in-internet-explorer
Which implementation is correct? The one in IE 11, or the one in the other browsers?

Thanks for reporting. Can you please update this issue to include all the information outlined in the reporting guidelines. If the issue is no longer relevant, please close it.


Rewrite recipe for GSTools 1.3

Since GSTools 1.3 will control the parallel build with an environment variable, we need to take care of this when it is released

https://docs.conda.io/projects/conda-build/en/latest/resources/define-metadata.html#use-environment-variables

Also we need to update the build command, to not create build dependencies now defined in pyproject.toml.

https://pip.pypa.io/en/stable/reference/pip/#pep-517-and-518-support
https://pip.pypa.io/en/stable/reference/pip_install/#cmdoption-no-build-isolation

Recipe could be updated as follows:
build:
  number: 0
  script_env:
   - GSTOOLS_BUILD_PARALLEL=1
  script:
    - {{ PYTHON }} -m pip install . --no-deps --no-build-isolation -vv

See: https://github.com/GeoStat-Framework/GSTools/pull/154

Done.


Docs: clarify how to set table location for trino on aws glue

Interesting, there's some overlap with Athena: https://github.com/TobikoData/sqlmesh/pull/3154/files#diff-810f5d6dd3e7fda3367cac3598147973d1db787a6026543dc9461dc9ac2f4872R29