gap jupyter

Checklist

[ ] Used a fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

The outputs section contained an unexpected subsection name. noarch is not a valid subsection name.

For recipe:

It looks like the 'gap-jupyter' output doesn't have any tests.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Organize Figma Foundations - Data Vis

Improving data vis and other pages on Foundations.

Check for a missing donut chart.


Obervability of state change

Hi,
this is a small PR that can be merged before #21, if you still need more time to review that.

It adds a way to subscribe a callback to any state change in a TreeNode. great for logging and debugging.

I added one more unit test where you can see how it works.

A minor change in ActionNode, that doesn't really affect much the overall behavior.

Cheers

Looks great! both the other PR, there are 3 pending questions (only 2 are actually meaningful)

Sorry... which pending questions? I am afraid I missed them :(


add model name before perform_skip

change line

location[0:5] = perform_skip(pdf_list, input_word)

to

location[0:5] = functioncaller.perform_skip(pdf_list, input_word)

Yes.
Sorry missed out on that.
Thank you very much. <3


Open extra file types with SCIFIO enabled

See https://github.com/imagej/imagej/issues/263

@ctrueden I think this PR is OK based on https://github.com/imagej/imagej/issues/263 discussion, but wanted your opinion on https://github.com/imagej/imagej-legacy/commit/f06818bc680cdbe5fc57d6ca3634f930fc77b2ab?


How can I link to other pages and keep localization?

Hi there! Really liking Enduro.js so far, it's awesome!
Well, I have 2 pages with 2 cultures (pt & en), and I understood that each culture adds a prefix in the URL.
How can I create a link from "/page-1" to "/page-2" that will work no matter if the prefix is "/pt" or "/en"?

Where do you place these links?
It can be programmed in template files or just written in both translations as needed in content files.

I'm trying to place them inside my template "page-1.hbs". Since I can't know for sure what locale is being applied, I don't know how I'm supposed to create a link to "page-2.hbs". I was hoping there would be an object to automatically apply the current locale.
Maybe I could do that by creating a custom helper that checks the URL, and if it matches a locale,   puts that prefix before the link. Is that the easiest way to accomplish that?

I'm not sure it is the easiest way, but yeah, I built such a feature by myself with helper functions:
Template
{{#if _meta.culture}}
    {{#ifeq fullpath '/index'}}
        {{linkTitle}}
    {{/ifeq}}
    {{#ifneq fullpath '/index'}}
        {{linkTitle}}
    {{/ifneq}}
{{else}}
    {{#ifeq fullpath '/index'}}
        {{linkTitle}}
    {{/ifeq}}
    {{#ifneq fullpath '/index'}}
        {{linkTitle}}
    {{/ifneq}}
{{/if}}

Helpers (assets\hbs_helpers)
/* global enduro */
enduro.templating_engine.registerHelper('ifeq', function (one, two, options) {
    return one == two? options.fn(this) : '';
});
enduro.templating_engine.registerHelper('ifneq', function (one, two, options) {
    return one != two? options.fn(this) : '';
});

Don't forget to replace ru in templates with your default language.

Thanks!

Hey @kikobr,
If your issue was solved, mark it as 'closed' for tracking purposes.

Sorry, I forgot about that. Thanks again.


function.dd: Table of Contents

https://github.com/D-Programming-Language/dlang.org/pull/1144 followup

I'd rather see it styled like the Jump to: menu used in Phobos: http://dlang.org/phobos/std_array.html
I believe that is using a list too, but styled as being horizontal.

I'd rather see it styled like the Jump to: menu used in Phobos: http://dlang.org/phobos/std_array.html

That's really more of a hack than a solution. Ideally we would be using summary tables / cheat sheets like those written manually for std.algorithm/std.range, or those generated by DDox. These horizontal lists are too dense to be usable, are difficult to scan (even though they're sorted you have to search two dimensions), their main benefit right now is that they're the first Ctrl+F occurrence that will lead to the sought symbol.

IMO this takes up way too much vertical space. It should be like the "Contents" box on Wikipedia pages, see example here: https://en.wikipedia.org/wiki/D_(programming_language).
Bonus points for making it hide-able.

@JackStouffer Do you want to take over this PR? I think this PR is still an improvement over the current intractable mess.

Do you want to take over this PR?

Sure, I'll whip something up tonight.

I think this PR is still an improvement over the current intractable mess.

It is an improvement, but I think that the correct solution here is to make something that is reusable on other pages, as I noticed when working on the overview page that it needed a contents section as well. The problem with putting something like this on other pages is that users decide in less than one second after the page is loaded whether they will stay on a web page or if they will do something else. Having the actual content of the page below the fold significantly reduces the chances that people will keep reading.

Sure, I'll whip something up tonight.

Any news?

@CyberShadow sorry it took so long: https://github.com/D-Programming-Language/dlang.org/pull/1151


SSH ProxyCommand support

I am trying Run House with a local pre-configured server. But that server needs to use "ProxyCommand" option to SSH into. Is there a way the PxoxyCommand can be specified in the Cluster API (like in the ssh_creds dict)?
Typical way to SSH into the server is something like this:
ssh -i  -o ProxyCommand="ssh -W %h:%p <user>>@<frontendproxyhost>" <user>@<targethost>>
I do have a workaround to add the ProxyCommand in ~/.ssh/config but would be nice to specify as params in the rh.cluster API for cases where the SSH command  are a bit dynamic (like in my case).

Thanks for bringing this up! This is something we should definitely support. Would adding a separate parameter for the proxy command work for your use case?
https://github.com/run-house/runhouse/commit/0dc953e6e89e5856e2c73b67e80605e418113e8f

We actually already support this, but haven't explicitly documented and tested it. Can you try adding ssh_proxy_command: {proxy string} to the ssh_creds dictionary?

Thanks for the tipcs @jlewitt1  and @dongreenberg . I tried to add the "ssh_proxy_command" to the ssh_creds dict. It seem to work (in terms of seeing an SSH connect) but then it threw an exception when RH tried to check connectivity again. This is the exception. I dont see this when I have the ProxyCommand  in my ~/.ssh/config. BTW - I have a couple of other SSH options (-o ) that I have in the config file which I did not have a way to pass in in the dict.
INFO | 2023-07-12 13:23:29,522 | Checking server myvm again.
---------------------------------------------------------------------------
BaseSSHTunnelForwarderError               Traceback (most recent call last)
File ~/miniconda3/envs/rh/lib/python3.9/site-packages/runhouse/rns/hardware/cluster.py:357, in Cluster.check_server(self, restart_server)
    356 try:
--> 357     self.connect_server_client()
    358     cluster_config = self.config_for_rns

File ~/miniconda3/envs/rh/lib/python3.9/site-packages/runhouse/rns/hardware/cluster.py:324, in Cluster.connect_server_client(self, tunnel, force_reconnect)
    323 else:
--> 324     self._rpc_tunnel, connected_port = self.ssh_tunnel(
    325         HTTPClient.DEFAULT_PORT,
    326         remote_port=DEFAULT_SERVER_PORT,
    327         num_ports_to_try=5,
    328     )
    329 open_cluster_tunnels[self.address] = (
    330     self._rpc_tunnel,
    331     connected_port,
    332     tunnel_refcount + 1,
    333 )

AttributeError                            Traceback (most recent call last)

gpu = rh.cluster(....)
File ~/miniconda3/envs/rh/lib/python3.9/site-packages/runhouse/rns/hardware/cluster_factory.py:59, in cluster(name, ips, ssh_creds, dryrun, **kwargs)
     50 if {"instance_type", "num_instances", "provider"} <= kwargs.keys():
     51     # Commenting out for now. If two creation paths creates confusion let's push people to use
     52     # ondemand_cluster() instead.
   (...)
     55     #     "If you would like to create an on-demand cluster, please use `rh.ondemand_cluster()` instead."
     56     # )
     57     return ondemand_cluster(name=name, **kwargs)
---> 59 return Cluster(ips=ips, ssh_creds=ssh_creds, name=name, dryrun=dryrun)

File ~/miniconda3/envs/rh/lib/python3.9/site-packages/runhouse/rns/hardware/cluster.py:58, in Cluster.__init__(self, name, ips, ssh_creds, dryrun, **kwargs)
     55 self.client = None
     57 if not dryrun and self.address:
---> 58     self.check_server()
     59     # OnDemandCluster will start ray itself, but will also set address later, so won't reach here.
     60     self.start_ray()

File ~/miniconda3/envs/rh/lib/python3.9/site-packages/runhouse/rns/hardware/cluster.py:379, in Cluster.check_server(self, restart_server)
    377     self.restart_server(resync_rh=False)
    378     logger.info(f"Checking server {self.name} again.")
--> 379     self.client.check_server(cluster_config=cluster_config)
    380 else:
    381     raise ValueError(f"Could not connect to cluster <{self.name}>")

AttributeError: 'NoneType' object has no attribute 'check_server'

Oh great point, we aren't passing the proxy into the tunnel. I can patch that and the options up shortly. Out of curiosity, you said this all works (through to running the remote function) when you've provided the options and proxy info in your SSH config?

Actually remote function also did not work with my ~/.ssh/config too. It hung for a long time. What worked was the setup of the functions (like creating the pip installs) and cluster.run_python.

Got it. I've been banging on this and the tunneling library we use (ironically to handle different credentials scenarios....) doesn't support proxies nicely (as in, it looks like it does, but I spent hours debugging and it still wouldn't proxy correctly despite working from the command line). I've implemented a fix going directly though the command line to remove that discrepancy, but I'll want to test it a bit further before releasing because it's a core execution path. Adding more ssh options is straightforward and I'll push that too shortly. If you're blocked and would like to give it a try so far, please feel free here:
pip install git+https://github.com/run-house/runhouse.git@proxy_tunneling

Thanks @dongreenberg  for the quick fix. I think there is still some issues after I used the proxy_tunneling branch. Good news is that the e2e (The stable diffusion tutorial) runs fine with remote functions etc when I use ~/.ssh/config to specify my ProxyCommand.
However when I use ssh_proxy_command dict item to pass that info I get some error in how the ssh /bash command is constructed. Seems like it is looking for SSH in my current dir.
/bin/bash: /home/user/runhouse/tutorials/t01_Stable_Diffusion/ssh -i ~/.ssh/id_rsa -W : -o StrictHostKeyChecking=no -o UserKnownHostsFile=/dev/null : No such file or directory
Then I see the remote pkill and start of the runhouse http server go thru fine. This is followed by a connection refused error.
INFO | 2023-07-13 13:11:47,807 | Checking server myvm again.
ConnectionRefusedError                    Traceback (most recent call last)
File ~/miniconda3/envs/rh/lib/python3.9/site-packages/urllib3/connection.py:174, in HTTPConnection._new_conn(self)
173 try:
--> 174     conn = connection.create_connection(
175         (self._dns_host, self.port), self.timeout, **extra_kw
176     )
178 except SocketTimeout:

Oh that is interesting. Glad to hear it works with the .ssh/config, and I appreciate you helping us through the dict case too. I think I spotted the error and just pushed a fix. It runs through on my side, but I've set up a phony jumpbox to test it, so it's really helpful that you've tried in on yours.

My environment is a bit custom (not sure how common it is). The target hostname (I pass in the ips field) is somewhat dynamic in nature (but follows some pattern which I specify in the .ssh/config) and wont resolve to any known IP address locally on my client machine and is only meaningful to the proxy host (which has a way to resolve these dynamic target hostname and route it correctly to my target  server). As a result if I dont use the ~/.ssh/config and let runhouse use the ssh_proxy_command, my proxy host somehow seems to not resolve the target  host passed in the -W option of proxycommand and returns a "Could not resolve IP address for  : Name or service not known".
The SSH client I run from command line passes the dynamic host name to the Proxy command (as I have -W %h:%p in the proxy command) and there I dont see my proxy failing to resolving the target.
For now,  I can use the ProxyCommand in ~/.ssh/config for now which seems to be working great for me with Runhouse as I was able to run several of the tutorials remotely. I am happy to add some logging on Runhouse locally to see how it is constructing the full SSH commands so I can check diff between my ssh/config setup vs passing ssh_proxy_command in rh.cluster. Where can I find that so I can debug it in my env to further isolate?

Good point, we should log the SSH commands, and I'm curious why they wouldn't be resolving the same way as through the command line. I have a gnarly commit in the works on a separate branch that I'll land shortly, and then add that logging and push to this branch. In general, would you say it's preferable for our ssh activity to run through the command line so it's consistent with whatever you know you can do directly, rather than use tools which depend on Python SSH tools (e.g. Paramiko, asyncssh)?


Creating Nether Issue

I tried creating a new nether and this happened.
I'm using the latest Spigot as well as the latest Multiverse and TerrainControl.
http://pastebin.com/bWWED6ju

Fixed.


useChecksum is incorrect

sha256 checksum is calculated incorrectly, workaround is "useChecksum: false" in options

done


Add SmtSpec for SMT proofs

This will be needed for implementing ADR-040 in Cosmos-SDK; specifically this PR depends on it: https://github.com/cosmos/cosmos-sdk/pull/10015

This will need a Go version bump, should it go to 0.6.8 or would 0.7.0 be more semver-appropriate?

@roysc bumping
I am happy to merge as soon as there are a couple test specs.
There is a new JSON format for them an all now (thanks to the work on solidity), which might be easier to use cross-language. But even adding a positive test in Go would be nice.

Sorry, I haven't had time to get to creating a proper serialization tool like ics23-iavl for the test data. I could look at just copying over some data as Go code in the meantime, but it's not particularly urgent yet to merge the spec into this repo, so I figured this could wait for the proper solution.

After testing more thoroughly, I found that the proof verification algorithm needs to be updated to check for empty child nodes in non-existence proof paths to support SMT proofs - see PR at https://github.com/confio/ics23/pull/61.
I've also written a ics23-smt  library to generate and check proof data (that depends on the above PR). I will rebase this branch on that fix and add the tests and data to complete it.

After testing more thoroughly, I found that the proof verification algorithm needs to be updated to check for empty child nodes in non-existence proof paths to support SMT proofs

😄 See I'm not just being annoying when asking for test cases... they do find issues.
I will review #61 now

Haha yes, the tests in the SDK were a bit too trivial it seems

Bumping this one to check the status.
Are you waiting for my review or am I waiting for more tests?

This is also ready for review, sorry for the delay. The JS part is not complete, but I will leave it in for now.

I've rebased and updated this now that the placeholder check is merged. I've also had to add a fix to the placeholder check logic, since it was being called with the wrong branch index. The fix is to detect the index from the padding within the empty check.


System Null Reference Exception while using a simple consumer

While testing out a consumer locally. I keep getting System.NullReferenceException
Exception: System.NullReferenceException Message: Object reference not set to an instance of an object. StackTrace: at Confluent.Kafka.Internal.Util.Marshal.PtrToStringUTF8(IntPtr strPtr) at Confluent.Kafka.Impl.SafeKafkaHandle.get_MemberId() at Confluent.Kafka.Consumer2.get_MemberId() at Test.Monitor.<>c.<Start>b__28_4(Object obj, List1 partitions) at Confluent.Kafka.Impl.LibRdKafka.NativeMethods.rd_kafka_destroy(IntPtr rk) at Confluent.Kafka.Impl.SafeKafkaHandle.ReleaseHandle() at System.Runtime.InteropServices.SafeHandle.InternalFinalize() at System.Runtime.InteropServices.SafeHandle.Dispose(Boolean disposing) at System.Runtime.InteropServices.SafeHandle.Finalize()
Error is occuring on OnParititionsRevoked even. At this point consumer.memberID is basically null.

Not sure what  could be causing this. Here is my memory usage

Here is my consumer code at this moment
_kafkaConsumer = new Consumer<Null, string>(consumerConfig, null,
new StringDeserializer(System.Text.Encoding.UTF8));
            _kafkaConsumer.OnMessage += (_, msg) =>
            {
                lock (Lock)
                {
                    LatestProcessedMessageOffset = msg.Offset;
                }
                ProcessMessage(msg);
            };

           _kafkaConsumer.OnError += (_, error) =>
            {
                                Logger.Error($"Error [{error}], Consumer{_kafkaConsumer.MemberId} on Server {Environment.MachineName}");
            };

            _kafkaConsumer.OnPartitionsAssigned += (obj, partitions) =>
            {
                var fromBeginning = partitions.Select(p => new TopicPartitionOffset(p.Topic, p.Partition,Offset.Beginning)).ToList();
                _kafkaConsumer.Assign(fromBeginning);
                Logger.Info($"Assigned partitions: [{string.Join(", ", partitions)}], Assigned to the Consumer{_kafkaConsumer.MemberId} on Server {Environment.MachineName}");

            };

            _kafkaConsumer.OnPartitionsRevoked += (obj, partitions) =>
            {
                _kafkaConsumer.Unassign();
                 Logger.Error($"Revoked partitions: [{string.Join(", ", partitions)}], Consumer{_kafkaConsumer.MemberId} on Server {Environment.MachineName}");
            };

            _kafkaConsumer.OnConsumeError += (_, msg) =>
            {
                //TODO: need to restart consumer subscription
                Logger.Error($"Error consuming from topic/partition/offset {msg.Topic}/{msg.Partition}/{msg.Offset}: {msg.Error},Consumer{_kafkaConsumer.MemberId} on Server {Environment.MachineName}");

            };

            _kafkaConsumer.Subscribe(config.Current.TARS.KafkaTopic_TestAttempt);

            while (true)
            {
                          _kafkaConsumer.Poll(TimeSpan.FromMilliseconds(100));
            }

Seems like Consumer is disposed from memory? Not sure how that happened

There is a registered event in the EventViewer. worker process gave up and app pool restarted automatically

Indeed there is a bug, we compare to null instead of IntPtr.Zero when converting MemberId
I don't know why you have a null memberId and dispose the safehandle though - more important bug. Can you try removing this MemberId  in your strings and see what happens?
Do you have any way to reproduce? (os, version, exact code, how many consumer you launch...)
Thanks!

Its a single consumer instance per each server(on which my app is running). We currently have two app servers so 2 consumer instances running. Seen this error in the event viewer of all severs. First I noticed this issue locally while doing some speed test.
1 Windows 7 (64 bit)
2. .NET 4.6.1
3. One kafka broker(running on my windows machine)
I did not notice this issue before. I have recently added logging on all the event handlers. I have noticed that OnPartitionedRevoked event is kind of stuck in an infinite loop or something because I keep getting call backs on that specific event because my logging statement has consumer.memberId(which was some how became null) is blowing up.
I'll test by removing memberId and see what happens.

I don't know why you have a null memberId and dispose the safehandle though - more important bug
I really don't know how this happened. I have not disposed my consumer instance explicitly it runs in an infinite loop. My consumer code is posted in my first comment. its plain and simple.

Here are my configuration parameters
private static Dictionary<string, object> GetConsumerConfig(string groupId, string brokerList, bool enableAutoCommit) =>
new Dictionary<string, object>
{
{ "group.id", groupId },
{ "enable.auto.commit", enableAutoCommit },
{ "auto.commit.interval.ms", 5000 },
{ "statistics.interval.ms", 60000 },
{ "bootstrap.servers", brokerList },
{ "default.topic.config", new Dictionary<string, object>()
{
{ "auto.offset.reset", "smallest" }
}
}
};

Thanks for feeback!
I wonder if it's not a problem in the pdb with unsafe code wrongly pointing the stack to releaseHandle - I think you indeed have a null memberId when partition are revoked due to coordintor mechanism (will have to check on librdkafka). You should be able to call MemberId before the Unassign, and the bug when calling it after Unassign will be fixed in #222 (but indeed, it's a problematic bug).
Also, why do you log the PartitionRevoked as an error? This is a normal behaviour - kafka mechanism with coordinator is to unsubscribe to all partition and reassign. If you have one consumer started and start a second one, the first one will receive a PartitionRevoked, then PartitionAssigned

@treziac . I have added logging to monitor whats going on behind the scenes. Its just a log in my case . I'll make that informational. I'll move the code of line and put it before I do the unassign call.

closing as #222 was merged. it's not clear to me if other discussion in this thread represents a problem with the library or it's use. Since this issue was reported we've changed some things that may be relevant - added object disposed checking to kafkaHandle methods (including MemberId) and fixed some issues with finalization. please open a new issue and reference this one if problems persist.


client request configuration

I think it would be better to set the client request url as a settable parameter since we need to replace all "http://localhost:3080" if the client and api are in different machine.

Yeah, probably best to set both SERVER_URL and CLIENT_URL as configurable parameters.

#26 replace all localhost:3080 request to relative path.
but I agree that SERVER_URL and CLIENT_URL are better design


Remove unnecessary dependency

This dependency is already provided by Jenkins core, so an explicit declaration is unnecessary and simply increases the maintenance burden of the plugin.

thx gyus


Important note doesn't render

Issue in ./help/components/filters/filters-overview.md
Incorrect Important syntax on this page.

I have fixed this and it's currently publishing.


Updating pom.xml

Changes version of http-client from 1,0.0 to 1.0.0-SNAPSHOT as it is only avaiable under this name in the Sonatype Repo.

According to https://mvnrepository.com/artifact/org.javawebstack/http-client , version 1.0.0 should be available on the Maven Central.

Like @x7airworker already mentioned, it's available on central. OSSRH snapshots is only required because the most recent snapshot of abstract-data contains new features and isn't yet released. If you are using gradle, make sure to add mavenCentral() to your repository declaration.


lint and fmt cause os error 2 after upgrade

Steps to Reproduce
I really have a feeling I've messed something up... but I have been scratching my head about this the last two days and maybe I'm missing something.
I ran rye self update and then rye lint and rye fmt now cause the following:
❯ rye fmt
error: No such file or directory (os error 2)

I ran rye self uninstall and then tried reinstalling and the same happend.

install rye via curl -sSf https://rye.astral.sh/get | bash
echo 'source "$HOME/.rye/env"' > ~/.zshrc
rye init testing123 && cd testing123
rye lint (or rye fmt)

Expected Result
Should run ruff fmt and ruff lint
Actual Result
❯ rye fmt
error: No such file or directory (os error 2)

Version Info
rye 0.38.0
commit: 0.38.0 (3e3c8540f 2024-08-02)
platform: macos (aarch64)
self-python<EMAIL_ADDRESS>symlink support: true
uv enabled: true

Stacktrace
No response

Apologies if there isn't enough information here to reproduce, but I'm not sure what other information I could provide that would be relevant.
❯ npx envinfo --system --languages --managers

  System:
    OS: macOS 14.5
    CPU: (14) arm64 Apple M3 Max
    Memory: 2.85 GB / 36.00 GB
    Shell: 5.9 - /bin/zsh
  Managers:
    Cargo: 1.78.0 - ~/.cargo/bin/cargo
    pip3: 21.2.4 - /usr/bin/pip3
    RubyGems: <IP_ADDRESS> - /usr/bin/gem
  Languages:
    Bash: 3.2.57 - /bin/bash
    Perl: 5.34.1 - /usr/bin/perl
    Python: 3.12.4 - /Users/manzt/.rye/shims/python
    Python3: 3.9.6 - /usr/bin/python3
    R: 4.4.1 - /usr/local/bin/R
    Ruby: 2.6.10 - /usr/bin/ruby
    Rust: 1.78.0 - /Users/manzt/.cargo/bin/rustc

Perhaps of note, I get the following erro while installing rye:
~/demos on  main [!?]
❯ curl -sSf https://rye.astral.sh/get | bash
This script will automatically download and install rye (latest) for you.
######################################################################## 100.0%
Welcome to Rye!

This installer will install rye to /Users/manzt/.rye
This path can be changed by exporting the RYE_HOME environment variable.

Details:
  Rye Version: 0.38.0
  Platform: macos (aarch64)

✔ Continue? · yes
Installed binary to /Users/manzt/.rye/shims/rye
Bootstrapping rye internals
Downloading<EMAIL_ADDRESS>Checking checksum
Unpacking
Downloaded<EMAIL_ADDRESS>error: No virtual environment found for executable name `python`
error: No virtual environment found for executable name `python`
Updated self-python installation at /Users/manzt/.rye/self

The rye directory /Users/manzt/.rye/shims was not detected on PATH.
It is highly recommended that you add it.
✔ Should the installer add Rye to PATH via .profile? · no
note: did not manipulate the path. To make it work, add this to your .profile manually:

    source "$HOME/.rye/env"

To make it work with zsh, you might need to add this to your .zprofile:

    source "$HOME/.rye/env"

For more information read https://rye.astral.sh/guide/installation/

All done!

now having an issue with upgrading:
❯ rye self update
Updating to latest
Checking checksum
error: Unable to perform update. This can happen because files are in use. Please stop running Python interpreters and retry the update.

Caused by:
    No such file or directory (os error 2)

Ok, I think I sorted this out.
I have UV_PYTHON=python as an environment variable.... but that python comes from Rye, so something happened when upgrading where python is momentarily not available and breaks the rye install (now that rye install is partially bootsrapped with uv?)


Tessa not working on Google Colab

I am in a simple colab environment, and can't get the pypi install to work, let me know if you understand whats going on.

Strange. That's from a Jupyter notebook? What happens if you leave out the version specification?

Same problem when I leave out the version specification, can you replicate it on Colab?

I gave this a try now and I get the same error as you in Colab. I assume the issue is because Colab runs Python v3.6 whereas tessa currently requires v3.9. – I will check if I can make this more lenient. See also https://github.com/ymyke/tessa/issues/8

Turns out that Google Colab runs Python 3.7. I just released v0.3.1 which should work with >=3.7, <3.10. I tested with Colab and it works there too.
(Nonrelated, but maybe useful in case you ever need to upgrade the Python environment in Colab: https://stackoverflow.com/questions/68657341/how-can-i-update-google-colabs-python-version)

I published v0.3.2 just now, which should finally work in Colab. Can you confirm, @danilogalisteu ?

v0.3.2 is working in Google Colab with Python 3.7.13. I'm not able to test on Python 3.9 at the moment. Thanks!

Great, thanks!


GPG error on mvn clean install

Keep getting a MojoExecutionException error due to Maven not being able to grab dependency: org.apache.maven.plugins:maven-gpg-plugin

I've looked into this... it's set up as:
		<plugin>
			<groupId>org.apache.maven.plugins</groupId>
			<artifactId>maven-gpg-plugin</artifactId>
			<version>1.5</version>
			<executions>
				<execution>
					<id>sign-artifacts</id>
					<phase>verify</phase>
					<goals>
						<goal>sign</goal>
					</goals>
				</execution>
			</executions>
		</plugin>

When I execute "mvn clean install" it doesn't generate signed files.  It only runs when I do "mvn verify". So I'm wondering why it's asking people who don't have gpg installed for it to be installed.

My plan is to redo how the build is deployed. Hopefully it will remove any further isses.


the obfuscated codes doesn't get beauty anymore

the obfuscated codes are doesn't get beauty anymore, there is a problem on reading variables. https://javascriptobfuscator.com/Javascript-Obfuscator.aspx

Please be more specific.


Remove autobuild from codeql

Based on what I'm reading here, autobuilding is something that can be removed for projects that have a different building method. Based on this information, we should be ok to remove it; which should also speed up the codeql process.

Also the fact that prompting to create a new workflow, comments out the autobuild code:

Managed to shave off roughly 2 minutes by removing this (somehow) - according to the python test on this pr and the current master build. Could just be a fluke though


Map GCReW met to GCW-W

This PR adds metadata needed to map GCReW met data into the GCW-W L1 data. @wilsonsj100 I would appreciate your careful eyes on this to make sure sensible and intuitive.
We are mapping

wx_gcrew_rain15   <-----
wx_tempavg15
wx_rh15
wx_vappress15
wx_vpd15   <-----
wx_svp15   <-----
wx_barpress15
wx_slr_fd15
wx_slr_tf15
wx_par_den15
wx_par_tot15
wx_winddir15
wx_maxws15
wx_svp15   <-----
wx_windspeed15

These are all the GCReW met outputs that map cleanly to COMPASS-FME met variables. There are three (wx_vpd15, wx_svp15, and wx_svp15) that do NOT appear at other synoptic sites but seemed useful so I included them.
There also is wx_gcrew_rain15. This is the rain output from GCReW met and is in cm, whereas the COMPASS sites' outputs are in mm. As a result, we send it to a NEW variable with a very distinctive name and you'll need to handle it specially.
The GCW metadata file now includes an explicit warning about this:

GCW_W_20230308-20230322_L1_v1-0.csv

Necessary for #158 but I need access to GCReW met raw data before closing that

Feedback still and always welcome @wilsonsj100 , but I want to merge this to easily proceed with other things.


For Spring Boot projects, do not recommend api dependencies.

Resolves https://github.com/autonomousapps/dependency-analysis-android-gradle-plugin/issues/222.

Build failed https://gradle.com/s/beitusvoda6xu


ZEPPELIN-61 Lens Interpreter

Add Lens interpreter. #100 has implementation from @praagarw.
This pr grabbing a necessary commits from #100 and rebase from current master branch.

Ready to merge

Merging if there is no more discussions


resize_handler: Convert module to Typescript

This PR migrates resize_handler.js to resize_handler.ts.

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[x] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

@Temidayo32 please go through the self-review checklist and make sure your commits follow the contributing guidelines.
Check out the Zulip commit message guidelines for more details.

@Temidayo32 please go through the self-review checklist and make sure your commits follow the contributing guidelines. Check out the Zulip commit message guidelines for more details.

Thanks for pointing this out 😃. I have made the changes

This is failing the linter; I think because #29082 isn't merged yet?

This is failing the linter; I think because #29082 isn't merged yet?

Yes, that's the reason

Merged, thanks @Temidayo32!
Though one small note: TypeScript is typically spell with the S capitalized.


[QUESTION]Is there any bigwig willing to start a major new branch

Is there any bigwig willing to start a major new branch.
Currently the project is not well maintained and tested.
Full of bugs, and not resolved in a timely manner.
Of course, I cannot deny that the author's contribution is great. Thanks to the author for his dedication and efforts.

Hey @mbdavid i have same question. If you don't have time\wish to continue maintain this project, i think this will be pretty easy to find 1-2 new maintainers to do it.


Consider ipyparallel

Seems like a potentially easy way to set up an SSH cluster:
https://ipyparallel.readthedocs.io/en/latest/tutorial/process.html#starting-a-cluster-with-ssh

Doesn't look that simple, actually. Will direct users to coiled instead.


vmdk_ops does not report attached volumes correctly: inspect volume and vmdk_ops list volumes results are different and getting error "vm doesn't belong to any tenant"

Environment Details:
#VMs: 2 photon vms (VM1, VM2)
ESX: VMware ESXi 6.0.0 build-3620759
Datastore: vmfs
Steps to Reproduce:

docker volume create --driver=vmdk --name=vmwDataVol -o size=10gb (on VM1)

root@photon-EHiCjpUvt [ ~ ]# docker volume create --driver=vmdk --name=vmwDataVol -o size=10gb
vmwDataVol

make sure volume is visible to another vm (VM2) and inspecting volume should say 'detached'

root@photon-w3ktCykjv [ ~ ]# docker volume ls
DRIVER              VOLUME NAME
vmdk                vmwDataVol

docker run --rm -it -v vmwDataVol:/mnt/myvol2 busybox
docker volume inspect vmwDataVol (on VM2)

[
    {
        "Name": "vmwDataVol",
        "Driver": "vmdk",
        "Mountpoint": "/mnt/vmdk/vmwDataVol",
        "Status": {
            "access": "read-write",
            "attach-as": "independent_persistent",
            "attached to VM": "photon2",
            "capacity": {
                "allocated": "155MB",
                "size": "10GB"
            },
            "created": "Wed Nov 16 18:57:57 2016",
            "created by VM": "photon2",
            "datastore": "datastore",
            "diskformat": "thin",
            "fstype": "ext4",
            "status": "**attached**"
        },
        "Labels": {},
        "Scope": "global"
    }
]

on ESX: list volumes

[root@promc-2n-dhcp105-97:/vmfs/volumes/4fc608aa-828f03c0-d4a2-d4ae52794a7a] /usr/lib/vmware/vmdkops/bin/vmdkops_admin.py ls
Volume           Datastore  Created By VM  Created                   Attached To VM  Policy  Capacity  Used      Filesystem Type  Access      Attach As
---------------  ---------  -------------  ------------------------  --------------  ------  --------  --------  ---------------  ----------  ----------------------
vmwDataVol       datastore  photon2        Wed Nov 16 18:57:57 2016  detached        N/A     10.00GB   175.00MB  ext4             read-write  independent_persistent

Expected Result:
while listing down volume, vmdkops command should say vmwDataVol is attached to VM1
Actual Result:
[root@promc-2n-dhcp105-97:/vmfs/volumes/4fc608aa-828f03c0-d4a2-d4ae52794a7a] /usr/lib/vmware/vmdkops/bin/vmdkops_admin.py ls
Volume           Datastore  Created By VM  Created                   Attached To VM  Policy  Capacity  Used      Filesystem Type  Access      Attach As
---------------  ---------  -------------  ------------------------  --------------  ------  --------  --------  ---------------  ----------  ----------------------
vmwDataVol       datastore  photon2        Wed Nov 16 18:57:57 2016  **detached**        N/A     10.00GB   175.00MB  ext4             read-write  independent_persistent

Triage:
---------------vmdk_ops.log------------------
11/16/16 18:57:56 1463911 [photon2-vmwDataVol] [INFO   ] executeRequest 'get' completed with ret={u'Error': 'Volume vmwDataVol not found (file: /vmfs/volumes/datastore/dockvols/vmwDataVol.vmdk)'}
11/16/16 18:57:56 1463911 [photon2-vmwDataVol] [INFO   ] *** createVMDK: /vmfs/volumes/datastore/dockvols/vmwDataVol.vmdk opts = {u'size': u'10gb', u'fstype': u'ext4'}
11/16/16 18:57:58 1463911 [photon2-vmwDataVol] [WARNING]  VM photon2 does not belong to any tenant
11/16/16 18:57:58 1463911 [photon2-vmwDataVol] [INFO   ] executeRequest 'create' completed with ret=None
11/16/16 18:57:58 1463911 [photon2-vmwDataVol] [INFO   ] *** attachVMDK: /vmfs/volumes/datastore/dockvols/vmwDataVol.vmdk to photon2 VM uuid = 564de5f9-3b8e-bc8c-96a6-eb359f4e9714
11/16/16 18:57:58 1463911 [photon2-vmwDataVol] [INFO   ] Attaching /vmfs/volumes/datastore/dockvols/vmwDataVol.vmdk as independent_persistent
11/16/16 18:57:59 1463911 [photon2-vmwDataVol] [INFO   ] Disk /vmfs/volumes/datastore/dockvols/vmwDataVol.vmdk successfully attached. controller pci_slot_number=192, disk_slot=1
11/16/16 18:57:59 1463911 [photon2-vmwDataVol] [INFO   ] executeRequest 'attach' completed with ret={'ControllerPciSlotNumber': '192', 'Unit': '1'}
11/16/16 18:58:15 1463911 [photon2-vmwDataVol] [INFO   ] *** detachVMDK: /vmfs/volumes/datastore/dockvols/vmwDataVol.vmdk from photon2 VM uuid = 564de5f9-3b8e-bc8c-96a6-eb359f4e9714
11/16/16 18:58:16 1463911 [photon2-vmwDataVol] [INFO   ] Disk detached /vmfs/volumes/datastore/dockvols/vmwDataVol.vmdk
11/16/16 18:58:16 1463911 [photon2-vmwDataVol] [INFO   ] executeRequest 'detach' completed with ret=None
11/16/16 18:58:39 1463911 [photon2-vmwDataVol] [INFO   ] executeRequest 'get' completed with ret={'status': u'detached', 'created by VM': u'photon2', 'capacity': {'allocated': '145MB', 'size': '10GB'}, 'c
11/16/16 18:58:39 1463911 [photon2-vmwDataVol] [INFO   ] *** attachVMDK: /vmfs/volumes/datastore/dockvols/vmwDataVol.vmdk to photon2 VM uuid = 564de5f9-3b8e-bc8c-96a6-eb359f4e9714
11/16/16 18:58:39 1463911 [photon2-vmwDataVol] [INFO   ] Attaching /vmfs/volumes/datastore/dockvols/vmwDataVol.vmdk as independent_persistent
11/16/16 18:58:40 1463911 [photon2-vmwDataVol] [INFO   ] Disk /vmfs/volumes/datastore/dockvols/vmwDataVol.vmdk successfully attached. controller pci_slot_number=192, disk_slot=1
11/16/16 18:58:40 1463911 [photon2-vmwDataVol] [INFO   ] executeRequest 'attach' completed with ret={'ControllerPciSlotNumber': '192', 'Unit': '1'}
11/16/16 18:58:48 1463911 [photon1-socket7] [INFO   ] executeRequest 'list' completed with ret=[{u'Attributes': {}, u'Name': 'refCountTestVol'}, {u'Attributes': {}, u'Name': 'datavol1'}, {u'Attributes': {
11/16/16 18:58:56 1463911 [photon1-vmwDataVol] [INFO   ] executeRequest 'get' completed with ret={'status': u'attached', 'attached to VM': 'photon2', 'created by VM': u'photon2', 'capacity': {'allocated':
~

Needs to get the VM name from the attached VM UUID in the KV. I'll fix this

Fixed via #745


Suggestion: Ability to turn on/off advanced oil processing & kovarex enrichment process

It would be really nice to be able to tell the calculator to omit using advanced oil processing and kovarex enrichment process (and maybe some other technologies as well, but those two are most important, I think) for example with two simple checkboxes. These technologies obviously require throwing some science packs at them, and are often not yet discovered when build some mid-game setups. Right now calculator (almost?) always goes for using those technologies (for example when calculating nuclear reactor cycle), which makes it hard to compute actual number of required drills/centrifuges/factories when you have not yet researched those technologies.
BTW - You have made one hell of a tool!

Added option to change the oil processing recipe in 1cbf8d9b957d3c9688ad46c6f414bbdc0bbd6068.

Added option to disable Kovarex processing in e5b74e42958a93da75d563ab66492beea1ff17d8.


Improve Q10 by using Map Reduce on Hive instead of UDTF

By running a map reduce task on the product reviews table, time to completion improved by 55% on 3 TB data set.

Zhou Yi has an updated version


NOT an issue, inquiry: how much are the duels modifiable?

How much are we able to inject things into the duel? for example is it within the realm of possibility to add a button that draws you or your opponent one card whenever pressed? or a similar button that allows an additional normal summon? how about buttons to manipulate LP?
Or is our ability to manipulate the client not that strong?

Duels aren't very moddable. While everything you mentioned is technically feasible you'd need to write a ton of code to support that type of thing including bot Unity UI mods and duel.dll edits. This isn't something I'll attempt.


docs: Switch to our own fork of sphinxcontrib-openapi

Sphinxcontrib-openapi relies on m2r. But m2r is unmaintained. It relies on mistune, but is not compatible with mistune versions >= 2.0. This makes it impossible to update mistune, even though there is a CVE in the old versions of the package.
Switch to sphinx-mdinclude instead.
There is also m2r2, but it is not a suitable alternative because they simply force the use of an older mistune version.
I built locally to check that the api.html page is identical before and after this PR.

API reference preview

I can't preview the link that you shared above, but assuming you're OK to merge this then I'm OK with it. Please add ready-to-merge if so :+1:


How to specify pretrained model ?

Hello, how to specify pretrained model in train_cub.yaml ?

Hi, @PapaMadeleine2022, nice to see you here again :)
The provided config is already using pretrained model of general purpose -- DINO
More models here and here.

I mean how to specify a local weights path.  I have done it. Modifying only the weights config path in  train_cub.yaml is ok.

yep, that's right
you can check training and validation configs on CARS example, where I left a lot of comments regarding every part of the configs


fix(from-entries): do not allow non supported array types

Current dependencies on/for this PR:

main

PR #89

PR #90   👈

This comment was auto-generated by Graphite.

@pevisscher started a stack merge that includes this pull request via Graphite.

Graphite rebased this pull request as part of a merge.

@pevisscher merged this pull request with Graphite.


Doesn't receive notification when in release mode and linker Sdk and User Assemblies

Hi. When I use Sdk Assemblies only , everything is working fine , but when I change linker option to link all, app doesn't receive notification at all!
Is anything most preserve or configure or skip?
I have tested in real device and emulator.

Did you uninstalled the debug version before installing release or increment your version number once you change to release?

I increment version number when publish it in release mode , but I dont sure that I uninstall debuge version before installing release mode.

@rdelrosario I can run my app and receive notification in release mode in sdk only.
My problem is when I choose linker option to linkAll !

I don't think this has to do with the plugin, this seems more a linking issue.

@rdelrosario Do you know how can i find any instruction or solution to solve it ?

This is on iOS or Android?

Android

Did you check these:
https://developer.xamarin.com/guides/android/advanced_topics/linking/
https://developer.xamarin.com/guides/android/deployment,_testing,_and_metrics/publishing_an_application/part_1_-_preparing_an_application_for_release/#Configure_the_Linker

@rdelrosario Thank you for your guidance. I investigated my build and I figured out it wasn’t linker problem. It was a connection problem to FCM.
In setup guide it says:
Important: While debugging set resetToken parameter to true.
Why we must not reset token in release mode ?

Because you don’t want to reset the token each time when released only when is actually needed.
When version number increases or token expired.

I solved my problem by linkskip your plugin assembly and firebase and google play services assemblies .
It's complicated when testing firebase by emulator. I tested it on real device and everything going back to normal, thank you for your support.


Adding changes to release.yml and readme to release collection

Adding changes to release.yml and readme to release collection

@sabre1041 I have updated with the changes, please take a look.

Pushed changes for readme, updated collection in playbook.yml


update readme for simple installation

If you want to use the commandline version, then this will do the trick:
go install github.com/gojp/goreportcard/cmd/goreportcard-cli
Perhaps you can put that in the readme (because then there's no need to get the software locally via git)

great work!!!


helm chart: fix broken default values

Fixes #1315, introduced in 3166d43.
Should be noted, that the current v5.5.0 helm chart is completely broken because of this. So a patch release might be necessary :)

Thanks for your quick help @tamcore , I should have spotted this, but apparently I didn't.
I will fix a release staright away.


does this project use semver?

You recently made a breaking change to the public API by making a number of methods private and only bumped the patch number. Should we expect future changes like this without a major version bump?

hi @akempe , sorry about that and thanks for bringing it up, we will follow the semver more strictly in the future.


Workaround for Jquery CSS method not working for safari browsers

Workaround for Jquery CSS method not working for safari browsers when applying CSS3 transform property. Vendor prefix needed for applying  css property.

-webkit- vendor prefix added for applying CSS3 transform property. Otherwise the rotate functionality was not working in older safari browsers. Not sure whether newer safari versions support transform without webkit prefix.

http://api.jquery.com/css

As of jQuery 1.8, the .css() setter will automatically take care of prefixing the property name. For example, take .css( "user-select", "none" ) in Chrome/Safari will set it as -webkit-user-select, Firefox will use -moz-user-select, and IE10 will use -ms-user-select.

Sure. I had to use this workaround because I'm working on an application that uses less jQuery 1.7.x. and upgrading jQuery didn't seem to be feasible for us right now.

I suggest you to hold this in your own branch as the plugin only support jQuery version > 1.9.1.

I see. Right. That makes sense.


python: add record size limit

Changelog

Added: the python MCAP library reader classes gained a new parameter, record_size_limit, defaulting to 4GiB. This allows callers to limit the size of records that their application will support. When a record is encountered with a greater length, the reader will raise an InvalidRecordLength exception. This limit can be removed by setting it to None. This helps applications avoid the issue where a corrupt MCAP can cause a MemoryError.

Docs
See generated python docs in the mcap.dev preview.
Description

BeforeAfter

Fixes: #1220

@indirectlylit nope updated. Thanks for the review!


Python SDK: Refactoring response from the server and adding warning logs

Example code to get data object and configure warning log level:
storage = FileKeyValueStorage(config_file)

secrets_manager = SecretsManager(
    token=token,
    config=storage,
    log_level="DEBUG"
)

all_records = secrets_manager.get_secrets(full_response=True)

print("App name: " + all_records.appData.title)
print("Access expires on: " + (str(all_records.expires_on_str()) if all_records.expiresOn is not 0 else "Never"))

for r in all_records.records:
    print(r)


Why no silhouette loss needed

Hi, I notice that in your paper it is emphasized that silhouette loss is not needed (which is a really nice feature!).
However, I also notice that all most all of the recent differentiable mesh renderer requires silhouette loss along with other bunch of regularizer to prevent the generation of self-intersecting faces.
I am curious about why this is the case.
Can dirt at the same time optimize mesh shape and mesh vertex color (texture with non-uniform color) without silhouette loss (in the paper only mesh shape optimization is shown)?
Is there anywhere that I can find a more detailed introduction of the technique that you employed to build dirt?
Thanks!

In short, yes, regularisation (or careful parameterisation) is important to avoid nasty self-intersecting meshes. In [1], we use local smoothness regularisers, and also show a parameterisation that avoids self-intersections a priori. We can avoid a silhouette loss even on natural images with texture by restricting the space of 'allowed' shapes for the foreground to live in some low-dimensional (learnt) manifold, using a VAE setup. Note that these techniques belong to the model rather than the renderer (DIRT) itself. Hence, I think that the same techniques will work with renderers other than DIRT (at least, the ones that have reasonably well-behaved derivatives everywhere) -- it does not have any particularly unusual properties.
The only reference for how DIRT works is a small section of my PhD thesis [2].
[1] Leveraging 2D data to learn textured 3D mesh generation, Henderson et al. CVPR 2020, https://arxiv.org/abs/2004.04180
[2] Section 3.4 of https://era.ed.ac.uk/handle/1842/35600

In short, yes, regularisation (or careful parameterisation) is important to avoid nasty self-intersecting meshes. In [1], we use local smoothness regularisers, and also show a parameterisation that avoids self-intersections a priori. We can avoid a silhouette loss even on natural images with texture by restricting the space of 'allowed' shapes for the foreground to live in some low-dimensional (learnt) manifold, using a VAE setup. Note that these techniques belong to the model rather than the renderer (DIRT) itself. Hence, I think that the same techniques will work with renderers other than DIRT (at least, the ones that have reasonably well-behaved derivatives everywhere) -- it does not have any particularly unusual properties.
The only reference for how DIRT works is a small section of my PhD thesis [2].
[1] Leveraging 2D data to learn textured 3D mesh generation, Henderson et al. CVPR 2020, https://arxiv.org/abs/2004.04180
[2] Section 3.4 of https://era.ed.ac.uk/handle/1842/35600


Connect plotting and signal retrieving

using sockets and python interpret functions

In the commit https://github.com/LorenzoMinutolo/SigViz/commit/7cbdd7c438f3a1d17f5ffbfd85c084bb79e4e754 there is the creation of the signals and kinds variables for the plotting.
The variables are created here
https://github.com/LorenzoMinutolo/SigViz/blob/7cbdd7c438f3a1d17f5ffbfd85c084bb79e4e754/templates/plotter.html#L8-L14
using Jinja2 nested loops and reflect the template defined here https://github.com/LorenzoMinutolo/SigViz/blob/7cbdd7c438f3a1d17f5ffbfd85c084bb79e4e754/static/js/plotter_view.js#L45-L47 except for the fact that now we have a triple nested list as signals[plot][row/col/pol][detector#]

multi windows sync works with commit https://github.com/LorenzoMinutolo/SigViz/commit/e49947bdbe2c863953fa70f54780640c1be896d0
I'm closing this


Add gauge to show the daily temp of Hanselman's tweets with sentiment analysis

this is huge.

I like this, would be a good contender for an Azure Function in the backend that runs on occasion.


Incorrect transaction date

Transaction dates are often not displayed correctly at Nault startup. [Reload] or [Force Reload] will sometimes correct the date column but will also inconsistently revert dates back to the wrong ones:

Comparison of transaction dates, incorrect data encircled in red on the left and the correct dates are visible on the right.
Only transactions on or before 2020-05-10 (May 10th) seem to be affected, these transactions are all incorrectly time-stamped 2020-05-10. I have tested 6 different accounts and 4 different wallets (including one Nano Ledger X) on 3 different systems, all displaying the same behavior.
(Nano Desktop v1.5.2 on Windows 10 Home v2004, Windows 10 Pro v1909 and Linux Mint v20 (Cinnamon) )

Thanks for your report. I see the same behavior. After some digging it seems like the timestamps that come from the server are actually incorrect. If your server is set to random that would explain the inconsistencies. I see a lot of May 2020 dates coming from MyNanoNinja. I also see alot of 0 timestamps (01-01-1970) from both Nanos.cc and NanoCrawler which we should probably handle as 'N/A'. Nanex.cc seems to be the most accurate.
Does it fix your problem if you set your server config to Nanex.cc instead of random?

@aleksre Thank you for investigation and pointing me in the right direction. My server was indeed set to "Random" hence the inconsistencies. I tried all servers from the list and Nanos.cc, Nanex.cc and NanoCrawler all return the correct dates. I can only reproduce the wrong May 10th transaction dates when using "My Nano Ninja". I have seen the 0 timestamp (01-01-1970) several times in the Nanocrawler block explorer but have not encountered it yet on my systems.

10th May is probably the date when I set up the new node and did a full sync. I don't know if there is currently a proper way of importing dates to a node.
Maybe we should clarify this on the frontend as well?

Won't fix, that's the only date we know. Adding another source for dates is probably overkill.
The node owner could fix the dates from a backup.


Fix of issue 828 - Missing @ sign in Razor view prints texts instead of value

Hi, So I just figured this to be the issue so I forked the code, fixed it, tested it and now it works.

Thanks


Debug an error "notEnoughGold"

Now when we have many users from Coinbase, this issue happens the most often and takes total 21% of all issues in the dashboard:
https://metabase.celo-networks-dev.org/dashboard/15

Fixed with the PR #452


ref 4671

ref #4671 .
Changes proposed in this pull request:

fix mysql ColumnDefinitionSegment,change type of dataType from String to DataTypeSegment

@terrymanu CI failed for the same reason: "t_log not found",help take a look.

Hi why is CreateTableStatement completely created in this PR? The same one has already existed in ShardingSphere, which makes me confused. Could you give me some hints?
@jingshanglu

Pull Request Test Coverage Report for Build 10370

0 of 45 (0.0%)  changed or added relevant lines in 7 files are covered.
50 unchanged lines in 7 files lost coverage.
Overall coverage decreased (-0.06%) to 59.689%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

shardingsphere-sql-parser/shardingsphere-sql-parser-statement/src/main/java/org/apache/shardingsphere/sql/parser/sql/segment/ddl/dialect/mysql/AddColumnDefinitionSegment.java
0
1
0.0%

shardingsphere-sql-parser/shardingsphere-sql-parser-statement/src/main/java/org/apache/shardingsphere/sql/parser/sql/segment/ddl/dialect/mysql/ModifyColumnDefinitionSegment.java
0
1
0.0%

shardingsphere-sql-parser/shardingsphere-sql-parser-statement/src/main/java/org/apache/shardingsphere/sql/parser/sql/segment/generic/dialect/mysql/DataTypeLengthSegment.java
0
1
0.0%

shardingsphere-sql-parser/shardingsphere-sql-parser-statement/src/main/java/org/apache/shardingsphere/sql/parser/sql/segment/generic/dialect/mysql/DataTypeSegment.java
0
1
0.0%

shardingsphere-sql-parser/shardingsphere-sql-parser-dialect/shardingsphere-sql-parser-mysql/src/main/java/org/apache/shardingsphere/sql/parser/mysql/visitor/impl/MySQLDDLVisitor.java
0
2
0.0%

shardingsphere-sql-parser/shardingsphere-sql-parser-statement/src/main/java/org/apache/shardingsphere/sql/parser/sql/segment/ddl/dialect/mysql/ColumnDefinitionSegment.java
0
8
0.0%

shardingsphere-sql-parser/shardingsphere-sql-parser-dialect/shardingsphere-sql-parser-mysql/src/main/java/org/apache/shardingsphere/sql/parser/mysql/visitor/MySQLVisitor.java
0
31
0.0%

Files with Coverage Reduction
New Missed Lines
%

sharding-scaling/sharding-scaling-mysql/src/main/java/org/apache/shardingsphere/shardingscaling/mysql/binlog/MySQLNegotiateHandler.java
1
95.24%

sharding-orchestration/sharding-orchestration-core/sharding-orchestration-core-registrycenter/src/main/java/org/apache/shardingsphere/orchestration/core/registrycenter/util/IpUtils.java
3
76.0%

shardingsphere-database-protocol/shardingsphere-database-protocol-mysql/src/main/java/org/apache/shardingsphere/database/protocol/mysql/packet/command/query/text/query/MySQLComQueryPacket.java
3
62.5%

sharding-sql-test/src/main/java/org/apache/shardingsphere/test/sql/loader/SQLCasesLoader.java
6
79.01%

shardingsphere-underlying/shardingsphere-common/src/main/java/org/apache/shardingsphere/underlying/common/config/TypeBasedSPIConfiguration.java
7
0%

sharding-scaling/sharding-scaling-mysql/src/main/java/org/apache/shardingsphere/shardingscaling/mysql/binlog/packet/binlog/RowsEventPacket.java
12
81.05%

sharding-scaling/sharding-scaling-mysql/src/main/java/org/apache/shardingsphere/shardingscaling/mysql/binlog/MySQLConnector.java
18
70.79%

Totals

Change from base Build 1089:
-0.06%

Covered Lines:
12524

Relevant Lines:
20982

💛  - Coveralls

It contains much workload in this PR, yet i want to know why you want to add the duplicated class named AlterTableStatement?

It contains much workload in this PR, yet i want to know why you want to add the duplicated class named AlterTableStatement?

The purpose is to modify the mysql code independently without affecting the code of other db

Hi Thanks for your effort, and I appended my opinions for this PR, please review them.

Here is my comment, please give it a look. @jingshanglu

Codecov Report

Merging #4789 into master will decrease coverage by 0.41%.
The diff coverage is 17.85%.

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

I have reverted to H2 coverage from previous test cases.

Hi @jingshanglu
I appreciated your modification, and current PR is clear enough. Good job,   only one point here is the naming, could you look at my comment?

Actually, firstxx and secondxx have  no meaning. It is like your name is Jingshang,  rather than firstPerson. My suggestion is precision and scale, or precisionLength and scaleLength. How about you?
@jingshanglu

Actually, firstxx and secondxx have no meaning. It is like your name is Jingshang, rather than firstPerson. My suggestion is precision and scale, or precisionLength and scaleLength. How about you?
@jingshanglu

ok

It seems correct, clear, and in line with the code of conduct currently. It is time to merge it.
@jingshanglu


[general-sdk] enable SwiftLint

SwiftLint can help us catch errors and ensure consistent formatting. Let's enable it on the rest-kit branch and address any warnings/errors before merging that branch to master.

We've ended up adopting a different style than the one that is checked and verified by SwiftLint. As a result, I think SwiftLint would cause us more frustration than help.


Institute automatic security warnings for dependencies.

Use depenendabot to automatically propose dependency updates in case of security issues.

Implemented with ac35935.


Makefile not respecting changes in stan submodule

Summary:
For a model is built in cmdstan, if the algorithms in the stan submodule are changed, on rebuilding make does not detect these changes and says there is nothing to do.
Description:
The discussion (and solution from @syclik) is here:
https://discourse.mc-stan.org/t/difficulty-modifying-stan-algorithms-with-cmdstan/10637
Current Output:
Make says there's nothing to do
Expected Output:
Make recompiles cmdstan main.cpp so that the algorithmic changes go into the model
Current Version:
v2.20.0

this is a duplicate of

Fixed by #769


Focus event

How can I use the ionFocus event?
I have tried this:
<ion-tags-input [(ngModel)]="tags" (ionFocus)="onFocus()"></ion-tags-input>

Now support, please upgrade.


List of scripts in editor lacks min width, can be fully hidden

Operating system or device, Godot version, GPU Model and driver (if graphics related):
godot v2.1.3
Issue description:

I could not see my script even if I open one. Picture https://www.reddit.com/r/godot/comments/6wify4/i_cant_access_any_script_what_did_i_do_wrong/
Steps to reproduce:
You have to push the "wall" between the list of script and the script all the way to the left
Link to minimal example project:

This no longer seems possible with Godot 3.0 (master) 4bde108d89113fb520ecfd4961d9fb3e7882e59f. Has it been fixed?

Yup, seems to be fixed. Has a min-width now for me.
@akien-mga This can be closed, I think?


Restrict KafkaInterceptor AOP to none final class

Similary to #159, there is some AOP issue :
Could not generate CGLIB subclass of class org.springframework.security.config.annotation.method.configuration.PrePostMethodSecurityConfiguration
KafkaInterceptor try to proxy PrePostMethodSecurityConfiguration which is a final class.

Codecov Report

Merging #161 (7d3b022) into main (bfec3ac) will decrease coverage by 0.04%.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@             Coverage Diff              @@
##               main     #161      +/-   ##
============================================
- Coverage     86.52%   86.48%   -0.04%
  Complexity       35       35
============================================
  Files            37       37
  Lines          2753     2753
  Branches        284      284
============================================
- Hits           2382     2381       -1
  Misses          248      248
- Partials        123      124       +1

Impacted Files
Coverage Δ

...com/decathlon/tzatziki/kafka/KafkaInterceptor.java
90.72% <ø> (-1.04%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


go mod tidy

Codecov Report

Merging #909 (9f225b0) into master (c1578d1) will increase coverage by 0.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #909      +/-   ##
==========================================
+ Coverage   45.70%   45.76%   +0.05%
==========================================
  Files         142      142
  Lines       10961    10961
==========================================
+ Hits         5010     5016       +6
+ Misses       5332     5326       -6
  Partials      619      619

Impacted Files
Coverage Δ

catalog/cataloger_create_entry.go
94.73% <0.00%> (-5.27%)
:arrow_down:

catalog/cataloger_create_branch.go
88.67% <0.00%> (+3.77%)
:arrow_up:

config/logger.go
75.00% <0.00%> (+10.71%)
:arrow_up:

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 c1578d1...9f225b0. Read the comment docs.

Oh yeah!
BTW, can a robot do this? (Asking for a cyborg friend...).

I think I saw a bot that does go tidy all day

Oh yeah!
BTW, can a robot do this? (Asking for a cyborg friend...).

I think I saw a bot that does go tidy all day


make traverse safe for delete operations

Currently, traverse does not play nice when mutating the children, signature, etc, of a reflection while traversing.
A current issue happens silently here
Every child is removed, but the iteration is done on the children collection:
    traverse(callback: TraverseCallback) {
        if (this.children) {
            this.children.forEach((child: DeclarationReflection) => {
                callback(child, TraverseProperty.Children);
            });
        }
    }

So the first child with index 0 is removed, then the index increments to 1 but because we altered the collection index 1 points to what used to be in index 2, i.e. we skipped an item.
The fix is simple, when working with collection work on a copy.
    traverse(callback: TraverseCallback) {
        if (this.children) {
            this.children.slice().forEach((child: DeclarationReflection) => {
                callback(child, TraverseProperty.Children);
            });
        }
    }

@aciccarello @blakeembrey can we kindly :) speed up PR merges?
I'm starting to pile up PR's and it creates a backlog 👍
thanks!

@shlomiassaf Thanks for the PRs. I'm really glad someone is making improvements. Sorry things are a bit behind. I don't have the time or knowledge of the codebase to process things very quickly. I'll see if I can get through the other 3 PRs sometime soon. Let me know if there is a priority. I'd guess #597 is the main blocker.
I'm also usually available on gitter if you need to discuss something.


multiple anonymous defines

For the record, I encountered this problem one day out of the blue and thankfully only lost a couple of hours before I tried disabling the browser extensions which fixed the problem (but left me without an excellent extension :( )

Which browser extension(s) did you have to disable?

Exif Viewer on Chrome.
On Mon, 1/18/16, John Hann<EMAIL_ADDRESS>wrote:
Subject: Re: [curl] multiple anonymous defines (#297)
To: "cujojs/curl"<EMAIL_ADDRESS>Cc: "molipha"<EMAIL_ADDRESS>Received: Monday, January 18, 2016, 9:55 PM
Which browser extension(s) did
you have to disable?
—
Reply to this email directly or view
it on GitHub.


Feature Request: Ability to remove chat prompts from the right click menu

The right click menu feels overtaken with options for this extension and just clutters the workspace. Id like to not have them at all, or at the very least move them all to a submenu.

@pkkid I thought I updated here but apparently not. I released a new version last week with Keybinding support and ability to remove the context menu prompts all together. Plus I added 2 new custom prompts that you can utilize apart from the predefined menu items. Unfortunately, vs-code doesn't allow dynamically updating the names/titles of the menu items so you will only have the option to hide them or update the promptPrefix themselves. Check out the announcement here: https://github.com/gencay/vscode-chatgpt/releases/tag/v3.6.0.
I hope this helps!


PageHeader: Address dom order issues (screen reader experience feedback from sign-off)

As a part of addressing accessibility sign-off review comments https://github.com/github/primer/issues/1115#issuecomment-1499501472, this PR updates the stories to ensure the order of the elements are the following;

PageHeader.Title
ContextArea (all elements)
PageHeader.LeadingAction
PageHeader.TrailingAction
PageHeader.Actions

instead of

ContextArea (all elements)
PageHeader.LeadingAction
PageHeader.Title
PageHeader.TrailingAction
PageHeader.Actions

The motivation behind is to make sure the actions on the page that are displayed above or before the main heading (Context area actions and leading action) should come after the main title (PageHeader.Title). With this way, we make sure screen reader users who navigate through heading menus don't miss any actions.
While we updated the dom order in the stories and this is how we will recommend to order the elements, we made sure everything, visually stays the same. (Leveraging CSS grid layout)
sign-off review comment reference
Changelog

New

Changed

Layout of the Root element from flex to grid
The order of PageHeader's sub components in the stories.

Removed

Rollout strategy

This is technically a breaking change but PageHeader is still a draft component, so we are good to realise this as a patch. There is one instance at github/github to be updated and I'll list the commit here to be included in the release PR.

[x] Patch release
[ ] Minor release
[ ] Major release; if selected, include a written rollout or migration plan

Testing & Reviewing

Merge checklist

[ ] Added/updated tests
[ ] Added/updated documentation (After I get 👍🏻 on the solution and the direction, I'll update the docs)
[ ] Added/updated previews (Storybook)
[ ] Changes are SSR compatible
[x] Tested in Chrome
[ ] Tested in Firefox
[ ] Tested in Safari
[ ] Tested in Edge

Take a look at the What we look for in reviews section of the contributing guidelines for more information on how we review PRs.

The grid gaps are creating some spacing issues. Maybe instead of using grid-gap, we could just put padding on the elements in the grid.

If you feel grid-gap is a better way to go, we could conditionally change how many columns the title area spans depending on what elements are present:

If there's a leading visual and trailing action, keep it as-is
If there's a leading visual and no trailing action, span from column 2 to 4
If there's no leading visual or trailing action, span from column 1 to 4

This solution could get kind of complicated though...

The grid gaps are creating some spacing issues.

@mperrotti  I saw the gaps in the VRT tests but didn't realised it was that tricky 😭 Thanks for bringing it to my attention!

Maybe instead of using grid-gap, we could just put padding on the elements in the grid.

I am a bit hesitant on this; mainly because I think we want these gaps to be around the cells not around the items. Also I feel like your second suggestion aligns more with the overall direction in the implementation. We have grid template areas and since we don't know what sub components might be used in a given PageHeader usage, creating the areas dynamically makes more sense to me, though it is quite complex, I agree 😬 I pushed a super WIP draft PR to explore your second suggestion. https://github.com/primer/react/pull/3970 I'll tag you there when it is less WIP 😄

@broccolinisoup - why does this have the "skip changeset" label? Wouldn't we want to cut a release with these changes?

@broccolinisoup - why does this have the "skip changeset" label? Wouldn't we want to cut a release with these changes?

Yes, I usually add skip label to the big PRs in the beginning to avoid the CI job failing. After changes are clear, I write a changeset that reflect the changes. Sorry I didn't communicate about it. Considering this is approved, I'll add a changeset now. For your info, because it requires updates at dotcom, I'll hold off merging the PR until I am back from holidays (Jan 11th)

closing this due to snapshot merge conflicts and opened another PR https://github.com/primer/react/pull/4358


decode adds extension to filename

decode automatically appends an extension to the filename, hard-coded to '.jpg'.  This is not a safe assumption to be made as either an extension may already be provided and/or the contents may not be a jpeg image.

Thank you. Will look into it.

Implementation details -

[ ] Infer extension from data string. Can be overridden by ext option if declared explicitly.
[ ] Save as jpg by default if Buffer
[ ] Save to provided ext if provided and Buffer is the input

@fastbean-au Thoughts?

it 's in line 104 of node-base64-image.js
(0, _fs.writeFile)(options.filename + '.jpg', imageBuffer, 'base64', function ...
instead of
(0, _fs.writeFile)(options.filename, imageBuffer, 'base64', function ...


Legend showing "Series 1" label rather than "This practice/ccg"

e.g. url https://openprescribing.net/measure/ktt9_cephalosporins/
It's working OK on CCG pages https://openprescribing.net/ccg/08E/measures/


Add HIP-729 Contract Nonces to ContractFunctionResult (PR)

Description:
This PR adds support for HIP-729 - adding ContractNonces to ContractFunctionResult.
Related issue(s):
Fixes #: https://github.com/hashgraph/hedera-sdk-cpp/issues/413
Notes for reviewer:
Checklist

[x] Documented (Code comments, README, etc.)
[x] Tested (unit, integration, etc.)

Integration tests suite for ContractNonceInfo will be postponed and will be added with https://github.com/hashgraph/hedera-sdk-cpp/issues/470.


MWPW-143414 migrate submit email block

Describe your specific features or fixes
Resolves: MWPW-NUMBER
Test URLs:

Pre Migration Link: https://adobe.com/express/learn/express-your-brand?martech=off
Before: https://main--express-milo--adobecom.hlx.page/express/learn/express-your-brand?martech=off
After: https://MWPW-143404-submit-email--express-milo--adobecom.hlx.page/express/learn/express-your-brand?martech=off

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Please upload report for BASE (main@1cb7d93). Learn more about missing BASE report.
Report is 7 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #84   +/-   ##
=======================================
  Coverage        ?   66.99%
=======================================
  Files           ?       58
  Lines           ?     9477
  Branches        ?        0
=======================================
  Hits            ?     6349
  Misses          ?     3128
  Partials        ?        0

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


After setting up 2FA it requires to pass 2FA Check

Well it is kind of inconvenient, but having this feature might lead to its abuse.

As if you set up 2fa & have no email ip check cookies, it will cause a IP check without sending a email with code, thus making it impossible.

So well
After setting up 2FA it requires to pass 2FA Check: Wontfix, it might lead to several security risks.
...it will cause a IP check without sending a email with code: Will be fixed in the next update.

Resolved | The issue will be fixed in the next release.


Feature/touch model when destroy or recover

:tophat: What? Why?
This is really important for us to understand when a model has been deleted.
:ghost: GIF

:+1:

:+1:


What is the prompt for the pseudo reading task?

In the Readme it says:
"For (Pseudo) Text Reading Task
The gt_parse looks like {"text_sequence" : "word1 word2 word3 ... "}
This task is also a pre-training task of Donut model.
You can use our SynthDoG 🐶 to generate synthetic images for the text reading task with proper gt_parse. See ./synthdog/README.md for details.

"
I saw on some guy Phiipp's blog he just used "" where as here: https://towardsdatascience.com/ocr-free-document-understanding-with-donut-1acfbdf099be
he or she uses "<s_sroie_donut>" ie. s followed by the file path of the dataset he or she used to fine tune donut.
Is this correct, to just use ""?
I am trying to read my students' homework submissions.
As an example, I am trying to read this in Colab:

It's actually very weak when I tried with the prompt "" in this notebook: https://gist.github.com/nyck33/ba445efc3f480a3be5b8a1bc8ffb3418
I'm not sure why in the code cell the prompt I used looks like "" empty string but you can see that I printed it out at the bottom to confirm it was "".


GLM Include Error

Hello, I'm fairly new to using pods. I'm currently getting the following error,

it has to do with this #include "type_half.inl" more specifically not being found.
I'm not sure what to do, any help would be appreciated. I don't know if this is connected at all, but I have an error when doing pod install, which is this:

Thanks!

This seems to be a ridiculous incompatibility introduced with cocoapods v0.39.0.
Downgrading back to v0.38.2 got working again for me for now.

How do you downgrade back to v0.38.2? I'm fairly new to this.
Thanks!
Sent from my iPhone

On Oct 14, 2015, at 8:31 AM, Andreas Wunderlich<EMAIL_ADDRESS>wrote:
This seems to be a ridiculous incompatibility introduced with cocoapods v0.39.0.
Downgrading back to v0.38.2 got working again for me for now.
—
Reply to this email directly or view it on GitHub.

This worked for me:

gem install cocoapods -v 0.38.2
gem uninstall cocoapods -v 0.39.0

Right, so I did that, and now I get this ugly mess.

Am I doing something wrong?

Well, all the middle part (lines 2 to 10) is just the update notification. Ignore that for now.
However, I think you cut off the interesting part below "Unable to satisfy the following requirements:"
Please post whats below there + your Podfile.

Pod file:
platform :ios, '8.0'

source 'https://github.com/CocoaPods/Specs.git'

# See: http://guides.cocoapods.org/making/making-a-cocoapod.html#release
# Once the pod has been released, use a version number here instead of a path.
#pod 'VideoCore', '~><IP_ADDRESS>'
pod 'VideoCore', path: '../..'

I fixed it by uncommenting the first VideoCore line, and commenting the second.
pod 'VideoCore', '~><IP_ADDRESS>'
#pod 'VideoCore', path: '../..'

Yeah, also note that the version is duplicated.
I think this is what you wanted:

pod 'VideoCore', '~>0.3'

Ah, good point. So that time I was able to actually run the app, and it actually worked! So, are these warnings okay?

Yes, those warnings are fine.  The code is setup to support legacy SDKs.

I solved this problem by change the search path setting in Pods project. Just remove all ""${PODS_ROOT}/Headers/Private" in “Header Search Paths"

So when I go to use VideoCore with a brand new project, I keep getting error about VCSessionDelegate "use of undeclared type 'VCSessionDelegate'". I have no clue what this means. I have set the project up with the Pods file, and I'm pretty sure video core is installed.

Fixed... Don't forget to go to build settings and enter the bridging file path. This should really be in the docs.

This has been solved (as mentioned way above).

@Craytor Hey, getting the same error as you did in the last comment. Could you elaborate more on what you meant/did by entering the bridging file path?
Thanks!

what's the mean of 'Don't forget to go to build settings and enter the bridging file path.'?what to to do exactly?

dourgulf has given the best and quickest solution to this problem.

@dourgulf  has given the best and quickest solution to this problem


PANIC:  XX000: ERRORDATA_STACK_SIZE exceeded

The following snippet produces the error in the title. The snippet has already been reduced in size somewhat, but it seems like the error may be more related to total query length than anything else. However, I was unable to reproduce the error simply by copypasting a large amount of simple SELECTs, so I assume something else is going on as well.
It isn't possible to avoid the bug by using several functions instead of one large function, all active modules in the project seem to count against the limit.
#![feature(plugin)]
#![plugin(postgres_macros)]

fn main() {
    sql!("
        LOCK TABLE foo IN ACCESS EXCLUSIVE MODE
    ");

    sql!("
        ALTER TABLE foo
            ADD CONSTRAINT foo PRIMARY KEY (foo)
    ");

    sql!("
        ALTER TABLE foo
            ADD CONSTRAINT foo
                FOREIGN KEY (foo)
                REFERENCES foo (foo)
                ON DELETE RESTRICT
                ON UPDATE RESTRICT
    ");

    sql!("
        CREATE INDEX foo ON foo (foo)
    ");

    sql!("
        INSERT INTO foo VALUES ($1)
    ");

    sql!("
        LOCK TABLE foo IN ACCESS EXCLUSIVE MODE
    ");

    sql!("
        ALTER TABLE foo
            ADD CONSTRAINT foo PRIMARY KEY (foo)
    ");
}

Here's the full error you get on cargo run:

PANIC:  XX000: ERRORDATA_STACK_SIZE exceeded
LOCATION:  elog_start, elog.c:1320

I'm using the latest 64-bit nightly and postgres 9.4.5 on OS X 10.11.

Awesome! Hoping for a quick v0.1.6 :)

Just released :)


pip: 18.0 -> 18.1

Motivation for this change
Version Bump
Things done

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

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

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

#47927 is already bumping pip

No problem. :)
Since you have tested this, maybe you can help @FRidh with the review of #47927 (just leaving a comment saying what testing you did for example).
Thanks for your contribution nonetheless!


Attaching the signature artifact to the Image artifact

When I use oras (or gcloud artifacts attachments create) to attach an artifact to an image artifact, it's displayed in a parent/child relation in the UI of Google Cloud Artifact registry.

When I use cosign this is not the case. The signature artifact is displayed like a normal artifact and no relationship is visible at first glance.
After comparing some of the manifests, I noticed that oras adds a field called subject to the manifest of the added artifact with a reference to the digest of the image artifact.
"subject": { "mediaType": "application/vnd.oci.image.index.v1+json", "digest": "sha256:the_digest_of_the_image", "size": 856 }
https://github.com/opencontainers/image-spec/pull/1020
https://github.com/docker/build-push-action/issues/1260
You can use the following command oras attach --artifact-type doc/example --annotation "key1=val1" --annotation "key2=val2" localhost:5000/hello:v1 to quickly create an attached artifact at that location.
Can cosign support the subject field as well?

OCI 1.1 support is planned - https://github.com/sigstore/cosign/blob/main/specs/BUNDLE_SPEC.md

https://github.com/moby/buildkit/pull/5573
buildkit is adding support in the next version.

The proposed Bundle-spec will be just another breaking change that makes all existing attachments and tooling incompatible, yet again. Also, I need to attach signatures and SBOMs in a future-proof way right now.

What other options besides cosign are there?

I am currently looking into Notation/NotaryV2 in combination with ORAS. My plan is to just push my SBOMs as true OCI attachments (Referrers API) using ORAS and maybe sign them using Notation. I am not saying that these tools are "better" than Cosign, but at least they seem committed to their specifications without deprecating everything every year.

The planned OCI 1.1 referring artifact support for attestations (Bundle-spec) is in active development here and should be available very soon as an optional feature in Cosign.
We've been very deliberate in designing the new specifications and this PR is part of a series of client standardization efforts that have been ongoing for nearly two years, centered on the Sigstore Bundle Format. The other language clients are cross-compatible and Cosign is the last to be updated, as we take breaking changes very seriously.

I'll also point out https://github.com/sigstore/cosign/issues/3927 as a solution for signatures as OCI artifacts, as the spec I linked is for attestations.
@ChristianCiach, adapting based on feedback from consumers and evolving the specification based on progress across the ecosystem is a natural part of development. Yes, in this example, there is a breaking change in terms of the format so old Cosign versions will not be compatible, but it's not accurate to say existing attachments will fail. We can continue to support verification of the current signature specification.
We want to standardize on the format of the signature and its verification material across Sigstore clients and SDKs. Cosign is actually the outlier in this case as bundles are already implemented in 5 Sigstore SDKs. We've made a lot of progress recently towards supporting this newer bundle format in Cosign, launched in Cosign v2.4.0. The bundle specification I've linked is our answer to OCI 1.1 and bundles - using OCI 1.1 with the current spec for the signature format keeps Cosign divergent and incompatible with the rest of the ecosystem, and using annotations to store bundles makes no progress towards OCI 1.1.
We are always open to feedback or PRs from the community.

Thanks a lot, guys! Everything is a lot clearer to me now.


Add additional audio codes which is not supported by browser?

Is it possible to add additional audio codec which is not supported by browser? For example, chrome doesn't support Dolby Digital Plus.  How can Shake Player support Dolby Digital Plus in chrome?

Shaka Player is limited to what the platform (the browser) supports. This means if Chrome/Firefox/Edge do not support a feature, we cannot play that content on that browser. The one exception is if it is something at the manifest-level (which we parse). If Chrome does not support it, there is not much we can do.


NTUEEPLUS-143. Out main page

What is this PR for?

merge all out components into one long page
scroll to corresponding part when click on sidebar
What type of PR is it?
[Bug Fix | Improvement | Feature | Documentation | Hot Fix | Refactoring]
improvement
Todos

[ ] - when login, user need to dbclick so that they could scroll
[ ] - header cover some space, I use br to avoid, but there could be better solution

What is the Jira issue?

https://ntueeplus.atlassian.net/browse/NTUEEPLUS-143
Screenshots (if appropriate)
out:

in:

Not use. Sorry QQ


bikeshed update can't download files

Once in a while when I do git pull --rebase; pip3 install -e .; bikeshed update I sometimes get (one or more) warnings like this:
WARNING: Couldn't download file 'https://raw.githubusercontent.com/tabatkins/bikeshed-data/master/data/wpt-tests.txt'.
Cannot connect to host raw.githubusercontent.com:443 ssl:True [SSLCertVerificationError: (1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1123)')]

Don't know if this is a problem with bikeshed or with github being messed up in some way.  But I thought I'd report this anyway. It seems more common these last week or so.  I don't recall ever seeing these before then.

...huh. I suspect this is just a transient network error? That said, I do have some retry logic built into the full update (querying all of the original data sources); I should probably apply it to the standard update process (via manifest, only querying static files on GitHub) as well. ^_^

When I run bikeshed update again, bikeshed just says everything is up-to-date (without any other messages) so it's hard to tell if it really is up-to-date or just failed to note that it still couldn't download the files.

Yeah, if there's a problem downloading, it looks like I'm not reflecting that in the manifest data, so there'll just be no way to tell (until that file updates again, at least). Definitely needs fixing!

Okay, at least partially fixed. Errors in downloading or saving files from the manifest will be recorded in the manifest, and will trigger a re-download next time you bikeshed update even if the remote data hasn't changed.
I still need to add retry logic into the function, but we're in a better spot now at least.

That's fantastic!  At least I know that if bikeshed spec does weird things, it could be because the update didn't work correctly.
I tried now and fails the first time (after doing a git pull; pip install -e .; bikeshed update).  When I update again, I get (roughly?) the same messages.
So, yeah, this is great!  I don't mind too much having to retry manually as long as I know I need to.

In case it matters, one of the messages I get is
WARNING: Couldn't download file 'https://raw.githubusercontent.com/tabatkins/bikeshed-data/master/data/anchors/anchors-ju.data'.
Cannot connect to host raw.githubusercontent.com:443 ssl:True [SSLCertVerificationError: (1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1123)')]

but curl https://raw.githubusercontent.com/tabatkins/bikeshed-data/master/data/anchors/anchors-ju.data downloads the data just fine.  Maybe my local setup is messed up?  I only have this problem on my mac, not on linux.

Given that all of the files are downloaded identically and from the same website (same repo, even) but only one of them is triggering problems, yeah, it's some weird local issue. 😃

I'm currently getting 35 failed files now.  I have no idea how to figure out my weird local issue.  Any hints?

So I just pushed retry logic in, so hopefully these'll fix?
And unfortunately, no, I have absolutely no idea how any of this works. It's basically magic and the fact that it works at all is due solely to other people being clever.
Is it failing for all of the files it now tries to update? I just now pushed an update to the message so it tells you how many total files it tried to update; is it failing for them all, or just some?

A couple of comments.  I dislike the new error messages which looks like:
Updating via manifest...
WARNING: Previous update had some download errors, so re-running...
Updating 47 files...
Updated 0/47, 1 errors...
FATAL ERROR: Done, but there were 47 errors (of 47 total) in downloading or saving. Run `bikeshed update` again to retry.
 ✘  Did not generate, due to fatal errors

I don't get any information on what failed.  That makes it hard for me and you to figure out what's failing.
However, having said that, I noticed there's a --skip-manifest option.  I tried that, and it took a while to download everything, but it succeeded to download everything.  Then I ran bikeshed update which said there's nothing to update. Sweet!
Perhaps my manifest file was messed up somehow?  Anyway, I'm going to try to do an update again tomorrow to see if the issue is gone.

Oh, duh, sorry, I did indeed strip out the individual error messages. That wasn't an intended change, I'll put those back.
Yeah, --skip-manifest does the full "download from sources, assemble all the data manually" steps that the bikeshed-data bot normally does. Because those sources are slow, and the assembling is slow, it's not generally great (plus I wanted to make sure that, as Bikeshed usage went up, I wasn't spamming the source servers with db queries; GH can handle being spammed with static file requests far better).
 It's not a general https failure, because the manual update also does everything over https, so it'd be somethign specific to GH. But some of the --skip-manifest sources are also just static files on GH anyway, so if all those succeeded, I'm really not sure what's up. :(
Let's hope we can figure this out!

Hmm. Maybe it's my python version.  I have 3 potentially different versions of python3 on my mac.  One in /opt/local/bin (macports), /usr/local/bin (don't know where that came from) and /usr/bin, in that order.
I'll try to test tomorrow if this makes a difference.

git pull; pip3 install; bikeshed update still gives a few warnings about not downloading files.  And some investigation shows that bikeshed is using python from macports.  Perhaps I should remove everything and start over and use a different python3.

Not entirely sure what was up, but I was recently stuck in a state where bikeshed would have outdated definitions for terminology and tests (and maybe more), and running bikeshed update would always report success but leave me in the same out of date state. bikeshed update --skip-manifest did fix it though.
Since it is now fixed, I cannot give more details as to what exactly was going on, but I thought collecting anecdotes might be helpful. (I was on the latest bikeshed, pulled from git, with python 3.8.6)


Gaze clicker experiment

object selection dropdown
trigger type dropdown

One comment for this one. Presently for an object to be "gaze clickable" the ClickableObject patch needs to be added after the object3d.
For this gaze clicker experiment to work as intended, all objects should either be ClickableObjects by default, or should be come ClickableObjects once selected from this dropdown.


Move ".exe" appending logic into ctx.Suffix()

This also updates the appropriate places in the code to use ctx.Suffix() where they weren't previously (and actually, nothing but gb info was using it).
One notable place that doesn't change here is gc.go, since it's using gohostos to find existing tool locations, not dealing with output locations (gotargetos).

It was really bothering me that we had .exe appending logic in so many places. :innocent:

This feeds wrong sorry, Suffux is composed of goos, goarch and tags, it shouldn't have anything to do with the nasty hack required for windows. The rest of the cleanup is good thou.

I'm not sure I follow -- right now in master, ctx.Suffix() is essentially if ctxString() is not empty, return "-" + ctxString(), but when we build we do if cross-compiling, append "-" + ctxString() to our binary name, which will be the same result?  So you're saying using tags + cross-compile will actually be kind of funky as-is with master?

it really bothers me that windows requires this gross haxk
On Thu, Sep 24, 2015 at 9:18 AM, Tianon Gravi<EMAIL_ADDRESS>wrote:

It was really bothering me that we had .exe appending logic in so many
places. [image: :innocent:]
—
Reply to this email directly or view it on GitHub
https://github.com/constabulary/gb/pull/411#issuecomment-142757879.

(and besides that, the only place ctx.Suffix() is currently used is in gb info for an output field called GB_BIN_SUFFIX, so I was confused as to what that means)

Yeah, nasty windows. :cry:

Would you prefer a new function for this, or should I modify the way ctx.Suffix() works to not include tags info?

(and maybe rename it to BinSuffix or something so it's clear?)

Neither. I don't want ctxString or Suffix to have any .exe entry, even on
windows. That should be handed in binffile/Binfile
On Thu, Sep 24, 2015 at 9:23 AM, Tianon Gravi<EMAIL_ADDRESS>wrote:

Would you prefer a new function for this, or should I modify the way
ctx.Suffix() works to not include tags info?
—
Reply to this email directly or view it on GitHub
https://github.com/constabulary/gb/pull/411#issuecomment-142758576.

There should be only one place where the names of binaries are computed,
even for test binaries
On Thu, Sep 24, 2015 at 9:24 AM, Dave Cheney<EMAIL_ADDRESS>wrote:

Neither. I don't want ctxString or Suffix to have any .exe entry, even on
windows. That should be handed in binffile/Binfile
On Thu, Sep 24, 2015 at 9:23 AM, Tianon Gravi<EMAIL_ADDRESS>wrote:

Would you prefer a new function for this, or should I modify the way
ctx.Suffix() works to not include tags info?
—
Reply to this email directly or view it on GitHub
https://github.com/constabulary/gb/pull/411#issuecomment-142758576.

Ok, that's fair, but does mean our tests will need to keep some of the same logic (like they have today).
Do you think it'd make sense to have a function somewhere to encapsulate our if statement for going from OS to "binary extension"? (ie, something like func binExtension(os string) string { if os == "windows" { return ".exe" } return "" } so we could just += binExtension(ctx.gotargetos) or += binExtension(ctx.gohostos))

I don't think this is worth a big effort. There are only two places that generate build.go via link and cmd/test.go, both of those should be using some kind of helped to generate the binary name. If not, we add a helper and a unit test. I don't think this needs to be more complex than that.

On 24 Sep 2015, at 09:32, Tianon Gravi<EMAIL_ADDRESS>wrote:
Ok, that's fair, but does mean our tests will need to keep some of the same logic (like they have today).
Do you think it'd make sense to have a function somewhere to encapsulate our if statement for going from OS to "binary extension"? (ie, something like func binExtension(os string) string { if os == "windows" { return ".exe" } return "" } so we could just += binExtension(ctx.gotargetos) or += binExtension(ctx.gohostos))
—
Reply to this email directly or view it on GitHub.