OrbitDB does not work with Svelte/Vite

I'm trying to build a Svelte browser app. I get ReferenceError: process is not defined. This appears to be caused by polyfill issues and/or because ipfs-core is no longer functional.
In contrast, Helia works great, but OrbitDB doesn't work with Helia because it seems to still be dependent on ipfs-core.
When I updated my vite config with the polyfills, the process is not defined error was gone, but other errors appeared related to ipfs-core not having a default export.
Anyway, there are so many interconnected errors that I'm not sure what to do. I've been studying OrbitDB for the past several days and I was excited to start working with it, but all these errors are killing my enthusiasm.
Can somebody please provide some concrete instructions on how to configure Svelte to work with either Helia (preferrable) or ipfs-core?
Thank you.

I have only problems with Svelte4! But you can have a look here:
https://github.com/silkroadnomad/ipfs-vite-svelte-kit
I am working on Svelte3 right now.
OrbitDb and IPFS must be bundeled with esbuild see package.json https://github.com/DougAnderson444/ipfs-vite-svelte-kit/blob/master/package.json#L12C18-L12C25

Thank you @silkroadnomad. Yes, I previously saw @douganderson444 's project and I can see that your project works on Svelte 3. Unfortunately, my project is already implemented with the latest version of Svelte 4 and Sveltekit and it's not possible for me to downgrade to an old version of Svelte for many reasons.
I tried to migrate your project to Svelte 4 using @Rich-Harris 's migration guide, but after the first step (npx svelte-migrate routes), I get the following error in the browser:
Not found: /
Error: Not found: /
    at resolve (file:///home/julia/practice/orbit/doug/.svelte-kit/runtime/server/index.js:3360:14)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async respond (file:///home/julia/practice/orbit/doug/.svelte-kit/runtime/server/index.js:3253:20)
    at async file:///home/julia/practice/orbit/doug/node_modules/@sveltejs/kit/dist/chunks/index.js:295:24

Does anybody have any ideas about how to make Orbit/IPFS cleanly work with the latest version of Svelte and SvelteKit?

Yes, this was my experience too. I also used the migration guide and tried many other things.  Was fighting full two days to make it work but then had to give up to save my life and the week;)
Then, I received another tip which you could try: https://github.com/orbitdb/orbitdb-examples/tree/main/vue-vite
This is a vue example with vite! I found it interesting because they are using vite too! (with latest Orbit) You could try this if you fancy to have look on it.

Thank you @JuliaBonita for going more profound here. I'd even say it's not an OrbitDb issue per se. I guess (I didn't try) the issue already occurs when dealing with IPFS alone in Svelte4. Maybe Helia-IPFS even works out of the box with Svelte4? I haven't tried either.
So there would be a possibility to check what IPFS community is doing with Svelte and vice verca. There should be some folks working on it. Maybe @DougAnderson444 has an opinion on IPFS with Svelte4.

@silkroadnomad Yes, Helia works fine with Svelte 4.x. That was the first thing I tried and described in my initial post. In fact, I would much rather use Helia. But OrbitDB still has a dependency on ipfs-core; so there are errors when I try to replace ipfs-core with Helia.
I would rather never use ipfs-core, but I don't know how long it will take the OrbitDB team to make it compatible with Helia. So using ipfs-core would have been better than nothing at all. But at this point, it seems we're still stuck with nothing at all for Svelte 4.x.
I agree that the IPFS team could potentially know more about why ipfs-core throws the errors I described above, but they are focused on Helia now and they don't provide any substantial support for the old ipfs-core code anymore.
In contrast, Helia works fine with Svelte 4.x; the problem is OrbitDB is still dependent upon ipfs-core. That means the IPFS team can't really help us and only the OrbitDB team can help fix the Helia compatibility problems.
Anybody have any other ideas?

@JuliaBonita It somewhat works for me now with the helia branch of Orbit-DB. Don't import ipfs-core only import helia! E.g. do like this:
npm create svelte@latest my-app & cd my-app
npm install helia
npm install "https://github.com/orbitdb/orbitdb.git#helia" --save

Then put this into your

Thank you @silkroadnomad . I remember getting to this point in some of my previous testing with Helia and the OrbitDB/helia branch, but I stopped focusing on Helia because the browser errors still prevent it from working in most of my tests:
Uncaught (in promise) TypeError: Failed to fetch dynamically imported module:

When I was investigating that error previously, I saw something that made me think it was caused because OrbitDB/Helia still had a dependency on ipfs-core. However, I just looked at the Node terminal errors, which are totally different:
PublishError.InsufficientPeers at GossipSub.publish

So maybe the browser error is really just a symptom of the insufficient peers error. That seems like it should be a relatively easy problem to fix by adding a list of bootstrap peers somewhere in the Helia/OrbitDB configs.
I don't have enough experience yet with these two libraries to know exactly how to do that, but I will perform some more tests later today.

You can get rid of this error very easily by adding a config to createHelia(config)
Have a lot of fun! :)
import { bootstrap } from '@libp2p/bootstrap';
import { pubsubPeerDiscovery } from '@libp2p/pubsub-peer-discovery';
import { circuitRelayTransport, circuitRelayServer } from 'libp2p/circuit-relay';
import { webRTC, webRTCDirect } from '@libp2p/webrtc';
import { webTransport } from '@libp2p/webtransport';
import { webSockets } from '@libp2p/websockets';
import { all } from '@libp2p/websockets/filters';
import { identifyService } from 'libp2p/identify';
import { autoNATService } from 'libp2p/autonat';
import { gossipsub } from '@chainsafe/libp2p-gossipsub';
import { kadDHT } from '@libp2p/kad-dht';
import { ipnsSelector } from 'ipns/selector';
import { ipnsValidator } from 'ipns/validator';

const multiaddrs = [ //add your own WebRTC Stars Servers here too!
			'/dnsaddr/bootstrap.libp2p.io/p2p/QmNnooDu7bfjPFoTZYxMNLWUQJyrVwtbZg5gBMjTezGAJN',
			'/dnsaddr/bootstrap.libp2p.io/p2p/QmQCU2EcMqAqQPR2i9bChDtGNJchTbq5TbXJJ16u19uLTa',
			'/dnsaddr/bootstrap.libp2p.io/p2p/QmbLHAnMoJPWSCR5Zhtx6BHJX9KiKNN6tpvbUcqanj75Nb',
			'/dnsaddr/bootstrap.libp2p.io/p2p/QmcZf59bWwK5XFi76CZX8cbJ4BhTzzA3gU1ZjYZcYW3dwt'
		];
		const bootstrapConfig = { list: multiaddrs };
		const config = {
			libp2p: {
				// https://github.com/ipfs/helia/blob/main/packages/helia/src/utils/libp2p-defaults.browser.ts#L27
				addresses: {
					listen: ['/webrtc', '/wss', '/ws']
				},
				transports: [
					webSockets({ filter: all }),
					webRTC(),
					webRTCDirect(),
					webTransport(),
					// https://github.com/libp2p/js-libp2p-websockets#libp2p-usage-example
					circuitRelayTransport({ discoverRelays: 3 })
				],
				//streamMuxers: [mplex()],
				// streamMuxers: [yamux()],
				peerDiscovery: [bootstrap(bootstrapConfig), pubsubPeerDiscovery()],
				services: {
					identify: identifyService(),
					autoNAT: autoNATService(),
					//pubsub: gossipsub({allowPublishToZeroPeers: true, emitSelf: false, canRelayMessage: true}),
					pubsub: gossipsub({
						allowPublishToZeroPeers: true,
						emitSelf: true,
						canRelayMessage: true
					}),
					dht: kadDHT({
						clientMode: true,
						validators: { ipns: ipnsValidator },
						selectors: { ipns: ipnsSelector }
					})
				}
				// https://github.com/libp2p/js-libp2p/blob/master/doc/CONFIGURATION.md#configuring-connection-gater
				//connectionGater: {denyDialMultiaddr: async (...args) => false},
			}
		};

@silkroadnomad Ah, I see now. Your example is fantastic! Thank you.
Now I see that the only way to use ODB effectively is to have a strong grasp of how to configure at least 3 different core libraries (libp2p, Helia and ODB) and many of their modules. It's not immediately obvious from the existing docs that we need to configure all these different pieces. And the examples in the docs are not really functional or correct right now because they still include configs and parameters that are only applicable for the older libraries and/or not applicable to the browser. This makes the onboarding process much more difficult.
I understand it's an open source project and resources are limited and the team is working as fast as possible to migrate everything to Helia. So I'm not complaining; I'm just pointing out an area for potential improvement to make it easier for others to get started.
In that spirit, your comments in this thread have helped me see exactly what I need to study and configure to make all these pieces work together. I think the project team should create a simple "Getting Started with Helia" page (similar to the old page) and integrate your examples into the project docs.
I appreciate everybody here (ODB team and community) for helping the world create great decentralized apps!

@JuliaBonita I accidentally found the reason for the error: https://github.com/ipfs/helia-ipns/issues/59


There are no black bars (gutters) visible on the screen (HTML/GWT)

Issue details
The game does not construct gutters (black bars) when resizing browser window (applied ExtendViewport with world from 720 480 to 864 552). gif: http://imgur.com/a/VkbtX
The proportions are fine as long as window size is within safezone. The gutters should appear when screen size is out of safezone. The game stretches the view instead (same code works fine on desktop and android).
Reproduction steps/code
Any game screen with this HtmlLauncher ...
public class HtmlLauncher extends GwtApplication {
    static final int WIDTH = 864;
    static final int HEIGHT = 552;
    static HtmlLauncher instance;

    @Override
    public GwtApplicationConfiguration getConfig() {
        GwtApplicationConfiguration config = new GwtApplicationConfiguration(Window.getClientWidth(), Window.getClientHeight());

        Element element = Document.get().getElementById("embed-html");
        VerticalPanel panel = new VerticalPanel();
        panel.setWidth("100%");
        panel.setHeight("100%");
        panel.setHorizontalAlignment(HasHorizontalAlignment.ALIGN_CENTER);
        panel.setVerticalAlignment(HasVerticalAlignment.ALIGN_MIDDLE);
        element.appendChild(panel.getElement());
        config.rootPanel = panel;

        return config;
    }

    @Override
    public ApplicationListener getApplicationListener() {
        instance = this;
        setLogLevel(LOG_NONE);
        setLoadingListener(new LoadingListener() {
            @Override
            public void beforeSetup() {

            }

            @Override
            public void afterSetup() {
                scaleCanvas();
                setupResizeHook();
            }
        });
        return new HiddenObjectGame();
    }

