Duplication of some names in step 1

In which chart does the bug happen?

Alluvial diagram
Sankey diagram

RAWGraphs Version
2.0
Web browser and version
Chrome Version 107.0.5304.89 (Official Build) (64-bit)
Operating System
Windows 10
Steps to reproduce this
Using a simple dataset with overall 19 universties describing the categories of courses they offer (3 different categories). I pull the university as the first step, then the next column is the number associated with the category of courses (1-3). For 15 of the universities, the name is only listed once in step 1 and the lines go to the numbers 1-3. But for 3 universities, no matter how i put the data in, the names are repeated and new lines go to the next step 1-3.
Project file

Additional context

For data vis.xlsx

Not clear to me the issue. This is the result that i get, it seems correct to me. Can you annotate on the image what is wrong? thanks! (I also upload the project file to reproduce it: viz.rawgraphs.zip).

Hi there,
The issue is that a number of universities are repeated on the right. So University of British Columbia (#2), Brock University (#3), and Universite de Montreal (#4) are all repeated in the list rather than only having one instance as they should.
Melissa MacKay, PhD (she/her)
Postdoctoral Scholar, Population Medicine
University of Guelph
@.***
From: Michele @.>
Sent: June 5, 2023 6:34 AM
To: @.>
Cc: Melissa @.>; @.>
Subject: Re: [rawgraphs/rawgraphs-charts] Duplication of some names in step 1 (Issue #136)
CAUTION: This email originated from outside of the University of Guelph. Do not click links or open attachments unless you recognize the sender and know the content is safe. If in doubt, forward suspicious emails to @.***
Not clear to me the issue. This is the result that i get, it seems correct to me. Can you annotate on the image what is wrong? thanks! (I also upload the project file to reproduce it: viz.rawgraphs.ziphttps://github.com/rawgraphs/rawgraphs-charts/files/11651546/viz.rawgraphs.zip).
[viz]https://user-images.githubusercontent.com/907536/243327191-bce7533f-88ac-437c-add8-5b6ef254ecba.png
—
Reply to this email directly, view it on GitHubhttps://github.com/rawgraphs/rawgraphs-charts/issues/136#issuecomment-1576540334, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A4DDG23AL7G6QOZIGXZKEUTXJWYRFANCNFSM6AAAAAAX3EGMZA.
You are receiving this because you authored the thread.Message ID: @.***>

The issue is related to the dataset: some values has double spaces, or spaces at the end of the string (see the screenshot, i highlighted some of them). RAWGraphs intepret them as different values. I suggest you to check the dataset with a tool like Open Refine to trim whitespaces and check inconsistencies (e.g. capitalization, typos)

Thanks Michele, the website is helpful as those differences are not visible in the Excel dataset.
Thanks, Melissa
Melissa MacKay, PhD (she/her)
Postdoctoral Scholar, Population Medicine
University of Guelph
@.***
From: Michele @.>
Sent: July 6, 2023 5:13 PM
To: @.>
Cc: Melissa @.>; @.>
Subject: Re: [rawgraphs/rawgraphs-charts] Duplication of some names in step 1 (Issue #136)
CAUTION: This email originated from outside of the University of Guelph. Do not click links or open attachments unless you recognize the sender and know the content is safe. If in doubt, forward suspicious emails to @.***
The issue is related to the dataset: some values has double spaces, or spaces at the end of the string (see the screenshot, i highlighted some of them). RAWGraphs intepret them as different values. I suggest you to check the dataset with a tool like Open Refinehttps://openrefine.org/ to trim whitespaces and check inconsistencies (e.g. capitalization, typos)
[Screenshot 2023-07-06 at 23 10 51]https://user-images.githubusercontent.com/907536/251575833-bf978b54-2431-430f-a19a-d568ba77ae00.png
—
Reply to this email directly, view it on GitHubhttps://github.com/rawgraphs/rawgraphs-charts/issues/136#issuecomment-1624317321, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A4DDG26O6N7BCRWHKZGH2UDXO4TANANCNFSM6AAAAAAX3EGMZA.
You are receiving this because you authored the thread.Message ID: @.***>


Address system test failures

Logging errors should report the errors to the error reporting service.  The system tests pulls the errors reported from the error reporting console to ensure that is the case.
Many changes have been made to try to make this pull more reliable based on the fact that it can take some time for errors to appear in the error reporting console.  However, the pulling mechanism is still not reliable enough to support consistent system test success.
I think we should just verify in the system tests that when a log at log level error is created, the appropriate payload used in calls to @google-cloud/logging has the error log level.

👋 just a heads up - bumping the priority to p1.  I don't think something can be both release blocking and a p2 at the same time :)

@DominicKramer I am not sure if this really needs to be labelled release-blocking.


Add rti-connext-dds-6.0.1 which is now in the ROS bootstrap repositories

Package name: rti-connext-dds-6.0.1
Distro packaging links:
Links to Distribution Packages
This package is hosted on the ROS bootstrap repository for Ubuntu. Since the package is not available on ARM platforms an empty package is provided to satisfy the build dependency and the packages which require rti-connext-dds to be installed are configured to become empty packages themselves if this dependency is not present at build time.

Debian: https://packages.debian.org/

NOT AVAILABLE

Ubuntu:

http://repos.ros.org/repos/ros_bootstrap/pool/main/r/rti-connext-dds-6.0.1/

Fedora: https://src.fedoraproject.org/browse/projects/

NOT AVAILABLE

Arch: https://www.archlinux.org/packages/

NOT AVAILABLE

Gentoo: https://packages.gentoo.org/

NOT AVAILABLE

macOS: https://formulae.brew.sh/

NOT AVAILABLE

Alpine: https://pkgs.alpinelinux.org/packages

NOT AVAILABLE

NixOS/nixpkgs: https://search.nixos.org/packages

NOT AVAILABLE

@lopsided98 I've just left the empty nixos dependency intact from the definitions for 5.3.1.


ENH: Add out-of-sample prediction to panel

All data to be used with predict to produce out-of-sample fits

This pull request introduces 7 alerts and fixes 1 - view on lgtm.com
new alerts:

4 for Non-callable called
3 for Potentially uninitialized local variable

fixed alerts:

1 for Non-callable called

Comment posted by lgtm.com


Total black or total white frames causes fps detection to drop

I detected this while testing captured content using ShadowPlay at max quality (around 130mbps to avoid quality loss as much as possible keeping the 60fps target) from EXE Drive specials and transform sequences from Hyperdimension Neptunia Re;Birth 1. There are some total white/black frames that are considered as unchanged by the program causing the fps value to drop while thats really not the case.
Tested in 0.3 alpha build.

trdrop works best with uncompressed video files which is not the case with ShadowPlay. ShadowPlay gives you a h264 video file, which is highly compressed, even with 130 mbit. You can try fraps which has less compression with avi mjpg, but best result is with totally uncompressed video files.

You are right. I don't have a capture card that can deliver uncompressed format directly so I can test to see what happens when using FRAPs content or nvenc at 444 color gamut and post results later.

It is true that trdrop works best with uncompressed video. However, a totally static scene with multiple identical consecutive frames (like a black screen or a totally static menu screen) will always be considered a frame drop as there is no way of differentiating whether it actually is one or if it's just a static scene.
Using uncompressed video might help in come cases where the compression would otherwise eliminate small differences that are not visible to the eye.
trdrop only analyzes framerates by comparing pixels of consecutive frames. This is of course a drawback when it comes to actually static scenes, but then again those are usually not the ones we're interested in :)
I'm not sure if we're going to provide a way around this in the future, but for now it is pretty much working as intended so I'll go ahead and close the issue.


ARROW-2411: [C++] Add StringBuilder::Append(const char **values)

This implementation uses not NULL-terminated C strings and the length of values instead of NULL-terminated C strings. Because other builder uses the interface such as PrimitiveBuilder::Append().

This implementation uses not NULL-terminated C strings and the length of values instead of NULL-terminated C strings

"NULL-terminated C strings" means each C string is terminated by a nul char, not that the array is terminated by a NULL pointer :-)

"NULL-terminated  C strings" means each C string is terminated by a nul char, not that  the array is terminated by a NULL pointer :-)

Thanks for the English lesson. :-)
It's very helpful to me because I'm not a native English speaker.

Thanks @kou !


replaced while true with while condition

While true :
if (condition) break
can simply become:
While not condition:

@eddyallard So do you recommend rejecting it and keep the first version or you recommend different changes ?

@jaykay9999 Considering that there are no do...while loops in python and that there are no checks associated with this pull request to validate it's functionnality, I would recommend rejecting it and keeping the first version.

Initial pull request rejected.


Add back missing favicon

nice

pleeeeeeeeease take me off this— I do not work at artsy

On May 17, 2018, at 11:08 AM, Orta<EMAIL_ADDRESS>wrote:
nice
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/artsy/artsy.github.io/pull/446#issuecomment-389958260, or mute the thread https://github.com/notifications/unsubscribe-auth/AHTqjdGWdtLuuFNwLaz1h-N4OEfwsbV8ks5tzby0gaJpZM4UDhKW.

I just checked and there is no way to remove someone as a watcher from a repo. I'll open a suggestion to GitHub.


test: check trivy scan in ci

A trivy scan test

Codecov Report

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

@@           Coverage Diff           @@
##           master      #53   +/-   ##
=======================================
  Coverage   93.15%   93.15%
=======================================
  Files          14       14
  Lines         628      628
=======================================
  Hits          585      585
  Misses         43       43

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 e51ef71...be8fd20. Read the comment docs.


release branch name collisions when releasing from multiple branches

It's possible when releasing from multiple feature branches that release-please tries to use the same version. In this case, release-please would stomp on the other feature branches' release versions.
We probably need to construct a release branch naming structure such that they can never collide across branches.

Calling this a feature request, since it mainly crops up in the multiple release branch scenario, and we have work in motion to fix the problem.

Calling this a feature request, since it mainly crops up in the multiple release branch scenario, and we have work in motion to fix the problem.


Support serialization of players handbook

addresses issue : https://github.com/two-shots-later/vitruvian-vtt/issues/9

Talked about this, we are pivoting away from 5E and moving toward Pathfinder because of their open Community license. Closing this PR


Remove files from disk?

The code is fine ~ the plugin works as announced.. however it could be nice if it could somehow remove the files from being written in disk.. or maybe (a dirty way) would be remove it after it has been created... so you don't end up with a .css and .js file in the directory....

remove-source-webpack-plugin can solve your problem

I have been able to successfully use ignore-emit-webpack-plugin.


⚠ Bofrost.at has degraded performance

In 63870aa, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1408 ms

Resolved: Bofrost.at performance has improved in f2768f4 after 28 minutes.


Task List

TODOs

[ ] 1. Remote file list (given a folder) and transfer (given a file name).

[ ] 2. Results table:

When a task is executed, it would be nice to have some sort of parameter like this:

store_results = None # Don't store results
store_results = ['*.txt'] # Store as results all *.txt files
store_results = ['filename1', 'filename2'] # Store as results files ( a result entry per file)
store_results = ["*", "!*.nxs"] # All files except *.nxs

[ ] 3. Examples with the password caching using something default in Django. Ideas:
SESSION_ENGINE = "django.contrib.sessions.backends.cache". It uses memory sessions (rather than the DB) for all sessions. The problem is when the memory runs out the sessions disappear..... The server would eventually purge the cache and make the user put on their passwords again...

https://docs.djangoproject.com/en/1.10/topics/http/sessions/#using-cached-sessions

Session data will be stored directly in your cache. However, session data may not be persistent: cached data can be evicted if the cache fills up or if the cache server is restarted.

[ ] 4. Log model needs a flag to tag the log either as stdout or stderr.

[ ] 5.  Choose the interpreter in submit_task. E.g. a parameter like interpreter='/usr/bin/python3 or  interpreter='/usr/bin/bash.

[ ] 6. Minimize dependencies (Let's keep just celery and redis for now).

[x] 7. submit_task needs a timeout. We have to see how to kill the process remotely and interrupt the celery task...

[ ] 8. Ideas (not urgent!):

Using django-channels to have a real update of the tasks. See example: https://github.com/VincentTide/django-channels-celery-example

Use django-celery-results to see in the admin the celery results tables.

Done:
7
Tanner is doing:
1,2
Ricardo is doing:
5

TODO
import os
import paramiko

from getpass import getpass

'''
Logs in the remote machine adds the client public key to the
authorised keys in the remote host

Set password = getpass()
For the first login if authorised keys does not exist yet in the remote machine
'''

remote_host = "lealpc.ornl.gov"
#username = os.getlogin()
username = os.environ['USER']
password = None # getpass()
public_key_filename=os.path.expanduser('~/.ssh/id_rsa.pub')

def start(server, username, password=None,  key_filename=None):
	'''
	starts the client
	If password=None uses the key_filename
	'''
    try :
        client = paramiko.SSHClient()
        client.set_missing_host_key_policy(paramiko.AutoAddPolicy())
        if password:
            client.connect(server, port=22, username=username, password=password)
        else:
            client.connect(server, port=22, username=username, key_filename=key_filename) # no passwd needed
    except Exception as e:
        raise RuntimeError(e)
    return client

def deploy_key_if_it_doesnt_exist(client, key_filename):
    '''

    '''
    key = open(key_filename).read()
    client.exec_command('mkdir -p ~/.ssh/')

    command = '''
    KEY="%s"
    if [ -z "$(grep \"$KEY\" ~/.ssh/authorized_keys )" ];
    then
        echo $KEY >> ~/.ssh/authorized_keys;
        echo key added.;
    fi;
    '''%key

    stdin, stdout, stderr  = client.exec_command(command)

    print(stdout.readlines())
    print(stderr.readlines())

    client.exec_command('chmod 644 ~/.ssh/authorized_keys')
    client.exec_command('chmod 700 ~/.ssh/')

client = start(remote_host, username, password,  key_filename=public_key_filename)

deploy_key_if_it_doesnt_exist(client, key_filename=public_key_filename)

while client:
    key = True
    cmd = input("Command to run: > ")
    if cmd == "":
        break
    chan = client.get_transport().open_session()
    print("** running '%s'" % cmd)
    chan.exec_command(cmd)
    while key:
        if chan.recv_ready():
            print("** recv:\n%s" % chan.recv(4096).decode('ascii'))
        if chan.recv_stderr_ready():
            print("** error:\n%s" % chan.recv_stderr(4096).decode('ascii'))
        if chan.exit_status_ready():
            print("** exit status: %s" % chan.recv_exit_status())
            key = False
            client.close()
client.close()


Fix profiling tests

You should look at the first 2 individual commits.  In the 3rd commit, I've moved a test to a different class and renamed the exising test class, so the full diff view isn't very useful.

Together with https://github.com/Microsoft/PTVS/pull/2515, all profiling tests pass now.

👍


Fix broken LDAP connector test case syntax

I noticed that a couple of pillar test cases were passing despite an error in the test.  This PR adjusts those test cases so they correctly fail upon discrepancies.
The diff is misleading because of indentation; there isn't that much changed.  Just an extra context wrapper to facilitate a begin block calculating unique values, use of those variables, and removing a couple of unneeded lines.

The factory doesn't return the number assigned by the internal sequence (or rather it doesn't do so in a way obvious to me), so the auto increment database ID doesn't necessarily reflect the value the factory uses. This provides predictable uniqueness.

The factory doesn't return the number assigned by the internal sequence (or rather it doesn't do so in a way obvious to me), so the auto increment database ID doesn't necessarily reflect the value the factory uses. This provides predictable uniqueness.

