Time decoders should not ignore superfluous characters

Given that the OpenAPI date-time decoding codegen defaults to UTCTime, it's way too easy to throw away the offset by accident. You end up parsing a local time as a UTC time, which gives the wrong result. Users of the library might be parsing timestamps from a source that initially doesn't send the offset, then adds it and specifies a regex. The codegen is run again, but the regex is ignored and the utcTime decoder will happily ignore the offset.
Proposed solutions:

Don't ignore superfluous characters
Require people to configure the appropriate datetime type instead of defaulting to UTCTime
Have the default date-time type be one that isn't a prefix of the others. For example ZonedTime instead of UTCTime

  ✗ prop_decodeZonedTimeAsUTCTime failed
    after 1 test and 3 shrinks.
    shrink path: 1:a3

        ┏━━ test/Spec.hs ━━━
    745 ┃ prop_decodeZonedTimeAsUTCTime :: HH.Property
    746 ┃ prop_decodeZonedTimeAsUTCTime =
    747 ┃   HH.property $ do
    748 ┃     zonedTime <- HH.forAll genZonedTime
        ┃     │ 2000-01-01 00:00:00 + 0000
    749 ┃     case FA.decode FC.utcTime $ FA.encode FC.zonedTime zonedTime of
    750 ┃       Right _ -> fail "Decoding should have failed"
    751 ┃       Left _ -> pure ()

    Decoding should have failed

Test patch
diff --git a/json-fleece-aeson/test/Spec.hs b/json-fleece-aeson/test/Spec.hs
index 412ae5e..6b2ae03 100644
--- a/json-fleece-aeson/test/Spec.hs
+++ b/json-fleece-aeson/test/Spec.hs
@@ -71,6 +71,7 @@ tests =
   , ("prop_encode_union", prop_encode_union)
   , ("prop_decode_taggedUnion", prop_decode_taggedUnion)
   , ("prop_encode_taggedUnion", prop_encode_taggedUnion)
+  , ("prop_decodeZonedTimeAsUTCTime", prop_decodeZonedTimeAsUTCTime)
   , ("prop_utcTimeAndZonedTime", prop_utcTimeAndZonedTime)
   ]

@@ -741,6 +742,14 @@ taggedUnionGen =
           <*> Gen.bool
     ]

+prop_decodeZonedTimeAsUTCTime :: HH.Property
+prop_decodeZonedTimeAsUTCTime =
+  HH.property $ do
+    zonedTime <- HH.forAll genZonedTime
+    case FA.decode FC.utcTime $ FA.encode FC.zonedTime zonedTime of
+      Right _ -> fail "Decoding should have failed"
+      Left _ -> pure ()
+
 prop_utcTimeAndZonedTime :: HH.Property
 prop_utcTimeAndZonedTime =
   HH.property $ do

We use attoparsec-iso8601 to parse the times and the UTCTime parser used by FC.utcTime does not throw away the offset. It works by parsing a local time and timezone first and then converting it to a UTCTime. Am I missing something?

Users of the library might be parsing timestamps from a source that initially doesn't send the offset, then adds it and specifies a regex.

What do you mean about the regex?

Also I don't understand why you'd expect that test to fail

Oh, I see, the offset is not being thrown away, it is getting integrated into the UTCTime. I was assuming that if the encoded data included an offset, and the decoded data didn't, then the data must have been lost. But that's not the case.

regex

I was referring to the pattern property.


Adapt for ws2801 lights

Hi, I'm interested in using your project with my ws2801 LED strand. Can you help me understand what modifications will be necessary to work with those lights? Thank you.

I second this!

Okay... So I just adapted it for the ws2801 led strip. It is a bit quick and dirty but here yah go!
Step 0: Follow instructions and install prerequisites from here:
sudo apt-get update
sudo apt-get install python-pip -y
sudo pip install adafruit-ws2801

Step 1 edit your config to the number of lights that you want. You do not need frequency and whatnot. The library will handle that for us.
Step 2 edit your led.py to this:
from __future__ import print_function
from __future__ import division

import platform
import numpy as np
import config

# ESP8266 uses WiFi communication
if config.DEVICE == 'esp8266':
    import socket
    _sock = socket.socket(socket.AF_INET, socket.SOCK_DGRAM)
# Raspberry Pi controls the LED strip directly
elif config.DEVICE == 'pi':
    import time
    import RPi.GPIO as GPIO

    # Import the WS2801 module.
    import Adafruit_WS2801
    import Adafruit_GPIO.SPI as SPI

    # Configure the count of pixels:
    PIXEL_COUNT = config.N_PIXELS

# Alternatively specify a hardware SPI connection on /dev/spidev0.0:
    SPI_PORT   = 0
    SPI_DEVICE = 0
    strip=Adafruit_WS2801.WS2801Pixels(PIXEL_COUNT, spi=SPI.SpiDev(SPI_PORT, SPI_DEVICE), gpio=GPIO)

    strip.clear()
    strip.show()
elif config.DEVICE == 'blinkstick':
    from blinkstick import blinkstick
    import signal
    import sys
    #Will turn all leds off when invoked.
    def signal_handler(signal, frame):
        all_off = [0]*(config.N_PIXELS*3)
        stick.set_led_data(0, all_off)
        sys.exit(0)

    stick = blinkstick.find_first()
    # Create a listener that turns the leds off when the program terminates
    signal.signal(signal.SIGTERM, signal_handler)
    signal.signal(signal.SIGINT, signal_handler)

_gamma = np.load(config.GAMMA_TABLE_PATH)
"""Gamma lookup table used for nonlinear brightness correction"""

_prev_pixels = np.tile(253, (3, config.N_PIXELS))
"""Pixel values that were most recently displayed on the LED strip"""

pixels = np.tile(255, (3, config.N_PIXELS))
"""Pixel values for the LED strip"""

_is_python_2 = int(platform.python_version_tuple()[0]) == 2

def _update_esp8266():
    """Sends UDP packets to ESP8266 to update LED strip values

    The ESP8266 will receive and decode the packets to determine what values
    to display on the LED strip. The communication protocol supports LED strips
    with a maximum of 256 LEDs.

    The packet encoding scheme is:
        |i|r|g|b|
    where
        i (0 to 255): Index of LED to change (zero-based)
        r (0 to 255): Red value of LED
        g (0 to 255): Green value of LED
        b (0 to 255): Blue value of LED
    """
    global pixels, _prev_pixels
    # Truncate values and cast to integer
    pixels = np.clip(pixels, 0, 255).astype(int)
    # Optionally apply gamma correc tio
    p = _gamma[pixels] if config.SOFTWARE_GAMMA_CORRECTION else np.copy(pixels)
    MAX_PIXELS_PER_PACKET = 126
    # Pixel indices
    idx = range(pixels.shape[1])
    idx = [i for i in idx if not np.array_equal(p[:, i], _prev_pixels[:, i])]
    n_packets = len(idx) // MAX_PIXELS_PER_PACKET + 1
    idx = np.array_split(idx, n_packets)
    for packet_indices in idx:
        m = '' if _is_python_2 else []
        for i in packet_indices:
            if _is_python_2:
                m += chr(i) + chr(p[0][i]) + chr(p[1][i]) + chr(p[2][i])
            else:
                m.append(i)  # Index of pixel to change
                m.append(p[0][i])  # Pixel red value
                m.append(p[1][i])  # Pixel green value
                m.append(p[2][i])  # Pixel blue value
        m = m if _is_python_2 else bytes(m)
        _sock.sendto(m, (config.UDP_IP, config.UDP_PORT))
    _prev_pixels = np.copy(p)

def _update_pi():
    """Writes new LED values to the Raspberry Pi's LED strip

    Raspberry Pi uses the rpi_ws281x to control the LED strip directly.
    This function updates the LED strip with new values.
    """
    global pixels, _prev_pixels
    # Truncate values and cast to integer
    pixels = np.clip(pixels, 0, 255).astype(int)
    # Optional gamma correction
    p = _gamma[pixels] if config.SOFTWARE_GAMMA_CORRECTION else np.copy(pixels)
    # Encode 24-bit LED values in 32 bit integers
    r = p[0][:].astype(int) #np.left_shift(p[0][:].astype(int), 8)
    g = p[1][:].astype(int) #np.left_shift(p[1][:].astype(int), 16)
    b = p[2][:].astype(int) #p[2][:].astype(int)
    # Update the pixels
    for i in range(config.N_PIXELS):
        # Ignore pixels if they haven't changed (saves bandwidth)
        if np.array_equal(p[:, i], _prev_pixels[:, i]):
            continue
        strip.set_pixel(i, Adafruit_WS2801.RGB_to_color( r[i], g[i], b[i] ))
    _prev_pixels = np.copy(p)
    strip.show()

def _update_blinkstick():
    """Writes new LED values to the Blinkstick.
        This function updates the LED strip with new values.
    """
    global pixels

    # Truncate values and cast to integer
    pixels = np.clip(pixels, 0, 255).astype(int)
    # Optional gamma correction
    p = _gamma[pixels] if config.SOFTWARE_GAMMA_CORRECTION else np.copy(pixels)
    # Read the rgb values
    r = p[0][:].astype(int)
    g = p[1][:].astype(int)
    b = p[2][:].astype(int)

    #create array in which we will store the led states
    newstrip = [None]*(config.N_PIXELS*3)

    for i in range(config.N_PIXELS):
        # blinkstick uses GRB format
        newstrip[i*3] = g[i]
        newstrip[i*3+1] = r[i]
        newstrip[i*3+2] = b[i]
    #send the data to the blinkstick
    stick.set_led_data(0, newstrip)

def update():
    """Updates the LED strip values"""
    if config.DEVICE == 'esp8266':
        _update_esp8266()
    elif config.DEVICE == 'pi':
        _update_pi()
    elif config.DEVICE == 'blinkstick':
        _update_blinkstick()
    else:
        raise ValueError('Invalid device selected')

# Execute this file to run a LED strand test
# If everything is working, you should see a red, green, and blue pixel scroll
# across the LED strip continously
if __name__ == '__main__':
    import time
    # Turn all pixels off
    pixels *= 0
    pixels[0, 0] = 255  # Set 1st pixel red
    pixels[1, 1] = 255  # Set 2nd pixel green
    pixels[2, 2] = 255  # Set 3rd pixel blue
    print('Starting LED strand test')
    while True:
        pixels = np.roll(pixels, 1, axis=1)
        update()
        time.sleep(.1)

if you get that buffer overflow follow this edit:
https://github.com/scottlawsonbc/audio-reactive-led-strip/commit/fa492bbffc13cc59820ffff1bf8767daad969620

@benwicks After some work I was able to set up a nice server to control the lights with this visualizer included. If you want to take a look at it check here

Looks great @kevkid! I don't have a microphone yet so am waiting a little longer to try this. I will keep a record of your work to try once I acquire a microphone. Thanks!

@benwicks Awesome! Good luck setting up your lights! If you have any questions about my project let me know!

I am also using a strip of WS2801 LEDs, they are connected to an ESP8266 running ESPlightNode, and I have patched led.py to send Artnet output (see issue 134). Works like a charm.

