Error when adding to dependencies

First thank you for this awesome library!
So I have done everything like you say but I get this error :
error: dependency 'SwiftSMTP' in target 'Resumos.nosync' requires explicit declaration; provide the name of the package dependency with '.package(name: "SwiftSMTP", url: "https://github.com/IBM-Swift/Swift-SMTP", .upToNextMinor("5.1.0"))' and now I can't install it

Maybe a bit late, but:
Try to specify a name for the package inside your Package.swift:
dependencies: [
    .package(name: "SwiftSMTP", url: "https://github.com/Kitura/Swift-SMTP", .upToNextMinor(from: "5.1.0")),
],

After specifying the name I could run swift build without problems.


parallel=True not efficient or faster (in some cases)

This is just to summarize an offline conversation:
Problem:
Doing some benchmarking I found running ctapipe-stage1 on a large number of events using 2 cases on a machine with a 6-core (12 with hyperthreading) Intel core i7 coffee lake processor:

standard case (just running it as usual)
setting NUMBA_NUM_THREADS=1

Both analyses finish at the same time (speed is essentially identical)
in case 1, all 6 processors are at 100% the whole time (and the fan kicks in quickly)
in case 2, only one processor is used, and is at around 60-90% (rest is probably IO), no fan

This means that the parallelism used in the numba calls in CameraCalibrator does not help the speed, but rather decreases the efficiency (lots more CPU power used with the same result).
This is likely true because the size of the arrays being looped over (N_pixels * N_samples * N_channels) is actually not very large from a parallelization standpoint, so the act of distributing data to multiple threads has more overhead than the speed increase provided.   It may not be the case on slower processors or those without good vector processing instructions.
Options:

leave things as they are, but recommend running NUMBA_NUM_THREADS=1 ctapipe-stage1 ..., so one can optimize to each processor
just remove the parallel=True from the @njit() decorators so the default is to run in single core mode for these functions.

