Another _Ember is undefined error

Symptoms
Same symptoms as #342, but the errors are coming up while loading tests in Travis. Every require fails with this error.
How to replicate
You should be able to replicate by running tests with CI=true.
It's related to this default line in targets.js. Removing ie 11 as a target removes the issue.
if (isCI || isProduction) {
  browsers.push('ie 11');
}

Tested on latest<EMAIL_ADDRESS>

Okay, so I've managed to track down and fix my issues with some much needed refactoring.
 I'll leave this up in case it helps improve anything here.
Original:
import { computed, getProperties } from '@ember/object';

function position() {
  return computed('lat', 'lng', function() {
    const { lat, lng } = getProperties(this, 'lat', 'lng');
    return (lat && lng) ? new google.maps.LatLng(lat, lng) : undefined;
  });
}

export { position };

Compiled:
define("ember-google-maps/utils/helpers", ["exports"], function (_exports) {
  "use strict";

  Object.defineProperty(_exports, "__esModule", {
    value: true
  });
  _exports.position = position;

  function position() {
    return (_Ember = Ember).computed.apply('lat', 'lng', function () {
      var _Ember$getProperties = Ember.getProperties(this, 'lat', 'lng'),
          lat = _Ember$getProperties.lat,
          lng = _Ember$getProperties.lng;

      return lat && lng ? new google.maps.LatLng(lat, lng) : undefined;
    });
  }
});

One last thing! This similar function below compiles correctly. I guess the transpiler is catching on to the ...args.
function computedPromise(...args) {
  let func = args.pop();
  return computed(...args, function() {
    return ObjectPromiseProxy.create({
      promise: func.apply(this)
    });
  });
}

loading tests in Travis

Which repo was this for? I’d like to repro locally to see what can be changed in the Babel plugin.

I'm seeing a similar thing, running CI=true ember serve with the following component
import Component from '@ember/component'
import { computed, set } from '@ember/object'

import layout from './template'

function setSomeComputedKeys(keys) {
  const someDataCP = computed(...keys, this.someDataCPFunction)
  set(this, 'someData', someDataCP)
}

function someDataCPFunction() {
  return []
}

export default Component.extend({
  layout,
  tagName: '',

  init(...args) {
    this._super(...args)
    setSomeComputedKeys.call(this, [])
  },

  someDataCPFunction,

  data: computed('someData.[]', function () {
    return []
  }),
})

Compiles to
;define("dummy/components/foo/component", ["exports", "dummy/components/foo/template"], function (_exports, _template) {
  "use strict";

  Object.defineProperty(_exports, "__esModule", {
    value: true
  });
  _exports.default = void 0;

  function _toConsumableArray(arr) { return _arrayWithoutHoles(arr) || _iterableToArray(arr) || _unsupportedIterableToArray(arr) || _nonIterableSpread(); }

  function _nonIterableSpread() { throw new TypeError("Invalid attempt to spread non-iterable instance.\nIn order to be iterable, non-array objects must have a [Symbol.iterator]() method."); }

  function _unsupportedIterableToArray(o, minLen) { if (!o) return; if (typeof o === "string") return _arrayLikeToArray(o, minLen); var n = Object.prototype.toString.call(o).slice(8, -1); if (n === "Object" && o.constructor) n = o.constructor.name; if (n === "Map" || n === "Set") return Array.from(o); if (n === "Arguments" || /^(?:Ui|I)nt(?:8|16|32)(?:Clamped)?Array$/.test(n)) return _arrayLikeToArray(o, minLen); }

  function _iterableToArray(iter) { if (typeof Symbol !== "undefined" && Symbol.iterator in Object(iter)) return Array.from(iter); }

  function _arrayWithoutHoles(arr) { if (Array.isArray(arr)) return _arrayLikeToArray(arr); }

  function _arrayLikeToArray(arr, len) { if (len == null || len > arr.length) len = arr.length; for (var i = 0, arr2 = new Array(len); i < len; i++) { arr2[i] = arr[i]; } return arr2; }

  function setSomeComputedKeys(keys) {
    var _Ember;

    var someDataCP = (_Ember = Ember).computed.apply(_Ember, _toConsumableArray(keys).concat([this.someDataCPFunction]));

    Ember.set(this, 'someData', someDataCP);
  }

  function someDataCPFunction() {
    return [];
  }

  var _default = Ember.Component.extend({
    layout: _template.default,
    tagName: '',
    init: function init() {
      this._super(arguments);

      setSomeComputedKeys.call(this, []);
    },
    someDataCPFunction: someDataCPFunction,
    data: (_Ember = Ember).computed.apply('someData.[]', function () {
      return [];
    })
  });

  _exports.default = _default;
});

For some reason, removing the usage of the spread operator (...) from the component appears to fix the issue.

Which repo was this for? I’d like to repro locally to see what can be changed in the Babel plugin.

This was in ember-google-maps. I've created a branch out of the old version for you. No setup necessary, will error out right away.
https://github.com/sandydoo/ember-google-maps/tree/ember-cli-babel-repro

Awesome, thank you @sandydoo!

I have the same problem as @sandydoo mentioned.
In my package.json I have  "ember-cli-babel": "~7.20.4" but I called yarn why ember-cli-babel and a lot of libraries depends on ember-cli-babel: 6.18.0, e.g. I use ember-cli-notifications: ^4.3.3 which depends on "ember-css-modules": "^0.7.10" which depends on ember-cli-babel: 6.18.0, seems two version of ember-cli-babel conflicts each other.

seems two version of ember-cli-babel conflicts each other

No, they should not conflict with each other. Each addon will use the version of ember-cli-babel that they directly depend on to transpile their own code.

@rwjblue if I turn on
if (isCI || isProduction) {
  browsers.push('ie 11');
}