@kevkid I had time to set up your project today. My strand has 25 lights and the basics are working ok. But, when I turn on the visualizer, I get this output and see no change on the lights. Do you think there might be an issue getting ahold of the microphone? Thanks for all the help so far!
ALSA lib pcm_dmix.c:1018:(snd_pcm_dmix_open) unable to open slave
ALSA lib pcm.c:2217:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.rear
ALSA lib pcm.c:2217:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.center_lfe
ALSA lib pcm.c:2217:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.side
ALSA lib pcm.c:2217:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.hdmi
ALSA lib pcm.c:2217:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.hdmi
ALSA lib pcm.c:2217:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.modem
ALSA lib pcm.c:2217:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.modem
ALSA lib pcm.c:2217:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.phoneline
ALSA lib pcm.c:2217:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.phoneline
ALSA lib pcm_dmix.c:957:(snd_pcm_dmix_open) The dmix plugin supports only playback stream
ALSA lib pcm_dmix.c:1018:(snd_pcm_dmix_open) unable to open slave
Cannot connect to server socket err = No such file or directory
Cannot connect to server request channel
jack server is not running or cannot be started
FPS 48 / 60
FPS 20 / 60
FPS 14 / 60
FPS 12 / 60
FPS 12 / 60
FPS 12 / 60
FPS 12 / 60
FPS 13 / 60
...

@benwicks there may be several things going on. Try this check list to make sure that everything is the way it should.

[ ] Did you set the correct number of LEDs? There are 2 sections in my project one in light_control.py and one in led.py

[ ] Is your microphone working? From my understanding you do have to edit some settings in linux to get it to make the microphone as the default device.

[ ] Did you run everything as sudo?

Thanks. I did notice I didn't have the right number of LEDs specified in the config.py file. Now, I'm not sure how to proceed with making sure the microphone is working. Do you have a link to walk through how to configure my microphone as the default device in linux?

@benwicks I used the default configuration from this github. Follow what was written here for the Pi and it should work.


[Checkbox]inputRef cannot display the checked status properly

[ ] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯

When the component is rendered, it gets the status from the server through useEffect() and updates it, but the checked status is not displayed, even though the status is indeed true. It can be achieved by replacing the Checkbox with a traditional input tag.
const Auxiliary = () => {
  ……
  const humitureRef = React.useRef();

  return (
    <>
      <Grid container spacing={1}>
        <Grid item xs="auto">
          <CheckboxLabel labelName={"humiture"} flagType={"humitureFlag"} ref={humitureRef} />
        </Grid>
      </Grid>
    </>
  );
};

const CheckboxLabel = React.forwardRef(({ labelName, flagType }, ref) => {
  const checkedChangeHandler = (event) => {
    ……Send the new status to the server.
  };

  React.useEffect(() => {
    // Gets the status from the server, assuming true.
    ref.current.checked = true;
  }, []);

  return (
    <>
      {/* <FormControlLabel control={<input onChange={checkedChangeHandler} type={"checkbox"} ref={ref} />} label={labelName} /> */}
      <FormControlLabel control={<Checkbox onChange={checkedChangeHandler} inputRef={ref} />} label={labelName} />
    </>
  );
});

Expected Behavior 🤔

The Checkbox component is in a checked state after rendering.
Steps to Reproduce 🕹

Steps:
1.Execute the above code.
2.As a comparison, replace the Checkbox with the traditional input tag.
Context 🔦

Each Checkbox component turns on or off the real-time state of a server.
Your Environment 🌎

Tech
Version

Material-UI
v4.11.0

React
v16.13.1

Browser
Chrome 87.0.4280.88、Firefox 78.5.0esr

TypeScript

etc.

@clxering We don't support it, and will probably never, you need to control the checked prop.


Expensive Upgrades

While playing the hardest thing about the mod is the upgrades. The early tier are not bad price wise, however the first tier of the upgrade with 4 tiers feels out of place recipe wise compared to the others.
For the rest tier1/2 feels okays. Tier 2/3 feels a bit pricy. Tier 3/4 feels overpriced.
Echoshards are a lot more rare than you might think, and the need for 2 upgrades per line make it spike up fast.
On average you get about 6-12 shards per city, at least that what it seems like from my playthrough as i got 6 in one city and 9 in another. That is also not taking into account needing shriekers.
Since i cleaned out a end city of all ground level sculk using silk touch, shriekers number wise felt right for the teir 3/4 upgrade, leaving me about a stack. However thats excluding how dangerous collecting all it is. As often times there is 3+ shriekers in an area so mining one alerts others, so you are constantly avoiding the warden/s. Which is harder since after 4 go off it only takes one for a new warden to spawn.
While i like that idea of using sculk blocks for the upgrade materials perhaps making the shrieker and echoshards something that less used is a better idea.
Like an "shriek" upgrade which will cause emitters to shriek every so often than gather nearby dropped items. I know i suggested such a feature before i am just giving a example. The idea is a limited use upgrade that is very useful but not needed to be crafted a lot.
Then change upgrades that are crafted a lot to cheaper but not cheap recipes.

More upgrades is a tricky one in general to implement, since we want the mod to be UI-less and two upgrades already take a lot of space on the block without being too crammed.
Disregarding possible new upgrades, what do you think good recipes would be?

Well you want them to be expensive for what you get, but easily reproducible without needing to spend possible tons of hours.
perhaps add a new item the "blank upgrade", this item takes the various sculk items, echo shards, catalyst, and shrieker. and gives you 4-16 of it.
Then for speed upgrades you do due a basic stack of, redstone, redstone block, redstone block and diamonds, redstone block and diamond block.
For stack size, Lapis, lapis blocks, gold ingot, gold blocks, and diamonds and diamond blocks.
this way you still need skulk, but the cost matches the recieved, and to then upgrade the upgrades its basic materials that increasingly get more expensive.


sleep_time_based_on_baud leads to incomplete response..

Hi,
I have an AT command that returns long response (actually a json repr of a LwM2M object).
When I excecute it, I get an error be cause the decoded response is incomplete: the OK line as per expected response, is not catched by the exec.
More precisely, the missed lines are due to this loop:
https://github.com/veeso/ATtila/blob/0f8548d2900b4c1c68de98523006919f754cfe3d/attila/atcommunicator.py#L204-L215
Indeed, debugging shows that it did not wait "enough" for data to be ready i.e polling the condition self._device.in_waiting.
So I multiplied by 10 the sleeping time in the lines
https://github.com/veeso/ATtila/blob/0f8548d2900b4c1c68de98523006919f754cfe3d/attila/atcommunicator.py#L191
to become
sleep_time_based_on_baud = 1000 / self.baud_rate  # Milliseconds

To my understanding the inverse of baudrate (units in bits/sec) is to be multiplied by 1000 to convert to ms !
@veeso, what do you think ?
This said, I did not have an issue when the response size was small...
Cheers,
Roudy

To allow enough time to capture long responses, adjust;
atcommunbicator.py line 202, mini_sleep_time = .001 ( example =9 for a 9 seconds wait)


How to get subview within an item?

Hello,
I was wondering what is the correct way of obtaining a view within one of the RecyclerView items? The very first item I am adding to my adapter is a calendar view. I need to set some listeners to the calendar view but I can't figure out a way to obtain the view.
I have tried recyclerView.findViewById and findViewHolderByPosition but they always return null. Any suggestion would be greatly appreciated!

So you have an item with a calendar in it right? You can get a reference to the view only inside the view holder of the view model. Listeners are setup in the view holder too. I suggest you to check the example provided in the readme in which I show how to handle click events. You should be able to get the point from there ;) Remember that you shouldn't need to reference a view inside a particular view holder from outside the recyclerview, otherwise I suggest you to revisit your layout architecture.

Thank you for your quick response! I am new to Android development so I reviewed the readme and examples again. I think I'm starting to get a hang of it. :) I was able to get my calendar working by doing everything in the view holder. I am going to play around with creating items that have buttons/text fields inside them. I should be able to do this and eventually pull out all of the values of buttons or text fields within items if I have some sort of submit feature?
Thank you again!


Update node-webkit.rb 0.11.6 => 0.12.0

node-webkit is renamed NW.js
They have got new website on GitHub and their official site.

Thank you for the contribution. It will be merged in a different pull request due to some alterations. Please check it as a reference as to how the cask should’ve been updated. Your contribution is still included (and still credited to you), with the appropriate modifications, when applicable.


Crypto: EIP-55 support

Hexadecimal representations of chain addresses + digests should be checksum encoded ala EIP-55.  For reference purposes the python implementation of the checksum encode/decode functions is defined here.

The mechanics of the encode function are as follows:
If the length of the input bytes is > 75 then simply return lower case hexadecimal, otherwise:

Map input bytes into an array of bits (see _bytes_to_bits_cycle function);
Get 32 byte blake2b hash of output of above mapping function;
Convert 32 byte hash into an array of 'nibbles', i.e. 4 bit numbers (see _bytes_to_nibbles function);
Convert array of nibbles into array of hexadecimal characters mapped from here;

The checksummed should also expose a set of higher-level helper encoding functions:

encode_account_key;
encode_block_id;
encode_deploy_id;
encode_digest;
encode_signature;


assets_dir:  Ignored in config.yml

Hi here are the specs of the server I'm using
Ubuntu 2004
Nginx 1.21.1
PHP 8.0.9
picocms 2.1.4
I was originally using the nextcloud plugin version of pico cms but the additional restrictions and security wasn't meeting my needs so I setup picocms as a standalone on its own subdomain.
I've installed picocms in /var/www/pico
But the thing is, I want to continue using nextcloud as a frontend for editing the website.
I managed to point content_dir at the content directory in my nextcloud files which works nicely.
But when I point
assets_dir to the assets directory in my files, the images stored there do not display on my site.
here is the configuration

content_dir: /var/server/nextcloud/data/me/files/www/content
assets_dir: /var/server/nextcloud/data/me/files/www/assets

A bit of troubleshooting and I've determined that even though assets_dir is set in the config.yml,  The /var/www/pico/assets directory is actually still in play.  The images show up on my site just fine when I copy the images there.
This is not ideal as I'd like to fully modify the site without having to ssh to the server.
Thank you!

You must somehow publish your assets dir with your webserver, otherwise a browser won't be able to access the resources. There are various solutions for this:

Publish /var/server/nextcloud/data/me/files/www/assets below a virtual directory. If you're using Apache, refer to the Alias directive.
Reverse the setup: Don't let Pico access the files below /var/server/nextcloud, but Nextcloud access the files below /var/www/pico: Use Nextcloud's files_external app to include arbitrary local paths (i.e. /var/www/pico/content and /var/www/pico/assets) in your Nextcloud.
Symlink /var/www/pico/assets to /var/server/nextcloud/data/me/files/www/assets and unset the assets_dir config. You can even do the same with the content dir.

