[jvm-packages] Add option to treat 0's as 0's in sparse vectors instead of missing values?

Many tree packages don't have a missing value branch on their tree nodes (e.g. Spark).  It is often useful to translate from XGBoost results to a different tree package.
Currently, XGBoost treats zeros in sparse vectors as missing values.  Is it possible to add an option to treat 0's as 0's instead of as missing values for sparse vectors?  That way data specified as sparse vectors will have no missing values... just 0's?  This will allow for translating the resulting trees into different packages.  It would be a very useful option, I think.

I think you are talking about XGBoost-Spark
are you saying that you do not like Spark's VectorAssembler's behavior that treat 0 as the only accepted missing value option, and you want XGBoost to revert that behavior by translating SparseVector to DenseVector?

You are right that I'm mostly referring to XGBoost-Spark, but I don't think I'm saying that... let me be more specific with an example.
We create an input vector that has a lot of dummy-coded features, so just 0/1 values but many coordinates of them.  Spark stores this as a Sparse Vector, to save memory, which is good because there are many coordinates of 0/1 values.  When putting this vector into XGBoost, it treats the 0's as missing values and uses the missing value branch.  This makes it hard to translate the results of XGBoost back to Spark or a different tree software, because they don't have missing value branches.
I'd still like to store things as a Sparse Vector because it saves large amounts of memory.  But, it would be nice to have XGBoost treat the 0's in the sparse vector (which are actually missing coordinates, because Sparse Vectors don't store 0's) as 0's instead of missing values.  That would allow translating the trees back to Spark.
So, I'm not saying translating Sparse Vector to Dense Vector -- that consumes a lot of memory.  I'm saying treat the missing coordinates in Sparse Vector as 0's, not missing values, if that makes sense?

that's the same thing, you don't want 0 to be taken as missing values, then you need to save it in somewhere to feed to XGBoost <= translating sparse to dense

As I said , spark's vector assembler only takes 0 as missing value, when it comes to XGBoost training phase, all those 0s have been removed from input

XGBoost by default treats NaN as the missing value, but because of 1, it will treat 0s as missing value as well

since 2 is very confusing, we require users to set missing value option in xgboost as 0 if they are using sparse vector since 0.9 otherwise the training / prediction will fail

I understand a bit more... let me say it another way then...
The XGBoost option I'd be asking for is to treat the missing values as 0's in XGBoost without actually increasing the memory storage.

I understand a bit more... let me say it another way then...
The XGBoost option I'd be asking for is to treat the missing values as 0's in XGBoost without actually increasing the memory storage.

I don't think we can do that, essentially it comes XGBoost treats a missing value as non-missing , then it confuses the all logic in handling missing values in XGBoost, more specifically
when we create a DMatrix, we treat 0 as missing value, later after we feed DMatrix to XGBoost, we have to leak this concrete value taken as missing to core training algorithm, which is expected to take those values back by manipulating all data files....
we can ask @hcho3 @RAMitchell @trivialfis for more ideas

I don't understand the XGBoost code base very well, but is it possible to do something like change the dereference operator of BatchIterator to insert the 0's for missing values right before use by the algorithm?
Something perhaps here in data.h line 316?
const SparsePage& operator*() const {
CHECK(impl_ != nullptr);
return *(*impl_);
}
Just a thought...

how you insert the 0's for missing values without actually increasing the memory storage.?

It would increase it just for the one row that is currently getting processed, as opposed to for the whole dataset, if that makes sense?
So, as the algorithm is iterating over the rows gathering statistics, only the current row getting processed would have the increased memory usage?

BatchIterator to insert the 0's for missing values right before use by the algorithm?

Let's not do that.

one row that is currently getting processed,

Tree algorithm doesn't necessarily process data row by row, actually I believe the whole dataset is loaded into updaters unless external memory is used.
So if I understand correctly, the 0s are removed in spark, but you want them somehow restored in XGBoost?

So if I understand correctly, the 0s are removed in spark, but you want them somehow restored in XGBoost?

Yeah, so they don't get treated as missing values but as 0's.  But to do that without blowing up memory usage...

Tree algorithm doesn't necessarily process data row by row, actually I believe the whole dataset is loaded into updaters unless external memory is used.

Ok, I need to read more to understand how the tree updaters work.

I don't think we can implement that.  There's no way to distinguish actual missing values from zeros.

@nedStackAdapt The first-class support for sparse data with missing values is one of strengths of XGBoost. Other tree packages (e.g. scikit-learn) will force you to impute missing values first before training, whereas XGBoost can handle missing values natively.
So there will always be some difference between XGBoost models and models created by other packages.

@hcho3 I understand and value the missing value handling in XGBoost, its top notch.  The only issue in this case is that the values are not actually missing -- they are just 0's.  The 0's don't get stored in the sparse vector only to save memory, not because they are missing values.
But I understand if it is too hard to disambiguate these things in the XGBoost code.  Thanks for thinking about it.

@nedStackAdapt I don't know how spark works, but I think implementing a sparse vector is quite simple.
To me your issue is in sparse vector doesn't support compressing values other than zeros, not in machine learning libraries...  So maybe a better solution would be improving the vector class.

@trivialfis Ok, then I wasn't communicating clearly.  There are no missing values in my dataset.  It is just big with lots of 0 values, so it is stored as a sparse vector to save memory.
But the 0 values get treated as missing values in XGBoost... That's the issue.

I will try to see if it's possible for XGBoost to eliminate copying data under some conditions.

@nedStackAdapt Got it.  Thanks for the clarification.  Let us think about solutions other than restoring 0 for categorical data.  It's quite difficult for XGBoost to do that right now.

My understanding is that when training with sparse input matrix and a special "missing" value is specified (say Float.NaN), XGBoost will first go through the non-zero entries of the sparse matrix and remove any entries that takes that value from the sparse data structure; afterwards, any "empty" entries in the training matrix will be consider as missing values.
I'd image if there is only a couple of non-missing zero values, one would be able to circumvent this behaviour by explicitly setting their values to 0.0 in the sparse matrix.
As @nedStackAdapt mentioned, for sparse matrices, the semantics is typically that the "empty" entries represents 0s, not missing values. To solve this issue, I wonder if we can ask the user to explicitly declare a value that would like XGBoost to consider as missing without removing them from the sparse matrix's internal structure?


User Config Settings

Few questions regarding default-config.json

weather mode: when setting default to "lonlat", I'd imagine when a new user first joins the page Hubley's would request permission to their location, however this does not seem to be the case? Rather, user's still have to manually select zipcode or geolocation within settings.

While on the topic of weather... any chance can their be an added environment variable/config that would allow us to set default to imperial vs metric system for temp/windspeed/visability?