I have the next error:
ReferenceError: _Ember is not defined
    at configurable (/usr/src/fastboot/dist/assets/addon-tree-output/torii/configuration.js:13:1)```

I use torii: 0.9.6 which depends on "ember-cli-babel": "^6.3.0". But I have a lot of libraries which depend on "ember-cli-babel": "^6.3.0"

OK, that sounds a lot like the scenario that @sandydoo called out and provided a reproduction for. I'll be looking into it this afternoon.

Same problem here with ember-metrics (just posted in the discord). Not sure what Babel was trying to do with (_Ember = Ember) but it looks like its going to be happening in lots of places in various addons. Assuming its something in the 7.20.x release

Just a note that the build for the PR at ember-intl/ember-intl#1340 is encountering this issue as well. It's using<EMAIL_ADDRESS>and the issue reproduces locally as well.
The Travis run is here: https://travis-ci.com/github/ember-intl/ember-intl/jobs/342607182

@rwjblue all of my addons, ember-shepherd. ember-math-helpers, ember-flatpickr, etc are experiencing this issue. I am unable to run any CI builds. Is there a workaround?

We too had this issue. Fixed it with pinning "babel-plugin-ember-modules-api-polyfill": "2.13.0". Kind of workaround but works.

We too had this issue. Fixed it with pinning "babel-plugin-ember-modules-api-polyfill": "2.13.0". Kind of workaround but works.

@ananddotmurali could you supply details on what you did to pin that, as what I tried did not work

I am unable to run any CI builds. Is there a workaround?

@rwwagner90, a few workarounds we know of.

Stay on<EMAIL_ADDRESS>for now. For CI builds with floating dependencies that means pinning down the version.

Remove the ie* target from targets.js. This is an ember-cli default, removing it will bypass the issue.

ember-math-helpers, ember-flatpickr, etc are experiencing this issue

So I was curious... Why would ember-math-helpers encounter this error? It doesn't even have any computed properties. Turns out it's because of ember-cli-addon-docs.
https://github.com/ember-learn/ember-cli-addon-docs/blob/f1089775f32b56a61392b62eee36d26e06a68983/addon/utils/computed.js#L8-L29

Interesting, my repro for this issue in ember-intl/ember-intl#1340 occurs in the exact same place inside ember-cli-adon-docs.

I think we've got a fix? Go @rwjblue🥇
https://github.com/ember-cli/babel-plugin-ember-modules-api-polyfill/commit/621360cb8ccc08d0c36f5bfe4f44548fce9648f9

Ya, sorry for this taking so long. It took a little while to figure out which specific babel plugins were mutating the nodes, and I only discovered that using the shared AST nodes was causing an issue due to stepping through the tranpsilation process.
#348 should land and be released as 7.20.5 soon.

https://github.com/babel/ember-cli-babel/releases/tag/v7.20.5 is published now with the updates from https://github.com/ember-cli/babel-plugin-ember-modules-api-polyfill/pull/109. I tried to explain exactly what was going on in #348 and https://github.com/ember-cli/babel-plugin-ember-modules-api-polyfill/pull/109.
Note: the actual fix was in https://github.com/ember-cli/babel-plugin-ember-modules-api-polyfill/pull/109, so you can almost certainly get these fixes without being on<EMAIL_ADDRESS>but in order to do so you'd need to ensure your lockfile doesn't keep the babel-plugin-ember-modules-api-polyfill pinned.
For example, for yarn users you can do:

add a resolution for babel-plugin-ember-modules-api-polyfill to use 2.13.4
run yarn
remove the resolution
run yarn again


Demo scenes in GUI2

We, devs, would like to have a possibility to test visuals of components without connecting to the backend, like the demo scenes in the old GUI. We could use Historie framework https://histoire.dev/guide/vue3/getting-started.html to manage such scenes.
In scope of this task is to make a scene displaying our project view with some mocked nodes, where we also could see the component browser.

oh, also adding mocked suggestion db store and project store


Add tags to disable caching of Home.html

While going through the tutorial I had problems with the new HTML elements not appearing across builds and then magically appearing the next day without any changes. Having researched this I find that with recent versions of Office (1910 - 12130.20238) and Windows (1909 - 18363.418) the Office clients when running add-ins will use the Edge WebViewHost to display the HTML that has a different caching behaviour as IE before.
Adding the following meta tag to Home.html has fixed this behaviour going forward:
<meta http-equiv="Pragma" content="no-cache" />

Of course as there already was a cached version of the file without the tag I would have needed to wait until that expired but the command line recommended by user ndee here on StackOverflow (see latest reply added on 25/10/2019) has helped to clear the WebViewHost cache and get me going: https://stackoverflow.com/questions/57909922/office-js-addin-with-excel-and-edge-webview-how-to-delete-cache

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

ID: 18abb533-5be7-d533-8106-c52d458ddc1d
Version Independent ID: 6af81176-d832-d3d5-5060-5218b2ba75ac
Content: PowerPoint add-in tutorial - Office Add-ins
Content Source: docs/tutorials/powerpoint-tutorial.md
Product: powerpoint
Technology: add-ins
GitHub Login: @o365devx
Microsoft Alias: o365devx

This needs to be done for the home.js file too - still looking for the best way to do that. The command line cache deletion keeps me going until I find that.

@LevDev1 thanks for the feedback about this. I've reassigned this issue to @Rick-Kirkham, since he was the person who engaged on the Stack Overflow thread you've linked to, and therefore has more context about this topic. @Rick-Kirkham -- FWIW, IMO, seems like this type of information belongs with the other caching-related info in the docs, rather than in the PowerPoint tutorial where this issue was logged.

Fixed by PR 1421.


feat: release v5.23.0

Improves overall documentation
Introduces support for earliest/latest parameters for singleSelectSplunkSearch type in alerts
app.conf["install"]["is_configured"] is now defaults to "false"
Internal improvements for the repository

:tada: This PR is included in version 5.23.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Feature/GitHub star

As discussed on #464
Added github start badge.

Duplicate of #603

@lucacasonato Yes, it is, since there was no activity so I pushed my changes which I suggested. If that is going to merge then feel free to close this or let me know if anything needs to change, I am available to do.

closed in favor of https://github.com/denoland/deno_website2/pull/603


[WFLY-19478] Remove usage of deprecated SimpleResourceDefinition methods

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

Oh, and thanks! This is helpful.

Ah, but this one does need to be changed to override the OperationStepHandler variant instead:
    /**
     * Overrides the default impl to use a special definition of the add op that includes additional parameter
     * {@link #DEFAULT_CLUSTERED_SFSB_CACHE}
     * {@inheritDoc}
     */
    @Override
    protected void registerAddOperation(ManagementResourceRegistration registration, AbstractAddStepHandler handler, OperationEntry.Flag... flags) {
        OperationDefinition od = new SimpleOperationDefinitionBuilder(ADD, getResourceDescriptionResolver())
                .setParameters(ATTRIBUTES)
                .addParameter(DEFAULT_CLUSTERED_SFSB_CACHE)
                .withFlags(flags)
                .build();
        registration.registerOperationHandler(od, handler);
    }

Perhaps revert what you have here and fix that instead?

Hi @bstansberry, thank you for your review, I have updated PR accordingly.

Thanks @lvydra


ORC-920: Use junit.version and mockito.version property and bump junit to 5.7.2

What changes were proposed in this pull request?
This PR aims to define junit.version and mockito.version properties and use it.
In addition, junit.version is updated to 5.7.2.
Why are the changes needed?
Currently, Dependabot makes separate PRs per dependencies.
This will advise it to make one PR in the future.
How was this patch tested?
Pass the CIs.
Closes #831

cc @williamhyun

Thank you, @williamhyun !

I backported this to branch-1.7 for Apache ORC 1.7.0.


XSLT Function stroom:hex-to-oct() is returning a decimal

<xsl:value-of select="stroom:hex-to-oct('FF')" />
returns
255

Is XSLT helpfully converting it back to decimal for me?  If so how do I use it?


Forwarding client's cookies to API when calling from on server

Sorry if this is a noob question. What happens when the API call needs user's session. I assume those session cookies won't be forwarded to the API endpoint when the rendering happens on server-side?
Any idea on how should I forward user's cookies to the API endpoint. I think it will be somewhere inside HttpClient.js?
Thanks

@asadm it should be sending the cookies automatically, but the problem I'm having with this is that it doesn't work on the server side. I keep getting Error: Failed to serialize user into session when the server sends a login request. In the browser or with curl, the login request works fine, though.

Feel free to send a PR with axios or fetch etc.

@jaredpalmer I haven't figured out how to solve the problem yet, but I can try to work on a PR once it's fixed. :sweat_smile:

I'm getting the same error with fetch, by the way.

Does this help? http://stackoverflow.com/questions/19948816/error-failed-to-serialize-user-into-session

@jaredpalmer not at all. This has to do with universal rendering, passport works fine. It also works fine to send requests from the browser or curl, just when the server in this boilerplate makes the request to get the initialState, it fails. I assume this is because the server-side doesn't store the user session. Is there any way to do this?

I think the easiest way to work around this issue is to return a token and store that in the state, then later authenticate requests with the token. Would be nice if client/server shared the correct user session automatically, though :smile:

This might be a related issue: http://stackoverflow.com/a/12735003/702288

can you link to an example repo? also are you using express-session?

@jaredpalmer Unfortunately I can't share the code, but it's not hard to add passport to your existing boilerplate: http://passportjs.org/docs
Yes I am using express-session. I have tried using superagent.agent(), but it didn't solve the issue.

@jaredpalmer I just cloned a fresh instance of your boilerplate and added express-session. Setting req.session crashes it (I did in src/server/api/posts.js). You can check this out at my fork omnidan/react-production-starter (relevant commit).

Ok, nevermind my issue seems to have been unrelated to this :blush:
I just found out that it couldn't serialize the user because the data was sent via json from the server, but urlencoded from the browser/curl. adding app.use(bodyParser.json()) to my server fixed the issue for me.

I still have the problem that the session doesn't seem to work correctly. I changed one request to be (for testing purposes):
callAPI: () => http.post('http://localhost:3003/login', {
      email<EMAIL_ADDRESS>      password: 'test'
    }).then(data => {
      console.log('login', data)
      return http.get('http://localhost:3003/user')
        .then(data => console.log('user data:', data))
    }),

The output of this is:
login { email<EMAIL_ADDRESS>}
user data: { error: 'Not authenticated, please log in.' }

If I do the /login request and then access /user in the browser, it works fine.

I kinda solved it this way:
First, when user requests the page from express server. The req holds any session cookies in it's header. I just forward req.headers.cookie down till HttpClient.js:
get: (cookies, path) => new Promise((resolve, reject) => {
    request
      .get(getUrl(path))
      .accept('application/json')
      .set("Cookie",cookies)
      .end((err, res) => {
        if (err) {
          if (err.status === 404) {
            resolve(null);
          } else {
            reject(err);
          }
        } else {
          resolve(res.body);
        }
      });
  }),

Does this make sense?

@asadm good idea! I was gonna solve this by storing a login token in the redux store, but this is pretty much the same concept using cookies. I'll try this out in a bit :grin:

@omnidan I have done this approach and it's pretty similar to the one @asadm provided.
  getWithToken: (path, token) => new Promise((resolve, reject) => {
    request
      .get(getUrl(path))
      .set('X-User-Token', token)
      .accept('application/json')
      .end((err, res) => {
        if (err) {
          reject(err);
        } else {
          resolve(res.body);
        }
      });
  })


Redfish date time end to end automation

[ ] Test plan
[ ] Automation

Automation completed. Thanks @sivassrr


AlgoFetch Component only user main-net

unless I am mistaken, the component AlgoFetch only use main-net.
On the component ( https://github.com/headline-design/pipeline-ui/blob/main/src/AlgoFetch/index.js ) we can see hard coded :
let url2 =       'https://algoexplorerapi.io/idx2/v2/accounts/' +       props.address +       '/transactions?limit=5'
it's not possible to add this url management on the Pipeline class ( https://github.com/headline-design/pipeline-ui/blob/main/src/Pipeline/index.js ) and add 'main' props on the component or on an Pipeline variable ?
an other improve on this  the AlgoFetch component, add an limit props to manage the     '/transactions?limit=5'`

You are correct - mainNet only right now. We will be adding this feature - thanks! Also, you can submit a pr for the doc page to specify that AlgoFetch is mainNet only right now.

Ok ..
I will try to push on the documentation or directly a proposal on the code. Thank you for your Quick response.

Closing this issue.


Allow html prettyPrint options

Package uses 'commonjs-html-prettyprinter'. Allow for options to be passed to html.prettyPrint function.

Requesting this PR to be merged into master. This PR allows for options to be passed to prettyPrint function. (e.g. prettyPrintOptions: { max_char: Infinity } to prevent line wrapping)


Allow -a option to add a file without explicit path

That is, "-a fred" fails, but "-a ./fred" works. The check_file method is probably seeing None as the directory for "-a fred", and that's not writeable. Should just be able to make the directory "." if not specified.

I'll fix and merge this one.
a
Alex Kleider (set from my current gizmo)
------- Original Message -------
On Saturday, August 20th, 2022 at 9:43 AM, Leam Hall @.***> wrote:

That is, "-a fred" fails, but "-a ./fred" works. The check_file method is probably seeing None as the directory for "-a fred", and that's not writeable. Should just be able to make the directory "." if not specified.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Fixed issue 14, ready to merge when you approve it.


[SUGGESTION] Use steam app protocols

Use steam app protocols in About > SCPSL on steam

The About/SCPSL item is supposed to show information about SCP: Secret Laboratory (if e.g. someone downloads the tool out of curiosity and wants to learn what it's for), not to open the game. Although an "open SL" shortcut seems like a good idea (e.g. View/Launch SCPSL)

In the next release there will be a shortcut to launch sl. I think I can finally mark this one as closed


ERROR: Failed to download m3u8 information: '' (caused by BadStatusLine("''",))

Hey, Folks!
I was in the middle, of downloading a series of dorumentaries, first two worked like charms, at the third I got an error.
Then I got an error on everything, that I attempted.
So, I connected to a remote machine, and got the same error there.
As you can see from the verbose protocol, Youtube-dl is up to date.
This is the verbose output:
youtube-dl --verbose https://tvthek.orf.at/profile/zeit-geschichte/13606695/zeit-geschichte-Wie-wir-wurden-Was-wir-sind-Generation-Oesterreich-6-6-1995-2008/14027254
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'--verbose', u'https://tvthek.orf.at/profile/zeit-geschichte/13606695/zeit-geschichte-Wie-wir-wurden-Was-wir-sind-Generation-Oesterreich-6-6-1995-2008/14027254']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2019.09.28
[debug] Python version 2.7.13 (CPython) - Linux-4.9.0-9-amd64-x86_64-with-debian-9.9
[debug] exe versions: ffmpeg 3.2.14-1, ffprobe 3.2.14-1, phantomjs 2.1.1
[debug] Proxy map: {}
[orf:tvthek] 14027254: Downloading webpage
[orf:tvthek] 14566586: Downloading m3u8 information
ERROR: Failed to download m3u8 information: '' (caused by BadStatusLine("''",)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
File "/usr/bin/youtube-dl/youtube_dl/extractor/common.py", line 627, in _request_webpage
return self._downloader.urlopen(url_or_request)
File "/usr/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 2237, in urlopen
return self._opener.open(req, timeout=self._socket_timeout)
File "/usr/lib/python2.7/urllib2.py", line 429, in open
response = self._open(req, data)
File "/usr/lib/python2.7/urllib2.py", line 447, in _open
'_open', req)
File "/usr/lib/python2.7/urllib2.py", line 407, in _call_chain
result = func(*args)
File "/usr/bin/youtube-dl/youtube_dl/utils.py", line 2724, in https_open
req, **kwargs)
File "/usr/lib/python2.7/urllib2.py", line 1201, in do_open
r = h.getresponse(buffering=True)
File "/usr/lib/python2.7/httplib.py", line 1121, in getresponse
response.begin()
File "/usr/lib/python2.7/httplib.py", line 438, in begin
version, status, reason = self._read_status()
File "/usr/lib/python2.7/httplib.py", line 402, in _read_status
raise BadStatusLine(line)
As stated, the error appears with whatever you attempt to download from that site.
They must have changed something today around 19:30 CET.
These documentaries will be available only 4 more days.
I'd therefore be very grateful, if you could fix the problem swiftly.
My appreciation in advance, and yours, Claus

Network issue on your side.


proper Cancellation?

Currently the app freezes for seconds when clicking the red X of the console window -- which is nearly the same as a task kill.
How to stop Socket.AcceptAsync from "blocking" infinitely?
Further:

what about proper disposing of IDisposables?
async Main method does not contain a single await .. why not await ProcessLinesAsync?
no .editorconfig so first contact with ctrl+k-d reformats everything

Keep in mind that this is an introductionary sample, not one that can and should be used in production directly. It's more or less a demo to get your feets wet with pipelines.
If this demo would be productized, then it would be too big for an intro.

Currently the app freezes for seconds when clicking the red X of the console window