What do you mean by "The factory doesn't return the number assigned by the internal sequence"? Maybe an example of what is not happening and you would like it to happen?

Sure. The factory has a sequence which is used in generating the hostname and connector name. Each time the factory is used, that number increments. In these tests, we need to be able to match those fields. When this test is run stand-alone, the database auto-increment id field and sequence both start at the same point, so we can get the hostname using the database id. Because of the db cleaning, when the whole test suite runs, though, the database ID might be 5 while the sequence is 27.
We could use a static string. That would fail a unique name constraint, though, and makes debugging harder. What we want here is to ensure that the row we inserted is what we get back, which means a unique name. So this uses the test name to generate that unique ID separate from the factory sequence, saving us from relying on a relationship between the auto-increment field in the database and the factory sequence.

You are passing the generated manually through the constructor and expecting it. Woudn't it be the same as using dex_connector_ldap.name and dex_connector_ldap.host? It's already done with dex_connector_ldap.id.

This is verifying that what we expected to store is returned. That would be
verifying that was was returned is returned. :)
On September 28, 2018 2:42:07 PM wrote:

You are passing the generated manually through the constructor and
expecting it. Woudn't it be the same as using dex_connector_ldap.name and
dex_connector_ldap.host? It's already done with dex_connector_ldap.id.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

As we go farther down the road of using the factory-generated object for data, we get closer to validating only the format of the response and not the data content. While that's fine for what this test ought to be doing, is that still a sufficient test with the other connector tests that are defined elsewhere? In other words, Is the model being fully exercised elsewhere such that it's ok to basically validate only the JSON structure here? My initial update did not make that assumption.

As we go farther down the road of using the factory-generated object for data, we get closer to validating only the format of the response and not the data content. While that's fine for what this test ought to be doing, is that still a sufficient test with the other connector tests that are defined elsewhere? In other words, Is the model being fully exercised elsewhere such that it's ok to basically validate only the JSON structure here? My initial update did not make that assumption.

The spec changes you've submitted belongs to the PillarsController. The controller has a single endpoint and its only purpose is return a JSON. Our responsibility in this context is to test whatever affects the return of that JSON.

The spec changes you've submitted belongs to the PillarsController. The controller has a single endpoint and its only purpose is return a JSON. Our responsibility in this context is to test whatever affects the return of that JSON.

Agreed.  With commit 2dba58d I've moved all of the JSON assembly logic into the expected_json definition.  That reduced the content of the test examples to merely varying by a couple of constant parameters, so I pulled those out into a loop.  The code has less repetition now.  As a side benefit, it also now tests all four possible combinations of bind method and security negotiation rather than just two.
I also agreed with @nanoscopic's reasoning and elected to leave the destroy call in as a comment, so some future person who finds a data persistence issue might have a starting point.  I think that (and the above) addresses all of the requested changes.
If we're good here, I'll squash the commits in the branch and repush.

Oh; I didn't switch to the let block because I don't see it adding anything here.  The object changes across every example and is used in every example, so the lazy evaluation doesn't save anything.  The example is so compact that, IMHO, the parens and colons and braces in let add more distracting visual clutter than simply using an = for a local var.


Error on usage

I receive this error while drawing the map:
Error: layers[35].filter[3][1][0]: Unknown expression "pitch". If you wanted a literal array, use ["literal", [...]].
The first part changes, but it's always highlighting the 'Unknown expression "pitch"'.

I am hitting the same error with a new mapbox style parsed through the request transformer.

This issue is not related to the request transformer. I assume an update attempt was made on the mapbox style. Mapbox Studio styles must support MapboxGL 1.1. If you auto update you will change the projection to globe [from mercator] and introduce the pitch filter which will break compatibility with maplibre.

Thanks for investigating @dbauszus-glx
If there is a particular map style someone can point me to I can investigate further.


Support both integer and double forms of aggregate histograms

From @metasim on August 25, 2017 16:18
Currently only double (which are approximate) are supported. Need to come up with a consistent naming scheme for double vs integer forms of things (like GT does).
Copied from original issue: s22s/raster-frames#4

Should just require creating an Int version of this:
https://github.com/s22s/raster-frames/blob/5fb23a260d514478c5480f847572b90104326b30/src/main/scala/astraea/spark/rasterframes/functions/AggregateHistogramFunction.scala#L32


Can't use run_if on a SystemSet group

Bevy version
0.11.3
What you did
I'm trying to make system sets only run in a specific state. Unfortunately, adding a run condition to a whole group of system sets at once doesn't seem to work. I can chain and order the group just fine--it's only run conditions that don't work:
.configure_sets(
	PostUpdate,
	(
		CollisionPhaseInternal::Prepare,
		CollisionPhase::CollideLayers,
		CollisionPhaseInternal::ShortenVelocity,
	)
	.chain().before(apply_velocity) // these work fine
	.run_if(in_state(crate::GameState::Playing)) // this doesn't compile!
)

Additional information
The exact error given by Rust Analyzer:
the method `run_if` exists for struct `SystemSetConfigs`, but its trait bounds were not satisfied
the following trait bounds were not satisfied:
`SystemSetConfigs: IntoSystem<(), (), _>`
which is required by `SystemSetConfigs: bevy::prelude::IntoSystemConfigs<_>`
`SystemSetConfigs: bevy::prelude::SystemSet`
which is required by `SystemSetConfigs: bevy::prelude::IntoSystemSetConfig`
`&SystemSetConfigs: IntoSystem<(), (), _>`
which is required by `&SystemSetConfigs: bevy::prelude::IntoSystemConfigs<_>`
`&SystemSetConfigs: bevy::prelude::SystemSet`
which is required by `&SystemSetConfigs: bevy::prelude::IntoSystemSetConfig`
`&mut SystemSetConfigs: IntoSystem<(), (), _>`
which is required by `&mut SystemSetConfigs: bevy::prelude::IntoSystemConfigs<_>`
`&mut SystemSetConfigs: bevy::prelude::SystemSet`
which is required by `&mut SystemSetConfigs: bevy::prelude::IntoSystemSetConfig`

