[go_router] Deep linking

I'm trying to implement deep linking in my test app. I already tried out go_router. So, in documentation there is not much about deep linking. First I created page and added it to router like this:
GoRoute(
 name: 'deepLink',
 path: '/deepLink',
 builder: (context, state) => const DeepLinkPage(),
),

after I added intent filter to my AndroidManifest file as written here:
<!-- Deep linking -->
<meta-data android:name="flutter_deeplinking_enabled" android:value="true" />
<intent-filter android:autoVerify="true">
 <action android:name="android.intent.action.VIEW" />
 <category android:name="android.intent.category.DEFAULT" />
 <category android:name="android.intent.category.BROWSABLE" />
 <data android:scheme="http" android:host="akzdeeplink" />
 <data android:scheme="https" android:host="akzdeeplink" />
</intent-filter>

And what do i do from here? What I want achieve is when user presses URL it opens app at deepLink page

Hi @AkzhanArdaki, Thanks for filing the issue. Please see the next section

Test on Android emulator
Running the specified command from the console should launch the app with the deeplink URL which must be handled within the app, probably in navigatorBuilder. If you need further help with handling deeplinks inside the app I would suggest asking a question on StackOverflow with a proper explanation and a minimal code sample.

Closing as this is neither a bug nor a feature request, In case you disagree feel free to write in the comments and we will reopen it.
Thank you.


Fix nullengine

Refer to https://github.com/BabylonJS/Babylon.js/pull/12693#issuecomment-1462684174

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/13615/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/13615/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/13615/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/13615/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/13615/merge
https://gui.babylonjs.com/?snapshot=refs/pull/13615/merge
https://nme.babylonjs.com/?snapshot=refs/pull/13615/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/13615/merge#BCU1XR#0


 Our listening sockets are v4 only which I was painfully made aware of today when I did a dual stack docker network and vr-xcon got v6 address back when resolving the names and couldn't connect :/
Happy eyeballs would be nice too...

Seems it does ipv4 only bind:
https://git.qemu.org/?p=qemu.git;a=blob;f=net/socket.c;h=90ef3517beb8984d742a4983766b7edd440bd828;hb=HEAD#l504~


Add Operation class to generated umbrella app

include Dry::Monads::Either::Mixin
include Dry::Matcher.for(:call, with: Dry::Matcher::EitherMatcher)

Hmm I'm wondering whether it'd make sense to add Operation class to dry-transcation. Folks could just do class DoSomething < Dry::Transaction::Operation. WDYT?

Sorry to interrupt but what do you mean with Operation are talking about Transaction operations like step, tee, map etc...

 For example

@GustavoCaso we're talking about typical "functional objects" here, the kind you build hundreds of when you're making a large app (hence the idea of having the basic functionality provided by a shared superclass). For example
class CreateComment < Some::Shared:Operation
  def call(attrs)
    # ...
  end
end

@solnic nice idea to put it in dry-transaction. The reason I didn't initially consider providing this via a gem was that the notion of what an "operation" is might differ from app to app, so by dry-web-roda generating one for you, it was more of a hint that you should consider it for your own app.
That said, if all we'd be looking to do initially is include those 2 mixins, then it does make sense that this could be a Dry::Transaction::Operation. And that might make adoption of operations/transactions just that little bit easier for people, too.
OK! So if we did that, I still have one question: even if Dry::Transaction::Operation exists, is there still merit in generating something like this in new dry-web-roda apps?
module MyApp
  class Operation < Dry::Transaction::Operation
    # shared logic goes here
  end
end

I guess if the class if going to be empty, maybe not, but perhaps worth having as a placeholder just in case there's some shared behaviour that an app wants to provide?

Thanks @timriley for explaining 😄
So if understood correctly Dry::Transaction::Operation will be such an interface to MyApp::Operation adding some shared method across all operations?

@GustavoCaso At this point we're basically just talking about this:
module Dry
  module Transaction
    class Operation
      include Dry::Monads::Either::Mixin
      include Dry::Matcher.for(:call, with: Dry::Matcher::EitherMatcher)
    end
  end
end

😄

Nice I got it 👍

OK! So if we did that, I still have one question: even if Dry::Transaction::Operation exists, is there still merit in generating something like this in new dry-web-roda apps?

Yes, I would say it should be a general good practice (just like we do with rom repos), because it gives you a place to inject stuff like loggers, notifiers, integrate with anything that's specific to your app. It also makes it easier to swap implementations for many classes once there's just one abstract class that you own.

Fixed in #45


allow 100% minimum health capacity setting for resident tasks

When push comes to shove, Marathon will allow one instance to be unhealthy vs. having an extra resident task or having a deployment get stuck.
Fixes https://github.com/mesosphere/marathon/issues/3901

Can one of the admins verify this patch?

Can one of the admins verify this patch?

We have this patch deployed and can confirm that allowing the specification of 100% minimum capacity (and 0% over capacity) provides the desired behavior of restarting one task at a time.

Thanks @timcharper, LGTM! Waiting for tests to pass ..


Update the federation admin guide to use the new, simplified deployment mechanism.

cc @kubernetes/sig-cluster-federation

This change is

@madhusudancs deploy/netlify will pass if you rebase and then we will be able to see how this doc looks on the rendered site

Thanks @madhusudancs
Mostly looks good, barring one suggestion.
I will try to find the right way to do that but since I dont know it right now, I dont want to block this PR.
So consider this LGTM from me

deploy/netlify will pass if you rebase and then we will be able to see how this doc looks on the rendered site

Done. I hope this works.

Since @nikhiljindal LGTM'ed in his comment, I am adding the "Tech Review LGTM" label.

Almost there. Just a couple of nits.

Thanks, LGTM. I'll merge into the 1.4 branch.

@devin-donnelly Awesome! Thanks!


python-jiocloud is throwing warnings "InsecurePlatformWarning: A true SSLContext object is not available

I see python-jiocloud is throwing warnings "InsecurePlatformWarning: A true SSLContext object is not available. This prevents urllib3 from configuring SSL appropriately and may cause certain SSL connections to fail."
https://urllib3.readthedocs.org/en/latest/security.html#openssl-pyopenssl have a solution to install pyopenssl ndg-httpsclient pyasn1. Adding this to requirements.txt should fix this warning - I tested it in a separate venv, and worked.

closed as PR merged has resolved this issue.


build: update bazel to 3.6.0

Signed-off-by: Lizan Zhou<EMAIL_ADDRESS>

@htuch no specific reason. that will happen soon.


Support for Visualization

Thanks for your excellent work!
Could LibMTL provide visualization support?  We can easily visualize the training progress of the model by using visualization tools, like tensorboardX, or Visdom.
Thanks

We will add it in the future version. Thank you for your suggestion.

Closed as no further updates.


running npm install fails on installing peer dependencies

To reproduce, start at the beginning of the instructions. On the third step:
$ npm install
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.

@amfarrell: Thank you for pointing this out.
I could reproduce this issue.
Would you like to solve this issue? I'm happy to render any help for the same.

I'm not experienced enough with React Native (I've just starting out) or with Node packaging for it to be something I could efficiently tackle, so no.

No worries dude. I'll be fixing this ASAP.
I'll keep you posted


Version 286

Set version to 286
Changelog:

Refactor multi_buffer
Refactor buffers_adapter
Refactor static_buffer
Refactor flat_buffer
Refactor flat_static_buffer
Fix missing include in sha1.hpp
Fix ostream warning
Field digest is endian-independent
update broken links in README
Fix ostream flush

API Changes:

Nested const and mutable buffer types for all
Beast dynamic buffers are refactored. Affected types:

buffers_adapter
flat_buffer
flat_static_buffer
multi_buffer
static_buffer

Nested mutable_data_type in Beast dynamic buffers is deprecated:

Changes Required:

Use nested mutable_buffers_type instead of mutable_data_type,
or define BOOST_BEAST_ALLOW_DEPRECATED

close #1861
close #1862
close #1864
close #1845
close #1866

I had to edit some commit messages


Kill ffmpeg process?

Question,
I include this library in a small FX application, and I would like to know how to go about killing (SIGINT) the ffmpeg process in the event of the user closing the application.

Additionally, when grabbing a screen:
		final FFmpegBuilder builder = new FFmpegBuilder()
				.setVerbosity(Verbosity.ERROR)
				.setFormat("x11grab") // Format is not always a media format name
				.addInput("+1920,540") // Input is not always a filename
				.overrideOutputFiles(true)
				.addOutput("/tmp/output.mp4")
				.setVideoFrameRate(25, 1)
				.setVideoResolution(640, 480)
				.addExtraArgs("-threads", "0",
						"-hide_banner")
				.done();

ffmpeg never terminates by itself, so it can only be terminated by external means.
It would be a good idea to also add a JVM shutdown hook to terminate any child processes remaining alive, just in case.

The problem can be worked around by using a custom ProcessFunction:
		final FFmpeg ffmpeg = new FFmpeg("/usr/bin/ffmpeg", new RunProcessFunction() {
			@Override
			public Process run(final List<String> args) throws IOException {
				final Process process = super.run(args);

				if (!(args.size() == 2 && args.get(1).equals("-version"))) {
					/*
					 * Remember all child processes except "ffmpeg -version"
					 */
					processes.add(process);
				}

				Runtime.getRuntime().addShutdownHook(new Thread(() -> {
					terminateFfmpeg(process);
				}, "FFmpeg process destroyer"));
				return process;
			}
		});

where terminateFfmpeg(Process) can be implemented like this:
	/**
	 * @return the exit code of the terminated process as an {@code unsigned
	 *         byte} (0..255 range), or -1 if the current thread has been
	 *         interrupted.
	 */
	int terminateFfmpeg(final Process process) {
		if (!process.isAlive()) {
			/*
			 * ffmpeg -version, do nothing
			 */
			return process.exitValue();
		}

		/*
		 * ffmpeg -f x11grab
		 */
		System.out.println("About to destroy the child process...");
		try (final OutputStreamWriter out = new OutputStreamWriter(process.getOutputStream(), UTF_8)) {
			out.write('q');
		} catch (final IOException ioe) {
			ioe.printStackTrace();
		}
		try {
			if (!process.waitFor(5L, TimeUnit.SECONDS)) {
				process.destroy();
				process.waitFor();
			}
			return process.exitValue();
		} catch (final InterruptedException ie) {
			System.out.println("Interrupted");
			ie.printStackTrace();
			Thread.currentThread().interrupt();
			return -1;
		}
	}


Rerun element-based hook when element changes