Personally, I prefer symlinks approach for simplicity. 😜
Just make sure the original files live in Nextcloud, because it won't play as nice with the symlinks when it's time to sync your files (if I remember correctly).  And definitely do the same for your content folder.
I just wanted to chime in and add an explanation of why this was happening to you.  Also, I was already thinking about it before @PhrozenByte responded, lol.
As far as I know, Pico doesn't actually use the assets_dir property for anything other than replacing the %assets_url% placeholder.  With the content_dir, you're actually pointing Pico at a different location to look for files.  These files are processed by Pico to make the pages, but they're not accessible through your web server at /content, so it really didn't matter that they were somewhere else.
Your assets on the other hand are handled directly by the web server and not Pico.  The %assets_url% placeholder just provides a convenient substitute for the url, it doesn't actually provide access to the files.  So, when you did this, it was either trying to point to /var/www/pico/var/server/nextcloud/data/me/files/www/assets, if you were using %assets_url%, which didn't exist, or just to an empty /var/www/pico/assets if you were typing the URL in your markdown files manually.

Ah!  That makes total sense!  Thanks guys!  Unfortunately it seems a little unintuitve. I expected assets_dir would be the same as content_dir in how it works.  Perhaps it should be renamed assets_url? or just not be there as it seems to just be a newb trap?  What would be a use case for it?
I thought to try the symlink approach before I went down this config file rabbit hole, but I had some weird unexpected result. I did "ln -s /var/server/nextcloud/data/me/files/www/assets /var/www/pico" and the symlink came up broken. Both the source directory and destination are owned by www-data...so beats me.  I'll get to the bottom of it.  Most  likely something stupid...
The other two ideas sound good too, I may give them a try.

Perhaps it should be renamed assets_url?

There's also a assets_url config :wink: Both assets_dir and assets_url are required config variables, they describe different things, even though Pico can often simply guess them from one another.


Help me Incorporate pos into scala-logging.

I wrote this library which features this macro. It allows me to do this:
import com.github.johnreedlol._

logger.debug("foo" + pos())

Which will print:
17:52:09.794 [main] DEBUG  Log - foo    at my.package.Main.main(Main.scala:8)
Since it looks like a stack trace, my text editor will take me to that line in the source code. The only problem is that I have to add + pos() to every log line, which is annoying.
How do I modify scala-logging so that I can just do:
logger.debug("foo")
and get:
17:52:09.794 [main] DEBUG  Log - foo    at my.package.Main.main(Main.scala:8)?

Actually, I think that maybe the best approach would be to provide pos() as an implicit, so that it can be passed to scala-logger similar to the way it is done with the sourcecode library. Do you think that this would be the right approach?


Publish extension for Firefox

See PR #80
Need to do more testing: settings (sync), sounds, notifications
Register with FF
Update build scripts
Publish extension
Update README to include FF support
Get valid extension ID:

See manifest.json
https://developer.mozilla.org/en-US/Add-ons/WebExtensions/manifest.json/applications
https://developer.mozilla.org/en-US/Add-ons/Install_Manifests#id
https://developer.mozilla.org/en-US/Add-ons/WebExtensions/WebExtensions_and_the_Add-on_ID

This is great! I'm actively running it and trying to get it working right now. As I find issues do you want me to file them against the repo, or just create a PR and work against that? I know for sure that the notifications have trouble right now as the buttons option isn't supported on chrome.notifications.create() or update().

I always welcome pull requests. If the existing code isn't too messy to understand and the changes aren't too burdensome, feel free to create a PR.
If something is too complex or you think it would take you forever to get up-to-speed just to change something, you can file an issue and I'll take a look at it.
No button support is a bummer, but Chrome on macOS uses the macOS-style notifications now, so the buttons are hidden behind a dropdown menu, anyway.

Sounds good! Thanks @schmich!

I had to remove the applications property from manifest.json since I was getting warnings in Chrome (see #88) and I'm planning on publishing an update soon.
To support both Chrome and Firefox, I think we'll need to dynamically generate manifest.json for Firefox. I filed #106 to track this.

Closing this non-specific issue in favor of the more specific Firefox issues created. See the Firefox milestone for more details.


DialogTrigger type="modal" warning

🐛 Bug Report
Warnings
🤔 Expected Behavior
No warnings
😯 Current Behavior
Only happens with type="modal":

Warning: Legacy context API has been detected within a strict-mode tree.
The old API will be supported in all 16.x releases, but applications using it should migrate to the new version.
Please update the following components: Transition

Warning: findDOMNode is deprecated in StrictMode. findDOMNode was passed an instance of Transition which is inside StrictMode. Instead, add a ref directly to the element you want to reference.

💻 Code Sample
Codesandbox
🌍 Your Environment

Software
Version(s)

react-spectrum
3.4.0

Browser
Chrome

Operating System
MacOS

🙏🏽

I believe this will be solved by https://github.com/adobe/react-spectrum/pull/1108


[Website] Python API References redirect to 404 (1.6.0 Release)

Description
Some hyperlinks of tutorials in python API/mxnet.gluon (1.6.0 Release) leads to 404, Including Gluon Guide, gluon.Parameter, gluon.Trainer, gluon.model_zoo.vision.
Error Message
404 Page
To Reproduce
Try to access these contents through master branch.
Steps to reproduce
(Paste the commands you ran that produced the error.)

What have you tried to solve it?

Environment
We recommend using our script for collecting the diagnositc information. Run the following command and paste the outputs below:
curl --retry 10 -s https://raw.githubusercontent.com/dmlc/gluon-nlp/master/tools/diagnose.py | python

# paste outputs here

cc @ys2843

These broken links were only fixed in master branch, need to backport to v1.6.

Fixed the broken links on gluon index page v1.6 website, please verify.
link: https://mxnet.apache.org/versions/1.6/api/python/docs/api/gluon/index.html


Add support for ReplSet and associated options

Hey again @wzrdtales !
Sorry if the title is unclear. Right now in the mongo driver we have the following:
db = config.db || new MongoClient(new Server(host, port));

And I'm hoping to change it to something which has support for ReplSet and associated options:
    db = config.db || new MongoClient(new ReplSet(servers, options));

I think my general approach would be to try to avoid changing the API and add additional optional options to the config. Perhaps something like:
var mongoClient;
if (config.replSet) {
  var hosts = config.replSet.split(',');
  var servers = hosts.map(function(host) {
    return new Server(host, port, config.replSetOptions);
  });
  db = config.db || new MongoClient(new ReplSet(servers, config.replSetOptions));
} else {
  db = config.db || new MongoClient(new Server(host, port));
}

Before I went ahead and figured the rest out I figured I'd double check with you that this seems reasonable and you'd be amenable to a PR for this. Also please let me know if you have any implementation requests. And, as always, thanks for the work you do :)

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

For now I'll proceed with ^ and make a PR when it's ready

@wzrdtales pinging as requested here too :)

@MartinLoeper is there a way to configure that while still just using a database uri and not individually configuring each piece? (ie host, user, password, database, etc.)

@wolfejw86 Are you referring to the mongodb protocol or the mongodb+srv protocol? I guess host and database must be set any case with the present code from my fork. That is not an optimal solution since the database can be discovered from the srv record when using the latter protocol.
Could you provide an example of a configuration which you would like to set (i.e. the minimal set of keys you want to set)?

Really I’d like it just to be consistent with the rest of dB-migrates api. So
‘’’
{
“dev”: { “ENV”: “MYMONGO_SRV_URI” }
}
‘’’
I would be happy to contribute.

Ok, I had to think about this, but finally I got your point. If I understand correctly, you state that:

db-migrate is not database vendor specific
the mongodb plugin for db-migrate does not conform to the de-facto standard of passing the connection string to the underlying db mongo driver
we should use a single attribute which contains the full connection string

I guess this is actually the way to go! I do not see any reason why it has to be so complicated as it currently is.
However, we must either provide an adapter to be backward compatible or we introduce a breaking change.
Personally, I would go for the first option.
However, I do not have the time to do this right now. If you want to do this @wolfejw86, go for it! It makes very much sense IMO.

@MartinLoeper thanks for the response!  Sorry if I wasn't as clear as I should have been, I was typing that last message on my phone 🤦 .
Regarding implementation, would a "one or the other" type implementation make sense instead of either of your suggested options?  I'm looking at this line in particular:
https://github.com/db-migrate/mongodb/blob/f820c1359dc848326afdb0b93e2442746505d474/index.js#L495
If we could just do a preliminary check above that line that says if (typeof config === 'string') then treat it like a mongodb connection string right?  That way we could parse it all out.  There would still be additional work for the srv portion I believe but I think logic gates at the start of that configuration check could make it easier.
Let me know if you think this makes sense, I'm a first time user of this package and the functionality is great however it appears it has a decent amount of backstory as to why things are the way they are and I don't want to "step" on that.

Ok, I looked at the implementation again and noticed the following:
a) You can fully restructure the exports.connect method. The only thing necessary is to return an instance of MongodbDriver.
b) The database must be extracted inside the connect method and passed to the MongodbDriver since I think it is used by some of the other methods, e.g. to switch back to default database. You just have to extract it from your connection string somehow. I guess mongo has released an API since then which makes it easier to parse connection strings, so this should be no issue.
c) backstory? Yes, I think there is. I think the connection string parsing was not as easy as today. When I forked this package and built the "useSrvRecord" option into it, my goal was to do as little refactoring as possible. However, I guess today the exports.connect method should be completely rewritten to adhere to mongo best practices and being better usable.
d) I do not think that the db-migrate base module imposes any restrictions on the config that can be passed. At least nothing I can find inside the docs [1]. I guess each database driver might specify its own configuration keys. That should make it possible to add a property "connectionString". But don't pin me down to that. This is really just a guess.
[1] https://db-migrate.readthedocs.io/en/latest/Getting Started/configuration/


Together MPT HalfTensor type error

Affected models: togethercomputer/mpt-30b-instruct, togethercomputer/mpt-30b
Error Expected tensor for argument #1 'indices' to have one of the following scalar types: Long, Int; but got torch.cuda.HalfTensor instead
https://api.together.xyz/jobs/job/83af28ef89f447290af38b4cac2eaf4252fe3a292d8f8adf2eff2fc73255cf

Together seems to not be returning this error any more.


852 analytical frequency curve should plot on normal probability axis

This does not fully address the issue of confidence limits yet. It applies them in one location, and adds the model logic to compute them.

I ran into an uncaught exception

The exception happens when I attempt to view a graphical frequency function.

The latest update updates the tracker to use the probability rather than the Z score, so you can check exact probabilities anywhere on the lines.
I was unable to reproduce your error. Please send me your dataset if that's still an issue.


Skip downloaded files

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm requesting a feature unrelated to a specific site
[X] I've looked through the README
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Provide a description that is worded well enough to be understood
scenario
I put all the urls in a txt file and use the following line to download all the files.
C:\Users\name\Videos>yt-dlp.exe -a C:\Users\name\Videos\二级翻译\list.txt
But something went wrong, the download failed with some of the files downloaded and some not yet.
If I repeat the above line, I believe it will download all the files, including those already exist, which is not what I want.
request
May I request a feature that can check if the file exist. If yes, skip; if not, continue to download.
Provide verbose output that clearly demonstrates the problem

[ ] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[ ] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
No response

yt-dlp will try and skip files that already exist, so if the settings didnt change and you are not downloading live streams and are in the same directory it will extract the video to get the title, but not redownload it if the file already exists

If you download large amounts of videos, it is also recommended to use --download-archive. That way, all downloaded videos will be recorded by ID, so you don't need to extract them again, saving on time and API requests.


langmap and langnoremap produces odd recording behaviour

When recording using qa the langmap isn't applied to the replayed recording into when langnoremap is set.
For example if .vimrc is
set langnoremap
set langmap=sl,ls

and the current file is
testing

with the cursor over the first t in testing and in normal mode. Entering qas~q@a yields the file
tE~ting

now in insert mode and the cursor over the '~'. The expected outcome is
tEsTing

in normal mode with the cursor over the 'i'. When langnoremap isn't set the expected outcome occurs. However in both cases the register a contains s~.

The same issue occurs with the new nolangremap as well.


renamechisel3 - "chisel" -> "chisel3"

This renames the package to chisel3 in order to co-exist with the Chisel compatibility object on case-insensitive file systems.
External packages should use the Chisel object in order to preserve backward compatibility with the existing Chisel2 code.

Looks good, but has a merge conflict...


Switch to id; name is obsolete.

This is good.

@XhmikosR duplicate ids might be a pain but that's a problem for another day

Yeah, noticed that already.


Upgrade commons-fileupload:commons-fileupload to version 1.3.2 or higher

Fixes #665.

#666 ....
Any dark forces had anything to do with this pull request ? ....

I sure hope not!!! :-D


Added a zero check for the parameter w.

In the function static int bytepad:549 there is a division by the function parameter w.
If this parameter is set to zero, a division by zero will occur.
Therefore, a zero check for the parameter w was added at the beginning of the function.
Below is an example of the sanitizer output if the function bytepad is passed a parameter w of 0.

This would be eligible for CLA: trivial. Please amend the git commit message body to include CLA: trivial on a separate line.
Or submit a regular CLA. https://www.openssl.org/policies/cla.html

I was thinking it might be trivial, too.

@Klavishnik While you're updating the commit to add CLA: trivial on a line on its own, would you mind updating the commit message?
Added a zero check for the parameter w is fine in the context of this function, but for someone just looking at commit messages it's a bit vague. Something like Avoid diviide-by-zero in kmac_prov.c's bytepad()?

My suggestion would be instead of fixing this in the bytepad() function, ensure that the parameter cannot be 0 at callers.
This is the block size of the hash which cannot really be 0 unless some internal error happens. Currently the code checks for EVP_MD_get_block_size() returning negative numbers, that should be just augmented to check for negative or zero return.
Doing what @tom-cosgrove-arm in addition to that would be also good, but not required.

How are you forcing it to get here?
The digest is set to either KECCAK-KMAC-128 or 256 internally..
EVP_MD_get_block_size() returns -1 if there is no digest.
So the only way I could see this happening is if the blocksize of the digest returned zero.
The point just after loading of the digest during kmac_fetch_new() is where it would then be incorrect.

This is the block size of the hash which cannot really be 0 unless some internal error happens

Which is why I think adding to the assert (and moving that assert to a better place) is most appropriate

In theory the underlying MD implementation for the KMAC can come from a different provider which means it can return anything including 0. It would be wrong to use ossl_assert() instead of proper error in that case.

What should we do about this issue?

What should we do about this issue?

Could you please do what I suggested and change the line https://github.com/openssl/openssl/blob/bec6633b66c3506699b87d2cc226fbeb7ea3d8d8/providers/implementations/macs/kmac_prov.c#L292 to have <= 0 there? You can keep your original change as is there as well.

@t8m this is not a sufficient change.. There are the 2 places that get the block_size.
Change lines 292 and 252 to <= 0
And move the assert line up.

This pull request is ready to merge

Squashed and merged to master, 3.1, and 3.0 branches. Thank you for your contribution.


ZyphronG/Zucchini ADPCM encoder is gone 😭😭😭

Please re-clone https://github.com/ZyphronG/Zucchini! It has been down for days!

ZyphronG's projects are not my own.  This issue is not relevant to this tool.


I have an idea, to implement a jsonresume using VueJs

Prefare
The Json resume is not so suitable for portfolio showcase.
I have been working on a proof of concept already.

Customize a json-resume theme? a) I doubt whether that's procrastination. b) Almost no one use json-resume anymore. c) Comparing to Markdown, the json-Resume lack of flexibility.


module 'kaolin' has no attribute 'models'

v0.10.0 has no attribute 'models'?

Hi @duzhenjiang113 ,
Kaolin doesn't have models since v0.9.0, see: https://github.com/NVIDIAGameWorks/kaolin/releases/tag/v0.9.0


@? macro does not go through getproperty

Really nice package!
I've started playing a bit with it, and it feels nice to use. The only thing I've noticed is that, while function calls are lifted, getproperty is not, but it could be useful.
MWE:
julia> using Maybe

julia> m = match(r"a", "aaa")
RegexMatch("a")

julia> @? m.offset
Some(1)

julia> m = match(r"b", "aaa")

julia> @? m.offset
ERROR: type Nothing has no field offset
Stacktrace:
 [1] getproperty(::Nothing, ::Symbol) at ./Base.jl:33
 [2] top-level scope at /home/pietro/.julia/packages/Maybe/TuhZO/src/lift.jl:148

Hi, thanks for the feedback. Good to know that you find it nice to use!
I was a bit hesitant to lift the property accesses because it might generate too much code.  So I thought I'd wait for examples in which this is useful to come up in practice. I think RegexMatch is a good example.

Side notes: Your example itself can actually be worked around by putting match in @? as well
julia> @? let m = match(r"a", "bbb")
           m.offset
       end

julia> @? let m = match(r"a", "aaa")
           m.offset
       end
Some(1)

But I think Union{RegexMatch,Nothing} could be wrapped in other things like a NamedTuple.


Random crash in Request.cURLRepresentation()

I see random crash happening in method Request.cURLRepresentation() relatively often (not every day, but few times in a week I catch this crash in Xcode during development).
The crash happens on line with command
if request.allHTTPHeaderFields != nil {
Weird thing is that if I try to print contents of request.allHTTPHeaderFields in debugger console it prints correctly but accessing it for some reason causes crash in some random cases.

I first thought it happens only during debugging in Simulator but I now see exact same crash in Crashlytics from some tester device.
Part of stack trace from crash up:
Thread : Crashed: com.apple.main-thread
0  CoreFoundation                 0x0000000181bf44e0 CFAllocatorDeallocate + 48
1  CFNetwork                      0x000000018164d01c HTTPMessage::ensureParserFinished() + 108
2  CFNetwork                      0x000000018164cf74 HTTPMessage::copyAllHeaderFields(__CFArray const**) + 28
3  CFNetwork                      0x000000018175f6e0 -[NSURLRequest(NSHTTPURLRequest) allHTTPHeaderFields] + 36
4  Alamofire                      0x0000000100834540 Alamofire.Request.cURLRepresentation (Alamofire.Request)() -> Swift.String (Alamofire.swift:1357)
5  Alamofire                      0x0000000100835a3c Alamofire.Request.debugDescription.getter : Swift.String (Alamofire.swift:1392)

Code which leads to crash - debug print request after it was sent:
let request: Alamofire.Request = ...
request.response() { (request, response, JSONObj, responseError) in
    // ...
}

#if DEBUG
debugPrintln(request) // <-- crash happens on this line
#endif

Xcode 6.3.2, deployment target iOS 8.0, Alamofire version 1.2.0, commit 6d1238d

Thank you for that very detailed write up @ibru. It is very much appreciated.
Unfortunately you have stumbled upon a nasty issue in CFNetwork. There is a known threading issue with ensureParserFinished that has already been filed with Apple. There is a massive thread full of fantastic information from the AFN community which can be found here.
I would like to take the same approach as AFN on this one. If anyone finds any possible workaround for Alamofire for this issue, we'd be happy to adopt the fix. Until then, it may be wise to avoid using debugPrintln(request) on consumer facing builds. If you see this crash in your debug builds, at least now you'll know the root cause of where the issue truly lies.
Thanks again.


Implement request caching

This PR fixes #8 .
Changes proposed in this pull request:

Uses memory-cache module to cache GET requests

@onlywicked thanks for your contribution!
Would you like a mention in the contributors file?

Sure.


[8.x] Fix the cached routes issue

This PR fixes issue #37639 related to route priorities after caching.
Also, it provides a new approach to cache the routes into other cache drivers like Redis.
I can do it in another PR if this one succeeds.
This is the test regarding the mentioned issue:
    public function testRedirectRoutes()
    {
        $this->routes(__DIR__.'/Fixtures/redirect_routes.php');

        $this->post('/foo/1')->assertRedirect('/foo/1/bar');
        $this->get('/foo/1/bar')->assertSee('Redirect response');
        $this->get('/foo/1')->assertSee('GET response'); // This line asserts the fix
    }

This reverts https://github.com/laravel/framework/pull/31188 which was intended to improve request latency when resolving routes. It replaced route caching's base64_encode(serialize($routes)) with var_export($routes->compile(), true), as suggested by core Symfony dev nicolas-grekas.
Going back to serialize() now leaves dead code in the framework:

src/Illuminate/Foundation/Console/stubs/routes.stub
src/Illuminate/Routing/Router.php method Router::setCompiledRoutes()
src/Illuminate/Routing/CompiledRouteCollection.php and test coverage:

tests/Integration/Routing/CompiledRouteCollectionTest.php
src/Illuminate/Routing/RouteCollection.php method RouteCollection::toCompiledRouteCollection()

What are the implications of CompiledRouteCollection::match() being dropped? It will obviously affect benchmarks, reducing the # of requests/second apps can handle. And this doesn't call base64_encode()/base64_decode() like Laravel used before the 7.x changes.
The performace improvements were made because Laravel's route caching served no purpose once an app reached a few hundred routes. In fact it could be slower than uncached routes because serialize()/unserialize() on a >5MB cache file string wasn't leveraging PHP's opcache.
Also, the 2 concrete examples given in the GitHub issue thread can both be fixed by userland defining their routes in an unambigious order. Route::patch() before redirect(). Route::get() before any(). Specific definitions must come before general pattern matching.

I don't think there's a bug here. If you define routes in such a way that there are multiple correct evaluations, then expect the cached implementation to possibly behave differently.

@GrahamCampbell It is based on an open issue with the Bug label and it was opened 30 days ago until now.
If you define routes in such a way that there are multiple correct evaluations, then expect the cached implementation to exactly behave in such a way.
@deadem

It leaves dead code in the project: It is a good point. They could be removed in the next steps and it makes the framework a little smaller in size. I just have to wait to see this PR will be merged or not.
Missing base64_encode()/base64_decode() and the file size you have talked about in the next paragraph: Encoding the routes using these two methods make the process much slower and the file size higher.
PHP opcache: If the file size is >5MB, you have to consider it a very larger file as compiled one (I can calculate/estimate the size and write it here) and there will be an issue regarding opcache memory limit and tracking the problem from a compiled file. However, this is a point I agree with you and I need to figure out how to fix this problem.

@derekmd I find out how to use the benefit of PHP opcache. I will update the PR with a new commit.

This will definitely break any application with an existing caching route when upgrading to a minor release.

@crynobone Yes, and it may need a release note to run php artisan route:cache for applications with an existing caching route or it may NOT be included in minor release. Let's see...

Yes, and it may need a release note to run php artisan route:cache

Developer doesn't read release note when upgrading weekly minor release.

Yes, and it may need a release note to run php artisan route:cache

Developer doesn't read release note when upgrading weekly minor release.

@crynobone
I'm creating a new commit for this PR for PHP opcache.
I will include the route cache filename change to avoid this kind of breaking change.
cache/routes-v7.php => cache/routes-v8.php

Thank you.

such a way that there are multiple correct evaluations, then expect the cached implementation to exactly behave in such a way.

I don't agree. Multiple correct evaluations means that you could see either happen. This is non-deterministic choice. That's the definition of saying there are multiple correct evaluations. You don't get to choose which one happens. Either can (but not necessarily either actually will).

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.


TLS support

Add TLS support to Eth2Signer. This will likely follow the approach taken by EthSigner and so we can copy the implementation from there.

Documented here:
https://docs.eth2signer.pegasys.tech/en/latest/HowTo/Configure-TLS/


Remove using multiple directories for test-resources

Purpose
Fixes #1046
Current there are 2 paths we have been keeping resource in CentOS instances of the stacks.

/opt/wso2/
/opt/testgrid/

This will stick all the resource to /opt/testgrid

Security checks

Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines? yes
Ran FindSecurityBugs plugin and verified report? yes
Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets? yes

Please Note: Please merge only after;

Using packer-build AMIs in both Dev and Prd environments.
Changes all the integration-test pipline jobs' properties:
REMOTE_WORKSPACE_DIR_UNIX=/opt/testgrid/workspace
SQL_DRIVERS_LOCATION_UNIX=/opt/testgrid/sql-drivers/


SLL with --last_layer 'lln' is not 1-Lipschitz

Dear Alexandre,
I am trying to train the SLL models on the cifar100 dataset with the option --last_layer 'lln'.
The learned model, which achieves similar performance in the paper,  is not 1-Lipschitz.  By searching the empirical lower Lipschitz bound
L=norm(model(x+dx)-model(x))/(norm(dx)+1e-6)
with PGD-like algorithm, I obtain L=1.8. I guess the possible reason might be that although Q in the output layer has unit row vectors, its singular value could be still larger than 1.
self.Q = F.normalize(self.weight, p=2, dim=1)
Best regards
Ray
--
Ruigang Wang
<EMAIL_ADDRESS>Postdoc, Australian Centre for Robotics (ACFR),
University of Sydney, Australia.

Dear Ruigang,
Thank you for your interest in our work!
Yes, you are right, the model trained on CIFAR100 is not 1-Lipschitz. We're sorry if we didn't make that clear!
For CIFAR100 models, all layers are 1-Lipschitz SLL layers except the last one, which is a normalization layer proposed by [1] (see Section 4).
Therefore, for CIFAR100 models, the certification is adapted given the Lipschitz of the last layer, as you can see in the function evaluate.py > eval_certified_lln.
Please do not hesitate to contact us if you have any further questions,
Best regards,
Alexandre
[1] Sahil Singla, Surbhi Singla, and Soheil Feizi. Improved deterministic l2 robustness on CIFAR-10 and CIFAR-100
[2] Lipschitz-Margin Training: Scalable Certification of Perturbation Invariance for Deep Neural Networks


Create a class schedule with two notebooks per week, on average

A basic schedule is now in the outline.


[WIP] pin customer_testing

Description
Replace this paragraph with a description of what this PR is doing. If you're modifying existing behavior, describe the existing behavior, how this PR is changing it, and what motivated the change. If you're changing visual properties, consider including before/after screenshots (and runnable code snippets to reproduce them).
Related Issues
Replace this paragraph with a list of issues related to this PR from our issue database. Indicate, which of these issues are resolved or fixed by this PR. There should be at least one issue listed here.
Tests
I added the following tests:
Replace this with a list of the tests that you added as part of this PR. A change in behaviour with no test covering it
will likely get reverted accidentally sooner or later. PRs must include tests for all changed/updated/fixed behaviors. See Test Coverage.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] I signed the CLA.
[ ] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[ ] I read the Tree Hygiene wiki page, which explains my responsibilities.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] All existing and new tests are passing.
[ ] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[ ] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[ ] No, no existing tests failed, so this is not a breaking change.
[ ] Yes, this is a breaking change. If not, delete the remainder of this section.