System set definitions, found in the same module:
#[derive(SystemSet, Debug, Hash, PartialEq, Eq, Copy, Clone)]
enum CollisionPhaseInternal {
	/// Clears recoil at the start of the collision phase,
	/// so that [CollisionPhase::CollideLayers] can write to it
	/// cumulatively.
	Prepare,
	/// Applies the result of [CollisionPhase::CollideLayers] to
	/// [Velocity2D]s before [apply_velocity] runs.
	ShortenVelocity,
}
#[derive(SystemSet, Debug, Hash, PartialEq, Eq, Copy, Clone)]
pub enum CollisionPhase {
	/// Collide all layers, in no particular order.
	CollideLayers,
	/// Update the representation of the world for physics tests,
	/// for all layers in no particular order.
	UpdatePhysicsWorld,
}

And in main.rs, crate::GameState:
#[derive(bevy::prelude::States, Default, Debug, Copy, Clone, Hash, PartialEq, Eq)]
pub enum GameState {
	#[default]
	Init,
	LevelLoading,
	Playing,
	Paused
}

Notably, everything works fine if I distribute the run_if across all system sets in the group, like so:
.configure_sets(
	PostUpdate,
	(
		CollisionPhaseInternal::Prepare.run_if(in_state(crate::GameState::Playing)),
		CollisionPhase::CollideLayers.run_if(in_state(crate::GameState::Playing)),
		CollisionPhaseInternal::ShortenVelocity.run_if(in_state(crate::GameState::Playing)),
	)
	.chain().before(apply_velocity)
)

Can you reproduce this on 0.12?

In a hastily-constructed test project with only the code I've mentioned, it appears I cannot.


Adding 3 new attributes to PriorBox op

While preparing the new version of specification we lost 3 new attributes for PriorBox. Now they are restored. Shape inference function is aligned with one that is implemented in OpenVINO ModelOptimizer considering those three attributes. Code that calculates the number of priors is moved to a separate function to be easily re-used in 3rd party code.

@slyalin Could you fix unit tests?
[  FAILED  ] 2 tests, listed below:
[  FAILED  ] type_prop_layers.prior_box1
[  FAILED  ] type_prop_layers.prior_box2

Created another PR to rov.4 branch: https://github.com/NervanaSystems/ngraph/pull/4114


Allow music to be played in background for Safari on iOS

Hello and thank you for this amazing project !
To date, when playing audio from Safari on iOS, webamp music can't be played in the background. Not even with this "manually playing with the control center" trick.
I am not talking about autoplay, but only playing in the background after pressing the "play" button.
A lot of web-based players can play in the background, for instance the native player on archive.org.
I am clueless about HTML/JS but it seems that Safari on iOS needs a proper <audio> tag in the HTML of the web page.
Can you adapt webamp to allow playing in background ?
Cheers!

I'm happy to look into this if someone can identify exactly what it is within Webamp that's causing it to not be playable in the background, or if someone is able to find some documentation about under which conditions iOS lets a webpage play audio in the background.

i think i found out what it is. so iPhone doesnt like javascript. you can enable it to run on mobile iOS, but the problem is is that it doesnt like javascript running in the background, ie when you swipe out. i ben trying to figure out ways into tricking it by messing w audio tags, basically trying to make it seem like its html audio coming from the site rather than java. am i making sense?

Sadly I don't think that will work. Webamp must attach JavaScript listeners to the audio output in order to do balance, eq, visualizers. Probably not very practical to disable all that stuff.

Thank you all for quickly looking at this issue.

Sadly I don't think that will work. Webamp must attach JavaScript listeners to the audio output in order to do balance, eq, visualizers. Probably not very practical to disable all that stuff.

If we could have, from a user point of view, a "simple" mode where these advanced features of equalizer etc. can be turned off (with a button/switch or whatever), in favor of being able to play in the background, it could be great !
Again, I am totally clueless in HTML/JS and iOS stuff, but I found these:

https://github.com/jellyfin/jellyfin-web/issues/6113
https://bugs.webkit.org/show_bug.cgi?id=198277
https://www.vindit.dk/play-audio-on-demand-in-safari-on-ios/
https://github.com/goldfire/howler.js/issues/239
https://stackoverflow.com/questions/14414654/stop-html5-audio-from-looping-when-ios-safari-is-closed
https://developer.apple.com/documentation/webkit/delivering_video_content_for_safari/
https://developer.apple.com/library/archive/documentation/AudioVideo/Conceptual/Using_HTML5_Audio_Video/Introduction/Introduction.html
https://github.com/advplyr/audiobookshelf/issues/2655
https://discussions.apple.com/thread/253331448?sortBy=rank
https://www.reddit.com/r/jailbreak/comments/kdaqvu/question_safari_audio_with_phone_locked/?rdt=36668

And again, the native media player from archive.org works correctly with Safari on iOS for background music: https://github.com/internetarchive/iaux-music-player

Any chance to have some fix ?


Create DataFrame.reject function

Currently, we have a filter function for DataFrames, but a usecase I have ran into a few times is that I want to return elements that don't match my DataFrame.filter()
The way I do this at the moment is like so
original_dataframe = ...

elements_i_dont_want = DataFrame.filter_with(...)

elements_mapset = elements_i_dont_want |> DataFrame.to_rows() |> MapSet.new()
original_dataframe = original_dataframe |> DataFrame.to_rows() |> MapSet.new() |> MapSet.difference(elements_mapset) |> MapSet.to_list() |> DataFrame.new()

This solution seems a bit suboptimal. I think a DataFrame.reject() function would be a lot cleaner.
Thoughts?

@appdevland I want to clarify that converting a dataframe to rows is a very expensive operation and you want to avoid doing that. Other than that, you should be able to reject elements by placing a Series.not at the top of filter_with. Is there any issue i doing so?

Oh, sorry! I missed that when looking through the docs. Thanks for pointing it out!

Cool. Closing this for now then! I feel like reject would be easy to implement, but we probably want to keep the amount of verbs low :)


Image-feature

Image feature that extracts images of diagrams/tables/equations, etc.

Done. Updated in v2.5.0.


HIVE-28576: Add jdbc tests for tpcds queries

What changes were proposed in this pull request?

Add a new test driver for jdbc tests TestMiniLlapLocalJdbcCliDriver to create tpcds tables in dockerized postgres, and create corresponding external tables. Generate explain plans for tpcds queries.
Why are the changes needed?

To improve JDBC test coverage
Does this PR introduce any user-facing change?

No
Is the change a dependency upgrade?

No
How was this patch tested?

mvn test -pl itests/qtest  -Pitests -Dtest=TestMiniLlapLocalJdbcCliDriver -Dtest.output.overwrite=true

@soumyakanti3578, why should it be MiniLlapDriver?

@deniskuzZ No, TestTezTPCDS30TBPerfCliDriver generates logical plans and uses a stats dump. This is different because here we actually create tpcds tables in Postgres, create external tables in Hive corresponding to the tables in Postgres, and then generate the jdbc plans.

@deniskuzZ No, TestTezTPCDS30TBPerfCliDriver generates logical plans and uses a stats dump. This is different because here we actually create tpcds tables in Postgres, create external tables in Hive corresponding to the tables in Postgres, and then generate the jdbc plans.

why not simply parameterize the db to run tests against? default is derby, but can't be overridden. why create a separate driver for that?

@deniskuzZ

why not simply parameterize the db to run tests against?

Only option I saw was to add something like --!qt:database:postgres:<script-to-run-in-postgres>. Did you have this in mind as well? I didn't go with this approach because I need to run all tpcds queries on external tables. There are 2 ways to do this:

Have a single qfile - test_all_tpcds_queries_postgres.q with --!qt:database:postgres:q_test_tpcds_tables.postgres.sql (q_test_tpcds_tables.postgres.sql) on the top of the file to create all tpcds queries in postgres. Then we would need the contents of q_test_external_tpcds_tables_postgres.q in test_all_tpcds_queries_postgres.q to create all external tables. And then, we will need all tpcds queries too. This would be a huge file, and the resulting qout file could be very difficult to maintain too.
Other approach that I can think of is to have individual q files for each tpcds query, but then we will need to create <script-to-run-in-postgres> for each one of them if we want to only create the required tables in postgres. We can also just create all tpcds tables for each q file but that would be inefficient I guess. We would still need to create external tables in each q file. For example, for cbo_query3.q, we would need to create 3 jdbc tables (date_dim, store_sales, item) in <script-to-run-in-postgres-for-cbo_query3>, and 3 external tables in cbo_query3_postgres.q. This could become very difficult to maintain too.

With the new driver, we can configure CliConfigs.java to run 1 file to create all tpcds tables once with
setJdbcInitScript("q_test_tpcds_tables.postgres.sql");

and run 1 file to create all external tables once with
setExternalTablesForJdbcInitScript("q_test_external_tpcds_tables_postgres.q");

and in CoreJdbcCliDriver.java, we can launch the docker container before starting the tests with beforeClass(), run all the tests, then stop the docker container in shutdown() method.
I think this approach is easier to maintain.
But please let me know what your thoughts are regarding this, and if you have something else in mind too!
Thanks!

hi @soumyakanti3578, sorry for not replying, please give me some time, I am overwhelmed with diff requests atm

@deniskuzZ No problem at all. Thanks for looking into this :)


Dft reverse axis

Handle decreasing coordinates in dft with true_phase = True.
Add a test to check

@roxyboy, merge ?

I'm personally ok to merge but I think we should wait for @rabernat 's approval as well. He should be back from vacation next week.
We should also probably update the docs page using realistic data from the xarray tutorial (e.g. #148 ).

Thanks! So is the plan to defer the documentation updates to a future PR? If so, I will merge.

Thanks! So is the plan to defer the documentation updates to a future PR? If so, I will merge.

I give a thumbs up to a separate PR for docs :)


Add controller test for users#add_pregnancy and users#remove_pregnancy

Adds a few bells and whistles to the routes, a rescue_from block to the users controller in case a pregnancy or id isn't found, and some controller behavior tests for call list building methods.
Thanks to @ajohnson052 for making these methods super easy to write tests for !
Fixes #125