In this demo there is no unbind on the socket, etc. and nothing that stops the accept-loop
https://github.com/davidfowl/TcpEcho/blob/07ad4df181151850974219b8ebf434eee37778a4/src/Program.cs#L22-L26
You could run this server in a generic host, and leverage cancellation as well as proper disposing.
But, as said before, this would be too much for an intro...

async Main method does not contain a single await

Here it is: https://github.com/davidfowl/TcpEcho/blob/07ad4df181151850974219b8ebf434eee37778a4/src/Program.cs#L24

.. why not await ProcessLinesAsync?

Imagine
https://github.com/davidfowl/TcpEcho/blob/07ad4df181151850974219b8ebf434eee37778a4/src/Program.cs#L25
would be written as
await ProcessLinesAsync(socket);

what would happen?

The accept-loop asynchronously awaits a client-connection https://github.com/davidfowl/TcpEcho/blob/07ad4df181151850974219b8ebf434eee37778a4/src/Program.cs#L24
ProcessLinesAsync is "started" for that client, the accept-loop asynchronously awaits ProcessLinesAsync to be completed
ProcessLinesAsync will complete only when reading and writing are done https://github.com/davidfowl/TcpEcho/blob/07ad4df181151850974219b8ebf434eee37778a4/src/Program.cs#L37
goto point 1

So this would handle one client at a time and when the one is done, the next client is handled.
Not very scalable.
With the implementation as is, the following happens:

The accept-loop asynchronously awaits a client-connection
ProcessLinesAsync is "started" for that client, the returning Task is thrown away, because we don't want to await it
goto point 1

So when ProcessLinesAsync is started -- or to be more precise, when it yields the first time (await) -- the accept-loop can await for other clients. Quite more scalable.
Mature implementations like Kestrel have a more complex accept-loop.

Alright, was not aware that this would become single-user then.
But the most important question is still unanswered:
How to cancel await listenSocket.AcceptAsync(); in a safe way?

Have a look at a generic host implementation of this demo.
Note that this is still a demo with lots of room for improvement (like proper connection-handling, exception-handling, and so on). If you'd like to see how to handle the things in a correct way, you can drive inspriation from Kestrel

So disposing the Socket instance on a random thread and catching exceptions selectively by observing an unloading state field is the most safe way? IMHO this AcceptAsync method should natively support CancellationToken. The same goes for UdpClient.ReceiveAsync().
Any async method not accepting a CancellationToken looks smelly to me.

Any async method not accepting a CancellationToken looks smelly to me.

Agreed.
I believe the problems here with accepting a CancellationToken are rooted in how managed code and the operating system interact on the sockets. The managed part is more or less just a wrapper for the os-native Berkeley socket, so cancelling on the managed side has to cancel in a proper way on the os-side too, and this seems to be not so trivial.
https://github.com/dotnet/corefx/issues/5749 is one issue that tracks this desire.

Adding two related issues:

https://github.com/dotnet/corefx/issues/5749 Add a clean way to cancel a long-running Socket operation (e.g. accept or connect)
https://github.com/dotnet/corefx/issues/9698 Add reliable timeouts to socket operations

Send me a pull request 😄


java.util.LinkedHashMap cannot be cast to org.springframework.social.gitlab.api.project.Project

Since this code, PagedList does not return a List<T> with getContent but a List<LinkedHashMap<?,?>>.
For example, if after this line, I add:
assertThat(projects.get(0).getId(), is(4L));

I get a java.util.LinkedHashMap cannot be cast to org.springframework.social.gitlab.api.project.Project.

Seems that Jackson has not enough type information to transform the response body into something usefull. So the fallback Map<String, Object> is used.
I will check what we can do here. The whole paging related code should be reviewed.

All tests are green.
Found solution on http://stackoverflow.com/questions/21987295/using-spring-resttemplate-in-generic-method-with-generic-parameter

@pbit I know this is a 2-year old post, but how exactly does the stackoverflow link resolve the issue?
According to the method signature of org.springframework.data.domain.Slice.getContent(), a List<T> should be returned. Instead, all you get is a LinkedHashMap that obviously cannot be traversed properly.
I have even tried a Reflection-based workaround, but to no avail since the depth of my T is just too much.
Any ideas @pbit @obazoud ?

hi @gminog Will try to revert my brain to 2015 ;)
We create a ParameterizedTypeReference to keep the information which is lost due to Javas  type erasure.
see this helper method

... the depth of my T is just too much

You mean something like List<Foo<Bar>> ?
The solution found in this issue works for a single level of parameters e.g. List<Foo>.
Can you provide some code?

Thank you, I will check it and let you know!
In the meantime, my Foo contains properties that are Lists of other classes that also contain Lists of other classes. Boy, that is endless!

This stuff is just needed for the top level Collection.
Nested types are handled by Jackson, which can be customized in multiple ways.
You could even try to create concrete classes for your Lists.
class FooList extends ArrayList<Foo>

Closing this as its not related to this issue.


Unable to run on ubuntu 15.04

