Adding LICENSE file

Hello
On Wed, Mar 9, 2022, 10:39 PM Eldar Gabdullin @.***>
wrote:

Merged #9 https://github.com/subsquid/squid-template/pull/9 into main.
—
Reply to this email directly, view it on GitHub
https://github.com/subsquid/squid-template/pull/9#event-6212189743, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AXZF2VPCTTQGGAJZSKYTEK3U7DV4ZANCNFSM5QKLZROQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


chore(more logs): add more logs to indicate more detailed errors

info level？

info level？

This log is used to indicate the actual reason for {closed, "The connection was lost"}. I hope that the closed log and gun_down log can appear together by default. Set it as warning to hope that there is a way to block this log.


[RN] Add the button "Official site" to the object page, that redirects to the object site

Add the button "Official site" to the object page for the categories "Reserves" and "Excursions"
Test steps:

Open Radzima app
Go to the object page
Check that the "Official site" button is displayed
Click on the "Official site" button

Expected result
The "Official site" button is displayed on the object page and after clicking on it, redirection to the object site is implemented
Actual result
There is "site" field

fixed

verified - closed.


Error protect bind system

The bind system currently will crash the game when attempting to call a non-existant function.
The bind system should prevent duplicated function bindings.

Mostly done with https://github.com/ForLoveOfCats/SkyOfSteel/commit/cc61cfbc444deb9a72405f582f07e304e4e9f696

I'm going to call this done


Clear command #2

Fixed Util.IsMod such that it uses .retrieveMember(). And cleaned up some of my comments.

Try to avoid nested if statements as seen in the ClearCommand class.
Everything else looks nice:-)


Roborock finishes clean but doesn’t return to dock

As the title says. The roborock finishes its cleaning and then just stays in its last spot where it finished. It no longer even says “going back To the dock”. Just sits there until it runs out of battery.

Just had a similar problem.  Can you see any error messages in the Homebridge log?

Just had a similar problem.  Can you see any error messages in the Homebridge log?

Nope, I have to go find him and then press the charge button to force him to return. very off, I dont believe there has been a new firmware update thats changing the behaviour when a clean session ends.
anyone else having the issue?

What model do you have, @JonoHaysom?
Can you run homebridge with the environment variables DEBUG="miio:*,thing:*" and provide the logs you get? This will help us understand what are the internal messages and errors returned by the vacuum? Thank you!
Also, I recall there was an issue with the way iOS13 presented the vacuum and the pause button. If you have only 1 tile grouping all the switches for this vacuum, iOS sometimes mixes the Off signal and the Pause signal itself.

What model do you have, @JonoHaysom?
Can you run homebridge with the environment variables DEBUG="miio:*,thing:*" and provide the logs you get? This will help us understand what are the internal messages and errors returned by the vacuum? Thank you!
Also, I recall there was an issue with the way iOS13 presented the vacuum and the pause button. If you have only 1 tile grouping all the switches for this vacuum, iOS sometimes mixes the Off signal and the Pause signal itself.

Will do.
just need a few days as im offsite. hope we can crack it

What model do you have, @JonoHaysom?
Can you run homebridge with the environment variables DEBUG="miio:*,thing:*" and provide the logs you get? This will help us understand what are the internal messages and errors returned by the vacuum? Thank you!
Also, I recall there was an issue with the way iOS13 presented the vacuum and the pause button. If you have only 1 tile grouping all the switches for this vacuum, iOS sometimes mixes the Off signal and the Pause signal itself.

Will do.
just need a few days as im offsite. hope we can crack it

ok, so I dont have logs but I can confirm the behaviour occurs when doing a "clean" and the vac automatically returns to charge before completing the house. when it does that and then goes out to complete the vacuuming it finishing but then doesn't return to the dock.

Just to make sure I understand:

You trigger a full clean
At some point in the middle of the full clean, the vacuum goes to charge (is it due to an out of battery reason or anything else?)
Then, somehow, the full clean is resumed and the vacuum goes on with the full clean where it left it earlier.
When it's fully done with the clean, it doesn't go back to the dock?

Did I get it right?

Completely correct. its the RoboRock S5 and it goes back to the dock when it hits 20% to charge. everything else is completely correct

Got it now! Thanks for explaining!
I wonder, though, if this is more of an issue with the vacuum cleaner itself (there is no logic in this module to control the battery and send it to charge based on it and then resume the cleaning). Does it happen when you trigger the cleaning via the Xiaomi Mi app?

No, and thats the weird part. with Mi app it doesn't occur. which is odd

behaviour is still there. at east we know the scenario in which it occurs. I wonder if its an initialisation string for the initial request to start the cleaning.. and one of the parameters isn't passed when using home kit upon cleaning completion if its interrupted with a battery charge in between imitation and completion. where it finishes and then rather than return to dock, it remains paused in the place it finished the original cleaning request.

Yeah... that's super weird! It must be related to what you say.
Maybe the debug logs can help us find out what happens and what messages are going through (if run with DEBUG="miio:*,thing:*" we should be able to see the underlying messages when talking to the robot).

This issue seems stale... I'll close it. Feel free to reopen it when you manage to retrieve the requested logs :)


Added assert of array size before dereferencing

Added asserts, to avoid crashes, primarily seen during testing.

Pull Request Test Coverage Report for Build 49

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

Totals

Change from base Build 46:
0.0%

Covered Lines:
379

Relevant Lines:
431

💛  - Coveralls


chore(main): release 0.4.0

:robot: I have created a release beep boop
0.4.0 (2024-06-06)
Features