The context: useResizeEffect hook. This hook takes two arguments: an element's ref and a callback function. The callback function is called whenever the element is resized, e.g. due to responsive styling. This hook only subscribes/unsubscribes to/from ResizeObserver when mounted/unmounted to reduce waste.
Currently the issue is that ref.current could change and this code does not take this into account. I see to options to remedy this:
/1/ Use dependency-based effect.
function useResizeEffect(ref, callback) {
  useEffect(() => {...}, [ref.current]);
}

The "bad" part here is that on initial rendering, the useEffect will be called twice: first with ref.current == null and then with ref.current == rendered-element. Thus this version does one extra subscribe/unsubscribe from the ResizeObserver.
/2/ Manually compare elements
This approach is more efficient, but really messy.
function useResizeEffect(ref, callback) {
  const prev = useRef();
  const observer = useRef();
  useEffect(() => {
    if (prev.current != ref.current) {
      if (observer.current) {
        observer.current.unobserve(prev.current);
      } else {
        observer.current = new ResizeObserver();
      }
      observer.current.observe(ref.current);
      prev.current = ref.current;
    }
    return () => {
      if (observer.current) {
        observer.current.disconnect();
        observer.current = null;
      }
    };
  }, [/*mount only*/]);
}

Which of these more idiomatic?

If the initial render has ref.current == null, is it really an extra subscribe? My understanding was that ResizeObserver would not be initialized if the ref's current value was null.
I find the second one reasonably idiomatic. If it's unlikely the ref will ever swap between two nodes in need of observation, it could be simplified as follows (incurs extra ResizeObserver if swapped):
function useResizeEffect(ref, callback) {
  const current = ref.current;
  const observer = useRef();
  useEffect(() => {
    if (current) {
      observer.current = new ResizeObserver();
      observer.current.observe(current);
    }
    return () => {
      if (observer.current) {
        observer.current.disconnect();
      }
    };
  }, [current]);
}

If the initial render has ref.current == null, is it really an extra subscribe? My understanding was that ResizeObserver would not be initialized if the ref's current value was null.

Yes. This is a bit of a misconnect with how effects are scheduled and executed when using a ref. In this example:
function useResizeEffect(ref, callback) {
  const current = ref.current;
  useEffect(() => {
    if (current) {             // <- Executed post-rendering so the current is now NOT null.
      ...
    }
    return () => {...};
  }, [current]);                // <- Executed during rendering phase, so current starts as null.
}

Another approach using useCallback: https://github.com/ampproject/amp-react-prototype/pull/46

Dedupping this to #55.


Handle exceptions extracting used props

Fix #595

Description
Add a try/catch in extractUsedProps when the rule is evaluated. If the rule throws an unhandled exception, returns an empty array.
Patch

fela-bindings

Checklist
Quality Assurance

You can also run yarn run check to run all 4 commands at once.

[x] The code was formatted using Prettier (yarn run format)
[x] The code has no linting errors (yarn run lint)
[x] All tests are passing (yarn run test)
[x] There are no flow-type errors (yarn run flow)

Changes
If one of the following checks doesn't make sense due to the type of PR, just check them.

[x] Tests have been added/updated
[x] Documentation has been added/updated
[x] My changes have proper flow-types

Thanks for the PR!


export pdf when online (LAN/Wifi) is not the same as if Offline (no Internet)

I often ride train and ther i found this strange behavour.
When im online (wifi on) export to pdf does not show the same menu as when im offline (wifi off/no internet connection).
You can check by going offline and hiting following sequence:

File
Export as
pdf

then toggle internet on or off and check again.
For me it looks like this when online:

and this when offline:

I personaly use the crop and selection only always so its quite anoing that i do not have this features when offline...
Is this a limitation of some packages or why is this behavoir implemented?
i just realized that i was maybe missleaded. Im suffering this features in draw.io.

this is answered here https://github.com/jgraph/drawio/issues/163
please close and delete if possible.


Pass worker to resource

pass worker amount to resource strategy to help allocating resources for suite

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@4ab8d0d). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #258   +/-   ##
=========================================
  Coverage          ?   97.29%
=========================================
  Files             ?       31
  Lines             ?      960
  Branches          ?        0
=========================================
  Hits              ?      934
  Misses            ?       26
  Partials          ?        0

Impacted Files
Coverage Δ

src/test_runner.js
97.92% <ø> (ø)

src/cli.js
94.94% <ø> (ø)

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 4ab8d0d...1f74812. Read the comment docs.


Fix #28

By updating YasumiTest to call the factory with a max of 9998 so that when it is +1'ed it is within the limit.

Ah just beat you to it :) I push a changed to the master branch covering this. (Covers both the 'next' and the 'previous' methods).

Shucks :p


gifs emojis still dosnt work for smart reaction also there is no Priority for reacting a msg

gifs emojis still dosnt work for smart reaction also there is no Priority for reacting a msg

I was able to add an animated emoji through the addreact command. Please make sure it is an emoji that the bot can see on one of the servers it's on.
What do you mean by "no priority for reacting a msg"?

Actually this was fixed in v3 of the cog not the v2 cog

like when you add first 1 then 2 then 3 and 4
it add like 4 2 1 3 emojis to a msg

Closing as this is a duplicate of #44. Fixes addressing this will be tied to that issue.


Release 1.2.5

[ ] Bump version number
[ ] Update ChangeLog
[ ] Check for performance regression
[ ] Test some important downstream projects
[ ] Tag the repository
[ ] Publish to npm

See PR #1085 for the change.

No conclusive performance change was observed.
Before:
    Underscore 1.5.2    42.5 KiB      7.07 ms ± 1.71%
      Backbone 1.1.0    58.7 KiB      7.47 ms ± 1.83%
      MooTools 1.4.5   156.7 KiB     37.34 ms ± 1.81%
        jQuery 1.9.1   262.1 KiB     39.87 ms ± 1.79%
          YUI 3.12.0   330.4 KiB     37.11 ms ± 2.07%
 jQuery.Mobile 1.4.2   442.2 KiB    116.08 ms ± 7.04%
       Angular 1.2.5   701.7 KiB     85.69 ms ± 3.07%
                     ------------------------
                      1994.3 KiB    330.63 ms ± 4.61%

After
    Underscore 1.5.2    42.5 KiB      6.88 ms ± 2.09%
      Backbone 1.1.0    58.7 KiB      7.79 ms ± 1.85%
      MooTools 1.4.5   156.7 KiB     36.65 ms ± 1.92%
        jQuery 1.9.1   262.1 KiB     39.05 ms ± 1.83%
          YUI 3.12.0   330.4 KiB     37.19 ms ± 2.09%
 jQuery.Mobile 1.4.2   442.2 KiB    112.65 ms ± 6.30%
       Angular 1.2.5   701.7 KiB     85.96 ms ± 3.11%
                     ------------------------
                      1994.3 KiB    326.18 ms ± 4.21%

No functional regression was observed on the following downstream projects:

Istanbul (current master)
JSCS (current master)
ESLint 0.10.2

1.2.5 has been published!


Study attainment expiry date inconsistency

Expiry date in study attainments editing view doesn’t match the one shown on course instance view. Could be caused by the mock data still used in the frontend

/course-view/1 shows the expiry date 14.09.2024 for Exercises, and clicking on edit still shows its expiry date is the 14th of September 2024. Updating Exercises to expire in 2025 in mockAttainmentsClient.tsx also updates the UI to 2025 in both places.
There doesn't seem to be any problem as far as I can tell.


PR Interview 2 - 03.02.2023

Requirements:

Implement a 64px NavBar to the top of the application.
Navbar shouldn't go over other content.
All content must be neatly aligned, centered and have consistent paddings.
Should support all screen sizes from mobile to desktop (375p to 1920p)

Navbar alignment is desktop looks good. Center alignment for mobile devices is not working correctly.
Navbar shouldn't go over other content. z-index value is 1 in few places. Default value is 0. Please check if 1 is still required.


[vite:css-post] Plugin error - Unable to get file name for unknown file "2fa9c3fa".

Related plugins

[X] plugin-vue

[ ] plugin-vue-jsx