Brief Description: When trying to run python openbazaar.py start I am getting the error
ImportError: No module named sanitizer
When importing bleach.
Operating System (OS and version): Ubuntu
OpenBazaar version: Master
Hardware: LXD VM
Reproducible (Always / Almost Always / Sometimes / Rarely / Couldn't Reproduce): Always
Steps to reproduce:

Clone
Install reqs
Attempt to start

Observed Behavior:
Error
Expected Behavior:
OpenBazaar would start
Additional info (links, images, etc go here):
I was able to get things to run by bumping the bleach version number to 1.4.3.
Full stack below.
Traceback (most recent call last):
File "openbazaard.py", line 11, in
from api.ws import WSFactory, AuthenticatedWebSocketProtocol, AuthenticatedWebSocketFactory
File "/root/OpenBazaar-Server/api/ws.py", line 15, in
from api.utils import smart_unicode, sanitize_html
File "/root/OpenBazaar-Server/api/utils.py", line 1, in
import bleach
File "/usr/local/lib/python2.7/dist-packages/bleach/init.py", line 8, in
from html5lib.sanitizer import HTMLSanitizer
ImportError: No module named sanitizer

This was broken in the lastest update of html5lib. Run this to clock it back
sudo pip install html5lib==0.999

installing html5lib==0.999 initially worked for me but now upon reboot its complaining about this:
sudo -H python /home/ubuntu/OpenBazaar-Server/openbazaard.py start -da <IP_ADDRESS>
Warning: configuration file not found: (ob.cfg), using default values
Traceback (most recent call last):
File "/home/ubuntu/OpenBazaar-Server/openbazaard.py", line 11, in
from api.ws import WSFactory, AuthenticatedWebSocketProtocol, AuthenticatedWebSocketFactory
File "/home/ubuntu/OpenBazaar-Server/api/ws.py", line 16, in
from config import DATA_FOLDER, str_to_bool
File "/home/ubuntu/OpenBazaar-Server/config.py", line 163, in
DATA_FOLDER = _platform_agnostic_data_path(cfg.get('CONSTANTS', 'DATA_FOLDER'))
File "/usr/lib/python2.7/ConfigParser.py", line 607, in get
raise NoSectionError(section)
ConfigParser.NoSectionError: No section: 'CONSTANTS'

Dont run it as root

nevermind the prev comment, run it in the local directory

Closing.


Cleanup core/views/generic.py file

Fixed withdraw endpoint query params access
was: request.query_params.get("usr")
corrected: request.query_params.get("usr")
Added missing response classes
Removed old comments and commeted out code
Deleted the unused auth_bearer.py file

lgtm


Incoming whisper results in NoneType error while handling commands

Describe the bug
An incoming whisper received in event_message causes handle_commands to die with a NoneType object has no attribte _ws error
Are you using TwitchIO within a Discord Bot?
No
What commit of TwitchIO are you using?
pip show twitchio
That didn't do anything for me (using pipenv).  This is in my pipfile lock
       "twitchio": {
            "hashes": [
                "sha256:3ebb03425bf486a0a3812608a073adc54bfa458032748c37be06a4da38390970",
                "sha256:cd7768c173ce4364dc262e1cdd0c4c4291d768fdf387b10b8921bd4d18a3b07b"
            ],
            "index": "pypi",
            "version": "==2.0.0b7"
        },

To Reproduce

have this event_message:

    async def event_message(self, message):
        print(f"{message.author} / {message.author}: {message.content}")

        # prevent us responding to ourselves.
        if message.echo:
            return

        await self.handle_commands(message)

        await message.channel.send(f"splunge {message.content}")

Run the bot.
in the chat for the stage the bot is listening in, post a message to test the bot is working.  typing "borK" should result in a response of "splunge bork"
in the chat, enter the command /w botname hello (or use other mechanisms

Expected behaviour
The event_message method processes the event (in this case, sending back to the channel the "splunge bork" response
What actually happened is this:
pipenv run python bot.py
Loading .env environment variables...
Logged in as | borkbot_songbot
<PartialChatter name: borkwaremarkd, channel: <Channel name: borkwaremarkd>> / <PartialChatter name: borkwaremarkd, channel: <Channel name: borkwaremarkd>>: spoon
None / None: splunge spoon
<WhisperChatter name: borkwaremarkd> / <WhisperChatter name: borkwaremarkd>: hello
Traceback (most recent call last):
  File "/Users/markd/.local/share/virtualenvs/zandemonium-w44pKbCL/lib/python3.7/site-packages/twitchio/client.py", line 149, in wrapped
    await func(*args)
  File "bot.py", line 35, in event_message
    await self.handle_commands(message)
  File "/Users/markd/.local/share/virtualenvs/zandemonium-w44pKbCL/lib/python3.7/site-packages/twitchio/ext/commands/bot.py", line 253, in handle_commands
    context = await self.get_context(message)
  File "/Users/markd/.local/share/virtualenvs/zandemonium-w44pKbCL/lib/python3.7/site-packages/twitchio/ext/commands/bot.py", line 223, in get_context
    return cls(message=message, prefix=prefix, valid=False, bot=self)
  File "/Users/markd/.local/share/virtualenvs/zandemonium-w44pKbCL/lib/python3.7/site-packages/twitchio/ext/commands/core.py", line 388, in __init__
    self._ws = self.channel._ws
AttributeError: 'NoneType' object has no attribute '_ws'

In particular, when running self.handle_commands to process out ! commands, it tries to access a NoneType
(please complete the following information):

OS: MacOS Catalina 10.15.6 (19G73)
Python 3.9.5

Additional context
This is my pipfile:
[[source]]
url = "https://pypi.org/simple"
verify_ssl = true
name = "pypi"

[packages]
twitchio = "*"

[dev-packages]

[requires]
python_version = "3.7"

[pipenv]
allow_prereleases = true

and I installed via pipenv install twitchio --pre to get v2.
In general, though, I have no idea what I'm doing.  I'm markd2#7307 in the Pythonista Dicord and originally posted this there: https://discord.com/channels/490948346773635102/491048464831086592/848718727032864799
Thanks!  Enjoying twitchio.  Making bots is fun :-)

Fixed, thanks


No data is returned for a few pairs using - currentCoinBalance

Describe your environment

Operating system: macos - 10.14.5
Python Version: 3.7.10

Describe the problem:
currentCoinBalance returns no data for recently added pairs.
this issue is observed in USDC, BUSD, RUNE, SOL, SAND, MANA etc (while opening issue, there were most recently added)
Steps to reproduce:

fetch current coin balance via currentCoinBalance function.

Observed Results:

What happened?

the data field returns empty value.

What did you expect to happen?

the data field to return 0 balance.

Relevant code exceptions or logs
Code Sample, a copy-pastable example
# Your code here
print(bitbnsObj.currentCoinBalance('BUSD'))

logs, a copy-pastable example
{'code': 200, 'data': {}, 'error': None, 'status': 1}

The work around this issue is, buying a small quantity of the asset, and then retrying the function, it seems to be working(even if the balance is reduced to 0 later)
print(bitbnsObj.currentCoinBalance('BUSD'))

{
 'code': 200,
 'data': {'availableorderBUSD': 0, 'inorderBUSD': 29},
 'error': None,
 'status': 1
}

This is sorted


Fix whitespaces

After merge of https://github.com/EnAccess/micropowermanager-cloud/pull/4 few whitespace issues are still left in the code base. Fixing them here.

LGTM 💪


null blocks in bucket-index.json.gz for cortex ruler on startup

e2e tests for opni-agent fail 100% of the time.
There are several symptoms i've observed besides the 503s when the agent tries to write metrics :

a null block in bucket-index.json.gz for cortex rules:
cortex compactor crash
rule sync pending state for agent

This happens at startup, and there is no bucket-index.json.gz file existing after startup, it must be getting written to later

Content of the rules bucket-index.json file is :
version	1
blocks	null
block_deletion_marks	[]
updated_at<PHONE_NUMBER>

2022-08-17T16:10:42.026Z ERROR test.env.bench failed to send batch {"agent": "ba3cef75-82a2-42e3-9dcd-35ed7c27e464", "error": "server returned HTTP status 503 Service Unavailable: "}
12:10PM INFO  actions.e2e.test._exec | #54 960.2     2022-08-17T16:10:42.032Z ERROR test.env.bench failed to send batch {"agent": "ba3cef75-82a2-42e3-9dcd-35ed7c27e464", "error": "server returned HTTP status 503 Service Unavailable: "}
12:10PM INFO  actions.e2e.test._exec | #54 960.2     2022-08-17T16:10:42.035Z ERROR test.env.bench failed to send batch {"agent": "ba3cef75-82a2-42e3-9dcd-35ed7c27e464", "error": "server returned HTTP status 503 Service Unavailable: "}
12:10PM INFO  actions.e2e.test._exec | #54 960.2     2022-08-17T16:10:43.027Z ERROR test.env.bench failed to send batch {"agent": "ba3cef75-82a2-42e3-9dcd-35ed7c27e464", "error": "server returned HTTP status 503 Service Unavailable: "}
12:10PM INFO  actions.e2e.test._exec | #54 960.2     2022-08-17T16:10:43.037Z ERROR test.env.bench failed to send batch {"agent": "ba3cef75-82a2-42e3-9dcd-35ed7c27e464", "error": "server returned HTTP status 503 Service Unavailable: "}
12:10PM INFO  actions.e2e.test._exec | #54 960.2     2022-08-17T16:10:43.040Z ERROR test.env.bench failed to send batch {"agent": "ba3cef75-82a2-42e3-9dcd-35ed7c27e464", "error": "server returned HTTP status 503 Service Unavailable: "}
12:10PM INFO  actions.e2e.test._exec | #54 960.2     2022-08-17T16:10:44.024Z ERROR test.env.bench failed to send batch {"agent": "ba3cef75-82a2-42e3-9dcd-35ed7c27e464", "error": "server returned HTTP status 503 Service Unavailable: "}
12:10PM INFO  actions.e2e.test._exec | #54 960.2     2022-08-17T16:10:44.032Z ERROR test.env.bench failed to send batch {"agent": "ba3cef75-82a2-42e3-9dcd-35ed7c27e464", "error": "server returned HTTP status 503 Service Unavailable: "}
12:10PM INFO  actions.e2e.test._exec | #54 960.2     2022-08-17T16:10:44.036Z ERROR test.env.bench failed to send batch {"agent": "ba3cef75-82a2-42e3-9dcd-35ed7c27e464", "error": "server returned HTTP status 503 Service Unavailable: "}
12:10PM INFO  actions.e2e.test._exec | #54 960.2     2022-08-17T16:10:45.027Z ERROR test.env.bench failed to send batch {"agent": "ba3cef75-82a2-42e3-9dcd-35ed7c27e464", "error": "server returned HTTP status 503 Service Unavailable: "}
12:10PM INFO  actions.e2e.test._exec | #54 960.2     2022-08-17T16:10:45.038Z ERROR test.env.bench failed to send batch {"agent": "ba3cef75-82a2-42e3-9dcd-35ed7c27e464", "error": "server returned HTTP status 503 Service Unavailable: "}
12:10PM INFO  actions.e2e.test._exec | #54 960.2     2022-08-17T16:10:45.044Z ERROR test.env.bench failed to send batch {"agent": "ba3cef75-82a2-42e3-9dcd-35ed7c27e464", "error": "server returned HTTP status 503 Service Unavailable: "}
12:10PM INFO  actions.e2e.test._exec | #54 960.2     2022-08-17T16:10:46.022Z ERROR test.env.bench failed to send batch {"agent": "ba3cef75-82a2-42e3-9dcd-35ed7c27e464", "error": "server returned HTTP status 503 Service Unavailable: "}
12:10PM INFO  actions.e2e.test._exec | #54 960.2     2022-08-17T16:10:46.028Z ERROR test.env.bench failed to send batch {"agent": "ba3cef75-82a2-42e3-9dcd-35ed7c27e464", "error": "server returned HTTP status 503 Service Unavailable: "}
12:10PM INFO  actions.e2e.test._exec | #54 960.2     2022-08-17T16:10:46.031Z ERROR test.env.bench failed to send batch {"agent": "ba3cef75-82a2-42e3-9dcd-35ed7c27e464", "error": "server returned HTTP status 503 Service Unavailable: "}
12:10PM INFO  actions.e2e.test._exec | #54 960.2     2022-08-17T16:10:47.023Z ERROR test.env.bench failed to send batch {"agent": "ba3cef75-82a2-42e3-9dcd-35ed7c27e464", "error": "server returned HTTP status 503 Service Unavailable: "}
12:10PM INFO  actions.e2e.test._exec | #54 960.2     2022-08-17T16:10:47.028Z ERROR test.env.bench failed to send batch {"agent": "ba3cef75-82a2-42e3-9dcd-35ed7c27e464", "error": "server returned HTTP status 503 Service Unavailable: "}
12:10PM INFO  actions.e2e.test._exec | #54 960.2     2022-08-17T16:10:47.032Z ERROR test.env.bench failed to send batch {"agent": "ba3cef75-82a2-42e3-9dcd-35ed7c27e464", "error": "server returned HTTP status 503 Service Unavailable: "}
12:10PM INFO  actions.e2e.test._exec | #54 960.2     2022-08-17T16:10:48.026Z ERROR test.env.bench failed to send batch {"agent": "ba3cef75-82a2-42e3-9dcd-35ed7c27e464", "error": "server returned HTTP status 503 Service Unavailable: "}
12:10PM INFO  actions.e2e.test._exec | #54 960.2     2022-08-17T16:10:48.036Z ERROR test.env.bench failed to send batch {"agent": "ba3cef75-82a2-42e3-9dcd-35ed7c27e464", "error": "server returned HTTP status 503 Service Unavailable: "}
12:10PM INFO  actions.e2e.test._exec | #54 960.2     2022-08-17T16:10:48.040Z ERROR test.env.bench failed to send batch {"agent": "ba3cef75-82a2-42e3-9dcd-35ed7c27e464", "error": "server returned HTTP status 503 Service Unavailable: "}
12:10PM INFO  actions.e2e.test._exec | #54 960.2     2022-08-17T16:10:49.022Z ERROR test.env.bench failed to send batch {"agent": "ba3cef75-82a2-42e3-9dcd-35ed7c27e464", "error": "server returned HTTP status 503 Service Unavailable: "}
12:10PM INFO  actions.e2e.test._exec | #54 960.2     2022-08-17T16:10:49.028Z ERROR test.env.bench failed to send batch {"agent": "ba3cef75-82a2-42e3-9dcd-35ed7c27e464", "error": "server returned HTTP status 503 Service Unavailable: "}
12:10PM INFO  actions.e2e.test._exec | #54 960.2     2022-08-17T16:10:49.032Z ERROR test.env.bench failed to send batch {"agent": "ba3cef75-82a2-42e3-9dcd-35ed7c27e464", "error": "server returned HTTP status 503 Service Unavailable: "}
12:10PM INFO  actions.e2e.test._exec | #54 960.2     2022-08-17T16:10:50.027Z ERROR test.env.bench failed to send batch {"agent": "ba3cef75-82a2-42e3-9dcd-35ed7c27e464", "error": "server returned HTTP status 503 Service Unavailable: "}
12:10PM INFO  actions.e2e.test._exec | #54 960.2     2022-08-17T16:10:50.037Z ERROR test.env.bench failed to send batch {"agent": "ba3cef75-82a2-42e3-9dcd-35ed7c27e464", "error": "server returned HTTP status 503 Service Unavailable: "}
12:10PM INFO  actions.e2e.test._exec | #54 960.2     2022-08-17T16:10:50.042Z ERROR test.env.bench failed to send batch {"agent": "ba3cef75-82a2-42e3-9dcd-35ed7c27e464", "error": "server returned HTTP status 503 Service Unavailable: "}
12:10PM INFO  actions.e2e.test._exec | #54 960.2     2022-08-17T16:10:51.026Z ERROR test.env.bench failed to send batch {"agent": "ba3cef75-82a2-42e3-9dcd-35ed7c27e464", "error": "server returned HTTP status 503 Service Unavailable: "}
12:10PM INFO  actions.e2e.test._exec | #54 960.2     2022-08-17T16:10:51.035Z ERROR test.env.bench failed to send batch {"agent": "ba3cef75-82a2-42e3-9dcd-35ed7c27e464", "error": "server returned HTTP status 503 Service Unavailable: "}
12:10PM INFO  actions.e2e.test._exec | #54 960.2     2022-08-17T16:10:51.039Z ERROR test.env.bench failed to send batch {"agent": "ba3cef75-82a2-42e3-9dcd-35ed7c27e464", "error": "server returned HTTP status 503 Service Unavailable: "}
12:10PM INFO  actions.e2e.test._exec | #54 960.2   << End Captured GinkgoWriter Output
12:10PM INFO  actions.e2e.test._exec | #54 960.2
12:10PM INFO  actions.e2e.test._exec | #54 960.2   Timed out after 10.001s.
12:10PM INFO  actions.e2e.test._exec | #54 960.2   Expected
12:10PM INFO  actions.e2e.test._exec | #54 960.2       : Rule Sync Pending
12:10PM INFO  actions.e2e.test._exec | #54 960.2   to equal
12:10PM INFO  actions.e2e.test._exec | #54 960.2       : ok

Possibly caused by #517


WIP: Fix linux circle docker

Linux broke. Testing this.

          1 Warning

:warning:
Pull Request is Work in Progress

  Generated by :no_entry_sign: Danger


Question: How can I retrieve an enum member by value?

Say for example I have this enum:
var Members = new Enum({
  A: "a",
  B: "b"
});

I'd like to be able to do something like:
var member = Members.byValue('a');

Did I miss something in the documentation, or is this currently impossible?

What is the reason for not allowing strings as values?  I see this as a pretty common use case.  In my system, use strings as values so that they are human-readable in the database.  This means that in addition to wanting to use strings as values, I'd also be looking for an option to have a constructor option that changes the behaviour of toString/toJSON to return the value rather than the key.

Try to do a pull request...


docs: add opts_extend

Description
I completely forgot we should probably add this, just in case. Not sure it's worth explaining why opts_extend is there?
Related Discussion
https://github.com/olimorris/codecompanion.nvim/discussions/578#discussion-7758770
Screenshots

Checklist

[x] I've read the CONTRIBUTING guidelines and have adhered to them in this PR
[x] I've updated the README and ran the make docs command

Awesome thank you!


Refactor Code

Changed anime to animes for plural uses
Changed aniList to animeList
Changed addressBook to animeList

Codecov Report

Merging #50 (c3b19d4) into master (2705ace) will not change coverage.
The diff coverage is 67.39%.

@@            Coverage Diff            @@
##             master      #50   +/-   ##
=========================================
  Coverage     69.85%   69.85%
  Complexity      360      360
=========================================
  Files            67       67
  Lines          1118     1118
  Branches        104      104
=========================================
  Hits            781      781
  Misses          308      308
  Partials         29       29

Impacted Files
Coverage Δ

src/main/java/seedu/anilist/MainApp.java
0.00% <0.00%> (ø)

...in/java/seedu/anilist/commons/core/LogsCenter.java
78.37% <ø> (ø)

src/main/java/seedu/anilist/model/Model.java
100.00% <ø> (ø)

src/main/java/seedu/anilist/ui/MainWindow.java
0.00% <0.00%> (ø)

...rc/main/java/seedu/anilist/logic/LogicManager.java
75.00% <60.00%> (ø)

src/main/java/seedu/anilist/model/UserPrefs.java
80.00% <87.50%> (ø)

...ava/seedu/anilist/logic/commands/ClearCommand.java
100.00% <100.00%> (ø)

...va/seedu/anilist/logic/parser/AnimeListParser.java
100.00% <100.00%> (ø)

...rc/main/java/seedu/anilist/model/ModelManager.java
100.00% <100.00%> (ø)

...va/seedu/anilist/storage/JsonAnimeListStorage.java
85.71% <100.00%> (ø)

... and 2 more

Continue to review full report at Codecov.

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


No LICENSE

I want to steal your hard work and to do that legally, I need a license

@Try3D Thank you so much for bringing this issue to my notice. I have added the license now.


Dual-stack: /healthz/ready should have both v4/v6 wildcard listeners

Please provide a description of this PR:

Enhance DualStack integration tests as per RFC - added gateway readiness probe tests
Fix issue with probe test as mentioned in https://github.com/istio/istio/issues/47013

@kfaseela: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
4736565675759dad65b41d6ba184e71659273716
link
unknown
/test unit-tests-arm64

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@kfaseela: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
4736565675759dad65b41d6ba184e71659273716
link
unknown
/test unit-tests-arm64

integ-pilot_istio
4736565675759dad65b41d6ba184e71659273716
link
unknown
/test integ-pilot

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test unit-tests-arm64

@kfaseela: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-pilot_istio
4736565675759dad65b41d6ba184e71659273716
link
unknown
/test integ-pilot

unit-tests-arm64_istio
4736565675759dad65b41d6ba184e71659273716
link
true
/test unit-tests-arm64

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@kfaseela: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-pilot-multicluster_istio
4736565675759dad65b41d6ba184e71659273716
link
true
/test integ-pilot-multicluster

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test integ-pilot-multicluster


[MagicAPI] can't get spell with UTF-8 char like (é,è,à,etc)

i try get a spell named F_Percée with the API its not work-

I seem to have missed a bunch of issues.. sorry
I don't know why this would be, though, I build with utf-8 encoding- does your plugin do the same?

I seem to have missed a bunch of issues.. sorry
I don't know why this would be, though, I build with utf-8 encoding- does your plugin do the same?

No problem.
Yep UTF-8, i think its maybe because some spell in config contains UTF8 char :/
(i have update spell with utf8 char btw. but i wanna tell you ^^...)

Please let me know if this is still an issue for you and I will look more closely, I use UTF in my config files a lot now so I'm not sure what the problem would've been.


Add README notice about Next.js 13 app router statically typed links

This is a notice for someone who has searched for a solution and found this repo, to tell them they can use a built in feature that may fit their needs.
Although I love the effort that has gone into this project, I think many people will prefer to consider a solution that doesn't need an extra dependency.

Related Issue (now closed) https://github.com/tatethurston/nextjs-routes/issues/174

Thanks @mcapodici

@mcapodici Does Next's experimental option still only work for the app directory, or does it now support pages?

@mcapodici Does Next's experimental option still only work for the app directory, or does it now support pages?

Looks that way: https://nextjs.org/docs/app/api-reference/next-config-js/typedRoutes


Add projects to components

This PR simply adds some props that allow you to configure the project execution that squiggle runs in.

Codecov Report

Merging #1142 (d619794) into develop (d9f4171) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop    #1142   +/-   ##
========================================
  Coverage    59.18%   59.18%
========================================
  Files           98       98
  Lines         4496     4496
========================================
  Hits          2661     2661
  Misses        1835     1835

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

I've decided to go with only two interfaces:

Standalone execution, a project is created for you
Project execution. You can't chose the source name and it is generated for you and cleaned up once completed.
I don't currently have any use for any other interfaces, so I thought I might as well drop them for now.


My first Node-RED node

Add a wiki page to describe how to create a new Node-RED node inspired by QRCode or Trello node.

V1 done


Fix 179: allow projection if .embedding_ set to non float32

Fixes #179
This meets the expectation that the embeddings passed to optimize_layout are float32. In the typical case, where they are float32, this should be no cost. When the stored embedding is not float32, this is a pretty cheap operation (13ms for ten million elements). Potentially, the existing embedding could be converted and re-stored as a float32 array (saving future conversions), but this could increase memory usage and break expectations about references.
As proof this works, the added test fails on c61cdfe124ff (current master) but passes here.
Edit: So travis failed, but it looks like condas servers are down at the moment. I'd appreciate a rebuild once they're back up!

Hello @ivirshup! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file umap/tests/test_umap.py:

Line 765:80: E501 line too long (84 > 79 characters)

Thanks -- and I greatly appreciate the added test as well.


don't force adding scheme prefix to string

users should be expected to write reasonable code, this also matches the rest of the clients, i.e. see https://github.com/nats-io/java-nats/blob/master/src/main/java/io/nats/client/Options.java line 584

for our use case we need to use the tcp scheme, but this forces us to use nats, our other languages don't have this problem

@harrisa1 Actually all other supported clients are moving to support simplified URLs. That is, if no scheme/port is specified, the default scheme and port are added.

that sounds great, but this client didn't do that, it forced the string to start with "nats://" even if you might have already supplied a scheme, and didn't touch the port at all, i can update this request with the changes to support defaults, but that is not what this code was doing, if like us you specified a scheme that wasn't "nats://" this prefixed a second scheme which obviously wouldn't work

@harrisa1 Understood that current code is wrong to prepend nats:// without checking if a scheme is already set. I think that it would be addressed with bringing the changes done for other clients, but I am sure that @ColinSullivan1 would not mind a PR ;-)

no problem, updated with support for simplified urls and defaults


Tabs messed up iOS 8.4 simulator

Chrome & Android device work fine，But iOS  device browser access（Have been deployed to http://ming.1g9f.com） and app run.
The screenshot：Before the three is wrong；After the three is right.

CengcengtekiMacBook-Air:Home cengceng$ ionic info
Your system information:
Cordova CLI: 5.4.1
Gulp version:  CLI version 3.9.0
Gulp local:
Ionic Version: 2.0.0-alpha.57
Ionic CLI Version: 2.0.0-beta.17
Ionic App Lib Version: 2.0.0-beta.8
ios-deploy version: 1.8.3
ios-sim version: 4.1.1
OS: Mac OS X El Capitan
Node Version: v0.12.7
Xcode version: Xcode 6.4 Build version 6E35b

I'm sorry but I don't understand the problem. Could you explain when the tabs don't look correct?

I'm sorry, I mistakenly said.Should be run to the apple machine, black screen and numerous according to the problems.There is no problem, you visit http://ming.1g9f.com?

I'm going to close this because it doesn't look like it's an issue with the conference app. Can you please create a separate issue on the ionic repo if you're still experiencing this problem? https://github.com/driftyco/ionic/issues
Thanks!


Add Arpit Gupta to Contributors List

Thanks for the tutorial

@Codarp, I'm quite elated about your pull request. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a pull request and learned that I merged it?
We’ve recently added Facebook share and tweet feature to our web app. Could you please go to https://roshanjossey.github.io/first-contributions/#social-share and share your first contribution to open source?
Also, check out projects with easy issues while you’re there.


Fixes small issues in CI

Runs with ppxlib no longer building with trunk revealed two issues that were unnoticed in the Windows workflows

CI is green modulo a Windows Out_channel test that failed to trigger under Windows trunk.
I'm eager to merge to try out a rebased #319...  :smiley:


huffyuv-beta: Add version 2.2

Relates to https://github.com/ScoopInstaller/Extras/issues/10726

[x] I have read the Contributing Guide.

/verify


Integration Test - Release - 2020/09/25 15:45:20

Auto create for integration test. IssueCommandFlow

@lamebrain : Thanks for your contribution! The author(s) have been notified to review your proposed change.


feat: Extract pytket parameters to input wires

Depends on #660
Closes #628

There PARAMETERIZED case in the pytket tests was already testing the roundtrip pytket -> tk2 -> pytket
https://github.com/CQCL/tket2/blob/af2c458d4620aa7a204e24fe9e21f8a0512f6bcc/tket2/src/serialize/pytket/tests.rs#L59-L73
I added another roundtrip test from tk2 to tk2 with some named and some unnamed params.


Fix parser test running 1-4 times

The dumpAst helper was imported from parser.test.ts in 3 other files which caused vitest to randomly run the parser.test.ts tests(371 tests) extra times. It wasn't consitent which is how I caught it the number of test passed by vitest changed significantly from one run to the other

:x: There is undocummented changes. Run chronus add to add a changeset or click here.
The following packages have changes but are not documented.

:x:@typespec/compiler

You can try these changes at https://cadlplayground.z22.web.core.windows.net/prs/3168/
Check the website changes at https://tspwebsitepr.z22.web.core.windows.net/prs/3168/


Could you please also provide the csv file of the data ?

Hi, Nick, I'm new to Spark, and I have trouble with reading parquet data file on my own notebook.
Could you please also provide the csv file of the data ?

Hi there. CSV may be more tricky as the raw text would often contain the separator character etc. Parquet is much cleaner. Could you share the exact issue or error message you are getting?


[cloud_functions] Migrage functions to new core

Description

Determine Functions dependency based on core plugin version.

@googlebot I consent.


[FEATURE REQ] Ensure Azure ServiceBusReceiver RenewMessageLockAsync returns DateTime of new lock

Library name
Azure.Messaging.ServiceBus 7.5.1
Please describe the feature.
While migrating from Microsoft.Azure.ServiceBus to Azure.Messaging.ServiceBus ran into this hiccup.
Previously SubscriptionClient supported this:
https://docs.microsoft.com/en-us/dotnet/api/microsoft.servicebus.messaging.subscriptionclient.renewmessagelockasync?view=azure-dotnet
And it would seem that ServiceBusReceiver implemented one of the two methods here:
https://docs.microsoft.com/en-us/dotnet/api/microsoft.azure.servicebus.core.messagereceiver.renewlockasync?view=azure-dotnet#microsoft-azure-servicebus-core-messagereceiver-renewlockasync(system-string)
Which I feel like should have been an issue, I cannot think of a reason why you'd want a return in one case and not the other, but I'm assuming it's carrying that forward, and returning nothing is ostensibly a loss of functionality.

Actually, did some sanity clawing at possible reasons, and the message has a LockedUntil property, so I guess that's why.


Please move Samsung v0.7 submission models to mlcommons/mobile_models

Please move Samsung v0.7 submission models to mlcommons/mobile_models
https://github.com/mlcommons/mobile_models/tree/main/v0_7/Samsung
so that the unified app can access the model from URL

The models are submitted


Rename everything from "mutang" to "mull"

While doing so we should also make the current Mutang namespace start with a lowercase letter.

Why lowercase?

For consistency. It's common practice to use lowercase with underscores for namespaces. The standard does this and so do almost all C++ projects, including LLVM for example.


feat(duplicate_element): duplicate element by alt dragging

Issue #165
By pressing the alt key while clicking an element it duplicates it.
Here is how it behaves:

Cool :+1:. One small thing --- I think it should work even if element is already selected.

I'll take a look on that :)

@dwelle It should be good :)

LGTM. Thanks! :+1:

@jeremyscatigna when I alt-drag with multiple elements, it just duplicates one single element

@Fausto95 true, didn't catch that. Will make a PR fixing this.


wechat.rb can only be launched once

well,  after brew cask install wechat ,  the app can be launched only at the first time.
after quit ( both from menu 'quit wechat'  or use command-Q ) the app, it can never be launched again.
after do
brew cask uninstall wechat
brew cask zap wechat
brew cask install wechat

I can launch the app, but still, only once
However, everything go well if i install from app store.  In fact, the two methods get different version of wechat
env:
MacOSX: 10.10.3
 brew: 0.9.5
cask :0.53.3

Can you execute /path/to/WeChat.app/Contents/MacOS/WeChat in Terminal and paste the output here?

install from app store:  wechat version: <IP_ADDRESS>
install via brew cask :  wechat version:  <IP_ADDRESS>
so I think cask doesn't get the latest version.
to execute /path/to/WeChat.app/Contents/MacOS/WeChat ,  I got nothing after the first launch.
$ pwd
/Users/<user>/Applications/WeChat.app/Contents/MacOS
$ ./WeChat  # the first launch
2015-04-29 11:06:29.211 WeChat[11260:454829] LoadContactsWithType, type=13, loadedCount=0
2015-04-29 11:06:29.211 WeChat[11260:454829] New LoadContactsWithType, type=13, loadedCount=0
2015-04-29 11:06:29.236 WeChat[11260:454829] windowDidBecomeKey NSConcreteNotification 0x7fc72d1d9c50 {name = NSWindowDidBecomeKeyNotification; object = <MMMainWindow: 0x7fc72ace69e0>}
2015-04-29 11:06:29.236 WeChat[11260:454829] windowDidBecomeKey NSConcreteNotification 0x7fc72d1d9c50 {name = NSWindowDidBecomeKeyNotification; object = <MMMainWindow: 0x7fc72ace69e0>}
2015-04-29 11:06:38.971 WeChat[11260:454829] windowDidResignKey NSConcreteNotification 0x7fc72d357ed0 {name = NSWindowDidResignKeyNotification; object = <MMMainWindow: 0x7fc72ace69e0>}
2015-04-29 11:06:38.971 WeChat[11260:454829] windowDidResignKey NSConcreteNotification 0x7fc72d357ed0 {name = NSWindowDidResignKeyNotification; object = <MMMainWindow: 0x7fc72ace69e0>}
2015-04-29 11:06:41.001 WeChat[11260:454829] windowDidBecomeKey NSConcreteNotification 0x7fc72d1c3760 {name = NSWindowDidBecomeKeyNotification; object = <MMMainWindow: 0x7fc72ace69e0>}
2015-04-29 11:06:41.002 WeChat[11260:454829] windowDidBecomeKey NSConcreteNotification 0x7fc72d1c3760 {name = NSWindowDidBecomeKeyNotification; object = <MMMainWindow: 0x7fc72ace69e0>}
2015-04-29 11:06:44.333 WeChat[11260:454829] sender: <MMApplication: 0x7fc72ad36b50>
2015-04-29 11:06:44.367 WeChat[11260:454829] windowDidResignKey NSConcreteNotification 0x7fc72d300d60 {name = NSWindowDidResignKeyNotification; object = <MMMainWindow: 0x7fc72ace69e0>}
2015-04-29 11:06:44.368 WeChat[11260:454829] windowDidResignKey NSConcreteNotification 0x7fc72d300d60 {name = NSWindowDidResignKeyNotification; object = <MMMainWindow: 0x7fc72ace69e0>}
$
$ ./WeChat  # the second
$
$ ./WeChat
$

so I think cask doesn't get the latest version.

Not only that, they seem to be actively pushing the Mac App Store version. Let’s remove this from homebrew-cask, then.


kgx: init at unstable-2021-03-12

Motivation for this change
King's Cross is a simple terminal emulator for GNOME. It's a popular terminal for use on mobile devices with Phosh.
The last release (0.2.1) was tagged a year ago and a lot of changes have taken place since then, thus the unstable version is being submitted here.
Upstream report for the missing gio-unix-2.0 dependency (#36468): https://gitlab.gnome.org/ZanderBrown/kgx/-/issues/53
Things done

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

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

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

Independent but related: #88767

Result of nixpkgs-review pr 116244 at f51ab134 run on aarch64-linux 1

1 package built successfully:

kgx

1 suggestion:

warning: unclear-gpl
gpl3 is a deprecated license, check if project uses gpl3Plus or gpl3Only and change meta.license accordingly.
Near pkgs/applications/terminal-emulators/kgx/default.nix:70:5:
   |
70 |     license = licenses.gpl3;
   |     ^

nice! i was just trying to get this packaged but it always said no kgx gschema found.
i'll try this later.


Adding Modular Exponentiation to Number Theory

Adding Modular Exponentiation to Number Theory

Hi! Thanks for PR.
This code doesn't really get along with the rest of the repository, and you didn't add header comments, test problem, README stuff, and so on. Closing.


Use ffmpeg-the-third crate

ffmpeg-the-third is a fork of the abandoned ffmpeg-next crate, which is a fork of the abandoned ffmpeg crate, adding features like FFmpeg 5.x and FFmpeg 6.x support.

Is there any comment from the original maintainer of ffmpeg-next? I can't seem to find a confirmation that it has gone unmaintained.

Is there any comment from the original maintainer of ffmpeg-next? I can't seem to find a confirmation that it has gone unmaintained.

not sure really, but sadly it doesn't seem like ffmpeg-next being maintained (last release 8 months ago), PRs are not being merged too.

Okay, I want to wait this out for a bit since the ffmpeg-the-third crate does not seem to be the de facto replacement just yet. Do agree though that ffmpeg-next seems unmaintained. Will get back to this in a couple months to see if things have changed!

seems ffmpeg-next is back in business :partying_face:


How to synthesize the comprehensive video description in InternVid?

The paper says "These individual image captions are then synthesized into a comprehensive video description using a pretrained language model."
In the paper, it is unclear how the individual image captions are synthesized into a comprehensive description. Could you please share more details?

As shown in Figure2 of the paper, we use Image Caption Models to analyze video frames and generate image descriptions. These descriptions are subsequently fed into T5, which has been pre-trained on summarization tasks, for further summarization.


HEIF/AVIF file signature (magic number)

Hi!
Want to share with you my recent issue, that I created for another pyheif library, just don't want to copy it, but want to inform you about bug that I found. I believe that one day you will fix it.
https://github.com/bigcat88/pillow_heif/issues/8
Thank you for your attention!

Hi,
thanks to notify me!
Probably this issue should by fixed on heif-image-plugin.


Fix typo in AKS getting started guide

This PR fixes a small typo in AKS docs.

Coverage decreased (-0.03%) to 36.954% when pulling 087d2d9b283d3d5065f830b23d284035d4061910 on ap4y:patch-1 into c4b2c1e6c7245bdaf2bb4a904ed80a4087623f4d on cilium:master.


i18n(ja): fix guides/images.mdx, output result was wrong

Description (required)

Fix guides/images.mdx.
Fixed an incorrect example in the output section of the Astro Image Guide. The example was incorrectly showing an html element, but it should output an img element instead, so I have replaced it accordingly.

Hello! Thank you for opening your first PR to Astro’s Docs! 🎉
Here’s what will happen next:

Our GitHub bots will run to check your changes.
If they spot any broken links you will see some error messages on this PR.
Don’t hesitate to ask any questions if you’re not sure what these mean!

In a few minutes, you’ll be able to see a preview of your changes on Netlify 🥳.

One or more of our maintainers will take a look and may ask you to make changes.
We try to be responsive, but don’t worry if this takes a few days.

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

ja
guides/images.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


[improve][broker] Added RoundRobinBrokerSelector

Motivation
Added a simple RoundRobin logic to select brokers for bundle ownership. This simple selector can help distribute the bundle load when there are many concurrent lookup requests(especially useful for performance tests where most lookups happen at the beginning of tests).
Modifications
Added RoundRobinBrokerSelector and its unit test.
Verifying this change

[x] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change added tests and can be verified as follows:

Added an unit test to cover RoundRobinBrokerSelector

Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

[ ] Dependencies (add or upgrade a dependency)
[ ] The public API
[ ] The schema
[ ] The default values of configurations
[ ] The threading model
[ ] The binary protocol
[ ] The REST endpoints
[ ] The admin CLI options
[ ] The metrics
[ ] Anything that affects deployment

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository: https://github.com/heesung-sn/pulsar/pull/22

Codecov Report

Merging #19321 (c18170f) into master (8d81392) will increase coverage by 0.31%.
The diff coverage is 54.34%.

@@             Coverage Diff              @@
##             master   #19321      +/-   ##
============================================
+ Coverage     63.93%   64.24%   +0.31%
- Complexity     3470    26018   +22548
============================================
  Files          1818     1819       +1
  Lines        133061   133132      +71
  Branches      14637    14649      +12
============================================
+ Hits          85067    85529     +462
+ Misses        40270    39797     -473
- Partials       7724     7806      +82

Flag
Coverage Δ

inttests
24.92% <0.00%> (-0.03%)
:arrow_down:

systests
25.56% <0.00%> (-0.15%)
:arrow_down:

unittests
61.39% <54.34%> (+0.07%)
:arrow_up:

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


Email attachment

File attachment in the email library seems to return a corrupt file when the mail is sent.

No further details provided. Can't reproduce?


Beta came out nothing seems to work accept /say hello          alpha 3

-test8hello,1,20707,Able To Cast,Resources Available,GCD Free,Not actively casting anything else.
used castable spell Soulstone(tried several different spells)       will not work in body or anywhere else
Message types work.

I would check what you have in KeyPress
If there is a GCD spell there and note that COOLDOWNS ARE NOT ON THE GCD then nothing in the macro will work.

I tried the macro that worked yesterday in alpha 2 in the alpha . As soon as it turned to the beta using alpha 3 noting worked I took everything out only one spell that was not on cool down I waited and tried to cast it. Did not work
If I substituted /say hello   it worked. I tried several spells in main body and nothing worked. I did not put them in key press or pre or post.
I remove completely the alpha 3 and installed fresh still not working I have not gotten rid of the variables
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10

From: Timothy Minahan<EMAIL_ADDRESS>Sent: Tuesday, April 24, 2018 11:03:52 PM
To: TimothyLuke/GnomeSequencer-Enhanced
Cc: woodgray; Author
Subject: Re: [TimothyLuke/GnomeSequencer-Enhanced] Beta came out nothing seems to work accept /say hello alpha 3 (#389)
I would check what you have in KeyPress
If there is a GCD spell there and note that COOLDOWNS ARE NOT ON THE GCD then nothing in the macro will work.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/TimothyLuke/GnomeSequencer-Enhanced/issues/389#issuecomment-384146607, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AXVP_PJtmhV5ZYjg7qY_-Yp906zoAE4Bks5tr-eYgaJpZM4TissW.

Fixes for Beta:
https://github.com/TimothyLuke/GnomeSequencer-Enhanced/releases/download/GSE_2.3.00-BfA-beta1/GSE.2.3.00.-.BfA.Beta.1.zip


Add Bitbucket support

Hello,
Since my company is actually working with Bitbucket, I would be glad to work in order to add the same features for Github to Bitbucket. For example listing Bitbucket project repositories, listing user repositories, and ensure that each feature of noseyparker is working on it.
I could take inspiration from the existing code for Github and make a Bitbucket version.
Why not GitLab too in the future?
:smile:

Hello again @Coruscant11 :)
Yes, having support for enumerating repositories from Bitbucket would be welcome.
I'm not all that familiar with Bitbucket. It looks like they have a cloud-hosted offering as well as self-hosted options. Which type of deployment is most relevant to you? I would start with getting that working.
I took a quick look at things, and have some high-level notes and pointers for you:

In Nosey Parker, the GitHub repository enumeration code is used in just two places currently: in the scan command, and in the github command. In both cases, the high-level wrapper function is noseyparker::github::enumerate_repo_urls, which is a synchronous API that takes the set of requested users/orgs as input and simply returns a vector of repo URLs. Internally, it uses async functions, but that's all hidden for now.

I didn't see an existing Bitbucket API library on crates.io, so building something simple for the Bitbucket APIs you need using the reqwest libraryis a reasonable path. (That's what I ended up doing for GitHub support, because the existing octocrab library was pretty big and didn't actually support the APIs I needed.)

Nosey Parker's GitHub support has a ClientBuilder, used to specify how to initialize an actual Client object that makes the API requests. You could introduce similar types in a new bitbucket module.

Nosey Parker's GitHub code only currently supports bearer token-based authentication. In Bitbucket, it looks like many different authentication schemes are supported, though it looks like in the examples that simpler bearer token-based authentication should work. I'd start with that, unless your use case needs something different.

In Nosey Parker, if someone wants to use a GitHub token, they must specify it via the NP_GITHUB_TOKEN environment variable. You could introduce a similar NP_BITBUCKET_TOKEN variable.

It looks like this Bitbucket API endpoint may be the most relevant. It can take filter parameters to restrict the scope to particular users or workspaces. (A "workspace" is a Bitbucket version of a GitHub organization?)

The pagination in Bitbucket responses seems to use the same kind of mechanism as GitHub's APIs, so you could probably adapt Nosey Parker's GitHub code for that. Specifically, this and this.

Nosey Parker's GitHub support detects and reports rate-limiting failures, but currently makes no attempt to retry failing requests.

Again, I'm happy to answer any questions you may have about this!

Great answer as always!
Yes, the main goal would be to allow people to also scan their private Bitbucket server. Like in my case for my company. Normally, the cloud and the self-hosted  has the same API scheme, so just giving the Bitbucket URL to Nosey Parker should be enough.
As you said, we only need to call the API in order to find the git url,  and reqwest should do the work perfectly.
We can generate authentication token with Bitbucket, and I think that it should be the way to use authentication on tools like scanners. Users should use read-only tokens for this kind of stuff and that is why I think that only supporting bearer token-based authentication is a good thing.
I will try to work on it and keep you updated if I need something!
Thank you so much :smiley:

Hello, sorry I was quite busy the past months. I will work on this issue very soon :smile:

No problem at all :)

I am also curious about the progress of that feature. Is there any news?

Hello @amuessig, I didn't work on it unfortunately.. I could in fact maybe try to look at it again, just that finally, I didn't need it anymore at my company. If someone want to take this issue, feel free to!
Otherwise, I will give an update this week end. In fact I want to do some Rust these days so this would be a good opportunity :yum:


{DJAN-4} Setup Github

Instructions on how to set up Github.

Having the direct links to Github pages for creating repositories and managing the team seems like it'll be very convenient.

Good points, thanks. This is updated.

:sunglasses:


Allow using objects as key (in Maps, Sets, ...)

Currently, keys are always used as string, since they are stored as property name of an object. This PR allows using objects as key by using a Map instead of an object for the storage.
I hope it makes sense. Great project!

Awesome! 🙂 That's a pretty nasty bug, thanks for catching and fixing it. I am checking the performance impact and merging this soon. (I don't think this will slow things down.)