enable dynamic zone discovery and restrict records to those zones (#29) (326be28)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/jobs62/opnsense_unbound_external-dns_webhook/releases/tag/v0.4.0 :sunflower:


Add live tests and recordings for azure-identity

Splitting this issue to track adding the live tests and recordings thereof separately (#5980).


fix(remix-dev): only call onInitialBuild when the build succeeds

Remix is calling onInitialBuild hook even when the build fails without any information about the error. In Hydrogen, we use this hook to start the MiniOxygen server but it fails because the server bundle has not been generated.
This PR changes the Remix watch logic to only call onInitialBuild (and onRebuildFinish for consistency) after the first successful build. However, feel free to close this PR if the intended behavior is to actually call onInitialBuild even when the build throws errors. We can still workaround this in Hydrogen by manually checking the file system for the server build.
Another thing I noticed is that onInitialBuild is not getting the asset manifest in the parameters, unlike onRebuildFinish. Not sure if this is on purpose so I have not modified it.

Hi @frandiox!
Are you still interested in getting this one merged?
If so, please rebase onto latest dev & resolve conflicts

onInitialBuild has been removed. use onBuildStart and onBuildFinish instead (called for initial build and subsequent rebuilds).


Json Marshal considers value '0' as nil

Hi, during json.Marshal, a 0 value for an int is considered as empty, and the corresponding struct field isn't even made a part of the output. For example, if this is my struct
type some struct {
	Name string `json:"name,omitempty"`
	Value int64 `json:"value,omitempty"`
}

And this is where I'll marshal it:
someEx := &some{
	Name: "blah",
	Value: 0,
}
bodyContent, _ := json.Marshal(someEx)

The value of string(bodyContent) will be {"name":"blah"}. But when I use the following value
someEx := &some{
	Name: "blah",
	Value: 1,
}

Then the value of string(bodyContent) is {"name":"blah","value":1}.
I think that even if the value is 0, it can be needed by some applications, is it possible to pass it along?

Just drop the ,omitempty part?
For questions about Go, see https://golang.org/wiki/Questions.
If this is a bug report, though, feel free to reopen this.

Ah my bad. Don't know how I didn't notice that


Scala.js SBT plugin ignores logBuffered setting

History: https://github.com/typelevel/cats-effect/pull/51
The logBuffered in Test := false SBT setting is designed to allow test frameworks (and really anything with outputs) to produce feedback incrementally, typically as each individual test case completes, rather than exclusively as the test suite completes.  This is really really useful for extremely long running test suites, both from a usability and from a "it works in Travis" standpoint.  For context, Travis watches stdout/stderr, and if it doesn't see any new output in 3 minutes, it will assume the process is hung and kill it.  You can delay this through various dodgy mechanisms, but it doesn't work all that well.  Incremental output is the more scalable solution for long-running suites comprised of numerous fast test cases.
Unfortunately, the setting has no effect with .js projects.  It appears that the setting is being overridden by Scala.js somehow, and it's not entirely clear how.  Alex reported on some more minimized testing here.

@gzm0 Does that ring a bell to you?

Yeah, I had to implement buffering in the adapter itself, so output from the test frameworks does not end up interleaved (from different threads).
So currently, Scala.js completely ignores the setting, but only outputs all the logs, once a test is complete. This is a hack to work around the interleaving issue that survived until now.
Ideally, we do not car about buffering at all but just forward everything to the appropriate logger and sbt decides whether to buffer or not.
I can try and take a look in the following days.

Relevant code is in: https://github.com/scala-js/scala-js/blob/master/test-adapter/src/main/scala/org/scalajs/sbttestadapter/ScalaJSTask.scala

I have investigated this and I still do not understand why I introduced this in the first place.
When testing on our test-suite now, it seems that, after removing the buffering logic in Scala.js, the logBuffered setting does exactly what it is supposed to: If on, output comes nicely per suite, if off output lines are interleaved as they show up. This behavior is consistent on both JS and JVM.
@sjrd do you think we should investigate how other major test frameworks behave under this or should I just nuke our buffering logic? The risk is that people start complaining at the next release that they can't understand their test output :S

IIUC, they will complain only if they explicitly set logBuffered in Test := false, right? In which case it's they own problem, and we can tell them to remove the setting. Or are you afraid that we might destroy the output even under logBuffered in Test := true, because of some corner case we don't see?
If the latter, what about we introduce a "backdoor" during 0.6.x, based on a Java system property, to reactivate the internal buffering? And if we don't get complaints in 0.6.x, we definitively nuke it in 1.0.0.

Or are you afraid that we might destroy the output even under logBuffered in Test := true, because of some corner case we don't see?

Yes. I think the backdoor is a reasonable solution.

Fixed in 74c9af0d85b08e92435b488b97c8f6ac5392002f


[AutoPR frontdoor/resource-manager] typo: frontdoor/resource-manager/Microsoft.Network

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/5624

This PR has been merged into https://github.com/Azure/azure-sdk-for-go/pull/4546


Jobs not being submitted to sites with RSEs ending with _Disk

Impact of the bug
WMAgent
Describe the bug
Jobs for a workflow that reads Pileup locally i.e. TrustPUSitelists: false will not be submitted to the site if its  phedex_name name ends with _Disk Because the possibleSites function used by WorkQueue in the agent does not convert the phedex_name  to psn_name as defined in CRIC[1].
The possibleSites  method checks the pileup locations and the NoPileupUpdate (which I am assuming is linked to TrustPUSitelists from WF Config). and returns the intersection between SiteWhitelist and the Pileup data locations. The Pileup data locations are not converted from the Storage Element to Computing Element leading to this issue.
How to reproduce it
For Example:
cmsunified_task_JME-Run3Winter24wmLHEGS-00018__v1_T_240524_171256_5046
This workflow has very high priority (310001) but only runs at CERN.

No Jobs submitted to FNAL.
Jobs are not submitted to FNAL even though it has lower priority wfs running because of the possibleSites method.
Example of One WorkQueue picked up by an FNAL Agent, the intersection of SiteWhitelist and PileupData will only return T2_CH_CERN.
        "SiteWhitelist": [
            "T1_US_FNAL",
            "T2_CH_CERN",
            "T2_CH_CERN_HLT",
            "T2_CH_CERN_P5"
        ],
        "SiteBlacklist": [],
        "StartPolicy": "MonteCarlo",
        "EndPolicy": {
            "policyName": "SingleShot"
        },
        "Priority": 310001,
        "PileupData": {
            "/MinBias_TuneCP5_13p6TeV-pythia8/Run3Winter24GS-133X_mcRun3_2024_realistic_v7-v1/GEN-SIM": [
                "T2_CH_CERN",
                "T1_US_FNAL_Disk"
            ]
        },

Background
This problem has become apparent now because previously for minBias/heavy read pileups that do not read data remotely were only assigned to a single site. A bug in unified, caused workflows to be only assigned to a single site. We usually keep these pileups at CERN and FNAL, this bug caused the workflow to only be assigned to CERN. I fixed this bug and workflows started being assigned to both CERN and FNAL. link.
Previously since the wf was only being assigned to CERN, it was running at CERN so we did not notice this, but now the Workflows are assigned to both sites but only running at CERN.
Verification
To verify this that no jobs whose wf configuration had TrustPUSitelists: false and pileup at T1_US_FNAL_Disk were running. I Compiled a list of all workflows that run at FNAL and CERN for the previous year. (2023-06-06 to 2024-06-06).
There were a total of 1866 wfs with jobs whose data.Site=T1_US_FNAL and data.Type=Production.
1664 had Pileup Datasets mentioned in the Steps/Tasks
0 wfs had Pileup Datasets mentioned in the Steps/Tasks and TrustPUSitelists:False

There were a total of 1600 wfs with jobs whose data.Site=T2_CH_CERN  and data.Type=Production.
620 had Pileup Datasets mentioned in the Steps/Tasks
212 wfs had Pileup Datasets mentioned in the Steps/Tasks and TrustPUSitelists:False

I assume the same issue effects workflows that have input Data and reading locally, i,e TrustSitelists: False, but I have not verified it yet.
Steps to reproduce the behavior:

Assign a workflow with TrustPUSitelists:false only to a site whose storage element ends with _Disk.
The Workflow will remain stuck in Acquired state and agents will not create jobs for it.

Expected behavior
Jobs to be properly submitted to all sites in the site whitelist even if the SE and CE names are different i.e "T1_US_FNAL_Disk",
"T1_US_FNAL".
I can clean up the notebook I used to calculate the workflows. I was able to get data from Monit thanks to @nikodemas
FYI @amaltaro @anpicci @haozturk
[1]https://cms-cric.cern.ch/api/cms/site/query/?json&preset=data-processing

@amaltaro do I remember correctly that there was a reason to not consider site ending with _Disk as good ones? Or am I confusing things?

@hassan11196 having a careful look at this issue, I agree that the possibleSites() function returns inconsistent information, where SiteWhitelist and SiteBlacklist are actually a list of sites. But Inputs and PileupData actually contain a list of locations (RSEs).
Now looking for places that use this possibleSites() method, we can see (which does the PSN2PNN conversion of the output):
https://github.com/dmwm/WMCore/blob/master/src/python/WMCore/WorkQueue/WorkQueue.py#L454-L455
but this module does not convert it:
https://github.com/dmwm/WMCore/blob/master/src/python/WMCore/WorkQueue/WorkQueueBackend.py#L385
In addition, I find apparently inconsistent use as well in this module:
https://github.com/dmwm/WMCore/blob/master/src/python/WMCore/WorkQueue/Policy/Start/StartPolicyInterface.py#L254
as we can see for getDatasetLocations(), getDatasetLocationsFromMSPileup() and blockLocationRucioPhedex().
Lastly, we need to make sure that Sites (PSNs) and Storage (RSEs/PNNs/SEs) are properly considered in this CouchDB list function:
https://github.com/dmwm/WMCore/blob/master/src/couchapps/WorkQueue/lists/workRestrictions.js
Having said it all, I feel like we need someone to actually review everything that I said, review the code, test whatever is needed, such that we can find out where we have problems and whether objects should correspond to storage location (RSE/PNN) or CPU location (site/PSN). Please let me know if you think I am missing anything.

Thank you for the investigation, Alan. How do you recommend proceeding with the next steps?
From your reply, I understand that we have the following 3 action items:

Identify all places where Sites / RSEs are used.
Decide whether it should be Site/PSN or RSE/PNN.
Implement fix (fixes?) that are consistent on all levels we use sites/PSN.

As we discussed yesterday in the WMCore meeting, we are going to get first the pileup issue under control, which we have already properly identified. In a later stage/ticket, we can revisit site vs location (PSN vs PNN) across WMCore and evaluate if there are other places in need of bug-fixing as well.
I wanted to have a full example of WQE configuration, which I found on https://cmsweb.cern.ch/couchdb/workqueue/_design/WorkQueue/_rewrite/element/29a66613a07a655cbda0bf0787633487
As one can see, this is how it is currently using sites vs locations:
    "Inputs": {
      "/SinglePhoton_Pt-200To500-gun/Run3Winter24GS-133X_mcRun3_2024_realistic_v7-v2/GEN-SIM#8fa9810e-0de6-42ce-85fc-183e6becae97": [
        "T1_US_FNAL",
        "T3_US_FNALLPC"
      ]
    },
    "NoInputUpdate": false,
    "NoPileupUpdate": false,
    "SiteWhitelist": [
      "T1_US_FNAL",
      "T2_CH_CERN",
      "T2_CH_CERN_HLT",
      "T2_CH_CERN_P5"
    ],
    "PileupData": {
      "/MinBias_TuneCP5_13p6TeV-pythia8/Run3Winter24GS-133X_mcRun3_2024_realistic_v7-v1/GEN-SIM": [
        "T2_CH_CERN",
        "T1_US_FNAL_Disk"
      ]
    },

in summary:

this element does not have primary or secondary AAA enabled
input data location is actually a list of PSNs
site list is also a list of PSNs
but pileup data contains a list of locations (PNNs/RSEs)

Reopen issue, based on discussion on MM:
https://mattermost.web.cern.ch/cms-o-and-c/pl/ng3zisdcrig3igobk6qkw4txdr


🚀 - ability to disable or alter custom scollbar

Which @taiga-ui/* package(s) are relevant/releated to the feature request?
No response
Description
I think the title is pretty self-explanatory

I believe it's about global page scrollbar, introduced by tui-root. I suppose a DI token makes sense here, contributions are welcome.


configuration from the filesystem

In the documentation, Confit mentions taking configurations from directories. I'm guessing that means it expects a file of a prescribed name in those directories. I would rather just have a function that takes a path and returns a configuration that I can pass to override (#9). Then, composing a colon-separated path of configuration files in precedence order turns into this:
reduce(confit.override, map(confit.from_file, PATH.split(':')))

I think there is a good opportunity for directory based configuration, though. Many projects (like the users of this other confit) want to use different sets of defaults based on different environments. Common environments include "development", "qa" or "staging", and "production". It would be convenient if they could have a directory of multiple configuration files named after the environments.
def from_file(env, path): # env comes first for partial application
  file = path
  if not file.is_file():
    file = path / env
  if not file.is_file():
    file = path / (env + '.yml')
  ... # more guesses
  with open(file, 'r') as stream:
    return yaml.load(stream.read())

Yes, all good points.
The main reason that directories showed up before files in Confit is because, for my use case, I needed a way to store other files alongside the configuration. That is, in beets, the configuration really consists of a whole directory, not just the YAML file itself.
That said, I realize that the common case is probably a single file. Supporting a whole directory would be a nice extra feature, but the primitive functionality should be more or less as you've described it.
Nice find on that other confit. It does seem like this kind of environment-based configuration could be easily implemented by overlaid named configurations—separate functionality could provide the name-based search in the filesystem.

As a general practice, I would encourage authors to always use a single configuration file, keeping in mind a distinction between configuration and data. Many popular tools follow this pattern. For example, Git and Mercurial have .git/config and .hg/hgrc, respectively, and the rest of the contents of their hidden directories is all data. npm and Bower have package.json and bower.json and the rest of the contents of the package directories (source code, tests, docs, etc) is all treated as data. Just a note that influenced my last design.
That said, though, I think the directory approach with environments generalizes. In that perspective, a project with a single config file is just the special case that has exactly one environment.

Indeed—and beets works exactly the same as git and hg (those other files are data, not configuration).


How to push new workspace up to pbi service?

I can add a new workspace/group locally, but I'm not seeing how to push it up to the service. Am I missing a method?

you are right, currently does not work.
it seems the the functionality to work with groups/workspaces was recently extended to also support the creation of new groups
I will add this functionality also to the PBI API Client!
thanks for the hint

Heh no problem, looking forward to it! Basically I'm looking for a way to do PBI service tasks without managing all the httpclient hassles! A complete wrapper would be really great!

I just added a new function to the PBI API Client to create groups:
PBIGroup myGroup = pbic.CreateGroup("MyTestGroup");


Need a BaseObject

Need to create a base object which allows developers the ability to access wrapped and unwrapped Sparkpost API resources easily (in cases where the SDK hasn't been updated to the latest API version).

#15 addressed this


Ensure our custom workflow tasks send notifications

Spotted this yesterday while going over business area capabilities
reference: https://github.com/wagtail/wagtail/blob/6affa04d320f50b6a3babd8afc1a590e02e84a5d/wagtail/admin/mail.py#L368-L373 - GroupApprovalTaskStateSubmissionEmailNotifier checks we have a GroupApprovalTask instance, but our tasks inherit from AbstractGroupApprovalTask

Done, issue 12328


[ENH] GLM with multiple distributions and link function support

Reference Issues/PRs

fixes #383 and #230
What does this implement/fix? Explain your changes.

This creates an adapter converting the statsmodels GLM families to skpro equivalent giving GLMs a broader capability of distributions and link functions.
Does your contribution introduce a new dependency? If yes, which one?

No
Did you add any tests for the change?

Yes
PR checklist

For all contributions

[ ] I've added myself to the list of contributors with any new badges I've earned :-)
How to: add yourself to the all-contributors file in the skpro root directory (not the CONTRIBUTORS.md). Common badges: code - fixing a bug, or adding code logic. doc - writing or improving documentation or docstrings. bug - reporting or diagnosing a bug (get this plus code if you also fixed the bug in the PR).maintenance - CI, test framework, release.
See here for full badge reference
[x] The PR title starts with either [ENH], [MNT], [DOC], or [BUG]. [BUG] - bugfix, [MNT] - CI, test framework, [ENH] - adding or improving code, [DOC] - writing or improving documentation or docstrings.

For new estimators

[ ] I've added the estimator to the API reference - in docs/source/api_reference/taskname.rst, follow the pattern.
[ ] I've added one or more illustrative usage examples to the docstring, in a pydocstyle compliant Examples section.
[ ] If the estimator relies on a soft dependency, I've set the python_dependencies tag and ensured
dependency isolation, see the estimator dependencies guide.

I have thought of 2 possible ways to pass the offset and exposure parameters while predicting.
Design 1 and Design2 .
Design 1
Idea behind this is that I initialize the object of the class with whether offset and exposure is going to be passed while predicting and will contain boolean values. Since we cannot have an argument offset and exposure in the _predict function user must pass additional columns with names offset and exposure which will be removed/dropped while fitting and predict_proba but will be converted to array and passed to statsmodels predict thereby keeping consistency in the number of columns and rows.
Pros

It will allow user to keep changing the offset and exposure params even after fitting data.
User doesn't have to keep re initializing the model in case they want to change the offset/exposure values.

Cons

User will have to create 2 extra columns with the exact same name of offset and exposure each time in the exog/X variable.
No way to add a test setting to check this as it cannot add extra columns with the name offset/exposure.

Design 2
Idea here is that I pass the offset/exposure array while initializing the object itself. It must have the same length as the number of rows in the X that will be passed while predicting.
Pros

Simple code :)
User doesn't have to struggle having to modify the X while fitting.

Cons

shape of X passed to predict must be known while initializing/constructor the object itself.
If the size of prediction X needs to be changed the offset/exposure must be changed for which it will need to be re-initialized again.
Re-fitting the model will need to happen each time X size changes for predict.
Even for this method there is no way to add a test setting as size of X used for predicting is not known prior.

My Conclusion
Both the designs above are not a foolproof way but they both give correct answers. Since we cannot add a test setting for both these I am not sure I can think of a better way to do it where it can be added to the test setting too.

A strong design principle in sklearn-like designs is separating data from model specification, because only that allows reliable model composition.
As Option 2 violates that principle (offset and exposure are part of the data), as your cons imply, I would have a very strong preference towards option 1.
I would vary the idea a little, by adding parameters exposure_var and offset_var, which are pandas index elements or int. If int, it is assumed iloc; otherwise loc. There are sensible defaults, I would say None, which means no exposure/offset is passed (and I believe statsmodels assumes constant exposure and zero offset then).

I would vary the idea a little, by adding parameters exposure_var and offset_var, which are pandas index elements or int. If int, it is assumed iloc; otherwise loc. There are sensible defaults, I would say None, which means no exposure/offset is passed (and I believe statsmodels assumes constant exposure and zero offset then).

I am little unsure on what this means. Do you mean to add these 2 parameters exposure_var and offset_var along with the already present exposure and offset bool parameters as I have done in Design1.
If we are adding along with these then I think there would be no need to have the extra columns in X that the user has to pass while fitting,predicting etc. This would again mean that we are not separating the model specification from the data.
Could you please elaborate the idea a little.

I am little unsure on what this means. Do you mean to add these 2 parameters exposure_var and offset_var along with the already present exposure and offset bool parameters as I have done in

I was suggesting to replace these with two more informative variables, concretely:
Replace exposure by exposure_var, type is pandas index element (e.g., string). If exposure_var = None, it behaves like your exposure = False. If exposure_var = "exposure", it behaves like your exposure = True. And exposure_var = "sth_else" can be used for pointing to another variable.

Could you please elaborate the idea a little as to what exactly the exposure_var and offset_var will contain.

The type would be a single pandas index element. For instance, str. If str, then it would point ot the column loc. I also suggest to use integers iloc rather than loc.

Replace exposure by exposure_var, type is pandas index element (e.g., string). If exposure_var = None, it behaves like your exposure = False. If exposure_var = "exposure", it behaves like your exposure = True. And exposure_var = "sth_else" can be used for pointing to another variable.

Yea this makes more sense and I have completed implementing it that way and also re ordered the new params to the end. Next we will have to work on adding test setting for it.

great! btw, if you want to move the position of the parameters later, we should follow the "move parameter position" recipe - we can make the change right away.


TLS 1.3 is now latest standard

Should correct that TLS 1.3 is now the newest released standard.
https://datatracker.ietf.org/doc/rfc8446/
Any idea when MSFT products will start supporting it in their products including AD/ADLDS, etc.

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

ID: 7341fa0a-f474-6c2d-d3c0-7e2df12fbb65
Version Independent ID: e99c489a-7518-d293-ad93-7eaca8b7191a
Content: Transport Layer Security (TLS) best practices with the .NET Framework
Content Source: docs/framework/network-programming/tls.md
Product: dotnet-framework
GitHub Login: @blowdart
Microsoft Alias: dotnetcontent

True, @mairaw can we change initial paragraph to something like.
The Transport Layer Security (TLS) protocol is an industry standard designed to help protect the privacy of information communicated over the Internet. TLS 1.2 is the now the widely accepted security standard, with TLS 1.3 being ratified in Octore 2018. This article presents recommendations to secure .NET Framework applications that use the TLS protocol by moving from TLS 1.0 and 1.1 to use TLS 1.2.
This document, and github wouldn't be the place to comment on future product planning.

@dagolinuxoid put up PR #8538, thanks!


Some typos fixes

Sorry for opening an issue without searching

Thanks for this !

:+1: :sparkles:


Re-use native executable when test is re-run by FailSafe plugin

Summary
Right now if you do mvn verify (or with clean but there is Quarkus maven plugin in your POM file) and there are no build-time properties, the native executable from /target is re-used. However when there are build-time properties, it is not re-used. Logic behind that is probably build-time properties requires new build of native executable. But what if you added a new class, or removed one, or changed source code, doesn't it require new native executable? That is why user should use clean like mvn clean verify when he made changes that require new executable/JAR (we are talking user changes, not changes done by the test itself).
This way, we can re-use custom builds kept in module target for test re-run by Failsafe plugin when test fails (which saves time and we do not need to test indempotency of Quarkus build). On the other hand it means if there is 10 different test classes with build properties, there will be 10 different native executable kept in target (we will keep custom builds). Native executable should be small, so I do not consider it an issue.
It safes a lot of time when test is re-run by the Failsafe plugin.
Another effect of this PR is that it makes also native executable build by this framework without build-time properties re-usable, which should allow us to disable Quarkus plugin build in native mode in the future. If we want to . I'll propose and explain that in a separate PR.
Please check the relevant options

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Dependency update
[x] Refactoring
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] This change requires a documentation update
[x] This change requires execution against OCP (use run tests phrase in comment)

Checklist:

[x] Example scenarios has been updated / added
[x] Methods and classes used in PR scenarios are meaningful
[x] Commits are well encapsulated and follow the best practices

run tests

run tests

run tests

run tests

run tests


Should cmdParts remove empty strings?

The line splits cmd string to parts. But some of these parts could be empty. So is it proper to remove empty strings?

@zzctmac Nice catch! That cleanup before sending to exec would be a safeguard. Please feel free to raise a PR if you want to or else I can fix it this weekend.

Thanks for your reply. It would be great if you can fix it.

@zzctmac This is fixed and available in the latest release!


Support for nested configurations

it would be very helpful to have nested configuration working, like AppConfig.emails.office

:+1:
there is ugly workaround: AppConfig.emails['office']

The problem is that at some keys we get hash instead of OrderedOptions object.

Fixed in https://github.com/netguru/app_konfig/pull/3
Closing.


Open vs Tab Open

If I launch Oni and go to Oni(File) -> Tab Open, all files will be opened in new tabs and gt will move between tabs.  If I launch Oni and go to Oni(File) -> Open, all files will be opened in new tabs but gt will not move between tabs.
I'm not sure what benefit there is to having both Open and Tab Open given that they seem to do the same thing.  The only difference between the two that I see is any files opened by Open are ignored when using gt.  If I open some files with Tab Open and other files with Open, gt will only iterate through the tabs opened by Tab Open.
I guess my recommendation is to just remove Open and keep the Tab Open functionality.  Though if we do that we should probably rename Tab Open to just Open.

Also, it looks like any files opened using the fuzzy finder follow the Open code path because those files are ignored by gt as well.

If I launch Oni and go to Oni(File) -> Tab Open, all files will be opened in new tabs and gt will move between tabs. If I launch Oni and go to Oni(File) -> Open, all files will be opened in new tabs but gt will not move between tabs.

If you open a tab in vanilla vim/neovim, you with a new Tab which you can cycle with gt
If you open a file normally, you will open up a buffer.
The oni tab bar handles both...
Due to how vim handles buffers and tabs it's hard to figure out what the correct behavior SHOULD be

Good catch... It's really confusing to have 'Open...', 'Tab Open..', 'Split Open...', AND 'Open Folder...' - especially when we now have the buffer line here.

I'm not sure what benefit there is to having both Open and Tab Open given that they seem to do the same thing

Right, it's confusing... especially in the context of making Oni inviting to 'new' Vim users - a tab means something very different in the vim-world versus in most other editors...

I guess my recommendation is to just remove Open and keep the Tab Open functionality. Though if we do that we should probably rename Tab Open to just Open.

This sounds good to me. It alleviates the confusion, a new user would understand what's going on... and it also behaves the way a vim-veteran-who-uses-tabs would expect (at least initially).

Also, it looks like any files opened using the fuzzy finder follow the Open code path because those files are ignored by gt as well.

Seems like we should be consistent - it's confusing that two different high-level gestures ('Open File') have a different outcome. To be consistent above, seems like we should use tabe.

Due to how vim handles buffers and tabs it's hard to figure out what the correct behavior SHOULD be

Right, Oni adds some extra confusion here. IMO I believe Vim-style tabs are confusing for new users (because they don't map conceptually to what users think of as tabs in other programs / editors - they're really window layouts), so I don't necessarily want to optimize for them. I also think that the vim-tab story will potentially get more confusing as multiplexing is implemented in #362. However, if we can get reasonable behavior that works for vim-tabs as well (like in the open cases), there's no reason not to support that (and advanced Vim users that prefer a tab workflow can set the tabs.showVimTabs option, and presumably they'd know enough to be able to wade the confusion..)
So here's the summary of changes I read from above:

Implement @keforbes suggestion to consolidate Open and Tab Open to just Open. I'd also like to change it to Open File just to be crystal clear.
Change QuickOpen to open a tab, so we have consistent behavior. It might be good to refactor this into a common command so that we can share the code path (like oni.openFile).


Metric learning with PaCMAP?

Would it be possible to add a feature that allows one to embed an unseen/unlabeled data point to an existing embedding using metric learning? This would be similar to the function currently available with UMAP:
https://umap-learn.readthedocs.io/en/latest/supervised.html
Thanks!

If this could be implemented, it would be incredible. I absolutely love PaCMAP and have been using the dimensions as predictor variables for supervised models. This would take things to a new level

Thank you for your feedback! This feature is a top priority on our to-do list, and we are committed to working on it!


Document GAPIC generated Credentials classes

Currently the GAPIC generated Credentials class does not have any documentation on how to use it. Most of the behavior is inherited from Google::Auth::Credentials, but it would be good to document these classes, including come code examples, so users can better understand how they are to be used if they choose to use them.

@jbolinger @dazuma Thoughts on this?

Moved to backlog, opened googleapis/gapic-generator-ruby#5.


Problem with stroke-width while plotting multiple Rect

Hey guys
Problem: stokeWidth={1} on a Rect doesn't seem to take effect when you are plotting multiple rectangles.
Concise test case: https://codesandbox.io/s/2jvo555lyj
Visual Output:

This seems really weird, I noticed it at first because while plotting multiple Rect the stotke is really inconsistent. Am I missing some really obvious?

Take a look here: https://stackoverflow.com/questions/13879322/drawing-a-1px-thick-line-in-canvas-creates-a-2px-thick-line
The first rectangle looks different, because it is near the edge of the canvas.

Thanks @lavrton. That helped me track it. Evidently svg suffers from it as well.
  if (val % 1 === 0) return val + 0.5;
  return val;
}

Just for reference this helped me fix it. It just moves x/y coordinates by 0.5 when needed.
https://codesandbox.io/s/4195vxv74


My working copy of nativescript-app-encryption

As title, here's my working copy of nativescript-app-encryption
just use
tns plugin add https://github.com/zenz/nativescript-app-encryption.git
to add my plugin into your project. And do
tns build ios|android --release [--for-device]
will do the rest of the works.

that work perfectly, thanks for the effort
my question is how to use app-encryp with webpack, i mean encrypt the code after minification ?

It's not compatible with Webpack yet. It prepare the encrypted js files before webpack handling, so it will failed while webpack doing it's job(some not exist files).

yeh, thanks for response. hoping to improve that plugin, it's a great tool


Remove Username

Hello!
Is there a way to remove the username field and only use email for registration/login/account management?

Hi julyprum,
in general that should be possible but it would be a large change of my library. My library bases on username and email address to be two different objects with different purposes. In other words, there is no easy setting or way to change it the way you describe.
Best regards,
George


Add infra to enable unit tests

Ideally we should be able to use a separate db portfolio-test for testing.
The test script should automatically pick this new db, and return it in the get_db() calls.
Custom data can be added on startup.

Refer this link for possible solution

Fixed in #20


Job addon

Proposal:
Proposal written by Denarioyt
Is it a proposal for the website or for the bot?: bot
What is the title of your proposal?: Job addon
Explain your proposal more accurately (It's best to give as much information as possible, so that we can implement the proposal better): My idea is that you can enter ?job <number of the job> so that you don‘t have to enter two commands.
Why should we add this feature?: Would be better because you can only enter one command instead of two
ReportID: 173

➤ LenoxBot commented:
Approve:
very useful
Nickkel#6515

➤ LenoxBot commented:
Approve:
would be very useful!
Profxy#5396

➤ LenoxBot commented:
Approve:
Agree with nickkel and profxy
SaVaGeCrEePeR28 | Furkan#3532


SQL adapters

See design here: https://docs.google.com/document/d/1-xVPX7PqyENOk31bXdcAVVPYSHaGMokeFiR85d_zoOo/edit.

TLDR -- an adapter that enables you to run either a transform or a load from sql. Framework can abstract away the executing/connecting piece. This would be especially nice with duckdb -- see https://github.com/stitchfix/hamilton/issues/158.

We are moving repositories! Please see the new version of this issue at https://github.com/DAGWorks-Inc/hamilton/issues/46. Also, please give us a star/update any of your internal links.
Note that everything else (slack community, pypi packages, etc...) will not change at all.


HoughLine was not working. Fixed two issues.

HoughLine was not working. The constant CV_PI was missing in lua code. Furthermore, the parameter "rho" was missing when calling the function.

@JHoef
Thank you very much for contribution! :)


Audio and subtitle stream cycling

It is now possible to cycle through various audio and subtitle streams
I also refactored some of the documentation code I had pasted earlier to use doctring.

thanks for this, made a couple changes in order to prevent mutex locking on the ui thread + simplify the ui a bit


How to make line chart multiple colors

Hi, thank for the great library.
But, I can't make a Line Chart multiple colors, look like this picture:

Is this possible to do this in Victory-native?

@thanhtuttm you can do it with a gradient! Check out this example: https://formidable.com/open-source/victory/docs/faq/#how-can-i-use-gradient-fills-in-victory
If that doesn't work, you can render multiple VictoryLine components

@boygirl - How do i render 1 single  with different colors, in reactnative?


Welcome message is duplicated in Facebook channel

I'm using the following pattern used in multiple samples and it works fine in WebChat and Emulator channels but when I connected my bot to the Facebook channel it sends a duplicate welcome message.
    protected override async Task OnMembersAddedAsync(IList<ChannelAccount> membersAdded, ITurnContext<IConversationUpdateActivity> turnContext, CancellationToken cancellationToken)
    {
        var welcomeText = "Hello and welcome!";
        foreach (var member in membersAdded)
        {
            if (member.Id != turnContext.Activity.Recipient.Id)
            {
                await turnContext.SendActivityAsync(MessageFactory.Text(welcomeText, welcomeText), cancellationToken);
            }
        }
    }

Is this a sample issue or something is wrong with the Facebook channel?

It looks like Bot's member.Id for Facebook channel is wrong, it has both user's and bot's ids combined with "-":

Is this intentional or should be considered a bug?

@turbozapekanka just to give more context on the same.

The way that the bot stores user information (userState) and the way Facebook does is very different. UserState is created and maintained by the bot developer (you). Facebook ALSO maintains a list of people that have connected to an app, either through the facebook apps such as games, or via Messenger. Facebook stores this information in either PSIDs or ASIDs, accessable through the Facebook Graph API v2.0

On the bot framework side, the turnContext.Activity.MembersAdded is hit when a facebook user first connects with the bot (and thus creates their PSID or ASID to be stored in the Facebook Graph API). You could, theoretically, go in and clear the PSIDs/ASIDs from your Facebook app (the bot) on the Facebook side, but bear in mind that once a ASID/PSID is created for a Facebook user, that's THEIR ASID/PSID. If they're removed from the bot so they could be "reconnected", Facebook Graph is going to give them back that same PSID/ASID and thus the .MembersAdded flag is not going to trip and you'll be left right where you were.

Having said that, I am having difficulty to repro the issue as I am getting the 'Hello and Welcome' message just once. Can you share the steps to repro so that I can give it a try on my end?

The repro steps are below:

Deploy Echo bot https://github.com/microsoft/BotBuilder-Samples/tree/master/samples/csharp_dotnetcore/02.echo-bot to Azure.
Connect bot to Facebook messenger using instructions https://docs.microsoft.com/en-us/azure/bot-service/bot-service-channel-connect-facebook?view=azure-bot-service-4.0#connect-a-bot-to-facebook-messenger
Message to the facebook page using facebook messenger

@turbozapekanka do you get the 'Hello and Welcome' message twice every time you connect your echo bot to the facebook channel(connected to the same user or the same page/app) or every time you test, do you create a new page and connect it to the bot(and still get the welcome message twice)?

I only get it the first time I send a message. To reproduce this I used another facebook account, added it to testers role in facebook app, and messaged from a new account. Subsequent messages do not trigger OnMembersAddedAsync method at all, so I do not receive any welcome message. I don't know how to make the bot forget about previous conversation and start over.

@turbozapekanka  its how Facebook handles the user information on their end. Facebook maintains a list of people that have connected to an app, either through the facebook apps such as games, or via Messenger and stores this information in either PSIDs or ASIDs, accessable through the Facebook Graph API v2.0.
As per this behavior, when you connect a new facebook account(user) to the bot for the first time, the 'OnMembersAddedAsync' is triggered and the welcome message is displayed. The next time when you connect the bot to the facebook channel on the same user, you will not be getting a welcome message as the 'OnMembersAddedAsync' is not triggered. Even if you go ahead and clear the PSIDs/ASIDs from your Facebook app (the bot) on the Facebook side, the next time you try to reconnect, Facebook Graph is going to give them back that same PSID/ASID and thus the .MembersAdded flag is not going to get triggered. Only when you connect to a new account for the first time, the OnMembersAddedAsync will be triggered. Hence, you cannot make the bot forget the previous conversation and start over on the facebook channel.
Note: I am going to test extensively using a new account and try to repro what you are encountering and update the thread.

@turbozapekanka apologize for the delay in response. I will update the thread shortly. I am trying to test using the business/brand page option.

@turbozapekanka I am able to repro the issue on my end when I connect a new account/app to my bot. I am suspecting it to be related to how it is handled on the Facebook end(ID generation) since as per the bot code, it should just trigger once. I will update the thread once I am able to get more info on how this is being handled on Facebook's end.

@anusharavindrar Great to hear that you could repro. The cause is most likely due to member's ID is incorrect as shown on my screenshot above.

@turbozapekanka thank you for your patience. You were right, the issue was due to the fact that the channel is sending the wrong recipient id in the conversation update activity. A PR for this fix is already underway and would be merged in next week's build. You could hopefully be able to test it later next week to get the correct behavior.
Hope this helps.

Closing the issue as the fix should be out in this week's merge. Please feel free to reopen this if the issue persists.


Document CS8032

This specific warning should be a high priority IMO. It's being reported a lot, and usually unclear to the developer to figure out why it happens.

The most common reason for the warning is using a Visual Studio version that doesn't support a given CodeAnalysis package version.
CodeAnalysis version 3.x requires VS 2019 16.x (e.g, 3.11 requires 16.11)
CodeAnalysis version 4.x requires VS 2022 17.x
This should be linked from the analyzer tutorial to warn analyzer authors that upgrading their CodeAnalysis packages will require consumers to upgrade VS.

This is still being reported repeatedly and needs attention.

CodeAnalysis version 3.x requires VS 2019 16.x (e.g, 3.11 requires 16.11)
CodeAnalysis version 4.x requires VS 2022 17.x

We need mapping like this but for command-line builds.


utils:gcov fails: [errno2] no such file or directory

My setup worked fine until recently. I have not changed my Ceedling setup for the project in question since the last time it worked and that was setup using Ceedling pulled at v0.22.0 (or maybe 178983d4, which is the commit following v.0.22.0) , though gcc, gcov (both v6.2) or gcovr (v3.3) may have had patch updates during a $sudo apt-get upgrade (still same major and minor version though). I am using Ruby with rbenv set for Ruby at version 2.3.1.
Running $rake clobber && rake gcov:all works as normal, but $rake utils:gcov fails with:
$ rake utils:gcov
(in /mnt/c/Users/user-name/git/repo-name)
Creating html report of gcov results...
Traceback (most recent call last):
  File "/usr/local/bin/gcovr", line 2323, in <module>
    print_html_report(covdata, options.html_details)
  File "/usr/local/bin/gcovr", line 1650, in print_html_report
    OUTPUT = open(options.output, 'w')
IOError: [Errno 2] No such file or directory: '/mnt/c/Users/user-name/git/repo-name/build/artifacts/gcov/GcovCoverageResults.html'
ERROR: Shell command failed.
> Shell executed command:
'gcovr -p -b -e '^vendor.*|^build.*|^test.*|^lib.*' --html -r . -o GcovCoverageResults.html -o build/artifacts/gcov/GcovCoverageResults.html'
> And exited with status: [1].

rake aborted!
ShellExecutionException: ShellExecutionException
/mnt/c/Users/user-name/git/repo-name/vendor/ceedling/lib/ceedling/tool_executor.rb:78:in `exec'
vendor/ceedling/plugins/gcov/gcov.rake:160:in `block (2 levels) in <top (required)>'
Tasks: TOP => utils:gcov
(See full trace by running task with --trace)

I tried running with sudo but root user doesn't have Ruby setup , d'oh!
How can I fix this?

This is definitely an issue with gcovr rather than ceedling. The only thing the utils:gcov command does as of now is use call the gcovr command and uses the output files of gcov:all.
This line specifically is the issue:
> Shell executed command:
'gcovr -p -b -e '^vendor.*|^build.*|^test.*|^lib.*' --html -r . -o GcovCoverageResults.html -o build/artifacts/gcov/GcovCoverageResults.html'

The option in that causes the issue specifically is the option -o build/artifacts/gcov/GcovCoverageResults.html as it seems  gcov will not write if it can't find the needed directories. This option is currently also not in the the plugin at the moment. I'm currently working on actually making this file appear there along with other improvements, so this is good info for me to account for.
On my machine I was able to recreate and fix it the following way:
dom@dev:~/dev/test_project$ gcovr -p -b -e '^vendor.*|^build.*|^test.*|^lib.*' --html -r . -o GcovCoverageResults.html -o build/artifacts/gcov/GcovCoverageResults.html
Traceback (most recent call last):
  File "/usr/bin/gcovr", line 1970, in <module>
    print_html_report(covdata, options.html_details)
  File "/usr/bin/gcovr", line 1427, in print_html_report
    OUTPUT = open(options.output, 'w')
IOError: [Errno 2] No such file or directory: 'build/artifacts/gcov/GcovCoverageResults.html'
dom@dev:~/dev/test_project$ mkdir build/artifacts/gcov
dom@dev:~/dev/test_project$ gcovr -p -b -e '^vendor.*|^build.*|^test.*|^lib.*' --html -r . -o GcovCoverageResults.html -o build/artifacts/gcov/GcovCoverageResults.html
dom@dev:~/dev/test_project$ ls build/
artifacts/ gcov/      .keepme    logs/      temp/      test/
dom@dev:~/dev/test_project$ ls build/artifacts/gcov/
GcovCoverageResults.html

Hope this helps.

This morning I have figured out for the issue I have had by using the shell command:
gcovr -p -b -e '^vendor.*|^build.*|^test.*|^lib.*' --html -r . -o GcovCoverageResults.html -o build/artifacts/gcov/GcovCoverageResults.html

That the reason is that I have my build directory renamed to non-standard test-build. I though I had accounted for this by changing the build_root value in project.yml, but either the gcov rake task does not take build_root into account or I need to set it somewhere else too.
Using the above line with the /build/ part of the path adjusted works OK without any prior mkdir even after a clobber.

Ah, I see that the -o argument is hard-coded separately in the :tools_gcov_post_report: :arguments: section... that would probably explain that! LOL

Is this resolved at this point? I'm not sure how you guys left this.

@mvandervoord  It's resolved. The issue itself is in the limits of gcovr.

Just revisited this thread as found that I'm having the same issue originally reported. I'm running on Win10 and swapping the ticks for quotes hasn't solved it. Also if I run the failing command manually it works fine.
Failing cmd:
λ ceedling.bat utils:gcov Creating a detailed html report of gcov results in build/artifacts/gcov/GcovCoverageResults.html... rake aborted! Errno::ENOENT: No such file or directory - gcovr.py -p -b -e "^vendor.*|^build.*|^test.*|^lib.*" --html --html-details -r . -o  "build/artifacts/gcov/GcovCoverageResults.html" c:/dev/Researching/vendor/ceedling/lib/ceedling/system_wrapper.rb:45:in ``' c:/dev/Researching/vendor/ceedling/lib/ceedling/system_wrapper.rb:45:in shell_backticks'
c:/dev/Researching/vendor/ceedling/lib/ceedling/tool_executor.rb:68:in exec' c:/dev/Researching/vendor/ceedling/plugins/gcov/gcov.rake:168:in block (2 levels) in <top (required)>'
C:/Ruby21/bin/ceedling:23:in load' C:/Ruby21/bin/ceedling:23:in '
Tasks: TOP => utils:gcov
(See full trace by running task with --trace)
ERROR: Ceedling Failed`
However, running this below is absolutely fine:
gcovr.py -p -b -e "^vendor.*|^build.*|^test.*|^lib.*" --html --html-details -r . -o  "build/artifacts/gcov/GcovCoverageResults.html"
Has anybody got any views on this?


Simple listing of what hooks are disabled

The only way I see this working is that we parse all the theme layout files and use a regex to find out if the hook names are present or not. Honestly, sounds like way too much trouble than it's worth.
I propose we delete this issue.

Closing for now. Re-open if needed later.


Locale should be filled

See review: https://github.com/graphhopper/directions-api-java-client/pull/20#pullrequestreview-778871

I checked for Nominatim and Opencage, but both do not return a locale, but both allow for querying a locale. But there seems to be no option to view the returned locale.
We could just return the queried locale. Or we could setup a check if the given locale is a valid locale, and only return it if it is valid. Otherwise, return en. Not sure if there is another option?

Then I also see no other possibility.

This was fixed in https://github.com/boldtrn/graphhopper-geocoder-converter/commit/2b41c0f3505dce85072bc942f2a1abda034cec91


Contacts_Service Must Have SWIFT VERSION

contacts_service does not specify a Swift version when flutter run iOS. How to solve this problem?

Delete the ios folder (you might want to create a backup first)
Run this command in the root of the project:  flutter create -i swift .

Version is Fixed At Swift4.2, if iOS Project use Swfit5.0 will lead error again


feature(loading): loading enhancements + re-haul to use material modules. (closes #255) (closes #84)

Description
Code re-haul of loading.module for enhancements and better usage + unit tests + better demos.
https://github.com/Teradata/covalent/issues/255
https://github.com/Teradata/covalent/issues/84
What's included?

feature(animations): Added tdFadeInOut animation and used it instead of tdFade directive.
feature(loading): Easier fullscreen loading creation + default fullscreen mask.
feature(loading): Better efficiency with OnPush strategy.
Deprecate ILoadingOptions in favor of ITdLoadingConfig
Deprecate createOverlayComponent() method from TdDialogService in favor of create()
Deprecate loadingType and loadingMode in favor of tdLoadingType and tdLoadingMode so the inputs can be used correctly in (*) syntax.    e.g. *tdLoading="'name'; type:'linear'"
refactor(loading): Leverage TemplatePortal and Overlay from material.
refactor(loading): separated code into creation (factory) and execution (service), and made TdLoadingFactory for internal usage only.
feature(loading): Added tdLoadingStrategy to change between overlay and replace strat. https://github.com/Teradata/covalent/issues/84
feature(loading): Added tdLoadingColor input for primary , accent or warn https://github.com/Teradata/covalent/issues/255
internal(loading): Better error messages.
internal(loading): More code blocks in methods.
docs(loading): Better demo and examples.
code-health(loading): unit tests for TdLoadingDirective and TdLoadingService

Test Steps

[x] ng serve
[ ] go to http://localhost:4200/#/components/loading and see new demo and examples.
[ ] npm run tslint
[ ] npm test and see code coverage now.
[ ] npm run build

Before unit tests:

After unit tests:

Screenshots or link to CodePen/Plunker/JSfiddle

Coverage decreased (-1.1%) to 69.945% when pulling 4486fbf9ae62af1fe21ebf7e9b8819debedd50d1 on feature/loading-enhancements into e382f16bcb5c5530d0a0fc92d49a15e078352813 on develop.

Coverage increased (+6.2%) to 77.275% when pulling af098ab7bfbe109ce927d520d7070ff0d226c23e on feature/loading-enhancements into e382f16bcb5c5530d0a0fc92d49a15e078352813 on develop.

It would be nice if:

[ ] the top demo had some loader running by default
[ ] the Register buttons read Toggle Loader

is the breaking change: Renamed ILoadingOptions into ITdLoadingConfig absolutely necessary? :)

Coverage increased (+6.2%) to 77.275% when pulling 8a7247881f3f3cacecedaa4d59379f7dea7998a6 on feature/loading-enhancements into 94997412bc580406f53494029ff01d08accdd152 on develop.

Coverage increased (+6.2%) to 77.275% when pulling d6e83d09ff697354921b399e1407c72ff4e9fed4 on feature/loading-enhancements into 94997412bc580406f53494029ff01d08accdd152 on develop.


[language] remove diem-types dependencies from move-lang

This removes the dependency on diem-types from move-lang.

/land


First Person is buggy when in dungeon by Dungeonz mod

for some reason, when I enter a dungeon (from: DungeonZ), my First Person goes glitchy and the camera is in my head again.
[realcamera-0.6.0-alpha.1]
video showcasing the problem:
https://drive.google.com/file/d/1Cu9dSi9xK29uo8eLwT3Qem1BYtxVLqNr/view?usp=sharing

(has nothing to do with the problem above)
also  in the newest version (also tested the newest snapshot) the first person model only shows when a better combat animation plays. (tested it without my modpack and it worked)
may be a performance mod breaking this?

The issue above has just been fixed
but I have no idea about the issue below, I need more information, such as the game log to reproduce it

I'll try to find the mod thats causing the issues.
If I find it I'll reopen

ok I got it, it's 3D Skin Layers [v.1.6.2]
it completely disables the first person model
using [realcamera-0.6.0-beta (latest snapshot)]
but for some reason when Emotecraft is enabled the First Person Model gets rendered when attacking with Better Combat,
but when disabled its not rendered entirely again

snapshot


Typescript generation

Title self explain :) i would like to generate typescript definitions, any suggestions how to do this better? right now i manually specify what classes i would like to use to generate types.
approach i think is the best is to hook clearscript  addhost* functions and based on that generate types

Hi @CrosRoad95,
ClearScript does not generate TypeScript declarations for .NET resources.
Sorry!

yes i already figure it out, just need some tip how  to do this? as i said i want to get all types added to js runtime but from what i see clearscript add more types than i declared

just need some tip how to do this?

Sorry, this isn't something we've ever investigated, so we have no tips for you. In theory, since all .NET type information is accessible via reflection, it should be possible to generate TypeScript declarations automatically.
However, capturing the entirety of ClearScript's .NET interop via TypeScript declarations – given all the ways in which you can customize that behavior – would be a large project.
In fact, since you can customize ClearScript's behavior at runtime – e.g., by changing the default script access level or installing a new custom attribute loader – it may not actually be possible to do it all via static TypeScript declarations.

i want to get all types added to js runtime but from what i see clearscript add more types than i declared

Please say more about this. ClearScript should not be exposing anything without your permission. The only exception is the EngineInternal object that's reserved for ClearScript's private use.


Have an application level config to enable/disable sid claim

Is your suggestion related to an experience ? Please describe.
According to the back-channel logout spec https://openid.net/specs/openid-connect-backchannel-1_0.html#LogoutToken , sid claim is optional. If sid claim is not found, then application can terminate all the sessions of the user. So We need to have an application level config to enable/ disable sid claim

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Support for Direct LAN Serial Communication

Describe the solution you'd like （描述需求）
I have replaced the Midea WiFi dongle with my own ESP8266 which acts as a serial bridge. I would like the midea-ac-py software to support serial communication with the Midea unit without any of the stuff needed for the Midea dongle (no ID/key/token). Instead you only need to know the IP and port number to the serial bridge. This type of thing exists for node.js in https://github.com/reneklootwijk/node-mideahvac/ as the "serialbridge" communication method.

Why not use ESPHome instead? A Midea component already exists and is likely compatible with your dongle.

Because it's overly complex for what I want to do?

It's really not. It's a few lines of YAML to configure, build it, flash it and add the node to HA. Easy.
I've got a config laying around I can post later today,

I mean the architecture is overly complex creating custom firmware per device, etc. It's just not my preferred design architecture. Also, I want the ability to read and write to the unit from outside of HA too.

I mean the architecture is overly complex creating custom firmware per device, etc.

More complex than writing your own firmware? 🙄
This project midea-ac-py is a Home Assistant integration. Your request would be better suited for the underlying library midea-msmart that handles the communication.


Move objects mentioned in documentation from details namespace

Done in https://github.com/evgeniums/cpp-validator/commit/c4b1c6920c05f216e796fd359c3ac0a0e12dcf4a.

Done in https://github.com/evgeniums/cpp-validator/commit/c4b1c6920c05f216e796fd359c3ac0a0e12dcf4a.


LEAN-2738: TCP-100C:  Authority

@mbartenev-atypon, the backend is ready for testing, please let me know when you're done updating to latest master.

@mbartenev-atypon, @mnatsheh, @hhusban
This is ready for review, it's also deployed on dev1 if you'd like to test it.

Can someone please review this?

rerun

rerun


AUI-1619 Cannot set zIndex on Tooltip

Hey @mairatma
This is a back port of an AUI fix for portal of branch 6.2.X.
This issue is fixed for master as it is merged in AUI master and 3.0.x.
See the fix: https://github.com/eduardolundgren/alloy-ui/pull/761.
I updated the test class since it failed, according to https://github.com/liferay/alloy-ui/blob/master/src/aui-tooltip/tests/unit/js/tests.js#L163.
Please help review this issue
Many thanks
John.
/cc @jonmak08

Just started reviewing :) :octocat: Sent from GH.

Thanks, this has been pushed to the 2.0.x liferay/alloy-ui branch

Hey @zxdgoal,
I will push in a new AUI zip and create a new release for this issue today.
Please let me know if you have any questions.  Thanks!

Hey @zxdgoal,
I created a new AUI release with this issue at https://github.com/liferay/alloy-ui/releases/tag/<IP_ADDRESS>-deprecated.
Please let me know if you have any questions.  Thanks!


fix: do not mark a node as awake when it sends a NonceGet

fixes: https://github.com/zwave-js/node-zwave-js/issues/5854

@zwave-js-bot automerge


updated chore provider to implement the assignment service

Made changes. The service provider needs to pass back a chore object for add chore service call and the UI need pass in an list of children's id in for the add chore function.

Last week, I added the following description to the Asana task associated with this PR. Unless you have any objections, I think it would be good to include the functionality I listed there in this PR:

This includes adding the ability to obtain a list of chores assigned to a given person ID, a list of person IDs assigned to a given chore ID, creating new chores with a set of person IDs assigned to them, and changing the set of person IDs assigned to a chore.


Unable to load DLL 'DevIL.dll' error

I had to do a fresh windows 10 install, now i get an error when trying to use vimage.
Unable to load DLL 'DevIL.dll': The specified module could not be found. (Exception from HRESULT: 0x8007007E)``
I have same windows 10 installed as before, same programs, same everything.
any suggestions please?
EDIT
issue resolved itself after i installed photoshop

Weird. I wonder if DevIL needed some sort of redistributable I didn't know about. If someone else has this problem try installing the Visual C++ Redistributable: https://www.microsoft.com/en-au/download/details.aspx?id=48145


setup: make setup work on windows

CI's failing on a dumb f-string error, but looks great otherwise.

Thanks James - I'll have to figure out how to run the various tests on my machine.
I fixed the flake8 issues

Great, thanks @billygarrison. (Apologies for the delay, going to be able to be more responsive going forward.)


updating to py 3.7.4

IBM Cloud toolchain: Delivery Pipeline deployed long-way-home-callforcode to chatbotSpace, including commit c618aa5e910cc5d9a7f60aa844e23b644d6977eb


per-host-subnet hostgw mode cannot work when agent hosts use public IP

We use host agent_ip to build routes, the public IP is implemented through NAT on some cloud, so routing will fail if we use public IP.
We will get this error:Network is unreachable..
This problem can be reproduced on AWS.
We can introduce a host label to define the routing IP, e.g.  io.rancher.network.per_host_subnet.agent_ip

Released: https://github.com/rancher/per-host-subnet/releases/tag/v0.1.1.
@niusmallnan can you please make a PR for rancher/net with the above version?
And also another PR for rancher/catalog? Next version of rancher/net would be v0.13.2

Rancher-net v0.13.3 released. Per-host-subnet has been updated. Ready to test.

Tested with rancher-server version- v1.6.11-rc5 with rancher/net:v0.13.3
Environment with 2 hosts.
The host adds label io.rancher.network.per_host_subnet.override_agent_ip=private IP, Test passed.


Android Basics: Fragments Navigation component

URL of codelab
https://developer.android.com/codelabs/basic-android-kotlin-training-fragments-navigation-component?authuser=1&continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-kotlin-unit-3-pathway-2%3Fauthuser%3D1%23codelab-https%3A%2F%2Fdeveloper.android.com%2Fcodelabs%2Fbasic-android-kotlin-training-fragments-navigation-component#3
In which task and step of the codelab can this issue be found?
Task No. 4
Step No. 3
Describe the problem
In the screenshot for fragment_letter_list.xml, the tools:context should =".LetterListFragment" instead of ".WordListFragment"
Additional information
Include screenshots if they would be useful in clarifying the problem.

Duplicate of #48

@hassan0599
Thank you so much for reporting this issue. This is fixed and will be published soon.


jquery wrapper broken

havnt really gotten a chance to take a look at it.  broken in the demo page

closing this: jquery slated to be removed


854 fw debug prints

Firmware debug infos will be printed to the console if log level is debug.
Fixes #854

Hi Sebastian,
sorry, I tried to change it but it fails.

I have no idea how to change it :(
Also how to add that it is coming from the firmware.
Regards
Ralf

Hi Sebastian,
sorry, I tried to change it but it fails.

I have no idea how to change it :(
Also how to add that it is coming from the firmware.
Regards
Ralf

debug is no variable and can't be substituted. Remove the curly braces from {debug}.


Documentation Clarification regarding dynamic URLs

Hi,
I have added this package to our Umbraco 11 app, and am successfully able to add a QR code to a Document Type and have this render within the template on the front end.  I currently have the Source provider set to Absolute URL - and this results in the generate QR code referencing the URL of the page on which it is placed.
(I did find that the example given: https://github.com/sebafelis/RoboLynx.Umbraco.QRCodeGenerator#using-on-frontend-page did not work for me - I had to call @RoboLynx.Umbraco.QRCodeGenerator.Frontend.UrlHelperExtensionsForQRCodeGenerator.GetQRCodeUrl(Url,qrCode) - rather than just @Url.GetQRCodeUrl(qrCode) - presumably there's some other reference that I have missed here that I could not see in the documentation?)
I would like to update the URL which is referenced, to be suffixed by a hard coded value (either a query string or /path) so that the QR code points to {pageURL}/path.
I've looked at the documentation under https://github.com/sebafelis/RoboLynx.Umbraco.QRCodeGenerator#a-content-property - and I presume this is how I would configure this, setting the url argument dynamically... however I'm not clear on the necessary syntax in order to achieve this...
Would it be possible to provide feedback as to how this can be achieved - and maybe give slightly more detailed examples in the documentation which can be followed for those who are completely unfamiliar with the details?
Thanks.

Sorry for that late answer, but somehow I missed it notification about issue and message on linked.in.
Currently Absolute URL data source gets only current node URL (URL of document or media where QRCodeGenerator property is add). This data source will be extended and new one will be added in near future.
At this moment you have to two way:

Most simple solution, but probably not enough for you:

You can simply create another property in the same document type where qr code is add.
Property should be of Textstring type. Name them i.e. myUrl.
Next go your qr code data type settings and set Source provider to Content Property. In Source provider settings field enter name of the propery what you create (myUrl in this example).
In Content section select document base on type that you modified and enter staticly the whole absolute URL what you want to pass to QR code.

Second solution is more difficult and require to create a custom Source provider.

I don't know what you exacly wont to achive, but I wrote some example witch I hope it's help you. You need create two class inherit from QRCodeSource and QRCodeSourceFactory. Umbraco durning startup should find them automaticly.
public class CustomUrlSourceFactory : QRCodeSourceFactory
{
	private readonly IPublishedValueFallback _publishedValueFallback;
	private readonly IPublishedUrlProvider _publishedUrlProvider;

	public CustomUrlSourceFactory(ILocalizedTextService localizedTextService, IPublishedValueFallback publishedValueFallback, IPublishedUrlProvider publishedUrlProvider) : base(localizedTextService)
	{
		_publishedValueFallback = publishedValueFallback;
		_publishedUrlProvider = publishedUrlProvider;
	}

	public override string Id => "CustomUrl"; // Unique ID

       // Description and Name property don't need to be override.
       // If you don't do this Name and Description will be geting from language file where you should add the follow tags:
       //<area alias="qrCodeSources">
       //   <key alias="customUrlSourceName">Custom URL</key>
       //   <key alias="customUrlSourceDescription">Description showing in blue border in data type configuration at Umbraco Backoffice.</key>
       //</area>
       // More about languange file => https://docs.umbraco.com/umbraco-cms/extending/language-files#user-language-files
	public override string Description => "Description showing in blue border in data type configuration at Umbraco Backoffice.";

	public override string Name => "Custom URL";

	public override IQRCodeSource Create(IPublishedContent publishedContent, string sourceSettings, string culture)
	{
		return new CustomUrlSource(_publishedUrlProvider, _publishedValueFallback, publishedContent, sourceSettings, culture);
	}
}

/// <summary>
/// Object gets an URL of current node (document or media) and add value of the other property from the same node to the end.
/// </summary>
public class CustomUrlSource : QRCodeSource
{
	private readonly IPublishedUrlProvider _publishedUrlProvider;
	private readonly IPublishedValueFallback _publishedValueFallback;
	private readonly IPublishedContent _content;
	private readonly string _sourceSettings;
	private readonly string _culture;

	public CustomUrlSource(IPublishedUrlProvider publishedUrlProvider, IPublishedValueFallback publishedValueFallback, IPublishedContent content, string sourceSettings, string culture) : base()
	{
		if (content is null)
		{
			throw new ArgumentNullException(nameof(content));
		}

		if (content.ContentType.ItemType == PublishedItemType.Member ||
			content.ContentType.ItemType == PublishedItemType.Element ||
			content.ContentType.ItemType == PublishedItemType.Unknown) // Only documents and media has an URL
		{
			throw new InvalidSettingQRCodeGeneratorException("codeSource", $"This source provider does not support {content.ItemType} nodes");
		}
		_publishedUrlProvider = publishedUrlProvider ?? throw new ArgumentNullException(nameof(publishedUrlProvider));
		_publishedValueFallback = publishedValueFallback ?? throw new ArgumentNullException(nameof(publishedValueFallback));
		_content = content;
		_sourceSettings = sourceSettings;
		_culture = culture;
	}

	public override T GetValue<T>(int index, string key)
	{
		var property = _content.GetProperty(_sourceSettings); // _sourceSettings is value of Source provider settings field. In this case should be name of Umbraco Document property containing the rest of URL address what you want to pass.
		if (property == null)
		{
			throw new SourceConfigurationQRCodeGeneratorException(GetType(), "Property not found.");
		}

		var urlSufix = property.Value(_publishedValueFallback, _culture);
		if (urlSufix is not string) { // In this case only property types that return string are support (e.g. Textstring). If you need more complex value type you need to get a correct value.
			throw new SourceConfigurationQRCodeGeneratorException(GetType(), "Property value is not string.");
		}

		string url;
		switch (_content.ContentType.ItemType)
		{
			case PublishedItemType.Content:
				url = _publishedUrlProvider.GetUrl(_content, UrlMode.Absolute, _culture) + urlSufix;
				break;

			case PublishedItemType.Media:
				url = _publishedUrlProvider.GetMediaUrl(_content, UrlMode.Absolute, _culture, UmbracoCoreConstants.Conventions.Media.File) + urlSufix;
				break;

			default:
				throw new NotSupportedException();
		}

		// There is no validation in here. Basic validation of parameters is carried out in next level of qr code creating.
                // Of course you can make more complex  validation in here and throw ValidationQRCodeGeneratorException if value is not valid.

		return (T)Convert.ChangeType(url, typeof(T));
	}
}

I you need more explenation, please write.


[cleanup] Remove mubi4_e and replace it with mubi4_t

also do the same for mubi8/16
As discussed with @msfschaffner, there is a convention to use _t suffix as a type in topgen.
Right now, design and DV mixed-use both mubi4_e and mubi4_t. It's better to remove mubi4_e
to avoid confusing.
If this looks good, will do the same for lc_tx_e/t
Signed-off-by: Weicai Yang<EMAIL_ADDRESS>

lgtm overall. Thanks for making the change.

@tjaychen I think this eliminates the mubi4_e name since it directly names the enum as mubi4_t...

@tjaychen I think this eliminates the mubi4_e name since it directly names the enum as mubi4_t...

Is there a value in keeping an alias for a typedef? Seems like changing the enum to be typedef'ed to mubi4_t and getting rid of the duplicate will solve this confusion once and for all (if _t suffix is desired).

btw, if the intent it to not use mubi4_e anywhere, should we consider taking it out of the prim_mubi_pkg file so that others
don't accidentally reference it?

Agree. That's what I have done in this PR.

I guess there's no concept or private variable here...just wondering if there's any value to try and emulate it.
I guess no way to make the enum type private.


build: disable commitlint in favor of github bot

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Reconciler: Retry failing deletions

Description
A failing cluster deletion should be retried until a maxRetry level is reached. We are already supporting retries for reconciliations (#496 ) - please extend the existing approach and support also retries for deletion.
AC:

Cluster which would retrieve an delete_error state should be marked as delete-error-retryable and deletion should happen again until  the maxRetries threshold is reached.

Reasons
Align reconciler behaviour: retry also clusters which are in delete_error state to behave comparable to clusters which were in error state after reconciliation.
Attachments

Please consider #528


Remove player circle from game on disconnect

Leaving player circles as a dummy seems unintuitive to players and creates many unwanted edge-cases. This makes them disappear immediately.
Also removes duplicate code (registerDeath function) in respawnHandler.

Removing the circle is also unintuitive and creates edge-cases. For example in Passthebomb if the player that currently has the bomb leaves and their circle is deleted the gamemode will breakdown. Maybe implement as a per gamemode option?


A general question : what'r the main advantages in v4 ?

Hello, guys,
As title ... After several comparisons we'r still not clear enough for the main advantages in V4 comparing to v3 ?
Thx a lot,

I'd suggest watching the main video: https://reacttraining.com/react-router or taking the free course: https://reacttraining.com/online/react-router
It's not really a question anyone can answer conclusively, because it depends highly on your use case, but it's now a smaller, more React-idiomatic library. That means less coding specifically for the router, instead fighting the API differences with React. It no longer overwrites or changes React behaviors and the stuff to learn is drastically smaller and simpler, because it's just a set of components.


Update to latest Tika version

Hi,
Can you update to the latest version? Version 1.14 is out since 19 October,
Thanks.

There is a how to on this if you want to make a PR. I would love to get
feedback from someone else doing this to help improve the documentation.
https://github.com/KevM/tikaondotnet/blob/master/Readme.md
On Sun, Dec 4, 2016 at 11:39 AM Kees<EMAIL_ADDRESS>wrote:

Hi,
Can you update to the latest version? Version 1.14 is out since 19 October,
Thanks.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/KevM/tikaondotnet/issues/67, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAGHR8_-wheoZbLtXb2JJSrcNf_TWRoks5rEvrjgaJpZM4LDpJL
.

I have a look into tomorrow... does not seem to be to difficult to do and I did use IKVM more then once :)

I looked into Tika a few years back and then decided to use iFilters --> https://github.com/Sicos1977/IFilterTextReader ... but Tika has made some big improvements last year so looking into Tika again.

Thank you. It should be turn key. It is all automated. Let me know if you
have any pain.
On Sun, Dec 4, 2016 at 11:53 AM Kees<EMAIL_ADDRESS>wrote:

I have a look into tomorrow... does not seem to be to difficult to do and
I did use IKVM more then once :)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/KevM/tikaondotnet/issues/67#issuecomment-264718925,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAGHWFhRG0SrBOasOAlRvZri2zO33VDks5rEv4FgaJpZM4LDpJL
.

Hi,
I did build the new version and tried to push it but that gives me this error:
C:\Users\Kees\Documents\GitHub\tikaondotnet [master ≡ +0 ~4 -0 !]> git push origin --tags
remote: Permission to KevM/tikaondotnet.git denied to Sicos1977.
fatal: unable to access 'https://github.com/KevM/tikaondotnet.git/': The requested URL returned error: 403
C:\Users\Kees\Documents\GitHub\tikaondotnet [master ≡ +0 ~4 -0 !]>

There is also a typo in you code... I think StringExtrensions should be StringExtensions (without the R)

You don't have push access to the repo. Can you create a Pull Request?

typo in you code

Thanks. Go ahead and fix it in an separate PR.

O well that I have to start over again because I now just cloned to my local drive and not to Github.

Also a thing for your build script..
Add this line:
./paket update
Because the paket.lock needs to be updated when I update the link to the new tika jar.

I now just cloned to my local drive and did not do a fork on Github.

You can always fork it on the website and then add the remote manually.

Because the paket.lock needs to be updated when I update the link to the new tika jar.

I think all that is needed is ./paket.exe install (like a nuget restore) which I believe is done for each build.
paket update will update dependencies not simply download them.

Checked and paket restore is being done by the build.cmd. Let me know where the docs could be improved to avoid this ambiguity.

Hi,
Sorry for the delay but I had some other things that kept me busy.
I added this line to your build.sfx file:

Nice automation... seems all went well and the new package is also pushed to NuGet

@Sicos1977 To help everyone could you create a PR for the two things you did?

Update Tika version
Add assembly signing

Derp. You already did. Just in one PR. Thanks.


Question about ES6 import

Hello,
Could this be possible to use one of your project in an ES6 project with just an import ? I don't work in typescript and don't really understand why there is no export feature in your projects, it looks like typescript doesn't need that.
You can close my issue right after your answer. I am just sad not being able to use any of your great projects :(

Nevermind, I found a way.

Could you please say how you solved it? I have the same problem.


Get-Help (linux, ubuntu)

Get-Help on Ubuntu 14.04
$ powershell --version
powershell v6.0.0-alpha.17
Steps to reproduce
PS /home/lj> Get-Help Find                                                          Searching Help for Find ...                                                     [oooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooo]                                                                                Get-Help                                                                           Searching Help for Find ...                                                     [oooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooo]
Expected behaviour
Full help text
Actual behaviour
Seems that text rendering has no LF just CR so text gets overwritten with the new text in the stream.
In essence Get-Help is not usable at the moment as help text do not render properly.
Environment data

PS /home/lj> $PSVersionTable
Name                           Value

PSVersion                      6.0.0-alpha
PSEdition                      Core
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   <IP_ADDRESS>
GitCommitId                    v6.0.0-alpha.17
CLRVersion
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
> $PSVersionTable

I think this is a side effect of a broken progressbar (which was fixed https://github.com/PowerShell/PowerShell/pull/3362).  if you set $progresspreference='silentlycontinue' it renders correctly.

This was not fixed with #3362 and something specific to formatting.  Investigating.

I can confirm that setting $progresspreference='silentlycontinue' appears to fix this problem in Debian Stretch

Thank you!
I got that this was honest mistake on my part, as later I figured out that as usual 'it is not what you are thinking you are saying but what you are actually saying'.
    Get-Help Find

is actually searching for non existent Find (cmdlet or other object) which I would expect to be something existent, or hoped to find 'finding' commands in search engine way. All other Get-Help actually (existing cmdlets help) renders well on Linux.
I did not expect to hit something deeper than that but as I see from the latter discussion it actually uncovered something deeper and as well something for me to learn about PS.
behaviour is the same for all unknown strings
tried: $progresspreference='silentlycontinue'
it works like charm even for Find.
    > Get-Help Find

    Name                              Category  Module                    Synopsis
    ----                              --------  ------                    --------
    Find-Command                      Function  PowerShellGet             ...
    Find-DscResource                  Function  PowerShellGet             ...
    Find-Script                       Function  PowerShellGet             ...
    Find-Module                       Function  PowerShellGet             ...
    Find-RoleCapability               Function  PowerShellGet             ...
    Find-PackageProvider              Cmdlet    PackageManagement         Find-P...
    Find-Package                      Cmdlet    PackageManagement         Find-P...

    > Get-Help mela
    Get-Help : Get-Help could not find mela in a help file in this session. To
    download updated help topics type: "Update-Help". To get help online, search
    for the help topic in the TechNet library at
    https://go.microsoft.com/fwlink/?LinkID=107116.
    At line:1 char:1
    + Get-Help mela
    + ~~~~~~~~~~~~~
        + CategoryInfo          : ResourceUnavailable: (:) [Get-Help], HelpNotFoun
       dException
        + FullyQualifiedErrorId : HelpNotFound,Microsoft.PowerShell.Commands.GetHe
       lpCommand

'mela' is not matching anything. Error makes sense.

@ljgww Get-Help tries to be smart.  It searches for exact match, and if not found, tries to match the title, and if not found, tries matching content before giving up.  If one match is found, it displays it, otherwise shows a list of potential topics.


DropdownMenu menu doesn't follow widget

If you put a DropdownMenu in a scrollable, then cause the DropdownMenu widget to show its menu and then scroll the scrollable (e.g. using a trackpad), the menu doesn't follow the widget. It should use a LeaderLayer/FollowerLayer to follow the text field the same way the text field toolbar does.
(Testing on main, 3.24.0-1.0.pre.104, on macOS 14.2.1 23C71 darwin-arm64.)

Reopening because https://github.com/flutter/flutter/pull/154667 was partially reverted.
See https://github.com/flutter/flutter/pull/158930 for details about the revert.
Marking this issue as P1 because will actively work on a fix.


Points missing after compaction

Bug report
System info:
version 1.2.0, branch master, commit b7bb7e8359642b6e071735b50ae41f5eb343fd42
Docker image: influxdb:1.2.0-alpine with id 3a859cac1ae5
Docker is running inside Docker For Mac 1.13.1 (15353) Build 94675c5a76 on host with macOS 10.12.3
Steps to reproduce:

Start a fresh container: docker run --name influxdb -d -p 8086:8086 -v $PWD:/var/lib/influxdb influxdb:1.2.0-alpine
Create a database named "test"
Create batches (BatchPoints) of 5000 points with timestamps starting at 0 and incrementing to 1,000,000. Batches should be configured to use write consistency = quorum. Each point has 2 tags and 5 fields.
Start 5 threads and send the batches as quickly as possible. I used the influxdb-java client.
Query for a count of points inserted once all requests are complete.

Expected behavior:
Query returns a count of 1,000,000.
Actual behavior:
Query sometimes returns 1,000,000 but often returns a count of slightly less (10-100 less on average).
Additional info:
I've observed the first query return a count of 1,000,000 and a subsequent query after compaction logs return a lower count.
Here's the logs from a bad run where the first query returned the correct count and a query after compaction returned a lower count.
Client logs:
Generated and wrote 1000000 points.
Querying for how many were persisted:
QueryResult [results=[Result [series=[Series [name=jordan|app1|0|1, tags=null, columns=[time, count], values=[[0.0, 1000000.0]]]], error=null]], error=null]

Querying again after waiting a bit:
QueryResult [results=[Result [series=[Series [name=jordan|app1|0|1, tags=null, columns=[time, count], values=[[0.0, 999947.0]]]], error=null]], error=null]

InfluxDB logs:
https://gist.github.com/jganoff/db12b62ece9f7dcb9b2e90ff5a73b4bc#file-influxdb-log
Here's the requested debug information: https://gist.github.com/jganoff/db12b62ece9f7dcb9b2e90ff5a73b4bc
It looks like the missing points are from somewhere near the middle this time though they can be scattered around so don't over index on their locality here:
> select count("value") from "jordan|app1|0|1" where time >= 0 group by time(1m) fill(none)
name: jordan|app1|0|1
time         count
----         -----
0            60000
60000000000  60000
120000000000 60000
180000000000 60000
240000000000 60000
300000000000 60000
360000000000 59970
420000000000 59977
480000000000 60000
540000000000 60000
600000000000 60000
660000000000 60000
720000000000 60000
780000000000 60000
840000000000 60000
900000000000 60000
960000000000 40000

With more resolution:
> select count("value") from "jordan|app1|0|1" where time >= 0 group by time(5s) fill(none)
time         count
----         -----
...
385000000000 5000
390000000000 5000
395000000000 4994
400000000000 4994
405000000000 4994
410000000000 4994
415000000000 4994
420000000000 4997
425000000000 4996
430000000000 4996
435000000000 4996
440000000000 4996
445000000000 4996
450000000000 5000
455000000000 5000
460000000000 5000
...

Would you be able to attach your script that you use to repro this?  Also, can you run influx_inspect dumptsm XXXX.tsm on each of the tsm files in the /var/lib/influxdb/data/test/autogen/2 dir and attach the output?

Here's the tsm dump:
/var/lib/influxdb/data/test/autogen/2 # influx_inspect dumptsm 000000003-000000002.tsm
Summary:
  File: 000000003-000000002.tsm
  Time Range: 1970-01-01T00:00:00Z - 1970-01-01T00:16:39.999Z
  Duration: 16m39.999s   Series: 5   File Size: 825355

Statistics
  Blocks:
    Total: 5005 Size: 684894 Min: 26 Max: 251 Avg: 136
  Index:
    Total: 5005 Size: 140448
  Points:
    Total: 4999735
  Encoding:
    Timestamp:  none: 0 (0%)    s8b: 85 (1%)    rle: 4920 (98%)
    Float:      none: 0 (0%)    gor: 4004 (80%)
    Int:        none: 0 (0%)    s8b: 0 (0%)     rle: 1001 (20%)
  Compression:
    Per block: 0.14 bytes/point
    Total: 0.17 bytes/point

Is there only 1 tsm file?  The stats indicate there should be two.

Only 1 for test.autogen:
/var/lib/influxdb/data # find .
.
./_internal
./_internal/monitor
./_internal/monitor/1
./_internal/monitor/1/000000001-000000001.tsm
./test
./test/autogen
./test/autogen/2
./test/autogen/2/000000003-000000002.tsm

The points might still be in the wal.  Try running influxd w/ INFLUXDB_DATA_CACHE_SNAPSHOT_WRITE_COLD_DURATION=1s influxd.  If they are in the wal, that should cause them to get snapshotted to a new TSM file after 1s.

Here's the test application I've been able to reproduce the issue with: https://github.com/jganoff/influxdb-concurrent-write-test
I'll test with a shorter write cold duration next.

INFLUXDB_DATA_CACHE_SNAPSHOT_WRITE_COLD_DURATION=1s should cause any points still in the WAL to get written to tsm files.   Something is off because that 1 tsm file you ran influx_inspect on only had 4.9M points and the stats indicate there should be 2 files.
After running w/ the above env var, can you see if there are more than 1 tsm files on disk and that the contents of /var/lib/influxdb/wal/test/autogen/wal is empty or just 0 length files?
If there is nothing in the wal, please run influx_inspect again on all the tsm files for that shard.
Thanks for the repro script. I'll see if I can repro as well.

Here's the state of the tsm and wal files after a few runs with INFLUXDB_DATA_CACHE_SNAPSHOT_WRITE_COLD_DURATION=1s:
/var/lib/influxdb # find wal/
wal/
wal/_internal
wal/_internal/monitor
wal/_internal/monitor/1
wal/_internal/monitor/1/_00201.wal
wal/test
wal/test/autogen
wal/test/autogen/11
wal/test/autogen/11/_00006.wal
/var/lib/influxdb # ls -l wal/test/autogen/11/_00006.wal
-rw-r--r--    1 root     root             0 Mar  2 19:33 wal/test/autogen/11/_00006.wal
/var/lib/influxdb # find data
data
data/_internal
data/_internal/monitor
data/_internal/monitor/1
data/_internal/monitor/1/000000017-000000004.tsm
data/_internal/monitor/1/000000193-000000005.tsm
data/_internal/monitor/1/000000197-000000003.tsm
data/_internal/monitor/1/000000201-000000003.tsm
data/_internal/monitor/1/000000202-000000001.tsm
data/test
data/test/autogen
data/test/autogen/11
data/test/autogen/11/000000002-000000002.tsm
data/test/autogen/11/000000003-000000001.tsm
/var/lib/influxdb # ls -l data/test/autogen/11/
total 804
-rw-r--r--    1 root     root        654164 Mar  2 19:33 000000002-000000002.tsm
-rw-r--r--    1 root     root        165996 Mar  2 19:33 000000003-000000001.tsm

And here's the tsm dump for those 2 files:
/var/lib/influxdb # influx_inspect dumptsm data/test/autogen/11/000000002-000000002.tsm
Summary:
  File: data/test/autogen/11/000000002-000000002.tsm
  Time Range: 1970-01-01T00:00:00Z - 1970-01-01T00:13:17.765Z
  Duration: 13m17.765s   Series: 5   File Size: 654164

Statistics
  Blocks:
    Total: 3990 Size: 542118 Min: 26 Max: 243 Avg: 135
  Index:
    Total: 3990 Size: 112033
  Points:
    Total: 3988815
  Encoding:
    Timestamp:  none: 0 (0%)    s8b: 15 (0%)    rle: 3975 (99%)
    Float:      none: 0 (0%)    gor: 3192 (80%)
    Int:        none: 0 (0%)    s8b: 0 (0%)     rle: 798 (20%)
  Compression:
    Per block: 0.14 bytes/point
    Total: 0.16 bytes/point
/var/lib/influxdb # influx_inspect dumptsm data/test/autogen/11/000000003-000000001.tsm
Summary:
  File: data/test/autogen/11/000000003-000000001.tsm
  Time Range: 1970-01-01T00:13:17.766Z - 1970-01-01T00:16:39.999Z
  Duration: 3m22.233s   Series: 5   File Size: 165996

Statistics
  Blocks:
    Total: 1015 Size: 137250 Min: 26 Max: 158 Avg: 135
  Index:
    Total: 1015 Size: 28733
  Points:
    Total: 1011170
  Encoding:
    Timestamp:  none: 0 (0%)    s8b: 0 (0%)     rle: 1015 (100%)
    Float:      none: 0 (0%)    gor: 812 (80%)
    Int:        none: 0 (0%)    s8b: 0 (0%)     rle: 203 (20%)
  Compression:
    Per block: 0.14 bytes/point
    Total: 0.16 bytes/point

@jganoff I'm not able to reproduce the issue with the repo you provided.  One thing I noticed in the client code is that it looks like you are using System.currentTimeMillis() via the InfluxDB java client.  Can you verify that the points you are writing do not have the same timestamp?  It looks like only 1 series is getting written so duplicate timestamps will cause fewer points to be stored.
For example,
curl "http://localhost:8086/write?db=test" --data-binary "cpu value=1 0
cpu value=2 0"

Would send 2 points, but only 1 would be stored because the second is overwriting the first.

I'm generating monotonically increasing timestamps starting at 0 and ending at 1,000,000. ConcurrentWriteTestApp.java @ line 80 creates points with a known timestamp value.
 I can usually reproduce it in under 10 runs.

I'm able to repro it now.

@jganoff I have a fix in #8094 for this issue.  If you are able to pull down that PR and test it that would be great.  I can no longer repro the issue using your test script.  Your test repo was really helpful in tracking this down.

I'm running 1,000 iterations of my test against branch jw-8084 (commit af46b29257cfb055d4f23548e89390f4167201ac) now. I'll let you know how it goes.

1000 iterations completed without fail. Looks like you nailed it. Thanks @jwilder!