show_settings_text:  when set to false does not appear to change anything. My understanding is this would disable the settings button so that users cannot modify backgrounds/weather/etc.

weather mode: when setting default to "lonlat", I'd imagine when a new user first joins the page Hubley's would request permission to their location, however this does not seem to be the case? Rather, user's still have to manually select zipcode or geolocation within settings.

that kind of function is not implemented, will look into it

While on the topic of weather... any chance can their be an added environment variable/config that would allow us to set default to imperial vs metric system for temp/windspeed/visability?

I assume you americans will never be able to get along with the metric system ;)
Maybe this should be implemented as a per user setting and not an instance-wide config. I will look into it

show_settings_text: when set to false does not appear to change anything. My understanding is this would disable the settings button so that users cannot modify backgrounds/weather/etc.

per default the settings link on the dashboard is icon+text. with the toggle set it will show just the icon

imperial system can now be selected in settings, please test the edge tag

imperial system can now be selected in settings, please test the edge tag

Imperial toggle working great! Just like how George Washington wanted


Preview Block. New overall view

Hi @nkarmazina @Alice-cla
the second Part of the Ticket (Bold Heading, etc) looks good.
The first part:
As proposed in the original requirement: Is it possible to widen the entire modal?
In a 1920 to 1080 resolution it is still quite small:

If not:
Yes please widen the "Preview" part. Maybe 2/3 or 65%?
We might need to rework the Block-Modals into a separate View later. Especially when concidering some of the upcoming requirements.


Location Activity menu

the share icon in locationActivity is not working ,though it doesn't seem usefull to me here. Do you want me to make it working or simply remove it, and further share button in bookmark fragment doesn't seem usefull

@utkarshnath Please go ahead and make it work.
Thanks

@mananwason but what should I share, In the locationActivity no specific track is selected.
Isn't the case here is similar to tracksActivity and there is no share button.


Created Tower of Hanoi

Tower of Hanoi is a mathematical puzzle where we have three rods (A, B, and C) and N disks. Initially, all the disks are stacked in decreasing value of diameter i.e., the smallest disk is placed on the top and they are on rod A. The objective of the puzzle is to move the entire stack to another rod (here considered C), obeying the following simple rules:
Only one disk can be moved at a time.
Each move consists of taking the upper disk from one of the stacks and placing it on top of another stack i.e. a disk can only be moved if it is the uppermost disk on a stack. No disk may be placed on top of a smaller disk.

this algo is already pushed


FOUNDATIONS: Interns LockedInternException Add

Closes: #94

Created PR to reduce merge conflict between issues #50 and #49


Error when using validator in nested model with case_sensitive=True and using upper case

Example:
class DjangoSettings(BaseModel):
    ALLOWED_HOSTS: list[str] = ["<IP_ADDRESS>", "localhost"]

    @field_validator("ALLOWED_HOSTS", mode="before")
    def parse_comma_separated(cls, value):
        if not isinstance(value, str):
            return value
        return [item.strip() for item in value.split(",") if item]

class EnvSettings(BaseSettings):
    DJ: DjangoSettings

    model_config = SettingsConfigDict(
        env_nested_delimiter="__",
        case_sensitive=True,
        extra="forbid",
    )