I'll let this marinate until the end of today and then merge.


Make sure mixing masternode follows bip69 before signing final mixing tx

utACK (only checked for obvious errors, as I'm not familiar with the touched code)
Can you explain why bip69 is needed here?

The idea behind bip69 https://github.com/bitcoin/bips/blob/master/bip-0069.mediawiki is that instead of shuffling inputs/outputs we can define deterministic and easily verifiable input/output ordering and make sure that no info is leaked. Dash Core already creates all txes this way, inc. mixing txes on masternode side. So this part of code basically ensures that masternode didn't "tweak" tx in any way to leak information about mixing participants.


THRIFT-5523: [rs lib] use latest rust toolchain and edition 2021

[x] Did you create an Apache Jira ticket?  (not required for trivial changes)
[x] If a ticket exists: Does your pull request title follow the pattern "THRIFT-NNNN: describe my issue"?
[x] Did you squash your changes to a single commit?  (not required, but preferred)
[x] Did you do your best to avoid breaking changes?  If one was needed, did you label the Jira ticket with "Breaking-Change"?
[x] If your change does not involve any code, include [skip ci] anywhere in the commit message to free up build resources.

not sure if the build can be fixed by merging

https://github.com/apache/thrift/pull/2522
first

@Jimexist Which version of Rust would you prefer we standardize on: 1.50 or 1.60? My take is 1.60, since that allows us to update to the 2021 edition as well?

this shall be rebased upon:

#2628

see #2628

So, merge #2628 first and then update everything to 1.61?
On Fri, Jul 1, 2022 at 11:50 AM Jiayu Liu @.***> wrote:

this shall be rebased upon:

#2628 https://github.com/apache/thrift/pull/2628

—
Reply to this email directly, view it on GitHub
https://github.com/apache/thrift/pull/2527#issuecomment-1172481737, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAQ55DC2LCOB6XVRXXKKJDVR4HV7ANCNFSM5O76TMDA
.
You are receiving this because you commented.Message ID:
@.***>


Adding support for in-source builds

This adds a check for in-source builds.
There is one caveat to this PR that I would like to discuss. Since this uses require to read the bsconfig to check if the build should be done in-source, it requires the bsconfig to be valid JSON, which is not required by bucklescript since it ships it's own JSON parser.
If there's some way to use that JSON parser instead that would be great, otherwise this would need to be a limitation of using bs-loader (which IMO is fine, but could be confusing).

Don't worry about the Travis build failing, there's some permission problems on the CI 😄

Yeah that sounds good 👍. We should probably document it then to make sure people don't forget to change their bsconfig when using this.
I'll add something about it as well.

I've fixed what we discussed (except I forgot to add docs). For some reason I can't get the tests to run locally any more (not even before I made the change).
Anyways, this should be equivalent to the bsconfig parsing thing I did before. 👍

I'm just gonna see if I can get the tests running locally, then merge this in!


Fixed: #24227: Partially revert 983c158da

https://code.djangoproject.com/ticket/24227

Same as https://github.com/django/django/pull/6411?

Oops! Yes :)


Update apache.json

Fixed Apache file location. Upgraded Apache to 2.4.16

Hi @barohatoum
You haven't updated the hash from the old version.

I realised that..will do later
On Aug 19, 2015 8:32 PM, "Simon Hartcher"<EMAIL_ADDRESS>wrote:

Hi @barohatoum https://github.com/barohatoum
You haven't updated the hash from the old version.
—
Reply to this email directly or view it on GitHub
https://github.com/lukesampson/scoop/pull/468#issuecomment-132836454.

:+1:


Document tf-coreml converter in lite/README.md

Can one of the admins verify this patch?

@sarahs1 FYI


issue importing from Android Studio 1.1.0 & add .gitignore & bump minSd...

...k from 8 to 11
issue importing from Android Studio 1.1.0
https://github.com/googlesamples/android-NetworkConnect/issues/2
add .gitignore
https://github.com/googlesamples/android-NetworkConnect/issues/3
bump up minSdkVersion from 8 to 11 (android v3)
https://github.com/googlesamples/android-NetworkConnect/issues/4

Signed CLA.

Thanks for the pull request. Please either update the email address on the commit(s), sign the CLA with the corresponding email address or create a new pull request with the correct address. Otherwise we won't be able to accept your contribution.

Hi Benjamin,
Adding e-mail to git commits, makes it public visible. Not safe.So, i added this e-mail (the e-mail i use on git for this commit).In the CLA, i added my git username (sreev).
Your this process seems a bit out of the normal.What is the ultimate intention behind it ?
I found, the similar change is needed for other examples apps as well.
Thanking you.
With Regards
Sree
 On Thursday, March 5, 2015 5:41 AM, Benjamin Weiss<EMAIL_ADDRESS>wrote:

Thanks for the pull request. Please either update the email address on the commit(s), sign the CLA with the corresponding email address or create a new pull request with the correct address. Otherwise we won't be able to accept your contribution.—
Reply to this email directly or view it on GitHub.

sreev: CLAs are used by many large open source projects, including Apache, Python, Canonical, Eclipse, etc. See http://en.wikipedia.org/wiki/Contributor_License_Agreement for a partial list.
We require the email address to be correct, because GitHub is not our system of record. Once your commit is exported to our internal Git repository, we still need a way to track it and ensure that you've signed the CLA. The only field that Git gives us to do this is the email address.
Similarly, replying to this thread via email is not sufficient, because (again) GitHub is not our system of record and that information is not preserved in Git's metadata.
You can update your commit by using "git commit --amend --reset-author".

On second thought, I'm going to go ahead and close this PR, since these are issues that need to be fixed across multiple samples, and the changes are simple. It's easier if I just address these bugs upstream, that way we can get everything fixed simultaneously. (We have some tools that allow us to bulk-edit all the samples simultaneously.)
That said, thanks for the contribution. I know the CLA adds some extra hoops to jump through, but we do appreciate it.


fix: articles/active-directory/saas-apps/hyperanna-tutorial.md

MD038/no-space-in-code Spaces inside code span elements
MD039/no-space-in-links Spaces inside link text

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

space corrections confirmed

#sign-off


perf(scroll-dispatcher): lazily subscribe to global events

Switches the ScrollDispatcher to only subscribe to scroll and resize events if there any registered callbacks. Also unsubscribes once there are no more callbacks.
Fixes #3237.

Can you add tests for this? At minimum we should check that globalSubscriptions is null when there are no registered scrollables, and it exists when at least one scrollable is registered.

It feels weird having to make the subscription public only to be able to test it, but I should be able to use a spy to check whether stuff gets invoked.

Just thought about the case of having no cdk-scrollables on the page but still wanting to respond to scroll events. We'll still want the tooltip to reposition in this case.
Does it make more sense to assign the global subscriptions when scrolled is called? It's invocation shows that there is intent from some component to listen to the scroll event.

I see your point @andrewseguin, but I'm not sure how we'd know to unbind the global handler once there are no more scrolled subscriptions.

Changed as discussed @andrewseguin. Could you take another look?

I might not understand all the stuff going behind the scene, but why are you not using RX operators such as refCount here?


Incorrect display of days in the calendar

You have incorrectly displayed days in your calendar. Look at the first and second panes and then compare it. Can you fix it?

the pic blocked @crimsonimp

Don't worry @crimsonimp . I found the bug on the Vue2 Calendar live demo! April 1, 2019 falls on a Monday. The Vue2 Calendar widget displays it as occurring on a Sunday.
http://blog.w3cub.com/vue2-calendar/demo/

I fixed it here https://github.com/icai/vue2-calendar/pull/41


Use a larger address space for Azure NetApp Files VNet than for its delegated subnet

According to Azure NetApp Files networking team, following a case raised by a customer, if the size of Azure NetApp Files VNet address space and the size of its delegated subnet address space are the same, this will cause the failure to create SMB volumes. They said this is a known issue, but this is not documented anywhere and I don't think all NetApp and Microsoft cloud solution architects are aware of it.
When a volume is mounted on a VM in the same VNet as Azure NetApp Files, the CIDRs are necessarily different because we need a subnet for the VMs, in addition to that of Azure NetApp Files. On the other hand, when a volume is mounted on a VM in a peered VNet (VNet peering between the VNet of the volume and the VNet of the VM), a customer may consider that a single subnet is needed and that the same CIDR for the VNet and the subnet is sufficient whereas it doesn't work and it's not mentioned anywhere.
To the section https://learn.microsoft.com/en-us/azure/azure-netapp-files/azure-netapp-files-network-topologies#subnets, I propse to add the following:
The address space size of Azure NetApp Files VNet should be larger than the one of its delegated subnet, otherwise this will cause the failure of SMB volumes creation.

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

ID: 31580c1b-34e7-acad-dfed-db47f7dae39f
Version Independent ID: 8a07d877-784a-3d88-54cd-8cd5341a2859
Content: Guidelines for Azure NetApp Files network planning
Content Source: articles/azure-netapp-files/azure-netapp-files-network-topologies.md
Service: azure-netapp-files
GitHub Login: @ram-kakani
Microsoft Alias: ramakk

There was an error when I submitted the issue #107240, so I submitted again, but after a checking the issues list, I saw it was there.


Java 9 fixes

Linked to https://github.com/cbeust/testng/issues/938
Two maven plugins, maven-jar-plugin and maven-javadoc-plugin also have issues with Java 9.
Once https://github.com/apache/maven-plugins/pull/76 is merged it will depending on a fixed version of plexus-archiver (see https://github.com/codehaus-plexus/plexus-archiver/pull/12).

Please, have a look on #947


Variables in arguments should be lazily evaluated

Variable values are substituted into commands by the Gitsh::Transformer as part of the parsing process. The result is that a Gitsh::Command::Tree::*—which composes two or more commands—doesn't work as expected when the LHS modified a variable that is used on the RHS.
Lazily evaluating argument values as part of the Gitsh::Command::*#execute method should solve this problem.
The expected behaviour is:
% :set x "1"
% :set x "2"; :echo $x
2
% :echo $x
2

The actual behaviour is:
% :set x "1"
% :set x "2"; :echo $x
1
% :echo $x
2

Now that unset variables are going to be an error, when they're lazily evaluated it should also be possible to handle this kind of situation:
% :set b "This is B"
% :echo $a || :echo $b
This is B

This is more-or-less done on the gb-lazy-variables-original branch. Waiting on #210 to open a PR.


Use Adam File with Hive