I updated the PR!

Perfect, thanks! The next patch release is rolling out later today with your fix 🙂

Published it as v4.1.3, the change log is here: https://github.com/nx-js/observer-util/releases/tag/v4.1.3. Thanks again 👍


Can't convert prediction results to supervision format

Describe the bug
The Visualisation page is out of date, I believe: convert_to_sv_format no longer exists in the deepforest.visualize module.
To Reproduce
from deepforest.visualize import convert_to_sv_format
import deepforest.visualize as visualize
print(dir(visualize))

Returns
['Image', '__builtins__', '__cached__', '__doc__', '__file__', '__loader__', '__name__', '__package__', '__spec__', 'cv2', 'format_boxes', 'label_to_color', 'matplotlib', 'np', 'os', 'pd', 'plot_points', 'plot_prediction_and_targets', 'plot_prediction_dataframe', 'plot_predictions', 'ptypes', 'random', 'view_dataset', 'warnings']
---------------------------------------------------------------------------
ImportError

Environment (please complete the following information):

OS: macOS Ventura
Python version and environment : 3.10.12

I see this function is mentioned in #697 and is in https://github.com/weecology/DeepForest/blob/main/deepforest/visualize.py so I'm not sure why the function can't be found

Sorry for the confusion @camappel - that function is new since the last release and the docs are currently being built for the main branch.
Assuming you have git installed you can install the current development version with that function using:
pip install git+https://github.com/weecology/deepforest