[ ] I wrote a design doc: https://flutter.dev/go/template Replace this with a link to your design doc's short link
[ ] I got input from the developer relations team, specifically from: Replace with the names of who gave advice
[ ] I wrote a migration guide: Replace with a link to your migration guide

(PR triage) just checking in since this PR has been sitting here for a while: Are there still plans for this or should we close it for now?

(PR triage) just checking in since this PR has been sitting here for a while: Are there still plans for this or should we close it for now?

Closing this, it's no longer necessary.


Handled the case when atom doesn't exist in the path

Whether or not an atom exists at a given point is a separate concern from navigating into the value of an atom. I strongly disagree with mixing these concerns together.
If you wanted to do something like create a new atom for a non-existent key in a map, I would do it like this:
(def NIL->ATOM (if-path nil? (view atom) STAY))
(setval [:a NIL->ATOM ATOM] 100 {})


Font size of blog posts

Jack - Hope you are doing well .
I was wondering if there is an easy way to increase the font size of blog posts. It could be my older eyes :-) but I think its too small for good readability
I am okay if the setting increases the font size across whole website . Even better .
Cheers

@ashutoshmjain  Long time no see :)
When I design this theme I didn't take blog font size into consideration, but it's not easy for you to read, so I found an open source solution: Markdown CSS. But it needs some edit before we use it. For changing font size I have the following code for reference:
.article p {
  font-size: 1.3em;
  margin-bottom: 1.3rem;
}

.article li {
  font-size: 1.3em;
}

.article code {
  font-size: 1.3em;
}

.article h1,
.article h2,
.article h3,
.article h4 {
  margin: 1.414rem 0 .5rem;
  font-weight: inherit;
  line-height: 1.42;
}

.article h1 {
  margin-top: 0;
  font-size: 4.398rem;
}

.article h2 {
  font-size: 3.11rem;
}

.article h3 {
  font-size: 2.199rem;
}

.article h4 {
  font-size: 1.555rem;
}

.article h5 {
  font-size: 1.233rem;
}

.article h6 {
  font-size: .968rem;
}

.article small {
  font-size: .778em;
}

You can create a file *static/css/blog.css, and link it at _layouts/post.html line 4. If the font size is not satisfied, you can change it.

Wow  .. It worked like a charm  .. Many thanks .


Map Viewer

Add a map viewer that lets you over or click a button to see the map in a panel.

To be added in v. 1.4.0


boost::json::serialize returns NULL string

PLEASE DON'T FORGET TO "STAR" THIS REPOSITORY :)
Hello Team,
I am having issues when trying to serialize my JSON into a string before sending it.
The problem is that the serialize command returns a NULL string.
From what I can see the string is not empty it only say unable to read memory.

Version of Boost
1.81.0
Steps necessary to reproduce the problem
All elements in the map are std::string.
	boost::json::object JSONObject;
	JSONObject["type"] = this->PV_DTOMessageType;
	for (auto iter = keyValueMap.begin(); iter != keyValueMap.end(); ++iter) {
		JSONObject[iter->first] = iter->second;
	}
	return boost::json::serialize(JSONObject);

What is a "NULL string"? I'm pretty sure we can't create an invalid std::string. Can you create a minimal reproducible example, so that I can investigate your issue?

Wait, do I understand you correctly that you build Boost.JSON for one platform, but build your program that links to Boost.JSON for another platform? If you do — don't. That's never guaranteed to work. And even if it seems to work, most likely it silently has errors.

@marincovic have you figured out why you were having this error?

Hello,
Sorry I did not post it anything sooner. I could not resolve the issue at all, so I had to switch JSON tools. From what I remember the issue was connected to the DLL that was being used (I tried building it with Clang-cl as well but there was still an issue).
Unfortunately it was some time ago so I am not 100% sure.
Once again sorry for the late response.
BR//
Marin

It seems like the issue was caused by using an incompatible binary. I'm closing this for now.


Added C++ code for issue #1554

What have you Changed(must)
Added C++ code for 744. Find Smallest Letter Greater Than Target
Issue no.(must) - #1554
Self Check(Tick After Making pull Request)