Hi everyone,
I´m really new in the field of Hadoop / Hive / Spark so I need some help:
I`ve opened the adam-submit shell and used the vcf2adam command to convert a vcf file to an adam formatted one. After that I got a folder within some .parquet data sets.
Now I try to import them into a HIVE table on my Hadoop cluster to query them. The import data command works fine but when I try to run a SELECT * FROM parquet_table; I get different errors.
Do you have any Ideas?

Closing as need more info to reproduce.


feat(log,notify): notification of action log exceed count

What this PR does / why we need it:

Does this PR need to be backport to the previous release branch?:

3.9
/cc @swordqiu
/area notify logger
/hold

/hold cancel


Improve user interface for changing plot defaults

Currently users have to access the default plotting parameters directly through the dictionary. It would be nice to change this structure so that it wasn't so cumbersome and a little more intuitive

fixed in f6ad07bbb43b622881f9a418d406f362302bea75


It does not work !

Hello !
I am trying to use a LED Display and an ESP32.
Display: 2727 SMD P5 64x32
Board: ESP32 Wrover-B DevKitV4
I've done the wiring as it was written in the ReadMe. I thought I did a mistake so I redid the wiring and still nothing.
From the Left HUB75 to the Right HUB75 there are:
R2 to R1
G1 to R2
G2 to G1
B1 to G2
B2 to B1
Where do I put Ground to ?
The only Ground I have is that on the ESP32. There are no pins on either the HUB75 that go into the Ground. Is that how it's supposed to be ?
I've also redefined the GPIO as follows:
#define P_LAT 22
#define P_A 19
#define P_B 23
#define P_C 18
#define P_D 4
#define P_OE 25
I don't understand what to do with E. Is E a GPIO on the ESP or is it a pin on the HUB75 ?
I've tried using the Pattern_Test example:
#include <PxMatrix.h>

#define P_LAT 22
#define P_A 19
#define P_B 23
#define P_C 18
#define P_D 4
#define P_E 15
#define P_OE 25

hw_timer_t * timer = NULL;
portMUX_TYPE timerMux = portMUX_INITIALIZER_UNLOCKED;

// Pins for LED MATRIX

//PxMATRIX display(32,16,P_LAT, P_OE,P_A,P_B,P_C);
PxMATRIX display(64,32,P_LAT, P_OE,P_A,P_B,P_C,P_D);
//PxMATRIX display(64,64,P_LAT, P_OE,P_A,P_B,P_C,P_D,P_E);

void IRAM_ATTR display_updater(){
  // Increment the counter and set the time of ISR
  portENTER_CRITICAL_ISR(&timerMux);
  //isplay.display(70);
  display.displayTestPattern(70);
  portEXIT_CRITICAL_ISR(&timerMux);
}

uint16_t myCYAN = display.color565(0, 255, 255);
void setup() {
  // put your setup code here, to run once:
Serial.begin(9600);
  display.begin(16);
  //display.setScanPattern(x);
  //display.setMuxPattern(STRAIGHT);
  display.flushDisplay();
  display.setTextColor(myCYAN);
  display.setCursor(2,0);
  display.print("Pixel");
  Serial.println("hello");

   timer = timerBegin(0, 80, true);
   timerAttachInterrupt(timer, &display_updater, true);
   timerAlarmWrite(timer, 2000, true);
   timerAlarmEnable(timer);

  delay(1000);
}

void loop() {

 delay(100);

}

I've tried using display.setScanPattern(x) with all the values: LINE, ZIGZAG, ZAGGIZ, WZAGZIG, VZAG.
I've also used: display.setMuxPattern(x) WIth BINARY and STRAIGHT.
And nothing changes the output. It stays the same with either functions.
With or without those 2 function the result is the same gibberish.
And the pattern test isn't like the one in your .gif. Mine is different with shifted rows, wrong colors, etc.
What do you think the issue is ?

Is this library not receiving support anymore ?


Decompilation broken for MIPS:BE:32:default

Describe the bug
Decompilation fails with error
 Low-level Error: Could not find op at target address:
To Reproduce

have MIPS:BE:32:default asm
opcodes clearly gets disassembled
functions are found, but
on decompilation it fails with above mentioned message

Screenshots

Environment:

OS: Win7
Java Version: 12.0.2
Ghidra Version: 9.1-BETA

Additional context
I have developed a loader for N64 Roms and want to upgrade it for ghidra 9.1 (the code in that repo was already upgraded for 9.1 but the dist was left for 9.0.4)

Can you post the decompile debug .xml output file for this function?  The issue may not be at the instruction, as I can't duplicate the issue with a similar set of instructions.

Sure, here 4 examples
debug.zip

For what it's worth, I'm seeing the same issue with the SH2 processor and 9.1-Beta, so probably not CPU related.
This is from a database upgraded from 9.0.4 and I'm also using a bunch of overlays. Let me know if you need extra info.

Possilbe overlay related.  It may be related to the new syscalls features, or the upgrade from 9.0.4.

@zeroKilo are any of your memory blocks overlays?

well im not sure what you mean by overlayed. I create an InputStream for most blocks from a single byte[] array, the rom, and the 2 .spXXX blocks that are zeroed byte arrays. these blocks dont overlap in memory, the layout is like following:

here the bytestring of fun_800007ec
here the disassembly of fun_800007ec
here the actual rom file
here the function making blocks
here is how the makeblock function is used

I reproduced the issue on my end with a clean new project in 9.1-dev and 2 segments (still with SH2)
Should I just attach the .gar file here? (1.7mb)

This is definitely an issue with overlays.  Your second parameter to MemoryBlockUtil.createInitialedBlock is actually creating an overlay block.
There appears to be a lookup issue with the decompiler and overlays, possibly associated with the new reference overrides for syscalls.

yes you seem to be right, changing the second parameter to false lets the decompilation finish

is there anywhere to read about what overlays mean? I still dont understand the difference, whats it used for normally?

@zeroKilo overlays are used when you have multiple code segments that can be loaded at the same address in memory. Their address would overlap in memory, so you need to create them a separate overlays.
The root issue remains, as overlays are basically unusable in 9.1 due to this.

Agreed.  The overlay issue is a regression related to system call override changes, it is being diagnosed and will be fixed before release.

@yaz0r thanks for the explanation

I would hardly call this issue resolved. @emteere do you need a new issue to track the overlay problem or want to reopen this?

Fixed by bf4a17b67dff93b6a2555ceb4eed9ba85d019e05


Fix the substrate networking Grafana dashboard.

As mentioned in PR #7038 :

Most importantly, the list of nodes was fetched from what the cpu_percentage metric reports, and would therefore show as empty.

Since the sysinfo has been removed from the Prometheus metrics, the metric we used to extract instances ${metric_namespace}_cpu_usage_percentage no longer exists. However, #7038 only updated the substrate-service-tasks dashboard, this PR fixes the substrate-networking dashboard.
This PR also:

Changes the time range from 24h to 12h and disables auto-refresh, in order to be consistent with the substrate-service-task dashboard.
Updated the dashboard JSON to Grafana 7.0 (I'm also willing to submit another PR to update the substrate-service-task dashboard).

Hi!
Sorry, I was waiting for 0.8.24 to be widely deployed before updating this dashboard because some metrics have been modified (#6634) in that version.
Do you mind if I open a PR myself that fixes the dashboard and also updates the metrics for 0.8.24?

Hi!
Sorry for that, I was waiting for 0.8.24 to be widely deployed before updating this dashboard because some metrics have been modified (#6634) in that version. Releasing 0.8.24 took way more time than expected.
Do you mind if I open a PR myself that fixes the dashboard and also updates the metrics for 0.8.24?

Oh, thank you for reminding me, I did't even notice that there was another change of the metrics. It would be great if you can open another PR to fix the dashboard and update the metrics as well! Feel free to close this one.


502 error on large previews when SESSION_ENGINE is set to signed_cookies

Issue Summary
We are trying to preview a page on wagtail. It seems previews are store in session unlike drafts. And we have  a huge description text. So we are getting 502 error on preview.
Cookie length should be less than or equal to 4096 characters.
Drafts and live are working fine with this session engine.
Steps to Reproduce

Set SESSION_ENGINE = 'django.contrib.sessions.backends.signed_cookies' in settings.py
login to cms
go tp pages
create a new child page or edit existing.
Add big text in description i.e sample text
click preview

cc @pdpinch
Screens:

Technical details

Python version: 3.6
Django version: 2.0.2
SESSION_ENGINE = 'django.contrib.sessions.backends.signed_cookies'

Wagtail version: 2.0
Browser version: chrome Version 66.0.3359.139 (Official Build) (64-bit) osx

Thanks for the report @amir-qayyum-khan.
For reference, preview data is stored in the session since #3383. While #3583 reduces preview data size, the issue will essentially still exists.
The fact that it might exceed the cookies max-size was raised at the time but it was agreed to add a warning to the documentation (although it seems to have never happened) given that not using the session probably meant creating an API endpoint which would have been a lot more work. Maybe it will be time to revisit this.
In the meantime, you should be able to configure uwsgi to accept bigger cookies (IRRC, nginx has some sort of auto-scaling but uwsgi doesn't).

thanks @loicteixeira for you feedback.
We already have uwsgi config https://github.com/mitodl/micromasters/blob/master/uwsgi.ini#L47. But still we are getting preview error.

@loicteixeira is there any particular reason not to treat previews as an as-yet unpublished revision? it seems as though that would be the most robust option for managing data that you would like to preview and eventually publish, though I'm happy to be proven wrong if there is something that I'm missing.

@blarghmatey By definition, I think a preview has to show the outcome of a proposed change without committing that change to the database (even just as a draft revision). If an editor hits preview, and then changes their mind about their edit, and navigates away without saving, the system should behave as if the edit was never made at all (and thus revisiting the edit view should take you back to the page's original state).

Agreed that at the moment, it doesn't seem right to save a revision for a preview.
Once revision data is decoupled from page data with RFC29, I imagine it would be possible to save a revision for the sole purpose of previewing as it would be possible to "tag" a revision as preview and exclude it from the list of revisions the user can look at (to reduce noise) and/or make sure a "preview draft" always deletes the previous one (so we don't overload the DB). I'm not sure that what we would like to do though, but it would certainly be less of a problem than with the current implementation of revisions.
In the meantime, if setting up uswgi and nginx doesn't help, the session engine could be set to anything but cookies (although it might not be possible for all projects).

RFC 29 was not adopted.
https://github.com/wagtail/rfcs/blob/master/text/029-immutable-revision-events.md
Updated title to include Preview and adding labels.


Replace double-quote with single-quote

Replace double-quote with single-quote for terminal commands.
Although the strings in the examples work fine, users who with randomly generated passwords may have character strings that cause strange behaviors with double-quotes. Using single-quotes prevents this.
Example of a command that causes odd behaviors with double-quotes:
echo -n "RandPasswordr3!$#$3aR" | base64
This example has a series of special characters that some terminals try to interpret and replace with other text.

/assign @bradtopol

/lgtm
/approve


Split out font table parsing into separate struct

This enables support for custom font table providers, which is necessary if you want to decode the font tables in some way before handing them to ttf-parser (i.e.: gzip decoding / brotli decoding, etc.).
Due to the Deref trait impls, this changes nothing about the public API - all methods that were previously available on the Font struct are now available on the ParsedFaceTables struct, too.

Everything is fine now.
Honestly, I don't really like this change. It complicates everything even more. But I must admit that yours solution is probably the best one.
PS: do I understand correctly, that WOFF compresses each table separately? Or why do you want to parse tables in this way?

I want to initialize the tables from decoded WOFF tables. In the WOFF2 file format, font tables are brotli-encoded as a single block. ttf-parser does not support brotli decoding (and that's okay, because I already use allsorts to decode the actual font). However, now I want to get the glyph outlines (using ttf-parser) - so I need some way to pass the already-decoded font tables to ttf-parser, so that I can call get_outline on the FontTables.
I know it's not great, but at first I tried copying the outline-parsing code into allsorts, but that was an even worse solution, so I left it at that. Right now I just let allsorts unzip the font tables, and then I create a cache of all outlines in the font (memory expensive, I know).

Can't you decode WOFF first and then pass it to allsorts/ttf-parser?

I've looked at the WOFF spec and looks like it compresses tables separately, therefore your patch is necessary anyway.

okay, could you release a new version so I can depend on it? thanks

Done.


Issue when deploying

Hello everyone,
I am writing this email to address some challenges we've encountered while attempting to deploy Firebase functions. Currently, when we deploy a single function using the command "firebase deploy --only functions:function_name," the deployment process works smoothly and the function is correctly deployed. However, when we utilize the command "firebase deploy --only functions" to deploy all functions, we're encountering an issue where all the functions end up with the same container image URL. As a result, this is causing requests to be directed to only one function instead of the intended target functions.
Thank you.

Hello @nikhilkotiya. Could you provide a reproducible code sample that I may test with? Could you also provide the output of pip freeze and your python version?

reproducible code
@on_document_created(document="geo_data/{userId}", region="asia-northeast1", max_instances=10)
def geo_grid(event: Event[DocumentSnapshot]) -> None:
print(' geo_grid')
return "done"
@scheduler_fn.on_schedule(schedule="*/2 * * * *", region="asia-northeast1", max_instances=10)
def helper(event: scheduler_fn.ScheduledEvent) -> None:
print(' helper')
return "done"
So when i run firebase deploy --only functions the container image URL is create same for both the functions it mean both the cron trigger only 1 function(not specified function but same). And their log is same also.
pip freeze
beautifulsoup4==4.12.2
blinker==1.6.2
bs4==0.0.1
CacheControl==0.13.1
cachetools==5.3.1
certifi==2023.7.22
pandas==2.0.3
cffi==1.15.1
charset-normalizer==3.2.0
click==8.1.6
cloudevents==1.9.0
cryptography==41.0.2
deprecation==2.1.0
dnspython==2.4.1
firebase-admin==6.2.0
firebase-functions==0.1.1
Flask==2.3.2
Flask-Cors==4.0.0
functions-framework==3.4.0
google-api-core==2.11.1
google-api-python-client==2.95.0
google-auth==2.22.0
google-auth-httplib2==0.1.0
google-cloud-core==2.3.3
google-cloud-firestore==2.11.1
google-cloud-storage==2.10.0
google-crc32c==1.5.0
google-events==0.10.0
google-resumable-media==2.5.0
googleapis-common-protos==1.59.1
grpcio==1.56.2
grpcio-status==1.56.2
gunicorn==20.1.0
httplib2==0.22.0
idna==3.4
itsdangerous==2.1.2
Jinja2==3.1.2
MarkupSafe==2.1.3
msgpack==1.0.5
ndg-httpsclient==0.5.1
packaging==23.1
proto-plus==1.22.3
protobuf==4.23.4
pyasn1==0.5.0
pyasn1-modules==0.3.0
pycparser==2.21
PyJWT==2.8.0
pymongo==4.3.3
pyOpenSSL==23.2.0
pyparsing==3.1.0
python-dotenv==1.0.0
PyYAML==6.0.1
requests==2.31.0
rsa==4.9
six==1.16.0
soupsieve==2.4.1
typing_extensions==4.7.1
uritemplate==4.1.1
urllib3==1.26.16
watchdog==3.0.0
Werkzeug==2.3.6
redis==4.6.0

python version Python 3.11

@exaby73  please look into this bug i have 30+ firebase functions. Its difficult for me to deploy single single function. @antont  Have the same issue.

@exaby73 please look into this bug i have 30+ firebase functions. Its difficult for me to deploy single single function. @antont Have the same issue.

Yep. I can confirm that this happened when trying to deploy all functions normally, but deploying one-by-one works. I am doing that for the few that have now, but it would not be practical for many.

Hello. Just a small update. I am working on this bug, trying to track it down and fix it. I think I'm close, so please bear with me :)

cool!

I had this issue. I upgraded to the latest firebase CLI and it fixed it. npm install -g firebase-tools

I had this issue. I upgraded to the latest firebase CLI and it fixed it. npm install -g firebase-tools

Yay that fixed it for me too! I had upgraded already earlier but hadn't retried this after that yet. Thanks for info!

Yeah fixed.
I have one more question, is there any way to use debugger with firebase functions

Based on https://github.com/firebase/firebase-functions-python/issues/128#issuecomment-1713923659, I will close this issue as fixed.


'fire' as pandas dataframe column throws ValueError with hvplot.scatter(y='fire')

software version info
pandas                   1.1.3
holoviews               1.13.4
hvplot                    0.6.0
jupyterlab               2.2.6
columns = ['earth','water','air','fire']
df = pd.DataFrame(data=np.random.rand(100,4)*np.random.randint(1,6,(1,4)),
                                            columns=columns)

df.hvplot.scatter(y='fire')

throws unexpected ValueError in jupyter notebook and jupyter lab.

ValueError [Call holoviews.ipython.show_traceback() for details]
Mapping a dimension to the "cmap" style option is not supported by the Scatter element using the bokeh backend. To map the >"fire" dimension to the cmap use a groupby operation to overlay your data along the dimension.

hv.ipython.show_traceback()

Traceback (most recent call last):
File "C:....\Continuum\anaconda3\envs\dc-cds\lib\site-packages\holoviews\plotting\bokeh\element.py", line 1323, in _init_glyphs
style = self._apply_transforms(element, data, ranges, style)
File "C:....\Continuum\anaconda3\envs\dc-cds\lib\site-packages\holoviews\plotting\bokeh\element.py", line 1092, in _apply_transforms
backend=self.renderer.backend))
ValueError: Mapping a dimension to the "cmap" style option is not supported by the Scatter element using the bokeh backend. To map the "fire" dimension to the cmap use a groupby operation to overlay your data along the dimension.

Similar code has no issues!
df.hvplot.scatter(y=['air','fire'])  # OK
df.hvplot.line(y='fire')  # OK

I can't reproduce.


UVI Request

--- UVI JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from v5.8 to before v5.10.43",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=7a88a6227dc7f2e723bba11ece05e57bd8dce8e4",
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=951f8ef71d691f754483cc9d871cb493379fdb35"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "7a88a6227dc7f2e723bba11ece05e57bd8dce8e4",
      "note": "introduced"
    },
    {
      "type": "commit",
      "value": "951f8ef71d691f754483cc9d871cb493379fdb35",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "efi/libstub: prevent read overflow in find_file_option()\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v5.10.43 by commit 951f8ef71d691f754483cc9d871cb493379fdb35, it was introduced in version v5.8 by commit 7a88a6227dc7f2e723bba11ece05e57bd8dce8e4. For more details please see the references link."
}
--- UVI JSON ---

This issue has been assigned UVI-2021-1000763


Make notifier have correct action, make use attachment card

The $BUILD variable wasn't in our environment variables for builds, so I copied the hipchat-notifier of checking for deploy and defaulting to build.
Also changed the message to be prettier, and use slack attachments instead: https://api.slack.com/docs/attachments
It now looks like this:

thanks for this @ibash! appreciate it and merged!


v2 On update scale, don't draw the line to last point

In bar-line chart, when scale is changed (number of labels), for example change from 6 elements to 3 (remove 3 labels). It doesn't draw the line to the last point.
See: http://jsfiddle.net/y5arojpo/
Attached some screenshots:

Please click "Remove April & May & June" to see bug

Good catch. I'll try and look into this tonight.

@etimberg Figure this one out?

The issue is that in the JSFiddle, the data is changed, but the remove functions are not used. removeData should be used. I think we should provide a function to remove an entire index including dataset label to make it easier.

As i was writing unit tests I think I found the problem. I'm trying to fix it as I go.

@waltermelow this is fixed by the code in PR #1427

Thank you very much.
Thanks for all your work on this great project :)


Researching APIs for desirability data (e.g. WalkScore)

[x] Start with what Apartments.com uses (like WalkScore) and find similar APIs online
[x] Evaluate each API by criteria including but not limited to:

Relevance of statistic(s) provided by API
Max # calls/day
Cost of usage, if any
Ease of use
Granularity

Complete above by Feb 23

[ ] Work with DB/Back-end and Recommender System owners to integrate any APIs into the project

Complete above by March 3

Walk Score's API provides

Walk Score
Bike Score
Transit Score
with 5000 calls/day for free. Annoyingly, while they also offer Public Transit Data and Transit Time, which would be very useful, it costs at least $100/mo. Would be good to figure out if Google Maps can take care of that for us, or if there's another API of similar quality that is free to use.

Has good documentation: https://www.walkscore.com/professional/api.php
Supports lookup by latitude+longitude
Great option overall, would be sufficient by itself but we may benefit from additional APIs

SDMTS's comprehensive static route dataset should give us the remaining information we need to inform users about the specifics of public transit, while not even requiring API calls. Using this dataset is not nearly as straightforward as the two APIs above, but if we can figure out a way to use it efficiently, it could be really useful to us. It is formatted in GTFS (General Transit Feed Specification), which is what Google Maps uses, so there is pretty good documentation out there to help us.
Will spend some time reading the documentation for GTFS to see what we can do, and will update with some ideas.

Actually, it would be way easier to use the Google Maps Directions API, since you can request distance/time estimates by mode of transit (car, bike, walk, public transit)
If that doesn't work, we can look into SDMTS's dataset, but Google Maps would give us not just transit but estimates for all relevant types of transit.


cargo-generate: 0.4.0 -> 0.5.0

Motivation for this change
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip" (no depend pkgs)
[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] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @turboMaCk

looks great to me. I can confirm this compiles and works on NixOS


The travis-ci link in CONTRIBUTING.md is broken.

Describe the bug (描述bug)
The travis-ci link in https://github.com/apache/incubator-brpc/blob/master/CONTRIBUTING.md is broken.

@zyearn fixed in #1481

Thanks for the quick fix!


applying generator-angular2-library

@bampakoa if no comments agains this PR I'm merging it to continue improving

@stevermeister I will not be able to comment on this before Friday.


Add action to be build objects for children under Source Orbit heading

Use command: makei b -t <object>

Awesome!

@edmundreinhardt How do we make this work in general for PE as users may use other build tools?


make mpie-cmti standalone

This PR makes the mpie_cmti image definition "standalone", i.e. the layering starts with jupyter/base-notebook:ubuntu-22.0.
Before, the image was drived from pyiron, which was itself derived from md, which was derived from base, which was the only standalone image definition in docker-stacks.
Especially for this image, this makes managing the env easier, which is important as the shared conda environment on our cluster is based on this.
Broken down, I made some changes to Dockerfile, concatenating neccessary parts from the Dockerfiles of the previous "inheritance-chain". Similarly, I concatenated the environment.yml from all those image definitions

@niklassiemer you think this is okay?
Ich started the container locally and ran
mamba list

and compared the output with the output of running the same command on cmti; they are the same.
@jan-janssen : Just added you to make you aware of this change.

Actually I think this image is smaller; I'm about to leave my desk, so I can check tomorrow or so.
The container started from the image defined here is basically empty.
Maybe (in another PR) we could add some notebooks for typical tests we do to check our environment. This way, one can do those checks easily from a local container instance. I'm thinking about the typical imports, plugins, ...; stuff like that.

Ok, so upon concatenating the environment.yml from the image definitions of pyiron, base and md there occurred some merge conflict. Upon resolving this I seem to have removed some packages accidentially... I hope the last commit fixed that.

Environment-wise everything seems to be okay now. But now the image(s) are too large.
The mpie_cmti image is not actrually too large. But it now, as it's "standalone", doesn't share any layers with the other images anymore, so the device volume (the runners diks space) is filled faster.
This already an issue in the past, right? I think we could circumvent this, if not all steps of the building/testing were performed on the same runner instance.
We could either

try to only build & test the images derived from the files on  the affected directory of a PR (e.g. in this case mpie_cmti only). I think this could be a bit ticky but doable. However it would increase the complexity of the CI/CD pipeline.
make seperate repos for each image and make "docker-stacks" an organization.

What do you think, @niklassiemer?

Now I tried to split the testing to two VMs like before in #347. Back then I failed (and the issue was not to pressing) since all the builds needed to be in one VM. Now, this is not intended this way any longer!

Let's merge this PR and do the splitting for the test and release in another PR. @niklassiemer , can you please approve the changes?

Hmm... the code in test and push is quite similar and I would expect the push to break due to the changes in the cmti/Dockerfile. Thus, let me take over the responsibility and I try to get the release action aligned with the test action before merging it in.

Yeah, you're right. You really want to do it? I can take care of it as well.

I really don't like to do all of this in this PR, it appears messy to me.
I opened another (draft) PR where I'm working on the CI improvements and applied the same changes done here to testing.yml.

The problem is a bit that these things are indeed coupled... i.e. you need the Dockerfile change to build standalone and you need the ci change that this fits into one VM... probably we need stacked PRs to have it clean :)


Network stack cleanup: connection

connection.c needs cleanup:
. revise the connection cache logic: use sys_slist instead of looping on an array. See what has been done in subsys/net/l2/ethernet/arp.c
declutter using #8723 rules

I'd like to propose simplification of connection callback functions, let me crossreference it from here: https://github.com/zephyrproject-rtos/zephyr/issues/8723#issuecomment-415040335

@pfalcon You are taking things out of the context and making false conclusions.

The point is that there's not enough context, guidelines, and requirements are provided, so one literally can only try to make conclusions from the sparse data provided, which can easily turn out to be false...

I think we all know what needs to be done here, no need to take by hand whoever is going to do that work.

I think we all know what needs to be done here, no need to take by hand whoever is going to do that work.

So let me clarify: I don't know, so that's why I didn't participate in it. Actually, I had issues with reviewing patches claiming to be cleanups/fixes based on this and related tickets - if it's known exactly what should be done and why, it's hard to assess whether a particular change is improvement, regression, or just sideways work.
Anyway, just a friendly feedback.


Python 3.6 support?

[ ] is it available on travis?
[ ] configure and run

Yes it is available 👍

merged and available, many many thx.


Add line endings gitattribute

Porting over https://github.com/microsoft/vscode/pull/106822 early so this stops being an annoyance in our repo. It'll be brought over in a vs code merge eventually but no reason not to get it in now.

Pull Request Test Coverage Report for Build 291887477

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 39.658%

Totals

Change from base Build 291824970:
0.0%

Covered Lines:
19947

Relevant Lines:
46034

💛  - Coveralls


feat(Topology/Profinite): extending cones in Profinite and LightProfinite

Let (Sᵢ)_{i : I} be a family of finite sets indexed by a cofiltered category I and let S be
its limit in Profinite. Let G be a functor from Profinite to a category C and suppose that
G preserves the limit described above. Suppose further that the projection maps S ⟶ Sᵢ are
epimorphic for all i. Then G.obj S is isomorphic to a limit indexed by
StructuredArrow S toProfinite (see Profinite.Extend.isLimitCone).
We also provide the dual result for a functor of the form G : Profiniteᵒᵖ ⥤ C.

Thanks!
bors r+


Does PowerBI support IE11?

The documentation mentions:
"Internet Explorer 11. Some advanced features, such as lineage view, aren't supported in Internet Explorer. See Data lineage (preview) for details."
Which would imply it supports IE11. However the next line is:
"Microsoft 365 apps and services, including Power BI, no longer support Internet Explorer 11 (IE 11). Refer to this Microsoft 365 blog post for more information."
Which states it no longer supports IE11.
Can this be updated with some clarification as to what is actually supported?

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

ID: 74662424-4c86-7d72-d7ad-4d8a669e83b8
Version Independent ID: dfa6ec25-3a0c-1ee9-63b9-851c3d2ce055
Content: Supported browsers for Power BI - Power BI
Content Source: powerbi-docs/fundamentals/power-bi-browsers.md
Service: powerbi
Sub-service: pbi-fundamentals
GitHub Login: @mihart
Microsoft Alias: mihart

Hi, @rikbrowning -- good question! Assigning to the author to investigate.

Good question, @rikbrowning ! Updated the article to read:
"Microsoft 365 apps and services, including Power BI, will no longer support Internet Explorer 11 (IE 11) starting in August 2021."
change will be live this afternoon June 14, Pacific time.
Thanks!


--x-cmake-args: Only imply editable if a variable is being set

Allows stuff like --trace etc. be used in manifest mode. Don't see a reason to add extra commands for all possible cmake options to vcpkg if it can be passed via --x-cmake-args.

There is one funny edge case where you might pass --x-cmake-args=-- --x-cmake-args=SOMETHING  and you can access SOMETHING via CMAKE_ARGV<N> but that feels so hacky I wouldn't really consider it. However it could be easily mitigated by checking if any of those args is exactly equal to --

Thanks :)


feat: move /sdk up

What :computer:

Move /sdk up

Why :hand:

DNS rules handle redirecting /sdk, files need to be at the root

@itsacoyote why does the top navigation look so distinctively different?


Making the website responsive and interactive

Is this a unique feature?

[X] I have checked "open" AND "closed" issues and this is not a duplicate

Is your feature request related to a problem/unavailable functionality? Please describe.
The UI/UX is a bit bland, I Saw an issue raised with adding flexbox and more colour gradients. We could add some features to make it more interactive.
Also there is not much info about the repo on main page.
Proposed Solution
-Added an assets folder with starImage.svg and forkImage.svg to make the UI more pleasent.
Added Hover effect on Repo cards so that they scale-up on hovering.
Added the prominent language used in the repo so users find it easier to select the repositries they are interested in.
Screenshots

Do you want to work on this issue?
Yes
If "yes" to above, please explain how you would technically implement this (issue will not be assigned if this is skipped)
Already made the changes and Added a PR.

This is the PR with just basic enhancement features.

Sure you can open a pr now:)
Thanks for raising the issue


yarn hardhat priceReset

This implementation doesn't make sense. Its primary purpose is to automatically set the price to 1.00 by removing uAD liquidity.
Bonus if I can input the price e.g. `yarn hardhat priceReset --to 1.001" and it'll do the math and reset to the specific price with the same logic.
@hpiyankov let me know if you have any inputs on the reset strategy, but I believe that removing single sided liquidity is the best approach.