In envs: DJ__ALLOWED_HOSTS: '<IP_ADDRESS>, localhost'
Crashes with an error:
Traceback (most recent call last):
  File "/opt/app/src/manage.py", line 22, in <module>
    main()
  File "/opt/app/src/manage.py", line 18, in main
    execute_from_command_line(sys.argv)
  File "/usr/local/lib/python3.11/site-packages/django/core/management/__init__.py", line 442, in execute_from_command_line
    utility.execute()
  File "/usr/local/lib/python3.11/site-packages/django/core/management/__init__.py", line 382, in execute
    settings.INSTALLED_APPS
  File "/usr/local/lib/python3.11/site-packages/django/conf/__init__.py", line 102, in __getattr__
    self._setup(name)
  File "/usr/local/lib/python3.11/site-packages/django/conf/__init__.py", line 89, in _setup
    self._wrapped = Settings(settings_module)
                    ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/django/conf/__init__.py", line 217, in __init__
    mod = importlib.import_module(self.SETTINGS_MODULE)
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "<frozen importlib._bootstrap>", line 1206, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1178, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1149, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 690, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 940, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/opt/app/src/webapp/settings.py", line 20, in <module>
    ENV = EnvSettings()
          ^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/pydantic_settings/main.py", line 141, in __init__
    **__pydantic_self__._settings_build_values(
      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/pydantic_settings/main.py", line 311, in _settings_build_values
    return deep_update(*reversed([source() for source in sources]))
                                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/pydantic_settings/main.py", line 311, in <listcomp>
    return deep_update(*reversed([source() for source in sources]))
                                  ^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/pydantic_settings/sources.py", line 377, in __call__
    raise SettingsError(
pydantic_settings.sources.SettingsError: error parsing value for field "DJ" from source "EnvSettingsSource"

Thanks @nstonic for reporting this.
As the ALLOWED_HOSTS field type is list[str], pydantic-settings considers this field as a complex field and parses the value as JSON. So, you can have your desired behavior by removing the field validator and passing the env value as a valid JSON.
class DjangoSettings(BaseModel):
    ALLOWED_HOSTS: list[str] = ["<IP_ADDRESS>", "localhost"]

class EnvSettings(BaseSettings):
    DJ: DjangoSettings

    model_config = SettingsConfigDict(
        env_nested_delimiter="__",
        case_sensitive=True,
        extra="forbid",
    )

Env: DJ__ALLOWED_HOSTS:'["<IP_ADDRESS>","localhost"]'

@hramezani
Thank you. I'll keep that in mind. But that's not the problem. This code crashes at any attempt to use field_validator

Which code? The one that I provided?
What error do you get?

Your code works if case_sensitive=False, but it crashes with ValidationError if case_sensitive=``True`
  Input should be a valid list [type=list_type, input_value='["<IP_ADDRESS>","localhost"]', input_type=str]```

@nstonic I created https://github.com/pydantic/pydantic-settings/pull/309 to fix the problem.
Could you please confirm that the fix works?

Confirmed! Thank you!


Annotations

With this PR we add annotations.
Annotations are useful to attach data to Nodes successively to their creation, without having to change the AST class and "polluting it" with data useful only for certain elaborations.
Given this is a new significant feature, an opinion from @alessiostalla would be useful
Related to #203

Looks good to me. As always, I'm raising a flag regarding exports – do we want to include annotations in EMF models, JSON exports, etc.? If we want, but it's out of scope, we should keep track with an issue.

You are right. I will add issues for this, and we can proceed with the implementation once we have used annotations and we are sure they are working well. Otherwise the risk is modifying them and then havin


unity corruption

I encountered a problem: when running the Lightgun01 scene, clicking the mouse will not respond, and then Unity will crash.
The computer only has one mouse
Unity uses versions 2021.3 and 2022.3

Hello, I have the same problem. I fixed it by simply using a different mouse. The Amazon Basics mouse did not work for me. But my Roccat Kone Pure and my Logitech Pro x Superlight worked completly fine. At the moment I cannot look into the code and debug it, because I am buissie with study, but I will try to keep you up to date.


The hover effect animation is not functioning properly on the Bank Account in workspace settings

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.36-0
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Navigate to Settings > Workspace > Bank Account
Click on it; the modal opens on the right side
The hover effect animation should be centered on the Bank Account section

Expected Result:
The hover effect animation should be centered on the Bank Account section
Actual Result:
The hover effect animation is not centered on the Bank Account section
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/e701d97a-8149-4880-a00d-61fd0add3df5
View all open jobs on GitHub

Taking over, related to ideal nav.

Proposal
Root cause
We are checking if the Menuitem is pressed based on whether the item route name is present in the route, but for bankAccount we are not passing any routeName
Solution
We should pass routeName property for bankAccount item in WorkspaceInitialPage

@trjExpensify Curious for your opinion on this one, I feel like we should eventually update the bank account to have its own page in the main pane same as all the other settings pages.
If we would do that, this change would not be necessary

Yeah, interesting one. We could move the Bank account page to the main pane like we have the others. Clicking the rows in the card open the modal in the RHP.
That said, with Simplified Collect: Workflows when we implement the new re-designed workspace editor, it'll actually be RHP again:

So perhaps the decision to do

We also have the case where we have a flow from central navigator to the bank account page in RHP, so this can't be moved to central pane at least

Nah, all of those weird redirects will go away with the redesigned workspace editor as those pages get replaced.

making this monthly and will confirm once we update this flow

This button will be removed from the LHP so I am going to close this issue


Upgraded dependencies and fixed not working dark theme

Hi,
I upgraded some dependencies and changed the Route pathTemplate to match Hangfire's pathTemplate definition. The dark theme was not working for me
I would be happy, if the changes could be reviewed and merged 😊

new release available
https://www.nuget.org/packages/Hangfire.Dashboard.Dark/

Thank you


[RFC] Use sha_1 from uuidm to avoid Nocrypto, which requires C

This is needed to use Irmin with js_of_ocaml.
Not sure if this function should be moved to a separate package, or whether the license should be changed to match Irmin's. If it stays here, we could change it to operate on cstructs directly rather than converting to a strings.
/cc @dbuenzli

Travis is failing because you need to regenerate the _oasis auto-crap. I'll do it after merging.
About that patch: SHA1 is also used in the Git implementation and I wanted to be able to functorised ocaml-git relatively to the SHA1 implementation (see https://github.com/mirage/ocaml-git/issues/69). That's not the same concern, but could be useful to port Irmin's Git backend to js_of_ocaml. Not sure it is worth having a separate package though, as by default people will want to have the fast SHA1 version anyway.
And I also agree with the string/cstruct conversion.

Good point. I've fixed the _oasis file anyway, as I'm using this branch in my Travis build. It now fails to build trying to installed conf-gmp for some reason though.

Seems that the Travis scripts do not install the depext for the test dependencies ...

@talex5 can you add nocrypto in the DEPOPTS in travis.yml? This should work-around the bug in opam-depext....

Doesn't seem to help...

Superseded by #185


socks layer failed with curl client

Traceback (most recent call last):
  File "/Users/han-pc/workspace/python/microProxy/microproxy/proxy.py", line 111, in handle_stream
    yield self.layer_manager.run_layers(stream)
  File "/Users/han-pc/.env/mp/lib/python2.7/site-packages/tornado/gen.py", line 1008, in run
    value = future.result()
  File "/Users/han-pc/.env/mp/lib/python2.7/site-packages/tornado/concurrent.py", line 232, in result
    raise_exc_info(self._exc_info)
  File "/Users/han-pc/.env/mp/lib/python2.7/site-packages/tornado/gen.py", line 1014, in run
    yielded = self.gen.throw(*exc_info)
  File "/Users/han-pc/workspace/python/microProxy/microproxy/proxy.py", line 31, in run_layers
    current_context = yield current_layer.process_and_return_context()
  File "/Users/han-pc/.env/mp/lib/python2.7/site-packages/tornado/gen.py", line 1008, in run
    value = future.result()
  File "/Users/han-pc/.env/mp/lib/python2.7/site-packages/tornado/concurrent.py", line 232, in result
    raise_exc_info(self._exc_info)
  File "/Users/han-pc/.env/mp/lib/python2.7/site-packages/tornado/gen.py", line 1014, in run
    yielded = self.gen.throw(*exc_info)
  File "/Users/han-pc/workspace/python/microProxy/microproxy/layer/proxy/socks.py", line 47, in process_and_return_context
    yield self.socks_greeting()
  File "/Users/han-pc/.env/mp/lib/python2.7/site-packages/tornado/gen.py", line 1008, in run
    value = future.result()
  File "/Users/han-pc/.env/mp/lib/python2.7/site-packages/tornado/concurrent.py", line 232, in result
    raise_exc_info(self._exc_info)
  File "/Users/han-pc/.env/mp/lib/python2.7/site-packages/tornado/gen.py", line 1017, in run
    yielded = self.gen.send(value)
  File "/Users/han-pc/workspace/python/microProxy/microproxy/layer/proxy/socks.py", line 72, in socks_greeting
    raise ProtocolError("socks5 auth not supported")
ProtocolError: socks5 auth not supported

commit 2339b88 fix this issue.


boot.initrd.secrets fixes

Description of changes
A couple of fixes for making the user experience of boot.initrd.secrets less miserable.
This completely fixes https://github.com/NixOS/nixpkgs/issues/73404.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested with nixosTests.installer.fullDiskEncryption
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg test grub installer.fullDiskEncryption

Not super happy about the bind mount thing but I don't see a better way.

Yeah, as I wrote in the commit message the proper solution is to build the system within chroot, but that has several problems too and it's a lot of work. It could be something for the future, but for now it's important to just get rid of this bug.

Could you rebase on master and change

Ok!

There are conflicts now

Fixed the conflicts and re-run the tests.

This (commit 9fc47e6db3f) broke our amazonImage build and thus is blocking both unstable NixOS channels.  https://hydra.nixos.org/build/205943384

https://github.com/NixOS/nixpkgs/pull/211218

Aaand another installer test regression from this PR, also blocking the nixos-unstable channel: https://hydra.nixos.org/build/206057315
$ nix build -f. nixosTests.installer.simpleUefiSystemdBoot
[...]
boot-after-install # cp: cannot stat '/mnt/etc/nixos/secret': No such file or directory
[...]
(finished: cleanup, in 0.00 seconds)
# and now hanging very long; after that occasionally some additional log line

Ah, that's not a regression: it's just that relative path have always been broken and I haven't fixed systemd-boot as well as grub.

That's because we have exposed the fact relative paths have never worked before. I have to make the failure a warning in systemd-boot as well.

It's unclear to me why we need to re-invoke append-initrd-secrets for old generations though.

Fair point, in this way it's neither fully stateless nor stateful.

When constructing the boot menu we have to process the old generation in addition to the new one, and appending its secrets using its append-initrd-secrets fails because it refers to the old path.

This sounds tangentially related to https://github.com/NixOS/nixpkgs/pull/209156

It's unclear to me why we need to re-invoke append-initrd-secrets for old generations though.

Theorizing: as with the kernel and initrd: it has the effect of reinstating the available kernels under /boot if they have been removed or modified. It does seem unfortunate though that the secrets were not by-system but instead by-initrd which doesn't make sense when you consider that you might have two system versions with incompatible secrets, and possibly want to boot into either. I tried to fix this in #209156 by naming the append-initrd-secrets after the system instead.


Ideas for designers

https://www.figma.com/file/KRjaoe8uyNagg2xdCPymaC/Hotel-Website?node-id=0%3A1

@MoronkeO thank you for these designs. Are you currently a member of #team-fury?

@MoronkeO thank you for these designs. Are you currently a member of #team-fury?

https://www.figma.com/file/ISaiBZVasyTRTVjsbL6MP9/HNG-INTERNSHIP-HOTEL-WEBSITE?node-id=0%3A1
I'm still  making my UX corrections though, happy to join team fury!😊

Awesome designs @Afeski . Keep working on it. We need to get a good headstart

My hng internship designs so far. I'm really interested in working with you guys. https://www.figma.com/file/9UH7K3mTwKJUKJ0uMODIF8/HNG-Internship-Projects?node-id=49%3A0

Please find attached all my hng designs from day one. I look forward to working with you guys.
https://www.figma.com/file/NQ3F05yM0mwGR8Se5T90yD/Untitled?node-id=0%3A1

I am a ui/ux designer this is my hotel website task https://www.figma.com/file/oLFGHq2g5CBWGyZLKzILK3/HOTEL-WEBSITE

Please as you come in with your designs, state what you scored on the board. This will be confirmed or not, you can never know.

I am a ui/ux designer this is my hotel website task https://www.figma.com/file/oLFGHq2g5CBWGyZLKzILK3/HOTEL-WEBSITE

I scored 6

Despite your score, you are all welcome. The score is not a form of judgement its just for feeling the waters and knowing what we are working with. Thank you for understanding 🙂

https://www.figma.com/file/qtg1GCRf1ha4QsmGGiFWkd/Hotel-Website-Task?node-id=0%3A1
Scored 5

Chidera had 8/10

This is the link to my figma file
https://www.figma.com/file/pZL0dFzFI36FkldXmKzm2U/HNG-Stage-2?node-id=0%3A1

https://www.figma.com/proto/y3gNv22AsFPeRZgJx2fji3/Untitled?node-id=25%3A19&viewport=44%2C895%2C0.699999988079071&scaling=min-zoom

This is the link to my figma file I scored 8
https://www.figma.com/file/pZL0dFzFI36FkldXmKzm2U/HNG-Stage-2?node-id=0%3A1

Was scored 3 but after Mark's re-evaluation, he passed me that nothing is wrong with my designs.

here is my hotel design link:
https://www.figma.com/file/mme9Hl65fHwFX83BKH62Lb/Hotel?node-id=6%3A56
i Scored 6

@holawalepelumi nice

@debsonalo @Chidera-Okemeziem @Bosslady21 @oladimeji16 please share your slack usernames

Here is the link to my hotel website design
I hope to work with your team
https://www.figma.com/file/nSVNiZzIrspCEhHewwcaRW?node-id=0%3A1&viewport=-200%2C73%2C0.09433379769325256

I got 5

I will like to suggest @paulthaprodigy. I stumbled across his work on #random channel. You can check his works here
https://xd.adobe.com/view/e2d9ab4d-cf5d-45d6-6608-97d5b1d9c7ab-94e8/
https://www.figma.com/file/gLlEL3eywZOnCrnyudLa7p/Express-Rest-Hotel-Website?node-id=0%3A1
https://xd.adobe.com/view/e2d9ab4d-cf5d-45d6-6608-97d5b1d9c7ab-94e8/
https://xd.adobe.com/view/74f61194-eccb-4b91-5ed5-6eb0b7eca461-02c7/

@debsonalo @Chidera-Okemeziem @Bosslady21 @oladimeji16 please share your slack usernames

my username is @Olatohdimeji

Slack username: @anibe

Slack username: Dera-O

slack username: techqueen


provide option to log script output to stdout

Having all the output (including the webhook scripts) in the console/log would make debugging easier.

It seems to be a good idea but only for debugging purpose. The script output can be huge depending what is done. Concurrent script executions can also make the log output  tricky to read. I will work on that. Thx

Done. Set APP_SCRIPTS_DEBUG=true to redirect the script output in the console.


HAWQ-547. When HAWQ RM does not have valid YARN resource queue status received the cluster report is not accepted

WARNING log will be generated to log file.
2016-03-16 12:27:25.005161 GMT-10,,,p52480,th-1590025816,,,,0,con4,,seg-10000,,,,,"WARNING","01000","YARN mode resource broker got invalid cluster report",,,,,,,0,,"resourcebroker_LIBYARN.c",639,
Externally, user will have no available cluster report.
postgres=# insert into t values(1);
ERROR:  failed to acquire resource from resource manager, resource request is timed out due to no available cluster (pquery.c:806)

+1

+1


PR: Fix switching to an empty project and add CI tests for the WeatherDataGapfiller

This PR fix a bug that occurs when setting the working directory to a location that does no contain any valid weather data file.

Hello @jnsebgosselin! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file cdprep/gapfill_data/tests/test_gapfill_weather_gui.py:

Line 21:1: E402 module level import not at top of file
Line 22:1: E402 module level import not at top of file
Line 23:1: E402 module level import not at top of file
Line 26:1: E402 module level import not at top of file
Line 27:1: E402 module level import not at top of file


layout design questions

I would like to ask why fixed size blocks are used. Considering that this WAL implementation is used as bitcask storage log, the waste of storage space due to padding + record type is significant. I noticed that RocksDB wal also has a similar structure. Compared to directly appending records, what are the advantages of doing it this way?

When you read a large wal file from the begining to end, read a fix-sized block is more efficient. Normally the size is equals to the page cache size.
if one record is small, like 1KB, you will read 32 tims to load 32 records.
but if the block size is 32k, we can read them all once, which will reduce the syscall(fread).

there's no index, fix-sized block is easier to extend a binary search for taget data.


Possible wrong image in raspberry pi tutorial

On the raspberry pi page ( https://kairos.io/docs/installation/raspberry/ ), under Flash the image, there is amd64 image variant, which I believe it's wrong, for rpi. I wasn't sure how to propose change docs, as there is some kind of template system, and couldn't find how to specify arch.

@zewelor thanks for opening the ticket. Indeed we use a templating system where you can pass the arch and model but it's not clear to find out. I'll merge a fix

ah shoot, the template is fixed for the image ones LOL. ok i'll fix

Great thanks, and thanks for working on such a great project !


[SPARK-42639][CONNECT] Add createDataFrame/createDataset methods

What changes were proposed in this pull request?
This PR adds all the SparkSession.createDataFrame(..) and SparkSession.createDataset(..) methods we can support in connect. The implicit conversion that uses this is also added.
I moved the ArrowWriter class from sql/core to sql/catalyst for the arrow writing.
Why are the changes needed?
API partity with the existing SQL APIs
Does this PR introduce any user-facing change?
Yes.
How was this patch tested?
I have added a number of tests to ClientE2ETestSuite.

Thank you, @hvanhovell and @HyukjinKwon . Merged to master/3.4.


fix: use base.xScale() to invoke current xScale function

Closes #60

First commit to fix #60
Tests are on their way, as well as a small refactoring to cache xScale and yScale at function caller scopes when used multiple times within the same function.

is the same fix needed for heatmapBase? https://github.com/d3fc/d3fc-series/commit/c677ea0f79a02d7a22d2fbd8780ecf139b803838#diff-bfecc83e07366cc89af51f7b84970584R37

It's needed for heatmapBase indeed, thanks. Should be fixed now.


safe development db path

[x] I have added the tests to cover my changes.
[x] I have updated the documentation and CHANGELOG accordingly.
[x] I have read the CONTRIBUTING document.

@FyzHsn this creates the DB on the _dev path, but no records are added to the DB. Should I expect the records to get added by persist() at this point?

I did put those in stdout on purpose so it tells you where it's putting the
DB in develop mode but I'm fine with putting them in logs if you think it's
better
On Mon, Jul 11, 2022, 8:22 AM Venkat Bala @.***> wrote:

@.**** commented on this pull request.
In covalent_dispatcher/_db/dispatchdb.py
https://github.com/AgnostiqHQ/covalent/pull/798#discussion_r917866752:

@@ -167,6 +171,23 @@ def exit(self, exc_type, exc_value, traceback):

     return False

def _db_dev_path(self):

   if get_config("develop"):

       sqlite = ".sqlite"

       path = "sqlite+pysqlite:///" + self._dbpath.split(sqlite)[0] + "_dev" + sqlite

       print("***SET DB PATH FOR DEVELOPMENT***")

are these print statements included for debugging? could we use logger?
—
Reply to this email directly, view it on GitHub
https://github.com/AgnostiqHQ/covalent/pull/798#pullrequestreview-1034247215,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGC7HLHTWJMOITR3KPXTWV3VTQGYPANCNFSM527AGNSA
.
You are receiving this because you authored the thread.Message ID:
@.***>


fix homepage preview code

What: Fixes linting and other issues on the homepage example

Why: Builds were breaking and content was not translated

How: Made home-page-example not a function because it doesn't have content anymore. Passed content from the index page.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@89a8336). Click here to learn what that means.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master     #120   +/-   ##
=========================================
  Coverage          ?   96.19%
=========================================
  Files             ?       57
  Lines             ?      210
  Branches          ?       20
=========================================
  Hits              ?      202
  Misses            ?        7
  Partials          ?        1

Impacted Files
Coverage Δ

pages/index/content/index.js
100% <ø> (ø)

pages/index/index.js
100% <100%> (ø)

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 89a8336...cfa1050. Read the comment docs.


Allow FancyZone keyshortcut to be configurable for editor

We use old-school IBM clicky keyboards. They have no Windows key. Please add a setting to trap an alternate key stroke/combo.

It could also be useful for people whose keyboard don't have a tilde key.
The shortcut seems impossible to use on my French Canadian keyboard.

This was fixed a few version ago.


Queue fetcher updates

Take another page from scrapy scheduling internals

Override _on_input_message, which runs in the Pika thread when a new
message is received from RabbitMQ, and rather than just queuing the
message to the internal work queue (for consumption by worker
threads), decode it, and see when it could next be started (using the
Slot.issue_interval calculated from avg_seconds for request completion
to keep "next_issue" time).  If the delay is less than the fast delay
queue time (set with --busy-delay-minutes), use the Pika connection
"call_later" method to delay putting the message on the work queue
until it's ripe to be issued.  If the delay is longer than
busy-delay-minutes, requeue the message to the -fast queue.

This GREATLY reduces use of the -fast queue (lower CPU load) AND means
that requests can be started as soon as possible, without waiting for
the message to come around through RabbitMQ (better thruput).

Also: default worker count to the number of available CPU cores.

Ah, I understand now....  Good catch!! I noted the "Any" issue in
February as https://github.com/mediacloud/story-indexer/issues/233
which we put off as "long-term"
It looks like there are 29 uses of Story sub-object "getter" methods
in "with" statments (plus a comment in indexer/workers/fetcher/rss-queuer.py
that notes the problem!):
# mypy reval_type(rss) in "with s.rss_entry() as rss" gives Any!!

And the explicit hint, does seem to solve the problem:
(venv) ***@***.***:~/story-indexer$ cat a.py
from indexer.story import BaseStory, RSSEntry

def foo(s: BaseStory) -> None:
reveal_type(s.rss_entry())

with s.rss_entry() as r:
    reveal_type(r)

r2: RSSEntry
with s.rss_entry() as r2:
    reveal_type(r2)

r3 = s.rss_entry()
reveal_type(r3)
with r3:
    reveal_type(r3)

(venv) ***@***.***:~/story-indexer$ mypy a.py
...
a.py:4: note: Revealed type is "indexer.story.RSSEntry"
a.py:6: note: Revealed type is "Any"
a.py:10: note: Revealed type is "indexer.story.RSSEntry"
a.py:13: note: Revealed type is "indexer.story.RSSEntry"
a.py:15: note: Revealed type is "indexer.story.RSSEntry"

I'd like to understand the failure before applying a work-around, and
I'd prefer the assignment (since it avoids needing to explicitly type
the variable), plus a comment like "mypy gets with ..... wrong" to a
variable declaration that looks extraneous.
The problem is easy to reproduce, with a simple class with mypy (but
pytype gets it right) so I've asked about it on a python/typing chat.


Bump version to 0.3.0-SNAPSHOT

LGTM

Codecov Report

Merging #230 into bundling-flowcontrol-branch will not change coverage.
The diff coverage is n/a.

@@                      Coverage Diff                       @@
##             bundling-flowcontrol-branch     #230   +/-   ##
==============================================================
  Coverage                          69.98%   69.98%
  Complexity                           486      486
==============================================================
  Files                                 68       68
  Lines                               2559     2559
  Branches                             269      269
==============================================================
  Hits                                1791     1791
  Misses                               669      669
  Partials                              99       99

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 2b72a80...9e6e910. Read the comment docs.


EC2 AllocationId missing from describe_addresses

Missing AllocationId field from EC2 describe_addresses function.

Thanks; setting as a feature request.

Ran into the same issue.. Looking for a workaround because the ec2_client.create_nat_gateway() requires an AllocationId https://boto3.amazonaws.com/v1/documentation/api/latest/reference/services/ec2.html#EC2.Client.create_nat_gateway
The moto_ec2 describe_addresses() is also missing the AllocationId
Here is a demo of the missing AllocationId
test.py
from moto import mock_ec2
import boto3
import json

@mock_ec2
def example():
    ec2_client = boto3.client("ec2")
    print(json.dumps(ec2_client.allocate_address(), indent=2))
    print()
    print(json.dumps(ec2_client.describe_addresses(), indent=2))

if __name__ == "__main__":
    example()

And the reponse
$ python test.py
{
  "PublicIp": "<IP_ADDRESS>",
  "Domain": "standard",
  "ResponseMetadata": {
    "RequestId": "59dbff89-35bd-4eac-99ed-be587EXAMPLE",
    "HTTPStatusCode": 200,
    "HTTPHeaders": {
      "server": "amazon.com"
    },
    "RetryAttempts": 0
  }
}

{
  "Addresses": [
    {
      "InstanceId": "",
      "PublicIp": "<IP_ADDRESS>",
      "Domain": "standard",
      "NetworkInterfaceId": ""
    }
  ],
  "ResponseMetadata": {
    "RequestId": "59dbff89-35bd-4eac-99ed-be587EXAMPLE",
    "HTTPStatusCode": 200,
    "HTTPHeaders": {
      "server": "amazon.com"
    },
    "RetryAttempts": 0
  }
}

Looked at the AWS documentation a little closer https://boto3.amazonaws.com/v1/documentation/api/latest/reference/services/ec2.html#EC2.Client.allocate_address and noticed that the default "Domain" is to be null, which I think will also not return an allocationId. Pretty sure moto behaves correctly.
AllocationId (string) --
[EC2-VPC] The ID that AWS assigns to represent the allocation of the Elastic IP address for use with instances in a VPC
I think the moto.ec2.models.ElasticAddress.init() acts correctly, where if the domain is not specified, the allocation id is None
moto/ec2/models.py
class ElasticAddress(object):
    def __init__(self, domain, address=None):
        if address:
            self.public_ip = address
        else:
            self.public_ip = random_ip()
        self.allocation_id = random_eip_allocation_id() if domain == "vpc" else None
        self.domain = domain
        self.instance = None
        self.eni = None
        self.association_id = None

I agree - the docs are pretty specific that it refers to addresses within a VPC:
AllocationId (string) --

[EC2-VPC] The ID that Amazon Web Services assigns to represent the allocation
of the Elastic IP address for use with instances in a VPC.

Closing this.


A handful of zip codes in the district addresses dataset are not parsed as strings

All nine-digit zip codes in legislators-district-offices.yaml are parsed as YAML strings (because they have non-numerical characters in them), and most five-digit zip codes are quoted (which makes them parse as strings), but these particular offices' zip codes are not quoted and parse as numbers.

Looks like these all originated from fcabd4c9e152885a1d6f242e0a3c878ce3d4cbab (#794).

Sorry for that, folks.


Version script

Keep all versions up to date when bumping the package version with npm version <part>
Update publish script to also publish the new version under tags/

I also notice the version in package-lock.json  doesn't update.
https://github.com/wikimedia/wikipediapreview-wordpress/blob/0aabc0fecba99d4ee43127e59799e842bcd9e645/package-lock.json#L2-L3

It will stay up to date when we handle version correctly with npm version. I have fixed it on main for now.

npm run version works for me, i didn't test npm run postvresion as it push a new tag.


--dry-run refuses to simulate creation of a directory

RClone v1.33, running:-
rclone --dry-run move provider1:/data provider2:/data
I get:-
2016/08/26 12:08:11 amazon drive root 'data': Not making directory as dry run is set
2016/08/26 12:08:11 Attempt 1/3 failed with 0 errors and: directory not found
2016/08/26 12:08:11 amazon drive root 'data': Not making directory as dry run is set
2016/08/26 12:08:11 Attempt 2/3 failed with 0 errors and: directory not found
2016/08/26 12:08:11 amazon drive root 'data': Not making directory as dry run is set
2016/08/26 12:08:11 Attempt 3/3 failed with 0 errors and: directory not found
2016/08/26 12:08:11 Failed to move: directory not found
Instead of the simulation of the directory being created, which is what I assume --dry-run is suppose to do.

Just do a rclone mkdir provider2:/data first and then --dry-run will have a directory tree to work on and show you something sensible.

Hopefully that answered your question


Support overriding chrome flags and remote debugging port via environ…

…ment variables.
Configuration should be possible via file or environment variables.

I was basically working around issues I encountered when trying to run npm start or node src/main.js on RHEL 7. After installing google chrome with these instructions https://www.tecmint.com/install-google-chrome-on-redhat-centos-fedora-linux/ I had
$ ls -hl /usr/bin/google-chrome*
lrwxrwxrwx 1 root root 31 Sep  7 15:03 /usr/bin/google-chrome -> /etc/alternatives/google-chrome
lrwxrwxrwx 1 root root 50 Sep  7 15:03 /usr/bin/google-chrome-unstable -> /opt/google/chrome-unstable/google-chrome-unstable

and these errors
$ node src/main.js
which: no google-chrome-stable in (/usr/local/bin:/usr/bin:/usr/local/sbin:/usr/sbin)
{ Error: connect ECONNREFUSED <IP_ADDRESS>:34376
    at Object._errnoException (util.js:1041:11)
    at _exceptionWithHostPort (util.js:1064:20)
    at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1153:14)
  code: 'ECONNREFUSED',
  errno: 'ECONNREFUSED',
  syscall: 'connect',
  address: '<IP_ADDRESS>',
  port: 34376 }

After
$ sudo ln -s google-chrome /usr/bin/google-chrome-stable

I got
$ node src/main.js
{ Error: connect ECONNREFUSED <IP_ADDRESS>:33827
    at Object._errnoException (util.js:1041:11)
    at _exceptionWithHostPort (util.js:1064:20)
    at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1153:14)
  code: 'ECONNREFUSED',
  errno: 'ECONNREFUSED',
  syscall: 'connect',
  address: '<IP_ADDRESS>',
  port: 33827 }

Because of https://bugs.chromium.org/p/chromium/issues/detail?id=757181 I currently run chrome without --headless and Xvfb instead.
Since I didn't deploy it as a docker container, but as a systemd service I made some other changes to use systemd's socket activation in https://github.com/konstantinblaesi/rendertron/commit/416d65084fd88957dae3e0fd81ad831c3738da0b . Socket activation allows me to use the privileged ports < 1024 without running the service as root.

This PR has been idling for a while, so please let me know if this is still required.


The command path could not be resolved

Resolved command path for "pod" is invalid.
Expanded GEM_PATH: /usr/bin
Above is error msg while clicked update Pods..
I have install xcode6.4 and xcode7 in ei capitan

Hi @culmek, do you have the cocoapods gem installed? What do you see when you run these commands in a terminal:

which ruby
which pod

By default, the plugin looks for Cocoapods to be installed at /usr/bin/pod, which should happen when you run gem install cocoapods using the default ruby command on OS X (/usr/bin/ruby).
According to some comments in #73, if you are running El Capitan, the default GEM_PATH is now /usr/local/bin. You can also try setting GEM_PATH to that and see if it helps.
If /usr/bin/pod and /usr/local/bin/pod do not exist and you do have cocoapods installed, you can instead replace GEM_PATH in the menu (Product > Cocoapods > GEM_PATH) with the output of:
dirname `which pod`

Let me know if this helps!

dirname which pod, works for me perfectly!

/usr/local/bin works, thanks

up is answer, thanks

/usr/local/bin works, thanks

works for me，thx!


Cannot handle path when working from WSL path in Windows

The insider build for 1903 has a new functionality that allows browsing files on WSL from explorer. The paths look like \\wsl$\Ubuntu\home\alecxvs\Git\myproject.
Running wslgit from this path gives this error:
thread 'main' panicked at 'Cannot handle path "\\\\wsl$\\Ubuntu\\home\\alecxvs\\Git\\myproject"', src\libcore\option.rs:1008:5

Yes, it would be nice to support these new paths. Right now, wslgit assumes that Windows paths start with a drive letter (to translate them into a proper unix path).
Adding support for these paths should be possible, e.g. by detecting and removing the prefix.

+1

Adding support for these paths should be possible, e.g. by detecting and removing the prefix.

@andy-5 why don't you use wslpath for things like that? E.g. wsl.exe wslpath -u "\\\\wsl$\\Ubuntu\\home\\alecxvs\\Git\\myproject" would return /home/alecxvs/Git/myproject. Would it make your life easier? :)

Thanks for the suggestion (and good to know that it supports these paths). There is already some discussion on wslpath in #12. Unfortunately, it does not solve all problems. The main reasons it is not used are:

it does not fully support manually mounted drives
executing a separate process is slow
nobody has done the work to implemented support for it

It should be super simple to convert WSL2 UNC paths: just add/remove the \\wsl$\Distroname and flip the backslashes. No need to resolve mount paths in that case, though you probably need to pass the distro name to the Linux side. However, I'm not sure if there's some special consideration needed for running executables when the current directory is a UNC path. At least .bat scripts have problems with those.
Ideally, this would detect the UNC paths and also work with DrvFs or 9p mounts, so it'd work whether the git repo resides on the Windows side or the Linux side.

I just released v0.9.0, which uses wslpath to translate paths (and should handle UNC paths). Does the new version resolve this issue?

Not quite, I have a repo at
\\wsl$\Ubuntu\home\stan\projects\personal\WeakAuras2
and when I go to it via CMD (which is T:\home\stan\projects\personal\WeakAuras2) and type git status it does not detect it as a git repo.

@carlolars any idea for this maybe?

but T IS the WSL mount, how would I mount it inside of WSL?

but T IS the WSL mount, how would I mount it inside of WSL?

You can't.

Well what do I do then? :D


version 3.3 crashed in environment 'Devito'

updating anaconda to 2.5.x, then on a pre-installed environment 'Devito' open spyder which happen to be an older version 3.3.x, then it crashes and keep opening crash window.

uhh just update...

Hey @ahmed-elbassiony, thanks for reporting. At this point Spyder 3 is not maintained anymore. Please update to our latest version by closing Spyder, opening the Anaconda Prompt and running there the commands mentioned in our documentation.
If that doesn't work, I'm afraid you'll have to remove Anaconda and install a more recent version of it.


Pass show_preds(**draw_sample_kwargs) also to annotations

🚀 Feature
Is your feature request related to a problem? Please describe.
show_preds() draw_sample_kwargs only get passed to preds but not annotations
Describe the solution you'd like
Pass **draw_sample_kwargs also to preds
Context
I cannot change the font_size of annotation bbox labels.

Thanks for opening the request @maivels! Is there a simple solution you have in mind?

This should be fixed.


Both sync and async exceptions for hystrix observable cmd

Adds more thorough tests as specified by #525, and fixes synchronous exception handling that was missing and caught by these tests

Hystrix-pull-requests #260 FAILURE
Looks like there's a problem with this pull request

NetflixOSS » Hystrix » Hystrix-pull-requests #21 SUCCESS
This pull request looks good


Very delayed Router fast path transfers (12-18min)

Another (small) tx stuck: https://connextscan.io/tx/0xa2366577259abb0da2a873870b620a251a6556991a5ae1eceb18f866b1324b09?source=search
Transfer | Connextscan
Transfer | Connextscan
Explorer for the Connext protocol
Image
This should be treated as ultra critical, including for token launch
if our routers aren't bidding properly, then this will mean our crosschain airdrop totally fails
cc/ @alexblanq
By the way, this has happened every single time i have used Connext in the past two weeks. Completely unusable
not sure how this is going to work for partners like fuji btw. The above bridge transaction is to GC to fill my wallet so I could play with the Fujii flow. I specifically picked GC because lots and lots of liquidity. I'm willing to bet rn that we see exact same issue with needing to go through slow path when I finally get liquidity and try to use Fuji's dapp
We likely need to escalate this into a p0 to investigate
arjun — Today at 1:51 PM
Completed. Odd?
eddie | MST — Today at 1:51 PM
can't glean from logs exactly what happened here, looks like routers indeed did not bid on this initially
arjun — Today at 1:51 PM
Why did it take 18 minutes?
eddie | MST — Today at 1:51 PM
but yeah it completed just now
arjun — Today at 1:51 PM
Yea this is very weird and bad
@Preetham | PST cc^
Preetham | PST — Today at 2:09 PM
Looking into this
arjun — Today at 2:09 PM
Another: https://connextscan.io/tx/0x108521bee91ec023fa7fc3a0e42d60d0862a77f8e5d3de576fca4be6ed2a7eb6

Routers that bid:

https://app.datadoghq.com/logs?query=region%3Aus-east-2 %40requestContext.transferId%3A0x108521bee91ec023fa7fc3a0e42d60d0862a77f8e5d3de576fca4be6ed2a7eb6 -service%3A(router-publisher OR router-subscriber)&agg_q=%40bid.router&cols=host%2Cservice%2C%40bid.router&index=&messageDisplay=inline&sort_m=&sort_t=&stream_sort=time%2Cdesc&top_n=10&top_o=top&viz=sunburst&x_missing=true&from_ts=1673810905780&to_ts=1673897305780&live=true

Router that bid for 0x9ee68898913a7c062f893c57969afdc5dfd8cd06b51c3ad4f3229a92b1bfae48:

https://app.datadoghq.com/logs?query=region%3Aus-east-2%20%40requestContext.transferId%3A0x9ee68898913a7c062f893c57969afdc5dfd8cd06b51c3ad4f3229a92b1bfae48%20-service%3A%28router-publisher%20OR%20router-subscriber%29%20&agg_q=%40bid.router&cols=host%2Cservice%2C%40bid.router&index=&messageDisplay=inline&sort_m=&sort_t=&stream_sort=time%2Cdesc⊤_n=10⊤_o=top&viz=sunburst&x_missing=true&from_ts=1673810905780&to_ts=1673897305780&live=true

Likely root cause:
Sequencer does not update auctions data structure, on subsequent bids if this check is false (which is the case when there is a pending execution):
https://github.com/connext/monorepo/blob/7e80436cc42f47a3b60621f349d8b6083e78dc7c/packages/agents/sequencer/src/lib/operations/execute/fastpath.ts#L81
This is where we update auctions:
https://github.com/connext/monorepo/blob/7e80436cc42f47a3b60621f349d8b6083e78dc7c/packages/agents/sequencer/src/lib/operations/execute/fastpath.ts#L97-L108


Can't install plugin ..Version problem

MariaDB [(none)]> show engines;
+--------------------+---------+--------------------------------------------------------------------------------------------------+--------------+------+------------+
| Engine             | Support | Comment                                                                                | Transactions | XA   | Savepoints |
+--------------------+---------+--------------------------------------------------------------------------------------------------+--------------+------+------------+
| MRG_MyISAM         | YES     | Collection of identical MyISAM tables                                                            | NO           | NO   | NO         |
| CSV                | YES     | Stores tables as CSV files                                                                       | NO           | NO   | NO         |
| MEMORY             | YES     | Hash based, stored in memory, useful for temporary tables                                        | NO           | NO   | NO         |
| MyISAM             | YES     | Non-transactional engine with good performance and small data footprint                          | NO           | NO   | NO         |
| SEQUENCE           | YES     | Generated tables filled with sequential values                                                   | YES          | NO   | YES        |
| Aria               | YES     | Crash-safe tables with MyISAM heritage                                                           | NO           | NO   | NO         |
| PERFORMANCE_SCHEMA | YES     | Performance Schema                                                                               | NO           | NO   | NO         |
| InnoDB             | DEFAULT | Percona-XtraDB, Supports transactions, row-level locking, foreign keys and encryption for tables | YES          | YES  | YES        |
+--------------------+---------+--------------------------------------------------------------------------------------------------+--------------+------+------------+
8 rows in set (0.00 sec)
MariaDB [(none)]> SET @inst=IF(EXISTS(SELECT * FROM mysql.plugin WHERE NAME='mroonga'),'DO 1', "INSTALL PLUGIN mroonga SONAME 'ha_mroonga'");
Query OK, 0 rows affected (0.00 sec)
MariaDB [(none)]> PREPARE s FROM @inst;
Query OK, 0 rows affected (0.00 sec)
Statement prepared
MariaDB [(none)]> EXECUTE s;
ERROR 1126 (HY000): Can't open shared library 'ha_mroonga.so' (errno: 8, API version for STORAGE ENGINE plugin Mroonga not supported by this version of the server)
MariaDB [(none)]>

And also can't update mariaDB server in bash...is there any way to add plugin inside the mariadb .
Help me Sir,..

Please read docker documentation on how to write a Dockerfile. You can extend the images provided by this project or fork the project and modify the Dockerfiles but this exercise is something I will not be able to assist you with.


Add option to add date downloaded to lyrics

How you think the new feature/improvement would be helpful, or what problem you think it would solve

After some time lyrics get improved move accurate, better formatting especially for songs release 1 year ago.
This song was released in 2016 was in billlboard hot 100 year end and still gets better formatting just 2 weeks after I download it lyrics.
shocking is now shockin'

Do you mean you'd want to see for each source when those lyrics were last updated? That seems unlikely to be possible because it relies entirely on sources providing this info.

Do you mean you'd want to see for each source when those lyrics were last updated? That seems unlikely to be possible because it relies entirely on sources providing this info.

Date the lyrics were downloaded.

Downloaded date is something we can do, but that wouldn't tell you whether the lyrics you have are behind what the remote source has. Where do you anticipate this would be shown? Can you given an example of what you'd like to see?

that wouldn't tell you whether the lyrics you have are behind what the remote source has.

Can you add option to compare automatically or manually?

Where do you anticipate this would be shown? Can you given an example of what you'd like to see?

At the end or start of lyrics where source is being planned to put. If you put date you need to put source.

Added a mention of showing a timestamp to lyrics info displayed in #52

@jacquesh would you reconsider ?

What do you want me to reconsider?
In general we have no way of knowing when last a particular lyric was updated on a remote source and I've already linked to an issue where we could store the date when local lyrics were retrieved.

@jacquesh also add date. related to https://github.com/jacquesh/foo_openlyrics/commit/b99f3163e5c99be7792fcbbb3438561d3d2fdc52

@jacquesh is it completed by b99f316

That commit adds info about when the lyrics were first/last retrieved and when they were last edited locally, but as I said above, we have no way of knowing in general when the lyrics on a remote source were last modified.

That commit adds info about when the lyrics were first/last retrieved and when they were last edited locally, but as I said above, we have no way of knowing in general when the lyrics on a remote source were last modified.

That metadata is store in foobar2000 not in file ? I would like it better inside lyrics

@jacquesh you may close this issue