Describe the bug
I use legacy on vite4.2.0+@ vitejs/plugin legacy4{
targets: ['chrome 58']
}Error reported during rendering chunks
Reproduction
not have
Steps to reproduce
Run 'npm run build'
:[vite:css-post] Plugin error - Unable to get file name for unknown file "2fa9c3fa".
✓ built in 7.80s
error during build:
RollupError: Plugin error - Unable to get file name for unknown file "2fa9c3fa".
at error (file:///vite-project-platform/node_modules/rollup/dist/es/shared/node-entry.js:2128:30)
at Object.FileEmitter.getFileName (file:///vite-project-platform/node_modules/rollup/dist/es/shared/node-entry.js:23972:24)
at file:///vite-project-platform/node_modules/vite/dist/node/chunks/dep-79892de8.js:38026:43
at String.replace ()
at resolveAssetUrlsInCss (file:///vite-project-platform/node_modules/vite/dist/node/chunks/dep-79892de8.js:38025:37)
at Object.renderChunk (fvite-project-platform/node_modules/vite/dist/node/chunks/dep-79892de8.js:38054:32)
at file:///vite-project-platform/node_modules/rollup/dist/es/shared/node-entry.js:24395:40
at processTicksAndRejections (node:internal/process/task_queues:96:5)
System Info
System:
    OS: Windows 10 10.0.19044
    CPU: (16) x64 AMD Ryzen 7 4800H with Radeon Graphics
    Memory: 2.45 GB / 15.37 GB
  Binaries:
    Node: 16.15.1 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.18 - ~\AppData\Roaming\npm\yarn.CMD
    npm: 8.11.0 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Edge: Spartan (44.19041.1266.0), Chromium (113.0.1774.35)
    Internet Explorer: 11.0.19041.1566
  npmPackages:
    @vitejs/plugin-legacy: ^4.0.2 => 4.0.2
    @vitejs/plugin-vue: ^4.1.0 => 4.1.0
    vite: ^4.2.0 => 4.2.1

Used Package Manager
npm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

import { defineConfig } from 'vite'
import path from 'path'
import vue from '@vitejs/plugin-vue'
import AutoImport from 'unplugin-auto-import/vite'
import VueComponents from 'unplugin-vue-components/vite'
import { ElementPlusResolver } from 'unplugin-vue-components/resolvers'
import { VantResolver } from 'unplugin-vue-components/resolvers'
import UNO_CSS from 'unocss/vite'
import legacy from '@vitejs/plugin-legacy'
// https://vitejs.dev/config/
export default defineConfig({
resolve: {
// 设置src别名
alias: {
'@': path.resolve(__dirname, 'src')
}
},
plugins: [
vue(),
AutoImport({
imports: ['vue', 'vue-router'],
resolvers: [ElementPlusResolver()],
dts: 'src/auto-import.d.ts'
}),
VueComponents({
dts: 'src/components.d.ts',
resolvers: [ElementPlusResolver(), VantResolver()]
}),
UNO_CSS(),
legacy({
targets: ['chrome 58']
})
],
publicDir: 'public',
base: './',
build: {
// target:['chrome 58'],
rollupOptions: {
output: {
chunkFileNames: 'js/[name]-[hash].js',
entryFileNames: 'js/[name]-[hash].js',
assetFileNames: '[ext]/[name]-[hash].[ext]'
}
}
}
})


Allow hosts to add a manual payment method for orders

Now that we support pending orders, we should allow hosts to add a manual payment method.
Use case:
AirBnB wants to give $10k to Webpack, they won't process such payment on a credit card. They want to do a wire. They should be able to select "wire" as a payment method and receive the bank details of the host. Then the host should be able to manually mark that order as paid when the money has been received on their bank account.

We will be needing this for the new expense flow, it'd be great to tackle it during the hackathon

This should be linked to the new business model. People should pay $10/month to be able to accept donations directly on their bank account with the wire transfer/pending order feature.
As a quick implementation to unblock some users and also to already start getting feedback on that feature and pricing, this is what I'd suggest:

We could link to the documentation page for the latest pricing regarding hosts while we update the current pricing page.
And I'd create a new $10/month tier on https://opencollective.com/opencollective for "Open Collective Pro" (by lack of better name, we can always change it later) and redirect people to the contribution flow for that tier.
I'd leave for another issue the part for actually enforcing becoming a member of that tier to use that feature after $1,000 received.

couple of thoughts:

It should be in the same place where you connect Stripe. It's weird that you are mixing it with a payout method instead of the other pay-in method.
Is the + sign taking you to a place where you can add the copy from the email to be sent? If yes, it needs to be super clear.
I think contextual FAQ is needed. Right now, I would expect almost like integrating a bank account, not sending an email and creating a pending order I need to manually check.

Please call it Open Collective Monthly Tier (don't use PRO)

I'd leave for another issue the part for actually enforcing becoming a member of that tier to use that feature after $1,000 received.

Please open this issue and start describing it if you are going to own the enforcement of this policy. So we don't end up doing a bunch of support for an issue no one is working on / owning.

Bear in mind that is meant as a quick fix compared to what we have now:

Right now Stripe is in “Connected Accounts” alongside Twitter which I think is fine for now but should be ideally moved or duplicated to the Payment Methods settings pane. Reason being that well, those are all “Payment Methods”, i.e. ways for receiving or sending money. So it’s more Stripe that should be moved here. The fact that it is right now in “Connected Accounts” alongside Twitter is a very engineer centric view (because they are both accounts connected via oAuth to your open collective account). But that’s beyond the scope of this.

The button will take you to a form where you can add your bank account details and private instructions that are sent by email. Given that you are not modifying anything yet, it’s safe for people to click to discover what’s behind.
Yes contextual FAQ is needed but it’s not designed yet for those pages. That’s why for now I’ve added help text below each button.

So it’s more Stripe that should be moved here
Please also open an issue about this with you as the owner. Otherwise it will never happen and the fix like this is confusing. You can then assign it to someone else. But you should own this feature.

That’s why for now I’ve added help text below each button.
Ok so maybe explain this properly in the form.
We are not enabling to receive donations automatically. So maybe the button should read, to send your account details to your contributors and receive donations.... ?

We definitely need this feature. Maybe it makes more sense to move Stripe to 'payment methods' too. In my ideal world, the host would be able to freely define the options here. For example, bitcoin, paypal (direct, not via contribution flow), cash, etc. For each, you'd just need a free text box where the host can input details, so I think it could be generalisable (let them set the payment method name too). But I'm happy to keep this to adding the wire transfer option only for now if it's best to keep it simple.

Maybe enable Contributors?
We are not using the word donors anywhere else.

Also, why "kindly ask you"? Are we doing this optional?

@alanna yes, would totally love to add paypal and possibly other payment manual methods (e.g. Venmo). Since the host has to manually confirm reception of the money, it is very versatile. Then based on usage we can prioritize automating each of them.


HDDS-5686. Support transfer leadership in raft based ha cluster

What changes were proposed in this pull request?
Background:
In a raft based ha cluster, for some reason, we want to transfer leadership to a specific follower gracefully, now, we have to kill the leader master to transfer leadership, but the leadership could transfer to an unexpected follower, we have to kill the leader master again, this is so tricky.
Usage:
This feature is assumed to utilizing a shell command like ozone admin failover  [SCM/OM]  [host/host:port/'random'] , with optional IpAddr we can transfer the leadership to the target follower, with 'random' we can transfer leadership to any node except leader node.
The whole procedure is as following:

Pseudo raft client -> get real raft group info -> check address
whether in raft peers -> real raft client -> set priority -> trigger
transferleadership -> new raft leader takes office

What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-5686
Please replace this section with the link to the Apache JIRA)
How was this patch tested?
manual tests.
the Integration test will be added soon
Here are some screenshots

@bharatviswa504 @bshashikant PTAL, THANKS!

i am not sure, if the cmd does any authetication/security validation here. The raft client is directly talking to the raft server. We need to fit the cmd to security model in ozone.
cc @bharatviswa504

This implementation looks a  little tricky, it's indeed directly talking to the raft server. I agree that authentication/security should be added in the future.  : )

i am not sure, if the cmd does any authetication/security validation here. The raft client is directly talking to the raft server. We need to fit the cmd to security model in ozone.
cc @bharatviswa504

This implementation looks a little tricky, it's indeed directly talking to the raft server. I agree that authentication/security should be added in the future. : )

One idea might be similar to DN, contact OM and get list of CA and use that in setting up with raft client. And we can make this an admin command, and in secure cluster we need kerberos ticket to run this command.
For security question this way we can solve it. And also we need this security/authorization implemented, otherwise this command cannot work in a secure cluster.
Sample code:
    final GrpcTlsConfig tlsConfig = RatisHelper.createTlsClientConfig(new
        SecurityConfig(ozoneConf), caCerts);

And use this tlsConfig when creating RaftClient. Here caCerts is obtained from OM getServiceList Call. You can refer RpcClient for this.

@bharatviswa504 , can you please check this one?

/pending

Hey @Xushaohong , could you update the patch according to our discussion at your convenient time?

Hey @Xushaohong , could you update the patch according to our discussion at your convenient time?

Sure， i will open a new PR ASAP


⚠ ClamAV has degraded performance

In fb2fe72, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 446 ms

Resolved: ClamAV performance has improved in bf7273d after 17 minutes.


Fix input_offset resetting in stream

Close #22, close #23
The incorrect input_offset resetting causes that, while the stream outputs are still decompress-able, when the input is long enough, the output would become incorrect (means decompression can not output the original input). The PR fixes it.

Amazing @myl7! Thanks for investigating this.
Do you know how long the input has to be to trigger this issue? Is it possible to write a test to reproduce it, just to check this fix and to ensure we don't run into the same issue again in future?

A ~570KB file is enough. How about to add a test with a 600KB file?

Yes, that sounds good to me. I'm happy with either committing a 600KB file or just generating it at runtime (we can use getRandomValues in a loop for 65K at a time) assuming that's fairly quick.
We need to remember to confirm that any new test for this does definitely fail in the previous implementation without this fix :smile:

Oops, sorry, that was closed automatically due to merging #26.

My fault😂. I mistakenly used the term like "the fix # 25" in that.
I checked the processing of input_offset.
The problem looks more like a design choice rather than logical error.
When the instance requires more input and returns NeedsMoreInput, input_offset is previously set to input.len(), which is fair since for this chunk all bytes have been consumed.
However, since in the TransformStream we use do {} while {} and go to the next chunk immediately after getting NeedMoreInput, input_offset should be 0 for the next chunk but incorrectly kept input.len() which is for the previous chunk.
It causes the problem.
In the tests since input_offset is manually managed so the problem does occur.
The above mistakes seem to cause the instance keeps some chunks so the output is still decompressable but can not be recovered to the original input.
Adding an extra step in the TransformStream helper to manually reset input_offset seems also OK.
But I personally slightly prefer resetting input_offset in the library like the fix because it is easier for users to use.
How about your preference?

I see, thanks for the explanation, I think I understand the issue much more clearly.
What's really going on here is that within brotli-wasm we're tracking state that tightly depends on how it's used in the stream wrapper - it's not really part of our Brotli streaming implementation at all. It's breaking here not because it's wrong, but because it doesn't match details of how that other implementation works. The fixed logic in this PR to update input_offset makes sense in the current suggested implementation, but might still be wrong in other implementations - for example reading from a Buffer that is being appended to elsewhere (i.e. where the input offset doesn't change between chunks), something like this:
const buffer = /* ... some fixed buffer that's being written to elsewhere */
let inputOffset = 0;

while (true) {
    const nextInput = buffer.slice(inputOffset, endOfBuffer);
    output.append(this.stream.compress(nextInput, outputSize));

    if (this.stream.result() === brotliWasm.BrotliStreamResult.NeedsMoreInput) {
        // Wait for data from elsewhere to update endOfBuffer
        await waitForMoreData();

        // Last_input_offset will often be wrong here - it really should be endOfBuffer
        // now, but in fact it will be 0, because it's assuming that buffer is reset.
    }
}

(Very rough code, but you get the idea)
Even when it's correct, it's a bit 'too clever' and I think moving that into the stream wrapper (but making it easy to implement there) would be simpler & clearer.
What would happen if we remove last_input_offset completely, and instead make compress and decompress return the details of what was consumed each time. Then it's up to the stream wrapper to handle that and track input offset itself, depending on how the input actually works. In the web stream case, the stream implementation would be something like this:
let inputOffset = 0;
do {
    const input = chunk.slice(inputOffset);
    const { output, consumedInputLength } = this.stream.compress(input, this.outputSize);
    inputOffset += consumedInputLength;
    controller.enqueue(output);
} while (this.stream.result() === brotliWasm.BrotliStreamResult.NeedsMoreOutput)

What do you think?

True. Returning input_offset other than keeping it in the stream looks better.
I have updated the stream Rust code with the tests.
If OK, I will then also update the stream decompression example PR #27 .


Not Working

it is throwning an exception saying that it is not allow to connect to Mysql Server

check the mysql connector

you can use sqlite or any sql server connector like Postgres over the mysql if you don`t have mysql

import sqlite3

class Database:

 path = './libs/'

    dbName = 'store.db'

    def __init__(self):
        self.connection =
      sqlite3.connect(self.path+self.dbName)
        self.cursor = self.connection.cursor()

    def fetch_query(self, query):
        self.cursor.execute(query)
        results = self.cursor.fetchall()
        return results

    def fetch_columns_names(self, table_name):
        query = "PRAGMA table_info({})".format(table_name)
        results = self.fetch_query(query)
        columns_names = []
        print(results)
        for row in results:
            if row[1]!='id':
                columns_names.append(row[1])
        self.connection.commit()

        return columns_names

    def create_table(self, table_name, columns):
        query = "CREATE TABLE IF NOT EXISTS {} ({})".format(table_name, columns)
        self.cursor.execute(query)

    def insert_data(self, table_name, data):
        query = "INSERT INTO {} ({}) VALUES ({})".format(table_name,self.fetch_columns_names(table_name), data).replace('[', '').replace(']', '')
        print(query, "--------> Running ")

        self.cursor.execute(query)
        self.connection.commit()
        self.close_connection()

    def select_data(self, table_name, where_clause=None,fetchall = True):
        query = f"SELECT * FROM {table_name} {where_clause}"
        if where_clause:
            query = query.format(where_clause)
        print(query, "--------> Running ")
        self.cursor.execute(query)
        if fetchall ==False:
            results01 = self.cursor.fetchone()

        else:
            results01 = self.cursor.fetchall()
        self.connection.commit()
        self.close_connection()

        return results01

I think this was a localhost connector problem 🙃 Closing this... feel free to create another issue if this issue still persists :)


GDScript: Fix @export_enum works only with int

Closes #54828.
There was a bug in the PROPERTY_HINT_ENUM docs, float is not supported (displays a regular field, not a list).

Thanks!


Upload finished: undefined

INFO  Now start qiniu sync.
INFO  Need upload file: static/images/2015-04-26.jpeg
INFO  Upload finished: undefined

+1

please use version >= 1.4.2


Initial Loss problem When Fine-Tuning TinyChart-3B-768 with TinyChartData

I encountered an issue while fine-tuning the TinyChart-3B-768 model using the TinyChartData dataset. The initial loss is unexpectedly high, reaching 7.6. Additionally, when using the original DeepSpeed script zero3_offload_decay.json, the loss remains constant at 0 throughout the training process.
I changed the version of deepspeed based on pyproject.toml, based on the llava-v1.5 environment; ran vit_add_tome.py against TinyChart-3B-768-siglip.
Are there any dependencies or configurations that I might be missing which could cause the initial loss to be so high?
Why does the loss remain at 0 when using the original DeepSpeed script?
I would appreciate any guidance or insights into potential dependency issues or misconfigurations that could lead to these problems.
Thank you for your assistance.

Hi @buckerF,
We also encounter the zero-loss issue as you mentioned sometimes. It is very likely that there is NaN in the forward/backward pass due to fp16 precision. You can try to change bf16 or fp32 instead.
We do not encounter the high-loss issue you mentioned. Since Tinychart has been trained on the data, the loss is intended to be low. Do you make sure that the pre-trained parameters are correctly loaded?


data: d4community

Fixes Issue

Changes proposed

Check List (Check all the applicable boxes)

[ ] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.
[ ] This PR does not contain plagiarized content.
[ ] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

I have to add one for my community organization

I have to add one for my community organization

If you create an account for your community org on GitHub, you will want to close this PR attached to your personal account and then open it while logged in as your community. You cannot submit a profile on behalf of another github account.  Let me know if this makes sense or if you need anymore help!

Make sense, but creating the community profile only for this is maybe not a good idea. Anything else is possible?

Make sense, but creating the community profile only for this is maybe not a good idea. Anything else is possible?

I am not sure there is any other way to have a separate linkfree for your community other than having the separate github account, but looping in @eddiejaoude to confirm as perhaps there is a newer feature I am not aware of yet or something coming that would allow for you to have these accounts separated


API: Add eye function

Hi @willow-ahrens @hameerabbasi,
This PR adds eyeArray API function.
@willow-ahrens I think that as a follow-up I could add format argument but, for COO:
using Finch
tns = Tensor(SparseCOO{2}(Element(0)), 3, 4)
@finch begin
    for i=_, j=_
        if i == j
           tns[i, j] += 1.0
        end
    end
end

it fails with:
ERROR: Finch.FinchProtocolError("cannot modify Finch.VirtualSparseCOOLevel in place (forgot to declare with .= ?)")

Declare it first with tns .= 0

Do you think we should just add speye and spdiags to finch? I'll work on that this evening.

eye is pretty simple to implement in finch-tensor, but sure - Finch functions that follow:

https://data-apis.org/array-api/latest/extensions/generated/array_api.linalg.diagonal.html
https://data-apis.org/array-api/latest/API_specification/generated/array_api.eye.html
Would make it even simpler in finch-tensor.

We could have format=... argument there? Or always return coo

I just realized that speye and spdiags are classic matlab functions, so we should probably also do it in Finch.jl

Sure!

I created an issue. The PR is ready I think.


Cap the maximum reward per ECDSA keep operator per interval

Refs #446
Set a limit on the amount of KEEP a single ECDSA keep operator can receive in a single reward interval.
Use the withdrawal pattern to track allocated and withdrawn rewards. Rewards exceeding the cap will be returned to the unallocated pool.
Because the reward payouts are changed to the withdrawal pattern, the existing ECDSA rewards tests should not work. For some reason this is not the case and the tests pass despite the breaking changes.

I fixed a problem in tests in https://github.com/keep-network/keep-ecdsa/pull/584/commits/7dc61ccae4c373b5340dd448c1cea32cf8744f72, now they are failing as expected.

... And added calls to withdrawRewards where applicable so they are now passing as expected.

Code looks good to me, but tests will be added in a separate PR.

The last three comments and then I am ready to merge this one and https://github.com/keep-network/keep-ecdsa/pull/587 soon after.


Problem switch php version

Hi
I end up whit this error when i try to switch:
root@Server1:~# phpbrew switch 5.6.17
You should not see this, if you see this, it means you didn't load the ~/.phpbrew/bashrc script, please check if bashrc is sourced in your shell.
I am not sure what i should check or change.

This wiki page can solve your problem.


Add Dice Game

Related Issue

Info about Issue or bug

Closes: #[issue number that will be closed through this PR]
Describe the changes you've made
A clear and concise description of what you have done to successfully close your assigned issue. Any new files? or anything you feel to let us know!
Describe if there is any unusual behaviour of your code(Write NA if there isn't)
A clear and concise description of it.
Checklist:

[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[ ] New and existing unit tests pass locally with my changes.

Screenshots
Put any screenshot(s) of the project here.

Good Evening Sir, My previous pull request have some issue so sent a another pull request . Please Check the issue

Please Review My Code

@iamrishavraj1 Please resolve conflicts !!

Done Sir.

@iamrishavraj1 Please resolve conflicts !!

Done sir @Vishal-raj-1

@iamrishavraj1 please resolve them again

@iamrishavraj1 please resolve them again

done mam

@Vishal-raj-1 @urvashi-code1255 @harshgupta20  Please review my work


Improve quantization flow

Description
@pengzhao-intel @TaoLv @xinyu-intel @reminisce @KellenSunderland @anirudh2290
Major changes:

Don't need calib_layer, the layer list which needs calibrated will be generated from quantization pass, so user don't need to specify that. Only needed layer get calibrated, thus improve the calibration speed.

Entropy is refactored, only histogram of output is saved, which will help to reduce memory consumption. Entropy calculation is refactored as c++ operator, accuracy is improved, then entropy method can get same speed as naive.

Model
FP32 Accuracy
INT8 Naïve
INT8 entropy_old
INT8 entropy_new

ResNet50-V1
76.340%
76.060%
76.006%
76.053%

Squeezenet 1.0
56.980%
56.790%
55.584%
56.997%

MobileNet 1.0
72.230%
72.060%
71.822%
71.822%

MobileNetV2 1.0
70.270%
69.820%
69.950%
70.016%

Inception V3
77.760%
78.050%
78.019%
78.053%

Inception-BN
72.280%
72.020%
72.084%
71.978%

mean_for_all
70.977%
70.800%
70.578%
70.820%

Add new quantization mode smart, which will automatically decide each op should be quantized or not. This mode will only quantize nodes which have performance benefit(e.g. convolution and FC), and necessary nodes. For example, A is convolution or FC, which will be all quantized. B is Relu or Add, which is quantizable and quantization flow will make decision whether to quantize it or not. C is non-quantized node. For A -> B -> A, B will be quantized as it can pass down int8 data. For C->B->C, A -> B -> C, or C -> B -> A, B won't be quantized.

Add log for quantization flow, this can help to user to understand what quantization flow does and what's changed.

Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[ ] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[ ] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

@KellenSunderland the requests from your team :)

@ZhennanQin @xinyu-intel please rebase the code and retrigger the CI


Upload media

@runongirlrunon

This closes https://github.com/summer17-csc648-team1/warehausmedia/issues/9.


Swdeden personalidentitynumber - format acceptance and age validations

https://swedish.identityinfo.net/personalidentitynumber
The format states the [-+] are optional.

11900102-2384 -> invalid because it needs + to indicate over 100 years of age, also im thinking 1800 should be a cutoff.
119001022384 would also be an acceptable format, but not pass due to 1800 cutoff (1190 is YYYY)
5505260017 -> valid because - is optional
198902255630 -> valid because - is optional
19781204+0009 -> invalid because + indicates age is over 100 years
19200102-0755 -> invalid because - indicates the age is under 100 years

I wonder if the - and + gets assigned only once in a person's life, and gets updated after a person turns 100 to make it accurate after they roll over 100 mark. I bet that doesn't happen often, most people don't live that long.

I've been working on this... It makes sense to "fix" the year to 4 digits and perform all validation after that. Not "stress" over the +- difference for the long form version.


config-examples: add fred-drake

Added my personal configuration, focused on automatic formatting, LSP and debugging of several popular languages.

@mergify queue


Correct way to interact with the value of Input , or Control child element content inside a component.

I am making a file manager app base on react-redux, and I find it is super in-convenient to use input in react-redux.
For example:
There is a PathForm, it need show path from two way

user click a file path from left tree view, Explorer get this path as currentPath, then pass to PathForm, and show currentPath in input
user directly type a path to the PathForm's input,  PathForm call handleSubmit(Explore's function) to change the currentPath

Additional 3: I want to keep PathForm as a stateless component
I can't figure out how to achieve above purpose in react-redux.
Because this input can be change at two place , can't achieve without connecting to store and  another state inputValue
After some effort , I solve them with this:
export default class PathForm extends Component {
  changeCurrentPath(path) {
    const pathInput = document.querySelector('.current-path-input')
    if (pathInput){
      pathInput.value = path
      this.lastPath = path
    }
  }

  render() {

    const { currentPath,  handleSubmit } = this.props;
    console.log('PathFormPathFormPathForm', this.props)

    this.changeCurrentPath(currentPath)

    return (

        <form onSubmit={handleSubmit}>

            <input type="text" className="current-path-input" placehoder="input path" />

          <button className="go-btn" type="submit">Go</button>
        </form>

    );
  }
}

But I think it is not the most correct way,
It would be much better if I can control child elements of PathForm  without document.querySelector(to avoid the class conflict ).
There is a detail about this(Due to no one against above answer, I come here to ask if I am right)
http://stackoverflow.com/questions/38218629/easy-change-input-value-from-different-wayreact-redux/38220100#38220100

This is a usage question, not a bug report or feature request.  Usage questions should be asked on Stack Overflow, which you've already done.
Also, the code you've shown doesn't even have anything to do with Redux or the React Redux connect function at all.
You might want to go read some of these articles on how to properly use forms in React: https://github.com/markerikson/react-redux-links/blob/master/react-redux-architecture.md#react-and-forms .  In particular, I think you want to read up on the "controlled inputs" pattern.  I also described this concept in a gist, over at https://gist.github.com/markerikson/d71cfc81687f11609d2559e8daee10cc.

Thanks, that gist is really helpful, why not add it to doc or provide an example app

Thanks, that gist is really helpful !


Set tuning to psychovisual by default.

Also correct help message, now that it works for 4x4.

Coverage remained the same at 79.626% when pulling edb6e523780b2c889e522c1c038d93f7c62f4789 on tdaede:enable_psy_defualt into 83aa2b04097b6af53b6a70a82346dfe38c77a07e on xiph:master.


Internal registry

publish containers to internal docker registry
publish to internal py registry

@miraai the docker pushing is finally working 🎉 but i have a permissions issue while pushing the pypi package to the internal google artifact registry:
ERROR: (gcloud.artifacts.print-settings.python) PERMISSION_DENIED: Permission 'artifactregistry.repositories.get' denied on resource '//artifactregistry.googleapis.com/projects/rasa-releases/locations/europe-west3/repositories/rasa-plus-py' (or it may not exist).
- '@type': type.googleapis.com/google.rpc.ErrorInfo
  domain: artifactregistry.googleapis.com
  metadata:
    permission: artifactregistry.repositories.get
    resource: projects/rasa-releases/locations/europe-west3/repositories/rasa-plus-py
  reason: IAM_PERMISSION_DENIED

The above error is from the tagged build 3.8.0a8.dev1. the google auth uses secrets.RASA_OSS_RELEASE_ACCOUNT_KEY which is the same one that is used for the niglty_release builds so i hoped it'd just work. Do you have any idea what might be off with the permissions?
https://github.com/RasaHQ/rasa/actions/runs/6277978314/job/17051495441#step:12:193

@miraai the docker pushing is finally working 🎉 but i have a permissions issue while pushing the pypi package to the internal google artifact registry:
ERROR: (gcloud.artifacts.print-settings.python) PERMISSION_DENIED: Permission 'artifactregistry.repositories.get' denied on resource '//artifactregistry.googleapis.com/projects/rasa-releases/locations/europe-west3/repositories/rasa-plus-py' (or it may not exist).
- '@type': type.googleapis.com/google.rpc.ErrorInfo
  domain: artifactregistry.googleapis.com
  metadata:
    permission: artifactregistry.repositories.get
    resource: projects/rasa-releases/locations/europe-west3/repositories/rasa-plus-py
  reason: IAM_PERMISSION_DENIED

The above error is from the tagged build 3.8.0a8.dev1. the google auth uses secrets.RASA_OSS_RELEASE_ACCOUNT_KEY which is the same one that is used for the niglty_release builds so i hoped it'd just work. Do you have any idea what might be off with the permissions?
https://github.com/RasaHQ/rasa/actions/runs/6277978314/job/17051495441#step:12:193

Let me take a look

@tmbo I added write permissions for that service account, try again and let me know!

superseded by https://github.com/RasaHQ/rasa-private/pull/15


Programmatically Clear Cache

Hey! Creator of the CustomJS plugin here. I have a side effect of using my plugin with dataviewjs blocks that I want to run by you to see if I can resolve it somehow.
CustomJS has simplified my dataviewjs blocks, since I'm able to move all of the logic into a reusable JS class. For example, retrieving my tasks without a due date is simply
```dataviewjs
const {DvTasks} = customJS
DvTasks.getTasksNoDueDate({app, dv, luxon, that:this})
```

Since dataview seems to cache the result, if I make a change to the underlying getTasksNoDueDate code, I have to then add/delete a space to my dataviewjs block to get it to rerun the code. Do you know of anything available in the dataview api — or obsidian api for that matter — that I could use to clear the cache programmatically for specific blocks and/or notes when I detect a change to a CustomJS js file?
Thanks for any help you can provide, and thank you for dataview! It's the sole plugin that got me to commit to Obsidian 😁

Setting up listeners for this may be a little hard - do you have a way in CustomJS or similar to detect when a file changes? The dataview index uses a monotonic ID to track changes (which is how it detects when the index has changed for re-rendering) - it would be easy to add a programmatic API call (like dv.forceReloadViews()) which increments this ID and causes views to update.

I've actually got the listeners working! I'm already reloading custom code whenever JS files change in the vault via this code:
this.registerEvent(this.app.vault.on('modify', this.reloadIfNeeded, this))
where reloadIfNeeded just checks the file extension. So I would be able to call something like dv.forceReloadViews() inside that block. Thanks for being willing to help out! If I am able to find the time I'll try to look through the dataview code and see if I can contribute.

Okay, great - I've added dv.index.touch() which increments the revision number and will cause views to reload. This will be part of release 0.4.11 which you can expect tonight.

Awesome! Thanks for the quick response! I'll get it tested out right away when it's released.

Sure thing!


Redesign Survey Top Appbar

This is a Pull Request for issue #429. It contains the following:

Made changes to the Survey Top Appbar, so that it conforms to the ODK-X Design Guidelines in the colors.xml and theme.xml
Built and run the Survey app on an Android device. The screenshot is shown below

@Chinex-Boroja while this PR was great for demonstrating your skills it also has duplicate of changes made in the new UI branch. @Redeem-Grimm-Satoshi this will duplicate values already available. Is there anything you want to keep? We will likely close this PR unless direction is given to revise to avoid duplicate issues.


Pingback link is rendered

We are getting a report that we have a bad value for the pinkback link. Not sure why since it should not be rendered in the first place:
https://github.com/gios-asu/ASU-Web-Standards-Wordpress-Theme/blob/7ba9e4c508fd1bf114a2ea91ccfe0401d28539e0/header.php#L141-L143
The error reported:

Bad value for the href on element link: Must be non-empty

  <link rel="profile" href="http://gmpg.org/xfn/11">
      <link rel="pingback" href="">

It is showing up on the School of Sustainabiltiy site. Simply view the source to see it.

While reading about pingback I came across xmlrpc attack which is kind of brute force attack, DOS, DDOS and spam. And I see many articles suggesting to remove them(profile and pingback) if we do not need comments tracking. As we are disabling comments do you think it is good idea to remove these links itself?

regardless of us wanting it or not, we shouldn't render an empty href attribute. I don't think we will want to use it.


No module named antfs_cli

Bonjour,
Install of openant and antfs-cli was very well but i have this error:
Traceback (most recent call last):
File "/usr/local/bin/antfs-cli", line 9, in
load_entry_point('antfs-cli==0.2', 'console_scripts', 'antfs-cli')()
File "/usr/lib/python2.7/dist-packages/pkg_resources.py", line 356, in load_entry_point
return get_distribution(dist).load_entry_point(group, name)
File "/usr/lib/python2.7/dist-packages/pkg_resources.py", line 2476, in load_entry_point
return ep.load()
File "/usr/lib/python2.7/dist-packages/pkg_resources.py", line 2190, in load
['name'])
ImportError: No module named antfs_cli
what can i do?
Merci beaucoup.

@Tigge There's a problem with this refactor, because module is now antfs_cli.antfs_cli instead of just antfs_cli. Maybe antfs_cli/antfs_cli.py could be renamed to antfs_cli/__init__.py for this issue to be solved.

Hm, I thought I tested this -- must have had some old version installed as well. So we need to change the install script to:
      entry_points={
          'console_scripts': ['antfs-cli=antfs_cli.antfs_cli:main']
      },

Might make sense to rename the module to something else than the package name to avoid confusion. But should that work?

Might be worth adding a test for this as well, just running "antfs_cli --version" or something.

I think that should work too, but I think it's be better to avoid the duplicate module name antfs_cli.antfs_cli using the __init__.py file in antfs_cli directory.

or simply move back this file in root folder, while keeping others in the directory with an empty __init__.py

In any case, there should be an __init__.py file in this directory, and imports should like this
from . import utilities
from . import scripting

thank you for the answer, I test.
2015-01-27 18:57 GMT+01:00 Rémi Alvergnat<EMAIL_ADDRESS>
@Tigge https://github.com/Tigge There's a problem with this refactor,
because module is now antfs_cli.antfs_cli instead of just antfs_cli.
Maybe antfs_cli/antfs_cli.py could be renamed to antfs_cli/init.py
for this issue to be solved.
—
Reply to this email directly or view it on GitHub
https://github.com/Tigge/antfs-cli/issues/124#issuecomment-71694811.

--
J.D.

Any luck with this?  I followed all the instructions but now can't run:
antfs-cli -h
I get:
Traceback (most recent call last):
File "/usr/local/bin/antfs-cli", line 9, in
load_entry_point('antfs-cli==0.2', 'console_scripts', 'antfs-cli')()
File "/usr/lib/python2.7/dist-packages/pkg_resources.py", line 351, in load_entry_point
return get_distribution(dist).load_entry_point(group, name)
File "/usr/lib/python2.7/dist-packages/pkg_resources.py", line 2363, in load_entry_point
return ep.load()
File "/usr/lib/python2.7/dist-packages/pkg_resources.py", line 2088, in load
entry = import(self.module_name, globals(),globals(), ['name'])
ImportError: No module named antfs_cli

I've pushed a fix for this and https://travis-ci.org/Tigge/antfs-cli reports that it can run and install it. @pfergi42 could you try again now?

Whoo it works!  Great job @Tigge!
Now... what do I do with these .fit files?  Can I sync them to strava somehow? =)  (I know that's beyond the scope of this project).
Well done!

Here my workflow:

Use this antfs-cli to download .fit files from your watch to your computer.
Use GcpUploader to upload to garmin connect.
Use tapiriik to sync from garmin connect to many other services, including strava.

Thanks SO much! I was already using tapiriiik so I was just missing that
2nd step.  I'll give it a go and report back.
On Thu, Jan 29, 2015 at 2:37 PM, Rémi Alvergnat<EMAIL_ADDRESS>wrote:

Here my workflow:

Use this antfs-cli to download .fit files from your watch to your
computer.
Use GcpUploader
https://github.com/Tigge/antfs-cli/blob/master/scripts/40-upload_to_garmin_connect.py
to upload to garmin connect.
Use tapiriik https://tapiriik.com/ to sync from garmin connect to
many other services, including strava.

—
Reply to this email directly or view it on GitHub
https://github.com/Tigge/antfs-cli/issues/124#issuecomment-72118692.

I'm using something like https://gist.github.com/Tigge/2ad3bf6387ed2c38fd48 placed in my script folder for antfs-cli to upload to Runkeeper. Could probably do something similar with http://strava.github.io/api/v3/activities/ for Strava if tapiriik doens't work out.

Bonjour,
I found this:
http://forum.ubuntu-fr.org/viewtopic.php?id=1267521
it works well.
Thanks.
Cordialement.
J. D.
Le 29 janv. 2015 23:57, "Gustav Tiger"<EMAIL_ADDRESS>a écrit :

I'm using something like
https://gist.github.com/Tigge/2ad3bf6387ed2c38fd48 placed in my script
folder for antfs-cli to upload to Runkeeper. Could probably do something
similar with http://strava.github.io/api/v3/activities/ for Strava if
tapiriik doens't work out.
—
Reply to this email directly or view it on GitHub
https://github.com/Tigge/antfs-cli/issues/124#issuecomment-72120531.

I also found this service to upload to Garmin Connect.  But it's manual and one file at a time:
https://connect.garmin.com/api/upload/widget/manualUpload.faces?uploadServiceVersion=1.1&cid=3447601

(issue seems fixed, closing)


Retrieve master page info in Shape object and master shape name in shape text when it does not exist

Hi @dave-howard ,
I have created this pr in order to fix the task Retrieve master page info in Shape object
Thanks for your effort and feel free to reach me in order to do any change.
Cordially,
Juan.

Hi - I made a few changes. I was a bit confused by the change to Shape.text, so have removed that and updated the test to check the master shape that has the expected text (which happens to be same as the master page name).
I didn't think that Shape.text should return master name when that master shape really had no text
If I have misunderstood please let me know :)
KInd regards
Dave

Hi @dave-howard , thanks for your comment.
The change of Shape.text it is due the following behaviour.
We are doing something like:
`
shapes = vis.pages[0].shapes[0]
for shape in self.page.sub_shapes():
shape.text
`
But for the file test_master_multiple_child_shapes.vsdx, the shape.text is empty:

as the master shape child has more than one child and the text is not in the first one:

The library is not retrieving that name correctly, I think that the issue, please, correct me if I'm missing something.
Cordially,
Juan.

Hi @jmgarcia-iriusrisk,
I think Shape.text returning an empty string in this case is correct - if we change the behaviour of Shape.text to return value of first child Shape with non-empty text, then that will break some other use cases (i.e. Page.find_shapes_by_text() method)
That said - if it would be useful to have a method of a Shape that returns all text, or first text value.
In the example above - if you get shape by id 7 and check it's text property - you'll get the expected text - but the other peer and patrent shapes have an empty text so that's what gets returned
I've added a new test in da2c7e1a that includes your sample file
Dave


Keybase package is incomplete

Describe the bug
After installing keybase, keybase-gui, and kbfs, I did not have access to the run_keybase script or the keybase service file. Running keybase ctl start gives the following error:
$ keybase ctl start
▶ INFO Starting keybase.service.
Failed to restart keybase.service: Unit keybase.service not found.
▶ ERROR Failed to start keybase.service.
▶ ERROR Error in spawning server process: exit status 5
▶ ERROR exit status 5

keybase init runs without error, but does not fix the problem. Things will run while active terminal windows are left open running keybase service and kbfsfuse, but stop working as soon as you close those terminals.
To Reproduce

install keybase
attempt to use run_keybase or keybase ctl start

Notify maintainers
@Avaq @carlsverre
Metadata
$ nix-shell -p nix-info --run "nix-info -m"

system: "x86_64-linux"
host os: Linux 5.4.85, NixOS, 20.09.2387.0c81b9a4f17 (Nightingale)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.3.9
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute:
# a list of nixos modules affected by the problem
module: keybase, kbfs, keybase-gui

keybase.service is (somewhat by necessity) included in the NixOS module with the same name: https://github.com/NixOS/nixpkgs/blob/master/nixos/modules/services/networking/keybase.nix
Have you tried services.keybase.enable = true;

keybase.service is (somewhat by necessity) included in the NixOS module with the same name: https://github.com/NixOS/nixpkgs/blob/master/nixos/modules/services/networking/keybase.nix
Have you tried services.keybase.enable = true;

Keybase (unluckily imho) needs the running service so you can put services.keybase.enable = true; in your system configuration or if you don't like having the service always active you can run keybase service to launch it in foreground without the need for a systemd unit

Keybase (unluckily imho) needs the running service so you can put services.keybase.enable = true; in your system configuration or if you don't like having the service always active you can run keybase service to launch it in foreground without the need for a systemd unit


VSCODE auto detected the issue and it generated this request.

 Environment data

Language Server version: 2024.6.1
OS and version: darwin arm64
Python version (and distribution if applicable, e.g. Anaconda):
python.analysis.indexing: true
python.analysis.typeCheckingMode: off

 Code Snippet

XXX

 Repro Steps

XXX

 Expected behavior
XXX
 Actual behavior
XXX
 Logs

XXX

Thanks for the report. Unfortunately, this is not enough information for us to investigate this issue.
If this continues to be a problem, would you please reopen the issue and add the information described in the troubleshooting guide?


SH1107: Check for opa when setting a pixel on set_px_cb

I've fixed in lvgl too. Now set_px_cb is not called when the pixel is transparent.

Thanks @an-erd and @kisvegabor!


Feature request: Read from/write to in-game chat

This request is very label-on-the-tin type.
I first briefly mentioned this in JoeStrout's MiniScript Discord server; it can be found in the logs here. That request has been split into two requests, however, as it seems to be asking for two very different things in the underlying game as far as I can tell.
Detailed Description
The game provides an in-game chat for players that can be used to run server commands and talk to one another. Messages in the chat provide a log accessible to the player.  (I am aware that Farmtronics doesn't presently support multiplayer; it's of note that this chat is able to be accessed even in a single-player session. Default key map appears to be t.)
What I'm proposing is that the MiniScript interpreter in Farmtronics needs to be able to read from and write to this interface element.
Here's some snapshots of it open at the bottom of the screen on a modded version of the game.

Usecase or Problem Being Solved
There are two main uses for this, which can be separated into two components - reading and writing.

Reading

Right now, in order to issue commands to a machine or bot interactively, it's necessary to actually visit it.

Users could instead issue commands via chat and using a prefix to their message

Writing

Right now, in order to check the state of a machine or bot in-game, it's necessary to either visit it directly or else read the SMAPI console log.

The machine or bot could instead be programmed to shout state into chat for things that actually matter to the player

Both

Some useful utilities only require access to arbitrary code execution and a text interface, and could be written using Farmtronics instead of a full blown Stardew Valley Mod

Dice rolling
Information services about the game state
and so on

Possible Implementation
As far as under the hood, I don't know much about Stardew Valley modding, sorry.
As far as a few key ideas and points, however, here:

Write: The bot or computer, when speaking, should probably use its name as a prefix, just like players do.
Write: It may be prudent to provide a mod config option to let players deny bots and computers the ability to run console commands, to future proof against potentially naïve or malicious behavior in the community or around it.
Read: It may be best to maintain some variation on a global queue for log messages, as opposed to forcing players to specifically catch each message themselves when it first comes in.

Limiting the size of this would be wise for various reasons - not the least of which, it shouldn't eat up RAM for players who don't make use of it
Message numbering and throwaway when the queue exceeds a certain size would probably be the best utilities to let players see if they potentially missed a message.

Environment

Version used:

Farmtronics 1.1.0

Host operating system:

Windows 10 Home x64

I would say part of my intention is to get the player away from having to check a computer/bot for status updates entirely. Otherwise "right on!"

Implemented for the next release.  New method world.chat takes a string argument and adds it to the chat (along with the bot name, or "Home Computer").  And new method world.chatMessages returns a list of the last 10 or so chat messages.


WIP: Add compaction analysis blog post

This PR adds a blog post about the recent change in promtool made in https://github.com/prometheus/prometheus/pull/8940.

Thanks for your contribution.
I think there is a misunderstanding in this blog post. The "fullness" of the chunks is not related to the bytes/sample they take. This is a misleading shortcut.
I am wondering if we have to publish that kind of deep tsdb blockposts in our blog, considering the very high complexity involved.
Even the "reference" tsdb blog by codesome are not in prometheus.io.
It means that this blog post would be "the" reference when you would look for info about chunks etc, and it would be very very confusing for users.

To clarify my last comment: I strongly believe that:
1/ if there is such an issue in Prometheus, it is a bug that must be fixed. Users should never run the promtool tsdb commands unless they are TSDB developers.
2/ if we publish this on prometheus.io, users will start using those commands and come back with numbers / bug reports / etc that will be hard to tackle. It will also create  a lot of frustration because those commands are not optimized and not targeted at end users.
So I would rather have this published on a 3rd party platform than in our main website.


Revert "Update moment to 2.21.0 as previous version had vulnerability"

Reverts SharePoint/sp-dev-fx-webparts#444

.


404: NOT_FOUND

When i try to access the deploy project on `/api/spotify" i have 404 not found but my files is here hwhen i acess https://novatorem-iy3pwkfqq.vercel.app/ and when i usei https://novatorem-iy3pwkfqq.vercel.app/api/spotify dont work and i have other problem
.py on end spotify don work

Internal Server Error
The server encountered an internal error and was unable to complete your request. Either the server is overloaded or there is an error in the application.

Please follow the steps in the debugging section of the setup, and if it persists, share your logs here.

Just load, i try to view again, but do have generate logs, just 404

Did you follow the video guide to make sure you didn't miss any steps?
Any info from the build logs?

yes and yes, i have all information, the name of env variables is ok, i gonna try again tomorow

probably i used the wrong url, i try with this https://novatorem.cristuker.vercel.app/api/spotify.py and works fine. Thanks for help.


Add support for anonymous struct returns

Motivation is documented here:

https://github.com/golang/mock/issues/153#issuecomment-1462571305

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


In Dockerfile, remove requirements.txt after pip install

See https://github.com/Senzing/stream-loader/issues/241

See "Post mordem" on https://github.com/Senzing/stream-loader/issues/241


⚠ Processors has degraded performance

In 3878577, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 46 ms

Resolved: Processors performance has improved in d17ee64.


Named generic arguments

This change is

Current dependencies on/for this PR:

main

PR #3903   👈

This comment was auto-generated by Graphite.


Invalid syntax with python 3.9

$ python --version
Python 3.9.5
$ python -c "import PyRoxy"
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/home/ubuntu/.virtualenvs/botnet-dds/lib/python3.9/site-packages/PyRoxy/__init__.py", line 13, in <module>
    from PyRoxy import GeoIP, Tools
  File "/home/ubuntu/.virtualenvs/botnet-dds/lib/python3.9/site-packages/PyRoxy/Tools/__init__.py", line 18
    rand_char: Callable[[int], chr] = lambda length=16: chr(Random.rand_int(0, 1000)) for _ in range(length)
                                                                                ^
SyntaxError: invalid syntax

To be honest, it's difficult to understand what code is meant to be in that line. There is no usage of rand_char() across all sources.

Because of last commit in PyProxy: https://github.com/MHProDev/PyRoxy/commit/ffecf3fb67306172e6e0444e851093a059bf12c1

Made a pull. Still getting this error.

fixed in last commit


Backtesting issue on selling ?

Buying correct amount, but selling 0
2016-08-24 14:07:10 (INFO):     Stochastic confirms the Alligator Uptrend movement -> buy
2016-08-24 14:07:10 (INFO):     Profit simulator got advice to long     @ 2016-08-23 03:22:00, buying 1.11274036 BTC       (1.11274036)
and :
2016-08-24 14:07:10 (INFO):     Stochastic confirms the Alligator DownTrend movement -> sell
2016-08-24 14:07:10 (INFO):     Profit simulator got advice to short    @ 2016-08-23 04:22:00, selling 0 BTC<PHONE_NUMBER>3)

I think this is just a logging error introduced when I refactored logging in backtesting. Will take a look asap!


knife bootstrap - error installing git-daemon

Knife boostrap fails while installing git. Here is the error.
Appreciate any help in this regard.
Chef::Exceptions::Package
No version specified, and no candidate version available for git-daemon
Resource Declaration:
In /var/chef/cache/cookbooks/git/recipes/server.rb
package 'git-daemon'

@ctippur Apologies that you never received a response on this.  This appears to be a problem with a particular cookbook you are attempting to use rather than a bug in knife or chef-client.  If you are still experiencing this problem, many in the Chef community would be happy to help, particular in our IRC channel on freenode and the mailing list.  Is this still an issue for you?  If not, I'll close this bug out.

@ctippur As Steven noted above, this is more of an issue with the cookbook or recipe you are using and the Chef mailing list or IRC channel are more appropriate forums for help with that.


frogmouth: also patch xdg dependency in pyproject.toml

The build for frogmouth already contains a postUnpack patch that replaces the xdg dependency with xdg-base-dirs, but that's unfortunately not enough for the Nixpkgs build to succeed: The python dependency check hook relies on the specifications within pyproject.toml (or the resulting wheel files) to determine whether all dependencies are present and fails because it still checks for xdg unless the project declaration is also patched.
See Hydra build: https://hydra.nixos.org/build/246124305
Hydra log: https://hydra.nixos.org/build/246124305/nixlog/2
Relevant log excerpt:
OK
/build/source
Finished executing pypaBuildPhase
Running phase: pythonRuntimeDepsCheckHook
Executing pythonRuntimeDepsCheck
Checking runtime dependencies for frogmouth-0.9.1-py3-none-any.whl
  - xdg not installed

Description of changes
Added a second sed line that patches pyproject.toml accordingly.

Things done

Built on platform(s)

[x] x86_64-linux
[x] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages
[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage

[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

frogmouth
frogmouth.dist

Result of nixpkgs-review run on aarch64-linux 1

2 packages built:

frogmouth
frogmouth.dist

Result of nixpkgs-review pr 281901 run on x86_64-linux 1

2 packages built:

frogmouth
frogmouth.dist

these changes don't make sense to me.  the python packages xdg and xdg-base-dirs are separate packages and if the upstream project wants to use the outdated xdg package, so be it.
i can remove the postUnpack and revert xdg-base-dirs to xdg (and remove the line in the pythonRelaxDeps) and package  builds and runs (well, starts, i didn't test further)
cc @philiptaron who might have a better take.

Thank you for working to fix this build break, btw @mfrischknecht!

Result of nixpkgs-review pr 281901 run on x86_64-linux 1

2 packages built:

frogmouth
frogmouth.dist

Result of nixpkgs-review pr 281901 run on x86_64-darwin 1

2 packages built:

frogmouth
frogmouth.dist


Call to undefined function Illuminate\Testing\Constraints\array_key_first()

Laravel Version: 7.11.0
PHP Version: 7.2.24
Database Driver & Version: mysql 5.7.19

Description:
I am writing a unit test and checking in pivot table if has some keys and throws error for undefined function. That function is for php 7.3 but laravel 7 requirement is 7.2.5+ right?
Why trait is using functions that are for 7.3 php version?
I am doing anything wrong?
Steps To Reproduce:
Create a table and add some records.
Create a test add assertion like this.
$this->assertDatabaseHas('user_role', [
            'user_id'     => $user->id,
            'role_id' => $role->id,
        ]);

It would be helpful if you shared the exact error message you get, preferably the whole stack trace.

Hello @lupinitylabs ,
This is the strack trace
Error : Call to undefined function Illuminate\Testing\Constraints\array_key_first()
 C:\laragon\www\zygos-gov\vendor\laravel\framework\src\Illuminate\Testing\Constraints\HasInDatabase.php:81
 C:\laragon\www\zygos-gov\vendor\laravel\framework\src\Illuminate\Testing\Constraints\HasInDatabase.php:66
 C:\laragon\www\zygos-gov\vendor\laravel\framework\src\Illuminate\Foundation\Testing\Concerns\InteractsWithDatabase.php:46

under further investigation it looks that the current error occures when the values you are looking are not in the table and the data are empty or missing the looking ids

The array_key_first method was introduced in php 7.3, so that probably shouldn't be there in the first place.

Good find!

@lupinitylabs  Thanks,testing is crucial indeed

Actually this is tested, but the tests load the PHP 7.3 polyfill...

This is now fixed on 6.x and 7.x.


install point_cloud_transport

how to install point_cloud_transport so that other ros nodes can use it ?
when i use
catkin_make_isolated --install --install-space /opt/ros/kinetic

but i get errors:
Install the project...
-- Install configuration: "Release"
CMake Error at cmake_install.cmake:44 (file):
  file cannot create directory: /opt/ros/kinetic/include/draco/metadata.
  Maybe need administrative privileges.

Makefile:115: recipe for target 'install' failed
make: *** [install] Error 1
<== Failed to process package 'draco':
  Command '['make', 'install']' returned non-zero exit status 2

so what is the right way to install it ?
looking forward to your reply !

Isn't this just a problem of insufficient permission? You need root access to write into /opt/*
It can be used as any other catkin package. E.g., you can make it part of your catkin workspace to use it.


Clarify gotcha severities.

It seems helpful to formalize this distinction — in other words, it's based on the severity of the bad consequences, not whether something works or not. Sound good? Thoughts?

Maybe two cases, though YMMV:

Warning: Things break/don't work and you can't figure out why by searching StackOverflow, the AWS docs, or og-aws with simple, intuitive keywords
Caution: Things don't work, but the cause is in the docs

I realize this is not backwards compatible with the current marking scheme, but reflects how I find problems IRL. I don't have enough context with the project to want to go against a dissenting consensus, though.

Thanks for the idea @rjhintz - I'm not sure I see the value in connecting the scheme to availability of the information in other docs or on  other sites, though. I'm open to discussing further here or on Slack, though.
@jlevy the PR LGTM. 👍

Agree, @rjhintz those considerations matter, and are a key reason this guide has value — that it contains things you can't find on SO/AWS docs/Google.
However, this PR is about how we annotate things by severity. Ideally, a reader looking to find really serious gotchas that are critical to know should be able to skim/search to find those in this guide. Let's go with this, as it's our current practice.

I'm fine with this outcome.


integrate invenio 3.4.0a1

Changes:

adds support for python 3.7 and 3.8 (untested)
sets THEME_LOGO to that of InvenioRDM. This value is already set in invenio-app-rdm and would not be needed. However, for clarity to the user it is better to leave the variable in invenio.cfg.

Closes https://github.com/inveniosoftware/cookiecutter-invenio-rdm/issues/71
Closes https://github.com/inveniosoftware/cookiecutter-invenio-rdm/issues/64

Coverage remained the same at 0.0% when pulling f5408ad3c6236b078041f8b6b701ac8e155c5795 on ppanero:invenio_34 into c8b95fb8c95f1e4e90b2ce89777d6b495ece5464 on inveniosoftware:master.

Closed in favour of #74


AOF coming soon?

This page says AOF is coming soon, while this image (https://docs.microsoft.com/en-us/azure/azure-cache-for-redis/cache-how-to-premium-persistence#configure-aof-persistence) suggests that its in Preview.
It would good to reflect the accurate state of this feature

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

ID: 7d381e32-2963-1767-3763-905ae05eca82
Version Independent ID: 18e36037-e1ba-fd52-28e3-7438ec61db96
Content: Azure Cache for Redis FAQ
Content Source: articles/azure-cache-for-redis/cache-faq.md
Service: cache
GitHub Login: @yegu-ms
Microsoft Alias: yegu

@abhirockzz We are checking this and will get back to you.

@yegu-ms Please convey the correct state of the AOF.

@abhirockzz we updated the doc. It should be published today or tomorrow.
#please-close


Redesign CLI syntax of racetrack client

What's missing?

racetrack list command to see the running fatmen as it's shown on Dashboard

Things to improve:

For instance, racetrack delete WORKDIR RACETRACK_URL command.
You need to pass working directory to take fatman name and version out from a local manifest file. What if you don't have the manifest and you want to delete something as an admin. Explicit parameters --name, --version would be more reasonable.

Use dedicated library for building CLI applications:

argparse is terrible at building sub-commands tree
maybe use Typer or Click instead

racetrack completion bash and maybe zsh?

I think the best choice is to use Typer.
Few facts / opinions:

typer has 10.2 k stars on Github
click has 13.4 k stars on Github
Typer makes use of Click
Typer has "automatic completion for all shells" <- that's what we're looking for
the syntax of Typer is more concise, simpler than Click
Both has good documentation, but Typer's docs seems to be more friendly IMO.
Click has BSD-3-Clause license, while Typer is released under MIT license. That's funny as Typer is dependent of Click, so then you use both.
Typer seems to be more modern, eg. by having type annotations support
Typer has better support for nested SubCommands (we need that, eg. for racetrack plugin install ...)

Another idea: In some cases having positional argument RACETRACK_URL doesn't really make sense, it could be converted to optional parameter --remote RACETRACK_URL.
I'm thinking about recommending a simpler workflow so you can select your current environment in first place (just as you do with kubectl config set-context --current --namespace=racetrack):
racetrack config racetrack_url my-local-docker

and then call operations without passing redundant RACETRACK_URL each time:
racetrack list
racetrack logs fatman-A
racetrack delete fatman-A

This would be possible too:
racetrack list --remote=my-local-docker

Yeah that feels like a good idea. But we should maybe pressure-test that idea with some users. @nielshelsoe ?

@JosefAssadERST you choose to ping the guy that just has come back from maternity leave? I might think that there is a better person to come up with some useful insights

Have chatted with Niels and Veronica, consensus is that the idea of setting context so avoid typing URLs (or RT alias) every single time is good.

Good. I can make it backwards-compatible for the most common commands, so it won't be a breaking change, ie. racetrack deploy . WHERE or racetrack deploy . --remote=WHERE


Add a helm chart for kpack

It will be useful to have a helm chart for kpack deployments.

We would like to have a Helm chart for Kpack.

I've love to see an official helm chart as well. In my use case I'd love to see an option to allow/disallow specific components based on the values.yaml file. I would be willing to write the chart and make a PR to this repo, but would need help keeping it updated with the latest relase

We would like to have a Helm chart for Kpack.


app install doesn't give back token or profile info only when app uninstall works with web

did you see any error in the console?

when app is installed this is the console output : "2017-08-16 19:00:16.706038+0100 appname[504:191749] appId=*******"
nothing more

I've seen similar behaviour, in my case I had additional delegate method implemented (for Facebook support).
func application(_ app: UIApplication, open url: URL, options: [UIApplicationOpenURLOptionsKey : Any] = [:]) -> Bool

This one is called as a callback from LinkedIn app. The issue here, that options parameter doesn't contain any token. However, when I remove this method and proper method is called - everything is good.
func application(_ application: UIApplication, open url: URL, sourceApplication: String?, annotation: Any) -> Bool

@moazkh60 check out new readme, it's include a bit detail there.
:warning: for iOS 9 and above use this instead:
    func application(_ app: UIApplication, open url: URL, options: [UIApplicationOpenURLOptionsKey : Any] = [:]) -> Bool {
        // Linkedin sdk handle redirect
        if LinkedinSwiftHelper.shouldHandle(url) {
            return LinkedinSwiftHelper.application(app, open: url, sourceApplication: nil, annotation: nil)
        }

        return false
    }


Replace DateTime model fields with Date

The following model fields need to change from DateTimeField to DateField as they don't need the time.

[ ] Mentor.birthday
[ ] Guardian.birthday
[ ] Student.birthday
[ ] Equipment.aquisition_date (also fix spelling to acquisition_date)

Fixed in #820

Fixed in #820


Maintain children ref

Hello :)
I'm writing a simple tooltip that should be able to use any child as a reference component. I don't want to render any extra DOM nodes around my reference components. To do this, I am cloning the children and adding a ref tag.
Simplified example:
<Reference>
	{({ ref }) => React.cloneElement(children, { ref })}
</Reference>

However, it's possible the child already has a ref, and I don't want to override it. So I am doing something like this:
<Reference>
	{({ ref }) => React.cloneElement(children, {
		ref: (node) => {
			ref(node);
			children.ref(node);
		},
	})}
</Reference>

Unfortunately, react-popper doesn't seem to like this.  Here is a simplified example reproducing the error:
Reproduction demo
https://codesandbox.io/s/7yv72zmw51
Steps to reproduce the problem
Replace
<Reference>
  {({ ref }) => (

      Reference element

  )}
</Reference>

with
<Reference>
  {({ ref }) => (
    // -- Change is on the next line --

 ref(node)} style={referenceStyle}>
      Reference element

  )}
</Reference>

What is the expected behavior?
Ref is set and everything continues to work.
What went wrong?
Getting a "Maximum update depth exceeded" error.
Packages versions
<EMAIL_ADDRESS>Is this indeed an issue or is this the expected behaviour? Please let me know if I can provide any more information. Happy to help 👍

The problem I think is that you are defining a new function on every render, so it forces an update that goes into a loop. Try to define you combined ref setter outsie of your component.

Thanks for the quick response. That's ultimately what I ended up doing.
I guess it just seems a bit odd that it behaves this way since using an arrow function for a ref typically doesn't cause a loop :). I imagine it is just a result of the unique behaviour of the Reference component.

I'm open to suggestions to avoid this loop!

I too have been bitten by this. In my case, I'm trying to upgrade from 0.9.5 to 1.0.0. In previous versions I was able to use an arrow function, and I agree that using an arrow function for a ref feels very common.

Try to define you combined ref setter outsie of your component

Unfortunately, this doesn't seem to help in my case; I tried both moving it out to a public class field as well as to a free-floating function, but both continue to cause the infinite loop. In my case, I'm trying to combine react-popper's ref with my own, passed in as another prop.
Here's my "ideal" code (currently non-working):
const PopButtonStateless: React.SFC<PopButtonStatelessProps> ({ button, children, isOpen, onClick, setButtonRef, setPopperRef }) => (
  <Manager>
    <Reference>{({ ref: setReferenceRef }) => (
      button({
        popButtonProps: {
          onClick, ref: r => {
            setButtonRef(r);
            setReferenceRef(r); // this causes infinite re-render
          }
        },
      }))}</Reference>
    {isOpen && <PopButtonPopper setPopperRef={setPopperRef}>{children}</PopButtonPopper>}
  </Manager >
);

I'll keep poking around for a solution, but I'm excited to see the progress in #197 which might help nip the problem in the bud.

I ran into this too, pulling the ref prop value as a class method, or completely outside of it made no difference in my case, same as @shepmaster. Invoking ref as a method with the node seems to be the problem.
My workaround was to wrap another div inside and attach my own ref to that instead.

In my case I call the ref manually in the callback, as I pass it a DOM node which is not in the subtree of the Reference component. This has worked fine in previous versions of the module.
I looked into the code and found out that getReferenceRef always updates the state, regardless of the reference actually changing. Is this intended? Could there be a check if the refs are unequal or would it break anything?

Hi,
I have faced the same issue too, and seems like there is only one workaround (suggested by @ebemunk) - add one more div and attach ref to it, .
However this solution looks like a HACK, it adds a surplus node in the DOM for each popper.
I have also tried to add reference change check inside Manager.getReferenceRef (suggested by @janizde), however it did not help.

Great comment here: https://github.com/FezVrasta/react-popper/issues/140#issuecomment-397031037.
Define the function to handle your refs externally and memoize it.


The express application listen fails

Describe the bug:
Port assigned to express application works only single time. when application is run again the container says that no server is listening on the port
Link to the blitz that caused the error
https://stackblitz.com/edit/google-drive-connector?file=index.js
Steps to reproduce:

Go to index.js
Check app.listen(PORT)
Scroll down to '....'
Run command in shell: node index.js
No errors, server starts listening
Nothing on output. No server listening on port

Expected behavior:
Code should be running on the defined port
Version of webcontainer:
Hash: 70a1ff62a0f0131ac0df3c30faf487fef744ac90
Desktop (please complete the following information):
Browser name  = Chrome
Full version  = 91.0.4472.106
Major version = 91
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/91.0.4472.106 Safari/537.36
performance.memory = {
  "totalJSHeapSize": 85320413,
  "usedJSHeapSize": 79289465,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}

The same thing happened to me while using the node http server. It's not just one time, sometimes it will listen on the same port after restarting but once it starts failing you have to refresh the page to reboot the web container.

Thanks, this seems to have resolved my issue for now.  A bit inconvenient but will do for now.

Hm I think this is actually a bug, so I am reopening this issue because I am sure I ran into the same a few times. It seems that when the process is killed that the server is not properly killed and restarting it will fail then cause it checks if the port is taken or not. And if it's taken it won't spin up the server. This needs a bit more investigation from our side. Thanks for reporting this issue!

@gokuney We're having trouble reproducing this bug now, a related fix might have solved the issue. Are you still able to reproduce it?
cc @nishantSg

Sorry, I had not used stackblitz for quite a while. I tried to run a few old projects and now I am not getting that problem. It might have been fixed. I will update here in future if It comes up.

Hey @jrvidal ,
I no longer get the error now. You're right, some related fix might have solved this. :smiley:
Let me close this bug for now. Thanks

Seems to me, this issue is present again (or still)

I open url https://stackblitz.com/edit/google-drive-connector?file=index.js (firefox dev. edition 109.0b3, or googe chrome 108)
in the vscode terminal type node index.js the in-editor-browser opens, its fine, BUT
click on 🔌 on the left (ports in use) and ... in chrome works (most of the time at least) and in ff like never.

i really not understand, how the magic works behind this stuff, but i strongly feel its related with the tracking protection.
Sry if its not appropriate, to comment on a closed issue, i just thought it might useful if written down


Add Lolivault

Closes #1588
Checklist:

[x] Updated extVersionCode value in build.gradle for individual extensions
[ ] Updated overrideVersionCode or baseVersionCode as needed for all multisrc extensions
[x] Referenced all related issues in the PR body (e.g. "Closes #xyz")
[x] Added the isNsfw = true flag in build.gradle when appropriate
[ ] Have not changed source names
[ ] Have explicitly kept the id if a source's name or language were changed
[x] Have tested the modifications by compiling and running the extension through Android Studio
[x] Have removed web_hi_res_512.png when adding a new extension

Site uses FoolSlide theme btw


[Feature Request] table component filter multilevel support

Existing Component
Yes
Component Name
Table
Description
请问table组件filter是否支持多级？

Sorry, we have no plan to support this.