    void scaleCanvas() {
        Element element = Document.get().getElementById("embed-html");

        int innerWidth = getWindowInnerWidth();
        int innerHeight = getWindowInnerHeight();

        NodeList<Element> nl = element.getElementsByTagName("canvas");

        if (nl != null && nl.getLength() > 0) {
            Element canvas = nl.getItem(0);
            canvas.setAttribute("width", "" + innerWidth + "px");
            canvas.setAttribute("height", "" + innerHeight + "px");
            canvas.getStyle().setWidth(innerWidth, Style.Unit.PX);
            canvas.getStyle().setHeight(innerHeight, Style.Unit.PX);
            canvas.getStyle().setTop(0, Style.Unit.PX);
            canvas.getStyle().setLeft(0, Style.Unit.PX);
            canvas.getStyle().setPosition(Style.Position.ABSOLUTE);
        }
    }

    native int getWindowInnerWidth() /*-{
        return $wnd.innerWidth;
    }-*/;

    native int getWindowInnerHeight() /*-{
        return $wnd.innerHeight;
    }-*/;

    native void setupResizeHook() /*-{
        var htmlLauncher_onWindowResize = $entry(@com.imaginalis.hog.client.HtmlLauncher::handleResize());
        $wnd.addEventListener('resize', htmlLauncher_onWindowResize, false);
    }-*/;

    public static void handleResize() {
        instance.scaleCanvas();
    }

	@Override
	public ApplicationListener createApplicationListener() {
		return getApplicationListener();
	}
}

and with this viewport in core:
viewport = new ExtendViewport(720, 480, 864, 552);
gameStage = new Stage(viewport);

And again, this solution works fine on other platforms. I realize I could do something wrong. If so, point it out bluntly.
EDIT
I'm putting Game and Screen class for your convenience. Just copy and paste these two into the core project:
Game
public class ViewportTestGame extends Game {

	public SpriteBatch spriteBatch;

    @Override
    public void create() {
        spriteBatch = new SpriteBatch();
		this.setScreen(new ViewportTestScreen(this));
    }

    @Override
    public void render() {
        super.render();
    }

    @Override
    public void dispose() {
        spriteBatch.dispose();
    }
}

Screen
public class ViewportTestScreen implements Screen {

	private Game game;
	private Stage gameStage;
	private Image backgroundTestImage;
	private SpriteBatch spriteBatch;

	public ViewportTestScreen(final ViewportTestGame game) {
		this.game = game;
		this.backgroundTestImage = new Image(new Texture(Gdx.files.internal("background_test3.png")));
		this.gameStage = new Stage(new ExtendViewport(720, 480, 864, 552));
		this.gameStage.addActor(backgroundTestImage);
		spriteBatch = new SpriteBatch();
		Gdx.input.setInputProcessor(gameStage);
		// if you need virtual 0,0 point...
//		float virtualZeroX = (864 - gameStage.getViewport().getWorldWidth()) / 2f;
//		float virtualZeroY = (552 - gameStage.getViewport().getWorldHeight()) / 2f;
	}

	@Override
	public void show() {
		gameStage.getViewport().getCamera().position.set(864 / 2f, 552 / 2f, 0);
	}

	@Override
	public void render(float delta) {
		Gdx.gl.glClearColor(0, 0, 0, 1);
		Gdx.gl.glClear(GL20.GL_COLOR_BUFFER_BIT);
		gameStage.act();
		gameStage.draw();
	}

	@Override
	public void resize(int width, int height) {
		gameStage.getViewport().update(width, height);
		gameStage.getViewport().getCamera().position.set(864 / 2f, 552 / 2f, 0);
	}

	@Override
	public void pause() {
	}

	@Override
	public void resume() {
	}

	@Override
	public void hide() {
	}

	@Override
	public void dispose() {
	}
}

In desktop project:
public static void main (String[] arg) {
		LwjglApplicationConfiguration config = new LwjglApplicationConfiguration();
				int width = 1366;
				int height = 768;
                config.width = width;
                config.height = height;
                config.resizable = true;
				new LwjglApplication(new ViewportTestGame(), config);

It's dangerous to go alone. Take this:

... and put it into the assets folder
Version of LibGDX and/or relevant dependencies
1.9.6, no additional dependencies
Please select the affected platforms

[ ] Android
[ ] iOS (robovm)
[ ] iOS (MOE)
[x] HTML/GWT
[ ] Windows
[ ] Linux
[ ] MacOS

This seems to be a problem with your resizing code. Have you checked that resize is being called? If not, then debug the size of the canvas.

Yes, here is a working resizing code.
    private GwtApplicationConfiguration cfg;

    @Override
    public GwtApplicationConfiguration getConfig() {
        // reduce by 20 to avoid scrolling in iframes
        int w = Window.getClientWidth() - 20;
        int h = Window.getClientHeight() - 20;
        cfg = new GwtApplicationConfiguration(w, h);
        Window.enableScrolling(false);
        Window.setMargin("0");
        Window.addResizeHandler(new ResizeListener());
        cfg.preferFlash = false;
        return cfg;
    }

    class ResizeListener implements ResizeHandler {
        @Override
        public void onResize(ResizeEvent event) {
            int width = event.getWidth();
            int height = event.getHeight();
            getApplicationListener().resize(width, height);
            Gdx.graphics.setWindowedMode(width, height);
            Gdx.gl.glViewport(0, 0, width, height);

            Window.scrollTo((cfg.width - width) / 2, (cfg.height - height) / 2);

        }
    }

@Tom-Ski if you assure me to pull it into the master and tell me where the template is, I would be glad to contribute this into the project.

This would make all games take up the whole window size (- padding), no?

Exactly, same behaviour than Android and Desktop.
Normally it is not the browser window, because HTML games are living in an iframe by the hosing site in most cases. So the game would fill the whole iframe, yes.

Sounds good.  Template file is here, feel free to contribute https://github.com/libgdx/libgdx/blob/master/extensions/gdx-setup/src/com/badlogic/gdx/setup/resources/html/src/HtmlLauncher

Just to drop a note on the current state. Despite I used the code above for all my games, I discovered when I wanted to contribute that it does not work correctly when the game is resized to a smaller size than the initial size: Clicks are not registered at the correct position anymore. I never noticed that because normally all users resize the game to a size bigger than initial.
I have another code here that works like charm on this, but it has the disadvantage that the preloader logo is not centered on start. I am working on this.

Ok, I merged the best of both worlds. :-)


Add argparse option to restart-zerver for --fill-cache

Currently, restart-server has this slow-but-useful step of filling the memcached caches before switching over to the new version.  This is great in the case of it being called from upgrade-zulip, but for most other applications (e.g. triggering a quick restart because the server is down and you're live-editing code, or something), you really just want it to restart the server quickly.
We should make that behavior sit being a --fill-cache option (default off, but passed by upgrade-zulip code and maybe some other callers).
@aero31aero @hackerkid this is probably a good project for one of your near-term queues.

claim

@GrugLife The syntax is @zulipbot claim

@zulipbot claim


MONDO:0011836 & MONDO:0006465 Thyroid gland oncocytic follicular carcinoma [Merge]

Mondo term (ID and Label)
MONDO:0006465 Thyroid gland oncocytic follicular carcinoma
Reason for deprecation
Many overlapping synonyms between these 2 terms.
NCI (http://purl.obolibrary.org/obo/NCIT_C4946) has
Legacy Concept Name: Thyroid_Hurthle_Cell_Carcinoma
(Up to you all which sources' name you will use as Mondo's primary name)
Term to be merged with
thyroid Hurthle cell carcinoma  MONDO:0011836
Your nano-attribution (ORCID)
If you don't have an ORCID, you can sign up for one here

Dear @kanems,
Thank you for reaching out to the Mondo Disease Ontology, part of the Monarch Initiative!
We have received your ticket and will review it as soon as possible.
If you have any additional information or updates relevant to your inquiry, please add them in a comment on this ticket. Your input is valuable in helping us understand and resolve the issue effectively.
Thank you for your patience and for your contributions to Mondo!
Sincerely,
The Mondo Team

I agree these terms should be merged based on the original post highlighting the overlap in synonyms.


False positive for touch cop

Expected behavior
Not to flag this as an error
Actual behavior
Avoid using touch because it skips validations.
    FileUtils.touch logfile

Steps to reproduce the problem
Use the above FileUtils command
RuboCop version
$ rubocop -V
0.48.1 (using Parser <IP_ADDRESS>, running on ruby 2.4.1 x86_64-darwin16)

Ops, that's one silly mistake we made.

😄 easy mistake to make!

True, seems this cop treats a bunch of methods in more or less the same manner. It will require a bit of work to split those to class methods and instance methods.


Restore can complete but fail writing to etcd with request too large

I created a backup for a specific namespace, and restored it into a rebuilt cluster. My particular cluster may contain a larger than normal amount of cluster objects. (cluster role bindings, etc) I saw quite a bit of throttling request messages in the log.
ark-restore.log
The restore completed, my namespace is restored (though failed with service account problems probably due to the new cluster, I will dig into that next), but fails to write status because etcd finds the request too large. My restore then is hung in progress.
I0929 12:56:11.805636       1 restore_controller.go:255] restore heptio-ark/test1-11-20170929125444 completed
I0929 12:56:11.805661       1 restore_controller.go:258] updating restore heptio-ark/test1-11-20170929125444 final status
I0929 12:56:11.991224       1 restore_controller.go:260] error updating restore heptio-ark/test1-11-20170929125444 final status: etcdserver: request is too large

$ ark restore get
NAME                      BACKUP     STATUS       WARNINGS   ERRORS    CREATED                         SELECTOR
test1-11-20170929125444   test1-11   InProgress   0          0         2017-09-29 12:54:45 +0000 UTC   <none>

Environment was OpenShift 3.7 based on Kubernetes 1.7. Etcd 3.2.5.

Can you grab the log for the restore itself (ark restore logs <restore name>)?
I'm thinking there are a couple of issues here:

We need to make the qps and burst configurable for the client the ark server uses
We may need to rethink how/where we store warnings and errors for a restore. I'm guessing that you had a ton of warnings and errors such that the overall Restore object was over the etcd raft message size limit (1.5MB).

I'm going to create a separate issue for 1 and we'll keep this to track 2

Adding restore log.
restore.log
Definitely looks like some interplay here with reloading all that cluster data, in this case it was an entirely new cluster and things are not entirely happy.

@dgoodwin are these really large lists?
If so, we should dbl-check where clayton's paging work is at.

@timothysc the problem is we store the "lists" as structured data within restore.status

I'm thinking of some ways around this:

Store only # of warnings & # of errors in restore.status
Users can download the restore log if they want to see warnings & errors (they'd have to sift through it), OR
We could store the structured data for warnings & errors in object storage and make it easy to combine that information with what's in etcd; e.g., ark restore get --details could grab the warnings & errors and display them.

Here's what I have so far:
$ ark restore describe b2-20171101135335
Name:         b2-20171101135335
Namespace:    heptio-ark
Labels:       <none>
Annotations:  <none>

Backup:  b2

Namespaces:
  Included:  *
  Excluded:  <none>

Resources:
  Included:        *
  Excluded:        nodes
  Cluster-scoped:  auto

Namespace mappings:  <none>

Label selector:  <none>

Restore PVs:  true

Phase:  Completed

Validation errors:  <none>

Warnings:
  Ark:        <none>
  Cluster:    <none>
  Namespaces:
    default:      services "kubernetes" already exists
    heptio-ark:   services "minio" already exists
                  services "some-svc" already exists
    kube-system:  services "kube-dns" already exists

Errors:
  Ark:        <none>
  Cluster:    <none>
  Namespaces: <none>

WDYT?

@ncdc The format here looks good to me. I assume you're using option 3 of your previous comment for storing the data? That one seems like the best UX, as opposed to just throwing a potentially huge log at the user.

Really nice functionality and output, how would this work around the etcd request size issue?

The warnings and errors are stored in a gzipped file in object storage and
retrieved on the fly
On Wed, Nov 1, 2017 at 2:08 PM Devan Goodwin<EMAIL_ADDRESS>wrote:

Really nice functionality and output, how would this work around the etcd
request size issue?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/heptio/ark/issues/108#issuecomment-341190963, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAABYjE-weEDoS7m9le29VPgUDERoNm4ks5syLOSgaJpZM4Poqni
.

Love it, seeing them broken out per ns/cluster/ark is fantastic. The describe subcommand threw me for a loop but forgot you can do whatever you want as it's not kubectl directly.

@nrb yes this is option 3, although using a new describe subcommand.


Implement auth_history_syncer

Signed-off-by: Ofer Levi<EMAIL_ADDRESS>

also as a general comment for all pom.xml - they need to be updated to 1.11.4-SNAPSHOT

Squashed everything into a single commit


Add FP16 NaN/infinity handling flags

Adds 2 flags for more control over special floating point values when converting to FP16.  May be useful for applications that depend on NaN/infinity for special handling, like using NaN as a transparency sentinel.
TEX_FILTER_FLOAT_SATURATE_TO_INF saturates out-of-range values to infinity, including existing infinities.  TEX_FILTER_FLOAT_KEEP_NANS keeps NaNs instead of forcing them to zero.
Default behavior is convert NaN to zero and saturate to max range.

I'd have implemented these as part of the Convert special-case path rather than LoadScanline/StoreScanline since it's specific to conversion codepaths.
Since it's specific to FP32 to FP16 conversions, maybe name the flags: TEX_FILTER_FLOAT16_*

I totally get what you are going for here, I'm just wondering about some other impacts this might have elsewhere. I'll think about it a bit more...

Revisiting this issue part of the problem I think is the functionality offered in CVT16/F16C. These instructions let you control rounding, but don't have any ability to control NANs and/or DENORMS. I of course can implement it in software, but it will slow it down a bit.
As such, I think the default behavior needs to be "use NANs and DENORMs" and provide explicit flags for flushing DENORMs (which would need to be implemented for both F16 and F32 to make sense) and preferring INFs to specials for conversions.
I have some DirectXMath bugs in the handling of NANs. and DENORMS to deal with first, and when I wrap those up I'll revisit this PR.


gw cannot start

(base) [zsun@hopper2 ~]$ gw
Traceback (most recent call last):
File "/home/zsun/anaconda3/bin/gw", line 5, in
from pygeoweaver.main import geoweaver
File "/home/zsun/anaconda3/lib/python3.8/site-packages/pygeoweaver/init.py", line 5, in
setup_logging()
File "/home/zsun/anaconda3/lib/python3.8/site-packages/pygeoweaver/log_config.py", line 29, in setup_logging
logging.config.fileConfig(config_file)
AttributeError: module 'logging' has no attribute 'config'

Fixed. Need to import logging.config


fix mysql TinyInt serialization

fixes #91

Thanks for writing this fix and including a test. I would like to simplify the test in such a way that it does not require the additional table setup.

Fixed :)


cite more work in Understudied Langauges

Michael Hahn
Lars Hellan Ga
Berthold Crysmann, Hausa (?), Fox
Ackermann Moro
more.

@crysmann ??


package.json: Update react-dom package dependency

Check for any outstanding update of point version of any outstanding NPM package dependency

[ ] npm-update

npm-update in progress on 1-cockpit-11.
Log: https://<IP_ADDRESS>:8493/logs/npm-update-813-20191026-131111/

npm-update in progress on 1-ci-srv-04.
Log: https://<IP_ADDRESS>:8493/logs/npm-update-813-20191026-131111/

npm-update react-dom done: https://github.com/cockpituous/welder-web/commits/npm-update-react-dom-20191026-131212

npm-update @patternfly/react-icons done: https://github.com/cockpituous/welder-web/commits/npm-update-@patternfly/react-icons-20191026-131212

Please manually check features related to these files:
main.js:import ReactDOM from "react-dom";
package.json:    "react-dom": "16.11.0",

Codecov Report

Merging #813 into master will decrease coverage by 0.14%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #813      +/-   ##
==========================================
- Coverage   78.67%   78.52%   -0.15%
==========================================
  Files          67       67
  Lines        3372     3372
  Branches      708      708
==========================================
- Hits         2653     2648       -5
- Misses        719      724       +5

Impacted Files
Coverage Δ

components/Form/Password.js
85.71% <0%> (-4.77%)
:arrow_down:

core/selectors.js
92.96% <0%> (-0.79%)
:arrow_down:

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 d422c37...1811508. Read the comment docs.


GEODE-117 - gfsh put ignores --skip-if-exists flag

Fixed test, which erroneously reported that the --skip-if-exists flag is honored.
Implemented check for flag within the put operation.

+1. I'll pull this in.


Implements linting and warnings

The goal of Kataw is to be as fast as possible so the JS linting will be inlined with the parser code.
It's still WIP
let x: any;

{ allowTypes: true, lint: { noAny: true } }