I'll open an issue to do some work on the docs to display the current release by default.
If this clears things up feel free to close.

Thanks for the response! The following code worked
pip uninstall deepforest -y
pip install git+https://github.com/weecology/deepforest

import deepforest.visualize as dv
dv

Returned
<module 'deepforest.visualize' from<EMAIL_ADDRESS>Drive/colab/DeepForest_Project/deepforest-env/lib/python3.9/site-packages/deepforest/visualize.py'>

Glad that took care of it and thanks for reporting! We'll get that aspect of the docs cleaned up to avoid this very confusing kind of experience in the future.


[install] gem installation fails due to access right issues on rubygems.org

I already reported this at rubygems.org and got this answer:

This support email is for issues related to RubyGems.org the website and RubyGems the client. Please use Stackoverflow or Github issues for help with specific gems.

Thanks!

Have you read and followed the installation tutorial at http://www.nokogiri.org/tutorials/installing_nokogiri.html?

[x] Yes!

What is the complete output of gem install or bundle install?

sudo gem install nokogiri
ERROR:  While executing gem ... (Gem::RemoteFetcher::FetchError)
    bad response Forbidden 403 (https://api.rubygems.org/quick/Marshal.4.8/nokogiri-1.13.6-x64-unknown.gemspec.rz)

Tell us about your system!
What is the output from ruby -v?
ruby 2.5.1p57 (2018-03-29 revision 63029) [x86_64-linux-gnu]

What is the output from gem -v?
2.7.6

What is the output from gem env?
RubyGems Environment:
  - RUBYGEMS VERSION: 2.7.6
  - RUBY VERSION: 2.5.1 (2018-03-29 patchlevel 57) [x86_64-linux-gnu]
  - INSTALLATION DIRECTORY: /var/lib/gems/2.5.0
  - USER INSTALLATION DIRECTORY: /home/wolfgang/.gem/ruby/2.5.0
  - RUBY EXECUTABLE: /usr/bin/ruby2.5
  - EXECUTABLE DIRECTORY: /usr/local/bin
  - SPEC CACHE DIRECTORY: /home/wolfgang/.gem/specs
  - SYSTEM CONFIGURATION DIRECTORY: /etc
  - RUBYGEMS PLATFORMS:
    - ruby
    - x86_64-linux
  - GEM PATHS:
     - /var/lib/gems/2.5.0
     - /home/wolfgang/.gem/ruby/2.5.0
     - /usr/lib/x86_64-linux-gnu/rubygems-integration/2.5.0
     - /usr/share/rubygems-integration/2.5.0
     - /usr/share/rubygems-integration/all
  - GEM CONFIGURATION:
     - :update_sources => true
     - :verbose => true
     - :backtrace => false
     - :bulk_threshold => 1000
  - REMOTE SOURCES:
     - https://rubygems.org/
  - SHELL PATH:
     - /home/wolfgang/bin
     - /home/wolfgang/.local/bin
     - /home/wolfgang/bin
     - /home/wolfgang/bin
     - /usr/local/sbin
     - /usr/local/bin
     - /usr/sbin
     - /usr/bin
     - /sbin
     - /bin
     - /usr/games
     - /usr/local/games
     - /snap/bin

If Linux or a BSD variant, please note:

the distro you're using: Ubuntu 18.04
the output of uname -a Linux secret 5.4.0-107-generic #121~18.04.1-Ubuntu SMP Thu Mar 24 17:21:33 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
the contents of /etc/lsb-release.

DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=18.04
DISTRIB_CODENAME=bionic
DISTRIB_DESCRIPTION="Ubuntu 18.04.6 LTS"

Thanks for opening this issue! I'm sorry you're getting passed around, but I can help.
The issue you're seeing is actually due to the behavior of older versions of gem which aren't smart enough to look at the required_ruby_version value in each gem version's gemspec.
The latest Nokogiri is v1.13.6 which doesn't support Ruby 2.5 (you can see this in the nokogiri.gemspec). However, the version of gem that you're running, v2.7.6, tries to grab the latest version without first checking that it is compatible with the version of Ruby you're running.
You can get around this in two ways:

manually pick a version of Nokogiri that supports your version of Ruby (the v1.12.x series is the last branch that supported Ruby 2.5, so you can try gem install nokogiri -v "~> v1.12.0")
upgrade to a newer version of rubygems by running gem update --system

I recommend the second option, upgrade rubygems. Here's what it looked like for me:
$ ruby -v
ruby 2.5.9p229 (2021-04-05 revision 67939) [x86_64-linux]

$ gem -v
<IP_ADDRESS>

$ gem install nokogiri
ERROR:  While executing gem ... (Gem::RemoteFetcher::FetchError)
    bad response Forbidden 403 (https://api.rubygems.org/quick/Marshal.4.8/nokogiri-1.13.6-x64-unknown.gemspec.rz)

$ gem update --system
Updating rubygems-update
rbenv: cannot rehash: /home/flavorjones/.rbenv/shims/.rbenv-shim exists
Successfully installed rubygems-update-3.3.16
... snip ...
RubyGems installed the following executables:
	/home/flavorjones/.rbenv/versions/2.5.9/bin/gem
	/home/flavorjones/.rbenv/versions/2.5.9/bin/bundle
	/home/flavorjones/.rbenv/versions/2.5.9/bin/bundler

RubyGems system software updated

$ gem -v
3.3.16

$ gem install nokogiri
Successfully installed nokogiri-1.12.5-x86_64-linux
Parsing documentation for nokogiri-1.12.5-x86_64-linux
Done installing documentation for nokogiri after 1 seconds
1 gem installed

I hope this helps!


Updated installation instructions for on-device training package

To reflect the updated installation instructions for installing onnxruntime for training.
Addresses ONNXRuntime issue #21149

@carzh
Please update this note book to the latest 1.18.1 version.
https://github.com/microsoft/onnxruntime-training-examples/blob/master/on_device_training/desktop/csharp/masked_language_modeling/mobilebert_offline.ipynb

What do you mean update the notebook to the latest 1.18.1 version? Are you running into issues with running the notebook with 1.18.1?
The README for the masked language modeling example already includes the updated installation instructions for ONNXRuntime, and this example was written for CPU EP only.

Try this on the latest 1.18.* onnxruntime-training
import onnxruntime.training.onnxblock as onnxblock

The whole problem with onnxruntime-training is lack of specific information on requirements for CUDA 12.* and lack of testing that when using c#

I'll update the notebook & add a requirements.txt file for the on-device training example.
We can add adding a CUDA C# example to the backlog. I understand your frustration with the lack of documentation and we do need to improve ONNXRuntime documentation, but creating good documentation and examples also takes time.

@carzh
Thx for updating the documentation and training codes. Onnxrunetime training will become increasingly important, due to possibilities to combine with generative AI e.g. Phi3

Try this on the latest 1.18.* onnxruntime-training import onnxruntime.training.onnxblock as onnxblock

Ah, I gave a try, and did not run into any issues with that import line. What error do you run into?

@carzh
I can not use python to upgrade onnxruntime-training from 1.15.1 to 1.18.1
No windows version is available
The link to download the windows version is hard to find

@GeorgeS2019 The link to download the windows version is available at onnxruntime.ai. All the documentation should point to that installation table.
Try the following:
pip uninstall onnxruntime-training -y
python -m pip install cerberus flatbuffers h5py numpy>=1.16.6 onnx packaging protobuf sympy setuptools>=41.4.0
pip install -i https://aiinfra.pkgs.visualstudio.com/PublicPackages/_packaging/ORT/pypi/simple/ onnxruntime-training-cpu --no-cache-dir

I appended the no-cache-dir flag to ensure that pip doesn't pick up on any locally cached onnxruntime-training python packages.

@carzh
I managed to download onnxruntime-training python package for windows: 1.18.0
I could run the notebook, but the generated checkpoint file is only called => checkpoint
mobilebert-uncased.ckpt is not created. Only checkpoint file is written.
https://github.com/carzh/onnxruntime-training-examples/blob/cebaf5cb5077007163d16720f139c5c847df66e6/on_device_training/desktop/csharp/masked_language_modeling/csharp_console_app/Program.cs#L36C84-L36C96

There are confusion with Cuda 12 support.
Which version of onnxruntime and onnxruntime-training I could use for Cuda 12?
When running the notebook python, which version of python package I need for cuda 12?  1.18.1?

For Cuda 12, Onnruntime-training in windows for both python and windows is not supported it seems

There are confusion with Cuda 12 support. Which version of onnxruntime and onnxruntime-training I could use for Cuda 12?

Use the most up-to-date package unless the example specifies otherwise. The latest release includes CUDA 12 support, but CUDA 12 is not supported for all configurations of ORT, as it looks like you've discovered.
For example, on-device training with Python for Linux supports CUDA 12, but on-device training with Python for Windows does not. The easiest way to check if a configuration supports CUDA 12 is with the installation table on onnxruntime.ai.

When running the notebook python, which version of python package I need for cuda 12? 1.18.1?

The README specifies to use the python package for CPU. The C# example was written for CPU EP, and not CUDA.
I'll try reproducing the issues you are running into later today and I'll push some updates to make it clearer that the masked language modeling example is for CPU.

I'll try reproducing the issues you are running into later today

Any update?


How to move licence to another device

Hello!
In a dev test, I'm unfornately trashing devices so I run out of the 3 free devices.
How to move the licence to new devices? I don't need more licences, just to use 3 at the same time.
Nicolas.

Unfortunately there is no mechanism to move activations from one device to another. You will have to wait for those activations to expire (30 days) and then you can activate again on the new devices.


Fallback behavior for measure units

What should happen if you call the API with a unit identifier that the implementation doesn't know about?
A few options, each with pros and cons:

Put the list of supported unit identifiers directly into the spec

Pro: No ambiguity as to which units are available
Con: Difficult to update; can't just push new units as soon as they become available in CLDR/ICU

Throw an exception if the implementation does not know about the unit

Pro: The user gets a signal and can deal with that case on their own
Con: Some browsers could throw exceptions and others not throw exceptions with the same code

Use the unit identifier itself as the fallback: "3 length-meter"

Pro: Well-defined, exceptionless fallback behavior
Con: The string output is not localized and not particularly clear

Thoughts?

The Ecma 402 committee wants to have a list in the spec of a minimum set of units that an implementation must support.
This still leaves open the question though of what behavior to adopt when using a unit identifier outside of the minimum required set.

@srl295 suggested doing our best to localize the unit identifier; for example, CLDR should have translations for the "length" part of the string.
Another option could be to strip off the type part and display just the unit part ("meter" instead of "length-meter").
Throw an exception if you don't support the unit: some implementations might throw where others don't.
Throw an exception for any unit not in the spec set, even if the implementation supports it: slower updates, and there could still be cases where some browsers throw where others don't, because some browsers may pull in spec updates sooner than others.

I'd like to put in a vote for option 4. It seems the most reliable to me. When more units are added, it would be an explicit and shared action by the group, rather than an implementation-specific thing. Since we have a standing group, it should not be hard to add units.

Making that explicit change whenever there are more units to support sounds good to me. We settled on this policy for Unicode properties in RegExps, cc @mathiasbynens.

Following the precedent set by Unicode properties SGTM.  Thanks for finding that.
I'll update the proposal spec text soon.

+1 — explicit > implicit.

+1

I am re-opening this issue just briefly.
With the new "core unit identifier" syntax, users are now passing strings like "meter" instead of "length-meter".  The new core unit identifiers are a little bit more user-friendly.
@rxaviers pointed out previously that throwing an exception for an unknown unit is inconsistent with what we do with unknown currency codes, which is to echo out the currency code itself.  The argument for the difference was that there was not a reasonable way to display the unit identifiers.  However, the simplified core unit identifiers may be "good enough" that we can use those as the fallback symbols.
Thoughts?

Re-affirmed the previous consensus at today's Ecma 402 meeting.


No response in sendAudio

Required Information

Operating system:Linux
PHP version:5.6.30
PHP Telegram Bot version:0.54.0
Using MySQL database: yes
MySQL version:5.5.60-cll
Update Method: Webhook
Self-signed certificate: no
RAW update (if available):

Expected behaviour

There should be mp3 file with play button for play it.
Actual behaviour

but not getting any response
Steps to reproduce

bellow is code for sending it:
$url = str_replace(" ", "%20", $audio['file']);
$data['caption'] = $audio['title'];
$data['title'] = $audio['artist'];
$data['audio'] = Request::encodeFile($url);

Request::sendAudio($data);

Extra details

bellow is log when i am sending audio file:
[2018-09-06 12:17:39] bot_log.ERROR: exception 'Longman\TelegramBot\Exception\TelegramException' with message 'Telegram returned an invalid response!' in /vendor/longman/telegram-bot/src/Request.php:472
Stack trace:
#0 [internal function]: Longman\TelegramBot\Request::send('sendAudio', Array)
#1 /vendor/longman/telegram-bot/src/Request.php(574): call_user_func_array('static::send', Array)
#2 /Commands/CallbackqueryCommand.php(102): Longman\TelegramBot\Request::__callStatic('sendAudio', Array)
#3 /Commands/CallbackqueryCommand.php(102): Longman\TelegramBot\Request::sendAudio(Array)
#4 /vendor/longman/telegram-bot/src/Commands/Command.php(173): Longman\TelegramBot\Commands\SystemCommands\CallbackqueryCommand->execute()
#5 /vendor/longman/telegram-bot/src/Telegram.php(537): Longman\TelegramBot\Commands\Command->preExecute()
#6 /vendor/longman/telegram-bot/src/Telegram.php(505): Longman\TelegramBot\Telegram->executeCommand('Callbackquery')
#7 /vendor/longman/telegram-bot/src/Telegram.php(423): Longman\TelegramBot\Telegram->processUpdate(Object(Longman\TelegramBot\Entities\Update))
#8 /hook.php(52): Longman\TelegramBot\Telegram->handle()
#9 {main} [] []

i tried but still no action, i attached log files check it
SHATest_Bot_update.log
SHATest_Bot_debug.log

Hi @sunildabhi
I have removed your log files, as they contained your bot API token.
:exclamation: Please reset them immediately via @BotFather :exclamation:
Regarding your issue, your debug log showed a 400 Bad Request error, which means your request has invalid data in it.
Have another look at the official sendAudio documentation to make sure you have filled all necessary fields and that they are valid.

i have audio file with size of 65mb can you tell me how to send it? because using url there is limit max size 20mb.

Sent it to your bot via  your messenger and get file_id of the file. Send file by it's file_id.

@sunildabhi Can we close off here?

yes it want to working because file size was about to 65mb, thanks for help.


updates for Xcode 9 beta 3

adds recommended project settings

Codecov Report

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

@@           Coverage Diff           @@
##           master     #144   +/-   ##
=======================================
  Coverage   26.63%   26.63%
=======================================
  Files          13       13
  Lines        1952     1952
=======================================
  Hits          520      520
  Misses       1432     1432

Continue to review full report at Codecov.

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

FYI - just pushed 4.1.1 for this change. Thanks!


Created custom_utils module to replace min/max functions from std library #90

Partially solves #90
Can also be used in a future for helper modules.

NOTE: Putting blocked tag on this until the upgrade remediation's are finalized and we're past the upgrade process.


Add ja.yaml for Japanese

I translated the page for Japanese.
Thank you.

@usergreen Thanks, I've added some missing translations. Can you verify them?

xoxys, thank for your fixes.
I think the followings might be more appropriate:
Line 47: propertylist_required: 必須
Line 49: propertylist_default: 既定値

Thanks!

It seems appears that I made a mistake when I tried to synchronize my repository with the main.

I've created a new one.

Thank you.


Ensure that 'Threat Detection types' is set to 'All'

Possible check to target the following resource

[ ] azurerm_mssql_server_security_alert_policy

closing, tracking in spreadsheet

closing, tracking in spreadsheet


Add pretty diff output to error output

It would be nice to have a pretty diff in the terminal when nock doesn't match. Not sure how to 1 to 1  the req and the nock but I lose a lot of tile inspecting the request against expectation.

I thought this may help other:
nock('http://something')
  .put('/foo', (body) => {
    assert.deepEqual(body, {
      id: 'hello',
      data: 'args'
    })

    return true
  })
  .reply(200)

This saved my and my teammates hours.