@pavlovcik I feel like I am lacking context. What doesn't make sense?

@zgorizzo69 walked me through this price reset script but all he's missing is the formula to calculate how many LP tokens to remove in order to set the twap back to 1.00, according to him.
After he has this information he can make the command line interface a one argument reset command vs how it's implemented now (a lot more than one)

This is pretty straightforward. Since we want to reset at the best balance point (and not at the first one), essentially we need to have equal amounts of 3CRV and uAD. This means that the amount of uAD which needs to be removed from the pool is equal to:
amt_to_remove = uAD_curr - 3CRV_curr

I'll let @zgorizzo69 jump in here no point of me speculating what his questions may be


__MUL_INTEGER__ is too slow

On add-integer-multiplication
at f28c1d5

https://www.acmicpc.net/source/57586167


As an employee at a company that as a GitHub Enterprise instance, I want to be able to specify the URL to the API endpoint

I know it's not your priority, but it would be nice to have.

Some some support for this was just merged in, can someone check if it addresses this ticket?

It should work with http://hackage.haskell.org/package/github-0.14.0/candidate (and 0.13.2 already IIRC)


feat(select): add md-optgroup component

Adds the md-optgroup component, which can be used to group options inside of md-select, in a similar way to the native optgroup.
Fixes #3182.