note this may also depend on which parallelization backend is used  and the options used, so merits some more study (see https://numba.pydata.org/numba-doc/dev/user/threading-layer.html).

Isn't there another explanation, that the CameraCalibrator is not the primary bottleneck in ctapipe-stage1, irregardless of the parallel=True? And just for reference, parallel=True is only used for one method in ctapipe, neighbor_average_waveform. One would need to benchmark this method directly to observe the advantage given. If I remember correctly, it is an improvement, but probably not to a level that a bottleneck is created without it.

It's true it is not the main bottleneck anymore - in fact parameterization is.  But that is the only place I see parallel=True being used.  The problem is not the overall speed, just the CPU efficiency (seems we are just wasting cycles doing parallelization).  However, I'm still not totally convinced that it needs to be removed - I still have to do a bit of testing to see if we just need to add TBB or OMP as a dependency to get it to  be more efficient.

I tested this again just now and it's still the same. Running with many threads completely saturates the machine without any speedup. We should just remove it.


Slow performance and broken tabbing

and have you...

[X] I updated to the latest spin2 version available
[X] I have restarted VSCode and the problem still exists

Version of Spin2 Language Server VScode Extension with the issue
I think it started happening in the last version?
Is this a regression?
Yes. This used to work, but now it doesn't.
Hardware, Operating System, Operating System version, VSCode version
Hardware:
OS Ver:
VSCode Ver: VSCodium 1.85.1 (yea I should probably update it one of those days)
Description of problem
Since the last version or two, there have been issues where the language server is extremely slow to respond and the semantic highlights will stay outdated for multiple seconds and the whole editor becomes somewhat sluggish. Also sometimes the symbol information disappears.
Also, at some point the tabbing stopped working at all. I have no idea.
Could also be because the file I'm mostly working on is getting really long now.
Additional information
Very long file I'm currently working on:
misoyume_lower.spin2.zip

I"ll give the highlighting of unused code a workout and see what we find.  Also tabbing.  More soon

I found the cause for the broken tabbing... fixes will appear in the next release.

Something I notice when there are slowdowns is that it appears that all processing freezes for a bit and then it quickly runs through all the changes that have happened in the meantime.


bot ran out of space

Command that caused the error
@esmBot  caption It's a 50 NCR minimum to withdraw from your Neos wallet. I would recommend just finding someone you trust and selling it for paypal instead.
Image that caused the error
Message: Magick: unable to extend cache 'It<redacted>apos;s a 50 NCR minimum to withdraw from your Neos wallet. I would recommend just finding someone you trust and selling it for paypal instead.': No space left on device @ error/cache.c/OpenPixelCache/3900  Stack Trace: undefined
Steps to reproduce
use the cap cmd
Self-hosted instance?
No
Error file

Uploading error.txt…
Additional context
No response

Seems there was something up with the system logs filling up a ton of space on one of the servers, should be fine now.


Aggregation support via the find method

Feature request
Context
engine.find allows full flexibility to query anything matching the read operation query syntax, via the
*queries parameter, by allowing to pass QueryExpression along raw queries (dicts).
engine.find uses collection.aggregate internally, and therefore can allow an equal flexibility to use any aggregation stage matching the aggregation pipeline syntax
But, the function limits the use of aggregation stages to the ones defined explicitly via the function parameters:
limit, sort, skip -  for the $limit, $skip and $sort aggregation stages.
And *queries - which are used inside the $match aggregation stage.
Solution

Support all aggregation stages, with a new aggregations parameter.
Add the expressions in the aggregations parameter to the pipeline outside the $match stage.

For extra convenience, create AggregationQueryExpression, to support simple creation of aggregation stages.
Ones not implemented will be passed to aggregations as raw queries (dicts).

Add more aggregation operators as QueryExpression.

Alternative solutions
Implement the db.collection.find() method in engine.find, and create an engine.aggregate method for aggregations related operations.
Additional context
This will allow the user to use implemented odemantic functions for aggregation, reducing the need to use Raw query helpers

I would like to contribute this feature, can you assign it to me?

Agreed! I need it


Client Memory

Clients don't release received responses from the nodes and memory is getting leaked.

any updates here?

?
On Fri, Mar 5, 2021 at 6:42 PM Sajjad<EMAIL_ADDRESS>wrote:

any updates here?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/resilientdb/resilientdb/issues/11#issuecomment-791846454,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AALAUYR3XTVZYLIYEGNDJALTCGI75ANCNFSM4Q33V5EQ
.

--
Best Regards,
Mohammad Sadoghi, PhD
Assistant Professor
Exploratory Systems Lab (ExpoLab)
Department of Computer Science
University of California, Davis


Distributed Tracing in Sentry

#655 changes a setting to get tracing working correctly in the backend.
We might want to have some more configurability on the tracing/profiling rates- via more environment variables, as in news-search-api?
Also wondering about configurability of frontend tracing- right now the SENTRY_DSN and related sentry init in react is all hard-coded.

As far as configurability goes: the sentry initialization happens in root.jsx, which doesn't receive template varables from the backend. One easy change would be to move that around.
The other piece of this is to investigate if we want more intentional instrumentation, to limit the scope of the tracing spans we use.

This is done!


[Suggestion] Move to Telegram instead of WeChat

I feel like Telegram will be a lot more accessible as compared to WeChat.

I thought it would be better to have something more interactive.
It's ok.


fix quaternion integration equation in backend np

Issue: when using backend=np, change yaw (nonzero) and position will make the attitude instable.
Reason: this is caused by incorrect quaternion integration equation:
rowan.calculus.integrate(self.state.quat, self.state.omega, dt),
which is defined for quaternion integration equation using global angular velocity, see Eq. 194. However, self.state.omega is defined in local frame.
Solution: to fix the issue, the local angular rate self.state.omega needs to be converted into the global frame using rowan.rotate, see Eq. 204.

The CI has one failure:
      =================================== FAILURES ===================================
      _________________________________ test_flake8 __________________________________

          @pytest.mark.flake8
          @pytest.mark.linter
          def test_flake8():
              rc, errors = main_with_errors(argv=[])
      >       assert rc == 0, \
                  'Found %d code style errors / warnings:\n' % len(errors) + \
                  '\n'.join(errors)
      E       AssertionError: Found 1 code style errors / warnings:
      E         ./crazyflie_sim/backend/np.py:113:10: W291 trailing whitespace
      E       assert 1 == 0

      test/test_flake8.py:23: AssertionError
      ----------------------------- Captured stdout call -----------------------------

      ./crazyflie_sim/backend/np.py:113:10: W291 trailing whitespace
              #
               ^

      1     W291 trailing whitespace

      12 files checked
      1 errors

      'W'-type errors: 1

Could you fix that @penghuanggp, please? Thanks for your contribution.

Thanks!


Wrapping long commands

break the long commands in docs/ and examples/
I created a munger to check if there are overlong command. I will improve it a bit and send another PR for the tool.
@thockin @lavalamp

This almost certainly is not going in tonight, FWIW.

superceded by #11784. Close now.


Pass a "nested.property" to Validator.validateProperty

I'm submitting a feature request
Current behavior:
Given a Rule with a nested property accessor (like "some.nested.property"), the accessor is treated as a single property rather than nested properties, e.g. obj["some.nested.property"]
Expected/desired behavior:
The property to be accessed like so: obj.some.nested.property
This is my proposed solution (line 108 for the function call; the function itself is at the bottom): https://github.com/fkleuver/validation/blob/master/src/implementation/standard-validator.ts
What is the motivation / use case for changing the behavior?
When dynamically generating validation rules from arbitrary JSON, it's not always possible/practical to stick to direct property accessors. It appears to be relatively simple to implement, but correct me if I'm overseeing something!

#283


chore: migrate to GitHub Actions for CI

Use headless browsers instead of xfvb where possible.
Replace commitlint-travis with usage of the base commitlint as there is no official CLI for GitHub Actions.
Remove CI badges as GitHub displays the CI status natively for each commit.

Remaining work (for which I don't have permissions):

[x] Enable GitHub Actions for the repository 🤔
[ ] Land https://github.com/karma-runner/integration-tests/pull/15 and update this PR to use upstream for integration tests.
[ ] Copy all relevant environment variables from Travis settings to repository secrets: https://github.com/karma-runner/karma/settings/secrets/actions. Note that secrets are available to branches in the repo, but not available to forks, so GitHub secrets provide the same functionality and security as Travis used to.

[ ] BROWSER_STACK_USERNAME
[ ] BROWSER_STACK_ACCESS_KEY
[ ] SAUCE_USERNAME
[ ] SAUCE_ACCESS_KEY
[x] NPM_TOKEN
[ ] GH_TOKEN

[ ] Remove/disable Travis/AppVeyor?

Why do we want to use headless browsers instead of xvfb?

Remove/disable Travis/AppVeyor?

I have deleted the Travis webhooks. Can you clarify what is missing for Travis?

I have deleted the Travis webhooks. Can you clarify what is missing for Travis?

I'll open a new PR to confirm that Travis and AppVeyor are not triggered to see if that's enough or we need to do something else.

Shouldn't we use BROWSER_STACK_ACCESS_KEY instead?

I don't understand. What exactly do you mean by this?

Where and how are these used?

SuaceLabs tokens are used in the integration tests, which are run by npm run test:integration.

Just to confirm, this is only used in the "Release" workflow by the npm run semantic-release step, right?

That's correct.

I think this is not needed :)

That's actually used by update-docs.js to update documenation when release is cut. I would suggest putting the same token as we have in Travis to secrets as it should have push access to the documentation website repo. Or alternatively, we can generate a new token specifically for that repo and store it under a different name (DOCS_GITHUB_TOKEN or something). Let me know if you would prefer the second solution?


[BUG] bloxstrap not launching up after this pop up

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.

What problem did you encounter?
my pc act weird today and when i try to load up roblox this error came up
"System.BadImageFormatException: Could not load file or assembly 'System.Security.Cryptography.Algorithms, Version=<IP_ADDRESS>, Culture=neutral, .The file or directory is corrupted and unreadable. (0x80070570)" (i deleted the pubilckeytoken)

the file that causing the problem

Assuming that the issue has been resolved. Please respond if there is still a problem.


aws: fix test flake when no IMDS can be found (#35633)

Commit Message: aws: fix test flake when no IMDS can be found
Additional Description: Lack of IMDS (<IP_ADDRESS> address) can cause a race condition and crash during testing due to the shutdown of cluster manager. This scenario should not occur normally, as cluster manager will still exist and the lack of IMDS is handled gracefully.

hi @dcillera - thanks for this - does this only affect the 1.31 branch or do other branches also need the fix?

hi @dcillera - thanks for this - does this only affect the 1.31 branch or do other branches also need the fix?
No, at the moment only 1.31 is affected as the failing test (TestWithTwoClustersStandardInstanceProfile) is not present in previous releases.

just for completeness, this is a backport of https://github.com/envoyproxy/envoy/pull/35633

/retest


Cannot import zip files from incoming directory.

If I want to deliver heavy zip files from my site, currently using the incoming directory to speed upload will not due.

I guess .zip files are rejected by concrete5 altogether.

Just add the .zip extension to the Allowed File Types dashboard page
https://documentation.concrete5.org/editors/dashboard/system-and-maintenance/files/allowed-file-types
On Fri, Jun 16, 2017 at 3:39 PM, ob7<EMAIL_ADDRESS>wrote:

I guess .zip files are rejected by concrete5 altogether.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/concrete5/concrete5/issues/5571#issuecomment-309152742,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAgNwW0ueO9tIsWkngYXA2byhIT8fgZhks5sEwQrgaJpZM4N87uF
.


Having problems with saving my Homebrewery file

Error: Bad Request
Error: Bad Request
    at Request.<anonymous> (https://homebrewery.naturalcrit.com/homebrew/bundle.js:71842:19)
    at Request.Emitter.emit (https://homebrewery.naturalcrit.com/homebrew/bundle.js:1768:20)
    at XMLHttpRequest.xhr.onreadystatechange (https://homebrewery.naturalcrit.com/homebrew/bundle.js:72151:10)
{
  "status": 400,
  "method": "POST",
  "url": "/api/newGoogle/"
}

So, I've been trying to save my new file and even after reloading the website many times it just doesn't save :( how can I fix that?

Check out discussion #1580 for a solution. If those steps don't work, swing back here.

Duplicate of #1734

Closing this issue due to no activity, but if the steps in #1580 didn't help resolve the saving issue let us know here and we can reopen this.


Add solrbot dependency upgrades to CHANGES for 9.2

Here I move all dep upgrade changes lines to a new section, and add all the PRs filed by solrbot. These do not have JIRAs but are identified by PR#123.
There were multiple upgrades of same dependency in 9.2, in that case I merged them into one line and comma-separated the Jira/PR refs. If anyone feel that they want to add themselves for credit in addition to solrbot for any of these, feel free to do so in this PR.
I modified changes2html.pl to recognize this pattern and add link in HTML version of changes, see http://www.cominvent.com/pub/releasenotes.html#v9.2.0.dependency_upgrades for preview
I also modify addVersion.py so that the Dependency Upgrades section is auto generated when a new version is added.

Lucene uses GITHUB#123 for their CHANGES github links. But I like PR#123 better as it is shorter and should be well understood.
In the changes2html script I added support for both GITHUB#123, GH#123 and PR#123. I also considered a plain #123 or (#123), which is a pretty standard way of linking to github issue or PR. I really like the plain #123 best, as it should be well understood by developers too. However, as long as we have both JIRA and GH it may be ambigous to read "number 123" and assume it is Github.
Shout out if you have strong opinions either way.
@HoustonPutman may will cause some merge conflicts for other in-flight PRs, and will also need syncing with branch_9x, which happens to be a release manager task that needs doing anyway. So if you want to take over this PR and finalize it as part of your RM task, that is OK with me.

Do you want me to complete this @HoustonPutman or will you merge it as part of your other rm work with CHANGES ?

Sorry I missed this somehow, I will merge it before starting the RC!


Removing broken check for height auto-sizing

I implemented this feature originally.
Not sure exactly what happened, probably some squashing issue, or something changing over the years, but without removing these elements, the font always scales down as low as possible, rather than just low enough to fit the text into the frame in the view.
Once they're removed, it works properly, and functions just like the native iOS adjustsFontSizeToFit.
With these changes, this is what we ship to Production. @shergin would you mind chiming in here?
Motivation
This has a bug where the text shrinks forever without these changes.
Test Plan
We ship this to production.

Feel free to open new PR. ❤

@shergin I haven't gotten a chance to checkout your update and get it running. I did however take a real deep dive into getting this to work on Android.
It's actually super easy to add it to a TextView. ReactTextView just needs to inherit from AppCompatTextView, and the RN version of the Android Support Library needs to be updated.
Here's the gist of it:
//ReactTextAnchorViewManager.java
  @ReactProp(name = "adjustsFontSizeToFit", defaultBoolean = false)
  public void setAdjustsFontSizeToFit(ReactTextView view, boolean enabled) {
    view.setAdjustsFontSizeToFit(enabled);
  }

  @ReactProp(name = "minimumFontScale", defaultFloat = 0.1f)
  public void setMinimumFontScale(ReactTextView view, float scale) {
    view.setMinimumFontScale(scale);
  }

//ReactTextView.java
  public void setMinimumFontScale(float scale) {
    if (this.mMinimumFontScale != scale) {
      this.mMinimumFontScale = scale;
      this.updateFontScaling();
    }
  }

  public void setAdjustsFontSizeToFit(boolean enabled) {
    if (this.mAdjustsFontSizeToFit != enabled) {
      this.mAdjustsFontSizeToFit = enabled;
      this.updateFontScaling();
    }
  }

  public float getMinimumFontScale() {
    return (this.mMinimumFontScale != 0) ? this.mMinimumFontScale : DEFAULT_SCALE;
  }

  public void updateFontScaling() {
    if (this.mAdjustsFontSizeToFit) {
      int min = (int)(this.getTextSize() * this.getMinimumFontScale());
      int max = (int)this.getTextSize();

      TextViewCompat.setAutoSizeTextTypeUniformWithConfiguration(this, min, max, 1, TypedValue.COMPLEX_UNIT_PX);
    } else {
      TextViewCompat.setAutoSizeTextTypeWithDefaults(this, TextViewCompat.AUTO_SIZE_TEXT_TYPE_NONE);
    }
  }

Buuuuuuutttt, I think maybe the ReactTextView isn't actually what's drawn on screen? I get how the custom text drawing works on iOS, but I still don't fully understand this on Android.
I assumed if I could get access to the ReactBaseTextShadowNode from the ReactTextView or ReactTextAnchorViewManager, I could just set mFontSize over there and tell it to redraw. But I couldn't figure out how to do that. If you point me in the right direction, we'll probably finish adding it.

@MattFoley "TextViewCompat.setAutoSizeTextTypeUniformWithConfiguration" don't work in rn,right?


[USERSNAP] teste

Sender<EMAIL_ADDRESS>Taken at: https://test-sanchonet.govtool.byron.network/dashboard
Fields:

Add a comment: teste

Browser: Chrome 123 (Android 6.0)
Screen size: 1846x708
Open #46 in Usersnap
Powered by Usersnap.

@kickloop I can see no information on this bug. When I open it in usersnap, nothing.


dbs-arch: arm add gic and cpu registers releated feature

This PR adds features related to ARM GIC and CPU registers, including VIRTUALIZATION of GIC and vcpu startup related configuration.

dbs-arh: arm add gic and regs dependent library
There is a typo for the commit title

dbs-arh: arm add gic and regs dependent library There is a typo for the commit title

done


[Feature]: Hide env dropdown in view mode if there's only one env enabled

Is there an existing issue for this?

[X] I have searched the existing issues

Summary
Proposed changes for visibility of env switch dropdown

for edit mode, we’ll show dropdown if user has access to one or more envs.

In case of one env access, the dropdown will be disabled

In view mode

Dropdown is visible and enabled only if user has access to two or more envs
If the user has access to only one env and that env is production, the dropdown will be hidden
If the user has access to only one env and that env is the default env(in this case production), the dropdown will be hidden
If the user has access to only one env and that env is not the default env(in this case staging), the dropdown will be shown but disabled

Why should this be worked on?
The env dropdown in view mode is not helpful in cases when only one env is there so we need to hide it.

I would also like this feature. I don't want to confuse our viewer users with that bar!

Hey @geekyme-fsmk @kostyay
This feature is in progress and will be available asap. Please follow this issue for the updates.

I think if the app is publicly available, no selection of env switch dropdown is needed. No footer is needed in public apps.
Thank you! 🙏

Same here. Hoping this feature is released soon! My users are getting confused by the env bar.

Same here.

This issue has shown up again, reopening it.

Issue is fixed for view mode for viewers


Update README

@ohmaddi can you update the README to mention that you are a co-author as well?

Done! Thank you.


Need new x-twitter icon in font awesome icon

Need new twitter icon in font-awesome.

You can achieve that by:
npm install @fortawesome/fontawesome-free --save-dev

Making sure that the version of @fortawesome/fontawesome-free is 6.4.2, or you can do that by
npm update @fortawesome/fontawesome-free

then
import { subsetIconfont, FaFreeProvider } from 'subset-iconfont';

const fa = new FaFreeProvider(['x-twitter']);

subsetIconfont([fa], './outputDir', { formats: ['ttf', 'woff2'] }).then(
  (result) => {
    console.log('Done!');
  }
);

That's it.

Wow. It works. You are awesome.Thanks a lot. We can close this issue.


Support for apps to configure cea708 maxCount

When filing a feature request:
Replace the content in the sections below.
[REQUIRED] Use case description
Some streams have malformed closed caption data that don't configure the CC window with the correct rowCount or rowLock bits.  This causes the CC data to render up to 15 rows (as currently configure in Cea708Decoder.CueInfoBuilder.MAXIMUM_ROW_COUNT = 15.
It would be ideal if the app can configure the MAXIMUM_ROW_COUNT as needed.
Proposed solution
A new API to allow the App to configure the MAXIMUM_ROW_COUNT used by Cea708Decoder.
Alternatives considered
It is possible to provided a custom Cea708Decoder using a custom RenderersFactory, SubtitleDecoderFactory and Cea708Decoder but it is a lot of code to maintain just to override a configuration value.

This seems reasonable, feel free to send a PR adding a new Cea708Parser constructor (we recently renamed this class on the main branch) that takes int maxRowCount. This can then be used by an app writing a custom SubtitleParserFactory.


Add DaysLeft method

When storing key information data in a file, the TimeLeft field will not be updated because it is dependent on SKM. Thus, a days left method should be added because it would allow to check the time remaining in offline mode.

New methods added: https://github.com/SerialKeyManager/SKGL-Extension-for-dot-NET/commit/e567af07ff79b14339a4e33fbbdea28624c4c304


Bunyan logger problems

I am attempting to upgrade from 2.x to 3.0.1 and have encountered several issues with the application insights shim.  Among them are differences in how our trace/error logging is impacted.

Trace messages are "flattened".  Here is what I mean...

with 2.x we see this form...

Errors logged with exception objects are not rendered at all in either traces or exceptions.  Since the shim does not support enableLoggerErrorToTrace I didn't expect it to show up in traces, but it does not even show up in exceptions.  Here is an example of it showing as a trace using 2.x

..again with 3.x no entries are found anywhere when this same error is logged.

Regarding the issue with "flattened" traces. Can you share your call to the Bunyan logger creating that record? The logs you're comparing between 2.X and 3.0.1 don't appear to be the same, so I'm trying to understand what the original structures look like so I can better understand what behavior you're expecting.
Regarding your second issue with errors, I'm not able to reproduce your issue with not seeing Bunyan error logging. The Telemetry type should be trace if it's coming from Bunyan, but with a severity level of "Error". An example trace from my testing:
.
If you're not able to find these error logs, please share what your Bunyan error log calls in your code look like. I'm happy to test them and take a look. Thanks!

@JacksonWeber Sure!
Here is where we create the logger...
import * as Logger from 'bunyan';

...

let logStreams = [];

if (process.env.enable_log_file === 'true') {
    logStreams.push({
        type: 'rotating-file',
        period: '1d',
        count: 10,
        path: `evolved${process.pid}.log`,
        level: process.env.log_level
    });
}

logStreams.push({
    stream: process.stderr,
    level: process.env.log_level
});

this.log = Logger.createLogger(<Logger.LoggerOptions>{
    name: 'EVolvED',
    streams: logStreams
});

Here is an example of an "info" level call...
this.log.info(
    `Retrieved ${options.length} CPOE order text options for site ${req.params.siteId} with configuration ${req.params.type}`
);

Here is an example of an "error" level call...
this.log.error(
    err,
    `Error finding encounter ${req.params.encounterId} or site ${req.params.siteId} to get encounter with export criteria`
);

@skhilliard Thank you for following up. Regarding the changes in Bunyan trace logging, I understand the changes you're referring to now. As a part of our move to OpenTelemetry we also moved away from our previous diagnostic-channel usage to an OpenTelemetry offering for tracking Bunyan, which explains why some fields are not identical here. Does this new data formatting break implementations on your end?
I also tested your error implementation, and I'm still seeing the logs being produced from these logging calls. The contained error object is not serialized correctly (this has just been fixed in the exporter and will roll out with the next release), but I'm not seeing issues with generation from the SDK. Are you seeing no traces with severityLevel: 3?

@JacksonWeber We do have some alerts and other queries that count on the structure of the messages field, so it'd be ideal if it was the same.
I do have some traces with severity level of 3 but only when I'm not passing in an error object.  E.g. this.log.error('this is an error') vs. this.log.error(err, 'The err here is an Error object')

@skhilliard I don't know that the message field is really "flattened" so much as it just reports the Bunyan's message field as the message attribute in Azure Monitor. For example, if I log a stringified object as Bunyan's message, I get:

so it should still be possible to log any necessary data in objects as a part of your message field.
I've tested further and haven't been able to find any type of object/data I can pass to the log.error() to cause it to not create an output in the traces table. Do you have an example of an error you're using there so I can test with the exact same data?

@JacksonWeber Thanks for delving into this!  For my testing of the error condition, I just intentionally threw an error using an Error object like this:
let error = new Error('fakeError')
this.log.error(error, 'This is a fake error');
Kelly

@JacksonWeber Any updates for this?  Thanks!

@skhilliard Apologies for the delay. I tested your above error with the following code:
let error = new Error('fakeError')

async function setupRoutes() {
    app.get('/', async (req, res) => {
        try {
            bunyanLog.error(
                error,
                `Error finding encounter`
            );
        } catch (error) {
            console.error(error);
        }
        res.send("Hello World!");
    });
}

setupRoutes().then(() => {
    app.listen(PORT);
    console.log(`Listening on http://localhost:${PORT}`);
});

And I get a resultant trace that looks like this:

Are you ensuring that the test you're using gives ample time to the SDK to export the error logs? If you want to make sure, you can await flush() on the default telemetry client.

@JacksonWeber Yeah....I am sure we were giving enough time for the logs to export.  What version of bunyan are you using?  We are using...
    "bunyan": "^1.8.15",
    "bunyan-middleware": "^1.0.2",

@skhilliard I'm using the same versions of both bunyan and the bunyan-middleware package. I also tested with both application insights 3.0.1 and 3.1.0, and both are behaving the same way for me.
shim-ts-sample-app.zip
Please let me know if my sample app is not representative of your scenario in any way!

@JacksonWeber Sorry for the long time since I sent a response....it looks like the other relevant information (including call stacks) are now being embedded in the customDimensions....so I think we can just adjust our queries a bit to work around this...
New:
traces |
where client_Type != "Browser" and severityLevel > 2 |
project timestamp,
app_error=message,
err_message=parse_json(tostring(customDimensions.err)).message,
err_details=parse_json(customDimensions).err,
app_location=client_City,
location_state=client_StateOrProvince,
app_instance=cloud_RoleInstance |
sort by timestamp desc
Old:
traces |
where client_Type != "Browser" and severityLevel > 2 |
project timestamp,
app_error=parse_json(message).msg,
err_message=parse_json(message).err.message,
err_details=parse_json(message).err,
app_location=client_City,
location_state=client_StateOrProvince,
app_instance=cloud_RoleInstance |
sort by timestamp desc
Thanks,
Kelly


Update repositories.txt

I see you have fixed the error reported by the bot. However, Library Manager works exclusively from Git tags, so the bot won't be able to see the fix you made until you create a new release or tag in the library repository.
If you look at the repository contents at the latest tag, you can see that the version the bot is checking still has the problem:
https://github.com/Mirco04/Simple_Robot_Motor_Control/blob/v0.1.0/library.properties#L1
name=Robot

Remember to update the version field in the library's library.properties file as needed before making a release/tag.
Once you have done that, simply mention @ArduinoBot in a comment here to trigger it to run the automated checks again. After those are passing, your library will automatically be added to Library Manager.
If you have any questions or concerns, please let me know.
Regards, Per


Unable to fetch embeded object directly where _id matches

I'm unsure of the correct approach here - I was hoping for an inverse reationship to "embedsMany" and "embedsOne" but can't seem to find it (if it exists).
I have an _id of what is a child object and wish to fetch it directly into a proper child object without iterating through the parent first.
doing something like the following returns empty:
$child = new ChildClass::where('_id', '=', $knownChildId)->first();

Any tips on how to do this would be wonderfully appreciated.
Many thanks!

So I was able to get what I want thought I hope there is a better way.
I implement a function in the child class that finds the instance of the parent which contains the matching child ID... then I filter where _id of the child = $id like so:
function getById($id) {
   return ParentClass::where('child_class._id', '=', $id)->first()->children()->where('_id', $id)->first();
}

Is there a better way to do this?
I'm worried about multiple-sublevels deep as I plan to use embedsmany in this way if possible.
However I can imagine needing to parse through child1.child2.child3.child4._id and then ->children->where('child2.child3.child4._id', $id)` and this solution then seems much less sexy
Your thoughts greatly appreciated

I do have the same problem =/


inline tailwind classes not working

Problem
Currently sveltekit provides 2 templates when using
npm init svelte@next

I attempted to run svelte-add in both templates and it didnt work out of the box
Fix

Sveltekit Demo Project
Here, the project has a app.css which is imported into the __layout.svelte. I added the tailwind directives in this file to make inline tailwind classes working again.

Skeleton project
Here, the project does not have a __layout.svelte. So I

added a global.css file
imported it in a new __layout.svelte

If this issue is caused by a mistake on my part else i would like to try and make a PR for this. thank you.

Related to https://github.com/svelte-add/tailwindcss/issues/42, so I'll close this so the issue is tracked there only.
Thanks for the offer to make a pull request! I'll have to do this though because it's part of the rewrite.


Add files via upload

Name:  joha_baker
Id: a022f2a18379f4f8
Full logs will be submitted

rejected:B3
reason:missing logs for 3 different startup types

rejected:B3
reason:missing logs for 3 different startup types


How to handle connection lost?

I use rxmqtt on android device.
I do something as follows, but I'm not sure problem is solved when connection lost.

mqttConnectOptions.setAutomaticReconnect(true); mqttConnectOptions.setCleanSession(false);

rxMqttClient     .connect()     .retry() //reconnect when exception happened      .andThen(rxMqttClient.subscribe(topics, qoses))     .subscribe();

Hi there,
I hadn't considered this scenario properly so it probably needs investigation. Are you getting any errors when the retry() is not succeeding? Have you had any thoughts about the best approach to resolve it?


crash on mod activation

setup a fresh install of ff7, converted, setup 7h settings based on the usual guide, installed reunion and menu overhaul, when i click activate it crashed with

System.NullReferenceException: Object reference not set to an instance of an object.
at Iros._7th.Workshop.fLibrary.CheckAllowedActivate(Guid modID) in C:\Users\TurBoss\Projetcs\7h\7thWorkshop\fLibrary.cs:line 564
at Iros._7th.Workshop.fLibrary.DoActivate(Guid modID) in C:\Users\TurBoss\Projetcs\7h\7thWorkshop\fLibrary.cs:line 624
at Iros._7th.Workshop.fLibrary.ToggleActivateMod(Guid modID) in C:\Users\TurBoss\Projetcs\7h\7thWorkshop\fLibrary.cs:line 864
at Iros._7th.Workshop.fLibrary.DoModBarAction(pMod pm, ModBarAction a) in C:\Users\TurBoss\Projetcs\7h\7thWorkshop\fLibrary.cs:line 920
at Iros._7th.Workshop.pMod.bActivate_Click(Object sender, EventArgs e) in C:\Users\TurBoss\Projetcs\7h\7thWorkshop\pMod.cs:line 200
at System.Windows.Forms.Control.OnClick(EventArgs e)
at System.Windows.Forms.Button.OnClick(EventArgs e)
at System.Windows.Forms.Button.OnMouseUp(MouseEventArgs mevent)
at System.Windows.Forms.Control.WmMouseUp(Message& m, MouseButtons button, Int32 clicks)
at System.Windows.Forms.Control.WndProc(Message& m)
at System.Windows.Forms.ButtonBase.WndProc(Message& m)
at System.Windows.Forms.Button.WndProc(Message& m)
at System.Windows.Forms.Control.ControlNativeWindow.OnMessage(Message& m)
at System.Windows.Forms.Control.ControlNativeWindow.WndProc(Message& m)
at System.Windows.Forms.NativeWindow.Callback(IntPtr hWnd, Int32 msg, IntPtr wparam, IntPtr lparam)

apparently it was because i hadn't noticed that some of the paths had \

no the back slash. the bottom two fields in config both looked like steamapps\common\Final\ Fantasy\ VII\ and it had actually tried to install the mods to steamapps\common\final\fantasy\vii\


Save & Retrieve fields for saving project social media info

Related to https://github.com/Giveth/giveth-dapps-v2/issues/3466
We'll add some more info to project's data for including social media handles that are associated with a given project, the types of social media handles we can accept are:

[ ] - facebook
[ ] - twitter/X
[ ] - linkedin
[ ] - discord
[ ] - instagram
[ ] - youtube
[ ] - farcaster
[ ] - Lens
[ ] - website
[ ] - reddit

This data is added to the project during the create/edit project flow

There is no change in cerat project page on STG @RamRamez

@maryjaf The backend is merged on staging but the FE is not done yet.

@RamRamez - Is there anyway to test this since it is in the QA column?

@divine-comedian Yes, Maryam can test with Postman, but when FE is merged, @maryjaf has to test it again. Anyway, I can show her how to test it with Postman.

Hello am here to bless 10 people with just $12,000 just say bless me to my dm


Make Build Fails When Building From Release Assets

Pretty new to LocalAI and still learning as much as possible from everyone from this group.
My objective is sertup LocalAI so that i can integrate with Nextcloud instance.
So far i have managed to clone LocalAI from git clone https://github.com/go-skynet/LocalAI  and use luna-ai-llama2 as model with tht following
curl http://localhost:8080/v1/chat/completions -H "Content-Type: application/json" -d '{
"model": "luna-ai-llama2",
"messages": [{"role": "user", "content": "How are you?"}],
"temperature": 0.9
}'
Next step and try to build using 'make build' command and encountered the following error and halted:
go mod edit -replace github.com/nomic-ai/gpt4all/gpt4all-bindings/golang=/root/LocalAI/sources/gpt4all/gpt4all-bindings/golang
make: go: No such file or directory
make: *** [Makefile:229: replace] Error 127
Also double check no such URL exist in https://github.com/nomic-ai/gpt4all/gpt4all-bindings/golang.

Do you have go installed in your system?   What is your go version output?

go version go1.21.1 linux/amd64

@kevinkuan1969 If your on master please go to image 1.40v or 1.30v for now, they are doing a full rework of master right now for 2.0v

After clone ~/LocalAI-1.40.0, perform the BUILD using Docker and Make also failed:
DOCKER

Make

Also tested in version LoalAI 1.30, both build method produced the same errors.

The Docker error looks like .git folder isn't present.  Are you invoking the make build from the LocalAI repo root?  Can you run ls -la to confirm .git is there?
Also, the make error says cmake isn't found.  Can you run cmake --version to check that it's installed?

The Docker error looks like .git folder isn't present. Are you invoking the make build from the LocalAI repo root? Can you run ls -la to confirm .git is there?

Cannot find .gif folder. Yes and invoking 'make build' inside LocalAI folder. Please find below the print-screen of the directory

Also, the make error says cmake isn't found. Can you run cmake --version to check that it's installed?

How did you get the LocalAI code on your computer, did you not git clone? Otherwise you'd have a .git folder. It looks like .git copying enables git status to work: https://github.com/mudler/LocalAI/commit/6d19a8bdb5a58b5ec546ef3fbf3bc4fe93756170. Are you able to run git init to create a .git folder?
Also, just wondering, why are you directly running make build instead of docker compose up?

How did you get the LocalAI code on your computer, did you not git clone? Otherwise you'd have a .git folder. It looks like .git copying enables git status to work: [6d19a8b]

I did not use git clone because the v14 is referring to https://github.com/mudler/LocalAI.git same as master copy.
I used 'wget https://github.com/mudler/LocalAI/archive/refs/tags/v1.40.0.zip' and unzip.

Also, just wondering, why are you directly running make build instead of docker compose up?
Understand from below, we can build either using 'docker build' or 'make build'. I tried both because none is successful. We are ok with either one.

I see per wget, yeah it's a valid use case thanks.  Cc @mudler for .git not always being present.
@kevinkuan1969 what do you think of renaming this issue to be:

make build fails when building from release assets

In the meantime, can you run git init after unzipping?  That will hopefully unblock you, just know the VERSION in the Makefile won't be correct.

@kevinkuan1969 what do you think of renaming this issue to be:

make build fails when building from release assets

No problem and please go ahead. Thanks.

@kevinkuan1969 unfortunately I don't have the permissions to rename your issue, I am not a repo admin

@kevinkuan1969 what do you think of renaming this issue to be:

make build fails when building from release assets

Done and thanks.

Still face the problem when performing BUILD
1. sudo make build

sudo docker build -t localai .

git master build error
touch get-sources
go mod edit -replace github.com/nomic-ai/gpt4all/gpt4all-bindings/golang=/opt/docker/LocalAI/sources/gpt4all/gpt4all-bindings/golang
make: go: No such file or directory
make: *** [Makefile:238: replace] Error 127

i am also getting the same error
PS E:\AI\GPT\LocalAI> make build process_begin: CreateProcess(NULL, uname -s, ...) failed. Makefile:56: pipe: Bad file descriptor process_begin: CreateProcess(NULL, uname -m, ...) failed. Makefile:57: pipe: Bad file descriptor process_begin: CreateProcess(NULL, tput -Txterm setaf 2, ...) failed. Makefile:58: pipe: Bad file descriptor process_begin: CreateProcess(NULL, tput -Txterm setaf 3, ...) failed. Makefile:59: pipe: Bad file descriptor process_begin: CreateProcess(NULL, tput -Txterm setaf 7, ...) failed. Makefile:60: pipe: Bad file descriptor process_begin: CreateProcess(NULL, tput -Txterm setaf 6, ...) failed. Makefile:61: pipe: Bad file descriptor process_begin: CreateProcess(NULL, tput -Txterm sgr0, ...) failed. Makefile:62: pipe: Bad file descriptor process_begin: CreateProcess(NULL, uname -s, ...) failed. Makefile:68: pipe: Bad file descriptor touch get-sources process_begin: CreateProcess(NULL, touch get-sources, ...) failed. make (e=2): The system cannot find the file specified. make: *** [Makefile:235: get-sources] Error 2 PS E:\AI\GPT\LocalAI>
please help me out


iOS: UIMenu very slow with 3D Touch

Could it be possible to disable the 3D Touch feature and leave just the long press, because the menu is very slow when calling it from 3D Touch, is a little bit faster with just long press.
I loved 3D Touch but now it's gone with the new devices, so I give up!
Here's a little screen record!
https://user-images.githubusercontent.com/72861758/107276556-7d2f1700-6a53-11eb-968c-2c31bd9b595c.mp4

With 3D Touch disabled it's the same!
It changes when 3D Touch is enabled and I try with 3DT and long press (is faster with long press)!
I think could be a conflict between the calls of UIGestureRecognizer

We changed how we generate the menu in version 6 so that it works faster now.  I'm going to close this issue, but if you still find it slow after we start testing NNW 6, we can reopen it.

Okay @vincode-io, thank you so much!
I'll let you know if I'll find this in the first test builds.

@vincode-io I don't know if It could help but I've just noticed that this happens only if the UIMenu has to show even "Mark All as Read in…" (so if the folder or web feed has unread articles), otherwise is fast as expected.


Support for react native 0.76

Thank you for writing and publishing this library!
I was wondering if you were planning on supporting the new architecture and making this library work with react native 0.76.

@gtokman for me it crashes on android in a flatlist and on iOS sometimes the images are cut off in a wired way in the list (1 from 200 times).

@philipheinser got it! Can try it out this weekend. Are you yoga 2 as well?

@gtokman I worked around in android by not passing children to the Image.

@gtokman Whatever version comes with 76 no special setup

if you provide the minimal repo it would easy for maintainer


The button retry is far too expensive.

Yesterday we had a bug on 6element and bin couldn't be retrieved. So we changed stuf and pressed many times on the retry button. But this button can be pressed so easily that it costed us 9.5euros:

I think we should at least impose a limit to number of times it can be pressed per minutes.

@iOiurson maybe we could shit that as part of the next update.

I'm surprised.
Of course, we can (and probably must) cap the number of times this button can be pressed, maybe 5 times every 10 minutes or so.
But

wasn't this sensor supposed to be connected by cable ? If so, it shouldn't cost us anything.
how could we have consumed 9 Mb by pressing this button ? It's a simple HTTP request (okay, wrapped within MQTT messages) that most of the times returned empty bin list... and I don't remember pressing it 50 times

Are you absolutely certain this is only due to the sole Retry Button ?

You are right pressing the button cannot be the only cause but maybe it's something around qos 1: N of queries are made and when it finally works pheromons sends N times the result (the big json of the bins)

We need to investigate this further.
From what i remember, we had plenty of error responses from 6element server (due to the http/https automatic switch thing). This means there was no bin list sent whatsoever: the sensor queried, and had a response. I don't see why bin lists would have been sent later on.
What is true though, is that the timeout set on the Retry button (about 5 secondes if i'm correct) can cause such behavior (the bin list received for nothing after the timeout has passed). But i don't believe we're in this case.
Is it possible to have a detailed version of the consumption ? Can we trust the timestamp on the image you shared ?

no we can't have more details, the timestamps are good

Ok, so this is not due to the http/https problem we had yesterday around 7pm.
This was indeed because of requests from pheromon to 6element that didn't have an answer at first (6element was down).
To fix this, 2 options:

cap the use of the Retry Button
fine tune the behavior of pheromon regarding requests from sensors to 6element

The former is probably easier to implement, but might not be generic enough.


Frontend: Allow renaming of partner

We allow to set a name/alias for a partner. Currently the frontend does not support changing the name after the partner was added.
In Partner View it should be possible to edit the name

Involves backend work as well. Requires a  PUT /api/partners/{id} route to send the updated alias

Fixed by #29


Update pam 2022

Which conference does this PR update?

pam 2022

Related URL

https://pam2022.nl/

Thanks for contributing!
To retain the history, new conference entries are expected to be appended below the existing lines.
Could you please revise the PR?


Allow custom semantic-ui builds

Adds two configuration options. One for the path to the dist folder of
Semantic. The other option to specify the theme used for images and
fonts.
Note that this does not add support for importing files outside of the
build tree (e.g. bower_components). One can use bower link for local
projects.

Thanks, I should get a chance to look into these in the next couple of days.

This is awesome , really needed. This will reduce a lot of pain in choosing themse. Please merge

The hesitation was mostly around not having unit tests to verify changes for this and for the future. I had an idea tonight, if we could pull the app import methods into a utilities class, then we could import that class to do the calls and test the results.
@khornberg would you be open to added that to this pull request? If so I can detail out more what I'm thinking

@aaronbhansen Sure, I would like to add tests if you have something in mind

@aaronbhansen I'm not sure what you were thinking; I am not sure how one can test if the proper file is included in a build of ember.  I have thought about how to test the the function that gives us the paths to import.
//index.js

from utils import 'importPaths';

included: function(app) {
    paths = importPaths(app.options['SemanticUI']); // returns an array of strings or objects ['', '', {}, {}]
    paths.forEach((path) => { app.import(path); });
}

//utils/importPaths.js

// here do the if checks for each option and append to array

seems all good ?

@khornberg sorry for the delay in responding, this weeks been crazy busy. A quick overview of my idea was to move the url generation into a utils file and then test that. I should get time this weekend to explain a little more and provide an update.

Ok, no worries. I'll push what I've worked on today.
A few hours ago I realized that the index.js is outside of babel and ember.

@v3ss0n It is ready to test now

CI is failing because contains is deprecated in 2.8 in favor of includes.
https://github.com/Semantic-Org/Semantic-UI-Ember/blob/99086c25a1750e1868f8df4453ea46c39ae53467/addon/mixins/base.js#L38

Sorry for the delay in responding, I think its fine to leave most the code in index.js, I think that separates out concerns better. Mostly what I wanted to test was to pull out the default code and some simple formatting code into a separate util, test that, and then import that util into the index.js to produce the final result. As an example index.js could look like
var generate  = require('./utils/generation');

....
// A check that ensures the options passed in aren't empty, relative to ember isPresent
var theme = generation.default('path', 'theme', [options, defaults]);

if (generate.allow('imports', 'images', [options, defaults])) {
  var imageOptions = {
    destDir: generate.format('assets/themes', theme, 'assets/images')
  };
 app.import(generate.format(path, 'themes', theme, 'assets/images/flags.png'), imageOptions);
}

The generation util has three new methods we can test. Those are
default (method):
Parameters
1. The nested path
2. The property name
3. Any properties to consider defaulting in order
The first argument default finds that exists on the objects passed in is returned. The object could be null or empty string. We just need to check if the key exists on the object.
Now that we have multiple settings, it would be nice to nest them in the object. Have the app settings look like
SemanticUI: {
  imports: {
    fonts: false,
    images: false
  },
  paths: {
    theme: 'something'
  }
}

but we also need to support a flat version for older projects.
SemanticUI: {
  fonts: false,
  images: false,
  paths: {
    theme: 'something'
  }
}

In essence what the default method does is try to first resolve any variables that exist on the full path first, then the fall back, then the default full path, then the default fallback.
The check would be something like

check imports/fonts on options
check fonts on options (if found here, log out deprecation warning)
check imports/fonts on defaults
check fonts on defaults (if found here, log out deprecation warning)

The next method would be
import (method):
Parameters
1. The nested path
2. The property name
3. Any properties to consider defaulting in order
Same functionality and idea as default, but its specifically looking for a true / false value instead of first argument found on the object.
Last method would be
format (method):
Parameters
* An array of values
The reason we allow null or empty strings on default is for here. Lets say that they don't want to put the theme assets under theme. They could do this.
SemanticUI: {
  paths: {
    theme: null
  }
}

The generation would look like
var theme = generation.default(...) // theme is null
var destination = generate.format('assets/themes', theme, 'assets/images');
// result = 'assets/themes/assets/images'
the format method checks each value in the array and if the value is null, undefined, or whitespace, it removes it from the array.
Then we just do a simple [remaining values].join('/') to return the value.
Wrapping up
In the end, if we can create those three methods, and have tests against them, that will allow us to have much more flexibility in the future and start to allow multiple configuration options for anyones needs.
Let me know what you think

@aaronbhansen Ok
Do you have a strong preference of qunit over another testing framework? I've not worked with qunit on node outside of ember. Thus far, I've had to install it separately from ember-qunit to get the binary.
The tests seem fairly straight forward so I think either one would work fine.

I don't necessarily love qunit, but its the default ember path, and its the path of less resistance. I'd rather just use qunit than have two testing frameworks installed.
Here is a helper with the default qunit test for non ember utils.
import { aOrAn } from 'shared/helpers/a-or-an';
import { module, test } from 'qunit';

module('Unit | Helper | a or an');

// Replace this with your real tests.
test('it works', function(assert) {
  let result = aOrAn(['test']);
  assert.ok(result);
});

@aaronbhansen Yep don't disagree. As far as I understand, index.js is not run through babel or run within the context of ember. Are you thinking that that file live in addon/util/ and then when ember tests are run it would also be tested?

Thats correct, looks like the path can be anywhere from the other examples I've seen.
https://github.com/emberjs/data/blob/master/index.js#L110

@aaronbhansen How would the imports work with ember expecting  exports default function generation()... and node expecting modules.export = function... from index.js as require('addon/utils/generation')?
Maybe I'm missing something but those are not working for me.

@aaronbhansen I don't think the import or allow method is necessary. Are they the same thing? allow was in your index.js example but seems to be doing what you described under import.
I've updated the index.js to a working version.
In manual tests of ember build everything seems to check out.

Yeah after looking at it, they aren't needed. Initially I was thinking of it differently, but since all the imports are bools, it doesn't matter. Lets switch over to qunit and I think thats enough to merge in. Then we can update your other pull request on documentation with the new recommend structure.

@aaronbhansen I can certainly try to add qunit to the addon if desired. It would have to be installed since ember-cli-qunit installs qunitjs which is not the same as qunit. From my understanding qunit installs a binary that can be called from the command line. I thought I had it runner earlier but I cannot get to run now in the 15 minutes I tried.
Since we have to add something anyways to the add for these tests what do you think we should do?
Do you happen to know how to add qunit tests in node?
Is there anything I need to add so that tests are run on CI?

I don't think it needs to be installed separately. We test utils just like we are doing in this repo with just ember-qunit installed. I believe that package adds qunit as a dependency

What does computed-sum-by look like? I don't understand how it can be imported. My knowledge of node modules is lacking.
As I understand it, index.js does not have the ability to have import generator from "utils/generator" since it is not transpiled and runs on node 4.  index.js uses module.exports, which I understand to mean that it is a commonjs module, not an es6 module.

It can't import like that, but you can required('addon/utils/generator') in index.js. And as long as it doesn't have any es6 code, it should import just fine. Then on the flip side you could import or require it yet again in the tests to test the code.  The computed sum by does have import statements, thats only because its only used within es6 code, but how generator is today it should be fine.

With that said, if you're stuck, I can merge in and then finish making the changes later this weekend or monday.

After trying a few things, I'm still not getting things to import.
I am curious how this all works out though.
I've rebased to master and a single commit.
From what I understand everything should be fairly easy to make testable with qunit.
I've stuck everything in /utils

@aaronbhansen I've check the Allow edits from maintainers. Not sure how well that will work but hey it's new :)

@aaronbhansen This is passing again and rebased to master.
I tried again to require(something) in an ember test unsuccessfully. If you have an example of how to do what you are thinking please let me know. I don't mind hooking it up that way. I'm out of ideas and I'm not sure how it would work to mix commonjs require and es6 imports.

Thank @khornberg I finally got time to start working on this again last night. Last week got a bit too busy. I'm going to update ember and dependencies, then look at getting this merged in and released this week.

Cool, totally understand

Ok can't wait to test.

Thanks, going to pull this in an finish the changes on master 👍

Awesome , gonna test tomorrow.

@khornberg take a look at this commit https://github.com/Semantic-Org/Semantic-UI-Ember/commit/9cc8eed70ba224916c4dd85907a5d4b958ceb811.
I started looking at it last night and realized that it would probably be easiest if the full path was totally customizable. I think that makes it easier to understand and configure now.
Also, I couldn't get it to work on es6 imports either. It was creating a loop when I did. I figured tape was the easiest and set it up like i've seen on other projects with node-tests

@aaronbhansen Thanks, those changes add clarity; everything seems to work fine with the customization.

Glad to hear it worked. Thanks for all the work you did on it.


Exercise 2 Level 4 : Error when extending

Hi, I am having an error when trying to make SolidCuboid extend Cuboid class , Cuboid is currently implementing Shape3D. However i do not see what the issue is.

what is your Cuboid implementation?

Did you perform javac *.java first? This will show any compilation errors that JShell is not showing

Regarding ur first error, can u show ur implementation for SolidCuboid? It could be u call super() wrongly on initialisation in SolidCuboid.
As for the subsequent errors, I think u can read and understand what those errors are saying. Second error states that this.volume() is not implemented in SolidCuboid, this could be a hint for your first error, it is possible that you did not call super() correctly during initialisation.
Third error states that your method where u put the @Override declarator is not overriding any superclass methods, this one check if you have any syntax issues when u declare ur method in the subclass.
Fourth error also shows that u might have used super() wrongly during initialisation.

Fixed the issue, realized i wasn't opening Shape3D.

Bruh moment


Request for PyPi release

Thanks for working through the updates for Sphinx v4.0.0, just wondering if there could be another PyPi release.
To pitch in and help I can offer a PR with a GitHub Action that would automate the package creation and uploading to PyPi
All the would be needed is a new GitHub Secret called PYPI_API_TOKEN and it can be automatically triggered on GitHub releases
# This yml file will trigger a Github Actions event that builds and upload the
# Python package to PiPy. This makes use of Twine and is triggered when a push
# to the master branch occures. For more information see:
# https://help.github.com/en/actions/language-and-framework-guides/using-python-with-github-actions#publishing-to-package-registries

name: Upload Python Package

on:
  release:
    types: [created]

jobs:
  deploy:

    runs-on: ubuntu-latest

    steps:
    - uses: actions/checkout@v2
    - name: Set up Python
      uses: actions/setup-python@v2
      with:
        python-version: '3.x'
    - name: Install dependencies
      run: |
        python -m pip install --upgrade pip
        pip install setuptools wheel twine
    - name: Build and publish
      env:
        TWINE_USERNAME: __token__
        TWINE_PASSWORD: ${{ secrets.PYPI_API_TOKEN }}
      run: |
        python setup.py sdist bdist_wheel
        twine upload dist/*

Let me know if you think this is worth a PR

@Shimwell It should be available now:
https://pypi.org/project/sphinxcadquery/#history
Can you confirm it works as expected? :blush:

Happy to confirm it is working for me https://paramak--836.org.readthedocs.build/en/836/example_neutronics_simulations.html
Many thanks

@Shimwell Thanks to you! :heart:


High CPU utilization of Executor

I wrote a custom executor with the new "pure" Go bindings. And I am observing the cpu utilization of the executor process to be always ~100% with load average of the server over or close to 1.00
Can you guys test whether it's the same case with your executors too?

@nqn @vladimirvivien ^

@diptanu our test executors are simple and don't have benchmarks yet.  Please share more detail about your executor or paste some sample code here.

@vladimirvivien I tried to use runtime/pprof to get some samples and see which part of the code is spending more time on cpus. But unfortunately the golang cpu profiler is broken on OSX :(

@diptanu Thanks you for pointing this out.  So the issue is on OS X ?
Also, anything you can pin down that could be the cause is appreciated (executor driver creation, executor driver start/run,  driver http communication, etc).
Also, what are you doing in your Executor implementation ?  Anything long-running ?

@vladimirvivien The issue can be spotted both on OS X and Linux.
Yeah, I am monitoring processes once I launch them in the executor which is long running. Basically the process is started on a go routine when then starts another go routine to monitor the newly spawned process.

@diptanu Do you have any more info about this issue ?  Any info on how to replicate this behavior ?

@diptanu any updates?

@jdef I am not able to test anymore since it's not possible to write an executor with mesos-go without specifying an IP and port where the executor can bind. Mesos Slave doesn't set them, so I am not sure how to write an executor with this library anymore.

@diptanu some changes were merged recently that allow for an executor driver to optionally be configured with a BindingAddress and BindingPort. They are not required. If they (and HostnameOverride) are not specified, then the hostname should be looked up and an IP address resolved from that.
As a designer of an executor you are now in charge of the command line flags for things like binding address/port, and using them in the DriverConfig (if you want to). You can also query things like the env variable LIBPROCESS_IP (xref https://github.com/mesosphere/kubernetes-mesos/blob/a0a37467a692e60d5c2315a2127605b8a9e15a95/cmd/k8sm-executor/main.go#L72) to determine binding address.
If you've experiencing buggy behavior with the new implementation, please file an issue for it.
xref: https://github.com/mesos/mesos-go/blob/master/executor/executor.go#L39


feat: add description to contact

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
I just added a description to contact, according solicitation of client.
Fixes # (issue)
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Please also note any relevant details for your test configuration.

[x] Test A
[x] Test B

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] Any dependent changes have been merged and published in downstream modules

Ok!!! very good now!!!


make deploy-local fails if there is no vsecm-system namespace present.

Steps to Repo
Start with a fresh cluster.
Then, execute the following:
make k8s-delete
make k8s-start
eval $(minikube -p minikube docker-env)
make build-local
make deploy-local

Expectation:

vsecm-system namespace should be created
vsecm-system namespace should have Safe and Sentinel up and running w/o any issues.

What happens:
I get the following error log that indicates that the process cannot find vsecm-system namespace"
aegis@aegis:~/WORKSPACE/VSecM$ make deploy-local
kubectl apply -f ./k8s/0.21.5/crds
customresourcedefinition.apiextensions.k8s.io/clusterfederatedtrustdomains.spire.spiffe.io created
customresourcedefinition.apiextensions.k8s.io/clusterspiffeids.spire.spiffe.io created
customresourcedefinition.apiextensions.k8s.io/clusterstaticentries.spire.spiffe.io created
customresourcedefinition.apiextensions.k8s.io/controllermanagerconfigs.spire.spiffe.io created
kubectl apply -f ./k8s/0.21.5/0.21.5-local-distroless.yaml
namespace/spire-system created
serviceaccount/spire-agent created
serviceaccount/spire-server created
configmap/spire-agent created
configmap/spire-controller-manager-config created
configmap/spire-bundle created
configmap/spire-server created
clusterrole.rbac.authorization.k8s.io/secret-readwriter created
clusterrole.rbac.authorization.k8s.io/manager-role created
clusterrole.rbac.authorization.k8s.io/spire-agent-cluster-role created
clusterrole.rbac.authorization.k8s.io/spire-server-cluster-role created
clusterrolebinding.rbac.authorization.k8s.io/secret-readwriter-binding created
clusterrolebinding.rbac.authorization.k8s.io/manager-rolebinding created
clusterrolebinding.rbac.authorization.k8s.io/spire-agent-cluster-role-binding created
clusterrolebinding.rbac.authorization.k8s.io/spire-server-cluster-role-binding created
role.rbac.authorization.k8s.io/leader-election-role created
role.rbac.authorization.k8s.io/spire-server-role created
rolebinding.rbac.authorization.k8s.io/leader-election-rolebinding created
rolebinding.rbac.authorization.k8s.io/spire-server-role-binding created
service/spire-server created
service/spire-server-bundle-endpoint created
service/spire-controller-manager-webhook-service created
daemonset.apps/spire-agent created
deployment.apps/spire-server created
csidriver.storage.k8s.io/csi.spiffe.io created
clusterspiffeid.spire.spiffe.io/vsecm-safe created
clusterspiffeid.spire.spiffe.io/vsecm-sentinel created
validatingwebhookconfiguration.admissionregistration.k8s.io/spire-controller-manager-webhook created
namespace/vsecm-system created
Error from server (NotFound): error when creating "./k8s/0.21.5/0.21.5-local-distroless.yaml": namespaces "vsecm-system" not found
Error from server (NotFound): error when creating "./k8s/0.21.5/0.21.5-local-distroless.yaml": namespaces "vsecm-system" not found
Error from server (NotFound): error when creating "./k8s/0.21.5/0.21.5-local-distroless.yaml": namespaces "vsecm-system" not found
Error from server (NotFound): error when creating "./k8s/0.21.5/0.21.5-local-distroless.yaml": namespaces "vsecm-system" not found
Error from server (NotFound): error when creating "./k8s/0.21.5/0.21.5-local-distroless.yaml": namespaces "vsecm-system" not found
Error from server (NotFound): error when creating "./k8s/0.21.5/0.21.5-local-distroless.yaml": namespaces "vsecm-system" not found
make: *** [makefiles/VSecMDeploy.mk:41: deploy-local] Error 1

Also, although the namespace is there, there are no pods in it:
aegis@aegis:~/WORKSPACE/VSecM$ k get ns
NAME              STATUS   AGE
default           Active   26m
kube-node-lease   Active   26m
kube-public       Active   26m
kube-system       Active   26m
spire-system      Active   10m
vsecm-system      Active   10m
aegis@aegis:~/WORKSPACE/VSecM$ k get po -n vsecm-system
No resources found in vsecm-system namespace.
aegis@aegis:~/WORKSPACE/VSecM$ k get deployment -n vsecm-system
No resources found in vsecm-system namespace.

I can work on this

I might have found the issue when we add this annotations value, helm template adds the vsecm-system Namespace resource to the bottom of the manifest. When I removed the podAnnotations value from values.yml and generated the manifests, deployment worked as expected.
Previously, the podAnnotations key was duplicated, so I guess it was overriding, and it had no effect.
podAnnotations:
  "helm.sh/hook": pre-install

https://github.com/vmware-tanzu/secrets-manager/blob/main/helm-charts/0.21.5/values.yaml#L60-L61

@rohatcan it’s all yours. have fun :) .

Working as expected right now.
Closing.
Please let me know if it needs to be reopened.


Implement a better UI

Currently, the user interface looks unfriendly to new users who use the app:

Here's a preview of how the UI should look like:

Now it's done! Closing...


Loading script on boot: [Errno 25] Inappropriate ioctl for device

I am trying to run my custom script as background daemon which is being loaded during startup via rc.local script. During execution any access to envirophat throws:

OSError: [Errno 25] Inappropriate ioctl for device

However, same script loaded as background daemon from command line does not raise mentioned exception. I have tried to run it as root and as regular user, but there is no difference. The only workaround I found is to defer loading library to function scope.
Please refer to script file for error reproduction. I am using RPi Zero W with latest Raspbian image, no other shields are connected.

Could this be related? https://forums.pimoroni.com/t/enviro-phat-and-daemonizing/6224


minor readme additions

hey, i'm just exploring this project and thought i'd contribute some very minor readme additions; basically driven by things I was looking for as a new end-user

document the npm install. I find this to be a common pattern so I think it'd be helpful for new folks who come along
add the npm badge. a minor flourish, but it actually led me to realizing that you haven't published 0.1.1 to npm yet. i'm happy to file an issue about this as well, if that's more productive
add a note about needing the chromecast extension installed to readme

poke

Whoops, it seems we all let this slide. Thanks, these are great.

happy to help. thanks for the project! :rowboat:


API Request: Function for showing an "Open File" dialog that returns the selected path

Is it possible to add an API function which allows an extension to show an "Open File" dialog which doesn't cause VS Code to open the selected file but instead returns the file path?  I'd like to allow the user to easily pick the location of a PowerShell executable to use for their editing session.
I've looked around for a while to find a way to do this without the need for a new API but couldn't find a solution.  If you know of a better method, I'd be more than happy to use that :)
Thanks!

So is there any way to do that right now?
Only way I found is to tell the user to open the file and place it in the second column. Then I can get the file with this code:
 let col2Editor = vscode.window.visibleTextEditors.find(editor => editor.viewColumn === vscode.ViewColumn.Two);
 let filename = col2Editor && col2Editor.document.fileName;

I'm sure there's a cleaner way to do this...

I'd like this feature as well.  I'm writing an extension that wants to allow the user to load a binary file (one that wouldn't normally open in vscode) and encode it into a data-uri string, and embed it into the current text document.

@chrisdias Please provide some details about your use-case/scenario

In a nutshell, I want to be able to write an extension that will scaffold out a "new project".  I want to prompt the user for a new folder, I'll then scaffold code into that folder, and then I will (re) open Code on that folder, or add as a new root folder.
So, I need:

An API that let's me prompt for a new folder

An API for opening or reopening VS Code on a folder.  To accomplish this today, I have to invoke the vscode.openFolder command, see the awesome open new instance extension:

// The explorer/context menu contribution receives the URI to the file/folder
    let cmd1 = vscode.commands.registerCommand('extension.openNewInstance', (e: vscode.Uri) => {
        vscode.commands.executeCommand("vscode.openFolder", e, true);
    });

An API for adding a new root level work space folder

Incidentally the PowerShell extension also needs this API for project templating UI, so it'd be awesome to have this :)

Ok, you want a lot more then a file picker dialog. Step by step...

An API that let's me prompt for a new folder

I assume that is outside the current workspace folders and you'd expect the native OS file picker, right? It's unclear to me if picking a file inside the workspace should use our Cmd+P picker... In general there are a few options to this but nothing dramatic.

An API for opening or reopening VS Code on a folder. To accomplish this today, I have to invoke the vscode.openFolder command, see the awesome open new instance extension:

That is a lot more and I am not sure the implication are clear to everyone here. When we open a new or different folder a different extension host comes to life and your current extension host might be shut down (when replace the current window). In any case you won't be able to talk to that "other" extension host.

An API for adding a new root level work space folder

Similar to above. For API-stability reasons we restart the extension host when going from 0-1, 1-N, (1-0, N-1) workspace folders. That also means that your extension state is getting restarted and that you need to keep your state somewhere else.

I assume that is outside the current workspace folder

Yes, that would be my general expectation where the user would pick. However, it seems reasonable that they could pick a folder inside the workspace. Whether or not I reload the workbench seems to be something that is then up to the extension author.

you'd expect the native OS file picker

Yes, very much.  I want the easy ability to switch drives, add a new folder, etc. all provided by the OS pickers today.

your current extension host might be shut down

In my scenario I'm perfectly fine with this.
I can live with using the vscode.openFolder command for now, but having a real API for this feels cleaner.  For adding a new root level workspace folder i'm assuming there will be a similar vscode.addFolderToWorkspace command I could leverage?

Yes, very much. I want the easy ability to switch drives, add a new folder, etc. all provided by the OS pickers today.

Check, it's pretty much exposing what electron offers to us

I can live with using the vscode.openFolder command for now, but having a real API for this feels cleaner.

Yes and no. To some extend command identifiers are API because folks reference them from the keybindings.json-file and therefore shouldn't change. When exposing this via blessed API I am afraid that people will be confused about the lifecycle... It's rare that you call an method (other than exit) to shutdown yourself.

For adding a new root level workspace folder i'm assuming there will be a similar vscode.addFolderToWorkspace command I could leverage?

Unsure, it has been discussed before. But it's the same concern, having an API that restarts everything feels wrong...

Sorry, maybe I wasn't clear.  I can live with using this to reload VS Code:
        vscode.commands.executeCommand("vscode.openFolder", e, true);

So I guess the last question is, how will an extension add or remove a root level workspace?

So I guess the last question is, how will an extension add or remove a root level workspace?

Unsure, it has been discussed before. It's the same problem, when going from 0-1, 1-N a restart is required.

This is the current proposed API:
	export interface OpenDialogOptions {
		uri: Uri;
		openFiles?: boolean;
		openFolders?: boolean;
		openMany?: boolean;
	}

	export namespace window {

		export function showOpenDialog(options: OpenDialogOptions): Thenable<Uri[]>;
	}

API is proposed, awaiting feedback, making it final in September

Any way we could have the same for 'showSaveDialog' ?
Some of my extensions need a way to save/export into new files. I'm currently using showInputBox to get a file path, but a Save dialog would be way better :)

Any way we could have the same for 'showSaveDialog' ?

Yeah, I like. Will be similar to the open-api. I'll let know when I have something

Sorry for the delay, new API looks good to me!  Looking forward to trying it out.

Yeah, looks good. This plus the save dialog would be very handy.
On Wed, Sep 6, 2017 at 10:49 AM, David Wilson<EMAIL_ADDRESS>wrote:

Sorry for the delay, new API looks good to me! Looking forward to trying
it out.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/vscode/issues/13807#issuecomment-327488820,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFpqzfnE-xYbT0Bb_gsyOEHfWR_94wVgks5sfqL2gaJpZM4KXxQs
.

I have pushed a change that also adds a save dialog
	export interface OpenDialogOptions {
		defaultResource?: Uri;
		openLabel?: string;
		openFiles?: boolean;
		openFolders?: boolean;
		openMany?: boolean;
	}

	export interface SaveDialogOptions {
		defaultResource?: Uri;
		saveLabel?: string;
	}

	export namespace window {
		export function showOpenDialog(options: OpenDialogOptions): Thenable<Uri[]>;
		export function showSaveDialog(options: OpenDialogOptions): Thenable<Uri>;
	}

Is it easy to add a filter list for these?  Some File (*.sf)|*.sf|All Files (*.*)|*.*

@jreiken did you intend to use SaveDialogOptions on the showSaveDialog method?  Thanks for adding that one also, this is going to be really helpful!

did you intend to use SaveDialogOptions on the showSaveDialog method?

Doh, I am a copy&paste programmer....

Is it easy to add a filter list for these? Some File (.sf)|.sf|All Files (.)|.

Yeah, it's not there yet but the underlying electron API has it. Unsure yet how it will look like, may some sort of object like { [title:string]:string[]} which makes filters: { 'Images': [*.png, *.gif], 'Text': [*.txt]}

@jrieken thanks for the feature 👍
Although, maybe i'm doing something wrong, but I can't get a SaveDialog with anything other dans 'Untitled' in the Save As input box (on Mac). Even with a valid defaultUri.
const filepath = getPhysicalPath(d.uri);
const defaultUri = vscode.Uri.file(filepath);
const option: vscode.SaveDialogOptions = { defaultUri, filters: {} };

vscode.window.showSaveDialog( option ).then(fileUri => {
    console.log(fileUri ? fileUri.toString() : 'undefined');
});

I'm on 1.17.0-insider (f7962f0682a76516df51de4856f8ccc5d8ad578a)

Hm, it works for me on master. You mean undefined is always the result instead of a meaning full fileUri?

No, I mean I can't pre-fill the SaveDialog with a default file.

Hum, but it works if I name it defaultResource. The Insider might be more behind that I though of...

Yeah, Insiders is behind, the code-sign infrastructure gets an overhaul. In the stable API we have named this thing defaultUri instead of defaultResource.

Thanks, sry for the false-alarm then :)


[CCXDEV-9825] Add panel splitting status codes by endpoint

Description
Add a panel splitting status codes by endpoint. This let us easier debug by selecting the endpoint we want to query on a variable in the top side of the page. For example, if we want to know the status codes of the upgrade risks predictions we select /v2/cluster/{cluster}/upgrade-risks-predictions and look at the panel on the bottom right side of the page.
Before:

After

Type of change

New feature (non-breaking change which adds functionality)

Testing steps
Stage UI.
Checklist

[ ] make before_commit passes
[ ] updated documentation wherever necessary
[ ] added or modified tests if necessary
[ ] updated schemas and validators in insights-data-schemas in case of input/output change

Codecov Report
Patch has no changes to coverable lines.
:loudspeaker: Thoughts on this report? Let us know!.


Reopen  HCT-611 WiFi Water Timer #685

https://github.com/make-all/tuya-local/issues/685
@make-all
Can you pls help me?
As I wrote in the mentioned ticket (now closed).. since there are many completely different case discussed there (See discussion https://github.com/make-all/tuya-local/discussions/522).. is there a way to "identify" exactly the issue by seeing at logs?
If you guide me how to collect the right log for you.. I will do..
I don't want to go by attempts.. and I would like to have it working..
Thanks in advance

The information is all there in #522

So there is no way to understand the problem by seeing at logs?
I can't believe that..


NetworkCompatibility display concise missing requirements.

Currently, the server's required mod list and the clients current mod list are presented to the user upon a failed connection.
We can make this more concise by simply listing the offending/required packages which are missing, or mis-versioned.

done in #66


Application does not start because of ValueError when converting coordinates from ipinfo.io to floats

Describe the bug
When trying to start the application using the .desktop file or through the terminal with the venv activated, the application will not start. There is no GUI visible. When trying to start the application from the terminal, I get the following traceback:
I have uninstalled and reinstalled the application using the packaged scripts, the error happened before and after the reinstall.
Traceback (most recent call last):
  File "<frozen runpy>", line 198, in _run_module_as_main
  File "<frozen runpy>", line 88, in _run_code
  File "/opt/yin-yang/yin_yang/__main__.py", line 134, in <module>
    window = main_window_connector.MainWindow()
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/yin-yang/yin_yang/ui/main_window_connector.py", line 55, in __init__
    self.load()
  File "/opt/yin-yang/yin_yang/ui/main_window_connector.py", line 89, in load
    self.load_times()
  File "/opt/yin-yang/yin_yang/ui/main_window_connector.py", line 95, in load_times
    time_light, time_dark = config.times
                            ^^^^^^^^^^^^
  File "/opt/yin-yang/yin_yang/config.py", line 455, in times
    latitude, longitude = self.location
                          ^^^^^^^^^^^^^
  File "/opt/yin-yang/yin_yang/config.py", line 426, in location
    coordinate = get_current_location()
                 ^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/yin-yang/yin_yang/config.py", line 142, in get_current_location
    loc: [float] = [float(coordinate) for coordinate in loc_response]
                    ^^^^^^^^^^^^^^^^^
ValueError: could not convert string to float: '{\n  "status": 429'

Affected versions

Yin-Yang version:
Commit de1f3eb5c81f90657685503eb317e1fe91c57273
Python version: 3.12.0
Qt version: qt5: 5.15.11, qt6: 6.6.0

Additional notes
My system is Fedora 39 with KDE Plasma 5.27.9.
This issue is first and foremost for completeness sake, I may be able to delve into the bug myself, but not at this time. In case anyone else is interested in exploring this or has the same issue please go ahead and see if you can find the cause and fix it.

Alright, I looked into it a little bit, it appears that the value of loc_response is ['{\n  "status": 429', '\n  "error": {\n    "title": "Rate limit exceeded"', '\n    "message": "You\'ve hit the daily limit for the unauthenticated API.  Create an API access token by signing up to get 50k req/month."\n  }\n}']
What worked for me is adding this line of code before the line where the error was (config.py line 142):  if len(loc_response) != 2: return QGeoCoordinate(0, 0), rerunning the installation script, and then turning off update location automatically and putting my location in manually


Installation script uses pnpm instead of npm

Feature description
Installation script uses pnpm instead of npm
Additional information
No response

PR is appreciated  about adding a new option for npm, yarn, pnpm in install script both for sh and pwsh.

And is there a good reason for using pnpm (usually an additional dependency) instead of just npm for everyone?

for example, some package manager will ship pnpm with nodejs, but some didn't, like scoop on Windows.
u need extra settings to get pnpm and yarn ready for every distros and OSs.
to avoid this kind of hustle, we tend to use just the standerd installer.
in fact, u can only use pnpm on your own system.

As @Jint-lzxy said, pnpm is an additional dependency. It looks better than npm but it's not the official tool and maybe not suit every user. So we won't support it.


bigquery: CAST AS x FORMAT support

https://cloud.google.com/bigquery/docs/reference/standard-sql/conversion_functions#cast_as_date

I think the CI failures are not related to the changes in this PR -- if you merge up / rebase from main they should be solved. See https://github.com/sqlparser-rs/sqlparser-rs/pull/995


Moving AttributesTest to Kotlin

Have you migrated many tests already?
There's a trick: https://stackoverflow.com/questions/44310285/converting-file-tracked-with-git-from-java-to-kotlin-in-android-studio/44320452#44320452
If you do the migration with two commits, then git history would be easier to analyze. I did that recently, and I do recommend making the migration in two commits:

first commit that renames *.java -> *.kt (just rename)
the second commit converts the syntax

Note: if you do the conversion and commit via IntelliJ IDEA, then there's Extra commit for .java > .kt renames checkbox:

Thanks @vlsi for the trick.
I have still some doubts about a full migration strategy because the gain is low and it won't fix the design issues.
Instead I'd like to use kotlin as a way to rethink things without breaking everything.
But it take a lot of time and free time is rare.

I have still some doubts about a full migration strategy

I do not mean you must do the full migration.
I just note that if you convert files, then having that as two git commits makes it easier to review even in GitHub as you will have side-by-side diffs.

I am going to have to close out this PR for a couple of reasons.

My Kotlin learning got disrupted due to work priorities and also some personal setbacks. So I am bit more rusted on this.
Given the fact that time is becoming a constraint, I am not sure if I would want to pursue this migration. Instead I would like to put my time to fix all the open issues in TestNG.


Added DSL tabs for EIPs docs

Description
Added DSL tabs for most of existing EIP pages
Target

[x] I checked that the commit is targeting the correct branch (note that Camel 3 uses camel-3.x, whereas Camel 4 uses the main branch)

Tracking

[x] If this is a large change, bug fix, or code improvement, I checked there is a JIRA issue filed for the change (usually before you start working on it).

Apache Camel coding standards and style

[x] I checked that each commit in the pull request has a meaningful subject line and body.

[x] I formatted the code using mvn -Pformat,fastinstall install && mvn -Psourcecheck

camel-website now has the tabs support, so lets see this in action when a new build is done later.

@davsclaus thanks
There was PR that removed tabs-sync feature from several pages: https://github.com/apache/camel/pull/9993
For example, here: https://camel.apache.org/components/next/eips/message-translator.html if you click XML, than corresponding tab will be actived in each of tabs-sections. But here: https://camel.apache.org/components/next/eips/choice-eip.html you need to choose XML/YAML be hand on every tabs-section.
It is inconsistent now and what do you think is better - to have tabs-sync feature or not?

Can you create a JIRA about this regen job removing those. We need to look and fix this in some internal build tool


Add Perceptron to Notebook

Added a new section to the Python Notebook for the Perceptron Classifier. Currently includes Overview and Implementation. I have also written an example for the learner, but I will add it once #333 is resolved.

Added example to Perceptron Learner section. The section is now complete.

the build fails because of flake8 errors in test_agents.py and test_text.py ,
fixed that in PR #373 ,will have to wait for merge.

the build fails due to --test_text.py and test_agents.py, it shows flake8 errors, fixed that in PR #373 , will have to wait for merge

Updated the PR with some minor fixed in the notebook.

(closing this to make a cleaner PR)


 element does not support "type", so it defaults to always "submit"

Bug Report
Steps
Imagine we have a form with a dynamic input field. Every time the user presses on "Add item" a new input field should appear, and when the user presses "Submit" the form will be sent to the server. (See the testcase, it's a lot simpler than it looks!)
Expected Result
When I press "Add item" a new input field appears, and only when I press "Submit" the form gets submitted.
Actual Result
Since there is no way for us to specify the type of the <button> element when using Button, all buttons default to submit. See also: https://dev.to/clairecodes/why-its-important-to-give-your-html-button-a-type-58k9
Version
latest
Testcase
https://codesandbox.io/s/semantic-ui-react-kyzl4

I take it back, apparently you can set the property type="button". Don't know why I thought it wasn't working. Closing as not-an-issue.


2022 april tremors

Hey Tremor party people!
As promised, here's a breakdown and editorial on some of the changes we've seen in the last month. I'm more than happy to edit, dive into, change any of this as we see fit to make sure we cover important bits appropriately. Let me know what you think!
I've dated this for the second working day in May, I'm aiming to publish these on a cadence like that moving forward.

First of all thanks for that lovely post!
This is a bit of a tricky one, we're somewhat stuck with main on RC.1 with new code gernation and www still on 0.11 with old code generation. I'm not 100% sure how to best resolve this conflict without doing a ton of changes to the current main on -www, the best idea I have would be targeting this at the mob-v12-refresh and then merging it together with that branch.
What do you think?

no problem at all. sec!

so just to be clear I now have two branches targeted for merging. This PR and this PR I think that's what you wanted but let me know if I missed a trick

Heya, sorry ja release time is all a bit chaotic, sorry, with #176 this can be closed, we'll merge the changes over there :)


Fix unit-tests when no-srp configured

Checklist

[ ] documentation is added or updated
[X] tests are added or updated

This needs a rebase...

Rebased.

Can someone merge this (@richsalz, @levitte)

i will do it now.

b6e3250 in master, thanks!


[vk] Remove Pin usage from pipeline creation

Doesn't fix anything, just simplifies the code a bit.

bors r+

bors r+

bors r+

bors r+


Doesn't seem to work inside tmux.

Note, that for sixel support in tmux you need you'll need at least https://github.com/tmux/tmux/commit/1a1290f30b21bcc07e64eb7cb837a3c522779685, but probably https://github.com/tmux/tmux/commit/f68d35c52962c095e81db0de28219529fd6f355e is safer.
Running MPLTERM=protocol=sixel ipython and then trying to call plt.show() in tmux results in the terminal becoming cleared and mplterm hanging here:
File ~/.pyenv/versions/3.11.1/lib/python3.11/site-packages/matplotlib/pyplot.py:446, in show(*args, **kwargs)
    402 """
    403 Display all open figures.
    404
   (...)
    443 explicitly there.
    444 """
    445 _warn_if_gui_out_of_main_thread()
--> 446 return _get_backend_mod().show(*args, **kwargs)

File ~/.pyenv/versions/3.11.1/lib/python3.11/site-packages/matplotlib/backend_bases.py:2894, in FigureManagerBase.pyplot_show(cls, block)
   2892 for manager in managers:
   2893     try:
-> 2894         manager.show()  # Emits a warning for non-interactive backend.
   2895     except NonGuiException as exc:
   2896         _api.warn_external(str(exc))

File ~/.pyenv/versions/3.11.1/lib/python3.11/site-packages/mplterm/__init__.py:101, in _MpltermFigureManager.show(self)
     99 fig = self.canvas.figure
    100 with ExitStack() as stack:
--> 101     size = proto.get_pixel_size()
    102     if size is not None:
    103         stack.enter_context(_shrinked(fig, size))

File ~/.pyenv/versions/3.11.1/lib/python3.11/site-packages/mplterm/_sixel.py:42, in Sixel.get_pixel_size()
     40 @staticmethod
     41 def get_pixel_size():  # XTWINOPS
---> 42     reply = _term_query(_csi("14t"), _csi_regex(r"4;(\d+);(\d+)t"))
     43     h, w = map(int, reply)
     44     return w, h

File ~/.pyenv/versions/3.11.1/lib/python3.11/site-packages/mplterm/_util.py:48, in _term_query(cmd, pattern)
     46 buf = b""
     47 while True:
---> 48     buf += sys.stdin.buffer.read(1)
     49     match = regex.fullmatch(buf)
     50     if match:

KeyboardInterrupt:

Works like a charm outside tmux.

tmux gets stuck because it does not seem to support (AFAICT) XTSMGRAPHICS and XTWINOPS (https://invisible-island.net/xterm/ctlseqs/ctlseqs.html) to query the number of color registers and the window (pane?) size in pixels; see also https://github.com/wez/wezterm/issues/609 re: the need for XTSMGRAPHICS for sixel support.
However even if I bypass these queries (and hard-code some reasonable values), tmux (installed from git HEAD) doesn't hang anymore but I still don't manage to get sixel showing; in fact even something as simple as python -c 'from pylab import *; plot(); savefig("/tmp/test.png")'; convert test.png -colors 256 sixel:- gives no output for me in tmux (whereas it works fine outside of it).  Does such a command (which is close to what mplterm is basically doing) work for you?

@anntzer Ah yes, sorry, I should've mentioned the flag. To be honest, my technical knowledge on these things is extremely limited, so although I can certainly report (and link back here), you'd probably be more convincing. Still, if you'd rather I do it, I'll do it :)
The INPUT_BUF_LIMIT is basically the largest sized sixel tmux will be able to render? Because in that case, yes, I guess that would be useful, yes.

Ah yes, sorry, I should've mentioned the flag.

No worries!

To be honest, my technical knowledge on these things is extremely limited, so although I can certainly report (and link back here), you'd probably be more convincing. Still, if you'd rather I do it, I'll do it :)

I don't know much more -- I've only read the paragraphs about XTWINOPS and XTSMGRAPHICS in the page linked above, and they're basically ways to query the pixel size and the number of colors supported by the terminal (which are useful for sixel output for obvious reasons) -- and you seem to have more interest than I do in getting this to work in tmux (I can try to make whatever changes are reasonable in mplterm, but I don't really want to get bogged down in a discussion about tmux internals), so go for it ;)

The INPUT_BUF_LIMIT is basically the largest sized sixel tmux will be able to render?

If I understand the following comment https://github.com/tmux/tmux/issues/3668#issuecomment-1689540241 correctly, yes.

I should probably learn how to do these csi things ...

Can you just patch mplterm to print out err, num at that line?

Oh, yeah sure, should have been thinking :D
print(err,num)
3 1024

Sure, I just patched mplterm to be more lenient there.

Oh, yeah!

Out of curiosity, what does err mean here, that you could be more lenient (I was surprised it was not a binary value also)?

The meaning of err is documented in https://invisible-island.net/xterm/ctlseqs/ctlseqs.html (search for XTSMGRAPHICS); I won't explain it better than the ref so I may just as well point you to it (err is Ps, num is Pv).
Let's keep this open for now; I think this generally points out that mplterm can be more robust when the underlying terminal doesn't handle certain codes (it should not hang, can assume 256 color registers are available per the sixel standard if the terminal fails to report that information, and skip figure resizing if the window size is not available).

mplterm should now be more lenient if the underlying terminal doesn't support certain xterm-specific codes.


Conditional check needed for guess_normalisation() function in compressor.py

Hi Fabio,
in line 105 & 106 of compressor.py:
config.get("normalisation", guess_normalisation(self.adata))
The guess_normalisation function is always executed regardless of whether the user has supplied a normalisation method or not. But we don't want the function to be run if a normalisation method is provided in the config.
Cheers,
Ying

I don't think that's what that line does?
On Wed, Jun 26, 2024, at 10:46, YingX97 wrote:

Hi Fabio,
in line 105 & 106 of compressor.py:
config.get("normalisation", guess_normalisation(self.adata))
The guess_normalisation function is always executed regardless of
whether the user has supplied a normalisation method or not. But we
don't want the function to be run if a normalisation method is provided
in the config.
Cheers,
Ying
—
Reply to this email directly, view it on GitHub
https://github.com/fabilab/scquill/issues/2, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJFEAFPLRCCXZQLU6CDJPDZJIFNFAVCNFSM6AAAAABJ426QTOVHI2DSMVQWIX3LMV43ASLTON2WKOZSGM3TGOJTGQ3DGNQ.
You are receiving this because you are subscribed to this
thread.Message ID: @.***>