Outputs Type declaration of 'any' loses type-safety. Consider replacing it with a more precise type
The linter diagnostics will have a different color in the CLI and point directly to the issue.
The parser automatically tries to auto-fix if possible. No option for it.
Auto-fix example:
// Input
kataw.parse('a=>b',  { lint: { arrowParens: true }}

// Output

// Parentheses are prohibited around the parameter in this single parameter arrow function
(a)=>b;

@aladdin-add Thoughts?

@aladdin-add This is how it can be done. This example is all low-level and gives complete control
import { DiagnosticKind } from 'kataw'

kataw.parse(
  'eval()',
  { lint: { noEval: true }, warnings: true },
  function (source, kind, message, start, end) {
    if (kind & DiagnosticKind.Linter) {
      // ...
    }
    if (kind & DiagnosticKind.Warnings) {
      // ...
		}
    if (kind & DiagnosticKind.Errors) {
      // ...
    }
  },
);

@aladdin-add  This is how it will looks like for the -end user. The colors change depends on the diagnostics.

Closing this now. The remaining rules will be implemented slowly and needs a series of bitwise masks, and some lint rules can be moved to printer or is basically parser errors.


[Tool]Analytics reporting state is not updated on user branch

jonahwilliams-macbookpro2:flutter_tools jonahwilliams$ flutter config --no-analytics
Analytics reporting disabled.

You may need to restart any open editors for them to read new settings.
jonahwilliams-macbookpro2:flutter_tools jonahwilliams$ flutter config
Configure Flutter settings.

To remove a setting, configure it to an empty string.

The Flutter tool anonymously reports feature usage statistics and basic crash reports to help improve Flutter tools over time. See Google's privacy policy: https://www.google.com/intl/en/policies/privacy/

Usage: flutter config [arguments]
-h, --help                                    Print this usage information.
    --[no-]analytics                          Enable or disable reporting anonymously tool usage statistics and crash reports.
    --clear-ios-signing-cert                  Clear the saved development certificate choice used to sign apps for iOS device deployment.
    --android-sdk                             The Android SDK directory.
    --android-studio-dir                      The Android Studio install directory.
    --build-dir=<out/>                        The relative path to override a projects build directory
    --[no-]enable-web                         Enable or disable Flutter for web. This setting will take effect on the master and dev channels.
    --[no-]enable-linux-desktop               Enable or disable Flutter for desktop on Linux. This setting will take effect on the master channel.
    --[no-]enable-macos-desktop               Enable or disable Flutter for desktop on macOS. This setting will take effect on the master channel.
    --[no-]enable-windows-desktop             Enable or disable Flutter for desktop on Windows. This setting will take effect on the master channel.
    --[no-]enable-build-plugin-as-aar         Enable or disable Build plugins independently as AARs in app projects. This setting will take effect on the master channel.
    --[no-]enable-android-embedding-v2        Enable or disable flutter create generates projects using the Android embedding V2. This setting will take effect on the master channel.
    --[no-]enable-web-incremental-compiler    Enable or disable Enable the incremental compiler for web builds. This setting will take effect on the master and dev channels.
    --clear-features                          Remove all configured features and restore them to the default values.

Run "flutter help" to see global options.

Settings:
  flutter-web: true
  flutter-linux-desktop: true
  enable-web: true
  enable-macos-desktop: true
  ios-signing-cert: iPhone Developer: Jonah Williams (87UWLUA5U2)
  enable-web-incremental-compiler: false

Analytics reporting is currently enabled.

Afterwards:
{
  "flutter-web": true,
  "flutter-linux-desktop": true,
  "enable-web": true,
  "enable-macos-desktop": true,
  "ios-signing-cert": "iPhone Developer: Jonah Williams (....)",
  "enable-web-incremental-compiler": false
}

False alarm! We have disabled analytics reporting for user branches, but as a result of this the flag does not get updated until you switch to a real branch. We should probably update this to still perist the setting, but it isn't as severe.

Updating the analytics flag on a user branch is a bit tricky to support without punching holes in package:usage. We should instead consider an error message like "Not updating analytics settings because you are on a local branch. Switch to master/dev/beta/stable and try again".

how do i change  enable-windows-desktop: true
enable-web: true to false

Fixed via https://github.com/flutter/flutter/pull/71728


Why this program cannot in my mac book pro with 2T ram and M2 pro CPU ?

Why this program cannot in my mac book pro with 2T ram and M2 pro CPU ?

javafx is too old to support m2 chip


Revert enabling image bitmap

This PR reverts #4584.
After doing some testing it seems ImageBitmap is still not well supported across browsers. Details have been left in the code for next time.
It might be worth investigating more closely which browsers have concrete support for ImageBitmap and start selectively enabling the interface on those browsers.

This PR removes support for WebGPU, which it should not.

This PR removes support for WebGPU, which it should not.

Webgp.. what? :D Thanks - fixed!


brew link is wrong

Hi when you try to install via homebrew via
brew install --cask https://raw.githubusercontent.com/wkaisertexas/ScreenTimeLapse/main/screentimelapse.rb

it will try to download this zip file (from github releases)
https://github.com/wkaisertexas/ScreenTimeLapse/releases/download/v1.0.0/ScreenTimeLapse.zip

but the actual link (the name of zip file itself is different)
https://github.com/wkaisertexas/ScreenTimeLapse/releases/download/v1.0.0/TimeLapze.zip

Thanks for submitting the PR for this. Even though it is unpublished, someone claimed the name ScreenTimeLapse for the Apple app store which is annoying. Thanks for the PR!


Find widget doesn't respect selection column for multi-line selection

Find scope respects column only in single line selection, but not in multi-line selection. It was designed to be like this in e1bb3ad6f3950e9a1a13fb988ead768f91b783ff: https://github.com/microsoft/vscode/blob/e1bb3ad6f3950e9a1a13fb988ead768f91b783ff/src/vs/editor/contrib/find/common/findModel.ts#L159-L162
Steps to reproduce:

Have the following text in your editor:abc
abc
abc
abc abc
abc

Select from 2nd abc to 4th abc
Enable find widget, possibly by pressing Ctrlf
Enable find in selection, possibly by pressing Altl
Enter abc

Notice that it highlights 2nd to 5th abc, instead of 2nd to 4th:

Does this issue occur when all extensions are disabled?: Yes
VS Code version: Code 1.63.2 (899d46d82c4c95423fb7e10e68eba52050e30ba3, 2021-12-15T09:39:46.686Z)
OS version: Linux x64 5.13.0-22-generic snap
Restricted Mode: No

System Info

Item
Value

CPUs
AMD Ryzen 3 5300U with Radeon Graphics (8 x 2842)

GPU Status
2d_canvas: unavailable_softwaregpu_compositing: disabled_softwaremultiple_raster_threads: enabled_onoop_rasterization: disabled_offopengl: disabled_offrasterization: disabled_softwareskia_renderer: enabled_onvideo_decode: disabled_softwarevulkan: disabled_offwebgl: unavailable_softwarewebgl2: unavailable_software

Load (avg)
2, 1, 1

Memory (System)
7.12GB (1.21GB free)

Process Argv
--no-sandbox --force-user-env --unity-launch --crash-reporter-id f9ef0483-9e99-46a3-9f8c-1e6a71b29e0e

Screen Reader
no

VM
0%

DESKTOP_SESSION
plasma

XDG_CURRENT_DESKTOP
KDE

XDG_SESSION_DESKTOP
KDE

XDG_SESSION_TYPE
x11

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
pythontb:30283811
pythonptprofiler:30281270
vsdfh931cf:30280410
vshan820:30294714
vstes263cf:30335440
vscorecescf:30384386
pythondataviewer:30285071
vscod805cf:30301675
pythonvspyt200:30340761
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593:30376534
pythonvs932:30410667
vscop804:30404766
vs360:30404995
vsrem710:30416614
py55gd98:30411514
vscexrecpromp3t1:30407762

there is a long history of why the find scope is extended to first line start and last line end, making it risky to change the behavior.

duplicate https://github.com/microsoft/vscode/issues/80008


Revert "test: write example files to disk rather than stdout"

This broke the build as at least one test reads these files. We should have a flag to rewrite the files instead of assuming it's ok on every test run.
This reverts commit 3c92d21b73edca86f323e4714b8c49bf520903b0.

Codecov Report

Merging #517 into master will increase coverage by 0.35%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #517      +/-   ##
==========================================
+ Coverage   86.47%   86.82%   +0.35%
==========================================
  Files         111      111
  Lines        6758     6604     -154
==========================================
- Hits         5844     5734     -110
+ Misses        589      567      -22
+ Partials      325      303      -22


PHOENIX-5553: Transactional views should not get GlobalIndexChecker c…

…oproc

JIRA already committed, closing.


Support Swashbuckle easily

Is your feature request related to a problem? Please describe.
I want to be able to output the definition of each table controller as a well formed OpenAPI specification integrated into Swashbuckle.
Describe the solution you'd like
Something like:
services.AddSwaggerGen(c =>
{
  // Extension method to do "the right thing"
  c.AddDatasyncControllers();
});

Describe alternatives you've considered

Swagger does not allow you to extend for generic types (which is needed for Datasync Services)

Completed in verified checkin e03d8d2


Do we have a "Load Generator" which simulates job-submits in a day/period??

Do we have a "Load Generator" plan which simulates job-submits in a day/period??
maybe it can simulate that multiple users submit jobs in a ? Possion distribution ? in 2 hours?

Hi @xiandong79!
This is not something that's currently implemented but is definitely on the roadmap!
Probably the first step will be implementing a Poisson distribution parameter for the Sleep workload.
Currently you can approximate this with features already in spark-bench. (Note to self, this would be helpful to add to the docs)
Start by building a few workload suites to run in parallel with the workloads within each suite running serially. Each workload suite would represent one user. Between each of the workloads, insert a Sleep workload and tune accordingly.

I found similar repo- https://github.com/IBM/SparkMultiuserBenchmark which is also designed to measure resource manager performance for Multi-User & Sync/Unsync batch/interactive Spark workloads.
I think the "User case" of SparkMultiuserBenchmark is what we researchers want.


Gourmands graces update2

name to object
nutrition to calories
spoils_in to string

These are all items with name_plural and spoils_in objects. It loads without errors now.

done a bobo


assets downloads missing

hello, I downloaded the project using the flutterflow vscode extension, but I saw that the fonts and images in the assets folder did not download.

Same problem here, any fix suggestion?


Uploaded mods contain garbage directories

Some recently uploaded mod zips (e.g. http://content.faforever.com/faf/vault/mods/tvg_working_v1_sera_fixed.v0001.zip) have an empty directory corresponding to every subdirectory of the mod, and named like the path to the subdirectory. I was told that this client is the only way to upload mods right now, so it might be caused by some bug in the mod upload code.

Are you sure you linked the right mod? I don't see such a directory:

In that case it must be a Linux-specific issue. This is what I get when treating this zip file with unzip on Linux:

To clarify - by Linux-specific I mean that Linux triggers the issue. These zips do (wrongly) contain these extra directories, it's just that apparently Windows does not display them, or unpack them.

I've seen (and supposedly fixed) this before, it happens when the ZIP entries are created with backslashes as path separators instead of forward slashes. Works under windows, fails under linux.
I'll take a look (or anyone else?) and fix it, thanks for reporting!

Fixed in next version


[Create dashboard] Cancel should delete the new dashboard

Since this is actually after the actual dashboard creation. both actually and logically. making a 'revert' , delete is a bit dirty.
i think that an elegant yet simple solution would be to rename from 'cancel' to 'skip' , because thats what this actually is...a dashboard with no connection (because the step was skipped).
Thoughts?

Add two buttons instead of 'Cancel' - 'Skip' & 'Delete'

Don't see the benefit of a skip button, as all you will end up with is a blank dashboard / with toast errors popping up.
Less options would be better, in fact only 2 options are required - save / delete.


List of virtual machines with cut layout

Describe the issue
The list of virtual machines appears with the layout cut, see the print.

Configuration

UTM Version: 2.1.1
OS Version: 11.3.1
Apple Silicon

Duplicate of #2335


Update version in docker pull

Proposed changes
Wrong version in the docker pull
Checklist
Before creating a PR, run through this checklist and mark each as complete.

[ ] I have read the CONTRIBUTING doc
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have checked that all unit tests pass after adding my changes
[ ] I have updated necessary documentation
[ ] I have rebased my branch onto main
[ ] I will ensure my PR is targeting the main branch and pulling from my branch from my own fork

Release notes
If this PR introduces a change that affects users and needs to be mentioned in the release notes,
please add a brief note that summarizes the change.

NONE

Seems that our workflow for the release PR is missing multiple docs now...

yeah it's kinda weird


API- Response Type check from the response body

Area ->  API
New Feature - Need to add support to  validate the list of keys from response body
Describe the bug - New feature request
Version - NO
Expected behaviour - Example: if a user info is fetched i want to validate the value of a particular key and if there is another key as data in list of array type value, i want to identify the data type and validate or get a particular value from the list of array
Actual behaviour - Feature not available in current version
How to Reproduce? -  NO
Anything else? - NO
Note: In current testman response validation is not included.

This ticket has been duplicated
https://github.com/testmanio/homebrew-tm/issues/53


[fix][broker] Logger.warn usage bug fix: the number is inconsistent for format and arguments

Motivation
Based on the previous two PRs #9552 and #16228,
Logger.warn usage bug fix: the number is inconsistent for format and arguments
Modifications
add a argument in Logger.warn's arguments, the value is the dead letter topic's name
Verifying this change

[x] Make sure that the change passes the CI checks.
This change is a trivial rework / code cleanup without any test coverage.

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

If the box was checked, please highlight the changes

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

Documentation
Check the box below or label this PR directly.
Need to update docs?

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

Matching PR in forked repository
PR in forked repository: https://github.com/StevenLuMT/pulsar/pull/6

@codelipenghui @shibd @HQebupt @AnonHxy
help me review this pr,thanks

/pulsarbot run-failure-checks


Fix IPv6 migration

Fix IPv6 migration
IPv6 address needs to be formatted as [ip]:port.
Support migration also in the following cases:

IPv6 is disabled (sysfs disable_ipv6)
bindv6only is enabled

Make createTcpListener to support IPv6 local address.
To support IPv6 migration on kind PR #3374 is mandatory.
Signed-off-by: Or Shoval<EMAIL_ADDRESS>None

do we have an ipv6 ci lane? how are we validating migration over ipv6?

do we have an ipv6 ci lane? how are we validating migration over ipv6?

Hi David,
yes there is IPv6 lane
https://prow.apps.ovirt.org/view/gcs/kubevirt-prow/pr-logs/pull/kubevirt_kubevirt/3371/pull-kubevirt-e2e-kind-k8s-1.17.0-ipv6/1258079281842491392
the lane is auto triggered but on "skip report" this is why it isnt shown here,
but soon it will be not skip report, once this PR will be merged,
because its on skip report due to a flaky test that this PR fixes as well

i will address everything mentioned here
thanks


How to close out a connection?

Issue
There doesn't seem to be an API to close out a created connection. Is there?

I see that there is a quit method available in the library. Would be nice to update the documentation with this.

@a-koka There's also now an asynchronous client.disconnect() method- had to dig into the codebase to find this though. Agree the new API should be better documented

@a-koka checkout the v3.1 branch to see the v3 docs

https://github.com/redis/node-redis#disconnecting


StyleInclude inside of MergedDictionaries shouldn't be allowed

Describe the issue
Right now it's possible to write code below, compiler won't trigger any errors
    <Window.Resources>
        <ResourceDictionary>
            <ResourceDictionary.MergedDictionaries>
                <StyleInclude Source="avares://AvaloniaTest.Styles/Generic.axaml" />
            </ResourceDictionary.MergedDictionaries>
        </ResourceDictionary>
    </Window.Resources>

It is allowed only because StyleInclude implements IResourceProvider and inner styles might include any resources.
But developers can expect that it also will include inner Styles, which isn't how it actually work (neither how it supposed to work).

If anybody has that issue with "styleinclude" not working, you should include it with Control.Styles instead of Window.Resources:
<Window.Styles>
	<StyleInclude Source="avares://AvaloniaTest.Styles/Generic.axaml" />
</Window.Styles>


Cluster to single node replication prints "Could not parse cookie from response headers cookie_format_invalid"

Description
We are running a CouchDB cluster of 9 nodes. For backup purposes et. al. we would like to replicate all of its databases to a standalone single node CouchDB instance. The replication is a push-replication, i.e. the cluster has multiple continuous replications set up that target the single node.
Both the cluster and the single node are protected, so the replication knows about BasicAuth credentials for accessing the target as well as the source. We can use this setup to replicate data as expected and will see changes to the cluster being reflected in the backup node instantly.
Yet, for reasons unclear to us, all of the nodes in the cluster will log a message like this on each request made to the replication target:
[error] 2019-12-17T12:43:59.736014Z<EMAIL_ADDRESS><0.24235.3> -------- couch_replicator_auth_session : Could not parse cookie from response headers cookie_format_invalid

Log output for the single node looks unsuspicious.
From what I understand, the replication should use the Authentication header defined in the replication docs and no HTTP cookies are involved, yet after digging through the codebase it looks like the second branch is hit here: https://github.com/apache/couchdb/blob/1347806d2feebce53325070b475f9e211d240ddf/src/couch_replicator/src/couch_replicator_auth_session.erl#L307-L312 when i would expect it to hit the above one saying cookie_not_found.
I'm not entirely sure if this expected, or if there is something going wrong in how we have set things up.
Your Environment

CouchDB Version used: 2.3.1
Browser name and version: n/a
Operating System and version: Ubuntu 16 running CouchDB in Docker

Both the cluster as well as the single node are behind NGINX using the following proxy settings:
  location / {
    proxy_pass http://couchdb:5984;
    proxy_redirect off;
    proxy_set_header Host $host;
    proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
  }

As far as I understand this should not add or remove any cookie related headers.
All of the nodes in the cluster share the same couchdb.uuid and couch_httpd_auth.secret when the single node has different values.
chttpd.authentication_handlers is not set / using the default value. Explicitly defining this to be basic auth only seemed to make no difference.

Working from memory, bear with me.  Is there a security cookie defined some where in configs somewhere?  If so, and you edited it, does it have any spaces in it?

Hi @m90 , thanks for this nicely detailed issue. As of #1462 (which landed in 2.2.0) the replicator will actually try to establish a session using cookie authentication. We made this change so that admins could crank up the work factor on authentication to protect against brute force attacks. So that explains why we're even talking about cookie-related code paths. The replicator will fall back to regular basic auth if it can't establish a session, so it makes sense that you're still seeing progress.
As far as why the CouchDB 2.3.1 replicator thinks the cookie it's receiving from the target instance is invalid ... that I'm not so sure about. Is the single node target instance also running 2.3.1?
If the log spam is getting to you and you just want the replicator to use basic auth you can change the [replicator] auth_plugins config setting on the cluster nodes:
[replicator]
auth_plugins = couch_replicator_auth_noop

See http://docs.couchdb.org/en/stable/config/replicator.html#replicator/auth_plugins for the config documentation.
/cc @nickva

@m90
If the error is cookie_format_invalid it seems like we got a cookie but it doesn't have the 'AuthSession=...` value in it.
https://github.com/apache/couchdb/blob/1347806d2feebce53325070b475f9e211d240ddf/src/couch_replicator/src/couch_replicator_auth_session.erl#L462-L464
Perhaps the proxy is adding another cookie and we end up parsing that one first. Or it's modifying CouchDB's cookie and we don't parse it correctly.
Would you be able to capture the headers and see what it looks like. Otherwise, to help reproduce the issue maybe paste your nginx.conf and I'll try it on my laptop.
In the meantime, good advice from @kocolosk regarding disabling the plugin, see if that helps while we figure out the issue.
@seigel, I do recall an issue related to that, good call

Thanks for the detailed reponse @nickva @kocolosk
This is what I could find out using your info:

When I disable the session auth plugin and use couch_replicator_auth_noop the log messages go away, replication continues working
When I have nginx log the cookies included in the request(s), I can see that a non-empty AuthSession is being included in requests coming from the CouchDB Replicator. Its value also does not contain whitespace or anything unexpected.

For me to debug further, it would be interesting to know how I can further map these error messages to a request.
If I look at the log messages I can see some identifiers like:
[notice] 2020-01-13T09:35:41.390874Z<EMAIL_ADDRESS><0.6072.1500> ce9077a0e1 some.hostname.example.com ...

or
[error] 2020-01-13T09:24:07.152363Z<EMAIL_ADDRESS><0.8946.1807> -------- couch_replicator_auth_session : Could not parse cookie from response headers cookie_format_invalid

but I'm not entirely sure what the 0.6072.1500 (or similar) and ce9077a0e1 are all about?

The 0.6072.1500 is an Erlang process identifier. It won't be helpful for mapping log messages to requests, but Is occasionally helpful for understanding that several messages were emitted from the same internal process.
The ce9077a0e1 is a unique identifier for the HTTP request and is included as the value of the x-couch-request-id header in the response generated by CouchDB.

The ce9077a0e1 is a unique identifier for the HTTP request and is included as the value of the x-couch-request-id header in the response generated by CouchDB.

As the error messages print -------- for this value, I assume I cannot really tell which request caused the error, right?

Correct.

@m90

Could you capture the responses from the client to the proxy, then from the proxy to CouchDB see anything is adding cookie. Based on where the log is emitted, it not might not be an CouchDB cookie. https://github.com/apache/couchdb/issues/2369#issuecomment-573292483 . Also notice if there are any differences at all between "Set-Cookie" headers before and after the proxy.

Just to debug the scenario, can you eliminate the nginx proxy from the connection chain altogether, by say having a local cluster or use an SSH tunnel. Then see if errors persist or not.

Based on the logic in the code https://github.com/apache/couchdb/blob/1347806d2feebce53325070b475f9e211d240ddf/src/couch_replicator/src/couch_replicator_auth_session.erl#L462-L464 it looks like there was a cookie found but it doesn't have an "AuthSession" part. That looks like a bug as CouchDB should ignore that cookie altogether and only pay attention to the ones having an "AuthSession" instead of failing nosily. However before making that change, let's see if we can confirm that theory.

@m90
I couldn't reproduce the issue on Ubuntu 18.04 with distro default nginx version and a with this config:
# Default server configuration
#
server {
listen 80 default_server;
listen [::]:80 default_server;

root /var/www/html;
index index.html index.htm index.nginx-debian.html;

server_name _;
        location / {
            proxy_pass http://localhost:15984;
            proxy_redirect off;
            #proxy_buffering off;
            proxy_set_header Host $host;
            proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
     }
}

I had noticed in the code we would emit that error if we somehow got some other, non-CouchDB emitted cookie. So made PR to ignore those:
https://github.com/apache/couchdb/pull/2454
Also noticed the issue @seigel mentioned that we did not parse some of the cookie bits like expiration time properly. That field is not used, as we use max-age, but does indicate that if nginx somehow re-writes the cookie in a certain way the same parsing error might trigger in a different way and the cookie would not be found anymore. The parsing issue should be fixed in https://github.com/apache/couchdb/pull/2452 with the new web server component update.
Thought I am still curios, if you get a chance to capture the traffic between client and proxy and proxy and CouchDB as to what was causing the issue in your case. Another way to do it is to re-build from source and add a few debug lines to this module:
-spec parse_cookie(list()) -> {ok, age(), string()} | {error, term()}.
  parse_cookie(Headers0) ->
+    couch_log:error("***** Headers0:~p", [Headers0]),
      Headers = mochiweb_headers:make(Headers0),
      case mochiweb_headers:get_value("Set-Cookie", Headers) of
          undefined ->
              {error, cookie_not_found};
          CookieHeader ->
              CookieKVs = mochiweb_cookies:parse_cookie(CookieHeader),
+            couch_log:error("****CookieKVs:~p", [CookieKVs]),
              CaseInsKVs = mochiweb_headers:make(CookieKVs),
+            couch_log:error("-----> AuthSession:~p", [

Ok, we finally found out where this was coming from (although it still doesn't make too much sense): the Cluster that was causing the issues was (for reasons unknown to me) running behind a Cloudflare Proxy which does some interesting and unexpected things to headers and such.
This still does not explain why it's a single database only, but it must be something in the combination of the two.

There is no validate_doc_update on the database.

Would you be willing to share the name of the database? My guess is Cloudflare has some sort of rule that's kicking in due to the nature of the data it thinks is in place.

The database name sounds relatively unsuspicious (to me), it's van-shipments.


[BUG][ENH] mpyez.list_.get_object_count not giving correct results

The get_object_count function returns a class function instead of results when the option get_tabular_form == True

It works, just needed to print the returned object.


fix: clustertree bugs

fix
What type of PR is this?

fix: clustertree bugs
What does this PR do?

Which issue(s) does this PR fix?

Fixes #
Special notes for your reviewer:

Does this PR introduce a user-facing change?

NONE

/approve

/lgtm


[Wave9] Detect first time users - check display name and purpose

https://linear.app/expensify-swm/issue/EXP-275/[wave-9]-new-onboarding-flow-detecting-first-time-users
Checks if user has his required onyx keys set (name + surname + purpose). If not, on each refresh the modal will show up until he fills the data.

Closing. Replaced by https://github.com/software-mansion-labs/expensify-app-fork/pull/68


Updated SimpleInjector to version 5.0.3 in nuspec

Set EnableAutoVerification to false because otherwise the unit tests will fail with registering an InstancePerCall component. It implements IDisposable and should therefore not be registered as Transient

Thanks, I'll do a release soon

Pushed 3.0.6 to nuget. Thanks again.


CB-17805 change volume size and type GCP

JIRA: CB-17805
Change datalake duty volume type and size at GCP.
Versions from: 7.2.7 to 7.2.16
New medium duty size:

AUXILIARY - Reduced to 128GB
GATEWAY - Reduced to 128GB
MASTER - Reduced to 128GB
CORE: Increase it to 512GB

New light duty size:

Light duty: Increase it to 512GB

new core type: ps-ssd

did we look at using balanced ssd?


Less vertical space at the beginning of a list

If a list occurs in a paragraph which in the latex file means it looks like this
This is a
\begin{enumerate}
\item List,
\item continuing
\end{enumerate}

then I think currently a combination of plastex and gerby-website puts too much space before the first list item (just as in the current server). I think Tag 00DY is a good example; if you don't think this is ugly, then don't fix this. My feeling is that in the pdf there is always a bit more vertical space between text and the beginning of an environment than inside an environment between different parts of the environment.

I don't think Tag 00DY is necessarily ugly at the moment, but the lessening of vertical space does make it prettier. A quick adjustment is done in 753c90f. Adjust it to your liking!

Amazing!


Fixed broken tests

Some tests didn't work. Pool request fixes this.

@Ne-Lexa thank you for contribution


Issue with k8s.io/docs/tutorials/hello-minikube/

Katacoda doesn’t work.  Minikube doesn’t work. Is there a good, official, tutorial that does work?  Or that is “printed” only without requiring finicky interactive stuff that never works?

I tested https://k8s.io/docs/tutorials/hello-minikube/ following the instructions, using Katacoda to host the cluster.
I noticed a missing detail. Before you run minikube dashboard, you should open a new terminal, start minikube dashboard there, and then switch back to the main terminal.
Alternatively, we could rely on shell process management and have the reader run minikube dashboard &. This doesn't work on Windows though.
Would you be willing to revise the issue description to include these details I discovered @rich-pixley?

I didn't have any luck with it on any of several browsers I tried.  It
failed out on all of them.
On 2/20/21 07:39, Tim Bannister wrote:

I tested https://k8s.io/docs/tutorials/hello-minikube/
https://k8s.io/docs/tutorials/hello-minikube/ following the
instructions, using Katacoda to host the cluster.
I noticed a missing detail. Before you run |minikube dashboard|, you
should open a /new/ terminal, start |minikube dashboard| there, and
then switch back to the main terminal.
Alternatively, we could rely on shell process management and have the
reader run |minikube dashboard &|. This doesn't work on Windows though.
Would you be willing to revise the issue description to include these
details I discovered @rich-pixley https://github.com/rich-pixley?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kubernetes/website/issues/26617#issuecomment-782691440,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAMESP3KUVYL4WF47VKOA63S77CQNANCNFSM4X2X3NYA.


remove dependency to @angular/http

Bug, feature request, or proposal:
remove dependency to @angular/http from the package.json file
What is the expected behavior?
no dependency to @angular/http
What is the current behavior?
has a dependency to @angular/http
What are the steps to reproduce?
ng new testApp
cd testApp
npm install --save @angular/material @angular/cdk
npm uninstall @angular/http --save

make sure your angular version is over 4.3
follow your getting started guide for using your lib
see this error when trying to build
ERROR in ./node_modules/<EMAIL_ADDRESS>Module not found: Error: Can't resolve '@angular/http' in '.\node_modules\@angular\material\@angular'

What is the use-case or motivation for changing an existing behavior?
first of all, none of your components need http, so you shouldn't depend on it
secondly, since version 4.3 of angular, they introduces httpClientModule under @angular/common/http. it's a rewrite of the httpModule of @angular/http with more features (like interceptors, native json parsing, typed response) and easier mocking
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
angular 4.3+ if you want to replace http with httpclient. any angular version if you just wanna remove the dependency
Is there anything else we should know?
this stack overflow answer explain the difference between httpModule and httpClientModule in more details

Will be solved by https://github.com/angular/material2/pull/6702.


Vertical Scrolling Box

Hello,
is there a way to make a box (https://almsaeedstudio.com/themes/AdminLTE/documentation/index.html#component-box) vertically scrollable with a fixed height?

Sure that's not hard 😃
the easiest way (css only) is give the box-body a id and create a declaration or add inline style with:
height: 600px; overflow-y: scroll; (change 600px to your desired height).
If you want to hide the scrollbars replace overflow-y: scroll; with overflow: hidden;.
regards REJack

Thanks. it does work.
When I scroll the table, the table header also scroll respectively. Is it possible to make the table header fixed-position?


Application is incompatible with my OS/device

CyanogenMod 13 on a Samsung Galaxy S4 GT-I9505. The Play Store says the app is not compatible with my device. It's Marshmallow, so def not an issue with API version. Also, it worked on CyanogenMod 12.1 (Android 5.1.1).

Custom ROMs aren't officially supported, sorry.


Polymorphic relations allowed by DTD/XSD

Polymorphic relations are supported a long time now (https://github.com/propelorm/Propel2/pull/864) but DTD/XSD aren't aware of that

Thanks!

I made mistake, please look at #1208


Utility to generate an environment map from a scene

IIRC @panxinmiao proposed this earlier. Then we can create stuff like this.

Yes, we also need a corresponding PMREM (Prefiltered, Mipmapped Radiance Environment Map) generation utility to provide physically-correct scene based lighting.


Images added as different sizes when dragging vs clicking

Bug Description
When you click to add an image to the page, it is added as one size.
Then if you drag the same image to the page, it is added as a larger size.
Expected Behaviour
It should be consistent.
Screenshots
https://xwp.slack.com/archives/CCCAUAH6F/p1591724046225200
Additional Context

Plugin Version:
Operating System:
Browser:

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria

Implementation Brief

We could scale images (and shapes) up when dragging starts to the same size they would have, if just placed by clicking - and we could potentially use an animation similarly to this:
https://twitter.com/siddharthkp/status/1273868440923713536
CC @samitron7, @pbakaus, @o-fernandez.

Drop behavior for images/video is in Figma now (added on Friday): https://www.figma.com/file/Cd9iacVknHK62RO4pRwb68/Stories-Editor?node-id=10064%3A0

yep that behavior sgtm.

@barklund Get outta here, can we do that animation please. Once the thumb hits the page, animate the resizing.
And Marcin and I chatted about all this on slack and I updated Figma for all things related to sizing when added to page on Figma. Let me know if any additional tickets are missing figma links.


Changes:

Remove redundant animation: none when the element is hidden
Use transforms for the animation

Ref.https://www.paulirish.com/2012/why-moving-elements-with-translate-is-better-than-posabs-topleft/

Hey, awesome!


Added the open source organisations.

Added the names of  open source organisations with their active links to the website.

Issue #25 PR Merged


Document clearly how Mbed Crypto provider keys are stored

The documentation for what gets stored on disk for the Mbed Crypto provider isn't clear enough, as it could be misunderstood (based on the store_path of the KeyInfoManager) that the keys are stored along with their mappings.
In reality, the keys are stored locally in the working directory of the process using the Mbed Crypto library. This can be changed at compile time (see this) - maybe we could also include this as part of our build, but the main point is that the behaviour should be described more thoroughly somewhere, preferably both in the book and in the config file

Related to #53

Solved in the two PRs linked above


History service

Description
Allows the user to access past attempts on PeerPrep. It also showcases the number of easy, medium and hard questions the user have done in MetricComponent and also a table of the attempts made by the user in PreviousQuestionDone
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Seems like there's this 'bug' where the first user that closes the room (and is returned home automatically via close room) doesn't have their history updated.
However, doesn't seem like an important issue since it's updated once we refresh the browser.

Merged master and resolved conflict.
Will merge in this PR to master first


Contact Page

Hey @Srijita-Mandal, I would like to open this issue under JWOC. I wish to add a contact us page:

a user can submit his information and message to communicate with the admin.
the page will have same UI as home page & will contain the contact form with illustrations
it will be responsive
it will be added in the navbar of all the pages
the contact form will be validated

I would like to work on this issue under JWOC

@ritika728 Go for it.


"Cannot connect to Mattermost" is now on top of the page

First of all, please read CONTRIBUTING.md

[x] Complete Mattermost Contributor Agreement
[x] Execute npm run prettify to format codes
[x] Write about environment which you tested

OS X 10.12.1

Should fix #299

You're awesome @magicmonty !! Highly appreciated
Artifacts available here to download, can we get community's help testing the pull request?
Original issue report can be found here: https://github.com/mattermost/desktop/issues/299

Confirmed on Windows 10

Anyone would like to help with Mac or Linux? Below are the builds to download:

Mac
Linux i386.deb
Linux amd64.deb
Linux ia32.tar.gz
Linux x64.tar.gz

To test, you can follow these steps:

Download one of the builds above and open the app
Disconnect your Internet
Refresh the page and observe the error message provided. The expected is a screenshot shown above, which should be at the top of the page, instead of the middle

This change should be platform agnostic, testing it like you did should cover all platforms.

Sure, so if the appearance is proper for us, it's ready to merge.

We should try to have two testers regardless, even if the change applies to all platforms the same way.
I'm okay if both tests are on Windows, as long as there are two for review.

Tested on macOS 10.12.

Thanks @yuya-oc! Good to merge


Floaty only in ViewController, but black overlay above tabor and NavBarHeightLargeState

How can I make black semi-transparent overlay to be on top on nav and tab bar, while having floaty button only in a single controller?

    UIApplication.shared.keyWindow!.addSubview(fab)
    UIApplication.shared.keyWindow!.bringSubviewToFront(fab)

What is fab? Where you changing this value
I added  plusButton via storyboard.
@IBOutlet weak var plusButton: Floaty!


fix(plexis): added cross-env to dev dependencies

When following the installation instructions I was presented with the
following error:
'NODE_ENV' is not recognized as an internal or external command,
operable program or batch file.

cross-env fixes the error on windows computers

Wow, thanks so much @Mercyfulsin.


support tls options in client connection

Adds support for tls options when connecting to a client that requires certificate, key and such.

Thanks! But how about moving TlsOptions from amqp-properties.js to its own file? It has nothing to do with AMQPProperties so doesn't make sense to put them in the same file.

flaky tests?

@carlhoerberg any update?

any update for https://github.com/cloudamqp/amqp-client.js/pull/51 ?

Sorry, will check later today/night

2.1.1 is out with this PR merged


404 at /compose/install/compose-desktop/

I found a broken link : https://docs.docker.com/compose/install/compose-desktop/
I arrived on this page through: https://docs.docker.jp/

Can you please assign me this issue?

Thanks for reporting the broken link. Please report this issue on the source page where the link is located.


[MINOR] docs: fix formatting (#2177)

What changes were proposed in this pull request?

fix formatting

Why are the changes needed?
Improve the table cell formatting
Does this PR introduce any user-facing change?
no
How was this patch tested?
Local test

As mentioned in #2177, this PR is for branch-0.4


Import/Export Job Updates

add-databicksnotebook takes settings as one object instead of switche…s; export jobs by job id and save settings only.
This is how I'd like to manage my Jobs in source -

create the job in the portal, because it's easy.

download the job via the API, save it as a json file in a folder in my repo.

pass the json to the API in a deployment.

wrt point 3 - The trouble is is that Add-DatabricksNotebookJob will take a bunch of parameters and create the json object for you.
So then either
A.  have yet another script that takes the job json stored locally, figure out which params to run then use Add-DatabricksNotebookJob to deploy the job (which then turns it back into json, so what's the point
B. store the jobs as cmds in like a csv or something else to run the Add-DatabricksNotebookJob for each job.
C. Alter Add-DatabricksNotebookJob to accept a PSCustomObject that contains everything required to make the job, which is the change I have submitted to the cmdlet.
wrt point 2 - all I need to save is the settings part of the Job, so Get-DatabricksJob has been created to get the Job based on a jobId and has the option to output only the settings of the response. And Export-DataBricksJobs uses this function to loop through an array of JobIds and save them to a local directory, using name stored in the settings to use as the filename.

/azp run

Likey, I've been wanting something like this for a while. I added the ability to pipe clusters last week. Jobs and Instance Pools all need these features.
Lack of tests but I'll let you walk!
Added one small request - but otherwise good to go.

Glad the changes are there for clusters; GMTA.
Ok no worries will change param name.
Ahaha you’re quite right though; no tests! Was lazy and set myself a 5PM deadline to create the PR. Will include some tests with the change!

/azp run

Huh, wondered why I was getting emails about jobs being deleted...
Expect another PR from me soon that will remove my emails from the jobs I added to the repo 👍


Add doc on function type

closes: #3973

@dvermd This is a good trial, but we don't manually add docs anymore. We have a auto-generated document db.
The db is generated by scripts/generate_docs.py and applied to each object and functions by rustpython-doc::Database (search for crate::doc::Database::shared() in source code)
are you interested in finding the bug of them?

done via https://github.com/RustPython/doc/pull/2


Dead links

On this page the link "JavaDoc as bundle" leads to https://odftoolkit.org/odfdom/odftoolkit.org/api/odfdom/index.html which results in 404.

We should collect all dead links into one issue in the future (as lone one is open), I will take care of them for now. Thx! :-)

The whole documentation section in the README is dead, e.g. https://tdf.github.io/odftoolkit/docs/odfdom/index.html

Yes, we realized the old/deprecated Apache CMS is creating HTML with an empty body without an error.
A quick workaround is given by Sebastian MD2HTML java-based generation is provided/tested on a fork of mine:
https://svanteschubert.github.io/tdf-odftoolkit-fork/docs/odfdom/index.html
I am currently working on the generator module enabling all tests to generate an ODF 1.3 DOM, doing three releases in a row with all updated documentation.

Aside from the references to the Maven artefacts 0.9.0 and 0.10.0 (which might follow later today, when I am able to tame the generator) all links should be fixed at the 0.10-docu branch viewed at the moment at:
https://svanteschubert.github.io/odftoolkit/
In addition, I have used https://www.drlinkcheck.com/ to check for broken links on the above draft site.


Rate limiting

Look into rate limiting the polling. Hopefully I can implement dynamic polling of some kind as we don't need to poll parcels once delivered. At the very least reduce the frequency of polling as 5 minutes has already resulted in my account being blocked after less than a day

This should hopefully be resolved in 2024.10.0.
Won't close just yet.

Polling interval of 30 minutes produces a API rate limit exception after about a week.
Will try 45 minutes and see

note, 30 minutes polling causes api rate limiting to happen after 26 hours


Language review of Edge Documentation

This PR aims to check the compliance of the Edge documentation with the SUSE Documentation Style Guide in terms of style, consistency and terminology.

@dariavladykina Would you prefer me to make fixes in this PR or go ahead and merge this then make fixes in my own branch?

Hi @agracey, I believe it's better to make fixes in this PR and then re-request another final eye review. Please bear with me to finish all the files so that we don't get validation errors. Thank you!


Lottie and check update

Closes:

#1905
https://app.zenhub.com/workspaces/weave-gitops-enterprise-6127680175876a0014fc5657/issues/weaveworks/weave-gitops-enterprise/578

What changed and why?

replaced SignIn svg background with Lottie animation (improved performance on Firefox)
the Error page now uses a Lottie animation as well - improve user experience (we used to have text only)
improve feedback to user whilst the FeatureFlags are being retrieved / if they error out

ewq:~/weave/weave-gitops(main $%=)$ npm run build

><EMAIL_ADDRESS>build
> parcel build --target default

✨ Built in 34.15s

cmd/gitops-server/cmd/dist/index.html                                417 B     3.84s
cmd/gitops-server/cmd/dist/gitops-LOGO.4e557fcc.ico              201.34 KB     1.08s
cmd/gitops-server/cmd/dist/index.2f1bd790.js                    ⚠  1.5 MB    15.02s
cmd/gitops-server/cmd/dist/ProximaNovaBold.87676319.otf           94.38 KB     1.07s
cmd/gitops-server/cmd/dist/ProximaNovaRegular.91a8c864.otf        92.45 KB     1.07s
cmd/gitops-server/cmd/dist/ProximaNovaSemibold.7fa90ba1.otf       89.75 KB     1.07s
cmd/gitops-server/cmd/dist/roboto-mono-regular.b5d84859.woff       19.5 KB     1.06s
cmd/gitops-server/cmd/dist/index.780a4a15.css                      7.82 KB     4.23s

ewq:~/weave/weave-gitops(main $%=)$ git co -

ewq:~/weave/weave-gitops(lottie-and-check-update $%=)$ npm run build

><EMAIL_ADDRESS>build
> parcel build --target default

✨ Built in 24.65s

cmd/gitops-server/cmd/dist/index.html                                 417 B     2.55s
cmd/gitops-server/cmd/dist/gitops-LOGO.4e557fcc.ico               201.34 KB     1.08s
cmd/gitops-server/cmd/dist/index.49a3208f.js                    ⚠  2.52 MB    12.28s
cmd/gitops-server/cmd/dist/ProximaNovaBold.87676319.otf            94.38 KB     1.07s
cmd/gitops-server/cmd/dist/ProximaNovaRegular.91a8c864.otf         92.45 KB     1.07s
cmd/gitops-server/cmd/dist/ProximaNovaSemibold.7fa90ba1.otf        89.75 KB     1.07s
cmd/gitops-server/cmd/dist/roboto-mono-regular.b5d84859.woff        19.5 KB     1.06s
cmd/gitops-server/cmd/dist/index.780a4a15.css                       7.82 KB     4.23s

The bundle size jumped up 1mb?! .. its a bit funky. Maybe we could quickly test out lazy loading the animation as in https://github.com/mifi/react-lottie-player#lazy-loading-example to see if that has an impact?

Good point, I hadn't checked that. It doesn't look that there's a prop for that, I've implemented the suggestion from https://github.com/mifi/react-lottie-player/issues/11. npm run build finished in 38s for the latest main and 33s for the branch for me

Awesome! Thanks for trying this out. That is super odd, will have a quick poke..

Could try the bundle analyzer on it to see whats taking up all the space? 🤔
https://parceljs.org/features/production/#bundle-analyzer

Lotti is 650kb and the error404 is 200kb

Maybe the json won't be correctly bundled into the lib tho 🤔 ... really need to find a better way..

Will reopen a new branch


TK103 decoding problem

HI Anton, I tried already few trackers with your traccar, now I'm testing tk103 and it's connected thru ODB2 port, so I was expecting it to send more data then just coordinates, below you can see decoded data, I tried to break it apart, it seems like it sending at least 5 useful events? In UI it just showing location, time, speed, and status code changing from 00000000 to 00000001, is there any other information that can be extracted from data that's coming, and what's status code change mean?
Also I attached txt with HEX data.
tk103 HEX info.txt
Thank you in advance.
Hope to hear from you soon.
(016023234823
BR00160701
A4725.4862
N12213.9120
W000.023580000
0.0000000000
L00000000)

(016023234823
BR00160701
A4725.4785
N12213.9349
W000.005354433
3.9900000001
L00000000)

I don't think it's possible to extract any more info.
As for the status, here a quote from documentation:

The 8 bits of IO The first bit representative of the main power switch, "0" means the main power-on, "1", means the main power-off.   The second bit on behalf of the ACC (ignition), "0" means    ACC off, "1" means ACC on.   Other reservations

About 8 bits of IO in our case only last "0" cganging to 1, maybe it's
ignition indicator?
On Monday, 4 July 2016, Anton Tananaev<EMAIL_ADDRESS>wrote:

I don't think it's possible to extract any more info.
As for the status, here a quote from documentation:
The 8 bits of IO The first bit representative of the main power switch,
"0" means the main power-on, "1", means the main power-off. The second bit
on behalf of the ACC (ignition), "0" means ACC off, "1" means ACC on. Other
reservations
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tananaev/traccar/issues/2075#issuecomment-230250991,
or mute the thread
https://github.com/notifications/unsubscribe/AINLAx1lyMciN61G3SUdt-FLSQ1q3Y1Dks5qSNbYgaJpZM4JEIE7
.

--
Sent from Gmail Mobile

It says that "the first bit representative of the main power switch".

Thank you for your help, have great week.


Copy #955 and #956 from release/0.14 to master

Redesigned the framework to make it less vulnerable to native exceptions. (#955)
Updates for react native examples.  (#956)

Checklist

[x] Example projects and code samples are up-to-date (in case of API changes)

@radetsky can we please use the same tactics and migrate PRs as two parts: examples and core?


move check_dummy_inputs_allowed to common export utils

What does this PR do?
check_dummy_inputs_allowed is helpful function for any optimum that involves pytorch models conversion (e.g. openvino). Current place of it import trigger initialization of full onnx configuration for other backends that wish to reuse it and it may lead to some troubles with resolving dependencies. I suggest to move it to common utilities for avoiding unexpected errors like represented bellow:
Example of logs (issue visible on windows with onnx>1.16.2 and related to https://github.com/onnx/onnx/issues/6267):
  File "C:\Users\chuquanc\openvino\py310\lib\site-packages\optimum\intel\openvino\__init__.py", line 51, in <module>
    from .quantization import OVQuantizer
  File "C:\Users\chuquanc\openvino\py310\lib\site-packages\optimum\intel\openvino\quantization.py", line 47, in <module>
    from optimum.exporters.onnx.convert import check_dummy_inputs_are_allowed
  File "C:\Users\chuquanc\openvino\py310\lib\site-packages\optimum\exporters\__init__.py", line 16, in <module>
    from .tasks import TasksManager  # noqa
  File "C:\Users\chuquanc\openvino\py310\lib\site-packages\optimum\exporters\tasks.py", line 173, in <module>
    class TasksManager:
  File "C:\Users\chuquanc\openvino\py310\lib\site-packages\optimum\exporters\tasks.py", line 338, in TasksManager
    "clip-text-model": supported_tasks_mapping(
  File "C:\Users\chuquanc\openvino\py310\lib\site-packages\optimum\exporters\tasks.py", line 117, in supported_tasks_mapping
    importlib.import_module(f"optimum.exporters.{backend}.model_configs"), config_cls_name
  File "C:\Users\chuquanc\packages\Python310\lib\importlib\__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "C:\Users\chuquanc\openvino\py310\lib\site-packages\optimum\exporters\onnx\model_configs.py", line 23, in <module>
    from ...onnx import merge_decoders
  File "<frozen importlib._bootstrap>", line 1075, in _handle_fromlist
  File "C:\Users\chuquanc\openvino\py310\lib\site-packages\transformers\utils\import_utils.py", line 1766, in __getattr__
    module = self._get_module(self._class_to_module[name])
  File "C:\Users\chuquanc\openvino\py310\lib\site-packages\transformers\utils\import_utils.py", line 1780, in _get_module
    raise RuntimeError(
RuntimeError: Failed to import optimum.onnx.graph_transformations because of the following error (look up to see its traceback):
DLL load failed while importing onnx_cpp2py_export: A dynamic link library (DLL) initialization routine failed.

workaround for this problem also proposed on optimum-intel side https://github.com/huggingface/optimum-intel/pull/1048

@echarlaix  @IlyasMoutawwakil  could you please review?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Fix ESBJAVA-3832 - Handling java.util.EmptyStackException

I'm closing this PR since we need to check the problem described in this thread -
<EMAIL_ADDRESS>


Added GitHub CLI option to Contributing section

Updated the Contributing section, in docs, to include GitHub CLI option. Please fill up the TODO sections for the regular git option

Codecov Report

Merging #81 into master will decrease coverage by 0.57%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #81      +/-   ##
==========================================
- Coverage   56.00%   55.43%   -0.58%
==========================================
  Files          55       55
  Lines        1714     1692      -22
==========================================
- Hits          960      938      -22
  Misses        754      754

Flag
Coverage Δ

#unittests
55.43% <ø> (-0.58%)
:arrow_down:

Impacted Files
Coverage Δ

mantisshrimp/core/bbox.py
63.26% <0.00%> (-4.01%)
:arrow_down:

mantisshrimp/metrics/metric.py
53.33% <0.00%> (-2.92%)
:arrow_down:

...antisshrimp/metrics/coco_metric/reference_utils.py
19.28% <0.00%> (-2.00%)
:arrow_down:

mantisshrimp/metrics/coco_metric/coco_metric.py
37.83% <0.00%> (-1.64%)
:arrow_down:

mantisshrimp/models/mantis_rcnn/mantis_rcnn.py
37.50% <0.00%> (-1.10%)
:arrow_down:

mantisshrimp/core/mask.py
80.20% <0.00%> (-0.98%)
:arrow_down:

mantisshrimp/core/category.py
85.00% <0.00%> (-0.72%)
:arrow_down:

mantisshrimp/parsers/parser.py
93.75% <0.00%> (-0.19%)
:arrow_down:

mantisshrimp/parsers/splits.py
95.65% <0.00%> (-0.19%)
:arrow_down:

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 51f5488...49565e8. Read the comment docs.

For regular git. Just update the repo after PR is opened. Add it commit it.
Push to GitHub. GitHub will automatically update the PR.

To confirm pr was created. Come to this repo.  Check in pr section.  simple.


Fix u128

Pulling in @omotorin commits and trying to finish https://github.com/crystal-lang/crystal/pull/5564

I am not seeing these errors locally is this a CI thing?

You need to run crystal tool format

By the way, we should probably run this check earlier in CI. It's a waste of time to run all specs and have them pass just to fail later because the formatting is not correct.

@wontruefree & what about https://github.com/crystal-lang/crystal/pull/5564#discussion_r16066589?

@Sija I did some googling a little bit ago and read that it is one cycle on 32bit machines anyways I dont know if I can just get that from the x86 spec but I am more then willing to explore that.  I am wondering if I can verify these problems in virtual box.

A simple benchmark of String#to_i before and after this PR will show why this can't be merged.

wow I didn't realize what the performance impact would be.
master:
$ ./.build/crystal benchmark.cr --release
String#to_i32  35.87M ( 27.88ns) (± 2.76%)  0 B/op   1.01× slower
String#to_i64  36.11M ( 27.69ns) (± 2.45%)  0 B/op        fastest

this branch:
$ ./.build/crystal benchmark.cr --release
String#to_i32   2.24M (446.08ns) (± 4.01%)  0 B/op   1.04× slower
String#to_i64   2.32M (430.29ns) (± 1.97%)  0 B/op        fastest

@asterite Thanks for the explanation.  I am getting more understanding the more I get into this.  I am working on it now.

Ok this is a little better but some extraction might be possible.
$ ./.build/crystal benchmark.cr --release
String#to_i32  35.96M ( 27.81ns) (± 2.35%)  0 B/op   1.00× slower
String#to_i64  36.12M ( 27.69ns) (± 1.86%)  0 B/op        fastest

It's not so much a bug as just a lack of intrinsics.

@Sija I have been kind of deep in the weeds in the compiler.
I think I have to add u128 and i128 to the compilers math interpreter. src/compiler/crystal/semantic/math_interpreter.cr
Do you know if that would be the next steps to working on UInt128 to crystal?

@wontruefree I think so. Adding missing functions from compiler-rt (in the vein of https://github.com/crystal-lang/crystal/pull/7206/files#diff-b1ba6a49a8ea0b8a8425cebf9552302) seems needed too.

@wontruefree Are you still planning on finishing this?

@Sija I am actually working on this today.  But I am not as well versed in the compiler and dont have a 32 bit machine. If this is easy for you to finish you are more then welcome to take it.

I have been trying to get things to work in vagrant.  I have not gotten repeatable results with cross compiling.

If I'm not mistaken you'd need to port below functions to make it work on 32-bit machines:

__multi3 => https://github.com/llvm-mirror/compiler-rt/blob/master/lib/builtins/multi3.c
__umodti3 => https://github.com/llvm-mirror/compiler-rt/blob/master/lib/builtins/umodti3.c
__udivti3 => https://github.com/llvm-mirror/compiler-rt/blob/master/lib/builtins/udivti3.c
__modti3 => https://github.com/llvm-mirror/compiler-rt/blob/master/lib/builtins/modti3.c
__divti3 => https://github.com/llvm-mirror/compiler-rt/blob/master/lib/builtins/divti3.c

there any news?

@delef I was working on it two weeks ago and having a really hard time getting consistent builds on 32bit machines.  Basically I cannot get master to build consistently on VirtualBox.  I have also tried cross-compiling, but I was not confident I got it working correctly.
If you want to take this over or have a way to build 32 bit consistently I would like to know.  If not I work on it when I have time but it is going really slowly until I get 32 bit builds working.

@wontruefree any updates?

@Sija I have identified the rt functions that would need to be implemented and found example implementations in the llvm source code.  I have been having trouble running code on 32-bit VMs, and I had some difficulties running rt functions in general.
If someone wants to take this over and has the experience to do this they should. I can take another pass at it but it probably won't be till later this month.
Also, I have been doing my work on another branch so I could experiment without pushing up code.  But if someone wants the code, it is here.
https://github.com/wontruefree/crystal/tree/fix_u128_bak

If people are following this I have some updates.  I have figured out how to implement and test this.  It is just a fair amount of work and I have been working through it.  I am hoping to have this by 0.30.0

@wontruefree maybe it will make sense to split this PR in 3 parts.

for the port of compiler-rt, bringing uniform support for overflow and int128 in all platforms by default.
support for parsing int128 numbers (probably by introducing ToU64Info and ToU128Info instead of the union)
bring support to the compiler/parser itself to handle 128 bits literals

I think this order will make things easier to review and merge (also, big thanks for porting compiler-rt with the current design 🙇 )
WDYT?

@bcardiff I like this idea.  I think this PR got more and more expansive then I thought.  It is kind of hard to verify all this works without getting to run in crystal.


Upgrade to handlebars >= 4.3.0 to resolve security vulnerability

See https://www.npmjs.com/advisories/1164

Thanks for the report! We've released 10.4.0 to resolve this issue


Missing Static Files

I cannot get the static files to properly show under the current docker settings. I think we need to add a STATICFILES_DIRS parameter to the django settings file. See: https://docs.djangoproject.com/en/1.9/ref/settings/#std:setting-STATICFILES_DIRS and a related post with a possible easy solution here: https://stackoverflow.com/questions/38627393/docker-django-404-for-web-static-files-but-fine-for-admin-static-files

Hello,
I encountered this issue before. Basically, 2 ways to solve:

if you are using a web server (nginx or apache), map the app/YtManagerApp/static directory to the path /static/ from the server's configuration (this is how the provided Docker compose is built).
run django either in debug mode (set Debug=True in config.ini file) or insecure mode (add --insecure flag when running manage.py.

I reread your message, I will investigate the issue when I get the chance.

In the docker compose, I map /static url to directory
app/YtManagerApp/static using nginx. I wrote about thid issue in the wiki
(troubleshooting page). Let me know if that works.
On Tue, Jan 1, 2019, 21:25 Frank Claycomb<EMAIL_ADDRESS>wrote:

I am also having this issue. I created a symlink to make the /static
directory exist and contain the needed files however django still cannot
see them.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/chibicitiberiu/ytsm/issues/40#issuecomment-450751512,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE8dkSoAmDXwO7y8jMGtRSortIf1Pb8Wks5u-7YtgaJpZM4Y9FEj
.


hk.add_loss

To enable users to easily create per layer weight and activity regularizers plus other forms of losses created by intermediate layers it would be very useful if haiku had a hk.add_loss utility that when called within a transform it would append a loss to a list of losses which the user could later retrieve as an additional output from apply. I guess that this would require an additional flag to hk.transform and friends.

Hey @cgarciae , thanks for the FR! In general we try to keep Haiku fairly lean and encourage features (e.g. training loops, optimizers etc) to be solved in other libraries (then they can benefit all JAX users not just Haiku users) or built from existing Haiku/JAX features.
Wrt using existing features, you might consider using hk.set_state for this. This is a fairly general mechanism in Haiku for logging values associated with modules. You could use hk.data_structures.filter to extract all losses from the state dict:
def f(x):
  y = hk.nets.ResNet50(1000)(x, True)
  loss_1 = y.sum()
  loss_2 = loss_1 ** 2
  hk.set_state("loss_1", loss_1)
  return loss_2

f = hk.transform_with_state(f)

rng = jax.random.PRNGKey(42)
x = jnp.ones([1, 224, 224, 3])
params, state = f.init(rng, x)

# Apply as usual:
params, state = f.apply(params, state, rng, x)

# Extract losses from state:
is_loss = lambda m, n, v: n.startswith("loss")
losses = hk.data_structures.filter(is_loss, state)
print(losses)  # frozendict({'~': frozendict({'loss_1': DeviceArray(0., dtype=float32)})})

If you want to implement this as a standalone feature (e.g. decoupled from hk.set_state) then I've forked the following from @ibab who has implemented something similar (his version is more robust with thread safety and nesting).
from contextlib import contextmanager

loggers = []

@contextmanager
def context():
  data = {}
  loggers.append(data)
  try:
    yield data
  finally:
    loggers.pop()

def log(name, value):
  # NOTE: log(..) ignored when not logging.
  if loggers:
    data = loggers[-1]
    data[name] = value

def f(x):
  x = x ** 2
  log("a", x)
  x = x ** 2
  log("b", x)
  return x

def g():
  with context() as data:
    y = f(2)
  return y, data

y, data = g()
assert y == 16
assert data == {'a': 4, 'b': 16}

I hope that's useful, please feel free to reopen if this does not solve your usecase.


Installation problem

Hello everyone! I have a problem with asahi linux installation, what i should do?
./ You probably ran the installer once as root, then tried to run it again as your regular user.