Needs rebase


Switch from Docker to Quay

As discussed within the TSC, badge and links for FIWARE clones of the container image should be switched to quay.io
Proposed changes
Describe the big picture of your changes here to communicate to the
maintainers why we should accept this pull request.
If it fixes a bug or resolves a feature request, be sure to link to
that issue.
Types of changes
What types of changes does your code introduce to the project?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Maintenance (update of libraries or other dependences)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[ ] I have read the CONTRIBUTING doc
[ ] I have signed the CLA
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have run all the existing tests locally (not just those related to my feature) and there are no errors
[ ] After the last push to the PR branch, I have run the lint script locally and there are no changes to the code base
[ ] I have updated the RELEASE NOTES
[ ] I have added necessary documentation (if appropriate)
[ ] Any dependent changes have been merged and published in downstream modules

Further comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...

great, thank you @jason-fox
note: the build failures are totally unrelated to this PR.


[release] prepare for 2.0.0-RC4

2.0.0-SNAPSHOT -> 2.0.0-RC4

Codecov Report

Merging #6528 into master will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #6528      +/-   ##
==========================================
+ Coverage    39.5%   39.52%   +0.01%
==========================================
  Files        3090     3090
  Lines       92125    92125
  Branches    11891    11891
==========================================
+ Hits        36398    36414      +16
+ Misses      52723    52710      -13
+ Partials     3004     3001       -3