[x] This issue was assigned to me if not it will be marked as spam.
[x] My file is in the proper folder
[x] I am following clean code and Documentation.
[x] I have added the title and what program will do first in the file
[x] I have named the file as (Problem #). (Problem Title(in Camel Case)).(extension)
ex. 2. AddToNumbers.py

Please merge this!


Remove proxy and clean up config

Proposed Changes

Load config from env with defaults
Remove proxy and direclty call the backend api

@ohcnetwork/code-reviewers
Merge Checklist

[ ] Add specs that demonstrate bug / test a new feature.
[ ] Update product documentation.
[ ] Ensure that UI text is kept in I18n files.
[ ] Prep screenshot or demo video for changelog entry, and attach it to issue.
[ ] Request for Peer Reviews
[ ] Completion of QA

Should we switch to camelCase?

@bodhish Raised a PR for the same with base branch as this PR's branch: https://github.com/ohcnetwork/care_fe/pull/8450

[ ]  Add i18n for kasp string if it differs for each language

@khavinshankar The KASP string is a tricky variable, this was added for the covid times, it more like a state run insurance provider, since we have a better way to capture insurance now, we would phase out the KASP variable. So lets not support internationalisation for that for now.


Proper BOM handing for encoding for remote resources

The contract basically is, depending on what encoding is picked to save a file:

UTF-8: we do not automatically add a UTF-8 BOM unless the file has a BOM on disk
UTF-8 with BOM: we add the UTF-8 BOM because the user explicitly asked for it
UTF-16 LE: we always add a UTF-16 BOM because this makes it easier for us to not falsely think the file is binary due to 0-bytes within the file
UTF-16 BE: we always add a UTF-16 BOM because this makes it easier for us to not falsely think the file is binary due to 0-bytes within the file

This logic could go into the encoding helper we already have if possible.

I pushed a change for the easy cases (https://github.com/Microsoft/vscode/commit/f50c390500a58f027cb5fc99b713d2f797b7adda). There is still the remaining (more tricky) one to actually preserve the BOM when it is already there for UTF-8 only.


Can't build in Windows 10

I followed instructions from https://github.com/Microsoft/vscode/wiki/How-to-Contribute#build-and-run-from-source , but the build is unsuccessful.
I'm using node v5.1.0, Visual Studio 2015 Community (I installed C++ build tools), MSBuild v14.0.24720.0
npm-debug.log file gives following error:
21173 verbose stack Error<EMAIL_ADDRESS>install: `node-gyp rebuild`
21173 verbose stack Exit status 1
21173 verbose stack     at EventEmitter.<anonymous> (C:\Program Files\nodejs\node_modules\npm\lib\utils\lifecycle.js:232:16)
21173 verbose stack     at emitTwo (events.js:87:13)
21173 verbose stack     at EventEmitter.emit (events.js:172:7)
21173 verbose stack     at ChildProcess.<anonymous> (C:\Program Files\nodejs\node_modules\npm\lib\utils\spawn.js:24:14)
21173 verbose stack     at emitTwo (events.js:87:13)
21173 verbose stack     at ChildProcess.emit (events.js:172:7)
21173 verbose stack     at maybeClose (internal/child_process.js:818:16)
21173 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:211:5)
21174 verbose pkgid<EMAIL_ADDRESS>21175 verbose cwd E:\Github\vscode
21176 error Windows_NT 10.0.10586
21177 error argv "C:\\Program Files\\nodejs\\node.exe" "C:\\Program Files\\nodejs\\node_modules\\npm\\bin\\npm-cli.js" "install"
21178 error node v5.1.0
21179 error npm  v3.3.12
21180 error code ELIFECYCLE
21181 error<EMAIL_ADDRESS>install: `node-gyp rebuild`
21181 error Exit status 1
21182 error Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
21182 error Make sure you have the latest version of node.js and npm installed.
21182 error If you do, this is most likely a problem with the oniguruma package,
21182 error not with npm itself.
21182 error Tell the author that this fails on your system:
21182 error     node-gyp rebuild
21182 error You can get their info via:
21182 error     npm owner ls oniguruma
21182 error There is likely additional logging output above.
21183 verbose exit [ 1, true ]

Git shell is giving following error:
C:\Program Files (x86)\MSBuild\Microsoft.Cpp\v4.0\V120\Microsoft.Cpp.Platform.targets(64,5):
error MSB8020: The build tools for v140 (Platform Toolset = 'v140') cannot be found.
To build using the v140 build tools, please install v140 build tools.  Alternatively, you may upgrade to
 the current Visual Studio tools by selecting the Project menu or right-click the solution, and then
selecting "Upgrade Solution...".
[E:\Github\vscode\node_modules\vscode-textmate\node_modules\oniguruma\build\oniguruma.vcxproj]

However, as I mentioned earlier, my MSBuild version is 14.0.24720.0
I tried following workarounds from * User/Repository#Num: nodejs/node-gyp#679 :

set environment variable GYP_MSVS_VERSION = 2015
tried to run ```scripts\npm install --msvs_version=2015
update Visual Studio
But still without success.

Steps to reproduce the bug:

install Visual Studio Community 2015
install node.js v5.1.0
clone repository using command ```git clone https://github.com/microsoft/vscode
go to repository cd vscode
install mocha gulp ```npm install -g mocha gulp
run install script scripts\npm install

@Umriyaev not sure how to fix the specific node-gyp issue you run into.
But here is a hackish workaround. It involves "manually installing"<EMAIL_ADDRESS>in the node_modules such that npm does not need to install it:

download VSCode for windows as a zip
extract the zip and take resources\app\node_modules\vscode-textmate and copy it to where you have checked out vscode in $\node_modules\vscode-textmate
the vscode-textmate version in that zip is 1.0.6 and we now use 1.0.8 on master, so npm install will still try to install it, therefore:
download<EMAIL_ADDRESS>extract the tgz on top of $\node_modules\vscode-textmate

Run scripts\npm install again. This time, since npm finds<EMAIL_ADDRESS>it won't try to install it, therefore it won't utilise node-gyp and will continue with the other node modules (that don't require node-gyp).

Can you try installing Visual Studio 2013 instead of 2015?

I did manage this on a Windows 10 pc, with only VS2015. If I recall correctly I installed alot of packages with npm install <name> --no-bin-links manually because of some very weird errors. Will try to check it out again tonight

@alexandrudima Thank you so much! Your workaround worked for me.
I guess it should be added to How to build and run from source .
Of course after finding more generic solution.

It's definitely just a workaround, let's try to find why you couldn't compile. I'll set up a VM tomorrow with exactly your setup and see what I get.

Just set up a brand new Windows 10 VM. Installed VS Community 2015, node 5.1.0 and Python 2.7. Got some errors at first, but realised I did not install the C++ tooling. Went to Control Panel, hit Modify in the Visual Studio item and added the Visual C++ tooling.

I successfully ran .\scripts\npm.bat install afterwards.
Not sure what is wrong with your system, but it's at best a Visual Studio and/or npm issue.

You seem to be in a system that has had Visual Studio 2013 installed before, since you mention the following path:
C:\Program Files (x86)\MSBuild\Microsoft.Cpp\v4.0\V120\Microsoft.Cpp.Platform.targets

In my brand new VM, I only have the V140 directory, not the V120:
C:\Program Files (x86)\MSBuild\Microsoft.Cpp\v4.0\V140

I see.
Thank you for your help.

In case you actually find a solution to your problem, can I ask you to post it here? Maybe it'll help others. :+1:

Sure! Of course I will


Add description to readme

A few readme words (from the top of the __init__). Also added a couple of links, but feel free to suggest dropping them if you don't feel they're appropriate (especially as there is no built documentation for netcdftime).

Shall we also add the same wording to the short description of the repo?


LPUART won't work below 1MHz

Not sure whether this is a problem or there is some particular way to configure it. What I'm doing is:
let mut lpserial = dp.LPUART1.usart(tx_pin, rx_pin, serial::Config::default(), &mut rcc).unwrap();
It works perfectly with MSI from Range6..Range4, but not below that. The calculation of the baudrate divisor should work OK, it adjusts for LPUART. But it won't work. Setting the baudrate explicitly to lower values (e.g. 1200 bps) doesn't change it. It just stops working below Range4.
Is there something else I have to do in my code to make it work?

It works, the problem was caused by the non working delay (as mentioned in another PR).


Updated Create SQL DB Section

Updated "Create a single database" section Point # 9
Resolves #https://github.com/MicrosoftDocs/azure-docs/issues/108269

Thanks @koneni for taking the initiative to create this PR! By opening a pull request, you're helping the author add your approved change directly to the documentation. I'll incorporate your commit into a future update already in progress. GitHub will list you as an article contributor. Closing this PR for now.


Version 1.1.0 is not published

Hello @tibdex,
Just to inform you that master is on 1.1.0 but it's not published on npm.
https://github.com/tibdex/github-rebase/blob/52aacd9836e54c7fadf34437f185e6449d512184/package.json#L50

Thanks, I've just published it.


Is it possible to download it straight as an Mp3

Using ffmpeg to convert it from mp4 to mp3 is quite a slow process, is it possible to rip it stright to mp3?

Sorry, Youtube does not provide mp3 format of videos, but there are some itags dedicated to adaptive formats that provide only audio files in AAC, Opus and ... format (not mp3). You can use them to download less and convert faster


Ability to destroy List object

I'm working on a page where there are multiple lists, but only one is visible at a time. When the lists are toggled, I'd like to destroy the previous List instance and create a new one for the newly visible list.
Basically, this is what I'd like to do:
// Restore previous list items to default state, and remove List functionality
oldList.removeClass('list')
listObj.destroy();

// Shows the new list and adds the List functionality to it
newList.addClass('list');
listObj = new List('files', listOptions);

I'm not seeing this method anywhere, is this something that would need added or am I looking over something in the code?
To give some context, this is what the page looks like (http://d.pr/i/iH0y).

Sorry for reviving this old post, but I struggled with the same issue and found a workaround, probably helpful for next (desperate) readers.
Similar as the TS posted, I'm creating a new search instances and would like to clean up the old one. Otherwise it interferes with the new search instance and overwrites results of the new one. The trick is to remove the listeners on the search field of the old instance. Easiest to do so is
var searchField = document.getElementById("searchinput"),
        searchFieldClone = searchField.cloneNode(true);
searchField.parentNode.replaceChild(searchFieldClone, searchField);

When cloning elements, the listeners are not discarded (details see here.


refactor store/store.go

refactor store/store.go: use defer statement to update Stats and report R/W Sucess/Failure, so that the logic of Store's CURD operation and Stats update logic can be separated.

@mqliang Can you please follow the contributing guidelines on the commit message: https://github.com/coreos/etcd/blob/master/CONTRIBUTING.md#format-of-the-commit-message

Overall I think this does make the code slightly better. But, I will let @xiang90 chime in.

It must be etcdErr.Error type, if we define err as Error type, then err will always NOT nil. Since internealGet(and some other statements) return a etcdErr.Error type variable(even if it's nil, but when wen assign a nil etcdErr.Error variable to err, err will  be NOT nil. For example:
var tempErr etcdErr.Error
var err error

err = tempError  //tempError==nil, but err!=nil

@xiang90 FYI https://golang.org/doc/faq#nil_error

@mqliang I understand that. However, I think we need to change either internalCreate or internalGet (with necessary type assertion)+Node.Remove. And I think changing internalGet + Node.Remove to return error is cleaner.
I can take this pr even if you keep the error type. I do not really care this that much.

@xiang90 Reworked: return and remove the else statement. I didn't replace all etcdErr.Error type to error, since some type assertion are needed, and would mess thing up.

LGTM.
@xiang90 Any objection to merge this?

@gyuho No. Just want to double check there is no mistakes.

Ok merging this. Thanks @mqliang


CASMINST-4117 updated .version and handoff-bss-metadata.go to handle sun interface …

Summary and Scope
Updated .version and handoff-bss-metadata.go to handle interface naming during upgrade of CSM 1.0 to 1.2 depending on whether we have 4 MACs or 2 MACs

Fixes # CAMINST-4117
Requires #
Relates to #

Issue Type

Bugfix Pull Request

Fix

[x] I have included documentation in my PR (or it is not required)
[ ] I tested this on internal system (x) (if yes, please include results or a description of the test)

Idempotency

Risks and Mitigations

I think the confusion is that this code is not for upgrading from 1.0 to 1.2.  This is for taking the configuration on m002 as a template and changing the MACs during the handoff part of the fresh install.  So m002 will already be in a 1.2 configuration.

I think the confusion is that this code is not for upgrading from 1.0 to 1.2. This is for taking the configuration on m002 as a template and changing the MACs during the handoff part of the fresh install. So m002 will already be in a 1.2 configuration.

Correct, I think should be correct this time.

Verification and test of code change was successful and corrected the MAC address interface leak issue. However in making this PR we realized there is an earlier bug for which I am opening a new jira and PR for.


Improve the profile Screen add firstname lastname phone number and email

Hey,
I'm interested in contributing to this issue #12.
Could you please assign the task and provide the Profile ui for reference?

Hey @LegendSumeet,
Yes I can do it, so please assign this task #12.

@LegendSumeet i can perform this task please assign issue #12 to me

Hey @LegendSumeet ! Can you pls assign this issue to me?


First version of the plugin

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

1.0.0
1.0.0
GitHub release

Your semantic-release bot :package::rocket:


[GLUTEN-1640] Support judging whether the execution plan has a fallback

What changes were proposed in this pull request?
Support judging whether the execution plan has a fallback
How was this patch tested?
UT

@PHILO-HE  pls review again


Allow stateful function in dataset.map

Removes the "test type" section in Dataset.map which would modify the state of the stateful function. Now, the return type of the map function is inferred after processing the first example.
Fixes #1940
@lhoestq Not very happy with the usage of nonlocal. Would like to hear your opinion on this.

@lhoestq Added a test. If you can come up with a better stateful callable, I'm all ears 😄.

Sorry I said earlier that it was good to have it inside the loop, my mistake !

@lhoestq Okay, did some refactoring and now the "cache" part comes before the for loop. Thanks for the guidance.
Think this is ready for the final review.


Cant make Spyder default opening program for .py files

Since installing Winpython-64bit-<IP_ADDRESS>Qt5, I have been unable to open .py files by double clicking on them, using spyder as the default program.
What steps will reproduce the problem?**
When I right click a .py file and select properties, then where it says  "Opens with:" I click the "Change" button and the "Open with" window appears. I click "Browse" and browse for the Spyder.exe in the winpython folder and click it and click open. It simply returns to the "Open with" window but doesnt show spyder in the list of programs.
I also thought to make an association with spyder and .py files from the Programs and Features list from the control panel, but spyder doesnt appear in the list of installed programs.
Opening Spyder by double clicking Spyder.exe in the winpython folder works fine and I can open .py files from within spyder.
I have never had this particular issue with previous winpython downloads

Spyder Version: spyder 3.1.0
Python Version: Python 3.5
Operating system: windows 7

@stonebig, could you help here?

I never tried this way of opening. I may have a look

apparently, it works.

I am having a similar issue. When I tried to select the default program for Windows 7 to use, I selected spyder.exe here:
C:\Users[myname]\AppData\Local\Continuum\Anaconda3\Scripts\Spyder.exe
However after doing that, I just get an open completely blank console window, and nothing else happens.
Suspiciously, if I look at the properties of my Spyder shortcut from the start menu, it doesn't refer to any executable. It's a long Python target (all on one line):
C:\Users\sdewey\AppData\Local\Continuum\Anaconda3\pythonw.exe C:\Users\sdewey\AppData\Local\Continuum\Anaconda3\cwp.py C:\Users\sdewey\AppData\Local\Continuum\Anaconda3 "C:/Users/sdewey/AppData/Local/Continuum/Anaconda3/pythonw.exe" "C:/Users/sdewey/AppData/L
This is more suspicious because as you can see it's actually cut off (probably from the maximum target name length in Windows 7). Spyder seems to work fine from this shortcut, however.
Any suggestions to get it working using Spyder as the default program for .py files?

I'm having the same issue as @eliot1785 above - is there a solution for this yet?

I have the same issue. This is the type of issue that may force me to switch to a different IDE. Hopefully this add this functionality soon.

Found a solution using a bat file. I think basically all the links in the "shortcut" part need to be opened at the same time.
Hopefully they build something more intuitive.
Solution:
https://stackoverflow.com/questions/24643474/set-spyder-as-default-python/42282994#42282994
Screenshots attached.

Do like the screenshot and then, right click the file and navigate to the bat file to open it with.

Thank you! That worked perfectly.

@alochen , now I can't get it to work. I found that when I right click on the file and select the .bat file (pictured above), Spyder opens, but the file I selected doesn't actually open. When you tried this, did the actual file you right clicked on open? If so, can you take some screenshots?

Hi, yes when I right click on the file it opens Spyder directly. The file type it lists is 'PY file' if that makes a difference (see screenshot). I've also taken a screenshot of my spyder batch file just in case. I noticed in your screenshot, your notepad file (which I'm assuming is your batch file) doesn't have 'start' at the beginning or the '%1' at the end, as in the stackoverflow answer you linked - could that be it?

@alochen ,
You are correct. After I made the updates you pointed it, it worked!
I'm glad we solved this. I am trying to give back to the community (I'm new to programming), so I was surprised anyone even responded. You must have some notifications or something.
I liked how you used notepad++, which made it easy to see the "start" and "%1" tokens. I missed this with notepad.
Thanks!

@Joel-Balmer
How do we close issues? I have never had the pleasure of doing this before. Maybe @Joel-Balmer  has to commission it.

Although we can't fix this directly, we plan to implement this at some point for Anaconda.

Good call. I literally almost decided to not use spyder because of this
issue, so I think it's an urgent request. It may be costing you users.
On Tue, Jan 14, 2020, 2:24 PM Carlos Cordoba<EMAIL_ADDRESS>wrote:

Although we can't fix this directly, we plan to implement this at some
point for Anaconda.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/spyder-ide/spyder/issues/3835?email_source=notifications&email_token=AELD2UR5CEN76XGQMCOQ4ALQ5YNPZA5CNFSM4CZRUYVKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEI572ZQ#issuecomment-574356838,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AELD2URAFK4PDLRVXSG7NHTQ5YNPZANCNFSM4CZRUYVA
.


Inconsistent name of Authentication Service

In step 8. name = sts, but in step 11. the name is "AzureSSO", which is not consistent.

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

ID: adf2d1dc-ab4d-5e4a-d18d-a9f4fba5aedb
Version Independent ID: 2a58c27f-19e8-1a9a-f14a-45db73b91cc2
Content: Tutorial: Azure Active Directory Single sign-on (SSO) integration with Salesforce
Content Source: articles/active-directory/saas-apps/salesforce-tutorial.md
Service: active-directory
Sub-service: saas-app-tutorial
GitHub Login: @jeevansd
Microsoft Alias: jeedes

@Kofl, Thank for pointing it out. We will investigate with the content author and update the same.

@chetansriv and @v-nagta @v-viinde Can you please fix this issue?

@Kofl Article is updated as per the feedback, please review
#please-close


fixed typos in the divisions handler

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/702/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/703/
Test PASSed.


fromSQL support

As a user
I want to be able to instantiate a DataFrame from a simple SQL query
So that I can use a convenient interface for relational data manipulation and analysis.
ie:
$pdo = new PDO("dns");
$df = DataFrame::fromSQL($pdo, "SELECT * FROM tableName WHERE x = y;");

Feature added


switch icon component to svg

Since filament disables blade icons components by default
it throws an exception component not found and this will fix it

Interesting, not sure why I didn't come across this when testing!


helm charts: override the ./kube/config file

Override the ./kube/config for in the flux pod when using helm to deploy flux

@brandon-bethke-neudesic how did you test this? If you install the chart with the defaults is Flux still working?

We tested by deploying flux.  We verrified, that the flux container had the correct kube/config file mounted as specified in the values.yaml. We haven't encountered any issues thus far and flux is deploying the FHR and the artifacts in our charts without issue. Are you encountering issues?
The updated flux chart is mounting the ./kube/config file to the same location as specified in Dockerfile.flux COPY ./kubeconfig /root/.kube/config
And the default value in values.yaml is the same value as found in file: flux/docker/kubeconfig
The Reason for the change
We needed to override the default namespace because we need to transition/promote releases between namespaces. Therefore putting the namespace in the FHR is not an option or portable and there was no other way, we could find, to force flux to install artifacts to a specific namespace.
Or to put it more generally. We have flux installed in a namespace and we want all kube artifacts to be installed in that namespace without having to specify the namespace in the FHR file.

Yes. Our plan is to run a flux instance in each of our namespaces which represent the environments: test, stage, prod, etc
We have a single 'release repository' which contains only helm charts and FHR files. Each flux instance is scanning that single repo, but looking at different branches. To promote a release, we simply merge a branch.

Besides FHRs and charts do you also have secrets or ingress definitions? I image you can't merge an ingress definition from one namespace to another without changing the public domain name.

We have those types of artifacts. I am not familiar with the details, but we have an instance of traefik that is configured to route requests to another instance of traefik in the desired namespace based on a cookie.


fix readme about join

I fixed README about the syntax of join.
I also added historical background about the join clause.

Excellent, thank you @kiibo382 !


Fix Model.forge for ES6 classes

Closes #924.

Neat. Beware that ES extension is still not completely the same as using extend. See #756

Thanks for the merge! Yeah, good to note. I use a getter in a base model, similar to https://github.com/tgriesser/bookshelf/issues/756#issuecomment-110113718, to do some of this automagically (a la ActiveRecord), e.g.
get tableName () {
    var tableName = this._tableName;
    if (tableName == null) {
        tableName = pluralize.plural(underscored(this.constructor.name));
        this._tableName = tableName;
    }
    return tableName;
}

Hey, that's clever.


Format of XSum-TRAINING-DEV-TEST-SPLIT-90-5-5.json

What's the format of the .json file used in the pre-processing of the XSum data? I need it for my own custom dataset.
Thank you in advance

you can download from https://github.com/EdinburghNLP/XSum/tree/master/XSum-Dataset, and this file is to split the dataset by mapping the index id.


I have created an asp.net web application with ADFS on premisis authentication this was done by .net framework . we have microservices needs to be consumed in .net framework application we want to provide adfs authentication for .net core web services when trying to create api with asp .net core it is not showing on premisis option

I have created an asp .net framework web application with ADFS on-premises authentication. we have microservices needs to be consumed in .net framework application we want to provide adfs authentication for .net core web services when trying to create api with asp .net core it is not showing on premisis option

Hi @mukeshmuvva , we have a doc on using WS-Fed with ASP.NET Core. It supports both ADFS (on-prem) and Azure AD (cloud). Please follow those steps and let us know if you have any problem.


[new: feature] register shenyu instance by zookeeper

about #2416

Make sure that:

[x] You have read the contribution guidelines.
[x] You submit test cases (unit or integration tests) that back your changes.
[x] Your local test passed mvn clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #2437 (aff8d52) into master (6f7c10b) will decrease coverage by 0.43%.
The diff coverage is 0.00%.

:exclamation: Current head aff8d52 differs from pull request most recent head b1332c1. Consider uploading reports for the commit b1332c1 to get more accurate results

@@             Coverage Diff              @@
##             master    #2437      +/-   ##
============================================
- Coverage     59.35%   58.91%   -0.44%
+ Complexity     4046     4017      -29
============================================
  Files           638      638
  Lines         18074    18096      +22
  Branches       1680     1678       -2
============================================
- Hits          10728    10662      -66
- Misses         6508     6593      +85
- Partials        838      841       +3

Impacted Files
Coverage Δ

.../org/apache/shenyu/common/config/ShenyuConfig.java
0.00% <0.00%> (ø)

...che/shenyu/sync/data/http/HttpSyncDataService.java
61.38% <0.00%> (-19.81%)
:arrow_down:

...henyu/plugin/grpc/resolver/ShenyuNameResolver.java
51.57% <0.00%> (-12.64%)
:arrow_down:

...a/org/apache/shenyu/common/utils/VersionUtils.java
67.85% <0.00%> (-10.72%)
:arrow_down:

.../apache/shenyu/admin/model/dto/BatchCommonDTO.java
50.00% <0.00%> (-7.15%)
:arrow_down:

...apache/shenyu/admin/model/dto/AuthPathWarpDTO.java
57.14% <0.00%> (-7.15%)
:arrow_down:

...org/apache/shenyu/admin/model/dto/AuthPathDTO.java
58.82% <0.00%> (-5.89%)
:arrow_down:

...che/shenyu/admin/model/vo/DashboardUserEditVO.java
62.50% <0.00%> (-4.17%)
:arrow_down:

...ache/shenyu/admin/model/dto/DataPermissionDTO.java
59.25% <0.00%> (-3.71%)
:arrow_down:

...yu/plugin/grpc/resolver/ShenyuServiceInstance.java
96.29% <0.00%> (-3.71%)
:arrow_down:

... and 25 more

Continue to review full report at Codecov.

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


Remove checkboxes from issue/pr templates

Match https://github.com/freechipsproject/chisel3/pull/783

:: problems summary ::
:::: ERRORS
	Server access Error: Connection reset url=https://repo.scala-sbt.org/scalasbt/maven-releases/org/sonatype/oss/oss-parent/6/oss-parent-6.jar

Thats probably just an intermittent failure that would go away if rerun...

retest this please


Divide and Conquer

Divide and Conquer algorithm is a problem-solving strategy that involves breaking down a complex problem into smaller, more manageable parts, solving each part individually, and then combining the solutions to solve the original problem.
Here, is Merge Sort and Quick Sort using Divide and Conquer approach.

#21 create a folder name Divide and Conquer
create directory structure like this.
Divide and Conquer
------->Merge Sort
---------------->Program.c
---------------->Readme.md
------->Quick Sort
---------------->Program.c
---------------->Readme.md

Ok sure
On Tue, Oct 1, 2024, 22:16 Pankaj Kumar Bind @.***>
wrote:

#21 https://github.com/AlgoGenesis/C/issues/21 create a folder name
Divide and Conquer
create directory structure like this.
Divide and Conquer
------->Merge Sort
---------------->Program.c
---------------->Readme.md
------->Quick Sort
---------------->Program.c
---------------->Readme.md
—
Reply to this email directly, view it on GitHub
https://github.com/AlgoGenesis/C/pull/35#issuecomment-2386496918, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/BJI6RSQ2CQXE4GLYNBF5HNDZZLGU7AVCNFSM6AAAAABPF7DMOOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOBWGQ4TMOJRHA
.
You are receiving this because you authored the thread.Message ID:
@.***>


Tooltip: Focus only when a user triggers an event (Fixes #15547)

Please make sure these boxes are checked before submitting your PR, thank you!
Related issue #15547

[x] Make sure you follow Element's contributing guide (中文 | English | Español | Français).
[x] Make sure you are merging your commits to dev branch.
[x] Add some descriptions and refer relative issues for you PR.

Sorry, This is an experimental technology, we have no plan to support this


UI - Ticket ID detection strips space before link in comment

Hi,
Thanks for the advice last week on getting the rules working for ticket ID detection. We've got it working nicely now.
We've noticed that the space before the detected link is stripped out of the silence - only a little thing but looks odd:

becomes

Thanks, should be fixed via #4713


Add Tweet button

How about to add a Tweet button.

How to embed a Tweet button.
https://developer.twitter.com/en/docs/twitter-for-websites/tweet-button/overview

I am going to use a Font Awesome icon.
https://fontawesome.com

What is the benefit of using Bulma for icons? According to Bulma's description.

Because the icons can take a few seconds to load, and because you want control over the space the icons will take, you can use the icon class as a reliable square container that will prevent the page to "jump" on page load.


macOS: CPU thread pool uses only 1 CPU core by default

Describe the bug
The macOS build of iree has IREE_ENABLE_CPUINFO=OFF.
consequentially, topology_cpuinfo.c::iree_task_topology_initialize_from_physical_cores falls back to iree_task_topology_initialize_fallback, which creates a topology with group count 1.
Consequently, jax models such as those used in dalle-playground report just 125% CPU usage on a 10-core M1 Max.
Additional history explained here by @stellaraccident.
To Reproduce
I'm afraid I don't know how to produce a minimal repro of this.
but we have a smoking gun of the cmake configuration + reading the source code produced when IREE_TASK_CPUINFO_DISABLED is defined.
An example jax integration via dalle-playground can be tried like so:
git clone https://github.com/saharmor/dalle-playground.git
cd dalle-playground/backend
python3 -m venv dalle
# warning: first ensure you have on your PATH: cargo + a beta release of rustc
pip install -r requirements.txt
python app.py 8080

The backend server will generate an image as part of its startup sequence (i.e. it performs a "warmup" task).
Further tasks can be issued to the backend, by starting up the frontend:
cd ../frontend
npm i
# deploys frontend on port 3000
npm start

and submitting the form at:
http://localhost:3000/dalle-playground?backendUrl=http://<IP_ADDRESS>:8080
The CPU will not reach 200% unless you issue multiple tasks concurrently.

Setting task_topology_group_count disables all automatic detection if non-zero, so if that didn't change anything then I'd suspect that the flag is not being passed down correctly.


feat: Chroma export usability

Made chroma export command more usable by splitting the CLI parts into a separate function than the core export function that only interacts with Chroma. Writing to stdin or file is now done within the CLI part
Refs: #129

#140  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @tazarov and the rest of your teammates on  Graphite


Deployment information of the Smart Contracts

Description
During initialization of the Light Client raiden.create the deployment information of the smart contracts, which the receiving Raiden Full Client is using needs to be provided.
Searching the deployment information manually is a pain in the ass atm :D
CC @eorituz @pirapira
Related to #226 and https://github.com/raiden-network/raiden-contracts/issues/1000
Acceptance criteria

Tasks

[ ]

A first solution might be to tie an SDK release to specific smart contracts as it is done with raiden-libs and the smart contracts (as far as I have understood it)

I have created a draft

The draft has been added to the wiki https://github.com/raiden-network/light-client/wiki/Using-the-SDK-in-a-private-chain-or-a-development-environment


Update biba to 3.14.3975

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

Thank you for the contribution. It was merged directly as 0c72f3833ba0560917dcfd782b185cc8ee9bc17d. Your contribution is still credited to you.


dag_drawer not working

Information

Qiskit Terra version: 0.8.2
Python version: 3.7.5
Operating system: MacOS 10.15.2

What is the current behavior?
dag_drawer raises:
AttributeError: 'MultiDiGraph' object has no attribute 'number_of_selfloops'

Steps to reproduce the problem
from qiskit import QuantumRegister, ClassicalRegister, QuantumCircuit
from qiskit.dagcircuit import DAGCircuit
from qiskit.converters import circuit_to_dag
from qiskit.visualization import dag_drawer

q = QuantumRegister(3, 'q')
c = ClassicalRegister(3, 'c')
circ = QuantumCircuit(q, c)
circ.h(q[0])
circ.cx(q[0], q[1])
circ.measure(q[0], c[0])
circ.rz(0.5, q[1]).c_if(c, 2)

dag = circuit_to_dag(circ)
dag_drawer(dag)

What is the expected behavior?
dag_drawer should draw the dag for the above circuit.
Suggested solutions

Hi @weiT1993 , thanks for reporting. I'm not currently able to replicate this. Can you post the full traceback along with your
networkx.__version__
qiskit.__version__
qiskit.__qiskit_version__

qiskit 0.8.2 is a bit old, i'd try updating. though i don't recall this failing at that version either.

This is an issue caused by networkx 2.4 on qiskit-terra <0.11.0 (caused by https://github.com/chebee7i/nxpd/issues/14). There are 2 ways to fix it either downgrade networkx to 2.3 or upgrade qiskit-terra to 0.11.0. I'm going to close this since it's already been fixed on the latest version of terra (by #3280), but please feel free to reopen if I'm missing something


Group impl blocks together with their respective functions

part of #252, does not solve the deduplication though

a nice diff: 56fec23 (#252)

It seems like deduplication is easily solved with public_api.items.dedup_by_key(|i| {i.to_string()}); when sorted.

It seems like deduplication is easily solved with public_api.items.dedup_by_key(|i| i.to_string()); when sorted.

There is a relatively high risk with doing that: in case items due to bugs or limitations and up being rendered the same, using that method will cause items to disappear from listing/diffing. I quite strongly prefer items to occasionally be duplicated, over not being able to fully trust the output.
So I think we should do the deduplication  specifically for impls somehow.
But maybe I am just overly pessimistic. Early on in the project I used the equivalent of what you propose, but that ended up having the problems I described. But nowadays the project is much more mature, so the risk of inappropriately identically rendered items is much lower.
Maybe we should be brave and go with your proposal. Because it is very simple. And I do like simple :)
(But let's iterate on this in a separate PR after this one is merged, as I think you also have planned already.)

The PR title sounds like a bugfix, so I'm changing label to category-bugfix. I hope that's OK.
I will make a release shortly.


Update meta-edison-bsp with reboot fix patch

Signed-off-by: Sebastian Panceac<EMAIL_ADDRESS>

I think you should redo this patch to target the HEAD of the pyro branch in that bsp layer


Ope njpegtracked

What is this pull request about? I think it's done in error ?
If you need help or support,please open an issue.

I don't see any changes here @bondareko, plus this branch you're merging to is no longer in use. Like, @tst-awankhede said, please start by opening an issue if you want something to be changed. I'm going to go ahead and close this pull request for now.


Is it possible to change the detection target of a-scene dynamically

I am trying to change the mind file for detecting different targets using JS, is there a way to change the value of " mindar-image="imageTargetSrc" for the element"a-scene"?

Nope, you cannot change the targetSrc dynamically. mindar engine was initialized when a-scene started, and it will not re-initialize again when you change the src property


Lemma 7.5.6

[blueprint]
This is ready with a single sorry, so leaving as a draft for now.
TODOs

[ ] prove have notMax : ¬IsMax J
[x] add a typo correction (64⁻¹ version of the hypothesis) to blueprint
[ ] add \leanok
[ ] refactor

I'm looking at the proof in the blueprint, and I think there is indeed a gap where you have your sorry. I'll ask the harmonic analysis group here what the right way to fix this is.

Nevermind, the proof is fine.
In your goal, assume by contradiction that s J + 3 < 𝔰 p. Then you can prove ¬IsMax J using Grid.isMax_iff and Grid.le_topCube

Great, that worked, thanks!

Is it okay if I leave refactoring to a later PR (either by myself or someone else, maybe add #TODO refactor to keep track of what theorems need to be refactored) and merge it, or is it better to refactor right away?
Assume I'll do minimal preparation before the merge, e.g. prepend every auxiliary lemma with limited_scale_impact (so lemma first_estimate would be renamed to lemma limited_scale_impact__first_estimate).

I think it's nice if you clean it up yourself first, there's no hurry to get this merged right away.