Impacted Files
Coverage Δ

...pinpoint/rpc/ChannelWriteFailListenableFuture.java
42.85% <0%> (-28.58%)
:arrow_down:

...m/navercorp/pinpoint/rpc/client/ConnectFuture.java
80% <0%> (-5%)
:arrow_down:

...rc/main/java/com/navercorp/pinpoint/test/Item.java
70% <0%> (-3.34%)
:arrow_down:

...m/navercorp/pinpoint/profiler/AgentInfoSender.java
85.14% <0%> (+0.99%)
:arrow_up:

...point/rpc/client/DefaultPinpointClientHandler.java
72.07% <0%> (+1.29%)
:arrow_up:

...ollector/cluster/zookeeper/ZookeeperJobWorker.java
80.93% <0%> (+1.39%)
:arrow_up:

...java/com/navercorp/pinpoint/rpc/DefaultFuture.java
74.16% <0%> (+1.66%)
:arrow_up:

...rp/pinpoint/rpc/server/PinpointServerAcceptor.java
88.74% <0%> (+4.63%)
:arrow_up:

...vercorp/pinpoint/rpc/packet/ServerClosePacket.java
68% <0%> (+16%)
: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 c5e6b7e...17da3e8. Read the comment docs.


build: explicitly link against dispatch and Foundation

On platforms where Foundation is not provided by the system and you wish
to build against a custom build of Foundation, you need a number of
flags.  This allows you to instead opt for passing -DFoundation_DIR=
and -Ddispatch_DIR= to CMake when configuring to pick up those paths.

@swift-ci please smoke test


Fixed: Changed month view to accept 'workdaysOnly' prop

A new pull request, i saw the travis build was failing. Should now pass. Please let me know if this can be merged into the main repo

Any updates on this?

When will this feature be ready?

Hi @danyel2010 , it is ready, just needs to be merged. The latests travis builds were failing because of missing payments.

any updates on this?

It looks like this PR has been approved by @Marcinthedev but not @jquense. Is it possible for you, @jquense, to review this with priority as my team wants to build a calendar with a monthly view without weekends and so it would be great if this PR can be merged in ASAP.

Would this work with localized calendars? E.g. the fr calendar has Monday as 0 and Sunday as 7.

This issue still exist till this day right? because i've been trying to just visualize the work days on the month view, but seems like there's no way to (other than getting into the source code)


Empty lines in source file mess up layout

When one or multiple empty lines are inserted at the top of the source document (e.g. because a number of labels from the sticker sheet have been used already and need to be skipped on the layout), the first line of each affected sticker is completely omitted and the date is printed instead. This results in a vertically compressed layout messing up all following stickers.
Solution:

Either print empty sample name when empty lines are detected
Or allow user to skip a certain number of stickers via command line input

Empty stickers could be generated by hiding content (e.g. the date) using one of the three phantom commands in LaTeX: https://tex.stackexchange.com/questions/4519/how-do-i-create-an-invisible-character


PixelTap: Daily Combo

04/07/2024

Posted on 2024-07-06 08:36:53

Posted on 2024-07-07 04:00:04

Posted on 2024-07-08 04:30:04


feat: playerController.HasPermission

Would you be so kind and add an example to the readme file as well?

much appreciated :)


llama: add control vector

Intentionally skipped while implementing alpaca-core/ac-local#3
We should be able to apply control vectors via llama_control_vector_apply
(in main demo this is done by reading files in llama_control_vector_load from common lib)

To generate control vector follow: https://github.com/ggerganov/llama.cpp/tree/master/examples/cvector-generator


Initial commit of generating random data

Hi @frewsxcv
As suggested here is a pull request for work initial work on generating random data.
Cheers
Rowan

If you want the commit to be tied to your GitHub username, you should follow these instructions:
https://help.github.com/articles/why-are-my-commits-linked-to-the-wrong-user/
Not mandatory, but might be nice to give you proper credit. Once you do that, you can just git commit --amend and git push -f to repush the commit.

Also, if you have time, there are style/linting issues which you can see here:
https://travis-ci.org/frewsxcv/python-geojson/jobs/69054890
If you don't get around to them, I can fix them myself later

Hi @frewsxcv
Thanks, I didn't realise you could actually see what was failing, now that you point it out its obvious! And I just installed a python linter so that helped :)
I've still got one issue to do with line length given all my params, im not sure about the best way to handle that.
Anyway let me know how you're getting on, no rush of course
Cheers
Rowan

Before:
 def generate_random(featureType, numberFeatures=1, numberVertices=3, boundingBox=[-180.0, -90.0, 180.0, 90.0]):

After:
 +def generate_random(featureType, numberFeatures=1, numberVertices=3,
                      boundingBox=[-180.0, -90.0, 180.0, 90.0]):

More info about indentation from the official style guide
http://legacy.python.org/dev/peps/pep-0008/#indentation

Hi @frewsxcv
Sorry for the delay in progress, just a quick update, I've been a bit stuck on the random polygon generation, my original idea of simply sorting the coords failed (given hindsight that was kind of obvious!). The geojson-random library from mapbox contains code to generate random polys although Im having a hard time understanding the maths, I think I need to sit down with a piece of paper to fully understand whats going on.
Cheers
Rowan

If you want, we can merge in a basic implementation of the random geometry algorithm you have now, then when you understand the math more, open a new PR improving it

Hi @frewsxcv
well I think I've pretty got things working now at a basic level, hooray!
For the polygon creation I took some code from here although this code wasn't designed for dealing with lat lons so its not working with the bounding box properly yet, I still need to finalise the maths around it but it's at least working for the timebeing.
This linting business took a while I get my head around, I've think I've now got a decent linter installed so fingers crossed things are looking better now!
Anyway let me know what you think, I hope it forms a useful part of the library.
Cheers
Rowan

I haven't forgotten about this! I'll be returning home from my month long traveling in a couple days. Hopefully I'll look at this later this week

@rowanwins Looks good! You're welcome to add yourself under the Credits in the README.md if you want before I merge this

Hi @frewsxcv , happy not to be credited
I do have one final question which it would be good to get your opinion on. I've been debating removing the multi-feature support. The current approach locks people into a geometrycollection and doest allow properties. It may be easier to let people write the loop themselves if they want multi-features. What do you think?
FYI I I just used this to generate 100,000 random points with properties in a feature collection using this approach, took ~5s to run. Very handy for test data!

I think it's okay to leave out the multi-feature support; writing a loop should be simple enough, though it's up to you whether you want to keep it or now

FYI I I just used this to generate 100,000 random points with properties in a feature collection using this approach, took ~5s to run.

:D

@frewsxcv have removed the multi-feature support and re-committed, this is it I reckon...

Looks great @rowanwins, thanks!

Just released 1.3.0 that includes this change
https://pypi.python.org/pypi/geojson
Thanks again!


Question about test the image

Hello, I have a little problem, I try to test the image on preduct,py.
When I input commend "python predict.py NYU_ResNet-UpProj.npy (my own image)", it's get a error like this:

How to fix it problem?
Thanks you!!

I have the same question.
Maybe the code should be fixed.

Hi guys,
You should probably load the checkpoint file instead of the npy.
Let me know if that helps!
Christian

Yes, I have also load the checkpoint file, it's still same problem.

What version of tensorflow are you using?

I use 1.3.0 version.

This is strange, it is working for us fine. What is the error that you get with the checkpoint?

Do I need change the name of NYU_FCRN.ckpt.xxx file??

Please try python predict.py NYU_FCRN.ckpt 1.JPG

It's can working!   Thanks you!! @TUSSON  and @chrirupp


Add get order by id endpoint

This adds a GET /v3/omis/order/<order-id> endpoint returning
the order details, e.g.
{
    'id': 'aa6998d6-ab5a-4ba8-8090-9b663813d0d2',
    'reference': 'TTU169/17',
    'company': {
        'name': 'My Corp',
        'id': 'd98da59a-84fd-4e08-8ffb-c702d4706548'
    },
    'contact': {
        'name': 'John Doe',
        'id': 'f56b4017-2132-4c98-9356-b3859f409490'
    },
    'primary_market': {
        'name': 'France',
        'id': '82756b9a-5d95-e211-a939-e4115bead28a'
    }
}

Codecov Report

Merging #311 into develop will increase coverage by 0.12%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop     #311      +/-   ##
===========================================
+ Coverage    92.79%   92.92%   +0.12%
===========================================
  Files           68       68
  Lines         2486     2487       +1
  Branches       220      220
===========================================
+ Hits          2307     2311       +4
+ Misses         148      146       -2
+ Partials        31       30       -1

Impacted Files
Coverage Δ

datahub/omis/order/urls.py
100% <100%> (ø)
:arrow_up:

datahub/company/admin.py
89.47% <0%> (ø)
:arrow_up:

datahub/omis/order/serializers.py
100% <0%> (+16.66%)
: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 0950a8b...6e747f6. Read the comment docs.

Docs: https://documenter.getpostman.com/view/2403194/omis/6fZxPC3#97cd0aa7-0aa3-cacb-46b0-2305cfcfabc1


locked version vs. missing lock file

My feature title
Would you merge a PR that adds an argument to fail the job when dependencies: locked but the lock file is missing?
Background/problem
I'd like to see that problem in form of a ❌
https://github.com/ramsey/composer-install/blob/c49029afecf85fda829d8c6c7f1b5f7b8bc3f7be/bin/composer_install.sh#L19-L22

Sure thing. As long as it's a configurable argument, I think that's be great.